Форум

[QUOTE]Евгений Жуков написал:
[QUOTE] Максим Егоров написал:
А если возможность как-то их получить при добавлении?[/QUOTE]
События складов используйте[/QUOTE]
Использую событие OnBeforeStoreProductAdd, но получается какая-то фигня. Есть вот такие данные, как в табличке. Необходимо если товар есть на первом или втором складе, то ставить сортировку 500, если на 4, то 1000, если нет нигде, то 1500.

[IMG WIDTH=513 HEIGHT=246]https://i.ibb.co/JqCN0Cy/0000000000000.png[/IMG]
[CODE]if ($arFields["STORE_ID"] == 4 and $arFields["AMOUNT"] >0){
   $arSort["SORT"] = "1000";
} elseif (($arFields["STORE_ID"] == 1 and $arFields["AMOUNT"] >0) or ($arFields["STORE_ID"] == 2 and $arFields["AMOUNT"] >0)){
   $arSort["SORT"] = "500";
}else{
   $arSort["SORT"] = "1500";
}

$ibp = new CIBlockElement();
$res = $ibp->Update($arFields["PRODUCT_ID"], $arSort);

if ($res) {
   AddMessage2Log("Для позиции " . $arFields["PRODUCT_ID"] . " была изменена сортировка (" . $arSort['SORT'] . ")", "Tehno");
}[/CODE]Это получается при добавлении одной записи она изменяется 3 раза (CIBlockElement::Update) и при таком условии получается так

[IMG WIDTH=640 HEIGHT=501]https://i.ibb.co/qCLfCkb/11111111111.png[/IMG]

Сейчас настроена сортировка по двум полям, сначала по индексу сортировки, потом по цене. При этом, необходимо, чтобы сортировка по индексу никогда не изменялась, а пользователь мог менять только значения ELEMENT_SORT_FIELD2. Возможно ли это как-то реализовать? Сортировка в catalog.section

Тоже самое и при запросе CCatalogStoreProduct в событии OnAfterIBlockElementAdd
[CODE]$store = CCatalogStoreProduct::GetList(
   array(),
   array("PRODUCT_ID" => $ar_fields["ID"], ">AMOUNT" => 0),
   false,
   false,
   array()
);[/CODE]
То есть получается количество на складах добавляется после события OnAfterIBlockElementAdd. А если возможность как-то их получить при добавлении?

В init есть данный код. При событии OnAfterIBlockElementUpdate все передается и работает, а вот при OnAfterIBlockElementAdd данные со складов пустые
[CODE]define("LOG_FILENAME", $_SERVER["DOCUMENT_ROOT"]."/bitrix/php_interface/logs/".date("d-m-Y").".txt");

AddEventHandler("iblock", "OnAfterIBlockElementAdd", Array("ElementTehno", "addSort"));
class ElementTehno
{
   //Чтобы у нас не было 504 ошибки необходимо прописать этот код, он не дает зациклится нашему скрипту
   public static $disableHandler = false;

   // Меняем сортировка и активность в зависимости от наличия на складах
   public function sort_price($ID, $STORE_AMOUNT_1, $STORE_AMOUNT_2, $STORE_AMOUNT_4, $DETAIL_PICTURE){

       if($STORE_AMOUNT_1 == 0 and $STORE_AMOUNT_2 == 0 and $STORE_AMOUNT_4 == 0){
           $arSort["SORT"] = "1500";
       } elseif ($STORE_AMOUNT_1 > 0 or $STORE_AMOUNT_2 > 0){
           $arSort["SORT"] = "500";
       } elseif($STORE_AMOUNT_4 > 0){
           $arSort["SORT"] = "1000";
       }

       if(isset($arSort)){
           $price = CPrice::GetBasePrice($ID);
           if($price["PRICE"] == 0){
               $arSort["ACTIVE"] = 'N';
           } elseif($price["PRICE"] >0 and isset($DETAIL_PICTURE)){
               $arSort["ACTIVE"] = 'Y';
           }
           return $arSort;
       }
   }

   function addSort(&$arFields){
       if($arFields["ID"]> 0 && $arFields["IBLOCK_ID"] == "26"){

           $addProps = CIBlockElement::GetList(
               Array("ID" => "ASC"),
               Array("IBLOCK_ID" => 26, "ID" => $arFields["ID"]),
               false,
               false,
               Array(
                   'ID', 'PURCHASING_CURRENCY', 'PRICE_1', 'CURRENCY_1', 'PURCHASING_PRICE', 'PROPERTY_MERLION_CODE', 'STORE_AMOUNT_1', 'STORE_AMOUNT_2', 'STORE_AMOUNT_4', 'DETAIL_PICTURE'
               )
           );

           self::$disableHandler = true;
           if($ar_fields = $addProps->GetNext())
           {
               $ElementUpdate = ElementTehno::sort_price($ar_fields["ID"], $ar_fields["STORE_AMOUNT_1"], $ar_fields["STORE_AMOUNT_2"], $ar_fields["STORE_AMOUNT_4"], $ar_fields["DETAIL_PICTURE"]);

               if(isset($ElementUpdate)){
                   $ibp = new CIBlockElement();
                   $res = $ibp->Update($arFields["ID"], $ElementUpdate);
                   if($res){
                       AddMessage2Log("При добавлении позиции ".$ar_fields["ID"]." была изменена сортировка или активность", "Tehno");
                   }
               }

           }

       }
   }
}[/CODE]

[QUOTE]Евгений Жуков написал:
[URL=https://dev.1c-bitrix.ru/community/blogs/vws/rabota-s-tsenami-v-forme-redaktirovaniya-elementa-infobloka.php]Документация[/URL] [/QUOTE]
Спасибо. Проверял работоспособность при добавлении через форму, а использоваться будет через api. Так все работает

В $price["PRICE"] пусто, но при этом в $arFields["ID"] правильный id элемента

Необходимо при добавлении товара проверять цену товара и если она нулевая деактивировать ее. Сделал вот так, но не работает[CODE]define("LOG_FILENAME", $_SERVER["DOCUMENT_ROOT"]."/bitrix/php_interface/logs/".date("d-m-Y").".txt");

AddEventHandler("iblock", "OnAfterIBlockElementAdd", Array("ElementTehno", "addNullPrice"));

class ElementTehno{
function addNullPrice(&$arFields){
   if($arFields["ID"]> 0 && $arFields["IBLOCK_ID"] == "26"){
       $price = CPrice::GetBasePrice($arFields["ID"]);
       if($price["PRICE"] == '0.00'){
           $arActive = Array(
               "ACTIVE" => "N",
           );
           $ibp = new CIBlockElement();
           $ibp->Update($arFields["ID"], $arActive);
           AddMessage2Log("ТОВАР С НУЛЕВОЙ ЦЕНОЙ | Позиция ".$arFields["ID"]." была деактивирована.".$price["PRICE"], "TEHNO_Price");
       }
   }
}

}

[/CODE]

[QUOTE]Игорь Долгополов написал:
[QUOTE]return editSort();[/QUOTE]
вы возращаете результат выполнения самой функции, а не ее название. Т.е. получается у вас получилась рекурсия без точки выхода... Вам следует возвращать так:[CODE] return "editSort();" ; [/CODE]А так либо дождаться пока процесс убьется по таймауту на сервере, но судя по всему у вас он отключен... Либо найти самому этот процесс и убить его. Почитайте что-нибудь об [URL=https://pingvinus.ru/note/ps-kill-killall]этом[/URL] .

upd.  ну собственно да, увидел, что вам в соседней теме посоветовали отключить таймаут... Тогда искать процесс ручками и убивать, либо может оказаться проще - ребутнуть сервев, тоже должно помочь.[/QUOTE]
На локальной машине сделал так, выполнение функции завершилось, загрузил на хостинг и снова выполняется функция постоянно. При этом, рекурсия не всей функции. Запись "Завершение изменения сортировки" в логах не получаю, исполняется только цикл. На локалке получил информацию о старте и завершении в логах и после файл не меняется.

[QUOTE]Игорь Долгополов написал:
[QUOTE]return editSort();[/QUOTE]
вы возращаете результат выполнения самой функции, а не ее название. Т.е. получается у вас получилась рекурсия без точки выхода... Вам следует возвращать так:[CODE] return "editSort();" ; [/CODE]А так либо дождаться пока процесс убьется по таймауту на сервере, но судя по всему у вас он отключен... Либо найти самому этот процесс и убить его. Почитайте что-нибудь об [URL=https://pingvinus.ru/note/ps-kill-killall]этом[/URL] .

upd.  ну собственно да, увидел, что вам в соседней теме посоветовали отключить таймаут... Тогда искать процесс ручками и убивать, либо может оказаться проще - ребутнуть сервев, тоже должно помочь.[/QUOTE]
Советом из соседней ветки не успел воспользоваться, написал хостеру чтобы ребутнули сервер. Спасибо за наводку на ошибку

Произошел небольшой казус. В /bitrix/php_interface/init.php создал функцию и добавил ее выполнение на агента. По итогу функция выполняется по кругу. Удалил агента, в логи данные все равно записываются. Удалил файл init.php, удалил dbconn.php, в логах все равно записываются ID элементов, удалил агента через класс CAgent::Delete, удалил файл /bitrix/modules/main/classes/mysql/agent.php, но даже при не работающем сайте логи записываются.
Логи: [CODE]Host:
Date: 2020-09-29 09:50:02
Module:
29.09.2020 09:50:02 | Изменна позиция -171300
   editSort < editSort < eval < CAgent::ExecuteAgents < CAgent::CheckAgents
/var/www/u16046btrx21906/data/www/u16046btrx21906.pyramis.cameta.in.net/bitrix/php_interface/init.php:53
/var/www/u16046btrx21906/data/www/u16046btrx21906.pyramis.cameta.in.net/bitrix/modules/main/classes/mysql/agent.php(168) : eval()'d code:1
/var/www/u16046btrx21906/data/www/u16046btrx21906.pyramis.cameta.in.net/bitrix/modules/main/classes/mysql/agent.php:168
/var/www/u16046btrx21906/data/www/u16046btrx21906.pyramis.cameta.in.net/bitrix/modules/main/classes/mysql/agent.php:40
/var/www/u16046btrx21906/data/www/u16046btrx21906.pyramis.cameta.in.net/bitrix/modules/main/include.php:271
----------

[/CODE]
Сам код в init.php [CODE]define("LOG_FILENAME", $_SERVER["DOCUMENT_ROOT"]."/bitrix/php_interface/log.txt");

function editSort(){
   if(CModule::IncludeModule("iblock")){

       $date = strtotime("-1 day");
       $addProps = CIBlockElement::GetList (
           Array("ID" => "ASC"),
           Array("IBLOCK_ID" => 26, "ID" => 172265),
           false,
           false,
           Array(
               'ID', 'SORT','CATALOG_QUANTITY', 'CATALOG_STORE_AMOUNT_1', 'CATALOG_STORE_AMOUNT_2', 'DETAIL_PICTURE'
           )
       );
       AddMessage2Log(date("d.m.Y H:i:s")." | Старт изменения сортировки", "");
       while($ar_fields = $addProps->GetNext())
       {

           if($ar_fields['CATALOG_QUANTITY'] == 0){
               $arSort = Array(
                   "SORT" => "1500",
               );
           } elseif ($ar_fields['CATALOG_STORE_AMOUNT_1'] > 0 or $ar_fields['CATALOG_STORE_AMOUNT_2'] > 0){
               $arSort = Array(
                   "SORT" => "500",
               );
           } else{
               $arSort = Array(
                   "SORT" => "1000",
               );
           }
           if(isset($arSort)){
               $ibp = new CIBlockElement();
               $price = CPrice::GetBasePrice($ar_fields['ID']);
               if($price["PRICE"] == 0){
                   $arSort["ACTIVE"] = 'N';
               } elseif($price["PRICE"] >0 and isset($ar_fields['DETAIL_PICTURE'])){
                   $arSort["ACTIVE"] = 'Y';
               }
               $ibp->Update($ar_fields["ID"], $arSort);
               AddMessage2Log(date("d.m.Y H:i:s")." | Изменна позиция -".$ar_fields['ID'], "");
           }
           exit();
       }

       AddMessage2Log(date("d.m.Y H:i:s")." | Завершение изменения сортировки", "");
   }
   return editSort();
}[/CODE]