Description of problem: Partially selected tree in role access control is not getting invert coloured on top tree item. Version-Release number of selected component (if applicable): How reproducible: Steps to Reproduce: 1. Create a new access control role. Save. 2. Edit access control role . Unselect some sub items in tree. Save. 3. Top tree checkbox colour is not getting inverted. Actual results: Top tree checkbox colour is not getting inverted. Expected results: Top tree checkbox colour getting inverted. Additional info:
https://github.com/ManageIQ/manageiq-ui-classic/pull/137
https://github.com/ManageIQ/manageiq/pull/13577
*** Bug 1428553 has been marked as a duplicate of this bug. ***
*** Bug 1429914 has been marked as a duplicate of this bug. ***
New commit detected on ManageIQ/manageiq/euwe: https://github.com/ManageIQ/manageiq/commit/e54076bf76cde1a19f7b9539801dae6eaa704abb commit e54076bf76cde1a19f7b9539801dae6eaa704abb Author: Eric Hayes <eric> AuthorDate: Wed Jan 18 16:18:40 2017 -0800 Commit: Eric Hayes <eric> CommitDate: Wed Jan 18 17:24:24 2017 -0800 Build tree by counting all descendants Currently selected boxes deeper in the tree do not “bubble up” the select state to their ancestors. This changes that by counting all descendants including hidden features, also taking into account 3 possible select states. https://bugzilla.redhat.com/show_bug.cgi?id=1348623 https://bugzilla.redhat.com/show_bug.cgi?id=1411831 app/controllers/ops_controller/rbac_tree.rb | 319 ++++++++++++++-------- spec/controllers/ops_controller/rbac_tree_spec.rb | 48 +++- 2 files changed, 252 insertions(+), 115 deletions(-)
https://github.com/ManageIQ/manageiq/pull/14421
New commit detected on ManageIQ/manageiq-ui-classic/master: https://github.com/ManageIQ/manageiq-ui-classic/commit/4f46dae639cd83fa3524b8503ea12557544affa1 commit 4f46dae639cd83fa3524b8503ea12557544affa1 Author: Eric Hayes <eric> AuthorDate: Tue Jan 3 13:17:53 2017 -0800 Commit: Eric Hayes <eric> CommitDate: Mon May 1 16:06:10 2017 -0700 Convert Menu Features Tree to use TreeBuilder Addresses: https://bugzilla.redhat.com/show_bug.cgi?id=1348623 https://bugzilla.redhat.com/show_bug.cgi?id=1411831 https://www.pivotaltracker.com/story/show/129518309 app/controllers/ops_controller/ops_rbac.rb | 11 +- app/controllers/ops_controller/rbac_tree.rb | 146 --------------------- app/presenters/menu/item.rb | 10 ++ app/presenters/menu/manager.rb | 12 +- app/presenters/menu/section.rb | 16 ++- app/presenters/tree_builder.rb | 8 +- app/presenters/tree_builder_ops_rbac_features.rb | 145 ++++++++++++++++++++ app/presenters/tree_node/menu/item.rb | 36 +++++ app/presenters/tree_node/menu/node.rb | 28 ++++ app/presenters/tree_node/menu/section.rb | 19 +++ app/presenters/tree_node/miq_product_feature.rb | 48 +++++++ app/views/ops/_rbac_role_details.html.haml | 118 ++++++++--------- spec/controllers/ops_controller/rbac_tree_spec.rb | 14 -- .../tree_builder_ops_rbac_features_spec.rb | 57 ++++++++ 14 files changed, 431 insertions(+), 237 deletions(-) delete mode 100644 app/controllers/ops_controller/rbac_tree.rb create mode 100644 app/presenters/tree_builder_ops_rbac_features.rb create mode 100644 app/presenters/tree_node/menu/item.rb create mode 100644 app/presenters/tree_node/menu/node.rb create mode 100644 app/presenters/tree_node/menu/section.rb create mode 100644 app/presenters/tree_node/miq_product_feature.rb delete mode 100644 spec/controllers/ops_controller/rbac_tree_spec.rb create mode 100644 spec/presenters/tree_builder_ops_rbac_features_spec.rb
This fix cannot be back ported. Targeted fixes for other branches have been submitted.
*** Bug 1464271 has been marked as a duplicate of this bug. ***
I believe this is fixed. Feel free to comment otherwise, if you still see the issue.