поменяйте пожалуйста 36ю строчку, добавив проверку на существование значения
muravikov opened this issue · comments
параметр event не всегда передаётся битриксом. из за этого выскакивают лишние notice, которые на пример в AJAX ответе могут всё испортить.
было
if ($_REQUEST[ 'event' ] == 'ONAPPINSTALL' && !empty($_REQUEST[ 'auth' ])) {
стало
if (isset($_REQUEST[ 'event' ]) && $_REQUEST[ 'event' ] == 'ONAPPINSTALL' && !empty($_REQUEST[ 'auth' ])) {
параметр event не всегда передаётся битриксом. из за этого выскакивают лишние notice, которые на пример в AJAX ответе могут всё испортить.
было if ($_REQUEST[ 'event' ] == 'ONAPPINSTALL' && !empty($_REQUEST[ 'auth' ])) { стало if (isset($_REQUEST[ 'event' ]) && $_REQUEST[ 'event' ] == 'ONAPPINSTALL' && !empty($_REQUEST[ 'auth' ])) {
ну тогда уж на:
if (($_REQUEST[ 'event' ] ?? null) === 'ONAPPINSTALL' && !empty($_REQUEST[ 'auth' ] ?? null)) {
параметр event не всегда передаётся битриксом. из за этого выскакивают лишние notice, которые на пример в AJAX ответе могут всё испортить.
было if ($_REQUEST[ 'event' ] == 'ONAPPINSTALL' && !empty($_REQUEST[ 'auth' ])) { стало if (isset($_REQUEST[ 'event' ]) && $_REQUEST[ 'event' ] == 'ONAPPINSTALL' && !empty($_REQUEST[ 'auth' ])) {ну тогда уж на:
if (($_REQUEST[ 'event' ] ?? null) === 'ONAPPINSTALL' && !empty($_REQUEST[ 'auth' ] ?? null)) {
Мда, жесть конечно.
bitrix программист не знает как работает empty