GH-723 Fix failing tests for PHP 8.3

lang/de/local_catquiz.php

@@ -174,9 +174,6 @@
 $string['central_host_help'] = 'Z.B. https://www.example.com';
 $string['central_scale_labels'] = 'Zentrale Skala-Labels';
 $string['central_scale_labels_desc'] = 'Ein Skala-Label pro Zeile eingeben. Nur diese Skalen werden vom Remote-Berechnungstask verarbeitet.';
-$string['syncbutton'] = 'Parameter abrufen';
-$string['sync_scale'] = 'Label der zu synchronisierenden CAT Skala';
-$string['sync_scale_desc'] = 'Parameter dieser Skala werden mit der zentralen Berechnungsinstanz synchronisiert.';
 $string['central_token'] = 'Token zum Zugriff auf die zentrale Berechnungsinstanz';
 $string['central_token_desc'] = 'Das webtoken, das für sie auf der zentralen Berechnungsinstanz eingerichtet worden ist';
 $string['central_token_help'] = 'Das webtoken, das für sie auf der zentralen Berechnungsinstanz eingerichtet worden ist';
@@ -621,13 +618,13 @@
 $string['reportscale'] = 'Skala für den Report der Ergebnisse berücksichtigen';
 $string['requesttimeout'] = 'Zeitüberschreitung beim Verbindungsaufbau';
 $string['response'] = 'Antwort';
-$string['responsesbyusercharttitle'] = 'Gesamtanzahl der gegebenen Antworten pro Person';
 $string['responses_added'] = 'Es wurden neue Antworten hinzugefügt';
 $string['responses_added_desc'] = '{$a->sourceurl} hat neue Antworten hinzugefügt. {$a->added} neue Antworten wurden hinzugefügt, '
     . '{$a->skipped} waren bereits vorhanden und es sind {$a->errors} Fehler aufgetreten';
 $string['responses_submitted'] = 'Neue Antworten geteilt';
 $string['responses_submitted_desc'] = 'Es wurden neue Antworten mit der zentralen Berechnungsinstanz {$a->centralhost} geteilt. {$a->added} neue Antworten wurden hinzugefügt, '
     . '{$a->skipped} waren bereits vorhanden und es sind {$a->errors} Fehler aufgetreten';
+$string['responsesbyusercharttitle'] = 'Gesamtanzahl der gegebenen Antworten pro Person';
 $string['rootscale:tooltiptitle'] = 'Globalskala „{$a}“';
 $string['saveconfig'] = 'Konfiguration speicher';
 $string['scaledetailviewheading'] = 'Detailansicht der CAT-Skala „{$a}“';
@@ -716,8 +713,11 @@
 $string['summarynumberofassignedcatscales'] = 'Anzahl der Ihnen zugeordneten Skalen';
 $string['summarynumberoftests'] = 'Anzahl der einsetzenden Tests';
 $string['summarytotalnumberofquestions'] = 'Anzahl der Fragen (insgesamt)';
+$string['sync_scale'] = 'Label der zu synchronisierenden CAT Skala';
+$string['sync_scale_desc'] = 'Parameter dieser Skala werden mit der zentralen Berechnungsinstanz synchronisiert.';
 $string['syncactions'] = 'Mit zentraler Instanz synchronisieren';
 $string['syncactionshelp'] = 'Hier können lokale Daten mit der zentralen Instanz abgeglichen werden';
+$string['syncbutton'] = 'Parameter abrufen';
 $string['target'] = 'Ziel';
 $string['task_recalculate_cat_model_params'] = 'CAT Parameter neu berechnen';
 $string['taskqueued'] = 'Die Berechnung der Parameter wurde in die Warteschlange eingereiht und wird in Kürze ausgeführt.';

lang/en/local_catquiz.php

@@ -169,9 +169,6 @@
 $string['central_host'] = 'Compute instance';
 $string['central_host_desc'] = 'The host that will perform parameter calculations. E.g. https://www.example.com';
 $string['central_host_help'] = 'The host that will perform parameter calculations. E.g. https://www.example.com';
-$string['syncbutton'] = 'Fetch item parameters';
-$string['sync_scale'] = 'Label of the CAT scale to sync';
-$string['sync_scale_desc'] = 'Parameters of this scale will be synced with the compute instance';
 $string['central_scale_labels'] = 'Central scale labels';
 $string['central_scale_labels_desc'] = 'Enter one scale label per line. Only these scales will be processed by the remote calculation task.';
 $string['central_token'] = 'Token to access compute instance';
@@ -602,13 +599,13 @@
 $string['reportscale'] = 'Include scale for report';
 $string['requesttimeout'] = 'The connection request timed out before it could complete';
 $string['response'] = 'Response';
-$string['responsesbyusercharttitle'] = 'Total number of responses per person';
 $string['responses_added'] = 'New responses were submitted';
 $string['responses_added_desc'] = '{$a->sourceurl} submitted new responses. {$a->added} new responses were added, {$a->skipped} '
     . 'were skipped and {$a->errors} errors occured';
 $string['responses_submitted'] = 'New responses were shared';
 $string['responses_submitted_desc'] = 'New responses were shared with central compute instance {$a->centralhost}. {$a->added} new '
     . 'responses were added, {$a->skipped} were skipped and {$a->errors} errors occured';
+$string['responsesbyusercharttitle'] = 'Total number of responses per person';
 $string['rootscale:tooltiptitle'] = 'root scale {$a}';
 $string['saveconfig'] = 'Save configuration';
 $string['scaledetailviewheading'] = 'Detailview of catscale {$a}';
@@ -698,6 +695,9 @@
 $string['summarynumberofassignedcatscales'] = 'Number of assigned CAT scales';
 $string['summarynumberoftests'] = 'Number of assigned tests';
 $string['summarytotalnumberofquestions'] = 'Number of questions (total)';
+$string['sync_scale'] = 'Label of the CAT scale to sync';
+$string['sync_scale_desc'] = 'Parameters of this scale will be synced with the compute instance';
+$string['syncbutton'] = 'Fetch item parameters';
 $string['syncactions'] = 'Syncronize with central instance';
 $string['syncactionshelp'] = 'Here you can perform actions that syncronize local data with a central CAT quiz instance';
 $string['target'] = 'Target';

tests/local/model/model_responses_test.php

@@ -138,6 +138,11 @@ public static function filtering_values_works_as_expected_provider(): array {
         ];
     }
 
+    /**
+     * Test if setting values works as expected
+     *
+     * @return void
+     */
     public function test_setting_values_works_as_expected(): void {
         $mr = new model_responses();
         $mr->set('P1', 'A1', 1.0);
@@ -150,6 +155,11 @@ public function test_setting_values_works_as_expected(): void {
         $this->assertEquals(2 / 3, $mr->get_item_fraction('A1'));
     }
 
+    /**
+     * Test if person params can be updated
+     *
+     * @return void
+     */
     public function test_personparam_can_be_updated(): void {
         $mr = new model_responses();
         $mr->set(1, 'A1', 1.0);
@@ -166,8 +176,10 @@ public function test_personparam_can_be_updated(): void {
 
     /**
      * Test creating responses from remote responses table
+     *
+     * @return void
      */
-    public function test_create_from_remote_responses() {
+    public function test_create_from_remote_responses(): void {
                 // Create some test data.
                 $testdata = [
                     (object) [