LP#1660059 - Allow for undef value for A/T Processing Group Context Field
authorChris Sharp <csharp@georgialibraries.org>
Sat, 28 Jan 2017 18:19:29 +0000 (13:19 -0500)
committerChris Sharp <csharp@georgialibraries.org>
Tue, 31 Jan 2017 16:17:17 +0000 (11:17 -0500)
commit9636eab1103b69b52d276ea6884bed3b8d549dee
treef7228c6a8126480478090389bf5ad4df96cd4e32
parent8f145a746d6a3bbe0c49e68fea5125018a21b818
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