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

Replace add_to_log with an event trigger - grade

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.6
    • Fix Version/s: 2.7
    • Component/s: Gradebook
    • Labels:
    • Testing Instructions:
      Hide

      1/execute phpunit tests
      2/ test all 3 ways to alter grades in gradebook - normal grader report, ajax grader report (see grader report settings) and individual grade edit form
      3/ verify the log entries are added to the mdl_logstore_standard_log table if the store is enabled

      Show
      1/execute phpunit tests 2/ test all 3 ways to alter grades in gradebook - normal grader report, ajax grader report (see grader report settings) and individual grade edit form 3/ verify the log entries are added to the mdl_logstore_standard_log table if the store is enabled
    • Affected Branches:
      MOODLE_26_STABLE
    • Fixed Branches:
      MOODLE_27_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      w14_MDL-40910_m27_usergraded
    • Story Points:
      13
    • Sprint:
      BACKEND Sprint 12

      Description

      Replace the add_to_log calls for the following area, with a call to the get_legacy_logdata function in the event class.

      grade
      ./grade/report/grader/ajax_callbacks.php:127: add_to_log($course->id, 'grade', 'update', $url, $info);
      ./grade/report/grader/lib.php:309: add_to_log($this->course->id, 'grade', 'update', $url, $info);
      ./grade/edit/tree/grade.php:212: add_to_log($course->id, 'grade', 'update', $url, $info);
      ./grade/edit/tree/grade.php:239: add_to_log($course->id, 'grade', 'update', $url, $info);

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              skodak Petr Skoda
              Reporter:
              abgreeve Adrian Greeve
              Peer reviewer:
              Ankit Agarwal
              Integrator:
              Marina Glancy
              Tester:
              Michael de Raadt
              Participants:
              Component watchers:
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Sujith Haridasan
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

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