[3.2][3.3] Ultimate phpBB SEO Friendly URL

[AlexOo]

Название расширения: Ultimate phpBB SEO Friendly URL
Автор: phpbb-seo
Описание расширения: This Extension will URL rewrite phpBB URLs in various manners, injecting, or not, forums and topic titles in their URLS, each URL being rewritten once, no matter the number of links using it on the page.
Скачать: https://github.com/phpbb-seo/usu
Язык: en

Всем привет. Словил неприятный баг авторизации, может кто сталкивался и нашел, как его пофиксить. URL rewriting type: Mixed

1. Заходим в любой форум (/forum/chapter1.html) и пробуем с него авторизироваться

Результат: "Запрошенного форума не существует."

Урл имеет следующий вид:

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

/forum/viewforum.php/chapter1.html?forum_uri=chapter1&start=&sid=d6a68facbe4c4ad3a393804c24365fee

2. Аналогичная проблема при авторизации по ссылке профиля (/forum/member2.html)

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

/forum/memberlist.php/member2.html?mode=viewprofile&u=2&sid=28b81b906014133a86c32247b774281e

и выводит ошибку

Скрытый текст

Общая ошибка
Попытка перенаправления на потенциально опасный сайт.

BACKTRACE

FILE: (not given by php)
LINE: (not given by php)
CALL: msg_handler()

FILE: [ROOT]/ext/phpbbseo/usu/core/traits/seo.php
LINE: 127
CALL: trigger_error()

FILE: [ROOT]/ext/phpbbseo/usu/core/core.php
LINE: 730
CALL: phpbbseo\usu\core\core->seo_redirect()

FILE: [ROOT]/ext/phpbbseo/usu/event/listener.php
LINE: 629
CALL: phpbbseo\usu\core\core->zero_dupe()

FILE: [ROOT]/vendor/symfony/event-dispatcher/EventDispatcher.php
LINE: 214
CALL: phpbbseo\usu\event\listener->core_memberlist_view_profile()

FILE: [ROOT]/vendor/symfony/event-dispatcher/EventDispatcher.php
LINE: 44
CALL: Symfony\Component\EventDispatcher\EventDispatcher->doDispatch()

FILE: [ROOT]/phpbb/event/dispatcher.php
LINE: 62
CALL: Symfony\Component\EventDispatcher\EventDispatcher->dispatch()

FILE: [ROOT]/phpbb/event/dispatcher.php
LINE: 46
CALL: phpbb\event\dispatcher->dispatch()

FILE: [ROOT]/memberlist.php
LINE: 757
CALL: phpbb\event\dispatcher->trigger_event()

3. При авторизации из топика - топик доступен, но не возвращает на исходный урл. В итоге адрес страницы после авторизации имеет следующий вид урла

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

/forum/viewtopic.php/topic3.html?forum_uri=&start=&t=3&sid=226aaecfcb49ea2202b6d30f9651442c

И все урлы страницы становятся без ЧПУ. После перехода куда-либо в рамках форума ЧПУ срабатывает.

Помогите пожалуйста пофиксить.

Спасибо.

[Nekstati]

AlexOo, попробуйте так:

найти в файле ext\phpbbseo\usu\event\listener.php

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

	public function core_append_sid($event)
	{

добавить после

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

		if (strpos($event['url'], '/ucp.php') !== false && strpos($event['params'], 'mode=login&redirect=') !== false)
			$event['params'] = str_replace(['viewforum.php%2F', 'viewtopic.php%2F', 'memberlist.php%2F'], '', $event['params']);
		if (strpos($event['url'], 'memberlist.php/member') !== false && strpos($event['url'], 'mode=viewprofile') !== false)
			$event['url'] = str_replace('memberlist.php/', '', $event['url']);

(Это не исправление причины, а лишь обходной путь)

[AlexOo]

День добрый. Решил не поднимать новый топик по багам, а задать вопрос тут. Проблема в следующем, словил, что первая страница топика, если в ней до 10 постов включительно генерирует дубль, т.к. она доступна по адресам:

/forum/topic1.html
/forum/topic1-10.html

Подскажите пожалуйста, как это можно пофиксить ? Ведь /forum/topic1-10.html - это должна быть именно вторая страница топика, но никак не первая (((

Спасибо.

[Nekstati]

AlexOo, эта вторая ссылка отображается в каком-то месте форума или вы её вводите руками? Если не отображается, то и проблемы нет. Так-то любая страница доступна по целому триллиону адресов:
/forum/topic1-10.html
/forum/topic1-9999999.html?beavis=butthead
/forum/viewtopic.php?t=1
/forum/viewtopic.php?t=1&start=999999999999999

Страница, на которой мы сейчас находимся, тоже:
https://www.phpbbguru.net/community/viewtopic.php?t=51395&start=9999999999&beavis=butthead

[AlexOo]

Nekstati, на форуме она никак не фигурирует, нашел подобное через панель Я.Вебмастера, ругается на дубли ((( Прописал редирект, а потом увидел, что это оказывается вторая страница

Все урлы с параметрами через ? у меня закрыты в роботсе. А тут даже не знаю, как поступить (((

[AlexOo]

Все еще актуально, решение пока не найдено

Пока придумал вот такую реализацию:
Если количество сообщений в топике меньше или равно10, то php редирект на topic<id>.html
Если больше 10, то ничего не делать

Но не знаю как это реализовать. Прошу помощи

[Nekstati]

AlexOo,
ext\phpbbseo\usu\event\listener.php

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

	public function core_viewtopic_modify_page_title($event)
	{

добавить после

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

		if ($this->request->variable('start', 0) > $event['start']) {
			send_status_line(301, 'Moved Permanently');
			redirect(append_sid("{$this->phpbb_root_path}viewtopic.php", "f={$this->forum_id}&t={$this->topic_id}&start={$event['start']}"));
		}

[AlexOo]

Nekstati, спасибо большущее, низкий поклон за твои труды

НАконец-таки дошли руки внедрить все вышеописанное. Редиректы работают как надо. А вот с ошибкой авторизации что-то не то. Не хочет работать возвращает "Запрошенного форума не существует.". Пробовал по разному добавлять, результат один. Буду очень признателен, если тыкнешь носом, где я накосячил. Я еще тот рукопоп

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

	public function core_viewtopic_modify_page_title($event)
	{
		
    		if ($this->request->variable('start', 0) > $event['start']) {
    			send_status_line(301, 'Moved Permanently');
    			redirect(append_sid("{$this->phpbb_root_path}viewtopic.php", "f={$this->forum_id}&t={$this->topic_id}&start={$event['start']}"));
    		}

    		if (strpos($event['url'], '/ucp.php') !== false && strpos($event['params'], 'mode=login&redirect=') !== false)
    			$event['params'] = str_replace(['viewforum.php%2F', 'viewtopic.php%2F', 'memberlist.php%2F'], '', $event['params']);
    		if (strpos($event['url'], 'memberlist.php/member') !== false && strpos($event['url'], 'mode=viewprofile') !== false)
    			$event['url'] = str_replace('memberlist.php/', '', $event['url']);			
		
		$this->template->assign_vars([
			'U_PRINT_TOPIC'		=> ($this->auth->acl_get('f_print', $this->forum_id)) ? append_sid("{$this->phpbb_root_path}viewtopic.{$this->php_ext}", "f={$this->forum_id}&t={$this->topic_id}&view=print") : '',
			'U_BOOKMARK_TOPIC'	=> ($this->user->data['is_registered'] && $this->config['allow_bookmarks']) ? append_sid("{$this->phpbb_root_path}viewtopic.{$this->php_ext}", "f={$this->forum_id}&t={$this->topic_id}&bookmark=1&hash=" . generate_link_hash("topic_{$this->topic_id}")) : '',
			'U_VIEW_RESULTS'	=> append_sid("{$this->phpbb_root_path}viewtopic.$this->php_ext", "f=$this->forum_id&t=$this->topic_id&view=viewpoll"),
		]);
	}

[Nekstati]

AlexOo,
1) откатить все правки из ext\phpbbseo\usu\event\listener.php и внести только эту одну (против дублей):
Re: [3.2][3.3] Ultimate phpBB SEO Friendly URL
2) для авторизации попробовать так:
includes\functions.php
найти

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

		$redirect = $request->variable('redirect', rawurlencode($user->page['page']));

заменить на

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

		$redirect = $request->variable('redirect', $request->server('REQUEST_URI'));

или

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

		$redirect = $request->variable('redirect', rawurlencode($request->server('REQUEST_URI')));

[AlexOo]

Спасибо. Вечером попробую протестировать

[AlexOo]

И снова здравствуйте. Продолжаю ковыряться с расширением и выявлять новые касяки.

В списках тем отображаются ссылки на последнее сообщение. При наведении вижу урл вида
/forum/post10619.html#p10619

При переходе по ссылке срабатывает редирект и ссылка приобретает вид

/forum/topic13-90.html#p10619

Отсюда вопрос, как избавиться от редиректа, чтобы в списке тем сразу отображалась корректная ссылка на пост с указанием топика?


P.S. Ошибку авторизации так и не победил. Не стал лезть в includes\functions.php, т.к. эти правки затрагивают ядро, что не есть правильно. Все еще в поисках решения.

Пока в голову пришло лишь прописать правило редиректа, чтобы при авторизации убиралось viewforum.php, viewtopic.php и memberlist.php. Но своей соображалкой пока так и не понял, как это прописать, чтобы срабатывало именно при авторизации и не сказалось на функционале форума

[Nekstati]

как избавиться от редиректа

Снимок экрана 2023-10-07 214031.png

Только учтите, что редиректы кэшируются в браузере.

Отправлено спустя 1 минуту 23 секунды:

чтобы в списке тем сразу отображалась корректная ссылка на пост с указанием топика

Смысла нет, всё равно такие "перманентные" ссылки (без номера топика) есть в каждом посте.

[AlexOo]

Nekstati, перевод настройки в Off отключает редирект с post10619.html#p10619 на topic13-90.html#p10619

Моя же задача, чтобы в списке форумов/топиков изначально стояла ссылка вида topic13-90.html#p10619

[Татьяна5]

чтобы в списке форумов/топиков изначально стояла ссылка вида topic13-90.html#p10619

Уж чего-чего, а этого точно делать не стоит. Обращение к таблице тем с главной серьёзно увеличит нагрузку, в разы

[AlexOo]

Татьяна5, можно пожалуйста поподробнее ? Не очень понимаю, что создаст доп нагрузку на базу. Если можно на примере, как это работает сейчас и как будет работать после желаемой мной доработки.

ИМХО как раз сейчас создается доп нагрузка, т.к. выводится одно, а потом дергается адрес топика для редиректа. Я же хочу это оптимизировать, выводя сразу правильный адрес перехода, там самым сократив время отображения страницы за счет отсутствия переадресации