Merge branch '4.3' into 4.4

src/Core/CoreKernel.php

@@ -291,18 +291,19 @@ class CoreKernel implements Kernel
 
         // Build error response
         $dv = new DebugView();
-        $body =
-            $dv->renderHeader() .
+        $body = implode([
+            $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

@@ -2112,8 +2112,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] =
-                    new UnsavedRelationList(
+                $this->unsavedRelations[$componentName] = new UnsavedRelationList(
                     $manyManyComponent['parentClass'],
                     $componentName,
                     $manyManyComponent['childClass']

src/Security/PermissionCheckboxSetField.php

@@ -268,15 +268,15 @@ class PermissionCheckboxSetField extends FormField
             }
         }
         if ($this->readonly) {
-            return
-                "<ul id=\"{$this->ID()}\" class=\"optionset checkboxsetfield{$this->extraClass()}\">\n" .
-                "<li class=\"help\">" .
-                _t(
+            $message = _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

@@ -42,13 +42,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'])
-                && preg_match(
+            if (!isset($backtraceItem['file'])) {
+                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

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