Priority: Braces before MethodChainingIndentation

PHP-CS-Fixer · Nov 8, 2018 · c0358e8 · c0358e8
1 parent d1e1ae2
commit c0358e8
Show file tree

Hide file tree

Showing 3 changed files with 47 additions and 0 deletions.
diff --git a/src/Fixer/Whitespace/MethodChainingIndentationFixer.php b/src/Fixer/Whitespace/MethodChainingIndentationFixer.php
@@ -37,6 +37,15 @@ public function getDefinition()
         );
     }
 
+    /**
+     * {@inheritdoc}
+     */
+    public function getPriority()
+    {
+        // Should run after BracesFixer
+        return -30;
+    }
+
     /**
      * {@inheritdoc}
      */

diff --git a/tests/AutoReview/FixerFactoryTest.php b/tests/AutoReview/FixerFactoryTest.php
@@ -66,6 +66,7 @@ public function provideFixersPriorityCases()
             [$fixers['backtick_to_shell_exec'], $fixers['escape_implicit_backslashes']],
             [$fixers['blank_line_after_opening_tag'], $fixers['no_blank_lines_before_namespace']],
             [$fixers['braces'], $fixers['array_indentation']],
+            [$fixers['braces'], $fixers['method_chaining_indentation']],
             [$fixers['class_attributes_separation'], $fixers['braces']],
             [$fixers['class_attributes_separation'], $fixers['indentation_type']],
             [$fixers['class_keyword_remove'], $fixers['no_unused_imports']],

diff --git a/tests/Fixtures/Integration/priority/braces,method_chaining_indentation.test b/tests/Fixtures/Integration/priority/braces,method_chaining_indentation.test
@@ -0,0 +1,37 @@
+--TEST--
+Integration of fixers: braces,method_chaining_indentation.
+--RULESET--
+{"braces": true, "method_chaining_indentation": true}
+--EXPECT--
+<?php
+class SomeUnitTest extends PHPUnit\Framework\TestCase
+{
+    public function testSomething()
+    {
+        // *Only* the next line is indented wrong _before_ fixing
+        $this->entity
+            ->expects($this->once())
+            ->method('toXml')
+            ->willReturnCallback(
+                function (\DOMDocument $dom) {
+                    return $dom->createElement('elem');
+                });
+    }
+}
+
+--INPUT--
+<?php
+class SomeUnitTest extends PHPUnit\Framework\TestCase
+{
+    public function testSomething()
+    {
+        // *Only* the next line is indented wrong _before_ fixing
+         $this->entity
+            ->expects($this->once())
+            ->method('toXml')
+            ->willReturnCallback(
+                function (\DOMDocument $dom) {
+                    return $dom->createElement('elem');
+                });
+    }
+}