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

Remove custom timezone handling when PHP5 is required

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Duplicate
    • 1.5, 1.5.1, 1.5.2, 1.5.3, 1.5.4, 1.6, 1.6.1, 1.6.2, 1.6.3, 1.6.4, 1.6.5, 1.7, 1.7.1, 1.7.2, 1.8, 1.8.1, 1.8.2
    • None
    • Libraries
    • MOODLE_15_STABLE, MOODLE_16_STABLE, MOODLE_17_STABLE, MOODLE_18_STABLE

    Description

      We have custom timezone handling code in Moodle versions 1.5 to 1.7. This was put in place to work around

      • A long-standing bug in PHP on Windows 95/98/Me where TZ info was missing and the TZ calls did not return an error. This was considered a critical problem. This bug did not apply to WindowsNT/2000 AFAIK.
      • Possibly outdated TZ info in all platforms. This is a minor issue.

      This naturally affected all PHP-based projects. From PHP5 onwards, PECL's timezonedb (see http://php.net/manual/en/ref.datetime.php ) addresses both problems.

      Moodle 1.8 is expected to require PHP5 so we can drop this nasty kludge.

      Attachments

        Issue Links

          duplicates

          New Feature - A new Moodle feature which has yet to be developed. MDL-49684 Replace custom Moodle timezone stuff with standard PHP date/time code

          • Critical - Crashes server, loss of data, severe memory leak
          • Closed
          has been marked as being related by

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-6795 Stats never runs. stats_monthly table gets very full

          • Major - Major loss of function, incorrect output
          • Closed
          will be (partly) resolved by

          Task - A task that needs to be completed. Tasks usually refer to work that must be done outside the product. MDL-7275 META BUG: Change minimum required PHP version to PHP 5

          • Minor - Minor loss of function where workaround is possible
          • Closed

          Activity

            People

              poltawski Dan Poltawski
              martinlanghoff Martín Langhoff
              David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: