Add trailing commas

Author: tvdijen

src/Controller/ConsentController.php

@@ -158,7 +158,7 @@ public function getconsent(Request $request)
             }
         }
         $para = [
-            'attributes' => &$attributes
+            'attributes' => &$attributes,
         ];
 
         // Reorder attributes according to attributepresentation hooks

tests/src/Auth/Process/ConsentTest.php

@@ -137,13 +137,13 @@ public function testAttributeHashIsConsistentWhenOrderOfValuesChange(): void
     {
         $attributes1 = [
             'attribute1' => ['val1', 'val2'],
-            'attribute2' => ['val1', 'val2']
+            'attribute2' => ['val1', 'val2'],
         ];
         $attributeHash1 = Consent::getAttributeHash($attributes1, true);
 
         $attributes2 = [
             'attribute1' => ['val1', 'val2'],
-            'attribute2' => ['val2', 'val1']
+            'attribute2' => ['val2', 'val1'],
         ];
         $attributeHash2 = Consent::getAttributeHash($attributes2, true);
 
@@ -157,20 +157,20 @@ public function testAttributeHashIsConsistentWhenOrderOfAttributesChange(): void
     {
         $attributes1 = [
             'attribute2' => ['val1', 'val2'],
-            'attribute1' => ['val1', 'val2']
+            'attribute1' => ['val1', 'val2'],
         ];
         $attributeHash1 = Consent::getAttributeHash($attributes1, true);
 
         $attributes2 = [
             'attribute1' => ['val1', 'val2'],
-            'attribute2' => ['val1', 'val2']
+            'attribute2' => ['val1', 'val2'],
         ];
         $attributeHash2 = Consent::getAttributeHash($attributes2, true);
 
         $this->assertEquals(
             $attributeHash1,
             $attributeHash2,
-            "Hash is not the same when the order of the attributes changes"
+            "Hash is not the same when the order of the attributes changes",
         );
     }
 
@@ -181,20 +181,20 @@ public function testAttributeHashIsConsistentWithoutValuesWhenOrderOfAttributesC
     {
         $attributes1 = [
             'attribute2' => ['val1', 'val2'],
-            'attribute1' => ['val1', 'val2']
+            'attribute1' => ['val1', 'val2'],
         ];
         $attributeHash1 = Consent::getAttributeHash($attributes1);
 
         $attributes2 = [
             'attribute1' => ['val1', 'val2'],
-            'attribute2' => ['val1', 'val2']
+            'attribute2' => ['val1', 'val2'],
         ];
         $attributeHash2 = Consent::getAttributeHash($attributes2);
 
         $this->assertEquals(
             $attributeHash1,
             $attributeHash2,
-            "Hash is not the same when the order of the attributes changes and the values are not included"
+            "Hash is not the same when the order of the attributes changes and the values are not included",
         );
     }
 
@@ -241,7 +241,7 @@ public function testConstructorSetsInstancePrivateVars(): void
         $this->assertEquals($instanceVars['noconsentattributes']->getValue($testcase), $config['attributes.exclude']);
         $this->assertEquals(
             $instanceVars['showNoConsentAboutService']->getValue($testcase),
-            $config['showNoConsentAboutService']
+            $config['showNoConsentAboutService'],
         );
     }
 }

tests/src/Controller/ConsentControllerTest.php

@@ -84,7 +84,7 @@ public static function loadState(string $id, string $stage, bool $allowMissing =
                         'entityid' => 'urn:some:sp',
                     ],
                     'Source' => [
-                        'entityid' => 'urn:some:idp'
+                        'entityid' => 'urn:some:idp',
                     ],
                     'Attributes' => ['uid' => 'jdoe'],
                     'consent:store.userId' => 'jdoe@example.org',
@@ -119,7 +119,7 @@ public function testGetconsentDecline(): void
         $request = Request::create(
             '/getconsent',
             'GET',
-            ['no' => '', 'StateId' => 'someStateId']
+            ['no' => '', 'StateId' => 'someStateId'],
         );
 
         $c = new Controller\ConsentController(self::$config, self::$session);
@@ -135,8 +135,8 @@ public static function loadState(string $id, string $stage, bool $allowMissing =
                     'Source' => [
                         'entityid' => 'urn:some:idp',
                         'UIInfo' => [
-                            'PrivacyStatementURL' => ['https://example.org/privacy']
-                        ]
+                            'PrivacyStatementURL' => ['https://example.org/privacy'],
+                        ],
                     ],
                     'Attributes' => ['uid' => 'jdoe', 'filteredAttribute' => 'this attribute should be filtered'],
                     'consent:noconsentattributes' => ['filteredAttribute'],
@@ -160,7 +160,7 @@ public function testNoconsent(): void
         $request = Request::create(
             '/noconsent',
             'GET',
-            ['StateId' => 'someStateId']
+            ['StateId' => 'someStateId'],
         );
 
         $c = new Controller\ConsentController(self::$config, self::$session);
@@ -193,15 +193,15 @@ public function testLogout(): void
         $request = Request::create(
             '/logout',
             'GET',
-            ['StateId' => 'someStateId']
+            ['StateId' => 'someStateId'],
         );
 
         $c = new Controller\ConsentController(self::$config, self::$session);
         $c->setAuthState(new class () extends State {
             public static function loadState(string $id, string $stage, bool $allowMissing = false): ?array
             {
                 return [
-                    'core:IdP' => 'saml2:something'
+                    'core:IdP' => 'saml2:something',
                 ];
             }
         });
@@ -233,7 +233,7 @@ public function testMissingStateId(string $controller): void
         $_SERVER['REQUEST_URI'] = '/module.php/consent/' . $controller;
         $request = Request::create(
             '/' . $controller,
-            'GET'
+            'GET',
         );
 
         $c = new Controller\ConsentController(self::$config, self::$session);
@@ -256,7 +256,7 @@ public function testNoState(string $controller): void
         $request = Request::create(
             '/' . $controller,
             'GET',
-            ['StateId' => 'someStateId']
+            ['StateId' => 'someStateId'],
         );
 
         $c = new Controller\ConsentController(self::$config, self::$session);