orderCreate upgrade
This commit is contained in:
parent
47452c4f25
commit
d2aa99fdad
@ -71,7 +71,7 @@ class ICrmOrderActions
|
|||||||
|
|
||||||
while ($arOrder = $dbOrder->GetNext()) { //here orders by id asc; with offset
|
while ($arOrder = $dbOrder->GetNext()) { //here orders by id asc; with offset
|
||||||
|
|
||||||
$order = self::orderCreate($arOrder['ID'], $api, $arParams);
|
$order = self::orderCreate($arOrder, $api, $arParams);
|
||||||
|
|
||||||
if (!$order)
|
if (!$order)
|
||||||
continue;
|
continue;
|
||||||
@ -99,7 +99,7 @@ class ICrmOrderActions
|
|||||||
|
|
||||||
while ($arOrder = $dbOrder->GetNext()) { // here orders by id asc
|
while ($arOrder = $dbOrder->GetNext()) { // here orders by id asc
|
||||||
|
|
||||||
$order = self::orderCreate($arOrder['ID'], $api, $arParams);
|
$order = self::orderCreate($arOrder, $api, $arParams);
|
||||||
|
|
||||||
if (!$order)
|
if (!$order)
|
||||||
continue;
|
continue;
|
||||||
@ -179,18 +179,16 @@ class ICrmOrderActions
|
|||||||
else return;
|
else return;
|
||||||
}
|
}
|
||||||
|
|
||||||
public static function orderCreate($orderId, $api, $arParams, $send = false) {
|
public static function orderCreate($arFields, $api, $arParams, $send = false) {
|
||||||
if(!$api || empty($arParams) || !$orderId) { // add cond to check $arParams
|
if(!$api || empty($arParams)) { // add cond to check $arParams
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
$arFields = CSaleOrder::GetById($orderId);
|
|
||||||
|
|
||||||
if (empty($arFields)) {
|
if (empty($arFields)) {
|
||||||
//handle err
|
//handle err
|
||||||
self::eventLog('ICrmOrderActions::orderCreate', 'empty($arFields)', 'incorrect order');
|
self::eventLog('ICrmOrderActions::orderCreate', 'empty($arFields)', 'incorrect order');
|
||||||
|
|
||||||
return false;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
$rsUser = CUser::GetByID($arFields['USER_ID']);
|
$rsUser = CUser::GetByID($arFields['USER_ID']);
|
||||||
|
Loading…
Reference in New Issue
Block a user