LP#1660059 - Allow for undef value for A/T Processing Group Context Field user/csharp/lp1660059_handle_undef_value_for_at_group_field
authorChris Sharp <csharp@georgialibraries.org>
Sat, 28 Jan 2017 18:19:29 +0000 (13:19 -0500)
committerChris Sharp <csharp@georgialibraries.org>
Sat, 28 Jan 2017 18:19:29 +0000 (13:19 -0500)
commite6f20b3415a2fab424144bc0dcd3e798578b1d6c
tree179b747bdd455165590f55f7f4d9d331e2aea6f5
parent117d1641c4a31df9994b80f307e833701266d1f1
LP#1660059 - Allow for undef value for A/T Processing Group Context Field

For an action_trigger.event_definition that uses action.hold_request.sms_notify
as the Processing Group Context Field/group_field, the event processing
shows multiple Perl warnings and the open-ils.trigger process dies.

Tracing this to the relevant code in Trigger.pm shows that there is a
check to make sure that the fieldmapper object actually exists ($node),
but then assumes that the $ident_value (the actual field) is defined/not null.

Adding an additional check fixes the problem.

Signed-off-by: Chris Sharp <csharp@georgialibraries.org>
Open-ILS/src/perlmods/lib/OpenILS/Application/Trigger.pm