Actions
Bug #32883
closedMenu::Item doesn't use the `:if` option
Description
The `:if` option is saved into @condition, but it doesn't appear to be used anywhere: https://github.com/theforeman/foreman/blob/3347fa49d500964f0209122d8d36c920d1feafcc/app/registries/menu/item.rb#L14
One suggestion is to use @condition in the `to_hash` method to control if the menu item is used or not.
It is also advertised that a Proc can be passed into the `:if` option. This functionality should also be re-implemented. (https://github.com/theforeman/foreman/blob/develop/app/registries/menu/manager.rb#L60)
Updated by The Foreman Bot over 3 years ago
- Status changed from New to Ready For Testing
- Assignee set to Ian Ballou
- Pull request https://github.com/theforeman/foreman/pull/8622 added
Updated by Anonymous over 3 years ago
- Status changed from Ready For Testing to Closed
Applied in changeset foreman|0e7b3dd50bb930a0500ddc834146aecbb527a837.
Actions