Univention Bugzilla – Full Text Bug Listing |
Summary: | No visible feedback if form input is invalid | ||
---|---|---|---|
Product: | UCS | Reporter: | Alexander Kläser <klaeser> |
Component: | UMC (Generic) | Assignee: | Jascha Geerds <geerds> |
Status: | CLOSED FIXED | QA Contact: | Alexander Kläser <klaeser> |
Severity: | normal | ||
Priority: | P1 | CC: | best, damrose, gohmann, klaeser, schwardt, walkenhorst, wiesenthal |
Version: | UCS 3.1 | ||
Target Milestone: | UCS 3.1-1-errata | ||
Hardware: | Other | ||
OS: | Linux | ||
What kind of report is it?: | --- | What type of bug is this?: | --- |
Who will be affected by this bug?: | --- | How will those affected feel about the bug?: | --- |
User Pain: | Enterprise Customer affected?: | ||
School Customer affected?: | ISV affected?: | ||
Waiting Support: | Flags outvoted (downgraded) after PO Review: | ||
Ticket number: | Bug group (optional): | Usability | |
Max CVSS v3 score: | |||
Bug Depends on: | 30091, 30109 | ||
Bug Blocks: |
Description
Alexander Kläser
2013-05-10 17:22:12 CEST
*** Bug 31585 has been marked as a duplicate of this bug. *** The patch has been applied. - ucs3.1-1 & YAML: univention-management-console-frontend (2.0.244-11) - ucs3.2-0 & Changelog: univention-management-console-frontend (2.0.257-1) I found a validation problem which is UDM specific → Bug 31716 isValid() checks first whether an invalid state is forced (via the property "valid"). This should be done in validate(), as well (before calling its super method). If this is done, we should be able to remove _isValidSubset() in umc/widgets/_FormWidgetMixin as it does not seem to apply here. Its super method in dijit/form/ValidationTextBox respects the outcome of isValid. But this does not need to be done via this Bug. As discussed, the behavior seems to be ok → Fixed Erratum 3.1-1 changes: OK YAML file: OK UCS 3.2 changes: Please increase first version number of source package → FAIL changelog: OK All in all ok, only the debian changelog number needs to be increased. (In reply to Alexander Kläser from comment #5) > All in all ok, only the debian changelog number needs to be increased. Done! OK → VERIFIED |