[open-ils-commits] r12949 - trunk/Open-ILS/src/perlmods/OpenILS/Application (erickson)

svn at svn.open-ils.org svn at svn.open-ils.org
Tue Apr 21 15:37:57 EDT 2009


Author: erickson
Date: 2009-04-21 15:37:53 -0400 (Tue, 21 Apr 2009)
New Revision: 12949

Modified:
   trunk/Open-ILS/src/perlmods/OpenILS/Application/Actor.pm
Log:
create events after the commit to prevent any possible race condition.  also, no need for the manual activity log entry

Modified: trunk/Open-ILS/src/perlmods/OpenILS/Application/Actor.pm
===================================================================
--- trunk/Open-ILS/src/perlmods/OpenILS/Application/Actor.pm	2009-04-21 19:29:55 UTC (rev 12948)
+++ trunk/Open-ILS/src/perlmods/OpenILS/Application/Actor.pm	2009-04-21 19:37:53 UTC (rev 12949)
@@ -316,7 +316,7 @@
 	($new_patron, $evt) = _create_perm_maps($session, $user_session, $patron, $new_patron, $user_obj);
 	return $evt if $evt;
 
-	$logger->activity("user ".$user_obj->id." updating/creating  user ".$new_patron->id);
+	$apputils->commit_db_session($session);
 
     my $tses = OpenSRF::AppSession->create('open-ils.trigger');
 	if($patron->isnew) {
@@ -325,8 +325,6 @@
         $tses->request('open-ils.trigger.event.autocreate', 'au.update', $new_patron, $new_patron->home_ou);
     }
 
-	$apputils->commit_db_session($session);
-
 	return flesh_user($new_patron->id(), new_editor(requestor => $user_obj));
 }
 



More information about the open-ils-commits mailing list