Skip Menu |
 

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

Report information
The Basics
Id: 125067
Status: open
Priority: 0/
Queue: DBIx-Class

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

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



Subject: set_inflated_columns incorrectly clears a column with an inflation
Download (untitled) / with headers
text/plain 676b
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...
Download (untitled) / with headers
text/plain 943b
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.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

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