User tests: Successful: Unsuccessful:
Reduce the number of page duplicates in com_content categories when pagination is used.
https://www.joomla.org/announcements.html?start=999999999999999999999 - it opens, while it should not. With this patch applied, pages will open if they really exist: https://www.joomla.org/announcements.html?start=390 - this will be the last page.
Status | New | ⇒ | Pending |
Category | ⇒ | Front End Components |
Labels |
Added:
?
|
Can you test it, please?
Is it the intended behaviour of this pr that ?start=999999999999999999999 gives a 404?
Yes, it does.
The same behaviour you can see in the content_pagebreak plugin - if an article page does not exist, it gives a 404. Why not apply this approach to category blogs? When you have 200 pages, search engines index them, then you change the item per page number and 200 becomes 100. The problem is that the other 100 pages no longer exist, but they cannot be removed from search index without editing robots.txt/htaccess. I think it is very inconvenient.
I have tested this item
With or -out patch pages open. Example: Show 1 of 2 Pages, append in URL ?start=99999999
– always 2 (the last page) is shown.
I use the RC demo pages (featured view). I set in the menu params the following configuration to make pagination:
Leading Articles - 1
Intro Articles - 1
Columns - 1
Links - 1
When I open page 4 that does not have any items, I get the 404. Could you, please, describe your testing instructions?
@philip-sorokin Test Instructions shown in opened Comment.
@franz-wohlkoenig
Can you give me a link to this comment. I am sorry, can't find it.
on Top of this Thread, its your opening Comment about Issue.
@franz-wohlkoenig
This is my testing instructions, I thought you would write yours. Please, give me more details on the failure, because I tested it with 3 views successfuly:
-archive
-category
-featured
I used your Instructions for Test. Have to retest, my Test is too long ago to remember.
Test on "Archive". 9 archived Articles (5 en-GB, 4 All Lang.), Menu set on Lang. "en-GB".
index.php/en/archived-articles?start=5
(expected):index.php/en/archived-articles?start=6
(unexpected – Articles with Lang.: All are shown):index.php/en/archived-articles?start=9
(expected – 404And what is your conclusion? What about other views?
Category | Front End Components | ⇒ | Front End com_content Components |
Category | Front End Components com_content | ⇒ | Front End com_content Libraries Components |
@philip-sorokin @franz-wohlkoenig don't expect this
3/4. Change Url to index.php/en/archived-articles?start=6 (unexpected – Articles with Lang.: All are shown):
@franz-wohlkoenig See #12004 (comment) first question
I have no Conclusion cause no Knowledge.
I am closing this PR as it is not going to be merged.
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2018-01-14 08:04:32 |
Closed_By | ⇒ | philip-sorokin |
@addondev can you fix the if syntax and add a space bettween the
if
and the openeing(
https://travis-ci.org/joomla/joomla-cms/jobs/159016449