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)

History Show all quoted text
  Tue Apr 10 17:42:19 2018 ether [...] cpan.org - Ticket created
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...
  Wed Apr 11 07:28:07 2018 ribasushi [...] leporine.io - Correspondence added
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.
  Wed Apr 11 07:28:09 2018 The RT System itself - Status changed from 'new' to 'open'
  Sat Oct 26 16:56:23 2019 ether [...] cpan.org - Correspondence added
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
  Sat Oct 26 16:56:39 2019 ether [...] cpan.org - Reference by ticket #130144 added
  Thu Nov 14 14:30:44 2019 ribasushi [...] leporine.io - Dependency on ticket #124246 added
  Thu Nov 14 14:31:26 2019 ribasushi [...] leporine.io - Dependency on ticket #124246 deleted
  Thu Nov 14 14:31:26 2019 ribasushi [...] leporine.io - Dependency by ticket #124246 added