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

Weekly stats breaking after MDL-41284

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. Enable stats
      2. run cron, if you don't see "Running daily statistics gathering", adjust your system clock for some hours ahead, repeat running cron until you got the message
      3. adjust the system clock plus one day, run cron again
      4. repeat until you saw in cron messages "Running weekly statistics gathering" and "Running monthly statistics gathering"
      Show
      Enable stats run cron, if you don't see "Running daily statistics gathering", adjust your system clock for some hours ahead, repeat running cron until you got the message adjust the system clock plus one day, run cron again repeat until you saw in cron messages "Running weekly statistics gathering" and "Running monthly statistics gathering"
    • Affected Branches:
      MOODLE_27_STABLE
    • Fixed Branches:
      MOODLE_27_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      wip-MDL-46545-master
    • Pull Master Diff URL:

      Description

      Stats cron breaks with database writing error on lib/statslib.php line 699.
      The problem is in lib/statslib.php on line 704, where is missing userid column.
      This totally breaks statistics calculations.
      The same for monthly stats line 864

        Attachments

          Issue Links

          is a regression caused by

          Improvement - An enhancement to an existing Moodle feature. MDL-41284 Convert Statistics report to new logging API

          • Major - Major loss of function, incorrect output
          • Closed
          Testing discovered

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-46855 Cover stats_cron_weekly() and stats_cron_monthly() with unittests

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

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-46239 Scheduled task manager does not properly handle timezones

          • Critical - Crashes server, loss of data, severe memory leak
          • Closed

            Activity

              People

              Assignee:
              marina Marina Glancy
              Reporter:
              vadimon Vadim Dvorovenko
              Peer reviewer:
              Ankit Agarwal Ankit Agarwal
              Integrator:
              Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
              Tester:
              Frédéric Massart Frédéric Massart
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Sara Arjona (@sarjona), Andrew Lyons, Dongsheng Cai, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                8/Sep/14