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

Non backwards compatible change in calendar_get_events behaviour

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Blocker Blocker
    • 3.3
    • 3.3
    • Calendar

      In 3.3 we changed calendar_get_events such that it tries to respect user and group overrides and returns only the most applicable event.

      The details of this rely on the current logged in user, which is not good. Previously you could call it with something like: calendar_get_events(PHP_INT_MIN, PHP_INT_MAX, [3,4], true, true); and you would recieve all the events for users with id 3 and 4.

      Now it will only give you events for the current logged in user (in the case that the logged in user is neither 3 or 4 as in the example above, you will get back an empty array).

      A big part of the problem is that we attempt to solve the overrides problem with SQL, when really it should be done by filtering the results returned by the original query (probably).

            cameron1729 cameron1729
            cameron1729 cameron1729
            Ryan Wyllie Ryan Wyllie
            Andrew Lyons Andrew Lyons
            Damyon Wiese Damyon Wiese
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

              Created:
              Updated:
              Resolved:

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