Skip to content

fix: do not issue certs to suspended users#781

Open
rhell4 wants to merge 1 commit intomdjnelson:MOODLE_404_STABLEfrom
catalyst:MOODLE_404_STABLE-suspended
Open

fix: do not issue certs to suspended users#781
rhell4 wants to merge 1 commit intomdjnelson:MOODLE_404_STABLEfrom
catalyst:MOODLE_404_STABLE-suspended

Conversation

@rhell4
Copy link
Contributor

@rhell4 rhell4 commented Mar 18, 2026

Currently there is a scenario where a suspended user could be issued a cert.

  1. Student is enrolled in course
  2. Student is later suspended
  3. Teacher manually removes the issued cert for the student (or the student didn't have a cert issues to them yet)
  4. Issue certificates task runs
  5. Suspended student is issued a cert
  6. If "Email students" was turned on the suspended student receives an email, leading to confusion

I think just a simple check of the suspended flag and skipping that user should suffice. I've also adjusted the unit test to cover this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant