Display the answer note in the list of deployed variants.

doc/en/Developer/Development_track.md

@@ -60,11 +60,10 @@ At this point STACK will be "ready" for use with students, although not all feat
 5. Validation of student inputs has too many brackets.  This is a Maxima issue.  To reproduce it type `simp:false;` then `-3*x^2`.  We get unneeded brackets.
 6. Form validation should reject a PRT where Node x next -> Node x. Actually, it should validate that we have a connected DAG.
 7. Find a way to make the answer test test-suite and input test test-suite available to question authors.
-8. Display the answer note in the list of deployed variants.
-9. Provide a way back from the question tests to the question preview window (or wherever).
-10. When validating the editing form, actually evaluate the Maxima code.
-11. When validating the editing form, ensure there are no @ and $ in the fields that expect Maxima code.
-12. Button to remove a node from a PRT.
+8. Provide a way back from the question tests to the question preview window (or wherever).
+9. When validating the editing form, actually evaluate the Maxima code.
+10. When validating the editing form, ensure there are no @ and $ in the fields that expect Maxima code.
+11. Button to remove a node from a PRT.
 
 ## Future plans
 

lang/en/qtype_stack.php

@@ -213,7 +213,7 @@ $string['deletetestcase'] = 'Delete test case {$a->no} for question {$a->questio
 $string['deletetestcaseareyousure'] = 'Are you sure you want to delete test case {$a->no} for question {$a->question}?';
 $string['deletethistestcase'] = 'Delete this test case...';
 $string['deploy'] = 'Deploy';
-$string['deployedvariantoptions'] = 'The following variants have been deployed: {$a}';
+$string['deployedvariantoptions'] = 'The following variants have been deployed:';
 $string['deployedvariants'] = 'Deployed variants';
 $string['editingtestcase'] = 'Editing test case {$a->no} for question {$a->question}';
 $string['editthistestcase'] = 'Edit this test case...';

questiontestrun.php

@@ -111,7 +111,14 @@ if (!$question->has_random_variants()) {
     echo html_writer::tag('p', get_string('questionnotdeployedyet', 'qtype_stack'));
 
 } else {
-    $seedchoices = array();
+
+    $notestable = new html_table();
+    $notestable->head = array(
+        get_string('deployedvariants', 'qtype_stack'),
+        get_string('questionnote', 'qtype_stack'),
+    );
+    $prtstable->attributes['class'] = 'generaltable stacktestsuite';
+
     foreach ($question->deployedseeds as $deployedseed) {
         if (!is_null($question->seed) && $question->seed == $deployedseed) {
             $choice= html_writer::tag('b', $deployedseed,
@@ -129,10 +136,25 @@ if (!$question->has_random_variants()) {
                 new pix_icon('t/delete', get_string('undeploy', 'qtype_stack')));
         }
 
-        $seedchoices[] = $choice;
+        // Print out question notes of all deployed versions
+        $qn = question_bank::load_question($questionid);
+        $qn->seed = (int) $deployedseed;
+        $cn = $qn->get_context();
+        $qunote = question_engine::make_questions_usage_by_activity('qtype_stack', $cn);
+        $qunote->set_preferred_behaviour('adaptive');
+        $slotnote = $qunote->add_question($qn, $qn->defaultmark);
+        $qunote->start_question($slotnote);
+
+        $notestable->data[] = array(
+            $choice,
+            $qn->get_question_summary(),
+        );
+    
+    
     }
-    echo html_writer::tag('p', get_string('deployedvariantoptions', 'qtype_stack',
-            implode(' | ', $seedchoices)));
+
+    echo html_writer::tag('p', get_string('deployedvariantoptions', 'qtype_stack'));
+    echo html_writer::table($notestable);
 }
 
 if (!$variantmatched) {

stack/cas/cassession.class.php

@@ -89,7 +89,7 @@ class stack_cas_session {
             if (is_int($seed)) {
                 $this->seed = $seed;
             } else {
-                throw new Exception('stack_cas_session: $seed must be a number.');
+                throw new Exception('stack_cas_session: $seed must be a number.  Got "'.$seed.'"');
             }
         } else {
             $this->seed = time();