Skip Menu |

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

Report information
The Basics
Id: 6649
Status: resolved
Priority: 0/
Queue: SQL-Statement
People
Owner: JZUCKER [...] cpan.org
Requestors: chromatic [...] wgz.org
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: 1.09
Fixed in: (no value)

History Show all quoted text
  Wed Jun 16 16:28:26 2004 Guest - Ticket created
Subject: SQL::Parser Misfeature in feature()?
Tracking down an apparent bug in Class::DBI::SQLite, I wonder if SQL::Parser::feature() should be setting options under its 'opts' key, not $opt_class. Judging by the return path, it should be.
--- SQL/Parser.pm.old 2004-06-16 13:24:45.000000000 -0700 +++ SQL/Parser.pm 2004-06-16 13:25:04.000000000 -0700 @@ -183,7 +183,7 @@ $self->set_feature_flags( undef, {$opt_name=>$opt_value} ); } else { - $self->{$opt_class}->{$opt_name} = $opt_value; + $self->{'opts'}->{$opt_class}->{$opt_name} = $opt_value; } } else {
  Sun Feb 27 15:13:58 2005 JZUCKER [...] cpan.org - Correspondence added
Show quoted text
> Tracking down an apparent bug in Class::DBI::SQLite, I wonder if > SQL::Parser::feature() should be setting options under its 'opts' > key, not $opt_class. Judging by the return path, it should be.
Spot on. Fixed with thanks to you in upcoming version 1.10. There still seems to be a problem with rt.cpan.org. It sometimes forgets I am the current maintainer of SQL::Parser. If you find anything else, please cc me the report.
  Sun Feb 27 15:13:59 2005 JZUCKER [...] cpan.org - Status changed from 'new' to 'resolved'
  Sun Feb 27 15:13:59 2005 JZUCKER [...] cpan.org - Given to JZUCKER