Making WordPress.org

Opened 3 years ago

Closed 12 months ago

Last modified 12 months ago

#6214 closed defect (bug) (fixed)

photos archive does not work when logged out

Reported by: binarymoon's profile BinaryMoon Owned by: coffee2code's profile coffee2code
Milestone: Priority: normal
Component: Photo Directory Keywords:
Cc:

Description

Hi - I was browsing the photo gallery found here:
https://wordpress.org/photos/

When viewing the site I noticed that page 50 was a 404, and page 49 worked.

I then logged in to report the issue and saw that the pages went up to 79 and they all loaded.

So it seems that when you are logged out you can't see all the pages, and the last one you can see doesn't load.

If you log out (or use a private browser) and view the following pages then you can see the issue.

https://wordpress.org/photos/page/49/
https://wordpress.org/photos/page/50/

Change History (6)

#1 @dd32
3 years ago

WordPress.org archives are limited to 50 pages when logged out, for seo purposes.

Usually the archive pagination will respect that, but I guess https://meta.trac.wordpress.org/browser/sites/trunk/wordpress.org/public_html/wp-content/mu-plugins/pub/wporg-page-limiter.php needs updating.

#2 follow-up: @Otto42
3 years ago

@dd32 Would it make sense to disable this code for /photos? We probably do want the later pages available for SEO in this case. Limiting it made sense for the support forums and other listings, but photos is somewhat lighter on content than those are, and paging shouldn't create the same load issues.

#3 @BinaryMoon
3 years ago

Thanks for explaining. I understand how it works now.

Personally i would agree with @Otto42 that it should be disabled on the photos directory. For the directory to be useful you would want to have direct access to all the photos. Because of the page limit I thought the photos archive was a lot smaller than it is.

Google image search has generated a lot of traffic for my sites so I would suggest its worth getting all the images ranked.

Thanks.

#4 in reply to: ↑ 2 @dd32
3 years ago

Replying to Otto42:

Would it make sense to disable this code for /photos? We probably do want the later pages available for SEO in this case.

It would, but you'll need to deal with Systems to remove the pagination block, preferably on all sites. There's an nginx rule which blocks access to /page/[5-9][0-9] for bots/etc, the pagination blocking code is just to ensure that we don't present links to search engines/etc and promptly cause 404's which cause more issue.

#5 @coffee2code
12 months ago

  • Owner set to coffee2code
  • Resolution set to fixed
  • Status changed from new to closed

In 12967:

Photo Directory, Posts: Take into account logged-out pagination limit on w.org.

Since the front page limit differs from the archive limits, the Photo Directory does its own max page calculation, which is not compatible with the WPORG_Page_Limiter's own generic calculation.

Fixes #6214.

#6 @coffee2code
12 months ago

In 12969:

Photo Directory: Show a message adjacent to pagination navigation on the last archive pagination page if user is not logged in.

Unless user is logged in, w.org enforces up to 50 pages of pagination, which can lead visitors to think that's all the photos we have. This message recommends the visitor log in to see more photos.

See #6214.

Note: See TracTickets for help on using tickets.