Generic.WhiteSpace.ScopeIndent now disables exact indent checking for multi-line chained methods (ref #2371)

Author: pfrenssen

src/Standards/Generic/Sniffs/WhiteSpace/ScopeIndentSniff.php

@@ -903,6 +903,15 @@ public function process(File $phpcsFile, $stackPtr)
                 $exact = true;
             }
 
+            // Don't perform strict checking on chained method calls since they
+            // are often covered by custom rules.
+            if ($checkToken !== null
+                && $tokens[$checkToken]['code'] === T_OBJECT_OPERATOR
+                && $exact === true
+            ) {
+                $exact = false;
+            }
+
             if ($checkIndent === null) {
                 $checkIndent = $currentIndent;
             }

src/Standards/Generic/Tests/WhiteSpace/ScopeIndentUnitTest.1.inc

@@ -1311,6 +1311,9 @@ function foo() {
     if ($foo) {
             echo 'baz';
     }
+    $this->foo()
+->bar()
+      ->baz();
 }
 // phpcs:set Generic.WhiteSpace.ScopeIndent exact false
 

src/Standards/Generic/Tests/WhiteSpace/ScopeIndentUnitTest.1.inc.fixed

@@ -1311,6 +1311,9 @@ function foo() {
     if ($foo) {
         echo 'baz';
     }
+    $this->foo()
+    ->bar()
+      ->baz();
 }
 // phpcs:set Generic.WhiteSpace.ScopeIndent exact false
 

src/Standards/Generic/Tests/WhiteSpace/ScopeIndentUnitTest.2.inc

@@ -1311,6 +1311,9 @@ function foo() {
 	if ($foo) {
 			echo 'baz';
 	}
+	$this->foo()
+->bar()
+		->baz();
 }
 // phpcs:set Generic.WhiteSpace.ScopeIndent exact false
 

src/Standards/Generic/Tests/WhiteSpace/ScopeIndentUnitTest.2.inc.fixed

@@ -1311,6 +1311,9 @@ function foo() {
 	if ($foo) {
 		echo 'baz';
 	}
+	$this->foo()
+	->bar()
+		->baz();
 }
 // phpcs:set Generic.WhiteSpace.ScopeIndent exact false
 

src/Standards/Generic/Tests/WhiteSpace/ScopeIndentUnitTest.php

@@ -164,20 +164,21 @@ public function getErrorList($testFile='ScopeIndentUnitTest.inc')
             1293 => 1,
             1310 => 1,
             1312 => 1,
-            1327 => 1,
-            1328 => 1,
-            1329 => 1,
+            1315 => 1,
             1330 => 1,
             1331 => 1,
             1332 => 1,
+            1333 => 1,
+            1334 => 1,
             1335 => 1,
-            1340 => 1,
-            1342 => 1,
+            1338 => 1,
+            1343 => 1,
             1345 => 1,
-            1364 => 1,
-            1365 => 1,
-            1366 => 1,
+            1348 => 1,
             1367 => 1,
+            1368 => 1,
+            1369 => 1,
+            1370 => 1,
         ];
 
     }//end getErrorList()