Uploaded image for project: '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 Master Branch:
      mdl38113-master

      Description

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

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              damyon 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 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 Damyon Wiese added a comment -

              Thanks Tim!

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

              Show
              tlock Tim Lock added a comment -
              Hide
              damyon 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 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 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 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
              poltawski 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
              poltawski 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
              rajeshtaneja Rajesh Taneja added a comment -

              Thanks for fixing this Tim,

              Email doesn't reveal any user information now.

              Show
              rajeshtaneja Rajesh Taneja added a comment - Thanks for fixing this Tim, Email doesn't reveal any user information now.
              Hide
              rajeshtaneja 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
              rajeshtaneja 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
              stronk7 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
              stronk7 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:
                    Fix Release Date:
                    11/Mar/13