xAPI-vle / moodle-logstore_xapi

A Moodle plugin to send xAPI statements to an LRS using events in the Moodle logstore.
GNU General Public License v3.0
75 stars 85 forks source link

PR-timezonefix: Set the timezone back to site default after the manua… #775

Closed cwarwicker closed 1 year ago

cwarwicker commented 3 years ago

…l Europe/London change.

Previously this set the timezone to Europe/London outside of this function, which was causing problems elsewhere in the system, as once the plugin settings were all loaded by Moodle, the timezone was changing from the site default. So this change puts it inside the function and then sets it back to the site default when we are done.

Description

(If the site default is Europe/Oslo)

On a page which does not call admin_externalpage_setup() the timezone is always Europe/Oslo However on a page which does call admin_externalpage_setup() various other calls are made to plugins and the timezone is overwritten as Europe/London.

PR Type