MDL-67132 mod_lti: document new param
authorClaude Vervoort <claude.vervoort@cengage.com>
Thu, 13 Feb 2020 20:19:48 +0000 (15:19 -0500)
committerClaude Vervoort <claude.vervoort@cengage.com>
Wed, 26 Feb 2020 02:22:45 +0000 (21:22 -0500)
lib/grade/grade_item.php
lib/upgrade.txt
mod/lti/service/gradebookservices/classes/local/service/gradebookservices.php

index 3bb5812..f67c71c 100644 (file)
@@ -1732,6 +1732,7 @@ class grade_item extends grade_object {
      * @param string $feedback Optional teacher feedback
      * @param int $feedbackformat A format like FORMAT_PLAIN or FORMAT_HTML
      * @param int $usermodified The ID of the user making the modification
+     * @param int $timemodified Optional parameter to set the time modified, if not present current time.
      * @return bool success
      */
     public function update_final_grade($userid, $finalgrade = false,
@@ -1804,7 +1805,7 @@ class grade_item extends grade_object {
         $gradechanged = false;
         if (empty($grade->id)) {
             $grade->timecreated = null;   // Hack alert - date submitted - no submission yet.
-            $grade->timemodified = $timemodified ?: time(); // Hack alert - date graded.
+            $grade->timemodified = $timemodified ?? time(); // Hack alert - date graded.
             $result = (bool)$grade->insert($source);
 
             // If the grade insert was successful and the final grade was not null then trigger a user_graded event.
@@ -1828,7 +1829,7 @@ class grade_item extends grade_object {
                 return $result;
             }
 
-            $grade->timemodified = $timemodified?:time(); // Hack alert - date graded.
+            $grade->timemodified = $timemodified ?? time(); // Hack alert - date graded.
             $result = $grade->update($source);
 
             // If the grade update was successful and the actual grade has changed then trigger a user_graded event.
index 58a2ef1..1863b78 100644 (file)
@@ -32,6 +32,7 @@ information provided here is intended especially for developers.
 * The database drivers (moodle_database and subclasses) don't need to implement get_columns() anymore.
   They have to implement fetch_columns instead.
 * Added function cleanup_after_drop to the database_manager class to take care of all the cleanups that need to be done after a table is dropped.
+* grade_item::update_final_grade() can now take an optional parameter to set the grade->timemodified. If not present the current time will carry on being used.
 
 === 3.8 ===
 * Add CLI option to notify all cron tasks to stop: admin/cli/cron.php --stop
index c7e852b..c73df89 100644 (file)
@@ -394,7 +394,7 @@ class gradebookservices extends service_base {
             $grade->feedbackformat = $feedbackformat;
             $grade->feedback = $feedback;
             $grade->rawgrade = $finalgrade;
-            $status = \grade_update($source, $gradeitem->courseid,
+            $status = grade_update($source, $gradeitem->courseid,
                          $gradeitem->itemtype, $gradeitem->itemmodule,
                          $gradeitem->iteminstance, $gradeitem->itemnumber,
                          $grade);