Parsing issue on INSERT query (missing space between "VALUES" and opening parenthesis

[radityopw]

Parsing issue on INSERT query (missing space between "VALUES" and opening parenthesis

[GoogleCodeExporter]

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.

Original comment by [email protected] on 9 May 2011 at 2:04

[GoogleCodeExporter]

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

Original comment by [email protected] on 9 May 2011 at 6:38

• Changed state: Started
• Added labels: Priority-High
• Removed labels: Priority-Medium

[GoogleCodeExporter]

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]

Original comment by [email protected] on 20 Oct 2011 at 3:24

[GoogleCodeExporter]

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!

Original comment by [email protected] on 20 Oct 2011 at 3:44

[GoogleCodeExporter]

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

Original comment by [email protected] on 2 Feb 2012 at 8:24

[GoogleCodeExporter]

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

Original comment by [email protected] on 5 Mar 2012 at 10:39

[GoogleCodeExporter]

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.

Original comment by [email protected] on 5 Mar 2012 at 12:10

[GoogleCodeExporter]

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

Original comment by [email protected] on 7 Mar 2012 at 2:40

[GoogleCodeExporter]

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

Original comment by [email protected] on 7 Mar 2012 at 2:42

[GoogleCodeExporter]

I have accepted the contribution which fixes this bug.

Original comment by [email protected] on 12 Mar 2012 at 9:49

• Changed state: Fixed