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

[xisp]

$('#username_logged_in li.icon-logout a')

Прощай пользователи без яваскрипта, прощай производительность на мобильных.

[Татьяна5]

Прощай пользователи без яваскрипта

У пользователей без js расширение, написанное полностью на js, не заработает, и вместо него останется стандартный функционал форума (чего не было при изменении кода модами на 3.0). Так что "прощай" это к 3.0 относится, а не к 3.1

[xisp]

У пользователей без js расширение, написанное полностью на js, не заработает,

А ежели частично? Вот половину вёрстки добавят через событие, половину- яваскриптом. С выключенным яваскриптом выйдет каша.

[Татьяна5]

А ежели частично?

А это уже вопрос к авторам расширений. Код, который должен работать без js, должен и добавляться без него

[angst66]

Не знаю, где точнее задать вопрос. Некоторые расширения добавляют свои страницы. И если в 3.0x мы могли внести правку в viewonline.php , чтобы при просмотре "Кто сейчас на конференции" видеть, что кто то просматривает эту самую новую страницу из мода, то на 3.1 мы просто видим "Главная страница". Уважаемые создатели расширений, возможно ли решать эту проблему с помощью расширений? (возможно было бы спросить как то попроще)

[Татьяна5]

angst66, да, есть событие core.viewonline_overwrite_location

[Webliberty]

Добрый вечер! Помогите пожалуйста разобраться, где ошибка. Делаю расширение, чтобы сделать редирект со страниц &viev=next и &viev=previous.

Для примера, редирект с такой страницы: viewtopic.php?f=1&t=1&view=previous на такую: viewtopic.php?f=1&t=1
Подобные страницы заблокированы в роботсе, но Гугл их все равно обрабатывает и хранит в индексе с сообщением о том что содержимое страницы заблокировано в файле роботс. Почему тогда он их хранит если заблокировано не понятно.

В viewtopic.php есть строки U_VIEW_OLDER_TOPIC и U_VIEW_NEWER_TOPIC (как раз previous - предыдущая тема, next - следующая тема соответственно). Сделал вот такое переопределение по аналогии с каноническими, но на странице в браузере возникает предупреждение о циклической переадресации и страница не открывается.... Сам код:

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

<?php
namespace webliberty\nextprev\event;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
class listener implements EventSubscriberInterface
{
	protected $template;
	protected $php_ext;
	public function __construct(\phpbb\template\template $template, $php_ext)
	{
		$this->template = $template;
		$this->php_ext = $php_ext;
	}

	static public function getSubscribedEvents()
	{
		return array(
			'core.viewtopic_get_post_data' => 'get_post_data',
		);
	}

	public function get_post_data($event)
	{
		$forum_id = $event['forum_id'];
		$topic_id = $event['topic_id'];
		$this->template->assign_vars(array(
		'U_VIEW_OLDER_TOPIC'	=> redirect(append_sid("viewtopic.$this->php_ext", "f=$forum_id&t=$topic_id")),
		'U_VIEW_NEWER_TOPIC'	=> redirect(append_sid("viewtopic.$this->php_ext", "f=$forum_id&t=$topic_id")),
		));
	}
}
?>

[LavIgor]

Добрый вечер! Помогите пожалуйста разобраться, где ошибка. Делаю расширение, чтобы сделать редирект со страниц &viev=next и &viev=previous.

Для примера, редирект с такой страницы: viewtopic.php?f=1&t=1&view=previous на такую: viewtopic.php?f=1&t=1

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

<?php
		'U_VIEW_OLDER_TOPIC'	=> redirect(append_sid("viewtopic.$this->php_ext", "f=$forum_id&t=$topic_id")),
		'U_VIEW_NEWER_TOPIC'	=> redirect(append_sid("viewtopic.$this->php_ext", "f=$forum_id&t=$topic_id")),

?>

Это неправильно. Зачем вы редирект добавили?
Уберите редирект, и всё заработает. Но при этом добавьте обёртку с проверкой на то, является ли пользователь ботом, иначе обычные пользователи будут испытывать трудности.

Ужасно неудобно здесь, что у последнего сообщения нет кнопки "Цитата".
Я привык править цитируемые сообщения, а не слать их полностью, отчего убрали кнопку.
Уже несколько раз отослал благодарности вместо цитаты.

Думаю, нужна просто кнопка в личном разделе "Показывать ссылку Цитата в последнем сообщении темы". Те люди, кто делал полное цитирование при ненадобности (по незнанию того, что цитату нужно сокращать), вряд ли станут искать, где вернуть эту кнопку.

[Webliberty]

Это неправильно. Зачем вы редирект добавили?

Если убираю редирект, то вообще ничего не происходит, как было так и остается

[LavIgor]

Вам нужен редирект со страниц по ссылкам или чтобы эти ссылки на странице менялись (чтобы роботы по ним попадали на ту же страницу)?

[Webliberty]

Мне нужен редирект со страницы с параметром view=previous на страницу без этого параметра. Как для пользователей так и для роботов. В ветке 3.1 вообще нет ссылок ведущих на такие страницы (даже в исходном коде страницы), в 3.0.х были вот и нашел их робот. Но сейчас то это зачем.

Можно вообще сделать чтобы при переходе на такие страницы сервер 404 ответ давал, тогда уж 100% страницы выпали бы из индекса, но не знаю как сделать. На phpbb.com спросил как сделать редирект с помощью htaccess - никто не ответил.

[LavIgor]

Мне нужен редирект со страницы с параметром view=previous на страницу без этого параметра. Как для пользователей так и для роботов. В ветке 3.1 вообще нет ссылок ведущих на такие страницы (даже в исходном коде страницы), в 3.0.х были вот и нашел их робот. Но сейчас то это зачем.

Тогда Вы можете проверить GET-параметр view на соответствие previous или next.
И только при этом соответствии делать редирект на отдельной строке.

[Anvar]

seodirection,
Сделай редирект с помощью .htaccess и замени свою функцию, что-бы заменить адреса на страницах.

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

	public function get_post_data($event)
	{
		$forum_id = $event['forum_id'];
		$topic_id = $event['topic_id'];
		$this->template->assign_vars(array(
			'U_VIEW_OLDER_TOPIC'	=> append_sid("{$this->phpbb_root_path}viewtopic.$this->php_ext", "f=$forum_id&t=$topic_id"),
			'U_VIEW_NEWER_TOPIC'	=> append_sid("{$this->phpbb_root_path}viewtopic.$this->php_ext", "f=$forum_id&t=$topic_id"),
		));
	}

[Webliberty]

Anvar, так нужно что-то одно наверно, если редирект в .htaccess сделать, то зачем расширение это, какую функцию выполняет? Попробовал заменить функцию и снова циклическая переадресация. Совсем запутался. Я редирект не знаю как сделать такой, гугл тоже не знает Поэтому и начал копать в сторону расширения этого...

[rxu]

seodirection, LavIgor дает правильный совет. Переменные шаблона тут в принципе не нужны, нужна проверка параметра и вызов в зависимости от этого redirect. Только не передавайте redirect в шаблон, достаточно просто вызова этой функции.