Как переписать на новый синтаксис функцию обновления поля Коэффициент единицы измерения?

Question

[sergeyviktorovich]

AddEventHandler("catalog", "OnPriceUpdate", "HandlerOnPriceUpdate");
function HandlerOnPriceUpdate($ID,&$arFields)
{
 
     
    $element = $arFields['PRODUCT_ID'];    //ID товара
    $iblockId = 9;
   /*$coeff= CIBlockElement::GetByID($arFields['PRODUCT_ID'])->GetNextElement()->GetProperties('VES_V_GRAMMAKH_SHAG');*/ //например
    $coeff= 100;
        $curElementRatio = CCatalogMeasureRatio::getList(
        Array(),
        array('IBLOCK_ID' => $iblockId, 'PRODUCT_ID' => $element ),
        false, false);

    while ($arRatio = $curElementRatio->GetNext()) {
        $ratioId = $arRatio['ID'];
        $arFieldsNew = Array(
            'RATIO' => $coeff,
        );

        CCatalogMeasureRatio::update($ratioId, $arFieldsNew);
    }

}

пробовал так

$eventManager = \Bitrix\Main\EventManager::getInstance();
$eventManager->addEventHandler(
    'catalog',
    '\Bitrix\Catalog\Price::OnAfterUpdate',
    static function ( $event ) {
       $result = new \Bitrix\Main\ORM\EventResult;
       $arFields   =   $event->getParameter('fields');
       $iblockId = 9;
         /*** получаем ID товара*/
       $element = $event->getParameter('primary')['ID'];
       $coeff= CIBlockElement::GetByID($arFields['PRODUCT_ID'])->GetNextElement()->GetProperties('VES_V_GRAMMAKH_SHAG'); //например
      
      
      $curElementRatio = CCatalogMeasureRatio::getList(
        Array(),
        array('IBLOCK_ID' => $iblockId, 'PRODUCT_ID' => $element ),
        false, false);

      while ($arRatio = $curElementRatio->GetNext()) {
          $ratioId = $arRatio['ID'];
          $arFieldsNew = Array(
              'RATIO' => $coeff,
          );

       return CCatalogMeasureRatio::update($ratioId, $arFieldsNew);
    }


    
    }
);

Comments

[PetrPo]

Если верхний вариант рабочий, в чем сложность возникла? fields-ы получаешь через объект и вся разница

$arFields   =   $event->getParameter('fields');
$element = $arFields['PRODUCT_ID'] ?? 0;

if($element) {
    // здесь все остальное
}

[sergeyviktorovich]

PetrPo, верхний не рабочий устарел его синтаксис

[PetrPo]

sergeyviktorovich, по документации устарело событие OnPriceUpdate, речь об этом или о том чтобы все переписать на d7?

[sergeyviktorovich]

PetrPo, да переписать на d7,второй вариант то как пробовал сделать,но не совсем понял что не так

[PetrPo]

sergeyviktorovich, не пойму, говоришь все переписать на d7 и во втором примере юзаешь старое api. Давай еще раз, чтобы новое событие сработало

$eventManager = \Bitrix\Main\EventManager::getInstance();
$eventManager->addEventHandler(
    'catalog',
    '\Bitrix\Catalog\Price::OnAfterUpdate',
    static function ( $event ) {
		$arFields = $event->getParameter('fields');
		$element = $arFields['PRODUCT_ID'] ?? 0;
		
		if($element) {
			// здесь все остальное из первого примера
                       // условно назовем это место UPDATE_MEASURE_RATIO
		}
    }

);

Вот где UPDATE_MEASURE_RATIO, если поставить все из твоего первого примера, то все сработает, т.е.

$iblockId = 9;
/*$coeff= CIBlockElement::GetByID($arFields['PRODUCT_ID'])->GetNextElement()->GetProperties('VES_V_GRAMMAKH_SHAG');*/
    //например
    $coeff= 100;
        $curElementRatio = CCatalogMeasureRatio::getList(
        Array(),
        array('IBLOCK_ID' => $iblockId, 'PRODUCT_ID' => $element ),
        false, false);

    while ($arRatio = $curElementRatio->GetNext()) {
        $ratioId = $arRatio['ID'];
        $arFieldsNew = Array(
            'RATIO' => $coeff,
        );

        CCatalogMeasureRatio::update($ratioId, $arFieldsNew);
    }

Вопрос, ты хочешь еще и вот эти методы - CIBlockElement::GetByID, CCatalogMeasureRatio::getList, CCatalogMeasureRatio::update заменить на d7?

[sergeyviktorovich]

PetrPo, да тоже полностью переписать на D7

[PetrPo]

sergeyviktorovich, могу подсказать план действий, переписывать за тебя я конечно не буду.
1. открываешь модуль, например /bitrix/modules/catalog
2. ищещь файл с описанием нужной сущности, например для CCatalogMeasureRatio - это файл /lib/measureratio.php в нем смотришь какие поля доступны и какие есть связи с другими сущностями
3. проводишь аналогию между полями в сущности d7 и тем что получал не через d7, и пишешь getlist. Все сущности d7 наследники \Bitrix\Main\Entity\DataManager, т.е. сигнатура методов (getList, update) )одинаковая. Примеры есть в доках
4. здесь можешь посмотреть update

Про этот метод CIBlockElement::GetByID просто скажу замени на getlist и по аналогии с выше сказанным

[sergeyviktorovich]

PetrPo, а где посмотреть какое ядро в админке битрикс это можно поcмотреть ? и ретурн нужен ?
у меня почему не работает варинт D7+ с CIBlockElement::GetByID, CCatalogMeasureRatio::getList, CCatalogMeasureRatio::update
хочу обновить Коэффициент единицы измерения при выгрузке из C1 цены товаров

$eventManager = \Bitrix\Main\EventManager::getInstance();
$eventManager->addEventHandler(
    'catalog',
    '\Bitrix\Catalog\Price::OnAfterUpdate',
    static function ( $event ) {
    $arFields = $event->getParameter('fields');
    $element = $arFields['PRODUCT_ID'] ?? 0;
    
    if($element) {
    
      $iblockId = 9;
$coeff= CIBlockElement::GetByID($arFields['PRODUCT_ID'])->GetNextElement()->GetProperties('VES_V_GRAMMAKH_SHAG');
    //например

        $curElementRatio = CCatalogMeasureRatio::getList(
        Array(),
        array('IBLOCK_ID' => $iblockId, 'PRODUCT_ID' => $element ),
        false, false);

    while ($arRatio = $curElementRatio->GetNext()) {
        $ratioId = $arRatio['ID'];
        $arFieldsNew = Array(
            'RATIO' => $coeff,
        );

        CCatalogMeasureRatio::update($ratioId, $arFieldsNew);
    }
       
    }
    }

);

[PetrPo]

sergeyviktorovich,

а где посмотреть какое ядро в админке битрикс это можно поcмотреть

версия > 14 значит d7 (вообщем не парься, было бы не d7 ты бы даже обработчик не зарегистрировал)

ретурн нужен ?

Там же в доках, есть пример события. В нем меняют поля и делают return. В твоем случае при update цены (одна сущность) меняется коэффициент (другая сущность), т.е. можно ничего не возвращать

[sergeyviktorovich]

PetrPo, вот эту строчку пришлось переписать так

$coeff= CIBlockElement::GetByID($arFields['PRODUCT_ID'])->GetNextElement()->GetProperties()['VES_V_GRAMMAKH_SHAG']['VALUE'];

[sergeyviktorovich]

при сохранении из админки работает а товары
Не удалось получить текущее состояние процесса обмена. Данные обмена отправлены, но не загружены.
обработал все элементы, но не загрузил

<?
/*коэффициент единицы измерения при выгрузке*/

$eventManager = \Bitrix\Main\EventManager::getInstance();
$eventManager->addEventHandler(
    'catalog',
    '\Bitrix\Catalog\Price::OnAfterUpdate',
    static function ( $event ) {
    $arFields = $event->getParameter('fields');
    $element = $arFields['PRODUCT_ID'] ?? 0;
    
    if($element) {
    
      $iblockId = 9;
      $coeff= CIBlockElement::GetByID($arFields['PRODUCT_ID'])->GetNextElement()->GetProperties()['VES_V_GRAMMAKH_SHAG']['VALUE'];
    //например
 if(!empty($coeff)) { 
        $curElementRatio = CCatalogMeasureRatio::getList(
        Array(),
        array('IBLOCK_ID' => $iblockId, 'PRODUCT_ID' => $element ),
        false, false);

    while ($arRatio = $curElementRatio->GetNext()) {
        $ratioId = $arRatio['ID'];
        $arFieldsNew = Array(
            'RATIO' => $coeff,
        );

        CCatalogMeasureRatio::update($ratioId, $arFieldsNew);
    }
     }  
    }
    }
   

);


?>