If a property value is invalid, the property page will fail to save the changes and there is not always a clear indication of a failure. This is especially true when the invalid property value appears on a lower place in the page that is outside of the active view port and in instances where the property appears on a collapsible section of the property page. This issue is even more hidden in a situation where a resource already in inventory may have one of its property values changed outside of the UI or a plug-in is upgraded that may result in a change of "required" value behavior. In such situations, the user updating unrelated properties via the UI may receive an error from the UI regarding the hidden/unchanged property value even though the user may have never interacted with such value. At bare minimum, any property value that is invalid and on a collapsible section should result in the collapsible section being expanded. Additionally, if any property value is invalid, a warning/error message should appear at the top of the page to give immediate user feedback. This would reduce the risk that unsaved property changes will be lost when the user navigates away from the property page. 1 Navigate to a JBoss AS Server resource's Inventory -> Connection page 2 Click Edit... button 3 Expand Snapshot Reports section 4 Remove value from Snapshot Config Directory * field 5 Collapse Snapshot Reports section 6 Click OK button * Notice page is still in Edit Mode * Notice that no succss or failure indication is shown * Scroll down the page and notice that everything appears normal * Notice that at the bottom of the page the OK button is still visible
let's make sure we do this right in gwt
This is handled in GWT/coregui. If there are any validation errors at configuration editor load time, an event is thrown that views can use to display a warning or error in the message bar, which is what is currently done by the resource config editor, conn props editor, and the test config editor. Also, if validation errors are detected within a property group, the collapsible pane for that group is auto-expanded, to make it harder for the user to miss the error icons next to the invalid props.
Verified on build#107 (Version: 4.1.0-SNAPSHOT Build Number: 9192306) Followed the steps and verified that removing the value from Snapshot Config Directory field displays a red icon and clicking save button displays a message "The following connection settings have invalid values: [Snapshot Config Directory]. The values must be corrected before the settings can be saved." Aslo verified that the collapsible pane for Snapshot Report section is auto-expanded. Marking as verified.
Bulk closing of old issues that are in VERIFIED state.