Actions
Bug #24980
closedSingle reference fields are wrongly rendered in yaml and json
Difficulty:
Triaged:
No
Bugzilla link:
Pull request:
Team Backlog:
Description
When there's a single reference in command's output definition, yaml and json output adapters print the whole root data structure in place of the reference field.
For example:
field nil, _("Domain"), Fields::SingleReference, :key => :domain
prints:
hammer --output=json hostgroup info --id 1 { "ID": 1, "NAME": "Provisioning", "TITLE": "Provisioning", "NETWORK": { "DOMAIN": { "CONTENT_SOURCE_ID": null, "CONTENT_SOURCE_NAME": null, "CONTENT_VIEW_ID": null, "CONTENT_VIEW_NAME": null, "LIFECYCLE_ENVIRONMENT_ID": null, "LIFECYCLE_ENVIRONMENT_NAME": null, "KICKSTART_REPOSITORY_ID": null, "KICKSTART_REPOSITORY_NAME": null, "SUBNET_ID": 1, "SUBNET_NAME": "tstracho-laptop", "OPERATINGSYSTEM_ID": 3, "OPERATINGSYSTEM_NAME": "CentOS 7", "DOMAIN_ID": 1, "DOMAIN_NAME": "tstracho-laptop", "ENVIRONMENT_ID": null, "ENVIRONMENT_NAME": null, "COMPUTE_PROFILE_ID": null, "COMPUTE_PROFILE_NAME": null, "ANCESTRY": null, "PARENT_ID": null, "PARENT_NAME": null, "PTABLE_ID": 87, "PTABLE_NAME": "Kickstart default", "MEDIUM_ID": 9, "MEDIUM_NAME": "CentOS Czech mirror", "PXE_LOADER": null, "SUBNET6_ID": null, "SUBNET6_NAME": null, "ARCHITECTURE_ID": 1, "ARCHITECTURE_NAME": "x86_64", "REALM_ID": null, "REALM_NAME": null, "CREATED_AT": "2018-01-24 12:22:29 -1100", "UPDATED_AT": "2018-01-24 12:22:29 -1100", ...
Expected behavior:
JSON and YAML adaptera should print the same fields as the base adapter.
Fixing this might require implementation of formatters specific to certain adapters.
Updated by Tomáš Strachota over 6 years ago
- Related to Tracker #14914: Tracker for output related issues added
Updated by Oleh Fedorenko over 6 years ago
- Status changed from New to Assigned
- Assignee set to Oleh Fedorenko
Updated by The Foreman Bot over 6 years ago
- Status changed from Assigned to Ready For Testing
- Pull request https://github.com/theforeman/hammer-cli/pull/287 added
Updated by The Foreman Bot over 6 years ago
- Pull request https://github.com/theforeman/hammer-cli-foreman/pull/389 added
Updated by Tomáš Strachota over 6 years ago
- Status changed from Ready For Testing to Assigned
- Assignee changed from Oleh Fedorenko to Rahul Bajaj
- Pull request deleted (
https://github.com/theforeman/hammer-cli-foreman/pull/389, https://github.com/theforeman/hammer-cli/pull/287)
Updated by The Foreman Bot over 6 years ago
- Status changed from Assigned to Ready For Testing
- Pull request https://github.com/theforeman/hammer-cli/pull/288 added
Updated by The Foreman Bot over 6 years ago
- Pull request https://github.com/theforeman/hammer-cli-foreman/pull/390 added
Updated by Rahul Bajaj over 6 years ago
- Status changed from Ready For Testing to Closed
Applied in changeset 70799cb08ae2c16aecc2dc913495d0c1d1b6d6b4.
Updated by Martin Bacovsky over 5 years ago
- Fixed in Releases hammer-cli-0.15.0 added
Actions