MDL-47494 gapselect: Latest gapselect qtype fixes from Jamie. #23

question/type/gapselect/questionbase.php

@@ -286,7 +286,8 @@ abstract class qtype_gapselect_question_base extends question_graded_automatical
             $choice = $this->choices[$group][$choiceno];
             $parts[$place] = new question_classified_response(
                     $choiceno, html_to_text($choice->text, 0, false),
-                    $this->get_right_choice_for($place) == $response[$fieldname]);
+                    ($this->get_right_choice_for($place) == $response[$fieldname])
+                            / count($this->places));
         }
         return $parts;
     }

question/type/gapselect/questiontypebase.php

@@ -299,7 +299,8 @@ abstract class qtype_gapselect_base extends question_type {
             foreach ($question->choices[$group] as $i => $choice) {
                 $choices[$i] = new question_possible_response(
                         html_to_text($choice->text, 0, false),
-                        $question->rightchoices[$place] == $i);
+                        ($question->rightchoices[$place] == $i)
+                                / count($question->places));
             }
             $choices[null] = question_possible_response::no_response();
 

question/type/gapselect/styles.css

@@ -15,30 +15,6 @@
     padding-bottom: 0.5em;
 }
 
-.que.gapselect .group1 {
-    background-color: #E4F1FA;
-}
-.que.gapselect .group2 {
-    background-color: #DCDCDC;
-}
-.que.gapselect .group3 {
-    background-color: #B0C4DE;
-}
-.que.gapselect .group4 {
-    background-color: #D8BFD8;
-}
-.que.gapselect .group5 {
-    background-color: #87CEFA;
-}
-.que.gapselect .group6 {
-    background-color: #DAA520;
-}
-.que.gapselect .group7 {
-    background-color: #FFD700;
-}
-.que.gapselect .group8 {
-    background-color: #F0E68C;
-}
 .que.sddl .control {
     padding: 0.2em;
 }