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.
Download (untitled) / with headers
text/plain 384b
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 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.