Moodle
  1. Moodle
  2. MDL-31348

After creating/editing/deleting the view shown is based off GMT not the user's timezone

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.4, 2.2.1, 2.3
    • Fix Version/s: 2.1.5, 2.2.2
    • Component/s: Calendar
    • Labels:
    • Database:
      Any
    • Testing Instructions:
      Hide

      [Adding Event]
      1/ Go to your user profile and set timezone to GMT +12
      2/ Go to calendar and create an event of 'April fools day' on 01/04/12 at 00:00
      3/ Submit

      Expected Result:

      • Directed to the calendar page of 01/04/12
        Actual Result
      • Directed to the calendar page of 01/03/12

      [Removing Event]
      1/ Carry on from that test find the event on 01/04 in the calendar
      2/ Click the delete button on the event
      3/ Confirm the deletion when asked

      Expected Result:

      • Directed to calendar page of month of April
        Actual Result:
      • Directed to calendar page of month of March

      Repeat tests with 'server timezone'

      Show
      [Adding Event] 1/ Go to your user profile and set timezone to GMT +12 2/ Go to calendar and create an event of 'April fools day' on 01/04/12 at 00:00 3/ Submit Expected Result: Directed to the calendar page of 01/04/12 Actual Result Directed to the calendar page of 01/03/12 [Removing Event] 1/ Carry on from that test find the event on 01/04 in the calendar 2/ Click the delete button on the event 3/ Confirm the deletion when asked Expected Result: Directed to calendar page of month of April Actual Result: Directed to calendar page of month of March Repeat tests with 'server timezone'
    • Workaround:
      Hide

      Apply the attach diff to fix event.php and delete.php

      NOTE: this patch also includes the fix for the minicalendar side block not showing any events (year was not feed in as full date) noted in MDL-31339

      Show
      Apply the attach diff to fix event.php and delete.php NOTE: this patch also includes the fix for the minicalendar side block not showing any events (year was not feed in as full date) noted in MDL-31339
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE

      Description

      Ultimately if your timezone differs significantly from GMT (e.g. AEST) and you use the default midnight event time, you'll be shown the previous day's list. In deletion's case you'll see the previous month if you delete an event on the first of the month.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            Michael de Raadt added a comment -

            Thanks for spotting that and providing a solution.

            Show
            Michael de Raadt added a comment - Thanks for spotting that and providing a solution.
            Hide
            Dan Poltawski added a comment -

            Verified and sumibbitn for peer review

            Show
            Dan Poltawski added a comment - Verified and sumibbitn for peer review
            Hide
            Andrew Davis added a comment -

            Code looks fine. Expand the testing instructions to include both a user with a manually set timezone (+12 in the instructions) and also one with their timezone set to "Server's local time".

            Show
            Andrew Davis added a comment - Code looks fine. Expand the testing instructions to include both a user with a manually set timezone (+12 in the instructions) and also one with their timezone set to "Server's local time".
            Hide
            Eloy Lafuente (stronk7) added a comment -

            Integrated, thanks! (21, 22 & master)

            Show
            Eloy Lafuente (stronk7) added a comment - Integrated, thanks! (21, 22 & master)
            Hide
            Rajesh Taneja added a comment -

            Thanks Ben and Dan,
            Patch works Great

            FYI:
            In master and 22 calendar_print_event is depreciated and is used in /calendar/delete.php, which has been reported in MDL-31133. Hence linking the same to this issue.

            Show
            Rajesh Taneja added a comment - Thanks Ben and Dan, Patch works Great FYI: In master and 22 calendar_print_event is depreciated and is used in /calendar/delete.php, which has been reported in MDL-31133 . Hence linking the same to this issue.
            Hide
            Dan Poltawski added a comment -

            thanks Raj - I spotted that an meant to report it myself

            Show
            Dan Poltawski added a comment - thanks Raj - I spotted that an meant to report it myself
            Hide
            Eloy Lafuente (stronk7) added a comment -

            Well,

            I wish I said it every time
            you do the things you do.
            You always lend a helping hand,
            and I'm filled with gratitude.

            You are strong and generous
            for each and everyone one of us.
            I am eternally grateful,
            I cannot say thanks enough.

            Sorry for the (un)cool bit above, lol. Closing this as fixed. Ciao

            Show
            Eloy Lafuente (stronk7) added a comment - Well, I wish I said it every time you do the things you do. You always lend a helping hand, and I'm filled with gratitude. You are strong and generous for each and everyone one of us. I am eternally grateful, I cannot say thanks enough. Sorry for the (un)cool bit above, lol. Closing this as fixed. Ciao

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: