Неверная работа роутера тегов в 3.4.6 - Баги Joomla

Решение:
- сделано, требуется сделать pull

Статус:
желательна проверка

Описание:
Я создал пункт меню в скрытом меню типа список меток и дал ему алиас tags

Ожидаемый результат
Ссылки на тэги будут иметь вид /tags/tagname.html

Полученный результат
После посещения странички /tags.html ссылки действительно имеют такой вид, однако через некоторое время они вновь приобретают вид
/component/tags/tag/tagname.html

Информация о системе
Joomla 3.4.6 + модуль меток стандартный опять же

однако через некоторое время они вновь приобретают вид
/component/tags/tag/tagname.html

Это как? Само по себе?

Это как? Само по себе?

Да.
На обновленном блоге заметил.

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

Так по моему они так всегда работали. Пока находишься внутри компонента тегов, всё ок, а как выходишь из него - нет. Или я не прав?

Не знаю.
Даже если выйти, модуль некоторое время показывает верные ссылки.

В любом случае - это некорректная работа роутера

Ну так надо создать пункты меню для всех тегов и всё будет ок. Это же Joomla.

Ну так надо создать пункты меню для всех тегов и всё будет ок. Это же Joomla.

Да вы издеваетесь!
Тем более что тэги создаются на лету и ссылки то ИНОГДА правильные.

А если тегов несколько тысяч?

Не пробовали в шаблоне модуля приписать к url тэгов Itemid того пункта меню?

Нет. А разве они не в хелпере генерируется?

Не знаю, я этот модуль не юзаю, но не думаю, что в хелпере определяется ID вашего пункта меню (что в принципе тоже можно делать). Могу ошибиться, но насколько я помню, router.php сначала ищет подходящий пункт меню по Itemid и в случае неуспеха оставляет url в виде "component/tags".

Кстати, как ни странно, но ссылки формируются уже в шаблоне вывода

Могу ошибиться, но насколько я помню, router.php сначала ищет подходящий пункт меню по Itemid и в случае неуспеха оставляет url в виде "component/tags".

Именно, стандартный роутер - ищет Itemid конкретного элемента. Он такой везде в Joomla - во всех компонентах. Нет фолбэка на список.

Нет фолбэка на список.

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

Сергей, попробуй сделать так в /components/com_tags/helpers/route.php:

Добавь после 155 строки код:


Потом после строки 178:

Исключение - если он активный. Интересно, почему?

Так активный пункт может быть не только список.

Ведь если не найдется пункта меню для конкретного элемента, то логично использовать существующий пункт меню списка, так?

Ведь если не найдется пункта меню для конкретного элемента, то логично использовать существующий пункт меню списка, так?

Как бы да, логично. Выше я как раз предложил такое решение. Но реально нужно давать выбор, что использовать для фолбэка, а не хардкодить.

Сергей, попробуй сделать так в /components/com_tags/helpers/route.php:

Добавь после 155 строки код:

Код: php

// Fallback to default tags view
if ($view == 'tags')
{
	self::$lookup[$language]['tags']['default'] = $item->id;
}

Потом после строки 178:

Код: php

// Return the default itemid
if (isset(self::$lookup[$language]['tags']['default']))
{
	return self::$lookup[$language]['tags']['default'];
}

Точно после этих строк?

Точно

https://github.com/joomla/joomla-cms/blob/staging/components/com_tags/helpers/route.php#L155
https://github.com/joomla/joomla-cms/blob/staging/components/com_tags/helpers/route.php#L178

а 178 до первой правки или после?

а 178 до первой правки или после?

До $active = $menus->getActive();

Угу, сделал.
Проверил - поведение не изменилось.

Аа, я затупил. Там же else, а значит он не отработает, так как $needles всегда есть. Убери else вообще.

И первую часть кода после строки 154 воткни.
https://github.com/joomla/joomla-cms/blob/staging/components/com_tags/helpers/route.php#L154

Не, всё же поставь первую часть после строки 155.

О, теперь работает!
Надо запуллить!

О, теперь работает!
Надо запуллить!

Я уже пояснял, что так просто пулл не сделать. А если кто-то не хочет, чтобы по дефу брался пункт меню списка тегов?

Ну если 99% это надо, то может есть смысл игнорировать этого кого-то?

Ты узко мыслишь... Сразу видно, пуллов не делал Харкодить нельзя, понимаешь? Ты же не для себя делаешь...

Нужна свобода выбора, а это значит, что надо параметр вводить какой-то. Возможно в настройки компонента.

Ты узко мыслишь... Сразу видно, пуллов не делал Харкодить нельзя, понимаешь? Ты же не для себя делаешь...

Нужна свобода выбора, а это значит, что надо параметр вводить какой-то. Возможно в настройки компонента.

Эх.. пока это случится