Change refence of file.itemid to questionnaire_response_file.id

lucaboesch · Apr 4, 2024 · 79f2b73 · 79f2b73
1 parent e4edc48
commit 79f2b73
Show file tree

Hide file tree

Showing 3 changed files with 45 additions and 8 deletions.
diff --git a/classes/question/file.php b/classes/question/file.php
@@ -81,7 +81,7 @@ protected function question_survey_display($formdata, $descendantsdata, $blankqu
         $elname = 'q' . $this->id;
         $draftitemid = file_get_submitted_draft_itemid($elname);
         $component = 'mod_questionnaire';
-        $options = $this->get_file_manager_option();
+        $options = self::get_file_manager_option();
         if ($draftitemid > 0) {
             file_prepare_draft_area($draftitemid, $this->context->id, $component, 'file', $this->id, $options);
         } else {
@@ -114,7 +114,7 @@ protected function question_survey_display($formdata, $descendantsdata, $blankqu
      *
      * @return array
      */
-    private function get_file_manager_option() {
+    public static function get_file_manager_option() {
         return [
             'mainfile' => '',
             'subdirs' => false,

diff --git a/classes/responsetype/file.php b/classes/responsetype/file.php
@@ -43,9 +43,12 @@ public static function answers_from_webform($responsedata, $question) {
             $record->responseid = $responsedata->rid;
             $record->questionid = $question->id;
 
-            file_save_draft_area_files($val, $question->context->id, 'mod_questionnaire', 'file', $question->id);
+            file_save_draft_area_files($val, $question->context->id,
+                'mod_questionnaire', 'file', $val,
+                \mod_questionnaire\question\file::get_file_manager_option());
             $fs = get_file_storage();
-            $files = $fs->get_area_files($question->context->id, 'mod_questionnaire', 'file', $question->id,
+            $files = $fs->get_area_files($question->context->id, 'mod_questionnaire',
+                'file', $val,
                 "itemid, filepath, filename",
                 false);
             if (!empty($files)) {
@@ -132,10 +135,44 @@ public function insert_response($responsedata) {
             $record->question_id = $this->question->id;
             $record->fileid = intval(clean_text($response->answers[$this->question->id][0]->value));
 
-            return $DB->insert_record(static::response_table(), $record);
-        } else {
-            return false;
+            // When saving the draft file, the itemid was the same as the draftfileid. This must now be
+            // corrected to the primary key that is questionaire_response_file.id to have a correct reference.
+            $recordid = $DB->insert_record(static::response_table(), $record);
+            if ($recordid) {
+                $olditem = $DB->get_record('files', ['id' => $record->fileid], 'itemid');
+                if (!$olditem) {
+                    return false;
+                }
+                $siblings = $DB->get_records('files',
+                    ['component' => 'mod_questionnaire', 'itemid' => $olditem->itemid]);
+                foreach ($siblings as $sibling) {
+                    if (!$this->fix_file_itemid($recordid, $sibling)) {
+                        return false;
+                    }
+                }
+                return $recordid;
+            }
         }
+        return false;
+    }
+
+    /**
+     * Update records in the table file with the new given itemid. To do this, the pathnamehash
+     * needs to be recalculated as well.
+     * @param int $recordid
+     * @param \stdClass $filerecord
+     * @return bool
+     * @throws \dml_exception
+     */
+    protected function fix_file_itemid(int $recordid, \stdClass $filerecord): bool {
+        global $DB;
+        $fs = get_file_storage();
+        $file = $fs->get_file_instance($filerecord);
+        $newhash = $fs->get_pathname_hash($filerecord->contextid, $filerecord->component,
+            $filerecord->filearea, $recordid, $file->get_filepath(), $file->get_filename());
+        $filerecord->itemid = $recordid;
+        $filerecord->pathnamehash = $newhash;
+        return $DB->update_record('files', $filerecord);
     }
 
     /**

diff --git a/lib.php b/lib.php
@@ -555,7 +555,7 @@ function questionnaire_pluginfile($course, $cm, $context, $filearea, $args, $for
             return false;
         }
     } else if ($filearea == 'file') {
-        if (!$DB->record_exists('questionnaire_response_file', ['response_id' => $componentid])) {
+        if (!$DB->record_exists('questionnaire_response_file', ['id' => $componentid])) {
             return false;
         }
     } else {