Moodle
  1. Moodle
  2. MDL-38113

Blind Marking grader email reveals submitters details

    Details

    • Testing Instructions:
      Hide
      1. Test prerequisites (working email)
      2. Create an assignment with blind marking enabled in a course with at least one teacher and at least 2 students.
      3. In assignment settings, make sure notify grader about submissions is set to "Yes" and "File submissions" are enabled.
      4. Make sure the teacher can receive emails from this Moodle (email is working)
      5. Login as a student and make a submission to the assignment
      6. Login as admin and go to admin/cron.php to trigger cron
      7. Verify the teacher received a notification by email about the student submission
      8. Verify the email does not reveal the identity of the student.
      Show
      Test prerequisites (working email) Create an assignment with blind marking enabled in a course with at least one teacher and at least 2 students. In assignment settings, make sure notify grader about submissions is set to "Yes" and "File submissions" are enabled. Make sure the teacher can receive emails from this Moodle (email is working) Login as a student and make a submission to the assignment Login as admin and go to admin/cron.php to trigger cron Verify the teacher received a notification by email about the student submission Verify the email does not reveal the identity of the student.
    • Affected Branches:
      MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE
    • Pull from Repository:
    • Pull 2.4 Branch:
      mdl38113-moodle24
    • Pull Master Branch:
      mdl38113-master
    • Rank:
      47931

      Description

      Blind Marking functionally notifies the graders once a submission has been made with their Moodle details and revealing their identity.

        Issue Links

          Activity

          Hide
          Damyon Wiese added a comment -

          Thanks for reporting this.

          I've put that on the backlog.

          In the meantime feel free to help us work on this issue. If you are able to provide a patch or links to your Git repository branch, please add a patch label so we will spot it.

          Show
          Damyon Wiese added a comment - Thanks for reporting this. I've put that on the backlog. In the meantime feel free to help us work on this issue. If you are able to provide a patch or links to your Git repository branch, please add a patch label so we will spot it.
          Hide
          Damyon Wiese added a comment -

          Thanks Tim!

          Show
          Damyon Wiese added a comment - Thanks Tim!
          Hide
          Tim Lock added a comment -

          Show
          Tim Lock added a comment -
          Hide
          Damyon Wiese added a comment -

          This patch looks good Tim. I just had to add testing instructions to the issue. Sending for integration.

          Show
          Damyon Wiese added a comment - This patch looks good Tim. I just had to add testing instructions to the issue. Sending for integration.
          Hide
          Damyon Wiese added a comment -

          The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

          Thanks!

          Show
          Damyon Wiese added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. Thanks!
          Hide
          Dan Poltawski added a comment -

          Integrated to master and 24.

          The branches seemed to have lots of other changes on, so I just cherry-picked the patches.

          Show
          Dan Poltawski added a comment - Integrated to master and 24. The branches seemed to have lots of other changes on, so I just cherry-picked the patches.
          Hide
          Rajesh Taneja added a comment -

          Thanks for fixing this Tim,

          Email doesn't reveal any user information now.

          Show
          Rajesh Taneja added a comment - Thanks for fixing this Tim, Email doesn't reveal any user information now.
          Hide
          Rajesh Taneja added a comment -

          Sorry guys just realized this patch is not sending correct mail to student.
          Student gets mail from "Participant 3<noreply@testsite>" which seems incorrect.

          Will open another issue to fix this.

          Show
          Rajesh Taneja added a comment - Sorry guys just realized this patch is not sending correct mail to student. Student gets mail from "Participant 3<noreply@testsite>" which seems incorrect. Will open another issue to fix this.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Because

          A
          MARVELOUS
          A       U
          Z  YOU  P
          I  ARE  E
          N  PPL  R
          G       B
            TNKS! 
          

          Closing, ciao

          Show
          Eloy Lafuente (stronk7) added a comment - Because A MARVELOUS A U Z YOU P I ARE E N PPL R G B TNKS! Closing, ciao

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: