This queue is for tickets about the DBIx-Class CPAN distribution.

Report information
The Basics
Id:
125067
Status:
open
Priority:
Low/Low
Queue:

People
Owner:
Nobody in particular
Requestors:
ether [...] cpan.org
Cc:
AdminCc:

BugTracker
Severity:
(no value)
Broken in:
  • 0.08270
  • 0.082841
Fixed in:
(no value)



Subject: set_inflated_columns incorrectly clears a column with an inflation
14:33 < ether> gah! $row->set_inflated_columns({ col => undef }) for a column with an inflation (e.g. a datetime) will result in undef getting correctly stored in _column_data, but the inflated object is still left behind in _inflated_column. 14:33 < ether> I see that set_inflated_columns has changed between the version I'm using (0.08270) and latest stable (0.082841), but both implementations handle this case incorrectly. 14:38 < ether> changing $self->set_inflated_columns($upd) to $self->set_inflated_column($_ => $upd->{$_}) foreach keys %{ $upd // {} }; works. 14:38 < ether> why set_inflated_columns doesn't simply do that... I don't know...
On Tue Apr 10 23:42:19 2018, ETHER wrote:
Show quoted text
> 14:33 < ether> gah! $row->set_inflated_columns({ col => undef }) > for a column with an inflation (e.g. a datetime) will result in undef > getting correctly stored in _column_data, but the inflated object is > still left behind in > _inflated_column. > 14:33 < ether> I see that set_inflated_columns has changed between the > version I'm using (0.08270) and latest stable (0.082841), but both > implementations handle this case incorrectly. > 14:38 < ether> changing $self->set_inflated_columns($upd) to $self-
> >set_inflated_column($_ => $upd->{$_}) foreach keys %{ $upd // {} };
> works. > 14:38 < ether> why set_inflated_columns doesn't simply do that... I > don't know...
This has been reported/addressed in https://github.com/Perl5/DBIx-Class/pull/121 It needs some extra backcompat work before going in, but is on top of the stack of things. No tentative release date yet.
On 2018-04-11 04:28:07, RIBASUSHI wrote:
Show quoted text
> This has been reported/addressed in https://github.com/Perl5/DBIx- > Class/pull/121 > > It needs some extra backcompat work before going in, but is on top of > the stack of things. No tentative release date yet.
The fix covers some, but not all, the issue described in this related issue: https://rt.cpan.org/Ticket/Display.html?id=130144


This service runs on Request Tracker, is sponsored by The Perl Foundation, and maintained by Best Practical Solutions.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.