commit 4cd8c250c65ed29705e2b6b3a50d2edb6efccae9
author Alexander Hansen <alexander.hansen@9elements.com> Wed Aug 06 17:29:40 2025 +0200
committer Alexander Hansen <alexander.hansen@9elements.com> Mon Sep 22 14:37:58 2025 +0200
tree 3f704b3471ffb9b5a728676ed8c92bed3ce65ef8
parent bd85baae199ebf0918b56158f01dd6d01d3a59c7

entity-manager: not throw in createAddObjectMethod

We do not throw at the other call site in entity_manager.cpp.

Looked at all 3 places where there's a 'catch'

```
$ rg catch src/entity_manager/
src/entity_manager/dbus_interface.cpp
39:    catch (std::exception& e)

src/entity_manager/configuration.hpp
39:    catch (const std::out_of_range&)

src/entity_manager/expression.cpp
100:            catch (const std::invalid_argument&)
```

but none of these would catch this exception.

Remove the throw here to align with the other call site and prevent
process termination.

Tested: Inspection only.

Change-Id: I90d70f3f59237b32fbf102c970cc1ed7de94e34c
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>

src/entity_manager/dbus_interface.cpp

diff --git a/src/entity_manager/dbus_interface.cpp b/src/entity_manager/dbus_interface.cpp
index cea5d42..3ae698c 100644
--- a/src/entity_manager/dbus_interface.cpp
+++ b/src/entity_manager/dbus_interface.cpp
@@ -344,7 +344,6 @@
             if (!writeJsonFiles(systemConfiguration))
             {
                 std::cerr << "Error writing json files\n";
-                throw DBusInternalError();
             }
             std::string dbusName = *name;