Разработчикам

Форум

После свежего обновления сломался сайт, Перестала работать выборка элементов инфоблока, вываливает все сразу

Пользователь 181407

Постоянный посетитель

Сообщений: 68 Баллов: 10 Регистрация: 09.08.2015

26.09.2022 11:41:02

Привет! Недавно вышло обновление системы, я обновил и на сайте перестали работать фильтры такого вида:

Код

<?
                      $arSelect = Array("CODE", "NAME", "ID", "ACTIVE_TO");
                      $arFilter = Array("IBLOCK_ID"=>"123","PROPERTY_USERID"=>$GLOBALS['USERID']);
                      $arOrder = array('SORT' => 'DESC'); 
                      $res = CIBlockElement::GetList($arOrder, $arFilter, false, Array("nPageSize"=>99), $arSelect);
                      while($ob = $res->GetNextElement())
                      {
                      $arFields = $ob->GetFields();
                      $channels[] = array(
                      'ID' => $arFields["ID"],
                      'NAME' => $arFields["NAME"],
                      'ACTIVE' => $arFields["ACTIVE_TO"],
                      'CODE' => $arFields["CODE"]);
                      };
?>

Фильтр должен был вывести элементы, привязанные к пользователю через свойство USERID. ID пользователя тут лежит в $GLOBALS['USERID']

Я думаю что это из-за этих изменений, но могу ошибаться:

[*]Разработчикам - изменена логика проверки глобального кеша свойств в методе CIBlockProperty::GetPropertyArray для возможности реализации интерфейса ArrayAccess.
[*]Разработчикам - исправлена ошибка фильтра вида PROPERTY_символьный_код с использованием фасетного индекса на установках с версией PHP 7.4 и выше.

Не подскажете, что могло сломать код? Пришлось откатывать базу, теперь боюсь обновлять систему.

Евгений Жуков Администратор Сообщений: 9534 Баллов: 1849 Регистрация: 05.08.2005 Epic fail - сказать "это так просто" и не суметь сделать	#2 0 26.09.2022 14:05:29 Обратитесь в техподдержку, пожалуйста. Потребуется административный доступ. Не надо сверлить зубы через задний проход дрелью от Сваровски

Пользователь 181407 Постоянный посетитель Сообщений: 68 Баллов: 10 Регистрация: 09.08.2015	#3 0 26.09.2022 17:39:45 Я боюсь что они опять что-то сломают или обновят систему без моего ведома. Страшно их пускать в админку. Или скажут опять "кастомные сборки не поддерживаем"

Пользователь 2062977

Посетитель

Сообщений: 37 Баллов: 5 Регистрация: 14.02.2019

27.09.2022 13:06:02

Код

function filterArrayByKeys(array $array, array $fields) {
    return array_map(function ($array) use ($fields) {
        $result = [];
        foreach ($fields as $v => $field) {
            if (array_key_exists($field, $array) || array_key_exists($v, $array)) {
                $result[is_string($v) ? $v : $field] = $array[$field];
            }
        }
        return $result;
    }, $array);
}

$entity = \Bitrix\Iblock\Iblock::wakeUp(123)->getEntityDataClass();

$elements = $entity::getList([
    'select' => ['CODE', 'NAME', 'ID', 'ACTIVE_TO', 'USERID_' => 'USERID'],
    'filter' => ['USERID_VALUE' => $GLOBALS['USERID']],
    'order'  => ['SORT' => 'ASC']
])->fetchAll();

$channels = filterArrayByKeys($elements, ['ID', 'CODE', 'NAME', 'ACTIVE' => 'ACTIVE_TO']);

Пользователь 181407

Постоянный посетитель

Сообщений: 68 Баллов: 10 Регистрация: 09.08.2015

28.09.2022 14:13:51

Цитата

написал:

Код

   function   filterArrayByKeys  (array  $array , array  $fields )  { 
     return  array_map(  function   ( $array )   use   ( $fields )  { 
         $result  = [];
         foreach  ( $fields   as   $v  =>  $field ) {
             if  (array_key_exists( $field ,  $array ) || array_key_exists( $v ,  $array )) {
                 $result [is_string( $v ) ?  $v  :  $field ] =  $array [ $field ];
            }
        }
         return   $result ;
    },  $array );
}

 $entity  = \Bitrix\Iblock\Iblock::wakeUp( 123 )->getEntityDataClass();

 $elements  =  $entity ::getList([
     'select'  => [ 'CODE' ,  'NAME' ,  'ID' ,  'ACTIVE_TO' ,  'USERID_'  =>  'USERID' ],
     'filter'  => [ 'USERID_VALUE'  =>  $GLOBALS [ 'USERID' ]],
     'order'   => [ 'SORT'  =>  'ASC' ]
])->fetchAll();

 $channels  = filterArrayByKeys( $elements , [ 'ID' ,  'CODE' ,  'NAME' ,  'ACTIVE'  =>  'ACTIVE_TO' ]);

Спасибо. Не могли бы вы пояснить ваш пример? Ведь мой пример из документации API, он теперь нерабочий? Нужно использовать такие громоздкие конструкции как вы показали?

Пользователь 2062977

Посетитель

Сообщений: 37 Баллов: 5 Регистрация: 14.02.2019

28.09.2022 14:42:34

Цитата

написал:

Цитата
Спасибо. Не могли бы вы пояснить ваш пример? Ведь мой пример из документации API, он теперь нерабочий? Нужно использовать такие громоздкие конструкции как вы показали?

В примере что я написал используется D7 API, и оно не громоздкое, обычные ORM конструкции, а самописная функция filterArrayByKeys делает выборку ключей из массива, а также меняет ключ, если в есть пара ключ-значение

Вместо конструкции, которая запрашивает класс сущности ORM для конкретного инфоблока:

Код
\Bitrix\Iblock\Iblock::wakeUp( 123 )->getEntityDataClass();

можно напрямую обращаться к сущности, зная его символьный код API, который задан в настройках инфоблока:

т.е:

Код
\Bitrix\Iblock\Elements\ElementCatalogTable::getList();

Опять же все подробно описано в документации

Пользователь 181407

Постоянный посетитель

Сообщений: 68 Баллов: 10 Регистрация: 09.08.2015

28.09.2022 17:13:17

Цитата

написал:

Цитата

написал:

Цитата
Спасибо. Не могли бы вы пояснить ваш пример? Ведь мой пример из документации API, он теперь нерабочий? Нужно использовать такие громоздкие конструкции как вы показали?

В примере что я написал используется D7 API, и оно не громоздкое, обычные ORM конструкции, а самописная функция filterArrayByKeys делает выборку ключей из массива, а также меняет ключ, если в есть пара ключ-значение

Вместо конструкции, которая запрашивает класс сущности ORM для конкретного инфоблока:

Код
\ Bitrix \ Iblock \ Iblock ::wakeUp( 123 ) - > getEntityDataClass ();

можно напрямую обращаться к сущности, зная его символьный код API, который задан в настройках инфоблока:

т.е:

Код
\ Bitrix \ Iblock \ Elements \ ElementCatalogTable ::getList() ;

Опять же все подробно описано в документации

Спасибо. Это означает, что апи до D7 теперь не работает? У наc на сайте множество мест, где использован мой пример выборки элементов. Их всех надо переписывать чтобы заработали после обновления?

Пользователь 114926 Постоянный посетитель Сообщений: 368 Баллов: 40 Регистрация: 13.08.2012	#8 0 29.09.2022 07:45:21 А значение $GLOBALS['USERID'] перед запросом вы проверяли, выводили в лог и т.п.? Там то, что нужно?

Евгений Жуков

Администратор

Сообщений: 9534 Баллов: 1849 Регистрация: 05.08.2005

Epic fail - сказать "это так просто" и не суметь сделать

29.09.2022 09:27:16

Цитата
написал: Или скажут опять "кастомные сборки не поддерживаем"

Что значит "кастомные сборки"?

Не надо сверлить зубы через задний проход дрелью от Сваровски

Пользователь 181407

Постоянный посетитель

Сообщений: 68 Баллов: 10 Регистрация: 09.08.2015

#10

03.10.2022 00:14:24

Цитата

написал:

Цитата
написал: Или скажут опять "кастомные сборки не поддерживаем"

Что значит "кастомные сборки"?

У нас несколько файлов модифицированы одним из решений. Из-за этого тех поддержка отказывается что-то делать.

Форум

Центр поддержки

Продукты

Управление сайтом

Битрикс24

Интернет-магазин + CRM

Решения

Для интернет-магазинов

Каталог готовых решений

Внедрение

Выбрать партнера

Проверить партнера

Стать партнером