gurucomkz / silverstripe-eagerloading

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