Как добавить событие изменения количества товара из свойства?

Question

[Darklt]

Раньше было следующее событие, когда подставлялось общее значение при добавлении товара.

AddEventHandler("sale", "OnProductAdd", "OnProductAdd");
function OnProductAdd(&$ID,$Fields)
{
    $res=Array("QUANTITY"=>"500");
    CCatalogProduct::Update($ID,$res);
}

Сейчас сделали что приходит количество в поле например

<?=$arResult["DISPLAY_PROPERTIES"]["kol"]["VALUE"]?>

Понимаю что событие надо поменять на OnProductUpdate, но как указать чтобы брал значение не 500 а из свойства ?

При этом работало только в arSection ID 400 и всех его подразделах.

Comments

[no_one_safe]

А как вы пытались сделать? Покажите что получается сейчас.

[Darklt]

no_one_safe,

AddEventHandler("catalog", "OnProductUpdate", "OnProductUpdate");
function OnProductUpdate(&$ID,$Fields)
{
   if ($SECTION_ID = 400)
      {

    $res=Array("QUANTITY"=>$Fields['kol']);
    CCatalogProduct::Update($ID,$res,$SECTION_ID );
	}
}

[no_one_safe]

Darklt, откуда у вас взялась $SECTION_ID? Почему вы уверены что в $Fields есть какое то 'kol'?

[Darklt]

no_one_safe, вот поэтому и спрашиваю совета, как сделать

[no_one_safe]

Darklt, смотрите. Давайте начнем с того, где хранится ID раздела, вернее каким способом мы можем отлучить его, зная ID элемента?
Каким способом можно получить свойства элемента?
Обе эти задачи прекрасно решает CIBlockElement::GetList

Answer 1

[no_one_safe]

Получить ID раздела и значение свойства поможет метод CIBlockElement::GetList

Comments

[Darklt]

no_one_safe

Вот что получилось, но почему то ставит значние последнего товара у всех товаров в разделе

AddEventHandler("catalog", "OnProductUpdate", "OnProductUpdate");
function OnProductUpdate(&$ID,$Fields)
{
$iblock_id = 99;
$IBLOCK_SECTION_ID = 999;
$my_elements = CIBlockElement::GetList (
Array("ID" => "ASC"),
Array("IBLOCK_ID" => $iblock_id, "ACTIVE" => "Y", "SECTION_ID" => $IBLOCK_SECTION_ID , "INCLUDE_SUBSECTIONS" => "Y"  ),
false,
false,

Array(
'ID', 
'PROPERTY_kol_VALUE',
'PROPERTY_kol'
)
);
while($ar_fields = $my_elements->GetNext())
{
  $res=Array("QUANTITY"=> $ar_fields['PROPERTY_kol_VALUE']);
    
}	
	CCatalogProduct::Update($ID,$res,$PRODUCT_ID);
}

[no_one_safe]

Darklt, не почему то. А потому что вы выбираете именно последний товар
Array("ID" => "ASC"),
Откуда вы взяли
$IBLOCK_SECTION_ID = 999;
? Что это такое?
Где тут участвует ваш товар? В запросе, очевидно, должен участвовать ID пришедшего товара, разве нет?

[Darklt]

no_one_safe
первое это просто сортировка.
$IBLOCK_SECTION_ID = 999; Потому что мне нужно только в этом разделе менять количество, а не во всем каталоге.
Да верное должно меняться значение у каждого товара, а не у всех на значение последнего, видимо что то тут
Array("ID" => "ASC"), но не могу понять и не вижу примеров как поменять в каждом товаре

[no_one_safe]

Darklt, зачем вам сортировка вообще какая то? Вы должны проверить IBLOCK_SECTION_ID товара, который изменяется.

Тем более, если вам нужно поменять поля, нужно использовать событие OnBeforeProductUpdate

[Darklt]

no_one_safe
Сортировку убрал, событие помнял, теперь вообще ничего не происходит при изменении.

AddEventHandler("catalog", "OnBeforeProductUpdate", "OnBeforeProductUpdate");
function OnBeforeProductUpdate(&$ID,$Fields)
{
$iblock_id = 99;
$IBLOCK_SECTION_ID = 999;
$my_elements = CIBlockElement::GetList (
Array("IBLOCK_ID" => $iblock_id, "ACTIVE" => "Y", "SECTION_ID" => $IBLOCK_SECTION_ID , "INCLUDE_SUBSECTIONS" => "Y"  ),
false,

Array(
'ID', 
'PROPERTY_kol_VALUE',
'PROPERTY_kol'
)
);
while($ar_fields = $my_elements->GetNext())
{
  $res=Array("QUANTITY"=> $ar_fields['PROPERTY_kol_VALUE']);
    
}	
  CCatalogProduct::Update($ID,$res,$PRODUCT_ID);
}

Покажите пожалуйста в коде, что я не так делаю.

[no_one_safe]

Darklt,
1. Почитайте что за событие OnBeforeProductUpdate. И посмотрите внимательно его аргументы.
2. В аргументах к вам приходит ID товара, верно? Вам нужно узнать - принадлежит этот товар к разделу 999 или нет, верно? Зачем вы выбирайте какой то не понятный элемент из бд, когда вам нужен конкретно тот, который пришел?
3. Если вы убедились что это ваш элемент и его количество нужно установить равному свойству - зачем вы опять вызываете CCatalogProduct::Update? Это запустит бесконечный цикл. Изменяйте поля Fields, а не вызывайте опять новое событие.
4. Код вам не поможет, пока вы не научитесь думать и рассуждать.