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

Calendar day view does not always show all events on a day

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 3.0.3
    • 2.9.6, 3.0.4
    • Calendar
    • MOODLE_30_STABLE
    • MOODLE_29_STABLE, MOODLE_30_STABLE
    • MDL-53757_master
    • Hide
      Test 1
      1. Create 2 entries in calendar at different times on the same day (today).
      2. In the upcoming events block, click on the linked word 'today' for the first, earlier, event.
      3. Ensure all events for the day are shown.
      4. Click on 'Dashboard' to go back to the front page.
      5. In the upcoming events block, click on the linked word 'today' for the second, later, event.
      6. Ensure all events for the day are shown.
      Show
      Test 1 Create 2 entries in calendar at different times on the same day (today). In the upcoming events block, click on the linked word 'today' for the first, earlier, event. Ensure all events for the day are shown. Click on 'Dashboard' to go back to the front page. In the upcoming events block, click on the linked word 'today' for the second, later, event. Ensure all events for the day are shown.

    Description

      Calendar day view takes a time as input parameter. While day view purports to show the calendar for the day, it actually only shows events from the timestamp provided onwards (although the time provided is not displayed). This is massively confusing for students, and has caused some to miss lectures.

      The logic around timestamp calculation at the start of calendar/view.php is contorted and not obvious. I assume that in introducing this behaviour, the author was primarily considering the "upcoming" view, for which this behaviour would be correct.

      Attachments

        Activity

          People

            markn Mark Nelson
            nwpotago Nick Phillips
            Mark Nelson Mark Nelson
            David Monllaó David Monllaó
            Jake Dallimore Jake Dallimore
            Andrew Lyons, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:
              9/May/16