Bug 1670293 - Setting `Edit Report Menu` back to default does not work as expected.
Summary: Setting `Edit Report Menu` back to default does not work as expected.
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.9.8
Hardware: Unspecified
OS: Unspecified
medium
high
Target Milestone: GA
: 5.11.0
Assignee: Dávid Halász
QA Contact: Parthvi Vala
Red Hat CloudForms Documentation
URL:
Whiteboard:
Depends On:
Blocks: 1702489
TreeView+ depends on / blocked
 
Reported: 2019-01-29 07:47 UTC by Parthvi Vala
Modified: 2019-12-13 15:09 UTC (History)
7 users (show)

Fixed In Version: 5.11.0.1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1702489 (view as bug list)
Environment:
Last Closed: 2019-12-13 15:09:46 UTC
Category: Bug
Cloudforms Team: CFME Core
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Bug Demo (1.80 MB, video/webm)
2019-01-29 07:47 UTC, Parthvi Vala
no flags Details

Description Parthvi Vala 2019-01-29 07:47:20 UTC
Created attachment 1524474 [details]
Bug Demo

Description of problem:
Setting the `Edit Report Menus` to default doesn't really reset the menu to default.

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

How reproducible:
Always

Steps to Reproduce:
1. Add a new report or copy an existing report, say `testing report`.
2. Move the newly created report to a menu, say 'Tenants > Tenant Quotas'.
3. Check if the report is visible under the menu.
4. Go back to `Edit Report Menus`, and click on `Default`.
5. Click on 'Tenant Quotas' and check if the `testing report` is moved back to `available reports`.
6. Click on `Save`.
7. Go to `Edit Report Menus` > group > Tenants > Tenant Quotas, and see if the report `testing report` is still visible under `Tenant Quotas`.

Actual results:
Report `testing report` is still visible under `Tenants > Tenant Quotas`.

Expected results:
Report `testing report` must not be visible under the menu and things should be set back to default.

Additional info:
`Default` does not work correctly until the changes are committed.
After the 5th step, if you go to `Tenants > Tenant Quotas` and click on `Commit` and then click `Save`, the menu will be set to default.

Issue exists on 5.10 as well.

Comment 5 CFME Bot 2019-02-14 04:37:30 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:

https://github.com/ManageIQ/manageiq-ui-classic/commit/828d1922b9901085417a6c80cae2726ac3fd9274
commit 828d1922b9901085417a6c80cae2726ac3fd9274
Author:     Dávid Halász <dhalasz>
AuthorDate: Thu Jan 31 04:46:58 2019 -0500
Commit:     Dávid Halász <dhalasz>
CommitDate: Thu Jan 31 04:46:58 2019 -0500

    Delete the "report_menus" instead of :report_menus when resetting

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1670293

 app/controllers/report_controller/menus.rb | 2 +-
 spec/controllers/miq_report_controller/menus_spec.rb | 18 +
 2 files changed, 19 insertions(+), 1 deletion(-)

Comment 7 Parthvi Vala 2019-05-02 11:14:22 UTC
FIXED. Verified on 5.11.0.2.20190430174828_0e34dea.


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