style: fix remaining trailing comma phpcs issues

There's a bug in phpcs that is preventing these checks from catching all cases. With this in mind, I've opted to use the Drupal standards to catch these missing cases which seems to work well. The rule is "Drupal.Arrays.Array.CommaLastItem"

classes/dataflow.php

@@ -209,7 +209,7 @@ public function get_variables(): array {
                 'DATAFLOW_RUN_NAME' => $this->engine->run->name ?? null,
             ],
             'dataflow' => $dataflow,
-            'steps' => $steps
+            'steps' => $steps,
         ];
         return $variables;
     }

classes/local/step/connector_s3.php

@@ -92,7 +92,7 @@ public function execute(): bool {
         $config = $this->enginestep->stepdef->config;
         $connectionoptions = [
             'version' => 'latest',
-            'region' => $config->region
+            'region' => $config->region,
         ];
         if ($config->key !== '') {
             $connectionoptions['credentials'] = [

classes/step.php

@@ -547,11 +547,12 @@ protected function validate_link_count(int $count, string $inputoutput, string $
         $steptype = $this->steptype;
         [$min, $max] = $steptype->$fn();
         if ($count < $min || $count > $max) {
-            return ["invalid_count_{$inputoutput}{$flowconnector}s_{$this->id}" => get_string(
-                "stepinvalid{$inputoutput}{$flowconnector}count",
-                'tool_dataflows',
-                $count
-            ) . ' ' . visualiser::get_link_expectations($steptype, $inputoutput)
+            return [
+                "invalid_count_{$inputoutput}{$flowconnector}s_{$this->id}" => get_string(
+                    "stepinvalid{$inputoutput}{$flowconnector}count",
+                    'tool_dataflows',
+                    $count
+                ) . ' ' . visualiser::get_link_expectations($steptype, $inputoutput),
             ];
         }
         return true;

edit.php

@@ -56,7 +56,7 @@
 // Render the specific dataflow form.
 $customdata = [
     'persistent' => $persistent ?? null, // An instance, or null.
-    'userid' => $USER->id // For the hidden userid field.
+    'userid' => $USER->id, // For the hidden userid field.
 ];
 $form = new dataflow_form($PAGE->url->out(false), $customdata);
 if ($form->is_cancelled()) {

tests/tool_dataflows_secret_service_test.php

@@ -56,7 +56,7 @@ protected static function form_define_fields(): array {
             'secret'            => ['type' => PARAM_TEXT, 'secret' => true],
             'source'            => ['type' => PARAM_TEXT],
             'target'            => ['type' => PARAM_TEXT],
-            'sourceremote'      => ['type' => PARAM_BOOL]
+            'sourceremote'      => ['type' => PARAM_BOOL],
         ];
     }
 }