MDL-62535 grading methods: Fix for multiple intances.
authorAdrian Greeve <abgreeve@gmail.com>
Thu, 30 Aug 2018 03:35:52 +0000 (11:35 +0800)
committerAdrian Greeve <abgreeve@gmail.com>
Thu, 30 Aug 2018 03:35:52 +0000 (11:35 +0800)
We were only exporting the one instance per item id.
Instead we should have been exporting all instances for each
item id.

grade/grading/form/guide/classes/privacy/provider.php
grade/grading/form/rubric/classes/privacy/provider.php

index e22883f..34a7aaa 100644 (file)
@@ -83,7 +83,7 @@ class provider implements
                  WHERE gf.instanceid = :instanceid";
         $records = $DB->get_records_sql($sql, $params);
         if ($records) {
-            $subcontext = array_merge($subcontext, [get_string('guide', 'gradingform_guide')]);
+            $subcontext = array_merge($subcontext, [get_string('guide', 'gradingform_guide'), $instanceid]);
             writer::with_context($context)->export_data($subcontext, (object) $records);
         }
     }
index e3304fc..0739744 100644 (file)
@@ -72,7 +72,7 @@ class provider implements
                  WHERE rf.instanceid = :instanceid";
         $records = $DB->get_records_sql($sql, $params);
         if ($records) {
-            $subcontext = array_merge($subcontext, [get_string('rubric', 'gradingform_rubric')]);
+            $subcontext = array_merge($subcontext, [get_string('rubric', 'gradingform_rubric'), $instanceid]);
             \core_privacy\local\request\writer::with_context($context)->export_data($subcontext, (object) $records);
         }
     }