diff options
Diffstat (limited to 'redfish-core/lib/chassis.hpp')
| -rw-r--r-- | redfish-core/lib/chassis.hpp | 36 |
1 files changed, 13 insertions, 23 deletions
diff --git a/redfish-core/lib/chassis.hpp b/redfish-core/lib/chassis.hpp index 745f5af768..8fc9129c20 100644 --- a/redfish-core/lib/chassis.hpp +++ b/redfish-core/lib/chassis.hpp @@ -189,8 +189,7 @@ inline void const std::variant<std::string>& property) { if (ec) { - BMCWEB_LOG_DEBUG << "DBUS response error for " - "Location"; + BMCWEB_LOG_DEBUG << "DBUS response error for Location"; messages::internalError(asyncResp->res); return; } @@ -198,8 +197,7 @@ inline void const std::string* value = std::get_if<std::string>(&property); if (value == nullptr) { - BMCWEB_LOG_DEBUG << "Null value returned " - "for locaton code"; + BMCWEB_LOG_DEBUG << "Null value returned for locaton code"; messages::internalError(asyncResp->res); return; } @@ -219,16 +217,14 @@ inline void getChassisUUID(const std::shared_ptr<bmcweb::AsyncResp>& asyncResp, const std::variant<std::string>& chassisUUID) { if (ec) { - BMCWEB_LOG_DEBUG << "DBUS response error for " - "UUID"; + BMCWEB_LOG_DEBUG << "DBUS response error for UUID"; messages::internalError(asyncResp->res); return; } const std::string* value = std::get_if<std::string>(&chassisUUID); if (value == nullptr) { - BMCWEB_LOG_DEBUG << "Null value returned " - "for UUID"; + BMCWEB_LOG_DEBUG << "Null value returned for UUID"; messages::internalError(asyncResp->res); return; } @@ -338,12 +334,10 @@ inline void requestRoutesChassis(App& app) connectionNames[0].second; const std::array<const char*, 2> hasIndicatorLed = { "xyz.openbmc_project.Inventory.Item.Panel", - "xyz.openbmc_project.Inventory.Item.Board." - "Motherboard"}; + "xyz.openbmc_project.Inventory.Item.Board.Motherboard"}; const std::string assetTagInterface = - "xyz.openbmc_project.Inventory.Decorator." - "AssetTag"; + "xyz.openbmc_project.Inventory.Decorator.AssetTag"; if (std::find(interfaces2.begin(), interfaces2.end(), assetTagInterface) != interfaces2.end()) { @@ -354,8 +348,7 @@ inline void requestRoutesChassis(App& app) if (ec) { BMCWEB_LOG_DEBUG - << "DBus response error for " - "AssetTag"; + << "DBus response error for AssetTag"; messages::internalError(asyncResp->res); return; } @@ -365,8 +358,7 @@ inline void requestRoutesChassis(App& app) if (assetTag == nullptr) { BMCWEB_LOG_DEBUG - << "Null value returned " - "for Chassis AssetTag"; + << "Null value returned for Chassis AssetTag"; messages::internalError(asyncResp->res); return; } @@ -458,9 +450,9 @@ inline void requestRoutesChassis(App& app) { getChassisUUID(asyncResp, connectionName, path); } - else if (interface == - "xyz.openbmc_project.Inventory." - "Decorator.LocationCode") + else if ( + interface == + "xyz.openbmc_project.Inventory.Decorator.LocationCode") { getChassisLocationCode(asyncResp, connectionName, path); @@ -513,8 +505,7 @@ inline void requestRoutesChassis(App& app) { asyncResp->res.addHeader( boost::beast::http::field::warning, - "299 - \"IndicatorLED is deprecated. Use " - "LocationIndicatorActive instead.\""); + "299 - \"IndicatorLED is deprecated. Use LocationIndicatorActive instead.\""); } const std::array<const char*, 2> interfaces = { @@ -562,8 +553,7 @@ inline void requestRoutesChassis(App& app) const std::array<const char*, 2> hasIndicatorLed = { "xyz.openbmc_project.Inventory.Item.Panel", - "xyz.openbmc_project.Inventory.Item.Board." - "Motherboard"}; + "xyz.openbmc_project.Inventory.Item.Board.Motherboard"}; bool indicatorChassis = false; for (const char* interface : hasIndicatorLed) { |