commit 9ae226fabb21700e6a73ef1d0a9d1cdead70fea5
author George Liu <liuxiwei@inspur.com> Wed Jun 21 17:56:46 2023 +0800
committer Ed Tanous <edtanous@google.com> Tue Jun 27 16:37:31 2023 -0700
tree 83b58acad99313a1992ba26d3e1c18f0f44ba54c
parent 0f25584d76ae584830f8c933caf1ac1943e9f853

Refactor setProperty method

SetProperty is a method we should use more, and use consistently in the
codebase, this commit makes it consistently used from the utility
namespace.

Tested: Refactor.  Code compiles.

Signed-off-by: George Liu <liuxiwei@inspur.com>
Change-Id: I5939317d23483e16bd98a8298f53e75604ef374d

redfish-core/lib/power.hpp

diff --git a/redfish-core/lib/power.hpp b/redfish-core/lib/power.hpp
index 3072b89..4aaac67 100644
--- a/redfish-core/lib/power.hpp
+++ b/redfish-core/lib/power.hpp
@@ -97,7 +97,10 @@
                 return;
             }
 
-            crow::connections::systemBus->async_method_call(
+            sdbusplus::asio::setProperty(
+                *crow::connections::systemBus, "xyz.openbmc_project.Settings",
+                "/xyz/openbmc_project/control/host0/power_cap",
+                "xyz.openbmc_project.Control.Power.Cap", "PowerCap", *value,
                 [sensorsAsyncResp](const boost::system::error_code& ec2) {
                 if (ec2)
                 {
@@ -107,12 +110,7 @@
                 }
                 sensorsAsyncResp->asyncResp->res.result(
                     boost::beast::http::status::no_content);
-                },
-                "xyz.openbmc_project.Settings",
-                "/xyz/openbmc_project/control/host0/power_cap",
-                "org.freedesktop.DBus.Properties", "Set",
-                "xyz.openbmc_project.Control.Power.Cap", "PowerCap",
-                std::variant<uint32_t>(*value));
+                });
             });
     };
     redfish::chassis_utils::getValidChassisPath(sensorsAsyncResp->asyncResp,