Форуми / National / Russian / Модули и плагины / [Fixed] Не работает пагинация в плагине tags

Ошибка на вот этой версии файла https://github.com/Cotonti/Cotonti/blob/master/plugins/tags/tags.php наблюдается ?

#41121 Dayver:

Ошибка на вот этой версии файла https://github.com/Cotonti/Cotonti/blob/master/plugins/tags/tags.php наблюдается ?

Наблюдается. 

Версия Cotonti  0.9.18,  версия тегов 0.9.18 , а ошибка у меня воспроизводится так:

Если выбрать в настройках :   Тегов на странице в облаке всех тегов, 0 - все теги сразу:     вместо 0 другое число, например 100, то появляется пагинация на странице всех тегов, даже при переключении что-то меняется, но кнопка подсветки остается 1 постоянно, куда бы я не переключал, класс pagenav pagenav_current постоянно находится на кнопке под номером 1, при переходе на страницу тегов , например /tags?a=all&dt=2      сами теги меняются, но в пагинации это никак не отображается, плюс в мета тайтл не появляется ( страница 2 ) . 

На тестовом пробовал в одном из файлов заменить переменную $t на $dt в одном месте , тогда в мета тайтл появляется ( страница 2 ) и тд, но сама пагинация не меняет класс, и остается на уровне 1 постоянно.

Я попробую обновить сайт до мастер ветки, посмотрим что будет. 

Добавлено 1 час спустя:

Обновил сайт мастер веткой, проблема осталась

Добавлено 13 минут спустя:

Вот живой пример проблемы:

Нулевой сайт, обновлен до мастер ветки, меньше 10 тегов, выставил для примера 1 на страницу, итог можно посмотреть здесь, некоторые страницы не включаются, некоторые не соответствуют цифре.

Второй пример:

Сайт с несколькими сотнями тегов, обновлен до мастер ветки, настройка выводить по 100, итог можно посмотреть здесь:

#41264 Macik:

Как выяснилось баг проявлялся только при включенном режиме `easypagenav`.
Исправлено в 2285300.

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

Протестировал на нескольких сайтах, проблема осталась. Пробовал и с включенными чпу и без, на рабочем сайте и на новом, чистом.

Если установить чистую базу, создать 4-5 тегов, и в админке поставить настройку:   Тегов на странице в облаке всех тегов   1  

При включении Дружественная паджинация:  просто переключает теги, но сама пагинация не меняет цифры, остается на первой.

При выключении Дружественная паджинация:  тоже не работает, но при малом количестве тегов еще и появляется бесконечная пагинация, 1..5..100...5000...

#41269 Macik:

Спасибо за оперативный тест. 
Вот скорректировал .

Попутно поправил еще один баг — «кривое» кеширование страниц облака тегов. Из-за чего некоторые могли дублироваться, а некоторые не отображались.

Просьба протестировать повторно (не забудьте обнулить кеш).

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

Добавлено 10 минут спустя:

=================

Отлично, всё работает теперь, и в режиме Дружественная паджинация и без.

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

Я себе делал небольшую доработку, подстановку из существующих переменных, сейчас поищу ее.

Добавлено 10 минут спустя:

Нашёл, я добавил заяку на гитхаб:  #1458

#41274 Macik:

 

В целом согласен. Но надо немного доработать — прокомментировал здесь.

Ок, код приготовил, сейчас на гитхабе добавлю, новый вариант будет такой:

// meta title
$out['subtitle'] = empty($qs) ? $L['Tags'] : htmlspecialchars(strip_tags($qs)) . ' - ' . mb_strtolower($L['tags_Search_tags']);
// meta descriptions
$out['desc'] = empty($qs) ? $L['Tags'] : mb_strtolower($L['tags_Search_tags'] . ': ' . strip_tags($qs) . ' - ' . $L['tags_Search_results']);
// meta keywords
$out['keywords'] = empty($qs) ? $L['Tags'] : mb_strtolower($L['Tag'] . ' ' . strip_tags($qs) . ' ' . $L['tags_Search_tags']);

Добавлено 17 минут спустя:

Правки внёс

#41276 Macik:

Правки влил. Спасибо за исправления.

Ок, видел. 

А что насчет правки #1404, по ней идеи уже есть ? 

я так понимаю, суть заявки отвязать конфиг смайлов CKEditor от самого плагина?