Passed
Pull Request — master (#70)
by Christian
01:42 queued 19s
created
Maphper/DataSource/Database.php 1 patch
Indentation   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -9,8 +9,8 @@  discard block
 block discarded – undo
9 9
 	private $table;
10 10
 
11 11
 	private $fields = '*';
12
-    private $databaseSelect;
13
-    private $databaseCrud;
12
+	private $databaseSelect;
13
+	private $databaseCrud;
14 14
 
15 15
 	public function __construct($db, $table, $primaryKey = 'id', array $options = []) {
16 16
 		$options = new DatabaseOptions($db, $options);
@@ -23,10 +23,10 @@  discard block
 block discarded – undo
23 23
 
24 24
 		$defaultSort = $options->read('defaultSort') !== false ? $options->read('defaultSort')  : implode(', ', $this->primaryKey);
25 25
 
26
-        $databaseModify = new DatabaseModify($adapter, $options->getEditMode(), $table);
26
+		$databaseModify = new DatabaseModify($adapter, $options->getEditMode(), $table);
27 27
 
28
-        $this->databaseSelect = new DatabaseSelect($adapter, $databaseModify, $table, $defaultSort, $options->getCacheMode());
29
-        $this->databaseCrud = new DatabaseCrud($adapter, $databaseModify, $this->databaseSelect, $table, $this->primaryKey);
28
+		$this->databaseSelect = new DatabaseSelect($adapter, $databaseModify, $table, $defaultSort, $options->getCacheMode());
29
+		$this->databaseCrud = new DatabaseCrud($adapter, $databaseModify, $this->databaseSelect, $table, $this->primaryKey);
30 30
 
31 31
 		$databaseModify->optimizeColumns();
32 32
 	}
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
 	}
53 53
 
54 54
 	public function findByField(array $fields, $options = []) {
55
-        return $this->databaseSelect->findByField($fields, $options);
55
+		return $this->databaseSelect->findByField($fields, $options);
56 56
 	}
57 57
 
58 58
 	public function deleteByField(array $fields, array $options = []) {
@@ -60,6 +60,6 @@  discard block
 block discarded – undo
60 60
 	}
61 61
 
62 62
 	public function save($data) {
63
-        $this->databaseCrud->save($data, true);
63
+		$this->databaseCrud->save($data, true);
64 64
 	}
65 65
 }
Please login to merge, or discard this patch.
Maphper/DataSource/DatabaseSelect.php 1 patch
Indentation   +83 added lines, -83 removed lines patch added patch discarded remove patch
@@ -2,41 +2,41 @@  discard block
 block discarded – undo
2 2
 namespace Maphper\DataSource;
3 3
 
4 4
 class DatabaseSelect {
5
-    private $resultCache = [];
6
-    private $idCache = [];
7
-    private $idCacheTime = [];
8
-    private $selectBuilder;
9
-    private $whereBuilder;
10
-    private $adapter;
11
-    private $databaseModify;
12
-    private $defaultSort;
13
-    private $table;
14
-
15
-    public function __construct(DatabaseAdapter $adapter, DatabaseModify $databaseModify, $table, $defaultSort, $cacheMode) {
16
-        $this->adapter = $adapter;
17
-        $this->databaseModify = $databaseModify;
18
-        $this->selectBuilder = new \Maphper\Lib\SelectBuilder();
19
-        $this->whereBuilder = new \Maphper\Lib\Sql\WhereBuilder();
20
-        $this->defaultSort = $defaultSort;
21
-        $this->cacheMode = $cacheMode;
22
-        $this->table = $table;
23
-    }
24
-
25
-    public function getColumns($table) {
26
-      return $this->adapter->getColumns($table);
27
-    }
28
-
29
-    private function cacheUpdateRequired($id) {
30
-      if ($this->cacheMode > 0) {
31
-        if (!isset($this->idCacheTime[$id])) return true; // Cache time has not been set for first time
32
-
33
-        if (time() - $this->idCacheTime[$id] > $this->cacheMode) return true; // Cache time has expired
34
-      }
35
-
36
-      if (!isset($this->idCache[$id])) return true; // Cache has not been set for first time
37
-
38
-      return false;
39
-    }
5
+	private $resultCache = [];
6
+	private $idCache = [];
7
+	private $idCacheTime = [];
8
+	private $selectBuilder;
9
+	private $whereBuilder;
10
+	private $adapter;
11
+	private $databaseModify;
12
+	private $defaultSort;
13
+	private $table;
14
+
15
+	public function __construct(DatabaseAdapter $adapter, DatabaseModify $databaseModify, $table, $defaultSort, $cacheMode) {
16
+		$this->adapter = $adapter;
17
+		$this->databaseModify = $databaseModify;
18
+		$this->selectBuilder = new \Maphper\Lib\SelectBuilder();
19
+		$this->whereBuilder = new \Maphper\Lib\Sql\WhereBuilder();
20
+		$this->defaultSort = $defaultSort;
21
+		$this->cacheMode = $cacheMode;
22
+		$this->table = $table;
23
+	}
24
+
25
+	public function getColumns($table) {
26
+	  return $this->adapter->getColumns($table);
27
+	}
28
+
29
+	private function cacheUpdateRequired($id) {
30
+	  if ($this->cacheMode > 0) {
31
+		if (!isset($this->idCacheTime[$id])) return true; // Cache time has not been set for first time
32
+
33
+		if (time() - $this->idCacheTime[$id] > $this->cacheMode) return true; // Cache time has expired
34
+	  }
35
+
36
+	  if (!isset($this->idCache[$id])) return true; // Cache has not been set for first time
37
+
38
+	  return false;
39
+	}
40 40
 
41 41
   public function findById($id, $pk) {
42 42
 		if ($this->cacheMode < 0 || $this->cacheUpdateRequired($id)) {
@@ -44,19 +44,19 @@  discard block
 block discarded – undo
44 44
 				$result = $this->selectQuery($this->selectBuilder->select($this->table, $pk . ' = :id', [':id' => $id], ['limit' => 1]));
45 45
 			}
46 46
 			catch (\Exception $e) {
47
-                // Don't issue an error if it cannot be found since we return null
47
+				// Don't issue an error if it cannot be found since we return null
48 48
 			}
49 49
 
50 50
 			if (isset($result[0])) $result = $result[0];
51 51
 			else return null;
52 52
 		}
53 53
 
54
-    if ($this->cacheMode < 0) return $result; // Cache mode is off
54
+	if ($this->cacheMode < 0) return $result; // Cache mode is off
55 55
 
56
-    if ($this->cacheUpdateRequired()) {
57
-      $this->idCache[$id] = $result;
58
-      $this->idCacheTime[$id] = time();
59
-    }
56
+	if ($this->cacheUpdateRequired()) {
57
+	  $this->idCache[$id] = $result;
58
+	  $this->idCacheTime[$id] = time();
59
+	}
60 60
 
61 61
 		return $this->idCache[$id];
62 62
 	}
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
   public function findByField(array $fields, $options = []) {
65 65
 		$cacheId = md5(serialize(func_get_args()));
66 66
 
67
-    if ($this->cacheMode < 0 || $this->cacheUpdateRequired($cacheId)) {
67
+	if ($this->cacheMode < 0 || $this->cacheUpdateRequired($cacheId)) {
68 68
 			$query = $this->whereBuilder->createSql($fields);
69 69
 
70 70
 			if (!isset($options['order'])) $options['order'] = $this->defaultSort;
@@ -80,17 +80,17 @@  discard block
 block discarded – undo
80 80
 			}
81 81
 		}
82 82
 
83
-    if ($this->cacheMode < 0) return $result; // Cache mode is off
83
+	if ($this->cacheMode < 0) return $result; // Cache mode is off
84 84
 
85
-    if ($this->cacheUpdateRequired($cacheId)) {
86
-      $this->idCache[$cacheId] = $result;
87
-      $this->idCacheTime[$cacheId] = time();
88
-    }
85
+	if ($this->cacheUpdateRequired($cacheId)) {
86
+	  $this->idCache[$cacheId] = $result;
87
+	  $this->idCacheTime[$cacheId] = time();
88
+	}
89 89
 
90 90
 		return $this->idCache[$cacheId];
91 91
 	}
92 92
 
93
-    public function findAggregate($function, $field, $group = null, array $criteria = [], array $options = []) {
93
+	public function findAggregate($function, $field, $group = null, array $criteria = [], array $options = []) {
94 94
 		//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
95 95
 		if (is_array($field)) $field = $field[0];
96 96
 		$query = $this->whereBuilder->createSql($criteria);
@@ -107,39 +107,39 @@  discard block
 block discarded – undo
107 107
 		}
108 108
 	}
109 109
 
110
-    private function determineAggregateResult($result, $group, $field) {
111
-        if ($group != null) {
112
-            $ret = [];
113
-            foreach ($result as $res) $ret[$res->$field] = $res->val;
114
-            return $ret;
115
-        }
116
-        else if (isset($result[0])) return $result[0]->val;
117
-        else return 0;
118
-    }
119
-
120
-    private function selectQuery(\Maphper\Lib\Query $query) {
121
-        return $this->adapter->query($query)->fetchAll(\PDO::FETCH_OBJ);
122
-    }
123
-
124
-    public function clearResultCache() {
125
-        if ($this->cacheMode >= 0) $this->resultCache = [];
126
-    }
127
-
128
-    public function clearIDCache() {
129
-        if ($this->cacheMode >= 0) $this->idCache = [];
130
-    }
131
-
132
-    public function updateCache($data, $pkValue) {
133
-        if ($this->cacheMode >= 0) {
134
-  		    if (isset($this->cache[$pkValue])) $this->cache[$pkValue] = (object) array_merge((array)$this->cache[$pkValue], (array)$data);
135
-  		    else $this->cache[$pkValue] = $data;
136
-        }
137
-    }
138
-
139
-    public function deleteIDFromCache($id) {
140
-        if ($this->cacheMode >= 0) {
141
-          unset($this->idCache[$id]);
142
-          unset($this->idCacheTime[$id]);
143
-        }
144
-    }
110
+	private function determineAggregateResult($result, $group, $field) {
111
+		if ($group != null) {
112
+			$ret = [];
113
+			foreach ($result as $res) $ret[$res->$field] = $res->val;
114
+			return $ret;
115
+		}
116
+		else if (isset($result[0])) return $result[0]->val;
117
+		else return 0;
118
+	}
119
+
120
+	private function selectQuery(\Maphper\Lib\Query $query) {
121
+		return $this->adapter->query($query)->fetchAll(\PDO::FETCH_OBJ);
122
+	}
123
+
124
+	public function clearResultCache() {
125
+		if ($this->cacheMode >= 0) $this->resultCache = [];
126
+	}
127
+
128
+	public function clearIDCache() {
129
+		if ($this->cacheMode >= 0) $this->idCache = [];
130
+	}
131
+
132
+	public function updateCache($data, $pkValue) {
133
+		if ($this->cacheMode >= 0) {
134
+  			if (isset($this->cache[$pkValue])) $this->cache[$pkValue] = (object) array_merge((array)$this->cache[$pkValue], (array)$data);
135
+  			else $this->cache[$pkValue] = $data;
136
+		}
137
+	}
138
+
139
+	public function deleteIDFromCache($id) {
140
+		if ($this->cacheMode >= 0) {
141
+		  unset($this->idCache[$id]);
142
+		  unset($this->idCacheTime[$id]);
143
+		}
144
+	}
145 145
 }
Please login to merge, or discard this patch.