Fix feedback section errors when scorecalculation is empty

PoetOS · Dec 21, 2022 · 80d8bd9 · 80d8bd9
1 parent d202a7b
commit 80d8bd9
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/classes/feedback/section.php b/classes/feedback/section.php
@@ -97,7 +97,7 @@ public static function new_section($surveyid, $sectionlabel = '') {
         $newsection->surveyid = $surveyid;
         $newsection->section = $maxsection + 1;
         $newsection->sectionlabel = $sectionlabel;
-        $newsection->scorecalculation = '';
+        $newsection->scorecalculation = $newsection->encode_scorecalculation([]);
         $newsecid = $DB->insert_record(self::TABLE, $newsection);
         $newsection->id = $newsecid;
         $newsection->scorecalculation = [];

diff --git a/classes/question/sectiontext.php b/classes/question/sectiontext.php
@@ -101,9 +101,10 @@ protected function question_survey_display($response, $descendantsdata, $blankqu
 
         // In which section(s) is this question?
         foreach ($fbsections as $key => $fbsection) {
-            $scorecalculation = unserialize($fbsection->scorecalculation);
-            if (array_key_exists($this->id, $scorecalculation)) {
-                array_push($filteredsections, $fbsection->section);
+            if ($scorecalculation = unserialize($fbsection->scorecalculation)) {
+                if (array_key_exists($this->id, $scorecalculation)) {
+                    array_push($filteredsections, $fbsection->section);
+                }
             }
         }