Skip Menu |

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

Report information
The Basics
Id: 4835
Status: resolved
Priority: 0/
Queue: SQL-Translator

People
Owner: Nobody in particular
Requestors: altblue [...] n0i.net
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 0.04
Fixed in: 0.11011



Subject: mysql parser's default_val field_qualifier definition is plain wrong
(i'm talking about SQL/Translator/Parser/MySQL.pm, revision 1.40) ...as it fails with anything besides [\s\w\d.-] So i changed that definition locally with something still creepy but at least working: default_val : /default/i /'(?:.*?\\')*.*?'|(?:')?[\w\d:.-]*(?:')?/ { $item[2] =~ s/^\s*'|'\s*$//g; $return = $item[2]; } (this modification was done somewhere around line 446)
* changes committed 25.01.04 19:09 by Ken Youens-Clark<kclark@cpan.org> * test cases exists in t/02mysql-parser.t
The parser allows now a complete missing default value. This disqualifies sqlt as a validation tool.
On Wed Nov 23 19:03:32 2011, SKNPP wrote: Show quoted text
> The parser allows now a complete missing default value. This > disqualifies sqlt as a validation tool.
In my eye SQLT was never meant to be a validation tool in the first place. It is a "best effort" translation, which is forward compatible (a validator must closely keep up with the advances in the RDBMS syntaxes). So I will close this bug as not-applicable. Feel free to reopen it for more comments.
On 2nd thought (and after seeing your patch) I decided to include this cange anyway, but addes explicit tests to support DEFAULT ''. Cheers!