From 7d36ce912c3bfb4ba09595063086fa6b018f590f Mon Sep 17 00:00:00 2001
From: ErickSkrauch <erickskrauch@yandex.ru>
Date: Sat, 8 Apr 2023 21:07:29 +0200
Subject: [PATCH] Fixes #13. Implemented Ely\multiline_if_statement_braces
 fixer

---
 CHANGELOG.md                                  |  2 +
 README.md                                     | 22 +++++
 composer.json                                 |  3 +-
 composer.lock                                 | 16 ++--
 .../MultilineIfStatementBracesFixer.php       | 92 +++++++++++++++++++
 src/Rules.php                                 |  1 +
 .../MultilineIfStatementBracesFixerTest.php   | 68 ++++++++++++++
 7 files changed, 195 insertions(+), 9 deletions(-)
 create mode 100644 src/Fixer/Whitespace/MultilineIfStatementBracesFixer.php
 create mode 100644 tests/Fixer/Whitespace/MultilineIfStatementBracesFixerTest.php

diff --git a/CHANGELOG.md b/CHANGELOG.md
index eff292d..474ac21 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -7,7 +7,9 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 ## [Unreleased]
 ### Added
 - Enh #12: Implemented `Ely\align_multiline_parameters` fixer.
+- Enh #13: Implemented `Ely\multiline_if_statement_braces` fixer.
 - Enabled `Ely\align_multiline_parameters` for Ely.by codestyle in `['types' => false, 'defaults' => false]` mode.
+- Enabled `Ely\multiline_if_statement_braces` for Ely.by codestyle in `['keep_on_own_line' => true]` mode.
 - Enabled
   [`PhpCsFixerCustomFixers/multiline_promoted_properties`](https://github.com/kubawerlos/php-cs-fixer-custom-fixers#multilinepromotedpropertiesfixer)
   fixer for Ely.by codestyle in 2+ parameters mode.
diff --git a/README.md b/README.md
index de27c59..04f1f49 100644
--- a/README.md
+++ b/README.md
@@ -286,6 +286,28 @@ and `do-while`.
  $c = 'next statement';
 ```
 
+### `Ely/multiline_if_statement_braces`
+
+Ensures that multiline if statement body curly brace placed on the right line.
+
+```diff
+--- Original
++++ New
+@@ @@
+ <?php
+ if ($condition1 === 123
+- && $condition2 = 321) {
++ && $condition2 = 321
++) {
+     // Do something here
+ }
+```
+
+**Configuration:**
+
+* `keep_on_own_line` - should this place closing bracket on its own line? If it's set to `false`, than
+  curly bracket will be placed right after the last condition statement. **Default**: `true`.
+
 ### `Ely/remove_class_name_method_usages` (Yii2)
 
 Replaces Yii2 [`BaseObject::className()`](https://github.com/yiisoft/yii2/blob/e53fc0ded1/framework/base/BaseObject.php#L84)
diff --git a/composer.json b/composer.json
index 839395d..2a1ff20 100644
--- a/composer.json
+++ b/composer.json
@@ -21,7 +21,8 @@
     "require": {
         "php": "^7.4 || ^8.0",
         "friendsofphp/php-cs-fixer": "^3.13",
-        "kubawerlos/php-cs-fixer-custom-fixers": "^3.13"
+        "kubawerlos/php-cs-fixer-custom-fixers": "^3.13",
+        "symfony/polyfill-php80": "^1.15"
     },
     "require-dev": {
         "ergebnis/composer-normalize": "^2.28",
diff --git a/composer.lock b/composer.lock
index 48f74cd..3bb5890 100644
--- a/composer.lock
+++ b/composer.lock
@@ -4,7 +4,7 @@
         "Read more about it at https://getcomposer.org/doc/01-basic-usage.md#installing-dependencies",
         "This file is @generated automatically"
     ],
-    "content-hash": "9fc362b1a074bf1a33695018a2b1b198",
+    "content-hash": "8f031e395244dc0c0de0135b5c34ceda",
     "packages": [
         {
             "name": "composer/pcre",
@@ -1708,16 +1708,16 @@
         },
         {
             "name": "symfony/polyfill-php80",
-            "version": "v1.26.0",
+            "version": "v1.27.0",
             "source": {
                 "type": "git",
                 "url": "https://github.com/symfony/polyfill-php80.git",
-                "reference": "cfa0ae98841b9e461207c13ab093d76b0fa7bace"
+                "reference": "7a6ff3f1959bb01aefccb463a0f2cd3d3d2fd936"
             },
             "dist": {
                 "type": "zip",
-                "url": "https://api.github.com/repos/symfony/polyfill-php80/zipball/cfa0ae98841b9e461207c13ab093d76b0fa7bace",
-                "reference": "cfa0ae98841b9e461207c13ab093d76b0fa7bace",
+                "url": "https://api.github.com/repos/symfony/polyfill-php80/zipball/7a6ff3f1959bb01aefccb463a0f2cd3d3d2fd936",
+                "reference": "7a6ff3f1959bb01aefccb463a0f2cd3d3d2fd936",
                 "shasum": ""
             },
             "require": {
@@ -1726,7 +1726,7 @@
             "type": "library",
             "extra": {
                 "branch-alias": {
-                    "dev-main": "1.26-dev"
+                    "dev-main": "1.27-dev"
                 },
                 "thanks": {
                     "name": "symfony/polyfill",
@@ -1771,7 +1771,7 @@
                 "shim"
             ],
             "support": {
-                "source": "https://github.com/symfony/polyfill-php80/tree/v1.26.0"
+                "source": "https://github.com/symfony/polyfill-php80/tree/v1.27.0"
             },
             "funding": [
                 {
@@ -1787,7 +1787,7 @@
                     "type": "tidelift"
                 }
             ],
-            "time": "2022-05-10T07:21:04+00:00"
+            "time": "2022-11-03T14:55:06+00:00"
         },
         {
             "name": "symfony/polyfill-php81",
diff --git a/src/Fixer/Whitespace/MultilineIfStatementBracesFixer.php b/src/Fixer/Whitespace/MultilineIfStatementBracesFixer.php
new file mode 100644
index 0000000..0fa1d1c
--- /dev/null
+++ b/src/Fixer/Whitespace/MultilineIfStatementBracesFixer.php
@@ -0,0 +1,92 @@
+<?php
+declare(strict_types=1);
+
+namespace Ely\CS\Fixer\Whitespace;
+
+use Ely\CS\Fixer\AbstractFixer;
+use PhpCsFixer\Fixer\ConfigurableFixerInterface;
+use PhpCsFixer\Fixer\WhitespacesAwareFixerInterface;
+use PhpCsFixer\FixerConfiguration\FixerConfigurationResolver;
+use PhpCsFixer\FixerConfiguration\FixerConfigurationResolverInterface;
+use PhpCsFixer\FixerConfiguration\FixerOptionBuilder;
+use PhpCsFixer\FixerDefinition\CodeSample;
+use PhpCsFixer\FixerDefinition\FixerDefinition;
+use PhpCsFixer\FixerDefinition\FixerDefinitionInterface;
+use PhpCsFixer\Tokenizer\Analyzer\WhitespacesAnalyzer;
+use PhpCsFixer\Tokenizer\Tokens;
+use PhpCsFixer\Tokenizer\TokensAnalyzer;
+use SplFileInfo;
+
+final class MultilineIfStatementBracesFixer extends AbstractFixer implements ConfigurableFixerInterface, WhitespacesAwareFixerInterface {
+
+    /**
+     * @internal
+     */
+    public const C_KEEP_ON_OWN_LINE = 'keep_on_own_line';
+
+    public function getDefinition(): FixerDefinitionInterface {
+        return new FixerDefinition(
+            'Ensures that multiline if statement body curly brace placed on the right line.',
+            [
+                new CodeSample(
+                    '<?php
+if ($condition1 == true
+ && $condition2 === false) {}
+',
+                ),
+                new CodeSample(
+                    '<?php
+if ($condition1 == true
+ && $condition2 === false
+) {}
+',
+                    [self::C_KEEP_ON_OWN_LINE => false],
+                ),
+            ],
+        );
+    }
+
+    public function isCandidate(Tokens $tokens): bool {
+        return $tokens->isTokenKindFound(T_IF);
+    }
+
+    protected function createConfigurationDefinition(): FixerConfigurationResolverInterface {
+        return new FixerConfigurationResolver([
+            (new FixerOptionBuilder(self::C_KEEP_ON_OWN_LINE, 'adjusts the position of condition closing brace.'))
+                ->setAllowedTypes(['bool'])
+                ->setDefault(true)
+                ->getOption(),
+        ]);
+    }
+
+    protected function applyFix(SplFileInfo $file, Tokens $tokens): void {
+        $keepOnOwnLine = $this->configuration[self::C_KEEP_ON_OWN_LINE];
+        $tokensAnalyzer = new TokensAnalyzer($tokens);
+        $eol = $this->whitespacesConfig->getLineEnding();
+        foreach ($tokens as $i => $token) {
+            if (!$token->isGivenKind(T_IF)) {
+                continue;
+            }
+
+            $openBraceIndex = $tokens->getNextTokenOfKind($i, ['(']);
+            if (!$tokensAnalyzer->isBlockMultiline($tokens, $openBraceIndex)) {
+                continue;
+            }
+
+            $closingBraceIndex = $tokens->findBlockEnd(Tokens::BLOCK_TYPE_PARENTHESIS_BRACE, $openBraceIndex);
+            /** @var \PhpCsFixer\Tokenizer\Token $statementBeforeClosingBrace */
+            $statementBeforeClosingBrace = $tokens[$closingBraceIndex - 1];
+            if ($keepOnOwnLine) {
+                if (!$statementBeforeClosingBrace->isWhitespace()
+                 || !str_contains($statementBeforeClosingBrace->getContent(), $eol)
+                ) {
+                    $indent = WhitespacesAnalyzer::detectIndent($tokens, $i);
+                    $tokens->ensureWhitespaceAtIndex($closingBraceIndex, 0, $eol . $indent);
+                }
+            } else {
+                $tokens->removeLeadingWhitespace($closingBraceIndex);
+            }
+        }
+    }
+
+}
diff --git a/src/Rules.php b/src/Rules.php
index 42b8630..7d93f57 100644
--- a/src/Rules.php
+++ b/src/Rules.php
@@ -228,6 +228,7 @@ public static function create(array $overwrittenRules = []): array {
             ],
             'Ely/blank_line_before_return' => true,
             'Ely/line_break_after_statements' => true,
+            'Ely/multiline_if_statement_braces' => true,
             'Ely/remove_class_name_method_usages' => true,
         ], $overwrittenRules);
     }
diff --git a/tests/Fixer/Whitespace/MultilineIfStatementBracesFixerTest.php b/tests/Fixer/Whitespace/MultilineIfStatementBracesFixerTest.php
new file mode 100644
index 0000000..dd2434b
--- /dev/null
+++ b/tests/Fixer/Whitespace/MultilineIfStatementBracesFixerTest.php
@@ -0,0 +1,68 @@
+<?php
+declare(strict_types=1);
+
+namespace Ely\CS\Test\Fixer\Whitespace;
+
+use Ely\CS\Fixer\Whitespace\MultilineIfStatementBracesFixer;
+use PhpCsFixer\AbstractFixer;
+use PhpCsFixer\Tests\Test\AbstractFixerTestCase;
+
+/**
+ * @covers \Ely\CS\Fixer\Whitespace\MultilineIfStatementBracesFixer
+ */
+class MultilineIfStatementBracesFixerTest extends AbstractFixerTestCase {
+
+    /**
+     * @dataProvider provideFixCases
+     */
+    public function testFixOnNewLine(string $expected, ?string $input = null): void {
+        $this->doTest($expected, $input);
+    }
+
+    public function provideFixCases(): iterable {
+        yield 'simple' => [
+            '<?php
+if ($condition1
+ && $condition2
+) {}',
+            '<?php
+if ($condition1
+ && $condition2) {}',
+        ];
+
+        yield 'nested' => [
+            '<?php
+function foo() {
+    if ($condition1
+     && $condition2
+    ) {}
+}',
+            '<?php
+function foo() {
+    if ($condition1
+     && $condition2) {}
+}',
+        ];
+    }
+
+    /**
+     * @dataProvider provideInvertedFixCases
+     */
+    public function testFixOnSameLine(string $expected, ?string $input = null): void {
+        $this->fixer->configure([
+            MultilineIfStatementBracesFixer::C_KEEP_ON_OWN_LINE => false,
+        ]);
+        $this->doTest($expected, $input);
+    }
+
+    public function provideInvertedFixCases(): iterable {
+        foreach ($this->provideFixCases() as $name => $case) {
+            yield $name => [$case[1], $case[0]];
+        }
+    }
+
+    protected function createFixer(): AbstractFixer {
+        return new MultilineIfStatementBracesFixer();
+    }
+
+}