From 5dc4d9f3f825bd459e4b347c1ad09d9622449f06 Mon Sep 17 00:00:00 2001 From: Korina Cordero Date: Mon, 13 Feb 2023 02:05:18 +0000 Subject: [PATCH] Add checking if reason is administrative. #738 --- src/Service/JobOrderHandler/ResqJobOrderHandler.php | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/src/Service/JobOrderHandler/ResqJobOrderHandler.php b/src/Service/JobOrderHandler/ResqJobOrderHandler.php index 9a9a44a5..3fe0a7bd 100644 --- a/src/Service/JobOrderHandler/ResqJobOrderHandler.php +++ b/src/Service/JobOrderHandler/ResqJobOrderHandler.php @@ -1661,10 +1661,14 @@ class ResqJobOrderHandler implements JobOrderHandlerInterface $em->persist($obj); $em->flush(); - // check if hub has valid mobile number - $phone_number = $this->country_code . $hub->getNotifNumber(); - if (!empty($phone_number)) - $this->sendSMSHubRejection($phone_number, $obj, $jo); + // check the reason. if administrative, do not send SMS + if ($obj->getReason() != JORejectionReason::ADMINISTRATIVE) + { + // check if hub has valid mobile number + $phone_number = $this->country_code . $hub->getNotifNumber(); + if (!empty($phone_number)) + $this->sendSMSHubRejection($phone_number, $obj, $jo); + } } return $error_array;