Расширения - это просто!

[Webliberty]

Поискал в инете примеры с проверкой GET, собрал вот такое:

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

	public function get_post_data($event)
	{
		$forum_id = $event['forum_id'];
		$topic_id = $event['topic_id'];
		$next = $_GET['next'];
		$previous = $_GET['previous'];

		if (!$next)
		{
		$this->template->assign_vars(array(
			'U_VIEW_NEWER_TOPIC'	=> redirect(append_sid("{$this->phpbb_root_path}viewtopic.$this->php_ext", "f=$forum_id&t=$topic_id"))
		));
		}
		return;

		if (!$previous)
		{
		$this->template->assign_vars(array(
			'U_VIEW_OLDER_TOPIC'	=> redirect(append_sid("{$this->phpbb_root_path}viewtopic.$this->php_ext", "f=$forum_id&t=$topic_id"))
		));
		}
		return;
	}

и получил ошибку:

Illegal use of $_GET. You must use the request class or request_var() to access input data.

[Anvar]

seodirection, Используй функцию request_var()

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

	public function get_post_data($event)
	{
		$forum_id = $event['forum_id'];
		$topic_id = $event['topic_id'];
		$next =  request_var('next', '');
		$previous =  request_var('previous', '');
		if ($next || $previous)
		{
			return redirect(append_sid("{$this->phpbb_root_path}viewtopic.$this->php_ext", "f=$forum_id&t=$topic_id"));
		}
	}

[rxu]

Используй функцию request_var()

А еще лучше - $request->variable().

[Shredder]

rxu
Чем это отличается?

[rxu]

request_var() устаревшая функция (deprecated), в 3.1 используются методы класса \phpbb\request\request. Прописан сервис @request. В целом методы класса более продвинуты, чем функция request_var().

[LavIgor]

Итоговая функция должна выглядеть как-то так.

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

public function get_post_data($event)
	{
		$forum_id = $event['forum_id'];
		$topic_id = $event['topic_id'];
		$view =  $request->variable('view', '');
		if ($view == 'next' || $view == 'previous')
		{
			redirect(append_sid("{$this->phpbb_root_path}viewtopic.$this->php_ext", "f=$forum_id&t=$topic_id"));
			return false;
		}
	}

[Webliberty]

Немного подумал и пришел к такому выводу: все-таки ссылки viewtopic.php?f=1&t=1&view=previous и viewtopic.php?f=1&t=1&view=next ведут на разные страницы, а не на viewtopic.php?f=1&t=1 поэтому скорее всего я просчитался, настраивать подобный редирект не логично.

Поэтому сделал такое решение, которое тоже крутилось в голове: все страницы с previous и next в url сделал несуществующими, проверил ответ сервера - 404. Это на сто процентов поможет исключить такие страницы из индекса раз и навсегда!

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

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

	public function remove_nextprev($event)
	{
		$nextprev =  request_var('view', '');
		if ($nextprev)
		{
			trigger_error('NO_TOPIC');
		}
	}

[Anvar]

seodirection, Ну это уже твои заморочки, кто важнее пс или пользователь. Я лично обделять последних не собираюсь.
Можно и остальные дубли ликвидировать, например определить тему с неуказанным параметром &f=, то тоже на редирект.

[Shredder]

Как использовать языковую переменную во время установки, в функции update_data?

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

array('config.add', array('my_message', $this->user->lang['MY_VARIABLE'])),

Это не прокатывает, хотя в 3.0.12 прекрасно работало, а $this->config и сейчас использовать можно.

Добавлено спустя 7 минут 39 секунд:
Вместо добавления переменной происходит General Error, где она якобы равна NULL

[rxu]

Объекта $user нет в миграторе. А можно подробности, зачем локализованную языковую строку толкаем в таблицу конфига?

[Shredder]

Дефолтное сообщение для мода.

Если простого способа нет и нужно заморачиваться, то фиг с ним, добавлю пустую строку.

[rxu]

Дефолтное сообщение для мода.

Так может, не локализованное значение записывать, а индекс языковой переменной, т.е.

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

array('config.add', array('my_message', 'MY_VARIABLE')),

и уже потом в коде расширения подсовывать это значение в $this->user->lang()?

[Shredder]

В данном случае это смысла не имеет, т.к. функция переменной планировалась та же самая, как и у "ваш_домен.ru" в настройках конференции - просто подставить "демо"-значение при установке, которое при желании потом можно будет сменить на что-то своё.

[rxu]

Shredder
ok, тупо global $user; не прокатывает?

[Shredder]

rxu
Я сначала так и хотел, но забыл подключить язык мода через $user->add_lang_ext, как делал в "оригинале" 3.0.12
Теперь всё работает.

Метод с global чем-то хуже?