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

12>>>

Пагинация не работает корректно

Roffun
#1 2015-10-27 17:24

Contributors
Thanked: 175 kez

Заметил что не работает пагинация на странице всех тегов если ограничить количество   тегов на странице в облаке всех тегов

Остальные места работают, проблема наблюдается именно на странице всех тегов.

Кстати на cotonti.com тоже проблема с тегами, при нажатии на тег вообще ошибка fatal error выскакивает: //www.cotonti.com/tags?a=all&t=siena - если заменить в адресе переменную t на dt (как в пагинации указано) то ошибка исчезает.

 

 

Улетел на другую планету, а там почты нету.. https://www.cotonti.com/forums/45298?m=posts
Bu gönderi Roffun tarafından düzenlendi (2016-11-27 19:51, 8 yıllar önce)
Dayver
#2 2015-10-27 22:08

Team
Thanked: 180 kez

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

Pavlo Tkachenko aka Dayver
Bu gönderi Dayver tarafından düzenlendi (2015-10-27 22:13, 9 yıllar önce)
Macik
#3 2015-10-28 00:18

Team
Thanked: 181 kez

Что касается работы плагина здесь на сайте — то тут стоит достаточно старая версия. Как только разберемся с багом, обновим.

https://github.com/macik
правильный хостинг — https://goo.gl/fjCa1F
Roffun
#4 2015-10-28 09:51

Contributors
Thanked: 175 kez

#41121 Dayver:

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

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

 

Улетел на другую планету, а там почты нету.. https://www.cotonti.com/forums/45298?m=posts
Macik
#5 2015-10-30 08:33

Team
Thanked: 181 kez

#41118 Roffun:

Заметил что не работает пагинация на странице всех тегов если ограничить количество   тегов на странице в облаке всех тегов

Остальные места работают, проблема наблюдается именно на странице всех тегов.

Roffun, можешь подробнее расписать (не могу повторить на посленей версии из мастер ветки). Какая версия Котонти и какая версия плагина.
Еще лучше если ты сравниш все свои файлы плагина с файлами в мастер-ветке (если сделать это сложно — запакуй и пришли мне).

У себя проверял так — создал пару страниц с тегами (всего около десятка) в настройках поставил  «Tags displayed per page in standalone cloud = 2». Как результат на странице тегов (/tags?a=all) выводится 2 первых и показывает несколько страниц. Пагинация тоже работает нормально.

https://github.com/macik
правильный хостинг — https://goo.gl/fjCa1F
Roffun
#6 2015-10-30 13:13

Contributors
Thanked: 175 kez

Версия 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, итог можно посмотреть здесь:

Улетел на другую планету, а там почты нету.. https://www.cotonti.com/forums/45298?m=posts
Bu gönderi Roffun tarafından düzenlendi (2015-11-02 10:01, 9 yıllar önce)
Macik
#7 2015-12-26 18:33

Team
Thanked: 181 kez

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

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

https://github.com/macik
правильный хостинг — https://goo.gl/fjCa1F
Roffun
#8 2015-12-26 19:52

Contributors
Thanked: 175 kez

#41264 Macik:

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

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

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

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

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

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

 

 

Улетел на другую планету, а там почты нету.. https://www.cotonti.com/forums/45298?m=posts
Macik
#9 2015-12-27 21:26

Team
Thanked: 181 kez

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

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

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

https://github.com/macik
правильный хостинг — https://goo.gl/fjCa1F
Roffun
#10 2015-12-27 21:56

Contributors
Thanked: 175 kez

#41269 Macik:

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

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

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

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

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

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

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

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

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

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

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

Улетел на другую планету, а там почты нету.. https://www.cotonti.com/forums/45298?m=posts
Bu gönderi Roffun tarafından düzenlendi (2015-12-27 22:17, 9 yıllar önce)
Macik
#11 2015-12-28 12:30

Team
Thanked: 181 kez

#41270 Roffun:

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

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

Еще замечание — ты правильно сделал открыв тикет 1458 (и рано его закрыл).
Если в пул-реквесте писать что-то типа `Fix #1458` или `Closes #1458` то открытый ранее тикет будет автоматом закрыт при принятии пулреквеста.

 

https://github.com/macik
правильный хостинг — https://goo.gl/fjCa1F
Roffun
#12 2015-12-28 16:15

Contributors
Thanked: 175 kez

#41274 Macik:
 

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

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

PHP
1
2
3
4
5
6
// 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 минут спустя:

Правки внёс

Улетел на другую планету, а там почты нету.. https://www.cotonti.com/forums/45298?m=posts
Bu gönderi Roffun tarafından düzenlendi (2015-12-28 16:33, 9 yıllar önce)
Macik
#13 2015-12-28 16:51

Team
Thanked: 181 kez

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

https://github.com/macik
правильный хостинг — https://goo.gl/fjCa1F
Roffun
#14 2015-12-28 17:46

Contributors
Thanked: 175 kez

#41276 Macik:

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

Ок, видел. 

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

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

Улетел на другую планету, а там почты нету.. https://www.cotonti.com/forums/45298?m=posts
Macik
#15 2015-12-29 00:56

Team
Thanked: 181 kez

#41277 Roffun:
 

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

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


В Котонти существовал свой собственный набор смайов (в `images\smilies\`), со своим собственным файлом описания на JS, который использовался в редакторе Markitup, который раньше был основным редактором для Котонти (корнями это уходит в 2008-09 года).
Этот набор позже использовался так же плагином BBCode (при соотв. настройках). Здесь на сайте даже представлено несколько кастомных наборов.

Так вот, суть, думаю в том, чтобы привязать этот штатный набор к CKEditor. 

Теоретически наборы в CKeditor настраиваемые через соотв. переменные конфига. Надо просто расширить `ckeditor.default.set.js`, чтобы была возможность задать доп. настройки через свой JS файл или переменную.

Вопрос в том, на сколько это сегодня актуально. Расширить `ckeditor.default.set.js`, для возможности задавать свои настройки смысл вижу. Прописывать ли через этот механизм дефолтные смайлы — для меня вопрос открытый.

https://github.com/macik
правильный хостинг — https://goo.gl/fjCa1F

12>>>