New username isn't checked for spaces when added to LDAP
Bug #1689312 reported by
Matt Di Giuseppe
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Mahara |
Opinion
|
Medium
|
Kevin Dibble |
Bug Description
Mahara version: 16.10.3
When adding a new user, if you select an institution/
If you try to do this with a non LDAP authentication method, it thows an error and will not continue (although the error message does not highlight the username field as the source of the error in this case).
I've fixed it locally with trim() around the username value before it's put into the databse.
Changed in mahara: | |
milestone: | 17.10.0 → 18.04.0 |
Changed in mahara: | |
milestone: | 18.04.0 → 18.10.0 |
Changed in mahara: | |
milestone: | 18.10.0 → none |
tags: | added: bite-sized |
Changed in mahara: | |
assignee: | nobody → Kevin Dibble (digitalskill) |
status: | Confirmed → Opinion |
To post a comment you must log in.
Hi Matt,
Thanks for this report. If you'd like to submit your solution for Mahara core, we'd love to see it. You can find information on how to set up your connection to our code review system at https:/ /wiki.mahara. org/wiki/ Developer_ Area/Contributi ng_Code
Or since it's your first fix for Mahara, if you want to submit it here to Launchpad as a patch file, we can push it into our review system.
Cheers
Kristina