style: have phpcbf replace string double-quotes with single-quotes

Author: MHajoha

classes/api/api.php

@@ -109,7 +109,7 @@ public static function extract_package_info(stored_file $file): package_raw {
             'package' => curl_file_create($filepath),
         ];
 
-        $response = $connector->post("/package-extract-info", $data);
+        $response = $connector->post('/package-extract-info', $data);
         $response->assert_2xx();
         return array_converter::from_array(package_raw::class, $response->get_data());
     }
@@ -122,7 +122,7 @@ public static function extract_package_info(stored_file $file): package_raw {
      */
     public static function get_server_status(): status {
         $connector = connector::default();
-        $response = $connector->get("/status");
+        $response = $connector->get('/status');
         $response->assert_2xx();
         return array_converter::from_array(status::class, $response->get_data());
     }

classes/api/attempt_file.php

@@ -31,7 +31,7 @@ class attempt_file {
     public string $name;
 
     /** @var string|null */
-    #[array_key("mime_type")]
+    #[array_key('mime_type')]
     public ?string $mimetype = null;
 
     /** @var string $data */

classes/api/attempt_scored.php

@@ -28,11 +28,11 @@
  */
 class attempt_scored extends attempt {
     /** @var string|null */
-    #[array_key("scoring_state")]
+    #[array_key('scoring_state')]
     public ?string $scoringstate;
 
     /** @var scoring_code */
-    #[array_key("scoring_code")]
+    #[array_key('scoring_code')]
     public scoring_code $scoringcode;
 
     /** @var float|null */

classes/api/attempt_started.php

@@ -28,7 +28,7 @@
  */
 class attempt_started extends attempt {
     /** @var string */
-    #[array_key("attempt_state")]
+    #[array_key('attempt_state')]
     public string $attemptstate;
 
     /**

classes/api/attempt_ui.php

@@ -32,31 +32,31 @@ class attempt_ui {
     public string $formulation;
 
     /** @var string|null */
-    #[array_key("general_feedback")]
+    #[array_key('general_feedback')]
     public ?string $generalfeedback = null;
 
     /** @var string|null */
-    #[array_key("specific_feedback")]
+    #[array_key('specific_feedback')]
     public ?string $specificfeedback = null;
 
     /** @var string|null */
-    #[array_key("right_answer")]
+    #[array_key('right_answer')]
     public ?string $rightanswer = null;
 
     /** @var array<string, string> string to string mapping of placeholder names to the values (to be replaced in the content) */
     public array $placeholders = [];
 
     /** @var string[]|null */
-    #[array_key("css_files")]
+    #[array_key('css_files')]
     public ?array $cssfiles = null;
 
     /** @var array<string, attempt_file> specifics TBD */
     #[array_element_class(attempt_file::class)]
     public array $files = [];
 
     /** @var string specifics TBD */
-    #[array_key("cache_control")]
-    public string $cachecontrol = "PRIVATE_CACHE";
+    #[array_key('cache_control')]
+    public string $cachecontrol = 'PRIVATE_CACHE';
 
     /**
      * Initializes a new instance.

classes/api/package_api.php

@@ -194,17 +194,17 @@ private function guzzle_post_and_maybe_retry(string $uri, array $options = [], b
 
             $fd = $this->file->get_content_file_handle();
             try {
-                $options["multipart"][] = [
-                    "name" => "package",
-                    "contents" => $fd,
+                $options['multipart'][] = [
+                    'name' => 'package',
+                    'contents' => $fd,
                 ];
 
                 return $this->guzzle_post_and_maybe_retry($uri, $options, allowretry: false);
             } finally {
                 @fclose($fd);
             }
         } catch (GuzzleException $e) {
-            throw new coding_exception("Request to QPy server failed: " . $e->getMessage());
+            throw new coding_exception('Request to QPy server failed: ' . $e->getMessage());
         }
     }
 
@@ -226,7 +226,7 @@ public function download_static_file(string $namespace, string $shortname, strin
         try {
             $res = $this->guzzle_post_and_maybe_retry(
                 "/packages/$this->hash/file/$namespace/$shortname/$kind/$path",
-                ["sink" => $targetpath]
+                ['sink' => $targetpath]
             );
         } catch (BadResponseException $e) {
             if ($e->getResponse()->getStatusCode() == 404) {
@@ -239,11 +239,11 @@ public function download_static_file(string $namespace, string $shortname, strin
             );
         }
 
-        if ($res->hasHeader("Content-Type")) {
-            return $res->getHeader("Content-Type")[0];
+        if ($res->hasHeader('Content-Type')) {
+            return $res->getHeader('Content-Type')[0];
         } else {
-            debugging("Server did not send Content-Type header, falling back to application/octet-stream");
-            return "application/octet-stream";
+            debugging('Server did not send Content-Type header, falling back to application/octet-stream');
+            return 'application/octet-stream';
         }
     }
 

classes/api/qpy_http_client.php

@@ -37,8 +37,8 @@ class qpy_http_client extends http_client {
      * @throws dml_exception
      */
     public function __construct(array $config = []) {
-        $config["base_uri"] = rtrim(get_config('qtype_questionpy', 'server_url'), "/") . "/";
-        $config["timeout"] = get_config('qtype_questionpy', 'server_timeout');
+        $config['base_uri'] = rtrim(get_config('qtype_questionpy', 'server_url'), '/') . '/';
+        $config['timeout'] = get_config('qtype_questionpy', 'server_timeout');
         parent::__construct($config);
     }
 
@@ -53,7 +53,7 @@ protected function get_handlers(array $settings): HandlerStack {
         /* This checks requests against Moodle's curlsecurityblockedhosts, which we don't want, since admins would need
            to ensure their QPy server isn't in this list otherwise. There may be ways to granularly allow the
            server_url, but this will do for now. */
-        $handlerstack->remove("moodle_check_initial_request");
+        $handlerstack->remove('moodle_check_initial_request');
         return $handlerstack;
     }
 }

classes/api/question_edit_form_response.php

@@ -32,7 +32,7 @@ class question_edit_form_response {
     public qpy_form $definition;
 
     /** @var array */
-    #[array_key("form_data")]
+    #[array_key('form_data')]
     public array $formdata;
 
     /**

classes/api/question_response.php

@@ -28,34 +28,34 @@
  */
 class question_response {
     /** @var string */
-    #[array_key("question_state")]
+    #[array_key('question_state')]
     public string $state;
 
     /** @var string */
-    #[array_key("scoring_method")]
+    #[array_key('scoring_method')]
     public string $scoringmethod;
 
     /** @var float|int */
-    #[array_key("score_min")]
+    #[array_key('score_min')]
     public float $scoremin = 0;
 
     /** @var float|int */
-    #[array_key("score_max")]
+    #[array_key('score_max')]
     public float $scoremax = 1;
 
     /** @var float|null */
     public ?float $penalty = null;
 
     /** @var float|null */
-    #[array_key("random_guess_score")]
+    #[array_key('random_guess_score')]
     public ?float $randomguessscore = null;
 
     /** @var bool */
-    #[array_key("render_every_view")]
+    #[array_key('render_every_view')]
     public bool $rendereveryview = false;
 
     /** @var string|null */
-    #[array_key("general_feedback")]
+    #[array_key('general_feedback')]
     public ?string $generalfeedback = null;
 
     /**

classes/api/scoring_code.php

@@ -26,8 +26,8 @@
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
 enum scoring_code: string {
-    case automatically_scored = "AUTOMATICALLY_SCORED";
-    case needs_manual_scoring = "NEEDS_MANUAL_SCORING";
-    case response_not_scorable = "RESPONSE_NOT_SCORABLE";
-    case invalid_response = "INVALID_RESPONSE";
+    case automatically_scored = 'AUTOMATICALLY_SCORED';
+    case needs_manual_scoring = 'NEEDS_MANUAL_SCORING';
+    case response_not_scorable = 'RESPONSE_NOT_SCORABLE';
+    case invalid_response = 'INVALID_RESPONSE';
 }