Merge pull request #6678 from paulbalandan/no-useless-concat-operator

Enable `no_useless_concat_operator`

Author: kenjis

.php-cs-fixer.dist.php

@@ -38,7 +38,11 @@
         __DIR__ . '/spark',
     ]);
 
-$overrides = [];
+$overrides = [
+    'no_useless_concat_operator' => [
+        'juggle_simple_strings' => true,
+    ],
+];
 
 $options = [
     'cacheFile'    => 'build/.php-cs-fixer.cache',

.php-cs-fixer.no-header.php

@@ -30,7 +30,11 @@
         __DIR__ . '/admin/starter/builds',
     ]);
 
-$overrides = [];
+$overrides = [
+    'no_useless_concat_operator' => [
+        'juggle_simple_strings' => true,
+    ],
+];
 
 $options = [
     'cacheFile'    => 'build/.php-cs-fixer.no-header.cache',

.php-cs-fixer.user-guide.php

@@ -33,6 +33,9 @@
     'php_unit_internal_class'     => false,
     'no_unused_imports'           => false,
     'class_attributes_separation' => false,
+    'no_useless_concat_operator'  => [
+        'juggle_simple_strings' => true,
+    ],
 ];
 
 $options = [

system/Database/BaseBuilder.php

@@ -1179,7 +1179,7 @@ public function unionAll($union)
      */
     protected function addUnionStatement($union, bool $all = false)
     {
-        $this->QBUnion[] = "\n" . 'UNION '
+        $this->QBUnion[] = "\nUNION "
             . ($all ? 'ALL ' : '')
             . 'SELECT * FROM '
             . $this->buildSubquery($union, true, 'uwrp' . (count($this->QBUnion) + 1));

system/Database/MySQLi/Connection.php

@@ -360,7 +360,7 @@ public function escapeLikeStringDirect($str)
         // Escape LIKE condition wildcards
         return str_replace(
             [$this->likeEscapeChar, '%', '_'],
-            ['\\' . $this->likeEscapeChar, '\\' . '%', '\\' . '_'],
+            ['\\' . $this->likeEscapeChar, '\\%', '\\_'],
             $str
         );
     }

system/Debug/Toolbar.php

@@ -386,7 +386,7 @@ public function prepare(?RequestInterface $request = null, ?ResponseInterface $r
                 mkdir(WRITEPATH . 'debugbar', 0777);
             }
 
-            write_file(WRITEPATH . 'debugbar/' . 'debugbar_' . $time . '.json', $data, 'w+');
+            write_file(WRITEPATH . 'debugbar/debugbar_' . $time . '.json', $data, 'w+');
 
             $format = $response->getHeaderLine('content-type');
 

system/Helpers/form_helper.php

@@ -484,13 +484,13 @@ function form_datalist(string $name, string $value, array $options): string
 
         $out = form_input($data) . "\n";
 
-        $out .= "<datalist id='" . $name . '_list' . "'>";
+        $out .= "<datalist id='" . $name . "_list'>";
 
         foreach ($options as $option) {
-            $out .= "<option value='{$option}'>" . "\n";
+            $out .= "<option value='{$option}'>\n";
         }
 
-        return $out . ('</datalist>' . "\n");
+        return $out . ("</datalist>\n");
     }
 }
 

system/View/Parser.php

@@ -277,7 +277,7 @@ protected function parsePair(string $variable, array $data, string $template): a
         // have something to loop over.
         preg_match_all(
             '#' . $this->leftDelimiter . '\s*' . preg_quote($variable, '#') . '\s*' . $this->rightDelimiter . '(.+?)' .
-            $this->leftDelimiter . '\s*' . '/' . preg_quote($variable, '#') . '\s*' . $this->rightDelimiter . '#s',
+            $this->leftDelimiter . '\s*/' . preg_quote($variable, '#') . '\s*' . $this->rightDelimiter . '#s',
             $template,
             $matches,
             PREG_SET_ORDER

tests/system/HTTP/MessageTest.php

@@ -156,7 +156,7 @@ public function testAppendBody()
 
         $this->message->appendBody('\n');
 
-        $this->assertSame('moo' . '\n', $this->message->getBody());
+        $this->assertSame('moo\n', $this->message->getBody());
     }
 
     public function testSetHeaderReplacingHeader()

tests/system/Images/GDHandlerTest.php

@@ -413,7 +413,7 @@ public function testImageConvert()
     public function testImageReorientLandscape()
     {
         for ($i = 0; $i <= 8; $i++) {
-            $source = $this->origin . 'EXIFsamples/landscape_' . '' . $i . '.jpg';
+            $source = $this->origin . 'EXIFsamples/landscape_' . $i . '.jpg';
 
             $this->handler->withFile($source);
             $this->handler->reorient();
@@ -429,7 +429,7 @@ public function testImageReorientLandscape()
     public function testImageReorientPortrait()
     {
         for ($i = 0; $i <= 8; $i++) {
-            $source = $this->origin . 'EXIFsamples/portrait_' . '' . $i . '.jpg';
+            $source = $this->origin . 'EXIFsamples/portrait_' . $i . '.jpg';
 
             $this->handler->withFile($source);
             $this->handler->reorient();