Bug 1217161

Summary:	Refreshing/returning to Configure/Configuration screen causes exception
Product:	Red Hat CloudForms Management Engine	Reporter:	Dan Clarizio <dclarizi>
Component:	UI - OPS	Assignee:	Harpreet Kataria <hkataria>
Status:	CLOSED ERRATA	QA Contact:	Nandini Chandra <nachandr>
Severity:	high	Docs Contact:
Priority:	high
Version:	5.4.0	CC:	hkataria, jhardy, mpovolny, nachandr, obarenbo, psavage, sshveta
Target Milestone:	GA
Target Release:	5.4.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2015-06-16 13:00:41 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Dan Clarizio 2015-04-29 16:40:49 UTC

Description of problem:
Exception when returning to Config screen

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

How reproducible:
Go to Configure/Configuration/Advanced Settings, hit browser refresh or go to a different tab and come back, exception occurs

Actual results:
[----] I, [2015-04-29T09:35:44.120216 #25491:c89e98]  INFO -- : Started GET "/ops/explorer" for 127.0.0.1 at 2015-04-29 09:35:44 -0700
[----] I, [2015-04-29T09:35:44.142746 #25491:c89e98]  INFO -- : Processing by OpsController#explorer as HTML
[----] F, [2015-04-29T09:35:44.957373 #25491:c89e98] FATAL -- : Error caught: [NoMethodError] undefined method `config' for #<Hash:0x0000000d0cad20>
/home/dclarizio/.rvm/gems/ruby-2.0.0-p598/bundler/gems/rails-8f014fba21f9/activesupport/lib/active_support/core_ext/object/try.rb:36:in `try'
/home/dclarizio/dev/manageiq/vmdb/app/controllers/ops_controller.rb:172:in `explorer'

Expected results:
No exception

Additional info:

Comment 2 CFME Bot 2015-04-29 21:51:05 UTC

New commit detected on manageiq/master:
https://github.com/ManageIQ/manageiq/commit/680c8df8f25d5f5bb6848b78892d95ade547bcb4

commit 680c8df8f25d5f5bb6848b78892d95ade547bcb4
Author:     Harpreet Kataria <hkataria>
AuthorDate: Wed Apr 29 14:14:53 2015 -0400
Commit:     Harpreet Kataria <hkataria>
CommitDate: Wed Apr 29 15:06:50 2015 -0400

    Fixed code that sets session[:changed]
    
    - Fixed to set session[:changed] based upon whether @edit[:current] is Hash or VMDB::Config
    - Added spec tests for the new method.
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1217161

 vmdb/app/controllers/ops_controller.rb       |  7 ++++-
 vmdb/spec/controllers/ops_controller_spec.rb | 45 ++++++++++++++++++++++++++++
 2 files changed, 51 insertions(+), 1 deletion(-)

Comment 3 Harpreet Kataria 2015-04-30 15:25:45 UTC

*** Bug 1217263 has been marked as a duplicate of this bug. ***

Comment 4 Harpreet Kataria 2015-05-01 13:25:51 UTC

*** Bug 1217693 has been marked as a duplicate of this bug. ***

Comment 6 Harpreet Kataria 2015-05-06 16:32:00 UTC

*** Bug 1219055 has been marked as a duplicate of this bug. ***

Comment 7 Nandini Chandra 2015-05-07 00:30:49 UTC

Verified in 5.4.0.0.25

Comment 9 errata-xmlrpc 2015-06-16 13:00:41 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://rhn.redhat.com/errata/RHBA-2015-1100.html