Parsing issue on INSERT query (missing space between "VALUES" and opening parenthesis about php-sql-parser HOT 10 CLOSED

From [email protected] on May 09, 2011 07:04:34

Hmm, I've written : There is no data "RESULT" into the array.
It should be corrected to : There is no data "RESULT" or "VALUES" into the array.

from php-sql-parser.

From [email protected] on May 09, 2011 11:38:24

Thanks for the report. I'll look into this ASAP.

Status: Started
Owner: [email protected]
Labels: -Priority-Medium Priority-High

from php-sql-parser.

From [email protected] on October 19, 2011 20:24:39

Here is a simple fix. Move case 'VALUES' into its own category, and check if the next token is blank before consuming two tokens:

[code]
case 'VALUES':
$token_category = $upper;
if($tokens[$token_number+1]=="")
continue 2;
else
continue;
break;
[/code]

from php-sql-parser.

From [email protected] on October 19, 2011 20:44:34

Just in case this isn't clear how to fix. Around line 334 in the source code, you see a list of case statements like this:

case 'SELECT':
case 'ORDER':
case 'LIMIT:
...

find the "case 'VALUES'" and remove it, and make a new section between the following "break;" statement and the next new "case" statement (as in case 'DELETE')). That new section should contain the code pasted in my previous comment.

Hope this helps!

from php-sql-parser.

From [email protected] on February 02, 2012 00:24:24

Solved in current version on http://www.phosco.info/php-sql-parser_current.zip

from php-sql-parser.

From [email protected] on March 05, 2012 02:39:33

[email protected], this also doesn't appear to have been implemented in the link provided above.

from php-sql-parser.

From [email protected] on March 05, 2012 04:10:48

I have added a test for the this issue (see file insert.php in directory "t"). The test is okay, so it seems, that the solution works.

Try the repository on https://www.phosco.info/publicsvn/php-sql-parser I have no commit rights on the original codebase, so I provide the changes on my own Subversion.

from php-sql-parser.

From [email protected] on March 07, 2012 06:40:50

I confirm that code provided by "[email protected]" fixed my issue.

from php-sql-parser.

From [email protected] on March 07, 2012 06:42:40

I confirm that code provided by "[email protected]" fixed my issue, at least for long text. About blob, I didn't test it yet.

from php-sql-parser.

From [email protected] on March 12, 2012 14:49:32

I have accepted the contribution which fixes this bug.

Status: Fixed

from php-sql-parser.