[open-ils-commits] [GIT] Evergreen ILS branch rel_2_12 updated. a4f8c22dbd8c1c4009d759b5491f0f9e0f65cfc0

[Evergreen Git]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Evergreen ILS".

The branch, rel_2_12 has been updated
       via  a4f8c22dbd8c1c4009d759b5491f0f9e0f65cfc0 (commit)
      from  8ee82281557089082d76b9508ade0d716804fe39 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit a4f8c22dbd8c1c4009d759b5491f0f9e0f65cfc0
Author: Jason Stephenson <jason at sigio.com>
Date:   Thu Jun 15 14:11:41 2017 -0400

    LP 1698176: Return undef with die event in Actor.pm _add_patron function.
    
    If the CStoreEditor create_actor_user call fails in the _add_patron
    helper function in OpenILS/Application/Actor.pm, the CStorEditor's
    die_event is returned as a scalar value.  The caller expects a list
    return with two values: the user object in index 0 and the event, if
    any, in index 1.
    
    Returning just the event causes the caller to use the event as if it
    were the user object.  This leads to a different error message later
    in the code: Can't call method "billing_address" on unblessed
    reference at /path/to/perlib/OpenILS/Application/Actor.pm line 743.
    
    This commit changes the return value in the event of a database insert
    failure to be the two member list as expected by the caller with undef
    in index 0 and the die_event in index 1.  This returns the proper
    error message to the client.
    
    Signed-off-by: Jason Stephenson <jason at sigio.com>
    Signed-off-by: Cesar Velez <cesar.velez at equinoxinitiative.org>

diff --git a/Open-ILS/src/perlmods/lib/OpenILS/Application/Actor.pm b/Open-ILS/src/perlmods/lib/OpenILS/Application/Actor.pm
index 0e936cf..47576b1 100644
--- a/Open-ILS/src/perlmods/lib/OpenILS/Application/Actor.pm
+++ b/Open-ILS/src/perlmods/lib/OpenILS/Application/Actor.pm
@@ -591,7 +591,7 @@ sub _add_patron {
     # do a dance to get the password hashed securely
     my $saved_password = $patron->passwd;
     $patron->passwd('');
-    $e->create_actor_user($patron) or return $e->die_event;
+    $e->create_actor_user($patron) or return (undef, $e->die_event);
     modify_migrated_user_password($e, $patron->id, $saved_password);
 
     my $id = $patron->id; # added by CStoreEditor

-----------------------------------------------------------------------

Summary of changes:
 .../src/perlmods/lib/OpenILS/Application/Actor.pm  |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)


hooks/post-receive
-- 
Evergreen ILS