diff options
| author | Yuvakumar Selvamani <yuvakumars@ami.com> | 2026-09-23 13:25:25 +0300 |
|---|---|---|
| committer | Yuvakumar Selvamani <yuvakumars@ami.com> | 2026-10-09 13:43:51 +0300 |
| commit | b83413c32874dd05359a912486ff7cb6103af1b0 (patch) | |
| tree | b9a9570a57c2a3a76ad6d42ca60e8a15d78eae9d /include/image_upload.hpp | |
| parent | dba5d4ddde45989e017d3842370852266983a243 (diff) | |
| download | bmcweb-master.tar.xz | |
getFabricAdapterLocation() registered its D-Bus LocationCode property
response handler as an inline lambda, requiring an ast-grep-ignore
suppression for the long-lambda rule.
This commit extracts the body into a named function
(afterGetFabricAdapterLocation) wired via std::bind_front into the safe
dbus::utility::getProperty<std::string> API. This is a pure refactor:
logic is unchanged. The ast-grep-ignore comment is removed since the
code is no longer flagged by the rule.
Tested:
- Booted an AST2600 SoC platform and exercised the endpoint that
actually invokes the refactored code live: GET
/redfish/v1/Systems/system/FabricAdapters/<id>/ (single-adapter
endpoint, not the collection) returned 200 OK with
Location/PartLocation/ServiceLabel populated from the adapter's
LocationCode D-Bus property, matching the value set on the test
object. No errors logged for fabric_adapters.hpp for the full test
session.
- Redfish Service Validator: 5923 Pass, 353 Warn, 0 Fail.
Change-Id: I1f6594f690b8131ac17d6d38502e5674f15e7cb6
Signed-off-by: Yuvakumar Selvamani <yuvakumars@ami.com>
Diffstat (limited to 'include/image_upload.hpp')
0 files changed, 0 insertions, 0 deletions
