PHP Error when parsing SQL containing the REPLACE function about php-sql-parser

[GoogleCodeExporter]

Please try the current version from http://tinyurl.com/mg8bh9p or /trunk first. 
Is the error still there?

Original comment by [email protected] on 10 Sep 2014 at 9:19

• Changed state: Accepted

[GoogleCodeExporter]

The output of the current version is wrong.

Array
(
    [SELECT] => Array
        (
            [0] => Array
                (
                    [delim] => 
                )

        )

    [REPLACE] => Array
        (
            [0] => Array
                (
                    [expr_type] => table
                    [table] => 
                    [no_quotes] => Array
                        (
                            [delim] => .
                            [parts] => Array
                                (
                                )

                        )

                    [alias] => 
                    [base_expr] => 
                )

        )

)

Original comment by [email protected] on 10 Sep 2014 at 1:14

[GoogleCodeExporter]

[deleted comment]

[GoogleCodeExporter]

OK, I tried with php-sql-parser-4.0.0.zip and had the same error :

Warning: strpos(): Empty needle in /Users/nico/Downloads/PHP-SQL-Parser 
2/src/PHPSQLParser/positions/PositionCalculator.php on line 137

Fatal error: Uncaught exception 
'PHPSQLParser\exceptions\UnableToCalculatePositionException' with message 
'cannot calculate position of  within (NOW(), '-', '')' i
n /Users/nico/Downloads/PHP-SQL-Parser 
2/src/PHPSQLParser/positions/PositionCalculator.php:238
Stack trace:
#0 /Users/nico/Downloads/PHP-SQL-Parser 
2/src/PHPSQLParser/positions/PositionCalculator.php(254): 
PHPSQLParser\positions\PositionCalculator->lookForBaseExpression(
'SELECT REPLACE(...', 14, Array, 0, Array)
#1 /Users/nico/Downloads/PHP-SQL-Parser 
2/src/PHPSQLParser/positions/PositionCalculator.php(254): 
PHPSQLParser\positions\PositionCalculator->lookForBaseExpression(
'SELECT REPLACE(...', 14, Array, 'REPLACE', Array)
#2 /Users/nico/Downloads/PHP-SQL-Parser 
2/src/PHPSQLParser/positions/PositionCalculator.php(127): 
PHPSQLParser\positions\PositionCalculator->lookForBaseExpression(
'SELECT REPLACE(...', 14, Array, 0, Array)
#3 /Users/nico/Downloads/PHP-SQL-Parser 
2/src/PHPSQLParser/PHPSQLParser.php(96): 
PHPSQLParser\positions\PositionCalculator->setPositionsWith in 
/Users/nico/Downloa
ds/PHP-SQL-Parser 2/src/PHPSQLParser/positions/PositionCalculator.php on line 
238






Then I tried with the svn trunk (rev 1368) and had some trouble making the 
example code work (had to run composer install and then require_once 
../vendor/autoload.php in the example.php file) and got the same error :

Warning: strpos(): Empty needle in 
/Users/nico/work/code/tmp/php-sql-parser-read-only/src/PHPSQLParser/positions/Po
sitionCalculator.php on line 133

Fatal error: Uncaught exception 
'PHPSQLParser\exceptions\UnableToCalculatePositionException' with message 
'cannot calculate position of  within (NOW(), '-', '')' in 
/Users/nico/work/code/tmp/php-sql-parser-read-only/src/PHPSQLParser/positions/Po
sitionCalculator.php:239
Stack trace:
#0 
/Users/nico/work/code/tmp/php-sql-parser-read-only/src/PHPSQLParser/positions/Po
sitionCalculator.php(255): 
PHPSQLParser\positions\PositionCalculator->lookForBaseExpression('SELECT 
REPLACE(...', 14, Array, 0, Array)
#1 
/Users/nico/work/code/tmp/php-sql-parser-read-only/src/PHPSQLParser/positions/Po
sitionCalculator.php(255): 
PHPSQLParser\positions\PositionCalculator->lookForBaseExpression('SELECT 
REPLACE(...', 14, Array, 'REPLACE', Array)
#2 
/Users/nico/work/code/tmp/php-sql-parser-read-only/src/PHPSQLParser/positions/Po
sitionCalculator.php(123): 
PHPSQLParser\positions\PositionCalculator->lookForBaseExpression('SELECT 
REPLACE(...', 14, Array, 0, Array)
#3 
/Users/nico/work/code/tmp/php-sql-parser-read-only/src/PHPSQLParser/PHPSQLParser
.php(92): in 
/Users/nico/work/code/tmp/php-sql-parser-read-only/src/PHPSQLParser/positions/Po
sitionCalculator.php on line 239

Original comment by [email protected] on 10 Sep 2014 at 1:25

[GoogleCodeExporter]

This is a problem of the SQLProcessor class, which should differ between the 
function and the keyword.

Original comment by [email protected] on 11 Sep 2014 at 7:13

• Changed state: Started

[GoogleCodeExporter]

Try r1377 on /trunk, this should fix the issue.

Original comment by [email protected] on 11 Sep 2014 at 7:52

• Changed state: Fixed