Ошибка в коде

[klmn]

При заходе на страницу с анонсом статьи выводится уведомление об использовании неопределенной переменной:

Notice: Undefined variable: html in /..../sape.php on line 820

PHP-клиент, версия 1.0.5 от 12.01.2010.

[chahlic]

Сообщение от klmn

При заходе на страницу с анонсом статьи выводится уведомление об использовании неопределенной переменной:

Notice: Undefined variable: html in /..../sape.php on line 820

PHP-клиент, версия 1.0.5 от 12.01.2010.

Мне помогла замена

PHP код:

$html .= join($this->_announcements_delimiter, $links); 


на

PHP код:

$html = join($this->_announcements_delimiter, $links); 


[klmn]

Спасибо.

Еще одна ошибка - используется неопределенное свойство класса $this->_sape_charset на 385 строке.
У меня ошибка видна на локальном сервере.

Хотелось бы, чтобы программисты были более внимательны.

[Jooz]

Сообщение от chahlic

Мне помогла замена

PHP код:

$html .= join($this->_announcements_delimiter, $links); 


на

PHP код:

$html = join($this->_announcements_delimiter, $links); 


Не смотрел, но "накопительная" переменная не случайно же?
Лучше попробовать так:

PHP код:

$html.=join($this->_announcements_delimiter, $links); 


т.е. убрать ведомые пробелы, в какой-то версии php с этим баг был (помню сутки убил), а по идее должно работать.

[chahlic]

Сообщение от Jooz

Не смотрел, но "накопительная" переменная не случайно же?

Не сильный спец в пхп, но перед тем, как менять код посмотрел реализацию вывода ссылок по частям (а мы говорим именно об ф-циях, которые позволяют разбить вывод ссылок и анонсов статей по блокам) в старой версии sape.php, который у меня год работал - там нет этого накопления. Появилось оно в новой версии, как для ссылок, так и для статей.

Вроде переменная $html не доступна за пределами этих функций, а значит накопление там и не надо - переменная каждый раз заново создается.

Ну, это мое нубское мнение, но пока проблем не возникало

Добавлено через 7 минут

Сообщение от Jooz

убрать ведомые пробелы, в какой-то версии php с этим баг был (помню сутки убил), а по идее должно работать.

попробовал - сразу те же сообщения полезли

Проблема не нова, просто не критична (еррор лог забивает), вот админы и не "чешутся", а вообще из бы услышать тоже не мешало бы