Bug 16895: [QA Follow-up] Resolve warning on $payment_type_writeoff
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Wed, 19 Apr 2017 14:25:17 +0000 (16:25 +0200)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 21 Apr 2017 14:59:53 +0000 (10:59 -0400)
Resolves:
  Use of uninitialized value $payment_type_writeoff in string eq at C4/SIP/Sip/MsgType.pm line 1072, <STDIN> line 2.
Would be raised if this parameter is not found in SIPconfig.xml.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
C4/SIP/Sip/MsgType.pm

index 5f66e9b..192239e 100644 (file)
@@ -1060,7 +1060,7 @@ sub handle_fee_paid {
     my $status;
     my $resp = FEE_PAID_RESP;
 
-    my $payment_type_writeoff = $server->{account}->{payment_type_writeoff};
+    my $payment_type_writeoff = $server->{account}->{payment_type_writeoff} || '';
     my $is_writeoff = $pay_type eq $payment_type_writeoff;
 
     $fee_amt    = $fields->{ (FID_FEE_AMT) };