Смайлы в комментариях пользователей

[ZlydenGL]

Давно уже замечал, что в комментариях пользователей к статьям смайлы выводились довольно забавно: прямоугольник типа "изображение недоступно" с нормальной подписью смайла в alt-теге, при этом путь к изображению задавался пустой.

Начал ковырять и выяснил, что из-за того, что класс textsanitizer'а в myarticles подключается мягко говоря криво (а точнее - через расширение этого класса), то не отрабатывает процедура задания стандартного пути для смайлов.

Решил проблему очень просто: в процедуре class/module.textsanitizer.php функцию

PHP код:

    function smiley($message)
    {
        if(!$this->smileys)
            $this->getAllSmilesFromDB();
        foreach ($this->smileys as $s)
        {
            $message = str_replace($s['code'], '<img src="'.formatURL($this->smilies_path, $s['smile_url']).'" alt="'.$s['code'].'" align="middle" />', $message);
        }
        return $message;
    } 


заменил на

PHP код:

    function smiley($message)
    {
        if(!$this->smileys)
            $this->getAllSmilesFromDB();
        foreach ($this->smileys as $s)
        {
            if (!$this->smilies_path) $this->smilies_path = XOOPS_IMG_URL.'/smilies/';
            $message = str_replace($s['code'], '<img src="'.formatURL($this->smilies_path, $s['smile_url']).'" alt="'.$s['code'].'" align="middle" />', $message);
        }
        return $message;
    } 


(т.е. просто добавил проверку на существование переменной; если она не задана - задаем ее вручную).

Комментарии в статьях тут же запестрели нормальными графическими смайлами

Наверняка можно ту же задачу сделать, отредактировав только класс-расширение санитайзера от модуля MyArticles (т.е. файл myarticles/class/textsanitizer.class.php), но как это сделать пока не знаю.

[Michael-XIII]

Хак

Как-бы бага тут

PHP код:

class ArtTextSanitizer extends MyTextSanitizer {
    function ArtTextSanitizer()
    {
        global $xoopsConfig;

        $this->setAllowImage($xoopsConfig['allow_image']);
        $this->setAllowLibrary($xoopsConfig['allow_library']);
    } 


Заменить на что-то типа

[php]

PHP код:

class ArtTextSanitizer extends MyTextSanitizer {
    function ArtTextSanitizer()
    {
        global $xoopsConfig;
                parent::MyTextSanitizer();

        $this->setAllowImage($xoopsConfig['allow_image']);
        $this->setAllowLibrary($xoopsConfig['allow_library']);
    } 


или для php 5 (что более правильно)

parent::__construct();

Не проверял ибо проект не поднят сейчас

[LARK]

Сообщение от ZlydenGL textsanitizer'а в myarticles подключается мягко говоря криво (а точнее - через расширение этого класса)

он не криво подключается, а расширяется с переопределением метода.

Сообщение от Michael-XIII Хак Как-бы бага тут

сенкс.

Баг характерен для старших версий. Последняя версия MyArticles писалась под RUNCMS 1.4, там не было свойства устанавливающего путь к смайлам, да и конструктора у MyTextSanitizer не было. Баг появляется в линейке 1.6

[Michael-XIII]

>Баг характерен для старших версий. Последняя версия MyArticles писалась под RUNCMS 1.4, там не было свойства устанавливающего путь к смайлам, да и конструктора у MyTextSanitizer не было. Баг появляется в линейке 1.6

Ларк, не юли ;-) Если наследуешься от базового класса - делай это правильно ;-) ;-) ;-)

[LARK]

Сообщение от Michael-XIII Ларк, не юли ;-) Если наследуешься от базового класса - делай это правильно ;-) ;-) ;-)

Так я о чем - не было в 1.4 для которой писался модуль в MyTextSanitizer конструктора и вызов несуществующей функции приведет к фатальной ошибке. А для версии 1.6 я модуль не писал. Мы вообще о PHP говорим или о каком другом языке?

[iltmpz]

Спасибо, оба варианта для комментариев работают.
А что делать с самим текстом статьи?
Там смайлик выглядит вот так: <img src="" alt=":-)" />