search

studentquiz / moodle-mod_studentquiz

Moodle-Plugin
GNU General Public License v3.0
38 stars 38 forks source link

The recipient name and the actor names are flipped in anonymous ranki… #423

Closed vuvanhieu143 closed 1 year ago

vuvanhieu143 commented 1 year ago

Hi @timhunt , I have updated the old logic to make sure we are not missing the case where the user is not enrolled in the course but has manager permission in the course (for example users like admin). I have added a unit test for this case as well. I can't write the provider function because we need data from the setUp.

Could you please review the changes?

timhunt commented 1 year ago

Thanks. Fix looks good. Merged.