Support » Plugin: True Lazy Analytics » После обновления весь лог забит PHP Warning

  • Resolved Yworld

    (@yworld)


    Здравствуйте.
    После обновления последнего, лог начал забиваться варнингами (setting-page.php):
    Trying to access array offset on value of type bool in – строки 453,477,499,543
    Undefined array key “tlap_yametrika_webvisor” in: строка 535
    Версия PHP 8, прошлая версия без проблем, откатил. Но судя по всему и в 7.4 должна быть проблема.
    Буду признателен, если обратите внимание.
    И возможно, в будущем плагину придется менять функцию, посмотрите предупреждение в хроме, но это в будущем))

    • This topic was modified 1 month, 4 weeks ago by Yworld.
Viewing 12 replies - 1 through 12 (of 12 total)
  • Thread Starter Yworld

    (@yworld)

    Как вариант исправления, добавить isset
    Что то типо того:
    $g_id = isset($all_options['tlap_analytics_id']) ? $all_options['tlap_analytics_id'] : '';
    line 453,477,499,543,527

    Plugin Author seojacky

    (@seojacky)

    Здравствуйте. К сожалению делал совместимость с версией PHP 5.6 и видимо этот вариант несовместим с другими версиями.
    Вы проверили вариант с isset?
    Сейчас нет времени тестировать. Если вариант выше рабочий, то могу быстро пофиксить и залить новую версию.

    • This reply was modified 1 month, 4 weeks ago by seojacky.
    Thread Starter Yworld

    (@yworld)

    @seojacky Здравствуйте.
    На 8 версии проверил без ошибок, единственно пока не дошел проверить на 7.4, и опцию адвизора, она так же пустая идет в скрипт и варнинг.
    Я отпишусь, как все проверю.
    Спасибо за вашу работу.

    Thread Starter Yworld

    (@yworld)

    @seojacky
    Проверил на PHP 7.4 и 8, ниже версии сейчас не могу проверить.
    Все работает без ошибок. Небольшое дополнение, к вебвизору дописал, что бы неопределенный ключ не попадал в логи.
    Под 529 строку:
    if(isset( $webvisor_checkbox ) && 0 == $webvisor_checkbox) { $webvisor = '';}

    Вроде все работает, но как писал выше, не знаю, что будет на ниже версиях php.

    Plugin Author seojacky

    (@seojacky)

    Спасибо. Вечером сделаю фикс

    Plugin Author seojacky

    (@seojacky)

    Внёс некоторые изменения в версию 2.2.7. Должно решить проблему.

    Thread Starter Yworld

    (@yworld)

    @seojacky Здравствуйте.
    Все как прежде, варнинги остались и вижу по коду, что isset добавился только в вебвизор и этого предупреждения больше нет, остальные на месте.
    строки 453,477,499.
    Внес изменения себе и предупреждения исчезли, а то сразу не проверил и их просто мего тонны))

    • This reply was modified 1 month, 3 weeks ago by Yworld.
    Plugin Author seojacky

    (@seojacky)

    Странно у меня не было замечаний. Поэтому и не изменял больше код. Ладно допишу isset в другие пункты.

    Thread Starter Yworld

    (@yworld)

    @seojacky, Здравствуйте.
    Вы на php 8 (я проверил и на 8.3) проверяли? По идеи не может не быть предупреждений, правила же в 8 изменились относительно “строгости” так сказать))

    • This reply was modified 1 month, 3 weeks ago by Yworld.
    Plugin Author seojacky

    (@seojacky)

    Здравствуйте. Обновил до версии 2.2.8. Проверил на PHP8 – никаких ошибок нет

    Thread Starter Yworld

    (@yworld)

    Здравствуйте @seojacky .
    Проверил на тестовом все ок после обновления. Единственно в админке предупреждение.
    Скрин: https://prnt.sc/1w5elx2
    Или в логе посмотрите.

    Спасибо за работу над плагином и конечно за полезный плагин!

    • This reply was modified 2 days ago by Yworld.
    Plugin Author seojacky

    (@seojacky)

    А у меня нет таких предупреждений…. Странно….
    В следующем релизе поправлю и это

Viewing 12 replies - 1 through 12 (of 12 total)
  • You must be logged in to reply to this topic.