exstreme / Jcomments-4

Modified Jcomments 3.0.7 with Recaptcha 2 in comments. Adapted to Joomla 4. No compatibility with Joomla 1.5\2.5\3.x
16 stars 12 forks source link

php 8.1 Undefined property: stdClass::$profileLinkTarget in tpl_comment.php on line 50 #168

Closed kureshtar closed 1 month ago

kureshtar commented 2 months ago

Добрый день. Jcomments v4.0.30, Joomla 4.4.4. После перехода с php 7.4 на php 8.1 в логах стало писать вот такой warning: "Undefined property: stdClass::$profileLinkTarget in /components/com_jcomments/tpl/default/tpl_comment.php on line 50" Подобные ворнинги вроде лечатся оборачиванием в isset: isset($comment->profileLinkTarget) правда не знаю на сколько это правильное решение. Заранее спасибо.

exstreme commented 2 months ago

@kureshtar А что в плагине Аватара выставлено для этого поля?

Globulopolis commented 2 months ago

@kureshtar попробуйте заменить ваш jcomments.php этим файлом https://raw.githubusercontent.com/Globulopolis/Jcomments-4/master/component/site/jcomments.php Дайте знать про результат.

kureshtar commented 2 months ago

Спасибо. Да, вы верно подметили. В плагине аватара у меня свое кастомное поле выставлено для зарегистрированных Seblod пользователей. Данный ворнинг появляется в логфайле только на страницах с комментариями, которые оставили пользователи, зарегистрированные через Seblod (в комментарии проставляется ссылка и фото на их страницу seblod-профиля). На страницах без комментариев или с комментариями, которые оставили незарегистрированные пользователи или пользователи Joomla, такого ворнинга нет.

Замена файла jcomments.php ничего не поменяла, ворнинг также появляется.

Globulopolis commented 2 months ago

@kureshtar скриньте скрин настроек плагина аватара.

kureshtar commented 2 months ago

Прикладываю скрин: image

Globulopolis commented 2 months ago

Откуда Seblod? В текущей версии плагина нет этого.

kureshtar commented 2 months ago

Верно. Выше писал, что это свое кастомное поле выставлено. Если это с ним проблема, тогда ок. Спасибо.

exstreme commented 2 months ago

Расскажите, как добавляли кастомное поле, думаю стоит просто добавить поддержку Seblod в плагин

kureshtar commented 1 month ago

Добрый день. Я не программист, поэтому у меня очень кастомное решение , оно не универсальное, т.к. там идет запрос к базе, в которой из-за специфики себлода названия полей и таблиц в себлоде для типов контента могут иметь абсолютно любые значения (какие задаст вебмастер). Там наверное нужно в свойствах плагина при выборе Seblod запрашивать еще имена себлодовских типов контента и имя поля в котором хранится аватар, а также номер категории, в которой публикуются страницы пользователей. А возможно у кого-то это вообще по другому реализовано в себлоде.