diff --git a/src/Service/HubSelector.php b/src/Service/HubSelector.php index d984ddff..85339bdd 100644 --- a/src/Service/HubSelector.php +++ b/src/Service/HubSelector.php @@ -228,7 +228,7 @@ class HubSelector $message = str_replace('item_display', trim($sku_text), $this->trans->trans('no_inventory_message')); error_log($message); $this->sendSMSMessage($hub, $items); - $this->hub_filter_logger->logFilteredHub($hub, 'inventory'); + $this->hub_filter_logger->logFilteredHub($hub, 'no_inventory'); } } if ($jo_type == ServiceType::BATTERY_REPLACEMENT_WARRANTY) @@ -254,7 +254,7 @@ class HubSelector $message = str_replace('item_display', trim($sku_text), $this->trans->trans('no_inventory_message')); error_log($message); $this->sendSMSMessage($hub, $items); - $this->hub_filter_logger->logFilteredHub($hub, 'inventory'); + $this->hub_filter_logger->logFilteredHub($hub, 'no_inventory'); } } } @@ -273,15 +273,16 @@ class HubSelector $sku_text .= ' ' . $key; } $message = str_replace('item_display', trim($sku_text), $this->trans->trans('no_inventory_message')); - error_log($message); // get hub notification number $mobile_number = $hub->getNotifNumber(); - // TODO: what if hub has no notif number? - // send SMS message - error_log('sending sms to - ' . $mobile_number); - $this->rt->sendSMS($mobile_number, 'MOTOLITE', $message); + if (!empty($mobile_number)) + { + // send SMS message + error_log('sending sms to - ' . $mobile_number); + $this->rt->sendSMS($mobile_number, 'MOTOLITE', $message); + } } protected function getClosestHubs(Point $point, $limit_distance)