Bug 1383682 - (Default) label is missing in the tree view for default filter after adding a new filter
Summary: (Default) label is missing in the tree view for default filter after adding a...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.7.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: GA
: 5.8.0
Assignee: Hilda Stastna
QA Contact: Parthvi Vala
URL:
Whiteboard: ui:filter:tree_view
Depends On:
Blocks: 1393025
TreeView+ depends on / blocked
 
Reported: 2016-10-11 12:29 UTC by Matouš Mojžíš
Modified: 2019-05-30 09:14 UTC (History)
7 users (show)

Fixed In Version: 5.8.0.0
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1393025 (view as bug list)
Environment:
Last Closed: 2017-06-12 17:39:26 UTC
Category: ---
Cloudforms Team: CFME Core
Target Upstream Version:


Attachments (Terms of Use)

Description Matouš Mojžíš 2016-10-11 12:29:44 UTC
Description of problem:


Version-Release number of selected component (if applicable):
5.7.0.4

How reproducible:
Always

Steps to Reproduce:
1.Go for example to Compute -> Infrastructure -> Providers
2.Add a filter and set it as default
3.Add another filter

Actual results:
After adding another filter no filter has (Default) label.
It appears after refresh.

Expected results:
(Default) label should be displayed even after adding another filter.

Additional info:

Comment 2 Harpreet Kataria 2016-10-11 17:25:11 UTC

*** This bug has been marked as a duplicate of bug 1376121 ***

Comment 3 Matouš Mojžíš 2016-10-11 17:33:55 UTC
This is different. This happens to global filters too and only when I save filter...

Comment 4 Harpreet Kataria 2016-10-13 17:30:28 UTC
Hilda,

Can you take a look at this, it looks like after  a new filter is added it removes label "Default" from existing default filter. LEt me know if you have questions.

Thanks,
~Harpreet

Comment 5 Hilda Stastna 2016-10-25 13:08:50 UTC
https://github.com/ManageIQ/manageiq/pull/12163

Comment 6 CFME Bot 2016-11-01 17:36:09 UTC
New commit detected on ManageIQ/manageiq/euwe:
https://github.com/ManageIQ/manageiq/commit/80afa9b5a7e83bfb1f94bf0159b9abc822c5b409

commit 80afa9b5a7e83bfb1f94bf0159b9abc822c5b409
Author:     Milan Zázrivec <mzazrivec>
AuthorDate: Mon Oct 31 18:24:21 2016 +0100
Commit:     Oleg Barenboim <chessbyte>
CommitDate: Tue Nov 1 13:31:22 2016 -0400

    Merge pull request #12163 from hstastna/Default_label_after_adding_new_filter
    
    Display Default label for filter in My Filters
    (cherry picked from commit bff813d3ebd1530fdbd5f72bce15a5f98b0dc128)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1383682

 app/controllers/application_controller/filter.rb | 2 ++
 1 file changed, 2 insertions(+)

Comment 9 Matouš Mojžíš 2017-04-25 11:01:03 UTC
Verified in 5.8.0.11. Label is not missing anymore.


Note You need to log in before you can comment on or make changes to this bug.