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

[LavIgor]

nissin, м.б. проще тогда амперсанд перед оператором присваивания добавить?

К тому же помню, что PHP без необходимости копирование не осуществляет, хотя, по-моему, это к аргументам функций относится.

[nissin]

Можно и амерсанд добавить, но проще вообще убрать.

[LavIgor]

проще вообще убрать.

Что убрать?
Амперсанд же в исходный вариант добавится, правок меньше и для понимания проще.

[nissin]

Найти:

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

		$context = $this->context->get_data_ref();
		$context_vars = $context['.'][0];

Заменить

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

		$context = $this->context->get_data_ref();
		$context_vars = &$context['.'][0];

$context дальше не используется, поэтому можно и так заменить:

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

		$context_vars = $this->context->get_root_ref();

[LavIgor]

поэтому можно и так заменить

Так даже лучше будет.

[Sumanai]

К тому же помню, что PHP без необходимости копирование не осуществляет, хотя, по-моему, это к аргументам функций относится.

Да почти везде используется Copy on Write.

[nissin]

Re: Не пересобирается поисковый индекс
В новой версии добавили проверку форму при создании поисковых индексов, но исключение неверно прописали.
OPEN includes\acp\acp_search.php
FIND

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

		if (!check_form_key($form_key) && in_array($action, array('delete', 'create')))

REPLACE WITH

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

		if (!check_form_key($form_key) && !in_array($action, array('delete', 'create')))

[rxu]

добавили проверку форму при создании поисковых индексов, но исключение неверно прописали

https://github.com/phpbb/phpbb/pull/4489

[nissin]

Спасибо. Я до сих пор git не освоил

[rxu]

nissin, сделал пулл и понял - а почему должен быть ! ? Если результат проверки формы неверный - должно быть без разницы.

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

[nissin]

Для 'delete' и 'create' должно быть исключение. А так получается, что только для них и срабатывает.

[rxu]

Не понял, поясни развернуто.

[nissin]

Переиндексация идёт частями.
Соответственно action 'delete' и 'create' во второй и далее раз вызываются не из формы, а через redirect, а там ключ не отправляется.

[rxu]

а там ключ не отправляется

А почему? Добавлено ведь и add_form_key($form_key); тоже.

[nissin]

А кто же его отправлять будет

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

meta_refresh(1, append_sid($this->u_action . '&action=create&skip_rows=' . $post_counter));

Даже если добавить здесь, через GET он сработает.