Comment 5 for bug 1843149

Reviewed: https://reviews.mahara.org/10337
Committed: https://git.mahara.org/mahara/mahara/commit/9dc7cbc6a3a6ecdaf132888c6d755a5af20496b7
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit 9dc7cbc6a3a6ecdaf132888c6d755a5af20496b7
Author: Lisa Seeto <email address hidden>
Date: Thu Sep 12 12:08:59 2019 +1200

Bug 1843149: Bulk import doesn't retain user details correctly

This change does the following:
- Gets rid of the static cache that was created during run
of the first record which would only save the first user's info.
- when username is taken, append numbers and not underscores
- get rid of the warning using count on globals var that were not
previously set to array()s.
- respects the email users about their accounts option

behatnotneeded

Change-Id: I57b1c5e4d401028defda0c35bfc8347149094eb2
Signed-off-by: Lisa Seeto <email address hidden>