commit 70015ec570dcc174e95546db28e45ee468f51b88
author Alexander Hansen <alexander.hansen@9elements.com> Tue Nov 11 11:14:39 2025 +0100
committer Alexander Hansen <alexander.hansen@9elements.com> Tue Nov 11 13:57:18 2025 +0100
tree c84f2d59a70fbbddbe0acd15b0cb0753788cfca0
parent a6046a7b051c230b011122667cffd2a5bcd54012

common: use PDI constants for ObjectMapper

Pull the method names from PDI.

Tested: Inspection only.

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

common/utils.cpp

diff --git a/common/utils.cpp b/common/utils.cpp
index 13ac144..1e41100 100644
--- a/common/utils.cpp
+++ b/common/utils.cpp
@@ -232,9 +232,9 @@
     std::map<std::string, std::vector<std::string>> mapperResponse;
     auto& bus = DBusHandler::getBus();
 
-    auto mapper = bus.new_method_call(ObjectMapper::default_service,
-                                      ObjectMapper::instance_path,
-                                      ObjectMapper::interface, "GetObject");
+    auto mapper = bus.new_method_call(
+        ObjectMapper::default_service, ObjectMapper::instance_path,
+        ObjectMapper::interface, ObjectMapper::method_names::get_object);
 
     if (interface)
     {
@@ -255,9 +255,9 @@
     const std::vector<std::string>& ifaceList) const
 {
     auto& bus = pldm::utils::DBusHandler::getBus();
-    auto method = bus.new_method_call(ObjectMapper::default_service,
-                                      ObjectMapper::instance_path,
-                                      ObjectMapper::interface, "GetSubTree");
+    auto method = bus.new_method_call(
+        ObjectMapper::default_service, ObjectMapper::instance_path,
+        ObjectMapper::interface, ObjectMapper::method_names::get_sub_tree);
     method.append(searchPath, depth, ifaceList);
     auto reply = bus.call(method, dbusTimeout);
     auto response = reply.unpack<GetSubTreeResponse>();
@@ -273,7 +273,8 @@
     auto& bus = pldm::utils::DBusHandler::getBus();
     auto method = bus.new_method_call(
         ObjectMapper::default_service, ObjectMapper::instance_path,
-        ObjectMapper::interface, "GetSubTreePaths");
+        ObjectMapper::interface,
+        ObjectMapper::method_names::get_sub_tree_paths);
     method.append(objectPath, depth, ifaceList);
     auto reply = bus.call(method, dbusTimeout);
 
@@ -285,9 +286,9 @@
     const std::string& path, const std::vector<std::string>& ifaceList) const
 {
     auto& bus = pldm::utils::DBusHandler::getBus();
-    auto method = bus.new_method_call(ObjectMapper::default_service,
-                                      ObjectMapper::instance_path,
-                                      ObjectMapper::interface, "GetAncestors");
+    auto method = bus.new_method_call(
+        ObjectMapper::default_service, ObjectMapper::instance_path,
+        ObjectMapper::interface, ObjectMapper::method_names::get_ancestors);
     method.append(path, ifaceList);
     auto reply = bus.call(method, dbusTimeout);
     auto response = reply.unpack<GetAncestorsResponse>();
@@ -421,7 +422,8 @@
     auto& bus = DBusHandler::getBus();
     auto method = bus.new_method_call(
         ObjectMapper::default_service, ObjectMapper::instance_path,
-        ObjectMapper::interface, "GetAssociatedSubTree");
+        ObjectMapper::interface,
+        ObjectMapper::method_names::get_associated_sub_tree);
     method.append(objectPath, subtree, depth, ifaceList);
     auto reply = bus.call(method, dbusTimeout);
     auto response = reply.unpack<GetAssociatedSubTreeResponse>();