Bug 47824 - "please check this box if you want to proceed" error for any unchecked box in app settings
"please check this box if you want to proceed" error for any unchecked box in...
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: App Center
UCS 4.3
Other Linux
: P5 normal (vote)
: UCS 4.3-2-errata
Assigned To: Jens Pelzetter
Dirk Wiesenthal
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2018-09-17 11:19 CEST by Jürn Brodersen
Modified: 2018-12-05 14:39 CET (History)
2 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 3: Simply Wrong: The implementation doesn't match the docu
Who will be affected by this bug?: 5: Will affect all installed domains
How will those affected feel about the bug?: 2: A Pain – users won’t like this once they notice it
User Pain: 0.171
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments
screenshot (78.84 KB, image/jpeg)
2018-09-17 11:19 CEST, Jürn Brodersen
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jürn Brodersen univentionstaff 2018-09-17 11:19:29 CEST
Created attachment 9676 [details]
screenshot

It looks like check boxes in the app settings are marked as required. Which result in the error "please check this box if you want to proceed" for any unchecked box.

Note: this only happens on pressing Enter and NOT then using the apply changes button.
Comment 1 Jens Pelzetter univentionstaff 2018-11-07 11:31:34 CET
For some widgets the data from which the widgets are generated contained the value null for the required property. Because of this the checkboxes in the HTML had a required="null" attribute. The required attribute is a value less attribute, the mere presence of the attribute activates the build in check of the browser. Solved by checking for null values in AppSettings.py and setting the required property of the widgets to false instead of null.
Comment 2 Dirk Wiesenthal univentionstaff 2018-11-30 15:29:02 CET
Ok, works now
Comment 3 Arvid Requate univentionstaff 2018-12-05 14:39:18 CET
<http://errata.software-univention.de/ucs/4.3/345.html>