@@ -33,7 +33,7 @@ discard block |
||
33 | 33 | */ |
34 | 34 | public function cd($path) |
35 | 35 | { |
36 | - return new self( $this->db, $this->queryParser, $this->resultHandler, \arc\path::collapse($path, $this->path) ); |
|
36 | + return new self($this->db, $this->queryParser, $this->resultHandler, \arc\path::collapse($path, $this->path)); |
|
37 | 37 | } |
38 | 38 | |
39 | 39 | /** |
@@ -42,11 +42,11 @@ discard block |
||
42 | 42 | * @param string $path |
43 | 43 | * @return mixed |
44 | 44 | */ |
45 | - public function find($query, $path='') |
|
45 | + public function find($query, $path = '') |
|
46 | 46 | { |
47 | 47 | $path = \arc\path::collapse($path, $this->path); |
48 | 48 | $sql = $this->queryParser->parse($query, $path); |
49 | - return ($this->resultHandler)( $sql, [] ); |
|
49 | + return ($this->resultHandler)($sql, []); |
|
50 | 50 | } |
51 | 51 | |
52 | 52 | /** |
@@ -54,7 +54,7 @@ discard block |
||
54 | 54 | * @param string $path |
55 | 55 | * @return mixed |
56 | 56 | */ |
57 | - public function get($path='') |
|
57 | + public function get($path = '') |
|
58 | 58 | { |
59 | 59 | $path = \arc\path::collapse($path, $this->path); |
60 | 60 | $parent = ($path=='/' ? '' : \arc\path::parent($path)); |
@@ -75,9 +75,9 @@ discard block |
||
75 | 75 | * @param string $top |
76 | 76 | * @return mixed |
77 | 77 | */ |
78 | - public function parents($path='', $top='/') |
|
78 | + public function parents($path = '', $top = '/') |
|
79 | 79 | { |
80 | - $path = \arc\path::collapse($path, $this->path); |
|
80 | + $path = \arc\path::collapse($path, $this->path); |
|
81 | 81 | return ($this->resultHandler)( |
82 | 82 | /** @lang sql */ |
83 | 83 | 'lower(path)=lower(substring(:path,1,length(path))) ' |
@@ -91,9 +91,9 @@ discard block |
||
91 | 91 | * @param string $path |
92 | 92 | * @return mixed |
93 | 93 | */ |
94 | - public function ls($path='') |
|
94 | + public function ls($path = '') |
|
95 | 95 | { |
96 | - $path = \arc\path::collapse($path, $this->path); |
|
96 | + $path = \arc\path::collapse($path, $this->path); |
|
97 | 97 | return ($this->resultHandler)('parent=:path', [':path' => $path]); |
98 | 98 | } |
99 | 99 | |
@@ -102,7 +102,7 @@ discard block |
||
102 | 102 | * @param string $path |
103 | 103 | * @return bool |
104 | 104 | */ |
105 | - public function exists($path='') |
|
105 | + public function exists($path = '') |
|
106 | 106 | { |
107 | 107 | $path = \arc\path::collapse($path, $this->path); |
108 | 108 | $query = $this->db->prepare('select count(*) from nodes where path=:path'); |
@@ -116,7 +116,7 @@ discard block |
||
116 | 116 | */ |
117 | 117 | public function initialize() { |
118 | 118 | try { |
119 | - if ($result=$this->exists('/')) { |
|
119 | + if ($result = $this->exists('/')) { |
|
120 | 120 | return false; |
121 | 121 | } |
122 | 122 | } catch (\PDOException $e) { |
@@ -151,7 +151,7 @@ discard block |
||
151 | 151 | SQL; |
152 | 152 | $queries[] = "create index link_from on links(from_id);"; |
153 | 153 | $queries[] = "create index link_to on links(to_id);"; |
154 | - foreach ( $queries as $query ) { |
|
154 | + foreach ($queries as $query) { |
|
155 | 155 | $result = $this->db->exec($query); |
156 | 156 | if ($result===false) { |
157 | 157 | $this->db->exec('rollback;'); |
@@ -162,7 +162,7 @@ discard block |
||
162 | 162 | |
163 | 163 | return $this->save(\arc\prototype::create([ |
164 | 164 | 'name' => 'Root' |
165 | - ]),'/'); |
|
165 | + ]), '/'); |
|
166 | 166 | } |
167 | 167 | |
168 | 168 | /** |
@@ -171,7 +171,7 @@ discard block |
||
171 | 171 | * @param string $path |
172 | 172 | * @return mixed |
173 | 173 | */ |
174 | - public function save($data, $path='') { |
|
174 | + public function save($data, $path = '') { |
|
175 | 175 | $path = \arc\path::collapse($path, $this->path); |
176 | 176 | $parent = ($path=='/' ? '' : \arc\path::parent($path)); |
177 | 177 | if ($path!='/' && !$this->exists($parent)) { |
@@ -4,11 +4,11 @@ |
||
4 | 4 | |
5 | 5 | final class PSQLQueryParser { |
6 | 6 | |
7 | - private $tokenizer; |
|
7 | + private $tokenizer; |
|
8 | 8 | |
9 | - public function __construct($tokenizer) { |
|
10 | - $this->tokenizer = $tokenizer; |
|
11 | - } |
|
9 | + public function __construct($tokenizer) { |
|
10 | + $this->tokenizer = $tokenizer; |
|
11 | + } |
|
12 | 12 | |
13 | 13 | /** |
14 | 14 | * @param string $query |
@@ -23,14 +23,14 @@ discard block |
||
23 | 23 | $position = 0; |
24 | 24 | $expect = 'name|parenthesis_open|not'; |
25 | 25 | |
26 | - foreach( call_user_func($this->tokenizer, $query) as $token ) { |
|
26 | + foreach (call_user_func($this->tokenizer, $query) as $token) { |
|
27 | 27 | $type = key($token); |
28 | - list($token, $offset)=$token[$type]; |
|
29 | - if ( !preg_match("/^$expect$/",$type) ) { |
|
28 | + list($token, $offset) = $token[$type]; |
|
29 | + if (!preg_match("/^$expect$/", $type)) { |
|
30 | 30 | throw new \LogicException('Parse error at '.$position.': expected '.$expect.', got '.$type.': ' |
31 | - .(substr($query,0, $position)." --> ".substr($query,$position)) ); |
|
31 | + .(substr($query, 0, $position)." --> ".substr($query, $position))); |
|
32 | 32 | } |
33 | - switch($type) { |
|
33 | + switch ($type) { |
|
34 | 34 | case 'number': |
35 | 35 | case 'string': |
36 | 36 | $sql .= $part.$token; |
@@ -47,13 +47,13 @@ discard block |
||
47 | 47 | $part = $token; |
48 | 48 | break; |
49 | 49 | default: |
50 | - $part = "nodes.data #>> '{".str_replace('.',',',$token)."}'"; |
|
50 | + $part = "nodes.data #>> '{".str_replace('.', ',', $token)."}'"; |
|
51 | 51 | break; |
52 | 52 | } |
53 | 53 | $expect = 'compare'; |
54 | 54 | break; |
55 | 55 | case 'compare': |
56 | - switch( $token ) { |
|
56 | + switch ($token) { |
|
57 | 57 | case '>': |
58 | 58 | case '>=': |
59 | 59 | case '<': |
@@ -61,17 +61,17 @@ discard block |
||
61 | 61 | case '=': |
62 | 62 | case '<>': |
63 | 63 | case '!=': |
64 | - $part.=$token; |
|
64 | + $part .= $token; |
|
65 | 65 | break; |
66 | 66 | case '?': |
67 | - $part.=$token; |
|
67 | + $part .= $token; |
|
68 | 68 | str_replace($part, '#>>', '#>'); |
69 | 69 | break; |
70 | 70 | case '~=': |
71 | - $part.=' like '; |
|
71 | + $part .= ' like '; |
|
72 | 72 | break; |
73 | 73 | case '!~': |
74 | - $part.=' not like '; |
|
74 | + $part .= ' not like '; |
|
75 | 75 | break; |
76 | 76 | } |
77 | 77 | $expect = 'number|string'; |
@@ -92,7 +92,7 @@ discard block |
||
92 | 92 | case 'parenthesis_close': |
93 | 93 | $sql .= $token; |
94 | 94 | $indent--; |
95 | - if ( $indent>0 ) { |
|
95 | + if ($indent>0) { |
|
96 | 96 | $expect = 'operator|parenthesis_close'; |
97 | 97 | } else { |
98 | 98 | $expect = 'operator'; |
@@ -101,11 +101,11 @@ discard block |
||
101 | 101 | } |
102 | 102 | $position += $offset + strlen($token); |
103 | 103 | } |
104 | - if ( $indent!=0 ) { |
|
104 | + if ($indent!=0) { |
|
105 | 105 | throw new \LogicException('unbalanced parenthesis'); |
106 | - } else if ( trim($part) ) { |
|
106 | + } else if (trim($part)) { |
|
107 | 107 | $position -= strlen($token); |
108 | - throw new \LogicException('parse error at '.$position.': '.(substr($query,0, $position)." --> ".substr($query,$position))); |
|
108 | + throw new \LogicException('parse error at '.$position.': '.(substr($query, 0, $position)." --> ".substr($query, $position))); |
|
109 | 109 | } else { |
110 | 110 | return $sql; |
111 | 111 | } |
@@ -6,9 +6,9 @@ |
||
6 | 6 | |
7 | 7 | private $tokenizer; |
8 | 8 | |
9 | - public function __construct($tokenizer) { |
|
10 | - $this->tokenizer = $tokenizer; |
|
11 | - } |
|
9 | + public function __construct($tokenizer) { |
|
10 | + $this->tokenizer = $tokenizer; |
|
11 | + } |
|
12 | 12 | |
13 | 13 | /** |
14 | 14 | * @param string $query |
@@ -23,14 +23,14 @@ discard block |
||
23 | 23 | $position = 0; |
24 | 24 | $expect = 'name|parenthesis_open|not'; |
25 | 25 | |
26 | - foreach( call_user_func($this->tokenizer, $query) as $token ) { |
|
26 | + foreach (call_user_func($this->tokenizer, $query) as $token) { |
|
27 | 27 | $type = key($token); |
28 | - list($token, $offset)=$token[$type]; |
|
29 | - if ( !preg_match("/^$expect$/",$type) ) { |
|
28 | + list($token, $offset) = $token[$type]; |
|
29 | + if (!preg_match("/^$expect$/", $type)) { |
|
30 | 30 | throw new \LogicException('Parse error at '.$position.': expected '.$expect.', got '.$type.': ' |
31 | - .(substr($query,0, $position)." --> ".substr($query,$position)) ); |
|
31 | + .(substr($query, 0, $position)." --> ".substr($query, $position))); |
|
32 | 32 | } |
33 | - switch($type) { |
|
33 | + switch ($type) { |
|
34 | 34 | case 'number': |
35 | 35 | case 'string': |
36 | 36 | $sql .= $part.$token; |
@@ -53,7 +53,7 @@ discard block |
||
53 | 53 | $expect = 'compare'; |
54 | 54 | break; |
55 | 55 | case 'compare': |
56 | - switch( $token ) { |
|
56 | + switch ($token) { |
|
57 | 57 | case '>': |
58 | 58 | case '>=': |
59 | 59 | case '<': |
@@ -61,17 +61,17 @@ discard block |
||
61 | 61 | case '=': |
62 | 62 | case '<>': |
63 | 63 | case '!=': |
64 | - $part.=$token; |
|
64 | + $part .= $token; |
|
65 | 65 | break; |
66 | 66 | case '?': |
67 | - $part.= ' IS NOT NULL'; |
|
67 | + $part .= ' IS NOT NULL'; |
|
68 | 68 | str_replace($part, '->>', '->'); |
69 | 69 | break; |
70 | 70 | case '~=': |
71 | - $part.=' like '; |
|
71 | + $part .= ' like '; |
|
72 | 72 | break; |
73 | 73 | case '!~': |
74 | - $part.=' not like '; |
|
74 | + $part .= ' not like '; |
|
75 | 75 | break; |
76 | 76 | } |
77 | 77 | $expect = 'number|string'; |
@@ -92,7 +92,7 @@ discard block |
||
92 | 92 | case 'parenthesis_close': |
93 | 93 | $sql .= $token; |
94 | 94 | $indent--; |
95 | - if ( $indent>0 ) { |
|
95 | + if ($indent>0) { |
|
96 | 96 | $expect = 'operator|parenthesis_close'; |
97 | 97 | } else { |
98 | 98 | $expect = 'operator'; |
@@ -101,11 +101,11 @@ discard block |
||
101 | 101 | } |
102 | 102 | $position += $offset + strlen($token); |
103 | 103 | } |
104 | - if ( $indent!=0 ) { |
|
104 | + if ($indent!=0) { |
|
105 | 105 | throw new \LogicException('unbalanced parenthesis'); |
106 | - } else if ( trim($part) ) { |
|
106 | + } else if (trim($part)) { |
|
107 | 107 | $position -= strlen($token); |
108 | - throw new \LogicException('parse error at '.$position.': '.(substr($query,0, $position)." --> ".substr($query,$position))); |
|
108 | + throw new \LogicException('parse error at '.$position.': '.(substr($query, 0, $position)." --> ".substr($query, $position))); |
|
109 | 109 | } else { |
110 | 110 | return $sql; |
111 | 111 | } |
@@ -33,7 +33,7 @@ discard block |
||
33 | 33 | */ |
34 | 34 | public function cd($path) |
35 | 35 | { |
36 | - return new self( $this->db, $this->queryParser, $this->resultHandler, \arc\path::collapse($path, $this->path) ); |
|
36 | + return new self($this->db, $this->queryParser, $this->resultHandler, \arc\path::collapse($path, $this->path)); |
|
37 | 37 | } |
38 | 38 | |
39 | 39 | /** |
@@ -42,11 +42,11 @@ discard block |
||
42 | 42 | * @param string $path |
43 | 43 | * @return mixed |
44 | 44 | */ |
45 | - public function find($query, $path='') |
|
45 | + public function find($query, $path = '') |
|
46 | 46 | { |
47 | 47 | $path = \arc\path::collapse($path, $this->path); |
48 | 48 | $sql = $this->queryParser->parse($query, $path); |
49 | - return ($this->resultHandler)( $sql, [] ); |
|
49 | + return ($this->resultHandler)($sql, []); |
|
50 | 50 | } |
51 | 51 | |
52 | 52 | /** |
@@ -54,7 +54,7 @@ discard block |
||
54 | 54 | * @param string $path |
55 | 55 | * @return mixed |
56 | 56 | */ |
57 | - public function get($path='') |
|
57 | + public function get($path = '') |
|
58 | 58 | { |
59 | 59 | $path = \arc\path::collapse($path, $this->path); |
60 | 60 | $parent = ($path=='/' ? '' : \arc\path::parent($path)); |
@@ -75,9 +75,9 @@ discard block |
||
75 | 75 | * @param string $top |
76 | 76 | * @return mixed |
77 | 77 | */ |
78 | - public function parents($path='', $top='/') |
|
78 | + public function parents($path = '', $top = '/') |
|
79 | 79 | { |
80 | - $path = \arc\path::collapse($path, $this->path); |
|
80 | + $path = \arc\path::collapse($path, $this->path); |
|
81 | 81 | return ($this->resultHandler)( |
82 | 82 | /** @lang sql */ |
83 | 83 | 'path=substring(:path,1,length(path)) ' |
@@ -91,9 +91,9 @@ discard block |
||
91 | 91 | * @param string $path |
92 | 92 | * @return mixed |
93 | 93 | */ |
94 | - public function ls($path='') |
|
94 | + public function ls($path = '') |
|
95 | 95 | { |
96 | - $path = \arc\path::collapse($path, $this->path); |
|
96 | + $path = \arc\path::collapse($path, $this->path); |
|
97 | 97 | return ($this->resultHandler)('parent=:path', [':path' => $path]); |
98 | 98 | } |
99 | 99 | |
@@ -102,7 +102,7 @@ discard block |
||
102 | 102 | * @param string $path |
103 | 103 | * @return bool |
104 | 104 | */ |
105 | - public function exists($path='') |
|
105 | + public function exists($path = '') |
|
106 | 106 | { |
107 | 107 | $path = \arc\path::collapse($path, $this->path); |
108 | 108 | $query = $this->db->prepare('select count(*) from nodes where path=:path'); |
@@ -116,7 +116,7 @@ discard block |
||
116 | 116 | */ |
117 | 117 | public function initialize() { |
118 | 118 | try { |
119 | - if ($result=$this->exists('/')) { |
|
119 | + if ($result = $this->exists('/')) { |
|
120 | 120 | return false; |
121 | 121 | } |
122 | 122 | } catch (\PDOException $e) { |
@@ -140,7 +140,7 @@ discard block |
||
140 | 140 | SQL; |
141 | 141 | // $queries[] = "create trigger before_insert_node before insert on nodes for each row set new.id = UUID_TO_BIN(uuid());"; |
142 | 142 | $queries[] = "create unique index path on nodes ( path(255) );"; |
143 | - foreach ( $queries as $query ) { |
|
143 | + foreach ($queries as $query) { |
|
144 | 144 | $result = $this->db->exec($query); |
145 | 145 | if ($result===false) { |
146 | 146 | $this->db->exec('rollback;'); |
@@ -151,7 +151,7 @@ discard block |
||
151 | 151 | |
152 | 152 | return $this->save(\arc\prototype::create([ |
153 | 153 | 'name' => 'Root' |
154 | - ]),'/'); |
|
154 | + ]), '/'); |
|
155 | 155 | } |
156 | 156 | |
157 | 157 | /** |
@@ -160,7 +160,7 @@ discard block |
||
160 | 160 | * @param string $path |
161 | 161 | * @return mixed |
162 | 162 | */ |
163 | - public function save($data, $path='') { |
|
163 | + public function save($data, $path = '') { |
|
164 | 164 | $path = \arc\path::collapse($path, $this->path); |
165 | 165 | $parent = ($path=='/' ? '' : \arc\path::parent($path)); |
166 | 166 | if ($path!='/' && !$this->exists($parent)) { |
@@ -19,7 +19,7 @@ discard block |
||
19 | 19 | * @param string $resultHandler handler that executes the sql query |
20 | 20 | * @return store\Store the store |
21 | 21 | */ |
22 | - public static function connect($dsn, $resultHandler=null) |
|
22 | + public static function connect($dsn, $resultHandler = null) |
|
23 | 23 | { |
24 | 24 | $db = new \PDO($dsn); |
25 | 25 | $db->setAttribute(\PDO::ATTR_ERRMODE, \PDO::ERRMODE_EXCEPTION); |
@@ -40,7 +40,7 @@ discard block |
||
40 | 40 | $className = $storeType.'Store'; |
41 | 41 | $store = new $className( |
42 | 42 | $db, |
43 | - new $queryParserClassName(array('\arc\store','tokenizer')), |
|
43 | + new $queryParserClassName(array('\arc\store', 'tokenizer')), |
|
44 | 44 | $resultHandler($db) |
45 | 45 | ); |
46 | 46 | |
@@ -186,8 +186,8 @@ discard block |
||
186 | 186 | }, ARRAY_FILTER_USE_KEY |
187 | 187 | ); |
188 | 188 | } |
189 | - } while($result); |
|
190 | - if ( trim($query) ) { |
|
189 | + } while ($result); |
|
190 | + if (trim($query)) { |
|
191 | 191 | throw new \LogicException('Could not parse '.$query); |
192 | 192 | } |
193 | 193 | } |
@@ -15,14 +15,14 @@ discard block |
||
15 | 15 | * @param string $path |
16 | 16 | * @return mixed |
17 | 17 | */ |
18 | - public function find($query, $path=''); |
|
18 | + public function find($query, $path = ''); |
|
19 | 19 | |
20 | 20 | /** |
21 | 21 | * get a single object from the store by path |
22 | 22 | * @param string $path |
23 | 23 | * @return mixed |
24 | 24 | */ |
25 | - public function get($path=''); |
|
25 | + public function get($path = ''); |
|
26 | 26 | |
27 | 27 | /** |
28 | 28 | * list all parents, including self, by path, starting from the root |
@@ -30,21 +30,21 @@ discard block |
||
30 | 30 | * @param string $top |
31 | 31 | * @return mixed |
32 | 32 | */ |
33 | - public function parents($path='', $top='/'); |
|
33 | + public function parents($path = '', $top = '/'); |
|
34 | 34 | |
35 | 35 | /** |
36 | 36 | * list all child objects by path |
37 | 37 | * @param string $path |
38 | 38 | * @return mixed |
39 | 39 | */ |
40 | - public function ls($path=''); |
|
40 | + public function ls($path = ''); |
|
41 | 41 | |
42 | 42 | /** |
43 | 43 | * returns true if an object with the given path exists |
44 | 44 | * @param string $path |
45 | 45 | * @return bool |
46 | 46 | */ |
47 | - public function exists($path=''); |
|
47 | + public function exists($path = ''); |
|
48 | 48 | |
49 | 49 | /** |
50 | 50 | * initialize the store, if it wasn't before |
@@ -58,7 +58,7 @@ discard block |
||
58 | 58 | * @param string $path |
59 | 59 | * @return mixed |
60 | 60 | */ |
61 | - public function save($data, $path=''); |
|
61 | + public function save($data, $path = ''); |
|
62 | 62 | |
63 | 63 | /** |
64 | 64 | * remove the object with the given path and all its children |
@@ -11,7 +11,7 @@ |
||
11 | 11 | $q = $db->prepare('select * from nodes where '.$query); |
12 | 12 | $result = $q->execute($args); |
13 | 13 | $dataset = []; |
14 | - while ( $data = $q->fetch(\PDO::FETCH_ASSOC) ) { |
|
14 | + while ($data = $q->fetch(\PDO::FETCH_ASSOC)) { |
|
15 | 15 | $value = (object) $data; |
16 | 16 | $value->data = json_decode($value->data); |
17 | 17 | $value->ctime = strtotime($value->ctime); |