Bug #18280 for DBIx-SearchBuilder: May be i find a little bug

Tue Mar 21 21:27:06 2006 dimid [...] no-more.kiev.ua - Ticket created

Subject:	May be i find a little bug
Date:	Tue, 21 Mar 2006 20:03:59 +0200
To:	jesse [...] fsck.com
From:	Alexey Grebenschikov <dimid [...] no-more.kiev.ua>

Hello. I work with DBIx::SearchBuilder::Record and find missing lc within PrimaryKeys: line 404: sub PrimaryKeys { my $self = shift; my %hash = map { $_ => $self->{'values'}->{$_} } @{$self->_PrimaryKeys}; return (%hash); ^ } | may be lc $_ ? if _PrimaryKeys return ['Id'] or any keys contain big litter, operation SetAnyField will be wrong beecause PrimaryKeys return Id => undef Good Bye. P.S.: Sorry to trouble you and my very bed english.

Tue Mar 21 21:29:29 2006 jesse [...] fsck.com - Correspondence added

CC:	undisclosed-recipients: ;
Subject:	Re: [rt.cpan.org #18280] May be i find a little bug
Date:	Tue, 21 Mar 2006 21:29:23 -0500
To:	Alexey Grebenschikov via RT <bug-DBIx-SearchBuilder [...] rt.cpan.org>
From:	jesse <jesse [...] fsck.com>

Yep. That looks like a bug.

Tue Mar 21 21:29:30 2006 The RT System itself - Status changed from 'new' to 'open'

Thu Jun 05 20:31:08 2008 RUZ [...] cpan.org - Severity Normal added

Mon Mar 22 09:56:05 2010 RUZ [...] cpan.org - Correspondence added

Resolved in the repository. On Tue Mar 21 21:27:06 2006, dimid@no-more.kiev.ua wrote: Show quoted text

> > Hello. > > I work with DBIx::SearchBuilder::Record and find missing lc within > PrimaryKeys: > > line 404: > sub PrimaryKeys { > my $self = shift; > my %hash = map { $_ => $self->{'values'}->{$_} } @{$self- >_PrimaryKeys}; > return (%hash); ^ > } | may be lc $_ ? > > if _PrimaryKeys return ['Id'] or any keys contain big litter, operation > SetAnyField will be wrong beecause PrimaryKeys return Id => undef > > Good Bye. > > P.S.: Sorry to trouble you and my very bed english. > >

-- Best regards, Ruslan.

Mon Mar 22 09:56:06 2010 RUZ [...] cpan.org - Status changed from 'open' to 'patched'

Mon Mar 22 09:56:06 2010 RUZ [...] cpan.org - Given to RUZ