Сообщаем об ошибках phpBB 3.1.x разработчикам

[rxu]

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

[VADR]

Обновился с 3.1.1 до 3.1.2. При проверке версий установленных расширений (Персонализация -> Управление насширениями -> Перепроверить все версии) вылетает ошибка:

Fatal error: Wrong parameters for Exception([string $exception [, long $code [, Exception $previous = NULL]]]) in /home/asutpforum/test31.asutpforum.ru/docs/phpbb/file_downloader.php on line 72

Проблема, как выяснилось, была в наличии очень кривого расширения (вернее, расширение в очень ранней стадии разработки, "phpBB Blog"). Такая же ошибка вылетает по ссылке "Информация" для этого расширения. На версиях 3.1.1 и 3.1.0 такого не было.

[владимир1983]

Данная тема предназначена для сообщения разработчикам об ошибках не модифицированного кода phpbb. Твоя проблема, это проблема расширения блога, и к данной теме не относится.

[VADR]

Данная тема предназначена для сообщения разработчикам об ошибках не модифицированного кода phpbb. Твоя проблема, это проблема расширения блога, и к данной теме не относится.

Я бы так не сказал. Да, расширение кривое, оно даже толком не поставилось - пришлось немного подпиливать руками composer.json, чтобы расширение хотя бы опозналось движком (и даже после этого включить его не удалось). При запросе ядром информации по расширению вылезает что-то неудобоваримое, что вызывает исключение, и вот тут - ошибка вызова исключения, которой в версиях 3.1.0 и 3.1.1 не было! Я всё же думаю, что проблема именно в коде ядра.

[Татьяна5]

Если ошибка вызвана расширением - то проблема в расширении
(Авторы меняют расширения под код ядра, но не наоборот)

[VADR]

Если ошибка вызвана расширением - то проблема в расширении

А если в обработке данных расширения ядром? Ведь исключения не просто так используются, а как раз на тот случай, если разработчики расширений что-то накосячили. И проблема вылезает именно при вызове исключения: ему переданы неправильные параметры и сделано это в ядре.
В общем, попробую снова этот блог поставить, прицеплюсь через firephp и посмотрю, что же там на самом деле происходит...

[владимир1983]

VADR Проблема с расширением? Иди за поддержкой в тему расширения.
Заканчиваем флудить в данной теме.

[VADR]

А если в обработке данных расширения ядром? Ведь исключения не просто так используются, а как раз на тот случай, если разработчики расширений что-то накосячили. И проблема вылезает именно при вызове исключения: ему переданы неправильные параметры и сделано это в ядре.

Как я и предполагал, проблема таки в ядре. В файле phpbb/file_downloader.php в 72-й строке:

Код: Выделить всё

throw new \RuntimeException(array('FILE_NOT_FOUND', $filename));

в то время как конструктор RuntimeException в качестве параметра при вызове принимает не массив, а строку. Изменение таким вот образом решает эту проблему:

Код: Выделить всё

throw new \RuntimeException('FILE_NOT_FOUND' . $filename);

Правда, при этом вылезают другие (а то ж... расширение-то кривое), но не в том суть - от неправильного обращения к классу избавляемся.
Впрочем, далее вылезает такое: в файле phpbb/version_helper.php в строке 262:

Код: Выделить всё

throw new \RuntimeException(call_user_func_array(array($this->user, 'lang'), $exception->getMessage()));

$exception - экземпляр класса RuntimeException, метод getMessage наследуется у класса Exception и возвращает строку, но выходит такое вот сообщение:

[phpBB Debug] PHP Warning: in file [ROOT]/phpbb/version_helper.php on line 262: call_user_func_array() expects parameter 2 to be array, string given

То есть функция call_user_func_array должна принимать вторым параметром массив, а пришла строка. Как раз получается, что это тот самый массив, который хотели отправить в 72-й строке файла file_downloader.php, но не смогли...
Да, рановато ставить версию 3.1.2 на рабочий форум...

VADR Проблема с расширением? Иди за поддержкой в тему расширения.
Заканчиваем флудить в данной теме.

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

[rxu]

VADR
Скорее всего прав https://www.phpbb.com/community/viewtop ... #p13826371

[Sumanai]

Здравствуйте!
Нашёл такой баг- если использовать галерею аватар, и попытаться выбрать аватару из галереи, не состоящую в категории, то выводится ошибка

Указанный файл аватары не найден

Ошибка в коде, файл \phpbb\avatar\driver\local.php, строка

Код: Выделить всё

		if (!isset($avatar_list[$category][urldecode($file)]))

Данная проверка корректна только для аватар, состоящих в какой- либо категории. И дальше при задании размеров, так же нужно учесть случай аватаров без категории.

[Sumanai]

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

[Борис Бердичевский]

Я в своём драйвере уже давно обнаружил одну штуку и применил один трюк

Вот, без этой поправки фиг будет работать драйвер sqlite3. Проверено!

phpbb\db\driver\sqlite3.php
Find:

Код: Выделить всё

class sqlite3 extends \phpbb\db\driver\driver
{

Add after:

Код: Выделить всё

	var $n_count = 2;		/* count of attempts */
	var $n_msec = 250000;	/* time-out in millisecons (bisyTimeout)*/

Find:

Код: Выделить всё

public function sql_query($query = '', $cache_ttl = 0)

Replace all the function:

Код: Выделить всё

	public function sql_query($query = '', $cache_ttl = 0)
	{
		if ($query != '')
		{
			global $cache;
			$was_error = false;

			// EXPLAIN only in extra debug mode
			if (defined('DEBUG'))
			{
				$this->sql_report('start', $query);
			}
			else if (defined('PHPBB_DISPLAY_LOAD_TIME'))
			{
				$this->curtime = microtime(true);
			}

			$this->last_query_text = $query;
			$this->query_result = ($cache && $cache_ttl) ? $cache->sql_load($query) : false;
			$this->sql_add_num_queries($this->query_result);

			if ($this->query_result === false)
			{
				$nc=0;
				for( $nc=0; $nc< $this->n_count; $nc++)
				{
					try {
			
						$this->dbo->busyTimeout($this->n_msec);
						$err_level = 0;
						if ($this->return_on_error) {
							$err_level = error_reporting(0);
						}

						if (($this->query_result = @$this->dbo->query($query)) === false)
						{
							$was_error = true;
						}

						if ($this->return_on_error && $nc >= $this->n_count -1) {
							error_reporting($err_level);
						}
					}
					catch  (Exception $error)
					{
						$was_error = true;
					}

					if (!$was_error) {
						break;
					}

					$was_error = false;
				}
				
				if (defined('DEBUG'))
				{
					$this->sql_report('stop', $query);
				}
				else if (defined('PHPBB_DISPLAY_LOAD_TIME'))
				{
					$this->sql_time += microtime(true) - $this->curtime;
				}

				if ($cache && $cache_ttl)
				{
					$this->query_result = $cache->sql_save($this, $query, $this->query_result, $cache_ttl);
				}
			}
			else if (defined('DEBUG'))
			{
				$this->sql_report('fromcache', $query);
			}
		}
		else
		{
			return false;
		}

		return $this->query_result;
	}

Добавлено спустя 26 минут 55 секунд:
https://tracker.phpbb.com/browse/PHPBB3-13423

[romaamor]

Заметил сегодня на всех своих форумах такое сообщение -



Это только у меня одного такое или у всех ?

[владимир1983]

У всех. Это говорит о том что сервер обновлений не доступен. Это не является ошибкой скрипта.

[romaamor]

Подумалось сразу что это мой хостинг шалить. Или готовят очередное обновление.