gurucomkz / silverstripe-eagerloading

src/DataFilterEagerLoadingExtension.php

Braces +3 added lines, -1 removed lines

@@ -16,7 +16,9 @@
         if(!isset($this->owner->withList)){
             $this->owner->withList = [];
         }
-        if(!is_array($list)) $list = [$list];
+        if(!is_array($list)) {
+            $list = [$list];
+        }
         $list = array_map(function($e){return explode('.',$e); },$list);
         $this->owner->withList = array_merge($this->owner->withList,$list);
         return EagerLoadedDataList::cloneFrom($this->owner);

src/GridFieldEagerLoadManipulator.php

Braces +6 added lines, -2 removed lines

@@ -40,7 +40,9 @@ 
                 $export_eager = [];
                 foreach($export_fields as $field=>$_title) {
                     $parts = explode('.',$field);
-                    if(count($parts) < 2) continue;
+                    if(count($parts) < 2) {
+                        continue;
+                    }
                     $main = implode('.', array_slice($parts,0,-1));
                     if(!in_array($main,$eager)){
                         $eager[] = $main;
@@ -48,7 +50,9 @@ 
                 }
             }
         }
-        if($eager) return $dataList->with($eager);
+        if($eager) {
+            return $dataList->with($eager);
+        }
         return $dataList;
     }
 }

src/EagerLoadedDataList.php

Braces +17 added lines, -6 removed lines

@@ -52,7 +52,9 @@ 
     {
         $result = [];
         foreach($arr as $k => $v){
-            if(!isset($result[$v])) $result[$v] = [];
+            if(!isset($result[$v])) {
+                $result[$v] = [];
+            }
             $result[$v][] = $k;
         }
         return $result;
@@ -76,7 +78,9 @@ 
     private $relationsPrepared = false;
 
     public function prepareEagerRelations() {
-        if($this->relationsPrepared) return;
+        if($this->relationsPrepared) {
+            return;
+        }
         $this->relationsPrepared = true;
         $localClass = $this->dataClass();
         $config = Config::forClass($localClass);
@@ -224,7 +228,9 @@ 
             foreach($idsQuery as $row){
                 $relID = $row[$component['childField']];
                 $localID = $row[$component['parentField']];
-                if(!isset($collection[$localID])) $collection[$localID] = [];
+                if(!isset($collection[$localID])) {
+                    $collection[$localID] = [];
+                }
                 $collection[$localID][] = $relID;
                 $relListReverted[$relID] = 1;//use ids as keys to avoid
             }
@@ -326,13 +332,18 @@ 
 
     public static function EnsureArray($arr, $kfield = null)
     {
-        if(is_array($arr)) return $arr;
+        if(is_array($arr)) {
+            return $arr;
+        }
         $result = [];
         foreach($arr as $k => $v){
             $key = $k;
             if($kfield!==null) {
-                if(is_array($v) && isset($v[$kfield])) $key = $v[$kfield];
-                elseif(is_object($v) && isset($v->$kfield)) $key = $v->$kfield;
+                if(is_array($v) && isset($v[$kfield])) {
+                    $key = $v[$kfield];
+                } elseif(is_object($v) && isset($v->$kfield)) {
+                    $key = $v->$kfield;
+                }
             }
             $result[$key] = $v;
         }