gurucomkz / silverstripe-eagerloading

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) {
@@ -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)));
                 }
@@ -179,7 +179,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)));
             }
@@ -220,8 +220,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);
@@ -238,12 +238,12 @@ 
 
             $idsQuery = SQLSelect::create(
                 [
-                    '"' . $childField . '"',
-                    '"' . $parentField . '"',
+                    '"'.$childField.'"',
+                    '"'.$parentField.'"',
                 ],
-                '"' . $table . '"',
+                '"'.$table.'"',
                 [
-                    '"' . $parentField . '" IN (' . implode(',', $data) . ')'
+                    '"'.$parentField.'" IN ('.implode(',', $data).')'
                 ]
             )->execute();
 
@@ -261,7 +261,7 @@ 
 
             if (count($relListReverted)) {
                 $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)));
                 }
@@ -302,8 +302,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"
                 );
             }
@@ -325,8 +325,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"
                 );
             }