foo

[gitmo/MooseX-Storage.git] / lib / MooseX / Storage / Base / WithChecksum.pm

diff --git a/lib/MooseX/Storage/Base/WithChecksum.pm b/lib/MooseX/Storage/Base/WithChecksum.pm
index ffa34b2..227c447 100644 (file)
--- a/lib/MooseX/Storage/Base/WithChecksum.pm
+++ b/lib/MooseX/Storage/Base/WithChecksum.pm
@@ -8,14 +8,16 @@ use MooseX::Storage::Engine;
 
 our $VERSION = '0.01';
 
+our $DIGEST_MARKER = '__DIGEST__';
+
 sub pack {
     my ($self, @args ) = @_;
 
     my $e = MooseX::Storage::Engine->new( object => $self );
 
-    my $collapsed = $e->collapse_object;
+    my $collapsed = $e->collapse_object(@args);
     
-    $collapsed->{__DIGEST__} = $self->_digest_packed($collapsed, @args);
+    $collapsed->{$DIGEST_MARKER} = $self->_digest_packed($collapsed, @args);
     
     return $collapsed;
 }
@@ -25,8 +27,8 @@ sub unpack {
 
     # check checksum on data
     
-    my $old_checksum = $data->{__DIGEST__};
-    delete $data->{__DIGEST__};
+    my $old_checksum = $data->{$DIGEST_MARKER};
+    delete $data->{$DIGEST_MARKER};
     
     my $checksum = $class->_digest_packed($data, @args);
 
@@ -34,36 +36,44 @@ sub unpack {
         || confess "Bad Checksum got=($checksum) expected=($old_checksum)";    
 
     my $e = MooseX::Storage::Engine->new(class => $class);
-    $class->new($e->expand_object($data));
+    $class->new($e->expand_object($data, @args));
 }
 
 
 sub _digest_packed {
     my ( $self, $collapsed, @args ) = @_;
 
-    my $d = shift @args;
+    my $d = $self->_digest_object(@args);
+
+
+    {
+        local $Storable::canonical = 1;
+        $d->add( Storable::nfreeze($collapsed) );
+    }
+
+    return $d->hexdigest;
+}
+
+sub _digest_object {
+    my ( $self, %options ) = @_;
+    my $digest_opts = $options{digest};
+    $digest_opts = [ $digest_opts ] if !ref($digest_opts) or ref($digest_opts) ne 'ARRAY';
+    my ( $d, @args ) = @$digest_opts;
 
     if ( ref $d ) {
         if ( $d->can("clone") ) {
-            $d = $d->clone;
+            return $d->clone;
         } elsif ( $d->can("reset") ) {
             $d->reset;
+            return $d;
         } else {
             die "Can't clone or reset digest object: $d";
         }
     } else {
-        $d = Digest->new($d || "SHA1", @args);
+        return Digest->new($d || "SHA1", @args);
     }
-
-    {
-        local $Storable::canonical = 1;
-        $d->add( Storable::nfreeze($collapsed) );
-    }
-
-    return $d->hexdigest;
 }
 
-
 1;
 
 __END__