Item11705: Bulk registration crashes with uninitialized variable if Password column provided, and other failures.
Priority: Normal
Current State: Closed
Released In: 2.0.0
Target Release: major
If a Password is provided, the code compares
$row->{Password}
to
$row->{Confirm}
, and crashes.
- If using bulk registration it doesn't make sense to confirm both passwords, so temporarily disable confirmation.
- Use of the
Password
column is not documented.
- Other validations are logged as fail, but actually ignored. User is still registered if minimum password length test fails.
See
Support.Question1064
--
GeorgeClark - 29 Mar 2012
Ugh. The
ManageDotPm::verify_bulkRegister
tests don't really validate much. No checking that users were actually created, or that the log was created without errors.
--
GeorgeClark - 18 Jun 2014
The issue with ignored validation failures. "return" in a catch block doesn't return to the calling routine, but continues on in-line.
--
GeorgeClark - 18 Jun 2014