Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-30059

Assignment Submission Notification Should Use Teacher's Timezone Rather Than Server Time

    Details

    • Testing Instructions:
      Hide

      As a teacher, change your timezone setting in your profile to something other than the server timezone.
      Create an assignment and elect to receive notifications of submissions.
      As a student, submit the assignment.
      Check your email account used for the teacher.
      You should have a notification of the submission and the submission time found in the body of the message should be relative to the teacher's timezone setting.

      Show
      As a teacher, change your timezone setting in your profile to something other than the server timezone. Create an assignment and elect to receive notifications of submissions. As a student, submit the assignment. Check your email account used for the teacher. You should have a notification of the submission and the submission time found in the body of the message should be relative to the teacher's timezone setting.
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      wip-MDL-30059-master

      Description

      The assignment submission notifications that are sent to teachers include the submission time. The time is given according to the server's time settings. When the server is in a different timezone, the time in the email is wrong. The email notification should adjust the submission time to the teacher's timezone since the teacher is the recipient of the message.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            cfollin Chris Follin added a comment -

            Attaching patch that adjusts the submission time for the message body to the teacher's timezone.

            Show
            cfollin Chris Follin added a comment - Attaching patch that adjusts the submission time for the message body to the teacher's timezone.
            Hide
            salvetore Michael de Raadt added a comment -

            Thanks for putting this forward.

            I've added this to the STABLE backlog, but please keep pushing it towards integration.

            Show
            salvetore Michael de Raadt added a comment - Thanks for putting this forward. I've added this to the STABLE backlog, but please keep pushing it towards integration.
            Hide
            phalacee Jason Fowler added a comment -

            Hi Chris, the code looks fine. Do you have the ability to create a branch on github with this change for 2.0, 2.1 and 2.2dev?

            Show
            phalacee Jason Fowler added a comment - Hi Chris, the code looks fine. Do you have the ability to create a branch on github with this change for 2.0, 2.1 and 2.2dev?
            Hide
            cfollin Chris Follin added a comment -

            Jason, I don't think I do.

            Show
            cfollin Chris Follin added a comment - Jason, I don't think I do.
            Hide
            bushido Mark Nielsen added a comment -

            Added github URLs

            Show
            bushido Mark Nielsen added a comment - Added github URLs
            Hide
            phalacee Jason Fowler added a comment -

            Updated all relevant branches with changes

            Show
            phalacee Jason Fowler added a comment - Updated all relevant branches with changes
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Thanks guys, this has been integrated now.

            Jason just as a heads up I had to cherry-pick the 20 stable branch as you had included your fix for another issue in with that branch.

            Cheers
            Sam

            Show
            samhemelryk Sam Hemelryk added a comment - Thanks guys, this has been integrated now. Jason just as a heads up I had to cherry-pick the 20 stable branch as you had included your fix for another issue in with that branch. Cheers Sam
            Hide
            rajeshtaneja Rajesh Taneja added a comment -

            Notification now has time as per Teacher's timezone.
            Thanks for fixing this Chris

            Show
            rajeshtaneja Rajesh Taneja added a comment - Notification now has time as per Teacher's timezone. Thanks for fixing this Chris
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Yes, you got this finally upstream, just in time for Moodle 2.2beta. Congrats and thanks!

            Ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Yes, you got this finally upstream, just in time for Moodle 2.2beta. Congrats and thanks! Ciao

              People

              • Votes:
                1 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  28/Nov/11