Всем привет. После установки последних обновлений битрикса появилась ошибка при выборке

Question

Всем привет. После установки последних обновлений битрикса появилась ошибка при выборке

разделов инфоблока через такое апи
\Bitrix\Iblock\Model\Section::compileEntityByIblock($arParams['IBLOCK_ID']);

Как оказалось необходимо приводить к типу данных $arParams['IBLOCK_ID']:
\Bitrix\Iblock\Model\Section::compileEntityByIblock((int)$arParams['IBLOCK_ID']);

Хотя раньше работало. Никто не знает почему в ядре они перестали приводить к типу данных номер инфоблока? Может это было сделано специально для чего-то?
Этот вопрос скорее всего к техподу битрикса. Может кому пригодится.

#1c #bitrix #cms #programming #russian #software

0

22.10.2021

9 ответов

30 просмотров

Владислав Vlad Автор вопроса

Anton Vaniushin
Полистайте выше уже тоже словили. Думаю починят от...

Спасибо. Но я весь чат не успеваю читать по времени. А то что читал, не встречал этого.

0

22.10.2021

Артём Листиков

Там усложнилась логика приведения входящей переменнной. Теперь на вход принимается как строка, инт, так и инстанс Iblock. Но если передаётся строка, то до проверки is_numeric заходит в проверку is_string где предполагается что если передаётся стринг, то это код апи. И пытается найти инфоблок по коду апи и зачем-то переопределяется входящая переменная результатом этого поиска. Логично что не будет существовать инфоблока с апи кодом равным айдишнику. Следующие проверки проваливаются потому что теперь входящая переменная null. И собственно поэтому возвращает null в основной функции.

0

22.10.2021

Владислав Vlad Автор вопроса

Артём Листиков
Там усложнилась логика приведения входящей перемен...

Отлично. Вообще по хорошему всегда лучше приводить к типам, но php нас разбаловал в этом.

0

22.10.2021

Артём Листиков

Владислав Vlad
Отлично. Вообще по хорошему всегда лучше приводить...

По хорошему не просто приводить, а строго типизировать в самой функции. php 8 теперь позволяет указывать несколько типов для одного аргумента.

0

22.10.2021

Артём Листиков

Артём Листиков
По хорошему не просто приводить, а строго типизиро...

Но учитывая что раньше у этих функций типизации не существовало - внедрение типов тоже обернётся проблемами

0

22.10.2021

Владислав Vlad Автор вопроса

Артём Листиков
Но учитывая что раньше у этих функций типизации не...

Да. Перевод на php 8 будет проблематичным. Да и там вроде как добавляется обязательно <?php вместо <? (исключение если выводится сразу что-то из переменной например)

0

22.10.2021

Eugene Zhukov

Владислав Vlad
Отлично. Вообще по хорошему всегда лучше приводить...

В тп напишите. Чем больше обращений, тем быстрее выпуск фикса клиентам.

0

22.10.2021

Eugene Zhukov

Артём Листиков
По хорошему не просто приводить, а строго типизиро...

(задумчиво) в новом коде ?тип мне пока хватает...

0

22.10.2021