Merge pull request #9293 from open-sausages/pulls/4.3/psr2-fix

PSR2 linting fixes

src/Core/CoreKernel.php

@@ -285,18 +285,19 @@ class CoreKernel implements Kernel
 
         // Build error response
         $dv = new DebugView();
-        $body =
+        $body = implode([
-            $dv->renderHeader() .
+            $dv->renderHeader(),
             $dv->renderInfo(
                 "Configuration Error",
                 Director::absoluteBaseURL()
-            ) .
+            ),
             $dv->renderParagraph(
                 'You need to replace your _ss_environment.php file with a .env file, or with environment variables.<br><br>'
                 . 'See the <a href="https://docs.silverstripe.org/en/4/getting_started/environment_management/">'
                 . 'Environment Management</a> docs for more information.'
-            ) .
+            ),
-            $dv->renderFooter();
+            $dv->renderFooter()
+        ]);
 
         // Raise error
         $response = new HTTPResponse($body, 500);

src/ORM/DataObject.php

@@ -2102,8 +2102,7 @@ class DataObject extends ViewableData implements DataObjectInterface, i18nEntity
         // If we haven't been written yet, we can't save these relations, so use a list that handles this case
         if (!$id) {
             if (!isset($this->unsavedRelations[$componentName])) {
-                $this->unsavedRelations[$componentName] =
+                $this->unsavedRelations[$componentName] = new UnsavedRelationList(
-                    new UnsavedRelationList(
                     $manyManyComponent['parentClass'],
                     $componentName,
                     $manyManyComponent['childClass']

src/Security/PermissionCheckboxSetField.php

@@ -268,15 +268,15 @@ class PermissionCheckboxSetField extends FormField
             }
         }
         if ($this->readonly) {
-            return
+            $message = _t(
-                "<ul id=\"{$this->ID()}\" class=\"optionset checkboxsetfield{$this->extraClass()}\">\n" .
-                "<li class=\"help\">" .
-                _t(
                 'SilverStripe\\Security\\Permission.UserPermissionsIntro',
                 'Assigning groups to this user will adjust the permissions they have.'
                 . ' See the groups section for details of permissions on individual groups.'
-                ) .
+            );
-                "</li>" .
+
+            return
+                "<ul id=\"{$this->ID()}\" class=\"optionset checkboxsetfield{$this->extraClass()}\">\n" .
+                "<li class=\"help\">" . $message . "</li>" .
                 $options .
                 "</ul>\n";
         } else {

src/includes/constants.php

@@ -41,13 +41,17 @@ if (!defined('BASE_PATH')) {
     define('BASE_PATH', call_user_func(function () {
         // Determine BASE_PATH based on the composer autoloader
         foreach (debug_backtrace() as $backtraceItem) {
-            if (isset($backtraceItem['file'])
+            if (!isset($backtraceItem['file'])) {
-                && preg_match(
+                continue;
+            }
+
+            $matched = preg_match(
                 '#^(?<base>.*)(/|\\\\)vendor(/|\\\\)composer(/|\\\\)autoload_real.php#',
                 $backtraceItem['file'],
                 $matches
-                )
+            );
-            ) {
+
+            if ($matched) {
                 return realpath($matches['base']) ?: DIRECTORY_SEPARATOR;
             }
         }

tests/php/ORM/DataListTest.php

@@ -1039,11 +1039,9 @@ class DataListTest extends SapphireTest
             ->leftJoin(
                 'DataObjectTest_Team',
                 '"DataObjectTest_Team"."ID" = "DataObjectTest_TeamComment"."TeamID"'
-            )->filter(
+            )->filter([
-                array(
                 'Title' => 'Team 1'
-                )
+            ]);
-            );
 
         $this->assertEquals(2, $list->count());
         $values = $list->column('Name');