Hyyan Polylang for Woocommerce — как исправить (очередную) ошибку, чтобы работало с 8 версией php?

Question

[alexiusgrey]

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

В этот раз в админке в настройках вукоммерса не показывались настройки емейлов.

Ошибка

Fatal error: Uncaught Error: Attempt to assign property "locale" on bool
in /var/тут-пути-к-моей-папке-на-хосте/wp-content/plugins/woo-poly-integration/src/Hyyan/WPI/Utilities.php on line 645

Нужный файл плагина на гитхабе

Строчка 645 выглядит как
$polylang->curlang->locale = $languageLocale;
Если закомментировать, то мейлы появляются.
Но наверно комментировать не вариант.
Где неправильность в том, что "locale" это bool?

Comments

[Василий]

там скорей всего тип явно указан. Найди то место и убери указание типа.

[Refguser]

С момента смерти Hyyan Polylang ВП обновился 20 раз, а WC 60. Куда его ещё тянуть.
Он о php 8 заморачивается :) Да с php 8 ещё не работает добрая половина актуальных плагинов...

[alexiusgrey]

Refguser, альтернативы такие себе. не переделывать же старые сайты с нуля

[Refguser]

alexiusgrey, раз плагин умер, то остаётся переделывать. Или не обновлять ничего, но это тупик, в который всё равно упрёшься. Поэтому чем раньше задумаешься о переделке, тем лучше.

А заморачиваться на php8 точно не стоит.

[alexiusgrey]

Refguser, значит все еще пока не так и плохо, мне пришлось пофиксить всего три моментика с типизацией(я в новой типизации вообще бум-бум). в остальном вроде лично у меня оно работало в мертвом виде.

[Refguser]

alexiusgrey, а, и да. Не думаю, что используя родной полиланговский аддон для WC придётся так уж сильно "переделывать". Но да, он стоит денег и наверняка не так хорош, как был бесплатный Hyyan (я их сравнивал ещё когда Hyyan был жив. Тогда платный не имел некоторых фич/удобств, которые были в Hyyan)

[alexiusgrey]

Refguser, на php8 хостинг. когда на 7.4 у меня с Hyyan багов не было, вот на 8ках начались. Когда не просто там что-то ругается, такого много даже в ядре, а когда оно прям мешает сайту. Это третий.

Я верю в Hyyan, jQuery и Windows 7 - пока все альтернативы хуже.
Почему тупик еще какое-то время потянуть его исправлениями ошибок? Это же я тупой в новой типизации, а кто не упые тем просто увидеть эти мелкие ошибки. Исправить и можно еще попользоваться или вообще вытянуть на энтузиазме, автор вроде даже не против.

[Refguser]

alexiusgrey,

в остальном вроде лично у меня оно работало в мертвом виде.

Вероятно ты пока не понимаешь что это риски. Риски для бизнеса, а не просто сайта. Рекомендую почитать что очень давно писал один умный человек #16 от RiDDi. (Только не надо мне писать по прочитанному)

А вообще да.. кто я такой чтобы уговаривать делать правильно..

[Refguser]

на php8 хостинг

Эту отмазку я вообще не понимаю. WC и шаред - понятия не совместимые в принципе, а тут ещё и мультияз. Или такой биз, что на ВПС нет 800 руб/мес?
Но даже на шареде есть выбор версий php. Не может быть шареда без 7,4.
Да и сменить хостинг всегда можно.

[alexiusgrey]

Есть 74 на выбор. Там от допотопных 5.5 до 8.2.
Решил что "пусть будет уже 8.0". Она там стояла по умолчанию.
А что 74 лучше оставить и не ставить пока 8?

Answer 1

[Сергей delphinpro]

Где неправильность в том, что "locale" это bool?

С чего вы это взяли?
Ясно же написано: попвытка обратиться к свойству locale у переменной типа bool
То есть $polylang->curlang - bool (false)
Нужно выяснить, почему там false? а не объект, и поправить.
Или тупо добавить проверку

if ($polylang->curlang) {
  $polylang->curlang->locale = ...
}

но это лечение симптомов, а не болезни.

Comments

[alexiusgrey]

проверка помогла, спасибо

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