@@ -69,7 +69,7 @@ |
||
69 | 69 | * |
70 | 70 | * @return string serverinfo string |
71 | 71 | */ |
72 | - public function getServerInfo():?string; |
|
72 | + public function getServerInfo(): ?string; |
|
73 | 73 | |
74 | 74 | /** |
75 | 75 | * Sanitizer. |
@@ -25,6 +25,7 @@ |
||
25 | 25 | * @param \chillerlan\Settings\SettingsContainerInterface $options |
26 | 26 | * @param \Psr\SimpleCache\CacheInterface|null $cache |
27 | 27 | * @param \Psr\Log\LoggerInterface|null $logger |
28 | + * @return void |
|
28 | 29 | */ |
29 | 30 | public function __construct(SettingsContainerInterface $options, CacheInterface $cache = null, LoggerInterface $logger = null); |
30 | 31 |
@@ -12,10 +12,10 @@ |
||
12 | 12 | |
13 | 13 | namespace chillerlan\Database\Drivers; |
14 | 14 | |
15 | -use chillerlan\Database\Dialects\Dialect; |
|
16 | -use chillerlan\Settings\SettingsContainerInterface; |
|
17 | 15 | use Psr\Log\LoggerInterface; |
18 | 16 | use Psr\SimpleCache\CacheInterface; |
17 | +use chillerlan\Database\Dialects\Dialect; |
|
18 | +use chillerlan\Settings\SettingsContainerInterface; |
|
19 | 19 | |
20 | 20 | interface DriverInterface{ |
21 | 21 |
@@ -83,7 +83,7 @@ |
||
83 | 83 | } |
84 | 84 | |
85 | 85 | /** @inheritdoc */ |
86 | - public function getServerInfo():?string{ |
|
86 | + public function getServerInfo(): ?string{ |
|
87 | 87 | return $this->db->getAttribute(PDO::ATTR_SERVER_INFO).', connected to: '.$this->options->database; |
88 | 88 | } |
89 | 89 |
@@ -34,7 +34,7 @@ |
||
34 | 34 | } |
35 | 35 | |
36 | 36 | /** @inheritdoc */ |
37 | - public function getServerInfo():?string { |
|
37 | + public function getServerInfo(): ?string { |
|
38 | 38 | return $this->drivername.', connected to: '.$this->options->database.' (PDO::ATTR_SERVER_INFO not available)'; |
39 | 39 | } |
40 | 40 |
@@ -205,7 +205,7 @@ discard block |
||
205 | 205 | /** @inheritdoc */ |
206 | 206 | public function cols(array $expressions):array { |
207 | 207 | |
208 | - $_col = function ($expr1, $expr2 = null, $func = null):string { |
|
208 | + $_col = function($expr1, $expr2 = null, $func = null):string { |
|
209 | 209 | switch(true){ |
210 | 210 | case $expr2 && $func: |
211 | 211 | $col = sprintf('%s(%s) AS %s', strtoupper($func), $expr1, $this->quote($expr2)); |
@@ -247,12 +247,12 @@ discard block |
||
247 | 247 | /** @inheritdoc */ |
248 | 248 | public function from(array $expressions):array { |
249 | 249 | |
250 | - $_from = function (string $table, string $ref = null):string { |
|
250 | + $_from = function(string $table, string $ref = null):string { |
|
251 | 251 | // @todo: quotes |
252 | 252 | $from = $this->quote($table); |
253 | 253 | |
254 | 254 | if($ref){ |
255 | - $from = sprintf('%s AS %s', $this->quote($ref), $this->quote($table));// @todo: index hint |
|
255 | + $from = sprintf('%s AS %s', $this->quote($ref), $this->quote($table)); // @todo: index hint |
|
256 | 256 | } |
257 | 257 | |
258 | 258 | return $from; |
@@ -119,7 +119,7 @@ |
||
119 | 119 | |
120 | 120 | // @todo: whitelist types? |
121 | 121 | $nolengthtypes = ['DATE', 'TINYBLOB', 'TINYTEXT', 'BLOB', 'TEXT', 'MEDIUMBLOB', |
122 | - 'MEDIUMTEXT', 'LONGBLOB', 'LONGTEXT', 'SERIAL', 'BOOLEAN', 'UUID']; |
|
122 | + 'MEDIUMTEXT', 'LONGBLOB', 'LONGTEXT', 'SERIAL', 'BOOLEAN', 'UUID']; |
|
123 | 123 | |
124 | 124 | $field[] = (is_int($length) || (is_string($length) && count(explode(',', $length)) === 2)) && !in_array($type, $nolengthtypes, true) |
125 | 125 | ? $type.'('. $length . ')' |
@@ -77,13 +77,13 @@ discard block |
||
77 | 77 | $sql[] = 'TEMPORARY'; |
78 | 78 | } |
79 | 79 | |
80 | - $sql[] = 'TABLE' ; |
|
80 | + $sql[] = 'TABLE'; |
|
81 | 81 | |
82 | 82 | if($ifNotExists){ |
83 | - $sql[] = 'IF NOT EXISTS' ; |
|
83 | + $sql[] = 'IF NOT EXISTS'; |
|
84 | 84 | } |
85 | 85 | |
86 | - $sql[] = $this->quote($table); |
|
86 | + $sql[] = $this->quote($table); |
|
87 | 87 | |
88 | 88 | if(!empty($cols)){ |
89 | 89 | $sql[] = '('; |
@@ -92,7 +92,7 @@ discard block |
||
92 | 92 | if(!empty($primaryKey)){ |
93 | 93 | $sql[] = ', PRIMARY KEY ('; |
94 | 94 | $sql[] = $this->quote($primaryKey); |
95 | - $sql[] = ')'; |
|
95 | + $sql[] = ')'; |
|
96 | 96 | } |
97 | 97 | |
98 | 98 | $sql[] = ')'; |
@@ -127,7 +127,7 @@ discard block |
||
127 | 127 | 'MEDIUMTEXT', 'LONGBLOB', 'LONGTEXT', 'SERIAL', 'BOOLEAN', 'UUID']; |
128 | 128 | |
129 | 129 | $field[] = (is_int($length) || (is_string($length) && count(explode(',', $length)) === 2)) && !in_array($type, $nolengthtypes, true) |
130 | - ? $type.'('. $length . ')' |
|
130 | + ? $type.'('.$length.')' |
|
131 | 131 | : $type; |
132 | 132 | |
133 | 133 | if($attribute){ |
@@ -56,7 +56,7 @@ |
||
56 | 56 | |
57 | 57 | /** @inheritdoc */ |
58 | 58 | public function createDatabase(string $dbname, bool $ifNotExists = null, string $collate = null):array{ |
59 | - $sql = [ 'CREATE DATABASE']; |
|
59 | + $sql = ['CREATE DATABASE']; |
|
60 | 60 | $sql[] = $this->quote($dbname); |
61 | 61 | |
62 | 62 | if($collate){ |
@@ -187,7 +187,7 @@ |
||
187 | 187 | |
188 | 188 | /** @inheritdoc */ |
189 | 189 | public function truncate(string $table):array{ |
190 | - $sql = ['DELETE FROM'];// RECREATE TABLE [table spec] ...stupid firebird 2.5 |
|
190 | + $sql = ['DELETE FROM']; // RECREATE TABLE [table spec] ...stupid firebird 2.5 |
|
191 | 191 | $sql[] = $this->quote($table); |
192 | 192 | |
193 | 193 | return $sql; |
@@ -108,7 +108,7 @@ |
||
108 | 108 | |
109 | 109 | /** @inheritdoc */ |
110 | 110 | public function truncate(string $table):array{ |
111 | - $sql = ['DELETE FROM'];// ??? sqlite |
|
111 | + $sql = ['DELETE FROM']; // ??? sqlite |
|
112 | 112 | $sql[] = $this->quote($table); |
113 | 113 | |
114 | 114 | return $sql; |
@@ -69,7 +69,7 @@ |
||
69 | 69 | throw new QueryException('postgres insert on conflict: no conflict target given'); |
70 | 70 | } |
71 | 71 | |
72 | - $sql[] = 'ON CONFLICT ('.$this->quote($conflictTarget).') DO'; |
|
72 | + $sql[] = 'ON CONFLICT ('.$this->quote($conflictTarget).') DO'; |
|
73 | 73 | |
74 | 74 | switch($onConflict){ |
75 | 75 | case 'IGNORE': |