Bug 794243 (JBEPP-1298)

Summary: Problems with saving WSRP producer options
Product: [JBoss] JBoss Enterprise Portal Platform 5 Reporter: Viliam Rockai <vrockai>
Component: PortalAssignee: claprun <claprun>
Status: CLOSED NEXTRELEASE QA Contact:
Severity: high Docs Contact:
Priority: high    
Version: 5.2.0.ER04CC: epp-bugs, mwringe, theute
Target Milestone: ---   
Target Release: 5.2.0.ER06   
Hardware: Unspecified   
OS: Unspecified   
URL: http://jira.jboss.org/jira/browse/JBEPP-1298
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-10-25 17:12:47 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:

Description Viliam Rockai 2011-10-19 12:35:12 UTC
project_key: JBEPP

When I try to save any option (besides adding new properties) on WSRP producer page, clicking on the save button won't do that. I have to click on it twice, to get the successful save message and actually save the data.

Comment 1 Viliam Rockai 2011-10-19 13:52:19 UTC
Link: Added: This issue is related to JBQA-5457


Comment 2 Matt Wringe 2011-10-25 00:35:23 UTC
This is due to the follow lines being added to the listeners ProducerBean class in the WSRP admin:

// bypass the rest of the life cycle and re-display page
FacesContext.getCurrentInstance().renderResponse();

So we are reloading the page whenever we see a change instead of letting the save method take care of it. On the second hit, there is no change, so the listener don't get called, so the renderResponse() method isn't invoked, and the save method can be invoked as expected.

Once these lines are removed, then it works as expected.

Note: this will require a new version of wsrp to be released for this to be fixed (and occurs in GateIn trunk as well).

Chris, do you know why those lines are in the ProducerBean class?

Comment 4 Jared MORGAN 2011-11-22 23:54:53 UTC
Release Notes Docs Status: Added: Not Required