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