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

The day_exporter class unnecessarily defines and calls get_course_filter_selector()

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • 3.4
    • 3.4
    • Calendar
    • MOODLE_34_STABLE
    • MOODLE_34_STABLE
    • MDL-60564_master
    • Hide
      1. Create a course.
      2. Enrol the admin as a student in the course.
      3. Go to the course calendar and add a few events (some course, some user, some whatever).
      4. Go to the home page and add the calendar block if it isn't there.
      5. Check the block displays correctly and the events are listed.
      6. Navigate between months
      7. Check that the months update both here and in the three-month view
      8. View a specific day by clicking the date on a day with an event
      9. Check that the events are listed properly
      10. Use the nav links to change days
      11. Check that the events are listed properly
      Show
      Create a course. Enrol the admin as a student in the course. Go to the course calendar and add a few events (some course, some user, some whatever). Go to the home page and add the calendar block if it isn't there. Check the block displays correctly and the events are listed. Navigate between months Check that the months update both here and in the three-month view View a specific day by clicking the date on a day with an event Check that the events are listed properly Use the nav links to change days Check that the events are listed properly

      The function get_course_filter_selector() in day_exporter.php copies the exact logic in the function course_filter_selector() in calendar/renderer.php. The other exporters use the renderer appropriately.

      There is also the random comment // TODO remove this code on MDL-60096 in the code, but MDL-60096 is closed.

            markn Mark Nelson
            markn Mark Nelson
            Andrew Lyons Andrew Lyons
            Damyon Wiese Damyon Wiese
            Simey Lameze Simey Lameze
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved:

                Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.