Align multiple subsequent assignments.

GoogleChromeLabs · Aug 29, 2024 · da4a2c2 · da4a2c2
1 parent 260b9f1
commit da4a2c2
Show file tree

Hide file tree

Showing 11 changed files with 20 additions and 18 deletions.
diff --git a/inc/Data/ThirdPartyDataFormatter.php b/inc/Data/ThirdPartyDataFormatter.php
@@ -28,7 +28,7 @@ class ThirdPartyDataFormatter
      */
     public static function formatData(ThirdPartyData $data, array $args): ThirdPartyOutput
     {
-        $htmlData = $data->getHtml();
+        $htmlData    = $data->getHtml();
         $scriptsData = $data->getScripts();
 
         $allScriptParams = array_reduce(
@@ -169,7 +169,7 @@ public static function formatUrl(string $url, array $params, array $args, array
             $path = parse_url($url, PHP_URL_PATH);
             if ($path) {
                 $trailingSlash = str_ends_with($path, '/') ? '/' : '';
-                $url = str_replace(
+                $url           = str_replace(
                     $path,
                     substr($path, 0, - strlen(basename($path) . $trailingSlash)) . $slug . $trailingSlash,
                     $url

diff --git a/inc/Data/ThirdPartySrcValue.php b/inc/Data/ThirdPartySrcValue.php
@@ -52,9 +52,9 @@ public function __construct(array $srcData)
             throw new InvalidThirdPartyDataException('Missing src url.');
         }
 
-        $this->url = $srcData['url'];
+        $this->url       = $srcData['url'];
         $this->slugParam = isset($srcData['slugParam']) ? (string) $srcData['slugParam'] : '';
-        $this->params = isset($srcData['params']) ? array_map('strval', $srcData['params']) : [];
+        $this->params    = isset($srcData['params']) ? array_map('strval', $srcData['params']) : [];
     }
 
     /**

diff --git a/phpcs.xml.dist b/phpcs.xml.dist
@@ -17,6 +17,8 @@
 		</properties>
 	</rule>
 
+	<rule ref="Generic.Formatting.MultipleStatementAlignment"/>
+
 	<rule ref="Generic.Files.LineLength">
 		<properties>
 			<property name="lineLimit" value="120"/>

diff --git a/tests/phpunit/tests/Data/ThirdPartyDataTest.php b/tests/phpunit/tests/Data/ThirdPartyDataTest.php
@@ -114,7 +114,7 @@ public function testToArray()
             ],
             'stylesheets' => ['https://example.com/style.css', 'https://example.com/style-2.css'],
         ];
-        $data = new ThirdPartyData($input);
+        $data  = new ThirdPartyData($input);
         $this->assertSame($input, $data->toArray());
     }
 }
diff --git a/tests/phpunit/tests/Data/ThirdPartyHtmlDataTest.php b/tests/phpunit/tests/Data/ThirdPartyHtmlDataTest.php
@@ -45,7 +45,7 @@ public function dataGetMethods()
 
     public function testToArray()
     {
-        $input = [
+        $input    = [
             'element'    => 'iframe',
             'attributes' => [
                 'src'    => [

diff --git a/tests/phpunit/tests/Data/ThirdPartyOutputTest.php b/tests/phpunit/tests/Data/ThirdPartyOutputTest.php
@@ -84,7 +84,7 @@ public function dataGetMethods()
 
     public function testToArray()
     {
-        $input = [
+        $input  = [
             'id'          => 'my-service',
             'description' => 'A service that allows embedding something.',
             'website'     => 'https://my-service.com/',

diff --git a/tests/phpunit/tests/Data/ThirdPartyScriptDataTest.php b/tests/phpunit/tests/Data/ThirdPartyScriptDataTest.php
@@ -76,15 +76,15 @@ public function dataGetMethods()
 
     public function testConstructorWithUrlAndNoCode()
     {
-        $data = ['url' => 'https://example.com/'];
+        $data       = ['url' => 'https://example.com/'];
         $scriptData = new ThirdPartyScriptData(array_merge($this->baseData, $data));
 
         $this->assertSame($data['url'], $scriptData->getUrl());
     }
 
     public function testConstructorWithCodeAndNoUrl()
     {
-        $data = ['code' => 'window.dataLayer=window.dataLayer'];
+        $data       = ['code' => 'window.dataLayer=window.dataLayer'];
         $scriptData = new ThirdPartyScriptData(array_merge($this->baseData, $data));
 
         $this->assertSame($data['code'], $scriptData->getCode());
@@ -101,7 +101,7 @@ public function testConstructorWithUrlAndCode()
     {
         $this->expectException(InvalidThirdPartyDataException::class);
 
-        $data = [
+        $data       = [
             'url'  => 'https://example.com/',
             'code' => 'window.dataLayer=window.dataLayer',
         ];
@@ -137,7 +137,7 @@ public function dataIsExternal()
 
     public function testToArray()
     {
-        $input = array_merge(
+        $input      = array_merge(
             $this->baseData,
             [
                 'url'    => 'https://example.com/',

diff --git a/tests/phpunit/tests/Data/ThirdPartyScriptOutputTest.php b/tests/phpunit/tests/Data/ThirdPartyScriptOutputTest.php
@@ -68,7 +68,7 @@ public function dataGetMethods()
 
     public function testToArray()
     {
-        $input = [
+        $input        = [
             'strategy' => ThirdPartyScriptData::STRATEGY_CLIENT,
             'location' => ThirdPartyScriptData::LOCATION_HEAD,
             'action'   => ThirdPartyScriptData::ACTION_APPEND,

diff --git a/tests/phpunit/tests/Data/ThirdPartySrcValueTest.php b/tests/phpunit/tests/Data/ThirdPartySrcValueTest.php
@@ -50,7 +50,7 @@ public function dataGetMethods()
 
     public function testToArray()
     {
-        $input = [
+        $input    = [
             'url'       => 'https://my-embed.com',
             'slugParam' => 'type',
             'params'    => ['id', 'mode'],

diff --git a/tests/phpunit/tests/Util/HtmlAttributesTest.php b/tests/phpunit/tests/Util/HtmlAttributesTest.php
@@ -77,7 +77,7 @@ public function testOffsetUnset()
 
     public function testGetIterator()
     {
-        $attrs = new HtmlAttributes([
+        $attrs  = new HtmlAttributes([
             'id'    => 'unique-id',
             'class' => 'test-class',
         ]);

diff --git a/tests/phpunit/utils/TestCase.php b/tests/phpunit/utils/TestCase.php
@@ -27,7 +27,7 @@ protected function runGetterTestCase(string $className, string $getMethod, array
         }
 
         $instance = new $className($args);
-        $result = call_user_func([$instance, $getMethod]);
+        $result   = call_user_func([$instance, $getMethod]);
         if ($result instanceof Arrayable) {
             $result = $result->toArray();
         } elseif (is_array($result)) {
@@ -57,7 +57,7 @@ protected function gettersToTestCases(array $getters, string $exceptionClass = n
             }
 
             if (!isset($getter['value'])) {
-                $type = isset($getter['default']) ? gettype($getter['default']) : 'string';
+                $type  = isset($getter['default']) ? gettype($getter['default']) : 'string';
                 $value = $this->createValueOfType($type);
             } else {
                 $value = $getter['value'];
@@ -68,12 +68,12 @@ protected function gettersToTestCases(array $getters, string $exceptionClass = n
         $testCases = [];
         foreach ($getters as $getter) {
             if (!isset($getter['value'])) {
-                $type = isset($getter['default']) ? gettype($getter['default']) : 'string';
+                $type  = isset($getter['default']) ? gettype($getter['default']) : 'string';
                 $value = $this->createValueOfType($type);
             } else {
                 $value = $getter['value'];
             }
-            $args = $requiredFields;
+            $args                   = $requiredFields;
             $args[$getter['field']] = $value;
 
             $testCases["{$getter['getter']} with value"] = [