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/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);

tests/ProxyDBCounterExtension.php

Spacing +2 added lines, -2 removed lines

@@ -20,10 +20,10 @@
     protected static $queries = [];
 
 
-    public function updateProxy(ProxyGenerator &$proxy)
+    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/EagerloadTest.php

Spacing +1 added lines, -1 removed lines

@@ -114,7 +114,7 @@
         foreach ($teams as $team) {
             $supporters = $team->Supporters()->map()->toArray();
             // print_r(ProxyDBCounterExtension::getQueries());
-            switch($team->Title){
+            switch ($team->Title) {
                 case 'The Hurricanes':
                     $this->assertCount(3, $supporters);
                     $this->assertTrue(in_array('Supporter 1', $supporters));

src/EagerLoadedDataList.php

Spacing +15 added lines, -15 removed lines

@@ -65,7 +65,7 @@ 
     {
         return array_filter(
             $this->withList,
-            function ($dep) use ($list) {
+            function($dep) use ($list) {
                 return array_key_exists($dep[0], $list);
             }
         );
@@ -124,7 +124,7 @@ 
             $fields[] = "\"{$dep}ID\"";
         }
         $table = DataObject::getSchema()->tableName($this->dataClass);
-        $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) {
@@ -143,7 +143,7 @@ 
                 $IDsubset = array_splice($uniqueIDs, 0, self::ID_LIMIT);
                 $result = DataObject::get($depClass)->filter('ID', $IDsubset);
                 $this->extend('onEagerLoadHasOneMany', $result, $data, $depSeq, $withHasOnes);
-                if (count($depSeq)>1) {
+                if (count($depSeq) > 1) {
                     $result = $result
                         ->with(implode('.', array_slice($depSeq, 1)));
                 }
@@ -181,7 +181,7 @@ 
             ];
             $result = DataObject::get($depClass)->filter($depKey, $data);
             $this->extend('onEagerLoadHasMany', $result, $depKey, $data, $depSeq, $withHasManys);
-            if (count($depSeq)>1) {
+            if (count($depSeq) > 1) {
                 $result = $result
                     ->with(implode('.', array_slice($depSeq, 1)));
             }
@@ -222,8 +222,8 @@ 
 
                 $table = DataObject::getSchema()->tableName($throughClass);
 
-                $childField = $depData['to']. 'ID';
-                $parentField = $depData['from']. 'ID';
+                $childField = $depData['to'].'ID';
+                $parentField = $depData['from'].'ID';
             } else {
                 $depClass = $depData;
                 $component = $schema->manyManyComponent($localClass, $dep);
@@ -240,12 +240,12 @@ 
 
             $idsQuery = SQLSelect::create(
                 [
-                    '"' . $childField . '"',
-                    '"' . $parentField . '"',
+                    '"'.$childField.'"',
+                    '"'.$parentField.'"',
                 ],
-                '"' . $table . '"',
+                '"'.$table.'"',
                 [
-                    '"' . $parentField . '" IN (' . implode(',', $data) . ')'
+                    '"'.$parentField.'" IN ('.implode(',', $data).')'
                 ]
             )->execute();
 
@@ -264,7 +264,7 @@ 
             if (count($relListReverted)) {
                 $result = DataObject::get($depClass)->filter('ID', array_keys($relListReverted));
                 $this->extend('onEagerLoadManyMany', $result, $data, $depSeq, $withManyManys);
-                if (count($depSeq)>1) {
+                if (count($depSeq) > 1) {
                     $result = $result
                         ->with(implode('.', array_slice($depSeq, 1)));
                 }
@@ -305,8 +305,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"
                 );
             }
@@ -328,8 +328,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"
                 );
             }