Bug 1383682

Summary:	(Default) label is missing in the tree view for default filter after adding a new filter
Product:	Red Hat CloudForms Management Engine	Reporter:	Matouš Mojžíš <mmojzis>
Component:	UI - OPS	Assignee:	Hilda Stastna <hstastna>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Parthvi Vala <pvala>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	5.7.0	CC:	cpelland, hkataria, hstastna, jhardy, mpovolny, obarenbo, simaishi
Target Milestone:	GA	Keywords:	Reopened, TestOnly
Target Release:	5.8.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:	ui:filter:tree_view
Fixed In Version:	5.8.0.0	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:
Clones:	1393025 (view as bug list)		Environment:
Last Closed:	2017-06-12 17:39:26 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	CFME Core	Target Upstream Version:
Bug Depends On:
Bug Blocks:	1393025

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.