Level-2 / Maphper

Maphper/DataSource/DatabaseSelect.php

Indentation +64 added lines, -64 removed lines

@@ -2,48 +2,48 @@ 
 namespace Maphper\DataSource;
 
 class DatabaseSelect {
-    private $resultCache = [];
-    private $idCache = [];
-    private $selectBuilder;
-    private $whereBuilder;
-    private $adapter;
-    private $databaseModify;
-    private $defaultSort;
-    private $table;
-
-    public function __construct(DatabaseAdapter $adapter, DatabaseModify $databaseModify, $table, $defaultSort, $cacheMode) {
-        $this->adapter = $adapter;
-        $this->databaseModify = $databaseModify;
-        $this->selectBuilder = new \Maphper\Lib\SelectBuilder();
-        $this->whereBuilder = new \Maphper\Lib\Sql\WhereBuilder();
-        $this->defaultSort = $defaultSort;
-        $this->cacheMode = $cacheMode;
-        $this->table = $table;
-    }
-
-    public function findById($id, $pk) {
+	private $resultCache = [];
+	private $idCache = [];
+	private $selectBuilder;
+	private $whereBuilder;
+	private $adapter;
+	private $databaseModify;
+	private $defaultSort;
+	private $table;
+
+	public function __construct(DatabaseAdapter $adapter, DatabaseModify $databaseModify, $table, $defaultSort, $cacheMode) {
+		$this->adapter = $adapter;
+		$this->databaseModify = $databaseModify;
+		$this->selectBuilder = new \Maphper\Lib\SelectBuilder();
+		$this->whereBuilder = new \Maphper\Lib\Sql\WhereBuilder();
+		$this->defaultSort = $defaultSort;
+		$this->cacheMode = $cacheMode;
+		$this->table = $table;
+	}
+
+	public function findById($id, $pk) {
 		if (($this->cacheMode && !isset($this->idCache[$id])) || !$this->cacheMode) {
 			try {
 				$result = $this->selectQuery($this->selectBuilder->select($this->table, $pk . ' = :id', [':id' => $id], ['limit' => 1]));
 			}
 			catch (\Exception $e) {
-                // Don't issue an error if it cannot be found since we return null
+				// Don't issue an error if it cannot be found since we return null
 			}
 
 			if (isset($result[0])) $result = $result[0];
 			else return null;
 
-            if (!$this->cacheMode) return $result;
-            else return $this->idCache[$id] = $result;
+			if (!$this->cacheMode) return $result;
+			else return $this->idCache[$id] = $result;
 		}
 		// cacheMode is true and cache is set
-        return $this->idCache[$id];
+		return $this->idCache[$id];
 	}
 
-    public function findByField(array $fields, $options = []) {
+	public function findByField(array $fields, $options = []) {
 		$cacheId = md5(serialize(func_get_args()));
 
-    if (($this->cacheMode && !isset($this->resultCache[$cacheId])) || !$this->cacheMode) {
+	if (($this->cacheMode && !isset($this->resultCache[$cacheId])) || !$this->cacheMode) {
 			$query = $this->whereBuilder->createSql($fields);
 
 			if (!isset($options['order'])) $options['order'] = $this->defaultSort;
@@ -59,15 +59,15 @@ 
 			}
 		}
 
-    if ($this->cacheMode) {
-      if (isset($result)) $this->resultCache[$cacheId] = $result;
-      if (isset($this->resultCache[$cacheId])) return $this->resultCache[$cacheId];
-    }
+	if ($this->cacheMode) {
+	  if (isset($result)) $this->resultCache[$cacheId] = $result;
+	  if (isset($this->resultCache[$cacheId])) return $this->resultCache[$cacheId];
+	}
 
-    return $result;
+	return $result;
 	}
 
-    public function findAggregate($function, $field, $group = null, array $criteria = [], array $options = []) {
+	public function findAggregate($function, $field, $group = null, array $criteria = [], array $options = []) {
 		//Cannot count/sum/max multiple fields, pick the first one. This should only come into play when trying to count() a mapper with multiple primary keys
 		if (is_array($field)) $field = $field[0];
 		$query = $this->whereBuilder->createSql($criteria);
@@ -84,36 +84,36 @@ 
 		}
 	}
 
-    private function determineAggregateResult($result, $group, $field) {
-        if ($group != null) {
-            $ret = [];
-            foreach ($result as $res) $ret[$res->$field] = $res->val;
-            return $ret;
-        }
-        else if (isset($result[0])) return $result[0]->val;
-        else return 0;
-    }
-
-    private function selectQuery(\Maphper\Lib\Query $query) {
-        return $this->adapter->query($query)->fetchAll(\PDO::FETCH_OBJ);
-    }
-
-    public function clearResultCache() {
-        if ($this->cacheMode) $this->resultCache = [];
-    }
-
-    public function clearIDCache() {
-        if ($this->cacheMode) $this->idCache = [];
-    }
-
-    public function updateCache($data, $pkValue) {
-        if ($this->cacheMode) {
-  		    if (isset($this->cache[$pkValue])) $this->cache[$pkValue] = (object) array_merge((array)$this->cache[$pkValue], (array)$data);
-  		    else $this->cache[$pkValue] = $data;
-        }
-    }
-
-    public function deleteIDFromCache($id) {
-        if ($this->cacheMode) unset($this->idCache[$id]);
-    }
+	private function determineAggregateResult($result, $group, $field) {
+		if ($group != null) {
+			$ret = [];
+			foreach ($result as $res) $ret[$res->$field] = $res->val;
+			return $ret;
+		}
+		else if (isset($result[0])) return $result[0]->val;
+		else return 0;
+	}
+
+	private function selectQuery(\Maphper\Lib\Query $query) {
+		return $this->adapter->query($query)->fetchAll(\PDO::FETCH_OBJ);
+	}
+
+	public function clearResultCache() {
+		if ($this->cacheMode) $this->resultCache = [];
+	}
+
+	public function clearIDCache() {
+		if ($this->cacheMode) $this->idCache = [];
+	}
+
+	public function updateCache($data, $pkValue) {
+		if ($this->cacheMode) {
+  			if (isset($this->cache[$pkValue])) $this->cache[$pkValue] = (object) array_merge((array)$this->cache[$pkValue], (array)$data);
+  			else $this->cache[$pkValue] = $data;
+		}
+	}
+
+	public function deleteIDFromCache($id) {
+		if ($this->cacheMode) unset($this->idCache[$id]);
+	}
 }

Braces +46 added lines, -22 removed lines

@@ -25,16 +25,21 @@ 
 		if (($this->cacheMode && !isset($this->idCache[$id])) || !$this->cacheMode) {
 			try {
 				$result = $this->selectQuery($this->selectBuilder->select($this->table, $pk . ' = :id', [':id' => $id], ['limit' => 1]));
-			}
-			catch (\Exception $e) {
+			} catch (\Exception $e) {
                 // Don't issue an error if it cannot be found since we return null
 			}
 
-			if (isset($result[0])) $result = $result[0];
-			else return null;
+			if (isset($result[0])) {
+				$result = $result[0];
+			} else {
+				return null;
+			}
 
-            if (!$this->cacheMode) return $result;
-            else return $this->idCache[$id] = $result;
+            if (!$this->cacheMode) {
+            	return $result;
+            } else {
+            	return $this->idCache[$id] = $result;
+            }
 		}
 		// cacheMode is true and cache is set
         return $this->idCache[$id];
@@ -46,22 +51,27 @@ 
     if (($this->cacheMode && !isset($this->resultCache[$cacheId])) || !$this->cacheMode) {
 			$query = $this->whereBuilder->createSql($fields);
 
-			if (!isset($options['order'])) $options['order'] = $this->defaultSort;
+			if (!isset($options['order'])) {
+				$options['order'] = $this->defaultSort;
+			}
 
 			try {
 				$result = $this->selectQuery($this->selectBuilder->select($this->table, $query['sql'], $query['args'], $options));
 				$this->databaseModify->addIndex(array_keys($query['args']));
 				$this->databaseModify->addIndex(explode(',', $options['order']));
-			}
-			catch (\Exception $e) {
+			} catch (\Exception $e) {
 				$this->errors[] = $e;
 				$result = [];
 			}
 		}
 
     if ($this->cacheMode) {
-      if (isset($result)) $this->resultCache[$cacheId] = $result;
-      if (isset($this->resultCache[$cacheId])) return $this->resultCache[$cacheId];
+      if (isset($result)) {
+      	$this->resultCache[$cacheId] = $result;
+      }
+      if (isset($this->resultCache[$cacheId])) {
+      	return $this->resultCache[$cacheId];
+      }
     }
 
     return $result;
@@ -69,7 +79,9 @@ 
 
     public function findAggregate($function, $field, $group = null, array $criteria = [], array $options = []) {
 		//Cannot count/sum/max multiple fields, pick the first one. This should only come into play when trying to count() a mapper with multiple primary keys
-		if (is_array($field)) $field = $field[0];
+		if (is_array($field)) {
+			$field = $field[0];
+		}
 		$query = $this->whereBuilder->createSql($criteria);
 
 		try {
@@ -78,8 +90,7 @@ 
 			$result = $this->selectQuery($this->selectBuilder->aggregate($this->table, $function, $field, $query['sql'], $query['args'], $group));
 
 			return $this->determineAggregateResult($result, $group, $field);
-		}
-		catch (\Exception $e) {
+		} catch (\Exception $e) {
 			return $group ? [] : 0;
 		}
 	}
@@ -87,11 +98,15 @@ 
     private function determineAggregateResult($result, $group, $field) {
         if ($group != null) {
             $ret = [];
-            foreach ($result as $res) $ret[$res->$field] = $res->val;
+            foreach ($result as $res) {
+            	$ret[$res->$field] = $res->val;
+            }
             return $ret;
+        } else if (isset($result[0])) {
+        	return $result[0]->val;
+        } else {
+        	return 0;
         }
-        else if (isset($result[0])) return $result[0]->val;
-        else return 0;
     }
 
     private function selectQuery(\Maphper\Lib\Query $query) {
@@ -99,21 +114,30 @@ 
     }
 
     public function clearResultCache() {
-        if ($this->cacheMode) $this->resultCache = [];
+        if ($this->cacheMode) {
+        	$this->resultCache = [];
+        }
     }
 
     public function clearIDCache() {
-        if ($this->cacheMode) $this->idCache = [];
+        if ($this->cacheMode) {
+        	$this->idCache = [];
+        }
     }
 
     public function updateCache($data, $pkValue) {
         if ($this->cacheMode) {
-  		    if (isset($this->cache[$pkValue])) $this->cache[$pkValue] = (object) array_merge((array)$this->cache[$pkValue], (array)$data);
-  		    else $this->cache[$pkValue] = $data;
+  		    if (isset($this->cache[$pkValue])) {
+  		    	$this->cache[$pkValue] = (object) array_merge((array)$this->cache[$pkValue], (array)$data);
+  		    } else {
+  		    	$this->cache[$pkValue] = $data;
+  		    }
         }
     }
 
     public function deleteIDFromCache($id) {
-        if ($this->cacheMode) unset($this->idCache[$id]);
+        if ($this->cacheMode) {
+        	unset($this->idCache[$id]);
+        }
     }
 }