[open-ils-commits] r14426 - branches/rel_1_6_0/Open-ILS/src/perlmods/OpenILS/Utils (erickson)

[svn at svn.open-ils.org]

Author: erickson
Date: 2009-10-14 17:02:51 -0400 (Wed, 14 Oct 2009)
New Revision: 14426

Modified:
   branches/rel_1_6_0/Open-ILS/src/perlmods/OpenILS/Utils/Penalty.pm
Log:
thinko on rollback vs. commit when creating a local cstoreEditor object for user penalty calculation/creation

Modified: branches/rel_1_6_0/Open-ILS/src/perlmods/OpenILS/Utils/Penalty.pm
===================================================================
--- branches/rel_1_6_0/Open-ILS/src/perlmods/OpenILS/Utils/Penalty.pm	2009-10-14 21:02:41 UTC (rev 14425)
+++ branches/rel_1_6_0/Open-ILS/src/perlmods/OpenILS/Utils/Penalty.pm	2009-10-14 21:02:51 UTC (rev 14426)
@@ -17,10 +17,10 @@
 sub calculate_penalties {
     my($class, $e, $user_id, $context_org) = @_;
 
-    my $rollback = 0;
+    my $commit = 0;
     unless($e) {
         $e = new_editor(xact =>1);
-        $rollback = 1;
+        $commit = 1;
     }
 
     my $penalties = $e->json_query({from => ['actor.calculate_system_penalties',$user_id, $context_org]});
@@ -60,7 +60,7 @@
         }
     }
 
-    $e->rollback if $rollback;
+    $e->commit if $commit;
     return undef;
 }