Bug 1290440 - UI: Report Editor - After resetting changes on edit screen then trying to change tab throws an error
Summary: UI: Report Editor - After resetting changes on edit screen then trying to cha...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: UI - OPS
Version: 5.5.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: GA
: 5.5.2
Assignee: Harpreet Kataria
QA Contact: Nandini Chandra
URL:
Whiteboard:
Depends On: 1290170
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-12-10 14:14 UTC by John Prause
Modified: 2016-02-10 15:24 UTC (History)
7 users (show)

Fixed In Version: 5.5.2.1
Doc Type: Bug Fix
Doc Text:
Clone Of: 1290170
Environment:
Last Closed: 2016-02-10 15:24:18 UTC
Category: ---
Cloudforms Team: ---
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2016:0159 0 normal SHIPPED_LIVE CFME 5.5.2 bug fixes and enhancement update 2016-02-10 20:19:02 UTC

Comment 1 Harpreet Kataria 2015-12-11 17:58:12 UTC
https://github.com/ManageIQ/manageiq/pull/5804

Comment 2 CFME Bot 2015-12-14 22:00:53 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/8ac9880d457acc56347ef07ae0da6586f8287b4c

commit 8ac9880d457acc56347ef07ae0da6586f8287b4c
Author:     Harpreet Kataria <hkataria>
AuthorDate: Fri Dec 11 12:20:14 2015 -0500
Commit:     Harpreet Kataria <hkataria>
CommitDate: Fri Dec 11 12:56:34 2015 -0500

    Fixed issue in change tab after Reset is pressed in report editor
    
    - Fixed code to set prefix correctly for tab ids on tabs in Reports editor after Reset button is pressed
    - Added spec test to verify tab ids
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1290440
    https://bugzilla.redhat.com/show_bug.cgi?id=1290170

 app/controllers/report_controller.rb               |  2 -
 app/controllers/report_controller/reports.rb       | 39 +++++-------------
 .../report_controller/reports/editor.rb            | 16 ++------
 .../miq_report_controller/reports/editor_spec.rb   | 47 ++++++++++++++++++++++
 4 files changed, 62 insertions(+), 42 deletions(-)

Comment 3 CFME Bot 2015-12-14 22:01:01 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/ba3e50ad6b813308547c376989f19222a5e4e8b0

commit ba3e50ad6b813308547c376989f19222a5e4e8b0
Author:     Harpreet Kataria <hkataria>
AuthorDate: Fri Dec 11 12:57:08 2015 -0500
Commit:     Harpreet Kataria <hkataria>
CommitDate: Fri Dec 11 12:57:08 2015 -0500

    Deleted commented code.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1290440
    https://bugzilla.redhat.com/show_bug.cgi?id=1290170

 app/controllers/report_controller/reports/editor.rb | 9 ---------
 1 file changed, 9 deletions(-)

Comment 5 CFME Bot 2015-12-16 20:56:59 UTC
New commit detected on cfme/5.5.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=acff00bc7b192283ae5e025ad7d61cb6e53938d2

commit acff00bc7b192283ae5e025ad7d61cb6e53938d2
Author:     Harpreet Kataria <hkataria>
AuthorDate: Fri Dec 11 12:20:14 2015 -0500
Commit:     Harpreet Kataria <hkataria>
CommitDate: Mon Dec 14 17:13:02 2015 -0500

    Fixed issue in change tab after Reset is pressed in report editor
    
    - Fixed code to set prefix correctly for tab ids on tabs in Reports editor after Reset button is pressed
    - Added spec test to verify tab ids
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1290440
    https://bugzilla.redhat.com/show_bug.cgi?id=1290170
    (cherry picked from commit 8ac9880)

 app/controllers/report_controller.rb               |  2 -
 app/controllers/report_controller/reports.rb       | 39 +++++-------------
 .../report_controller/reports/editor.rb            | 16 ++------
 .../miq_report_controller/reports/editor_spec.rb   | 47 ++++++++++++++++++++++
 4 files changed, 62 insertions(+), 42 deletions(-)

Comment 6 CFME Bot 2015-12-16 20:57:06 UTC
New commit detected on cfme/5.5.z:
https://code.engineering.redhat.com/gerrit/gitweb?p=cfme.git;a=commitdiff;h=2fb2d7f773695e92d797e67cbcaa2c90d26457fe

commit 2fb2d7f773695e92d797e67cbcaa2c90d26457fe
Author:     Harpreet Kataria <hkataria>
AuthorDate: Fri Dec 11 12:57:08 2015 -0500
Commit:     Harpreet Kataria <hkataria>
CommitDate: Mon Dec 14 17:13:12 2015 -0500

    Deleted commented code.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1290440
    https://bugzilla.redhat.com/show_bug.cgi?id=1290170
    (cherry picked from commit ba3e50a)

 app/controllers/report_controller/reports/editor.rb | 9 ---------
 1 file changed, 9 deletions(-)

Comment 8 Nandini Chandra 2016-01-18 17:40:06 UTC
Verified in 5.5.2.1

Comment 9 errata-xmlrpc 2016-02-10 15:24:18 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/RHBA-2016:0159


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