commit e93abac6d60c4b409ad72086dac153f23d793ac1
author Ginu George <ginugeorge@ami.com> Fri Jun 14 17:35:27 2024 +0530
committer Ed Tanous <ed@tanous.net> Thu Jun 27 18:41:38 2024 +0000
tree 907337985b5a9b598353535ae8f99594d944fc01
parent ff0a08815ee289a7a67c31e9bb460cb240daad67

Pass redfishPropertyName earlier argument

It was pointed out that the setDbusProperty method should have an end
that approximately matches dbus-send and busctl set-property in its
arguments, to aid with debug.  This seems reasonable.

Tested: Redfish service validator passes.

Change-Id: Ic20295d93c71c957e3e76704e1eda9da187861b1
Signed-off-by: Ginu George <ginugeorge@ami.com>
Signed-off-by: Ed Tanous <ed@tanous.net>

redfish-core/lib/update_service.hpp

diff --git a/redfish-core/lib/update_service.hpp b/redfish-core/lib/update_service.hpp
index 36ee8a0..2888cbf 100644
--- a/redfish-core/lib/update_service.hpp
+++ b/redfish-core/lib/update_service.hpp
@@ -734,11 +734,11 @@
         return;
     }
 
-    setDbusProperty(asyncResp, "xyz.openbmc_project.Settings",
+    setDbusProperty(asyncResp, "ApplyTime", "xyz.openbmc_project.Settings",
                     sdbusplus::message::object_path(
                         "/xyz/openbmc_project/software/apply_time"),
                     "xyz.openbmc_project.Software.ApplyTime",
-                    "RequestedApplyTime", "ApplyTime", applyTimeNewVal);
+                    "RequestedApplyTime", applyTimeNewVal);
 }
 
 struct MultiPartUpdateParameters