Teststrategies: add standarderror before stopping quiz

Even if the maximum question limit is reached, calculate the ability
and standarderror based on the last response before finishing.
This means moving the `updatepersonability` and `addscalestandarderror`
classes to the top of the pre-select task lists.

classes/teststrategy/strategy/classicalcat.php

@@ -71,13 +71,13 @@ class classicalcat extends strategy {
      */
     public function get_preselecttasks(): array {
         return [
+            updatepersonability::class,
+            addscalestandarderror::class,
             maximumquestionscheck::class,
             removeplayedquestions::class,
             noremainingquestions::class,
             mayberemovescale::class,
             fisherinformation::class,
-            updatepersonability::class,
-            addscalestandarderror::class,
             strategyclassicscore::class,
         ];
     }

classes/teststrategy/strategy/inferallsubscales.php

@@ -75,8 +75,9 @@ class inferallsubscales extends strategy {
      */
     public function get_preselecttasks(): array {
         return [
-            maximumquestionscheck::class, // Cancel quiz attempt if we reached maximum of questions.
             updatepersonability::class,
+            addscalestandarderror::class,
+            maximumquestionscheck::class, // Cancel quiz attempt if we reached maximum of questions.
             removeplayedquestions::class,
             noremainingquestions::class,
             fisherinformation::class, // Add the fisher information to each question.
@@ -85,7 +86,6 @@ public function get_preselecttasks(): array {
             mayberemovescale::class, // Remove questions from excluded subscales.
             maybe_return_pilot::class,
             remove_uncalculated::class, // Remove items that do not have item parameters.
-            addscalestandarderror::class,
             filterbystandarderror::class,
             noremainingquestions::class, // Cancel quiz attempt if no questions are left.
             strategyfastestscore::class,

classes/teststrategy/strategy/inferlowestskillgap.php

@@ -75,8 +75,9 @@ class inferlowestskillgap extends strategy {
      */
     public function get_preselecttasks(): array {
         return [
-            maximumquestionscheck::class, // Cancel quiz attempt if we reached maximum of questions.
             updatepersonability::class,
+            addscalestandarderror::class,
+            maximumquestionscheck::class, // Cancel quiz attempt if we reached maximum of questions.
             removeplayedquestions::class,
             noremainingquestions::class,
             fisherinformation::class, // Add the fisher information to each question.
@@ -86,8 +87,6 @@ public function get_preselecttasks(): array {
             maybe_return_pilot::class,
             remove_uncalculated::class, // Remove items that do not have item parameters.
             noremainingquestions::class, // Cancel quiz attempt if no questions are left.
-            // Keep only questions that are assigned to the subscale where the user has the lowest ability.
-            addscalestandarderror::class,
             filterbystandarderror::class,
             strategydeficitscore::class,
         ];

classes/teststrategy/strategy/teststrategy_fastest.php

@@ -74,16 +74,16 @@ class teststrategy_fastest extends strategy {
      */
     public function get_preselecttasks(): array {
         return [
+            updatepersonability::class,
+            addscalestandarderror::class,
             maximumquestionscheck::class,
             removeplayedquestions::class,
             remove_uncalculated::class,
             noremainingquestions::class,
             mayberemovescale::class,
             firstquestionselector::class,
-            updatepersonability::class,
             lasttimeplayedpenalty::class,
             fisherinformation::class,
-            addscalestandarderror::class,
             strategyfastestscore::class,
         ];
     }