Bug #14136 for SQL-Statement: Terminating semicolon and LIMIT clause

Fri Aug 12 07:57:06 2005 Guest - Ticket created

Subject:

Terminating semicolon and LIMIT clause

Hi, I have noticed that DBD::Google does not work when terminating a SELECT statement with a semicolon, if a LIMIT clause is included in the query. For example, $dbh->prepare("SELECT URL FROM google WHERE q=$search LIMIT 970,10;"); produces "SQL ERROR: Bad limit clause!" in SQL::Parser because the terminating semicolon shows up in $limit_clause. Without the semicolon the query will succeed, however this is a violation of the SQL standard. I would be glad if you could fix this error. Thanks, Hendrik

Fri Aug 12 09:57:47 2005 darren [...] cpan.org - Status changed from 'new' to 'open'

Fri Aug 12 09:57:47 2005 darren [...] cpan.org - Correspondence added

[guest - Fri Aug 12 07:57:06 2005]: Show quoted text

> I have noticed that DBD::Google does not work when terminating a > SELECT statement with a semicolon, if a LIMIT clause is included in > the query.

Actually, this is a problem with SQL::Parser, in the LIMIT_CLAUSE method (line 1178 in SQL::Parser version 1.13), and the way SQL::Parser is subclassed, I can't fix the error in DBD::Google. I'm going to move this ticket to the SQL::Parser queue.

Fri Aug 12 09:58:22 2005 darren [...] cpan.org - Severity Normal changed to Important

Fri Aug 12 09:58:22 2005 darren [...] cpan.org - Status changed from 'open' to 'new'

Fri Aug 12 10:00:01 2005 darren [...] cpan.org - Queue changed from DBD-google to SQL-Statement

Wed Feb 22 16:44:58 2006 JZUCKER [...] cpan.org - Correspondence added

Hendrik wrote: For example, $dbh->prepare("SELECT URL FROM google WHERE q=$search LIMIT 970,10;"); produces "SQL ERROR: Bad limit clause!" in SQL::Parser because the terminating semicolon shows up in $limit_clause. Without the semicolon the query will succeed, however this is a violation of the SQL standard. Um, it has nothing to do with the SQL standard. DBI never expects and should not accept trailing semicolons for any statements.

Wed Feb 22 16:44:59 2006 The RT System itself - Status changed from 'new' to 'open'

Wed Feb 22 16:44:59 2006 JZUCKER [...] cpan.org - Status changed from 'open' to 'resolved'

Wed Feb 22 16:45:00 2006 JZUCKER [...] cpan.org - Given to JZUCKER