From e3a90202ab0154ec1a6c1a87aca8357e51735f67 Mon Sep 17 00:00:00 2001
From: Vincent Vanwaelscappel <vincent@cubedesigners.com>
Date: Fri, 29 Jan 2021 17:18:21 +0100
Subject: [PATCH] wait #4231 @3

---
 app/Models/Quiz.php       | 10 +++++++---
 resources/quiz/js/main.js | 13 +++----------
 2 files changed, 10 insertions(+), 13 deletions(-)

diff --git a/app/Models/Quiz.php b/app/Models/Quiz.php
index 5eae05606..543ca8382 100644
--- a/app/Models/Quiz.php
+++ b/app/Models/Quiz.php
@@ -303,10 +303,14 @@ class Quiz extends CubistMagicAbstractModel
         $d['countriesList'] = \Cubist\Locale\Country::getList($translation->getAttribute('locale'));
 
         // Fix boolean
-        $booleans = ['multiple', 'required', 'count_for_score'];
+        $booleans = ['multiple' => false, 'required' => true, 'count_for_score' => true];
         foreach ($d['questions'] as $qn => $q) {
-            foreach ($booleans as $b) {
-                $d['questions'][$qn][$b] = !!($q[$b]);
+            foreach ($booleans as $b => $default) {
+                if (!isset($d['questions'][$qn][$b])) {
+                    $d['questions'][$qn][$b] = $default;
+                } else {
+                    $d['questions'][$qn][$b] = !!($q[$b]);
+                }
             }
         }
 
diff --git a/resources/quiz/js/main.js b/resources/quiz/js/main.js
index acfa2e6de..3e53619ae 100644
--- a/resources/quiz/js/main.js
+++ b/resources/quiz/js/main.js
@@ -92,16 +92,13 @@
             countQuestions = 0;
             // Create questions
             $.each(DATA.questions, function (k, question) {
-                console.log(question);
                 if (question.count) {
                     countQuestions++;
                 }
-
-                var multipleclass = question.multiple ? ' multiple' : '';
-                var questionHtml = '<section class="question' + multipleclass + '" data-count="' + question.count + '" data-q="' + qn + '" data-min-score="' + question.min_score + '" data-type="' + question.type + '">';
+                var multipleClass = question.multiple ? ' multiple' : '';
+                var questionHtml = '<section class="question' + multipleClass + '" data-count="' + question.count_for_score + '" data-q="' + qn + '" data-min-score="' + question.min_score + '" data-type="' + question.type + '">';
                 questionHtml += '<div class="label"><h3>' + sprintf(getTranslation('question'), qn) + '</h3> ' + question.question + '</div>';
-                questionHtml += '<div class="answers">';
-                questionHtml += '</div>';
+                questionHtml += '<div class="answers"></div>';
                 if (question.multiple) {
                     questionHtml += '<a href="#" class="button validatemulti">' + getTranslation('validateAnswer') + '</a>';
                 } else if (question.type !== 'multiple') {
@@ -141,10 +138,6 @@
                         $(holder).append($(a));
                         an++;
                     });
-
-                    if (correctCount > 1 || $(this).attr('multiple') !== undefined) {
-                        q.addClass('multiple');
-                    }
                 } else if (question.type === 'text' || question.type === "email") {
                     $(holder).append('<input ' + required + ' type="' + question.type + '" name="answer" class="focusme" placeholder="' + question.placeholder + '" />')
                 } else if (question.type === 'textarea') {
-- 
2.39.5