-
Bug
-
Resolution: Fixed
-
Minor
-
3.4.4, 3.5.1, 3.6
-
MOODLE_34_STABLE, MOODLE_35_STABLE, MOODLE_36_STABLE
-
MOODLE_34_STABLE, MOODLE_35_STABLE
-
MDL-62920-master -
- Create a cohort on your site
- Bulk upload users using: Test user upload.csv
- Select 'Add new users' and complete the bulk upload
- In the database, set the 'deleted' field for the new user to 1
- Bulk upload the users again, but this time select 'Update existing users' and set 'Select for bulk user actions' to 'All users'
- After the upload is complete, with the selected users choose 'Add to cohort'
- Attempt to add the users to a cohort
(A similar result would occur if instead of manually setting the 'deleted' field to 1, you deleted the user between the upload completing and selecting 'Add to cohort').
Expected outcome:
- The deleted user is ignored (and, if there had been any other users in the upload they would have been added to the cohort)
Actual outcome:
- Fatal error as line 69 of user_bulk_cohortadd.php only loads users who are not deleted, but does not check to see if a valid record has been returned, before trying to process it.
- will be (partly) resolved by
-
MDL-60160 Task cohort_role_sync attempts to sync deleted users
- Development in progress