Just wanting to see if there's anything I can help with to move this bug
along. It just bit me again because I'm still having to manually put in
my patch.
On Sat Mar 06 13:30:40 2010, CAPTTOFU wrote:
Show quoted text> Thank you for the patch! I'll test it with the latest DBD::mysql and
> include it in an upcoming release if all checks out.
>
> On Mon Feb 15 16:23:10 2010, KBOSAK wrote:
> > This bug bit me again so I've looked at this more and did some
testing.
Show quoted text> > My previous fix wasn't close, but this seems to work for me in
version
Show quoted text> > 4.007 and 4.013. I've also run the test suite win 4.013 with this
> > change and did fail a few tests but I think they're unrelated
(40catalog
Show quoted text> > and 80process). If there's anything else I can do to help get this
bug
Show quoted text> > in the next release, I'm more than happy to help.
> >
> > @@ -443,10 +443,10 @@
> > for my $row (@$desc)
> > {
> > my $type = $row->{type};
> > - $type =~ m/^(\w+)(?:\((.*?)\))?\s*(.*)/;
> > + $type =~ m/^(\w+)(\((.+)\))?\s?(.*)?$/;
> > my $basetype = lc($1);
> > - my $typemod = $2;
> > - my $attr = $3;
> > + my $typemod = $3;
> > + my $attr = $4;
> >
> > my $info = $col_info{ $row->{field} }= {
> > TABLE_CAT => $catalog,
> >
> >
> >
> > On Wed Oct 07 14:40:58 2009, KBOSAK wrote:
> > > I ran across a case where I was calling column_info on a table
where set
Show quoted text> > > values had parentheses. The column_info code was not returning
the
Show quoted text> > > expected set values. It appeared to have ignored everything in
the list
Show quoted text> > > of set values after the first closing parenthesis it found.
> > >
> > > After some digging I found this regex is the culprit:
> > > $type =~ m/^(\w+)(?:\((.*?)\))?\s*(.*)/;
> > >
> > > I've not tested thoroughly, but I suspect something like this
would work
Show quoted text> > > better:
> > >
> > > $type = substr($type, 0, rindex($type, ')')+1);
> > > $type =~ m/^(\w+)(?:\((.*)\))/;
> > >
> > > I found this bug in version 4.005 but 4.013 also appears to have
the
Show quoted text> > > same regex.
> >
> >
>
>
