Skip Menu |

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

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

People
Owner: Nobody in particular
Requestors: filibard [...] gmail.com
Cc:
AdminCc:

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



Subject: error in SQL::Parser
Date: Wed, 21 Oct 2009 15:19:21 +0200
To: bug-SQL-Statement [...] rt.cpan.org
From: Filippo Bardelli <filibard [...] gmail.com>
I've noticed that after an update DBD::CSV was broken with an error due to the case of the file name. I've inquiryed and i've seen that the problem comes from SQL::Parser. I've tested the module with this simple script _____ use SQL::Parser; # CREATE A PARSER OBJECT my $parser = SQL::Parser->new(); my $dialect_name = 'CSV' ; $parser->dialect( $dialect_name ); my $success = $parser->parse('SELECT * FROM foo '); my $struct = $parser->structure ; print "tab: $_ \n" foreach @{ $struct->{'table_names'} } ; Show quoted text
______ result is 'foo' with the last version (1.22) and 'FOO' with version 1.15 my fix has been to uninstall version 1.22 and reinstall the older one. Thanks in advance Filippo
The location you found in SQL::Parser fixes an internal issue in SQL::Statement which gets confused when the same name name for a table is used in mixed case within a query. That's why I decided to convert all table/column names into lower case except the name is quoted. Could you please try whether quoting the table name fixes your problem? Best regards, Jens
Though over your report for a while: The described behaviour is described in the Changes file, too - so there is no need for describing it again. You didn't tell what you want or expect to be changed, so this is a report and I can say: Ok, you're right.