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

Reimplement access to log table in backup/util/helper/backup_cron_helper.class.php

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.6
    • Fix Version/s: 2.7
    • Component/s: Backup, Logging
    • Labels:
    • Testing Instructions:
      Hide
      1. Set your site's automated backup config, to run manually and to skip backup if course is not modified (My home ► Site administration ► Courses ► Backups ► Automated backup setup)
      2. Run php admin/cli/automated_backups.php and make sure no error is noticed and backups happen successfully.
      3. Run the script again, and make sure all courses are skipped.
      4. Login as admin and add a module in a course.
      5. Run the script again and make sure all courses except the one just modified is skipped.
      6. Run unit tests in admin/tool/log/store/legacy/tests/store_test.php and make sure they pass.
      Show
      Set your site's automated backup config, to run manually and to skip backup if course is not modified (My home ► Site administration ► Courses ► Backups ► Automated backup setup) Run php admin/cli/automated_backups.php and make sure no error is noticed and backups happen successfully. Run the script again, and make sure all courses are skipped. Login as admin and add a module in a course. Run the script again and make sure all courses except the one just modified is skipped. Run unit tests in admin/tool/log/store/legacy/tests/store_test.php and make sure they pass.
    • Affected Branches:
      MOODLE_26_STABLE
    • Fixed Branches:
      MOODLE_27_STABLE
    • Epic Link:
    • Pull Master Branch:
      MDL-42892-master_42890
    • Story Points:
      13
    • Sprint:
      BACKEND Sprint 11

      Description

      Currently this class uses the legacy logging table. It needs to be upgraded to use the new logging API.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              ankit_frenz Ankit Agarwal
              Reporter:
              salvetore Michael de Raadt
              Peer reviewer:
              Petr Skoda
              Integrator:
              Marina Glancy
              Tester:
              Michael de Raadt
              Participants:
              Component watchers:
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                12/May/14