bitrix-tools / crest

Tiny PHP SDK to call Bitrix24 REST methods via webhooks and OAuth 2.0

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

поменяйте пожалуйста 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' ])) {

commented

параметр 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)) {

commented

параметр 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