Bug 1269115 - Flash message is displayed twice in reporting schedule page
Summary: Flash message is displayed twice in reporting schedule page
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.4.0
Hardware: Unspecified
OS: Unspecified
medium
low
Target Milestone: GA
: 5.5.0
Assignee: Libor Pichler
QA Contact: Ramesh A
URL:
Whiteboard:
Depends On:
Blocks: 1305171
TreeView+ depends on / blocked
 
Reported: 2015-10-06 11:41 UTC by Ramesh A
Modified: 2016-02-09 19:18 UTC (History)
6 users (show)

Fixed In Version: 5.5.0.7
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-12-08 13:35:28 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Add Schedule (59.51 KB, image/png)
2015-10-06 11:41 UTC, Ramesh A
no flags Details
Edit-cancel operation (61.47 KB, image/png)
2015-10-06 11:42 UTC, Ramesh A
no flags Details
Edit-Reset operation (55.55 KB, image/png)
2015-10-06 11:43 UTC, Ramesh A
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHSA-2015:2551 0 normal SHIPPED_LIVE Moderate: CFME 5.5.0 bug fixes and enhancement update 2015-12-08 17:58:09 UTC

Description Ramesh A 2015-10-06 11:41:19 UTC
Created attachment 1080202 [details]
Add Schedule

Description of problem:
Flash message is displayed twice in the Reports Schedule page

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

How reproducible:
Very

Steps to Reproduce:
1.Navigate to cloud intelligence ==> Reports ==> Schedules
2. Case-1: Configuration ==> Add a new Schedule ==> Click on Save button
3. Case-2: Select an existing schedule ==> Configuration ==> Edit this Schedule ==> Click on Cancel button
3. Case-2: Select an existing schedule ==> Configuration ==> Edit this Schedule ==> Make some changes and Click on Reset button

Actual results:
Displays flash twice in the UI.  Refer the screenshots for more information

Expected results:
Should display the flash message only once

Additional info:

Comment 1 Ramesh A 2015-10-06 11:42:28 UTC
Created attachment 1080204 [details]
Edit-cancel operation

Comment 2 Ramesh A 2015-10-06 11:43:00 UTC
Created attachment 1080205 [details]
Edit-Reset operation

Comment 3 Ramesh A 2015-10-06 11:43:48 UTC
Found this issue while testing BZ#1222920. Could be regression issue related to it??

Comment 5 Ramesh A 2015-10-06 11:52:40 UTC
Ignore the bug number mentioned in Comment#3, the correct bug number is BZ#1263868

Comment 7 CFME Bot 2015-10-16 13:15:27 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/e3af61d40e5499d8af07e64a848d4069dd76705a

commit e3af61d40e5499d8af07e64a848d4069dd76705a
Author:     Libor Pichler <lpichler>
AuthorDate: Thu Oct 15 09:30:41 2015 +0200
Commit:     Libor Pichler <lpichler>
CommitDate: Thu Oct 15 09:30:41 2015 +0200

    Remove duplicated flash message from reporting schedule page
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1269115

 app/views/report/_schedule_list.html.haml | 1 -
 1 file changed, 1 deletion(-)

Comment 9 Ramesh A 2015-10-28 06:02:30 UTC
Good to go.  Verified and working fine in 5.5.0.7-beta1.3.20151021133253_ed40d96

Comment 11 errata-xmlrpc 2015-12-08 13:35:28 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2015:2551

Comment 12 CFME Bot 2016-02-09 19:17:43 UTC
New commit detected on cfme/5.4.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=824fb6cde49129d3dfd52b96ff17c3bdbb9e2ff9

commit 824fb6cde49129d3dfd52b96ff17c3bdbb9e2ff9
Author:     Harpreet Kataria <hkataria>
AuthorDate: Fri Feb 5 17:55:24 2016 -0500
Commit:     Harpreet Kataria <hkataria>
CommitDate: Fri Feb 5 17:55:24 2016 -0500

    Return to the schedule info instead of schedules list
    
    after adding, saving and cancelling schedule from
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1269115
    https://bugzilla.redhat.com/show_bug.cgi?id=1305171
    (cherry-picked from commit fb84c947e374bd7ec618d03fdfb6dc0950cc2c04)

 vmdb/app/controllers/report_controller/schedules.rb | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

Comment 13 CFME Bot 2016-02-09 19:17:52 UTC
New commit detected on cfme/5.4.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=ccf711b8c30d97bc19e9d9b8f99c535178e36cae

commit ccf711b8c30d97bc19e9d9b8f99c535178e36cae
Merge: fc5f57c 3604939
Author:     Dan Clarizio <dclarizi>
AuthorDate: Tue Feb 9 13:36:58 2016 -0500
Commit:     Dan Clarizio <dclarizi>
CommitDate: Tue Feb 9 13:36:58 2016 -0500

    Merge branch '54z_pr_4879' into '5.4.z'
    
    Remove duplicated flash message from reporting schedule page
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1269115
    https://bugzilla.redhat.com/show_bug.cgi?id=1305171
    
    @dclarizi please review, clean patch was applied for changes in https://github.com/ManageIQ/manageiq/pull/4879
    
    See merge request !771

 vmdb/app/controllers/report_controller/schedules.rb | 4 +---
 vmdb/app/views/report/_schedule_list.html.haml      | 1 -
 2 files changed, 1 insertion(+), 4 deletions(-)

Comment 14 CFME Bot 2016-02-09 19:18:01 UTC
New commit detected on cfme/5.4.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=3604939e1131ae117232c95cab01872ceb02fe22

commit 3604939e1131ae117232c95cab01872ceb02fe22
Author:     Harpreet Kataria <hkataria>
AuthorDate: Fri Feb 5 17:56:34 2016 -0500
Commit:     Harpreet Kataria <hkataria>
CommitDate: Fri Feb 5 17:56:34 2016 -0500

    Remove duplicated flash message from reporting schedule page
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1269115
    https://bugzilla.redhat.com/show_bug.cgi?id=1305171
    (cherry-picked from commit e3af61d40e5499d8af07e64a848d4069dd76705a)

 vmdb/app/views/report/_schedule_list.html.haml | 1 -
 1 file changed, 1 deletion(-)


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