Bug #88894 for SQL-Statement: Documentation error

Sun Sep 22 20:03:05 2013 MJD [...] cpan.org - Ticket created

Subject:

Documentation error

The manual says: # determine if the LIKE # operator is supported my $LIKE = $parser->feature('valid_operators','LIKE'); If 'valid_operators' is an error, it should be corrected. If it's not an error, then why isn't it listed in the list of five classes immediately above? * valid_commands * valid_options * valid_comparison_operators * valid_data_types * reserved_words

Sun Sep 22 20:04:15 2013 MJD [...] cpan.org - Ticket #88895: Ticket created

Subject:

No way to find out which commands are valid

Using the ->feature method, I can find out if a particular reserved word or data type is recognized by the parser. But there is no documented way to get a list of all the reserved words or data types that the parser will recognize.

Mon Sep 23 01:53:39 2013 REHSACK [...] cpan.org - Ticket #88895: Merged into ticket #88894

Mon Sep 23 01:53:39 2013 REHSACK [...] cpan.org - Merged into ticket #88894

Mon Sep 23 02:08:11 2013 REHSACK [...] cpan.org - Correspondence added

Well, thanks for the hint how the example misleads people. I've patched that and it'll be part of the upcoming release. Anyway - if you want the feature() method enhanced, finally hack some documenation for it, send patch and I'll hack the corresponding implementation (according to your wishes submitted before). This offer counts only for "get all items of a feature group" wish. Cheers, Jens

Mon Sep 23 02:08:12 2013 The RT System itself - Status changed from 'new' to 'open'

Mon Sep 23 03:53:35 2013 MJD [...] cpan.org - Correspondence added

On Mon Sep 23 02:08:11 2013, REHSACK wrote: Show quoted text

> Well, thanks for the hint how the example misleads people. I've > patched that and it'll be part of the upcoming release.

Thanks, but I still don't know which of the two is correct.

Mon Sep 23 05:53:24 2013 REHSACK [...] cpan.org - Correspondence added

For now: $ perl -Mblib -MSQL::Parser -MData::Dumper -le 'my $parser = SQL::Parser->new(); print Dumper [keys %{$parser->{opts}}]' You could also add (I'm on IRC in #dbi and happy to discuss) a new API description to get those features. I could imagine returning the list of supported features when called without feature and want_array is true.