search

davosmith / moodle-checklist

Checklist plugin for Moodle - allows a teacher to create a checklist for their students to tick-off
GNU General Public License v3.0
24 stars 68 forks source link

Show reset report only when required #83

Closed jboulen closed 3 years ago

jboulen commented 3 years ago

Currently, when you reset your course, even if you don't check the box to reset data around mod_checklist, Moodle shows a message that "Reset checklist progress and user items" is OK. Actually, this message is wrong. The checklist data is kept as expected, there is no data loss.

With this modification, an empty array is returned when user does not request to reset checklist data.

davosmith commented 3 years ago

Sorry it took a while to look at this - it was a bit harder to understand from the diff (due the the whitespace changes), so I needed a few spare minutes (and a bit of clear headspace) to look at it properly.

The fix looks great, so thank for submitting it - now merged.