| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| |
| | |
Signed-off-by: William Desportes <williamdes@wdes.fr>
|
| |
| |
| |
| | |
Signed-off-by: Hugues Peccatte <hugues.peccatte@aareon.fr>
|
|\ \
| |/
| |
| |
| |
| | |
This merge includes phpcs fixes
Signed-off-by: William Desportes <williamdes@wdes.fr>
|
| |
| |
| |
| |
| |
| |
| | |
Even if this keyword is ignored by MySQL, it should be parsed.
Ref: #167
Signed-off-by: Hugues Peccatte <hugues.peccatte@aareon.fr>
|
| |
| |
| |
| | |
Signed-off-by: William Desportes <williamdes@wdes.fr>
|
| |
| |
| |
| |
| |
| | |
See: 86c5baebda24c1721fb6881df8671a3c7df60e8b
Related-to: 930a860bc7f0684c45babd012070f88a9505d427, 5d5089a259d0195c4a1e4aa3588c31c839954067
Signed-off-by: William Desportes <williamdes@wdes.fr>
|
| |
| |
| |
| |
| | |
This reverts commit 17da8a34b81442524a8e01fead7b5968fa1cc39d.
Signed-off-by: William Desportes <williamdes@wdes.fr>
|
| |
| |
| |
| |
| | |
See: 86c5baebda24c1721fb6881df8671a3c7df60e8b
Signed-off-by: William Desportes <williamdes@wdes.fr>
|
| |
| |
| |
| |
| | |
See: 86c5baebda24c1721fb6881df8671a3c7df60e8b
Signed-off-by: William Desportes <williamdes@wdes.fr>
|
| |
| |
| |
| |
| |
| | |
Ref: #261
Signed-off-by: William Desportes <williamdes@wdes.fr>
|
|/
|
|
|
|
| |
Closes: #261
Signed-off-by: William Desportes <williamdes@wdes.fr>
|
|\
| |
| |
| | |
Signed-off-by: Isaac Bennetch <bennetch@gmail.com>
|
| | |
|
| |
| |
| |
| | |
Signed-off-by: Maurício Meneghini Fauth <mauricio@fauth.dev>
|
| |
| |
| |
| | |
Signed-off-by: Isaac Bennetch <bennetch@gmail.com>
|
|\ \
| | |
| | | |
Fix PHP warnings when building an incomplete CASE expression
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Issue can be reproduced with the following code snippet:
```php
new \PhpMyAdmin\SqlParser\Parser('SELECT a CASE');
// PHP Warning: count(): Parameter must be an array or an object that implements Countable in src/Components/CaseExpression.php on line 296
// PHP Warning: count(): Parameter must be an array or an object that implements Countable in src/Components/CaseExpression.php on line 297
```
|
|/ /
| |
| |
| | |
Signed-off-by: Isaac Bennetch <bennetch@gmail.com>
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| | |
This commit fixes implementation of `build()` methods to produce more consistent SQL code without redundant spaces.
|
| |
| |
| |
| |
| |
| |
| | |
- Rename test providers
- Mark tests as incomplete
Signed-off-by: Maurício Meneghini Fauth <mauriciofauth@gmail.com>
|
| |
| |
| |
| | |
Signed-off-by: Maurício Meneghini Fauth <mauriciofauth@gmail.com>
|
|/
|
|
| |
Signed-off-by: Maurício Meneghini Fauth <mauriciofauth@gmail.com>
|
|
|
|
| |
Signed-off-by: Deven Bansod <devenbansod.bits@gmail.com>
|
|\
| |
| | |
Add support for Aliases on CASE expressions
|
| | |
|
| | |
|
|\ \
| | |
| | | |
Add support for INDEX hints in SELECT statement
|
| | |
| | |
| | |
| | | |
Signed-off-by: Deven Bansod <devenbansod.bits@gmail.com>
|
|\ \ \
| |/ /
| | | |
Add support for LOCK and UNLOCK Statements
|
| | |
| | |
| | |
| | | |
Signed-off-by: Deven Bansod <devenbansod.bits@gmail.com>
|
|\ \ \
| |/ /
|/| | |
Throw a parser error on extraneous comma in UPDATE statement
|
| | |
| | |
| | |
| | | |
Signed-off-by: Deven Bansod <devenbansod.bits@gmail.com>
|
|\ \ \
| |/ /
|/| | |
Throw a parse error if a comma missing between two alter operations
|
| | |
| | |
| | |
| | | |
Signed-off-by: Deven Bansod <devenbansod.bits@gmail.com>
|
|/ /
| |
| |
| |
| |
| |
| |
| | |
* This was originally added in v3.0.8 but was erroneously removed in https://github.com/phpmyadmin/sql-parser/commit/848610a2004e37ec374c2c7a688c8c9131cd5687#diff-1041b02aae148a89853dc0d2710afc9c
* Added a test case for this so that we don't remove it in future releases
Signed-off-by: Deven Bansod <devenbansod.bits@gmail.com>
|
|\ \
| | |
| | | |
Add support for end options in SET statement
|
| | |
| | |
| | |
| | |
| | |
| | | |
* Support `[{ COLLATE expr | DEFAULT }] in `SET NAMES` statement
Signed-off-by: Deven Bansod <devenbansod.bits@gmail.com>
|
|\ \ \
| | | |
| | | | |
Fix building of RenameStatement
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
* The nuance here is that we don't treat `RENAME TABLE` as the keyword of the clause, only `RENAME` is considered.
* That's why we can't use `Statement::$CLAUSES` directly here (even with value of `3` (see `Statement::$CLAUSES`))
* Providing an overriden definition of `build()` in RenameStatement
Signed-off-by: Deven Bansod <devenbansod.bits@gmail.com>
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
Add tests for builder of CREATE or REPLACE VIEW statement
Signed-off-by: William Desportes <williamdes@wdes.fr>
|
| |/ /
| | |
| | |
| | | |
Signed-off-by: Deven Bansod <devenbansod.bits@gmail.com>
|
|/ /
| |
| |
| |
| |
| | |
Fix #207
Signed-off-by: Deven Bansod <devenbansod.bits@gmail.com>
|
|/
|
|
|
| |
Avoid duplicate if conditions
Use switch/case instead of ifs when possible
|
| |
|
|
|
|
|
|
| |
Newer phpunit versions pass parameters to __construct
Signed-off-by: Michal Čihař <michal@cihar.com>
|
|
|
|
|
|
| |
We still support 5.3 here.
Signed-off-by: Michal Čihař <michal@cihar.com>
|
|
|
|
|
|
|
|
|
|
| |
- add fallback in case there is no version match
- do not zero one by one, byt by two to match version parts
- better handle corner cases
Fixes https://github.com/phpmyadmin/phpmyadmin/issues/13728
Signed-off-by: Michal Čihař <michal@cihar.com>
|
|
|
|
|
|
| |
Fixes #182
Signed-off-by: Michal Čihař <michal@cihar.com>
|
| |
|
|
|
|
|
|
| |
Fixes #174
Signed-off-by: Michal Čihař <michal@cihar.com>
|
|
|
|
| |
Signed-off-by: Michal Čihař <michal@cihar.com>
|
|
|
|
|
|
|
|
| |
This seems to work just fine.
Issue #174
Signed-off-by: Michal Čihař <michal@cihar.com>
|
|
|
|
| |
Signed-off-by: Madhura Jayaratne <madhura.cj@gmail.com>
|
|
|
|
|
|
|
|
| |
This also includes INTERSECT and EXPECT reserved words.
Issue #166
Signed-off-by: Michal Čihař <michal@cihar.com>
|
|
|
|
|
|
| |
Issue #2
Signed-off-by: Michal Čihař <michal@cihar.com>
|
|
|
|
|
|
|
| |
This is probably better approach to address
https://github.com/phpmyadmin/phpmyadmin/issues/13487
Signed-off-by: Michal Čihař <michal@cihar.com>
|
|
|
|
|
|
| |
Fixes https://github.com/phpmyadmin/phpmyadmin/issues/13487
Signed-off-by: Michal Čihař <michal@cihar.com>
|
|
|
|
| |
Signed-off-by: Michal Čihař <michal@cihar.com>
|
|
|
|
|
|
| |
Fixes https://github.com/phpmyadmin/phpmyadmin/issues/13486
Signed-off-by: Michal Čihař <michal@cihar.com>
|
|
|
|
|
|
| |
Fixes https://github.com/phpmyadmin/phpmyadmin/issues/13485
Signed-off-by: Michal Čihař <michal@cihar.com>
|
|
|
|
| |
Signed-off-by: Michal Čihař <michal@cihar.com>
|
|
|
|
|
|
|
|
| |
Fix #157
Add Tests for SET statement flag
Signed-off-by: Deven Bansod <devenbansod.bits@gmail.com>
|
|
|
|
|
|
|
|
|
| |
First check whether string is actually utf-8 before trying to process it
as it.
Fixes https://github.com/phpmyadmin/phpmyadmin/issues/13385
Signed-off-by: Michal Čihař <michal@cihar.com>
|
|
|
|
|
|
| |
Fixes #156
Signed-off-by: Michal Čihař <michal@cihar.com>
|
|
|
|
|
|
|
|
|
| |
Previously the simple ones were not prefixed while the combination ones
were, what make it impossible to set the combination ones.
Fixes https://github.com/phpmyadmin/phpmyadmin/issues/13358
Signed-off-by: Michal Čihař <michal@cihar.com>
|
|
|
|
|
|
|
|
| |
It should not be there since beginning.
Issue #154
Signed-off-by: Michal Čihař <michal@cihar.com>
|
|\ |
|
| | |
|
|/
|
|
|
|
| |
Issue #154
Signed-off-by: Michal Čihař <michal@cihar.com>
|
|\ |
|
| | |
|
|/ |
|
|
|
|
| |
Signed-off-by: Dan Ungureanu <udan1107@gmail.com>
|
|
|
|
|
|
|
|
| |
We really can't use anyting as label, it has way stricter syntax.
Fixes https://github.com/phpmyadmin/phpmyadmin/issues/13272
Signed-off-by: Michal Čihař <michal@cihar.com>
|
|
|
|
|
|
| |
Fixes #12.
Signed-off-by: Dan Ungureanu <udan1107@gmail.com>
|
|
|
|
| |
Signed-off-by: Michal Čihař <michal@cihar.com>
|
|
|
|
|
|
|
|
| |
Actually all statements where main token is used as an option.
Fixes https://github.com/phpmyadmin/phpmyadmin/issues/13240
Signed-off-by: Michal Čihař <michal@cihar.com>
|
|
|
|
|
|
| |
Fixes #40.
Signed-off-by: Dan Ungureanu <udan1107@gmail.com>
|
|\
| |
| | |
Fixed inline comments parsing
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The newline is now included as separate token, previously it was
silently discarded, so it was not present in output.
See https://github.com/phpmyadmin/phpmyadmin/issues/13139
Signed-off-by: Michal Čihař <michal@cihar.com>
|
|\ \
| |/
|/| |
Fixes striping of inline comments #11717
|
| |
| |
| |
| | |
Signed-off-by: Raghuram Vadapalli <raghuram.vadapalli@research.iiit.ac.in>
|
|/
|
|
|
|
|
|
| |
It's mostly useful for debugging changes in parser.
See #147.
Signed-off-by: Michal Čihař <michal@cihar.com>
|
|\
| |
| | |
Fix parsing of DELETE clauses with JOINs
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Ref: https://dev.mysql.com/doc/refman/5.7/en/delete.html
Fix #144
Fix phpmyadmin/phpmyadmin#13129
Signed-off-by: Deven Bansod <devenbansod.bits@gmail.com>
|
| |
| |
| |
| |
| |
| | |
Ensure we never have negative indentation
Signed-off-by: Michal Čihař <michal@cihar.com>
|
| |
| |
| |
| | |
Signed-off-by: Michal Čihař <michal@cihar.com>
|
| |
| |
| |
| |
| |
| | |
The test was wrongly using old class names without PhpMyAdmin prefix.
Signed-off-by: Michal Čihař <michal@cihar.com>
|
|\ \
| | |
| | | |
Parse LOAD statement properly
|
| |/
| |
| |
| |
| |
| |
| |
| | |
Fix #131
Might help to fix phpmyadmin/phpmyadmin#12345
Signed-off-by: Deven Bansod <devenbansod.bits@gmail.com>
|
|\ \
| | |
| | | |
Inline JOINs
|
| |/ |
|
|/
|
|
|
|
| |
Fix #113
Signed-off-by: Deven Bansod <devenbansod.bits@gmail.com>
|
|\ |
|
| | |
|
|/ |
|