Passed
Pull Request — master (#70)
by Christian
02:13 queued 50s
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.
Maphper/DataSource/MysqlAdapter.php 1 patch
Indentation   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -3,14 +3,14 @@  discard block
 block discarded – undo
3 3
 class MysqlAdapter implements DatabaseAdapter {
4 4
 	private $pdo;
5 5
 	private $stmtCache;
6
-    private $generalEditor;
6
+	private $generalEditor;
7 7
 
8 8
 	public function __construct(\PDO $pdo) {
9 9
 		$this->pdo = $pdo;
10 10
 		//Set to strict mode to detect 'out of range' errors, action at a distance but it needs to be set for all INSERT queries
11 11
 		$this->pdo->query('SET sql_mode = STRICT_ALL_TABLES');
12
-        $this->stmtCache = new StmtCache($pdo);
13
-        $this->generalEditor = new GeneralEditDatabase($this->pdo, ['short_string_max_len' => 191]);
12
+		$this->stmtCache = new StmtCache($pdo);
13
+		$this->generalEditor = new GeneralEditDatabase($this->pdo, ['short_string_max_len' => 191]);
14 14
 	}
15 15
 
16 16
 	public function quote($str) {
@@ -20,32 +20,32 @@  discard block
 block discarded – undo
20 20
 	public function query(\Maphper\Lib\Query $query) {
21 21
 		$stmt = $this->stmtCache->getCachedStmt($query->getSql());
22 22
 		$args = $query->getArgs();
23
-        $stmt->execute($args);
23
+		$stmt->execute($args);
24 24
 
25 25
 		return $stmt;
26 26
 	}
27 27
 
28
-    private function alterColumns($table, array $primaryKey, $data) {
29
-        foreach ($data as $key => $value) {
28
+	private function alterColumns($table, array $primaryKey, $data) {
29
+		foreach ($data as $key => $value) {
30 30
 			if ($this->generalEditor->isNotSavableType($value, $key, $primaryKey)) continue;
31 31
 
32 32
 			$type = $this->generalEditor->getType($value);
33 33
 			$this->tryAlteringColumn($table, $key, $type);
34 34
 		}
35
-    }
35
+	}
36 36
 
37
-    private function tryAlteringColumn($table, $key, $type) {
38
-        try {
39
-            if (!$this->pdo->query('ALTER TABLE ' . $table . ' ADD ' . $this->quote($key) . ' ' . $type)) throw new \Exception('Could not alter table');
40
-        }
41
-        catch (\Exception $e) {
42
-            $this->pdo->query('ALTER TABLE ' . $table . ' MODIFY ' . $this->quote($key) . ' ' . $type);
43
-        }
44
-    }
37
+	private function tryAlteringColumn($table, $key, $type) {
38
+		try {
39
+			if (!$this->pdo->query('ALTER TABLE ' . $table . ' ADD ' . $this->quote($key) . ' ' . $type)) throw new \Exception('Could not alter table');
40
+		}
41
+		catch (\Exception $e) {
42
+			$this->pdo->query('ALTER TABLE ' . $table . ' MODIFY ' . $this->quote($key) . ' ' . $type);
43
+		}
44
+	}
45 45
 
46 46
 	public function alterDatabase($table, array $primaryKey, $data) {
47 47
 		$this->generalEditor->createTable($table, $primaryKey, $data);
48
-        $this->alterColumns($table, $primaryKey, $data);
48
+		$this->alterColumns($table, $primaryKey, $data);
49 49
 	}
50 50
 
51 51
 	public function getColumns($table) {
Please login to merge, or discard this patch.