move common audit parts back into store

[pxelator] / lib / PXElator / CouchDB.pm

diff --git a/lib/PXElator/CouchDB.pm b/lib/PXElator/CouchDB.pm
index 2b3357f..1d1401b 100644 (file)
--- a/lib/PXElator/CouchDB.pm
+++ b/lib/PXElator/CouchDB.pm
@@ -12,6 +12,7 @@ use Time::HiRes qw/time/;
 use Data::Structure::Util qw(unbless);
 use Scalar::Util qw/blessed/;
 use Storable qw/dclone/;
+use Carp qw/carp/;
 
 sub new {
        my ($class, $host, $port, $options) = @_;
@@ -60,10 +61,18 @@ sub request {
 
 our $rev;
 
+sub rev {
+       my ($self,$url) = @_;
+       my $rev = $rev->{$url};
+       $rev  ||= eval { $self->get( $url )->{_rev} };
+#      warn "# rev $url $rev";
+       return $rev;
+}
+
 sub delete {
        my ($self, $url) = @_;
 
-       $self->request(DELETE => $url);
+       $self->request(DELETE => $url . '?rev=' . $self->rev($url) );
 }
 
 sub get {
@@ -74,18 +83,27 @@ sub get {
 
 sub put {
        my ($self, $url, $json) = @_;
-       warn "put $url ",dump($json);
 
-       if ( ! defined $json->{_rev} ) {
-               my $old = eval { $self->get( $url )->{_rev} };
-               $rev->{$url} = $json->{_rev} = $old if defined $old;
-       }
+       $json->{_rev} = $rev->{$url} if defined $rev->{$url};
+
+       my $data = dclone $json;
+       $data = unbless $data if blessed $data;
+
+#      warn "# put ",dump( $data );
 
-       $json = unbless dclone $json if blessed $json;
+       $json = JSON->new->utf8->encode( $data );
 
-       $json = JSON->new->utf8->encode( $json ) if $json;
+       carp "# put ",$json;
 
-       $self->request(PUT => $url, $json);
+       do {
+               my $json = eval { $self->request(PUT => $url, $json) };
+               if ( $@ ) {
+                       $rev->{$url} = $self->rev( $url );
+                       warn "refresh rev $url = ", $rev->{$url};
+               } else {
+                       $rev->{$url} = JSON->new->decode( $json )->{rev};
+               }
+       } until ! $@;
 }
 
 sub post {
@@ -96,25 +114,9 @@ sub post {
 
 our $audit = __PACKAGE__->new;
 
-sub audit {
-       my $data = pop @_;
-
-       my $url = join(' ', @_);
-       $url =~ s/-\S+//g;
-       $url =~ s/\W+/-/g;
-
-       my ( $package, undef, $line, $sub ) = caller(1);
-       ( $package, undef, $line ) = caller(0) if $package eq 'main';
-
-       $data->{x_meta} = {
-               'ident' => [ @_ ],
-               'time' => time(),
-               'package' => $package,
-               'line' => $line,
-               'sub' => $sub,
-       };
-
-       $audit->put( "pxelator/$package.$url", $data );
+sub _store_audit {
+       my ( $id, $data ) = @_;
+       $audit->put( "pxelator/$id", $data );
 }
 
 1;