Fixes #38805 - Remove options related to filter overrides

[adamruzicka]

Companion to theforeman/foreman#10370

[ofedoren]

Thanks, @adamruzicka, LGTM, just one inline comment:

[ofedoren]

This information is still "there". WDYT about including it, but since it's on a different level, we could do it in two ways:

1. Show at the same level to be "compatible" with the changes:

      from :role do
        collection :locations, _("Locations"), numbered: false, hide_blank: true do
          custom_field Fields::Reference, name_key: :title
        end
        collection :organizations, _("Organizations"), numbered: false, hide_blank: true do
          custom_field Fields::Reference, name_key: :title
        end
      end

2. Include it at role level, but this would require some additional changes:

      field :role, _("Role"), Fields::Reference, details: [
        { key: :locations, structured_label: _('Locations'), except: :serialized },
        { key: :organizations, structured_label: _('Organizations'), except: :serialized }
      ], display_field_key: :name

and then in lib/hammer_cli_foreman/output/formatters.rb:36:

# ...
# The second check is for `base` adapter, since it's quite limited and this is a quick workaround instead of adding a better handler for nested structures (which I'm not sure we want for `base` though)
next if (detail[:id] && !show_ids) || (required_features & Array(detail[:except])).any?
# ...

[adamruzicka]

"same level" meaning showing it as if it was a attribute of the filter itself?

[ofedoren]

Yeap. Maybe include in the label (inherited), but that wouldn't as "compatible" then :/

[adamruzicka]

Went with this

# hammer filter info --id 338
Id:                        338
Resource type:             Location
Search:                    none
Role:                      zGptuCUriW
Permissions:               view_locations, assign_locations
Created at:                2025/10/10 09:52:59
Updated at:                2025/10/10 09:52:59
Locations (inherited):
    DVAErp
Organizations (inherited):
    GxRhDkKoNeDD

[ofedoren]

Thanks, @adamruzicka, I think this is ready.

[ofedoren]

The main PR is merged, thus merging, thanks, @adamruzicka !