From 1e0d63ec82c23f7d0c077ad03c40fac474a1d14a Mon Sep 17 00:00:00 2001 From: Galen Charlton Date: Tue, 14 Dec 2021 11:09:29 -0500 Subject: [PATCH] LP#1942220: fix glitch in direct charges deletion testing Signed-off-by: Galen Charlton --- Open-ILS/src/eg2/src/app/staff/acq/po/charges.component.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Open-ILS/src/eg2/src/app/staff/acq/po/charges.component.ts b/Open-ILS/src/eg2/src/app/staff/acq/po/charges.component.ts index 21464f993c..d38f54b802 100644 --- a/Open-ILS/src/eg2/src/app/staff/acq/po/charges.component.ts +++ b/Open-ILS/src/eg2/src/app/staff/acq/po/charges.component.ts @@ -122,12 +122,12 @@ export class PoChargesComponent implements OnInit, OnDestroy { if (debit && debit.encumbrance() === 'f') { return false; // if it's expended, we can't just delete it } - if (debit.invoice_entry()) { + if (debit && debit.invoice_entry()) { return false; // we shouldn't actually be a po_item that is // linked to an invoice_entry, but if we are, // do NOT touch } - if (debit.invoice_items() && debit.invoice_items().length) { + if (debit && debit.invoice_items() && debit.invoice_items().length) { return false; // we're linked to an invoice item, so the disposition of the // invoice entry should govern things } -- 2.11.0