Red Hat Bugzilla – Bug 1274685
AssetMgmt: New Repository dialog validation broken when Managed Repository is checked
Last modified: 2015-11-18 09:31:59 EST
Description of problem: * "Finish" button is activated after each change in the form (changing repo name, selecting org. unit) even if the new state is not ready for submitting and so the Finish button is disabled in the next moment. This results in a "blinking" effect, which is distracting and not expected. * "Next" button is always enabled even if repository name is not valid or no org. unit is selected. Version-Release number of selected component (if applicable): 6.2.0.ER4 How reproducible: With ***asset management enabled*** only (you have to log in with kiemgmt role (+ additional role like admin). Steps to Reproduce: 1. Go to Administration perspective, select Repositories -> New repository. 2. Check Managed Repository checkbox. 3. Select any org. unit. 4. Type some text into Repository Name field. 5. Select a different org unit. Actual results: * In step 2, Next button is enabled. * In steps 4 and 5, Finish button will become enabled for a short period of time. Expected results: * Next button should only be enabled if the provided repository name is valid and existing org. unit is selected. * Finish button should not be enabled in the first step of the wizard when Managed Repository is checked. In the next step of the wizard, it should only be enabled after a change event that results in a state that is ready for submitting. Additional info:
The following commits fixes the issue. 0.7.x https://github.com/uberfire/uberfire-extensions/commit/7a29bcdcf44cf6da27648ef8531a2316cc8ef3a1 6.3.x: https://github.com/droolsjbpm/droolsjbpm-build-bootstrap/commit/ec17dd2bca1a82ee18e7156fc27c85eb6871d341 https://github.com/droolsjbpm/guvnor/commit/c28d95a4e001801e425ab4631487a9b363b82e62 master: https://github.com/uberfire/uberfire-extensions/commit/c03701768b3569203ea31eeb0982e06cb844237f https://github.com/droolsjbpm/droolsjbpm-build-bootstrap/commit/92c0b0b5b339d469ca381fb067738fad9ea93486 https://github.com/droolsjbpm/guvnor/commit/708823061234c91e48368289f49d108da6b33636
Works perfectly now. Great job, Walter!
Thanks, it's nice to see the BZ list decrease.