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

src/EagerLoadedDataList.php

Spacing +11 added lines, -11 removed lines

@@ -63,7 +63,7 @@ 
     {
         return array_filter(
             $this->withList,
-            function ($dep) use ($list) {
+            function($dep) use ($list) {
                 return array_key_exists($dep[0], $list);
             }
         );
@@ -90,7 +90,7 @@ 
 
         if (!count($withHasOnes) && !count($withHasManys) && !count($withManyManys) && !count($withBelongsManyManys)) {
             throw new EagerLoadingException(
-                "Invalid names supplied for ->with(" . implode(', ', $this->withListOriginal) . ")"
+                "Invalid names supplied for ->with(".implode(', ', $this->withListOriginal).")"
             );
         }
 
@@ -124,7 +124,7 @@ 
             $fields[] = "{$dep}ID";
         }
         $table = Config::forClass($this->dataClass)->get('table_name');
-        $data = new SQLSelect(implode(',', $fields), [$table], ["ID IN (" . implode(',', $ids) . ")"]);
+        $data = new SQLSelect(implode(',', $fields), [$table], ["ID IN (".implode(',', $ids).")"]);
         $data = Utils::EnsureArray($data->execute(), 'ID');
 
         foreach ($withHasOnes as $depSeq) {
@@ -142,7 +142,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)));
                 }
@@ -173,7 +173,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)));
             }
@@ -212,7 +212,7 @@ 
                 implode(',', [$component['childField'], $component['parentField']]),
                 $component['join'],
                 [
-                    $component['parentField'] . ' IN (' . implode(',', $data) . ')'
+                    $component['parentField'].' IN ('.implode(',', $data).')'
                 ]
             )->execute();
 
@@ -229,7 +229,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)));
             }
@@ -269,8 +269,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"
                 );
             }
@@ -292,8 +292,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"
                 );
             }

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