404 и пагинация

Постим сюда все найденые ошибки
Post Reply
D.N.
Posts: 17
Joined: Wed Aug 29, 2018 9:12 am

404 и пагинация

Post by D.N. »

Версия:
SmartCJ Pro V 2.1 Build 201808261105

Урл:
/?force_template=content_list&group=group1&page=9999

Шаблон:

Code: Select all

<thumb num=1-120><!--GALLERY_ID--> - <!--GALLERY_SLUG-->/</thumb>
<ul>
<pagination first_pages=3 adjustment=1 last_pages=3 range_separator="<li>...</li>">
                <li><a href="/?force_template=content_list&group=group1&page=<!--PAGE_NUM-->/"><!--PAGE_NUM--></a></li>
</pagination>
</ul>
URLs with 0 result thumbs return 404 - YES

Всё нормально работает на страницах где есть тумбы, как и должно, но проблема в том, что на несуществующий страницах (например 9999) всёравно находятся 300 видео и выводятся по 20 тумб.
Вместо этого должен быть 404 код ошибки и шаблон content_not_found. Как так сделать?
admin
Site Admin
Posts: 37202
Joined: Wed Sep 10, 2008 11:43 am

Re: 404 и пагинация

Post by admin »

Тут вопрос в усложнении логики

например у нас 8 галер, по 2 на страницу = 4 страницы

1 из них - место под тестирование

когда мы открываем 5ю страницу (которой кстати в навигации нет но допустим руками кто-то зачем-то ввел)

так у нас появляется вомзонжость показать тестируемые, а так 404 ..

в целом не такой критичный вопрос, учитывая что туда попадет только бот или ручной траф
добавил ваш вариант
Don't forget to run script update
D.N.
Posts: 17
Joined: Wed Aug 29, 2018 9:12 am

Re: 404 и пагинация

Post by D.N. »

Незнаю связано ли с этим топиком или нет, но после сегодняшнего апдейта (Build 201808291928) все страницы перестали работать:
Sorry, there was an error. Please, reload page in a minute.

В логах:
2018-08-30 09:58:07 :: 1054 :: Unknown column 'rt.thumb_id' in 'field list' :: SELECT m.model_id, m.model_slug, m.model_name, md.model_description, md.model_custom_var1, md.model_custom_var2, md.model_custom_var3, m.model_total_galleries, md.model_average_ctr, rt.thumb_id, rt.height, rt.width FROM rot_models as m JOIN rot_models_data as md ON m.model_id = md.model_id and site_id = '1' WHERE 1=1 AND m.model_total_galleries >= 1 ORDER BY md.model_average_ctr DESC LIMIT 0, 20
admin
Site Admin
Posts: 37202
Joined: Wed Sep 10, 2008 11:43 am

Re: 404 и пагинация

Post by admin »

проведите плз ап и будет ок
Don't forget to run script update
D.N.
Posts: 17
Joined: Wed Aug 29, 2018 9:12 am

Re: 404 и пагинация

Post by D.N. »

Спасибо! Заработало как надо.
admin
Site Admin
Posts: 37202
Joined: Wed Sep 10, 2008 11:43 am

Re: 404 и пагинация

Post by admin »

Отлично)
Don't forget to run script update
Post Reply