Bug 1348813 - Default view settings fails for some pages
Summary: Default view settings fails for some pages
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.6.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: GA
: 5.8.0
Assignee: Jozef Zigmund
QA Contact: Jiri Stefanisin
URL:
Whiteboard: view:ui:general
: 1283118 1342567 (view as bug list)
Depends On:
Blocks: 1349636 1395813
TreeView+ depends on / blocked
 
Reported: 2016-06-22 06:55 UTC by Aziza Karol
Modified: 2017-06-12 17:38 UTC (History)
11 users (show)

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


Attachments (Terms of Use)

Description Aziza Karol 2016-06-22 06:55:25 UTC
Description of problem:


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

How reproducible:
100%

Steps to Reproduce:
1.Navigate to settings->My setting->default views
2.change view to tile or list for: 
  clouds:cloud providers,instances
  infrastructure:Infrastructure providers,Vms
3.Navigate to the respective pages to check if the setting gets applied



Actual results:
View does not get changed to tile or list

Expected results:


Additional info:

Comment 1 Harpreet Kataria 2016-06-22 14:03:33 UTC
Eric,

Need to make sure that correct keys exist for each of the above metnioned views in this list https://github.com/ManageIQ/manageiq/blob/master/app/helpers/ui_constants.rb#L169, i see some extar ones that should be deleted from this list, after that need to make sure that keys match up in this view with DEFAULT_SETTINGS[:views] hash https://github.com/ManageIQ/manageiq/blob/master/app/views/configuration/_ui_2.html.haml 

Keys to be used in https://github.com/ManageIQ/manageiq/blob/master/app/helpers/ui_constants.rb#L169 should be as following:
manageiq_providers_cloudmanager - Cloud Provider
manageiq_providers_cloudmanager_vm - Cloud Instances
manageiq_providers_cloudmanager_template - Cloud Images
manageiq_providers_inframanager - Infra Providers
vmortemplate - Infra VMs & Templates
manageiq_providers_inframanager_vm - Infra VMs
manageiq_providers_inframanager_template - Infra Templates

Should clean up keys such as emscloud, emsinfra, vmcloud, vminfra

Let me know if you have questions.

Thanks,
~Harpreet

Comment 4 CFME Bot 2016-06-23 16:06:48 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/0cc4339f04bfddc0353d8af392d0f58b1b1cc99e

commit 0cc4339f04bfddc0353d8af392d0f58b1b1cc99e
Author:     Eric Hayes <eric>
AuthorDate: Wed Jun 22 17:33:48 2016 -0700
Commit:     Eric Hayes <eric>
CommitDate: Thu Jun 23 07:56:35 2016 -0700

    Update settings keys to match name changes
    and remove leftovers from UI constants.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1348813

 app/helpers/ui_constants.rb             |  6 +-----
 app/views/configuration/_ui_2.html.haml | 18 ++++++++++++------
 2 files changed, 13 insertions(+), 11 deletions(-)

Comment 6 Jozef Zigmund 2016-06-27 15:45:29 UTC
*** Bug 1342567 has been marked as a duplicate of this bug. ***

Comment 8 Matouš Mojžíš 2016-09-30 14:51:44 UTC
Middleware domains, middleware messaging, cloud providers and cloud tenants still fail.

Comment 10 CFME Bot 2016-10-25 00:36:10 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/505c9d3621ce01e9bf009a56dc89ccf9a2aec6cc

commit 505c9d3621ce01e9bf009a56dc89ccf9a2aec6cc
Author:     Jozef Zigmund <jzigmund>
AuthorDate: Mon Oct 24 17:01:36 2016 +0200
Commit:     Jozef Zigmund <jzigmund>
CommitDate: Mon Oct 24 17:03:18 2016 +0200

    Add missing VIEW_RESOURCES
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1348813

 app/helpers/ui_constants.rb | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

Comment 11 CFME Bot 2016-10-25 02:05:49 UTC
New commit detected on ManageIQ/manageiq/euwe:
https://github.com/ManageIQ/manageiq/commit/8136ea12de818c238ee52635cb144039f7a7e172

commit 8136ea12de818c238ee52635cb144039f7a7e172
Author:     Dan Clarizio <dclarizi>
AuthorDate: Mon Oct 24 14:31:24 2016 -1000
Commit:     Oleg Barenboim <chessbyte>
CommitDate: Tue Oct 25 04:01:18 2016 +0200

    Merge pull request #12136 from jzigmund/bz1348813-fix_default_view_settings
    
    Fix default view settings
    (cherry picked from commit 0632cd6360b53ce2e9ec0d31189f7fe7b685c804)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1348813

 app/helpers/ui_constants.rb             | 7 +++++--
 app/views/configuration/_ui_2.html.haml | 4 ++--
 2 files changed, 7 insertions(+), 4 deletions(-)

Comment 17 Satoe Imaishi 2016-12-13 19:21:27 UTC
*** Bug 1283118 has been marked as a duplicate of this bug. ***


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