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)



From xenoterracide [...] gmail.com Mon Jan 23 20: 41:08 2012
MIME-Version: 1.0
X-Spam-Status: No, score=-4.192 tagged_above=-99.9 required=10 tests=[AWL=-2.972, BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, SPF_NEUTRAL=0.779] autolearn=no
X-Spam-Flag: NO
Content-Type: text/plain; charset=UTF-8
Message-ID: <CAAHKNRGifycWGGE0Pum5pQZScA_==BDHAQAYX5M92ih1nSgvjQ [...] mail.gmail.com>
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
X-Spam-Score: -4.192
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id 315FE241261 for <cpan-bug+moosex-storage [...] hipster.bestpractical.com>; Mon, 23 Jan 2012 20:41:08 -0500 (EST)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id yJalx-udWWpo for <cpan-bug+moosex-storage [...] hipster.bestpractical.com>; Mon, 23 Jan 2012 20:41:07 -0500 (EST)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id 1F5D9240C2D for <bugs-moosex-storage [...] rt.cpan.org>; Mon, 23 Jan 2012 20:41:07 -0500 (EST)
Received: (qmail 22921 invoked by uid 103); 24 Jan 2012 01:41:06 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 24 Jan 2012 01:41:06 -0000
Received: from mail-pw0-f50.google.com (HELO mail-pw0-f50.google.com) (209.85.160.50) by 16.mx.develooper.com (qpsmtpd/0.80/v0.80-19-gf52d165) with ESMTP; Mon, 23 Jan 2012 17:41:03 -0800
Received: by pbdv3 with SMTP id v3so1879192pbd.9 for <bugs-moosex-storage [...] rt.cpan.org>; Mon, 23 Jan 2012 17:41:01 -0800 (PST)
Received: by 10.68.75.132 with SMTP id c4mr26452765pbw.23.1327369261072; Mon, 23 Jan 2012 17:41:01 -0800 (PST)
Received: by 10.68.54.68 with HTTP; Mon, 23 Jan 2012 17:41:01 -0800 (PST)
Authentication-Results: hipster.bestpractical.com (amavisd-new); dkim=pass header.i= [...] gmail.com
Delivered-To: cpan-bug+moosex-storage [...] hipster.bestpractical.com
Subject: Don't delete unset attributes
Return-Path: <xenoterracide [...] gmail.com>
X-RT-Mail-Extension: moosex-storage
X-Original-To: cpan-bug+moosex-storage [...] hipster.bestpractical.com
X-Spam-Check-BY: 16.mx.develooper.com
Dkim-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=mime-version:date:message-id:subject:from:to:content-type :content-transfer-encoding; bh=2eh7T99yHJYlA5IBVb9XyPmVr/NMspSIXCODRzOIFN4=; b=QldBGNIaStcMSe7dilG/n3KR5NhDFJfb5LZ3JkIa44Z0Kh1Un4362DOLGU/QVeQZiO uTsQ5J74EGpAqR4U9WVQvu3/11OyfDl4VmSyCftEF429821qYvP2+TvFACNcWY1QHJpU 2iX2w36stvkqxhFVBtGxTtzpRHLfbGXA8V9uA=
Date: Mon, 23 Jan 2012 19:41:01 -0600
X-Spam-Level:
To: bugs-moosex-storage [...] rt.cpan.org
Content-Transfer-Encoding: quoted-printable
From: Caleb Cushing <xenoterracide [...] gmail.com>
X-RT-Original-Encoding: utf-8
Content-Length: 572
Download (untitled) / with headers
text/plain 572b
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
MIME-Version: 1.0
In-Reply-To: <CAAHKNRGifycWGGE0Pum5pQZScA_==BDHAQAYX5M92ih1nSgvjQ [...] mail.gmail.com>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
References: <CAAHKNRGifycWGGE0Pum5pQZScA_==BDHAQAYX5M92ih1nSgvjQ [...] mail.gmail.com>
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.18-6859-1402917217-272.74299-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 437
Download (untitled) / with headers
text/plain 437b
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
MIME-Version: 1.0
In-Reply-To: <CAAHKNRGifycWGGE0Pum5pQZScA_==BDHAQAYX5M92ih1nSgvjQ [...] mail.gmail.com>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
References: <CAAHKNRGifycWGGE0Pum5pQZScA_==BDHAQAYX5M92ih1nSgvjQ [...] mail.gmail.com>
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.18-31131-1408833691-856.74299-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 412
Download (untitled) / with headers
text/plain 412b
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.


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.