Bei save/delete das richtige Ergebnis zurückgeben
authorMoritz Bunkus <m.bunkus@linet-services.de>
Thu, 25 Nov 2010 14:17:35 +0000 (15:17 +0100)
committerMoritz Bunkus <m.bunkus@linet-services.de>
Thu, 12 May 2011 08:45:39 +0000 (10:45 +0200)
Und zwar das von RDBO::save/delete, und nicht das von do_transaction.

SL/DB/Object.pm

index 4e6e15c..a80640c 100755 (executable)
@@ -110,24 +110,30 @@ sub load {
 
 sub save {
   my ($self, @args) = @_;
-  my $worker        = sub {
+
+  my $result;
+  my $worker = sub {
     SL::DB::Object::Hooks::run_hooks($self, 'before_save');
-    my $result = $self->SUPER::save(@args);
+    $result = $self->SUPER::save(@args);
     SL::DB::Object::Hooks::run_hooks($self, 'after_save', $result);
   };
 
-  return $self->db->in_transaction ? $worker->() : $self->db->do_transaction($worker);
+  $self->db->in_transaction ? $worker->() : $self->db->do_transaction($worker);
+  return $result;
 }
 
 sub delete {
   my ($self, @args) = @_;
-  my $worker        = sub {
+
+  my $result;
+  my $worker = sub {
     SL::DB::Object::Hooks::run_hooks($self, 'before_delete');
-    my $result = $self->SUPER::delete(@args);
+    $result = $self->SUPER::delete(@args);
     SL::DB::Object::Hooks::run_hooks($self, 'after_delete', $result);
   };
 
-  return $self->db->in_transaction ? $worker->() : $self->db->do_transaction($worker);
+  $self->db->in_transaction ? $worker->() : $self->db->do_transaction($worker);
+  return $result;
 }
 
 1;