Skip Menu |

This queue is for tickets about the SQL-Statement CPAN distribution.

Report information
The Basics
Id: 29274
Status: resolved
Priority: 0/
Queue: SQL-Statement

People
Owner: REHSACK [...] cpan.org
Requestors: maxim.sloyko [...] bulyon.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Some bugz...
Date: Mon, 10 Sep 2007 11:58:25 +0400
To: <bug-SQL-Statement [...] rt.cpan.org>
From: "Maxim Sloyko" <maxim.sloyko [...] bulyon.com>
Hello! Thanks for good software, keep up the good work! There are couple of issues I found in 1.15 version: 1. Call to $stmt->where hangs deadly (infinite recursion) if the SQL Statement was invalid. This is how you can easily reproduce the problem: perl -MSQL::Parser -MData::Dumper -MSQL::Statement -e 'my $stmt = SQL::Statement->new("SELECT * FROoM stuff WHERE asdf => 234 AND somsing Show quoted text
> 45"); print Dumper($stmt->where);'
2. Success indicator seems to be broken: it is True even if the statement is invalid. Here is how to easily reproduce: perl -MSQL::Parser -MData::Dumper -MSQL::Statement -e 'my $p = SQL::Parser->new; my $res = $p->parse("SELECT * FROoooM stuff WHERE asdf => 234 AND somshit > 45"); print "OK" if $res;' This command reports an error, but happily prints "OK" nevertheless. -- Maxim Sloyko
Download signature.asc
application/pgp-signature 189b

Message body not shown because it is not plain text.

If you could provide a test and if possible a patch, I could handle it more quickly.
With 1.23 the bugs are not reproducible any more. Seems since my last answer there's got something fixed. Provide a test next time to get a quicker response when something becomes ready.