Remove the last instances of json pattern

In the past, we've tried to erradicate the use of
nlohmann::json(initiatlizer_list<...>) because it bloats binary sizes,
as every type is given a new nlohmann constructor.

This commit hunts down the last few places where we call this.  There is
still 2 remaining in openbmc_dbus_rest after this, but those are variant
accesses that are difficult to triage, and considering it's a less used
api, they're left as is.

Tested: WIP

Change-Id: Iaac24584bb78bb238da69010b511c1d598bd38bc
Signed-off-by: Ed Tanous <ed@tanous.net>
diff --git a/include/openbmc_dbus_rest.hpp b/include/openbmc_dbus_rest.hpp
index 72e1c31..a2f68b4 100644
--- a/include/openbmc_dbus_rest.hpp
+++ b/include/openbmc_dbus_rest.hpp
@@ -2332,10 +2332,10 @@
                     const char* type = arg->Attribute("type");
                     if (name != nullptr && type != nullptr)
                     {
-                        argsArray.push_back({
-                            {"name", name},
-                            {"type", type},
-                        });
+                        nlohmann::json::object_t params;
+                        params["name"] = name;
+                        params["type"] = type;
+                        argsArray.push_back(std::move(params));
                     }
                     arg = arg->NextSiblingElement("arg");
                 }