Skip Menu |

This queue is for tickets about the MooseX-Storage CPAN distribution.

Report information
The Basics
Id: 74299
Status: open
Priority: 0/
Queue: MooseX-Storage

People
Owner: Nobody in particular
Requestors: xenoterracide [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Don't delete unset attributes
Date: Mon, 23 Jan 2012 19:41:01 -0600
To: bugs-moosex-storage [...] rt.cpan.org
From: Caleb Cushing <xenoterracide [...] gmail.com>
I wrote this and an associated role for $job, any interest in having it as part of the main dist? I can of course provide an actual patch.     package MooseX::Storage::Engine::Trait::DontDeleteUnset;     use 5.014;     use warnings;     # VERSION     use Moose::Role;     around collapse_attribute => sub {         my ( $orig, $self, $attr, $options ) = @_;         my $value = $self->collapse_attribute_value( $attr, $options );         $self->storage->{$attr->name} = $value;     };     1; -- Caleb Cushing http://xenoterracide.com
On Mon Jan 23 20:41:09 2012, XENO wrote: Show quoted text
> I wrote this and an associated role for $job, any interest in having > it as part of the main dist? I can of course provide an actual patch.
We have the same problem and like this solution (which of course need a MooseX::Storage::Traits::SerializeUndef and MooseX::Storage::Engine::Trait::SerializeUndef ) Could you please apply the patch, or comment why you don't want it. Greetings, domm
On 2012-01-23 17:41:09, XENO wrote: Show quoted text
> I wrote this and an associated role for $job, any interest in having > it as part of the main dist? I can of course provide an actual patch.
I'm not quite exactly sure what you're intending the behaviour here to be, but version 0.47 changes how undef attributes (both set and unset) are handled, so you should take a look at that and let me know if this does what you need.