From 4994f9a2ac6d545e9f3ede6edf0b18857529ac43 Mon Sep 17 00:00:00 2001 From: erickson Date: Wed, 14 Oct 2009 21:02:14 +0000 Subject: [PATCH] thinko on rollback vs. commit when creating a local cstoreEditor object for user penalty calculation/creation git-svn-id: svn://svn.open-ils.org/ILS/trunk@14424 dcc99617-32d9-48b4-a31d-7c20da2025e4 --- Open-ILS/src/perlmods/OpenILS/Utils/Penalty.pm | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Open-ILS/src/perlmods/OpenILS/Utils/Penalty.pm b/Open-ILS/src/perlmods/OpenILS/Utils/Penalty.pm index e6256ae87e..098444c4f8 100644 --- a/Open-ILS/src/perlmods/OpenILS/Utils/Penalty.pm +++ b/Open-ILS/src/perlmods/OpenILS/Utils/Penalty.pm @@ -17,10 +17,10 @@ my $U = "OpenILS::Application::AppUtils"; 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 @@ sub calculate_penalties { } } - $e->rollback if $rollback; + $e->commit if $commit; return undef; } -- 2.43.2