It looks like the site administrator is a required field as a site must have a least one administrator. This pieform relies on the required field to validate that there is at least one admin selected. By taking out the rule that makes it required this validation is skipped and it's possible that someone may try to remove all admins and save resulting in a database error (see the attached screen shot 'no-administrators.png')
We could try to remove this required field off the form element and use a validation function on the form instead to catch this behaviour. -- You received this bug notification because you are a member of Mahara Contributors, which is subscribed to Mahara. Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it! https://bugs.launchpad.net/bugs/1894614 Title: Change lang string for the search for staff and admins Status in Mahara: In Progress Bug description: When you are on Admin menu -> Institutions -> Staff / Administrators, you see above the search field 'Site staff' and 'Site administrator' respectively, but you would actually just search for a person to potentially become a staff member or admin. Therefore, it would be better to say 'Search for a person'. This also applies to Admin menu -> People -> Site staff/ Site administrators. Question: Why is the site administrator search a required field? You can just pick from the people in the box? To manage notifications about this bug go to: https://bugs.launchpad.net/mahara/+bug/1894614/+subscriptions _______________________________________________ Mailing list: https://launchpad.net/~mahara-contributors Post to : [email protected] Unsubscribe : https://launchpad.net/~mahara-contributors More help : https://help.launchpad.net/ListHelp
