[ERP-209] отфильтрованы отсутствующие телефоны

diff --git a/erp24/actions/orders/FillPhoneAction.php b/erp24/actions/orders/FillPhoneAction.php
index 0cbe882275f4e1a55ba218de674642bdda1c9e8f..d8b61583b76051256d2115daa5fda3765c408cdb 100644 (file)
--- a/erp24/actions/orders/FillPhoneAction.php
+++ b/erp24/actions/orders/FillPhoneAction.php
@@ -35,7 +35,7 @@ class FillPhoneAction extends Action
             $orderIds = ArrayHelper::getColumn($sales, 'order_id');
             $orderIdsClean = [];
             foreach ($orderIds as $orderId) {
-                if (is_int($orderId)) {
+                if (intval($orderId) == $orderId) {
                     $orderIdsClean[] = $orderId;
                 }
             }
@@ -44,12 +44,22 @@ class FillPhoneAction extends Action
                 ->andWhere(['not in', 'phone', ['', null]])
                 ->all();
             $phoneByAmoIdMap = ArrayHelper::map($ordersAmo, 'amo_id', 'phone');
+
+            $salesClean = [];
+            foreach ($sales as $sale) {
+                /** @var $sale Sales */
+                if (!empty($phoneByAmoIdMap[$sale->order_id]) &&
+                    intval($phoneByAmoIdMap[$sale->order_id]) == $phoneByAmoIdMap[$sale->order_id]) {
+                    $salesClean[] = $sale;
+                }
+            }
+
+            $sales = $salesClean;
+
             if ($action == 'update') {
                 foreach ($sales as $sale) {
                     /** @var $sale Sales */
-                    if (!empty($phoneByAmoIdMap[$sale->order_id]) && is_int($phoneByAmoIdMap[$sale->order_id])) {
-                        Sales::updateAll(['phone' => $phoneByAmoIdMap[$sale->order_id]], ['order_id' => $sale->order_id]);
-                    }
+                    Sales::updateAll(['phone' => $phoneByAmoIdMap[$sale->order_id]], ['order_id' => $sale->order_id]);
                 }
             }
         }