gurucomkz / silverstripe-eagerloading

src/DataFilterEagerLoadingExtension.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
         }
         $this->owner->withListOriginal = $list;
         $list = array_map(
-            function ($e) {
+            function($e) {
                 return explode('.', $e);
             },
             $list

tests/ProxyDBCounterExtension.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
     public function updateProxy(ProxyGenerator &$proxy)
     {
         // In the closure, $this is the proxied database
-        $callback = function ($args, $next) {
+        $callback = function($args, $next) {
 
             // The first argument is always the sql query
             $sql = $args[0];

tests/EagerloadAdminTest.php

Spacing +5 added lines, -5 removed lines

@@ -37,7 +37,7 @@ 
     {
         $this->autoFollowRedirection = false;
         $this->logInAs('admin');
-        $this->assertTrue((bool)Permission::check("ADMIN"));
+        $this->assertTrue((bool) Permission::check("ADMIN"));
         $this->assertEquals(200, $this->get('admin/eltestadmin')->getStatusCode());
     }
 
@@ -66,14 +66,14 @@ 
         $exportActionStateID = $this->getActionSessionKey('export');
         $exportActionText = _t('SilverStripe\\Forms\\GridField\\GridField.CSVEXPORT', 'Export to CSV');
         $getVars = [
-            'action_gridFieldAlterAction?StateID=' . $exportActionStateID => $exportActionText,
+            'action_gridFieldAlterAction?StateID='.$exportActionStateID => $exportActionText,
         ];
 
         $this->autoFollowRedirection = false;
         $this->logInAs('admin');
-        $this->assertTrue((bool)Permission::check("ADMIN"));
+        $this->assertTrue((bool) Permission::check("ADMIN"));
         $modelClassName = "Gurucomkz-EagerLoading-Tests-Models-Player";
-        $getLink = "admin/eltestadmin/$modelClassName/EditForm/field/$modelClassName?" . http_build_query($getVars);
+        $getLink = "admin/eltestadmin/$modelClassName/EditForm/field/$modelClassName?".http_build_query($getVars);
 
         $this->assertEquals(200, $this->get($getLink)->getStatusCode());
     }
@@ -90,7 +90,7 @@ 
         preg_match_all('/"action_gridFieldAlterAction\?StateID=([^"]+)"/', $body, $matches);
 
         /** @var StateStore $store */
-        $store = Injector::inst()->create(StateStore::class . '.' . 'Gurucomkz-EagerLoading-Tests-Models-Player');
+        $store = Injector::inst()->create(StateStore::class.'.'.'Gurucomkz-EagerLoading-Tests-Models-Player');
 
         foreach ($matches[1] as $actionCode) {
             $actionData = $store->load($actionCode);

src/EagerLoadedDataList.php

Spacing +13 added lines, -13 removed lines

@@ -64,7 +64,7 @@ 
     {
         return array_filter(
             $this->withList,
-            function ($dep) use ($list) {
+            function($dep) use ($list) {
                 return array_key_exists($dep[0], $list);
             }
         );
@@ -123,7 +123,7 @@ 
             $fields[] = "\"{$dep}ID\"";
         }
         $table = Config::forClass($this->dataClass)->get('table_name');
-        $data = new SQLSelect($fields, '"' . $table . '"', ['"ID" IN (' . implode(',', $ids) . ')']);
+        $data = new SQLSelect($fields, '"'.$table.'"', ['"ID" IN ('.implode(',', $ids).')']);
         $data = Utils::EnsureArray($data->execute(), 'ID');
 
         foreach ($withHasOnes as $depSeq) {
@@ -141,7 +141,7 @@ 
             while (count($uniqueIDs)) {
                 $IDsubset = array_splice($uniqueIDs, 0, self::ID_LIMIT);
                 $result = DataObject::get($depClass)->filter('ID', $IDsubset);
-                if (count($depSeq)>1) {
+                if (count($depSeq) > 1) {
                     $result = $result
                         ->with(implode('.', array_slice($depSeq, 1)));
                 }
@@ -172,7 +172,7 @@ 
                 'map' => [],
             ];
             $result = DataObject::get($depClass)->filter($depKey, $data);
-            if (count($depSeq)>1) {
+            if (count($depSeq) > 1) {
                 $result = $result
                     ->with(implode('.', array_slice($depSeq, 1)));
             }
@@ -209,12 +209,12 @@ 
 
             $idsQuery = SQLSelect::create(
                 [
-                    '"' . $component['childField'] . '"',
-                    '"' . $component['parentField'] . '"',
+                    '"'.$component['childField'].'"',
+                    '"'.$component['parentField'].'"',
                 ],
-                '"' . $component['join'] . '"',
+                '"'.$component['join'].'"',
                 [
-                    '"' . $component['parentField'] . '" IN (' . implode(',', $data) . ')'
+                    '"'.$component['parentField'].'" IN ('.implode(',', $data).')'
                 ]
             )->execute();
 
@@ -231,7 +231,7 @@ 
             }
 
             $result = DataObject::get($depClass)->filter('ID', array_keys($relListReverted));
-            if (count($depSeq)>1) {
+            if (count($depSeq) > 1) {
                 $result = $result
                     ->with(implode('.', array_slice($depSeq, 1)));
             }
@@ -271,8 +271,8 @@ 
             }
             if (!method_exists($item, 'addEagerRelation')) {
                 throw new EagerLoadingException(
-                    "Model {$item->ClassName} must include " .
-                    EagerLoaderMultiAccessor::class .
+                    "Model {$item->ClassName} must include ".
+                    EagerLoaderMultiAccessor::class.
                     " trait to use eager loading for \$has_many"
                 );
             }
@@ -294,8 +294,8 @@ 
             }
             if (!method_exists($item, 'addEagerRelation')) {
                 throw new EagerLoadingException(
-                    "Model {$item->ClassName} must include " .
-                    EagerLoaderMultiAccessor::class .
+                    "Model {$item->ClassName} must include ".
+                    EagerLoaderMultiAccessor::class.
                     " trait to use eager loading for \$many_many"
                 );
             }