Bug 748111 - config editor: a prop value changed event fired when a prop becomes invalid incorrectly indicates that the set of invalid props did *not* change
Summary: config editor: a prop value changed event fired when a prop becomes invalid i...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: RHQ Project
Classification: Other
Component: Core UI
Version: 4.1
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: ---
Assignee: Ian Springer
QA Contact: Mike Foley
URL:
Whiteboard:
Depends On:
Blocks: rhq42
TreeView+ depends on / blocked
 
Reported: 2011-10-22 01:48 UTC by Ian Springer
Modified: 2013-08-06 00:41 UTC (History)
3 users (show)

Fixed In Version: 4.2
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-02-07 19:21:49 UTC
Embargoed:

Attachments (Terms of Use)

Description Ian Springer 2011-10-22 01:48:51 UTC 
When a prop becomes invalid, the various config edit views are supposed to display a red message to the user saying e.g. "The following properties have invalid values: [Foo] - the values must be corrected before the configuration can be saved.", but this is not happening due to the bad information in fired prop value changed events.
Comment 1 Ian Springer 2011-10-22 01:52:13 UTC 
[master dc7b71b] (http://git.fedorahosted.org/git/?p=rhq/rhq.git;a=commitdiff;h=dc7b71b) fixes this.
Comment 2 Sunil Kondkar 2011-10-24 12:09:13 UTC 
Verified on build#619 (Version: 4.1.0-SNAPSHOT Build Number: 7911e52)

It displays a red mark and a message for invalid property (Ex: :	
The following configuration properties have invalid values: [Banner]. The values must be corrected before the configuration can be saved.)
Also Verified for a group config (Ex: The following configuration properties have invalid values and must be corrected before the configuration can be saved: [[Plugins Directory]] )
Comment 3 Mike Foley 2012-02-07 19:21:49 UTC 
changing status of VERIFIED BZs for JON 2.4.2 and JON 3.0 to CLOSED/CURRENTRELEASE
Note You need to log in before you can comment on or make changes to this bug.