Bug 841045

Summary: advisoryInfo-plain.xhtml is missing parameters on the repo-plain.xhtml link
Product: [Other] RHQ Project Reporter: James Livingston <jlivings>
Component: Core UIAssignee: Mike Thompson <mithomps>
Status: CLOSED CURRENTRELEASE QA Contact: Mike Foley <mfoley>
Severity: high Docs Contact:
Priority: high    
Version: 4.3CC: hrupp, joallen, rpradeep
Target Milestone: ---   
Target Release: RHQ 4.5.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 4.5.0 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 841049 (view as bug list) Environment:
Last Closed: 2013-09-01 09:57:05 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:
Bug Depends On:    
Bug Blocks: 841049    
Attachments:
Description Flags
fix none

Description James Livingston 2012-07-18 00:48:20 UTC 
advisoryInfo-plain.xhtml has a <h:outputLink>  pointing to repo-plain.xhtml which does not include the "id" parameter, which results in an error when the link is followed:
  Error reading 'percentComplete' on type org.rhq.enterprise.gui.content.RepoDetailsUIBean


The fix is simply to add the missing parameters
Comment 1 James Livingston 2012-07-18 00:52:59 UTC 
Created attachment 598769 [details]
fix

Trivial fix, confirmed working by customer against JON 3.1
Comment 2 Charles Crouch 2012-08-02 20:52:13 UTC 
Making this bug target RHQ4.5, will use Bug 841049 for the JON311 target
Comment 3 Mike Thompson 2012-08-03 14:38:36 UTC 
I applied James patch to master commit id: 970715c

I was unable to test this as I cannot find the advisory pages and reproduction steps are missing. Discussed with the group and will check in patch anyway. The patch will not be picked to JON 3.1.1 until test steps are confirmed.
Comment 4 James Livingston 2012-08-05 22:42:15 UTC 
That's an area of JON I don't know well, so I'm not sure how to get there in the UI. Basically you need to view an advisory (advisoryInfo-plain) and then follow the link to the repo it's from.

The bug was fairly obvious, so was easy to fix without being able to test it.
Comment 5 Heiko W. Rupp 2013-09-01 09:57:05 UTC 
Bulk closing of items that are on_qa and in old RHQ releases, which are out for a long time and where the issue has not been re-opened since.