LP 1698176: Return undef with die event in Actor.pm _add_patron function. user/cesardv/dyrcona_lp1698176_add_patron_die_event_fix-signoff
authorJason Stephenson <jason@sigio.com>
Thu, 15 Jun 2017 18:11:41 +0000 (14:11 -0400)
committerCesar Velez <cesar.velez@equinoxinitiative.org>
Tue, 14 Nov 2017 22:34:42 +0000 (17:34 -0500)
commit52addf1dcdcde785aafd369b5b9635a80a20da88
tree3c1991fd816d2c9bdd601f33d6d0804dbe8ac386
parentd4cac73c5d8d564d2dfa640fe5a37b14b955a49e
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@sigio.com>
Signed-off-by: Cesar Velez <cesar.velez@equinoxinitiative.org>
Open-ILS/src/perlmods/lib/OpenILS/Application/Actor.pm