Make WordPress Core

Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#35344 closed defect (bug) (fixed)

Strange pagination issue on front page after 4.4.1 update

Reported by: northeme's profile northeme Owned by: dd32's profile dd32
Milestone: 4.4.2 Priority: normal
Severity: normal Version: 4.4.1
Component: Canonical Keywords: has-patch needs-unit-tests needs-testing fixed-major
Focuses: Cc:

Description

If a custom post type or blog listing page is set as a front page, pagination URLs don't work properly since 4.4.1 - E.g. /page/2, /page/3. However, it works just fine if permalinks set to Default or listing page unselected at Settings > Reading > Front Page.

There was no problem with front page pagination until 4.4.1

Attachments (2)

35344.diff (622 bytes) - added by pento 9 years ago.
35344.2.diff (2.8 KB) - added by dd32 9 years ago.

Download all attachments as: .zip

Change History (79)

#1 @northeme
9 years ago

I forgot to mention that pagination links redirect to homepage instead of redirecting to 404 page. In other words, pagination links should be working fine yet I couldn't find out the reason why it's redirecting back to home.

#2 @pento
9 years ago

  • Component changed from General to Query
  • Milestone changed from Awaiting Review to 4.4.2

Thanks for the bug report!

I can reproduce the behaviour you're describing. We'll get this fixed up. :-)

@pento
9 years ago

#3 @northeme
9 years ago

I appreciate your swift reply @pento ! It's good to know that you'll look into that, you definitely made my day :)

#4 @pento
9 years ago

  • Keywords has-patch needs-unit-tests needs-testing added

35344.diff is a first pass at fixing this, though given the edge cases we've encountered in this code, it'll need some careful consideration.

#5 @northeme
9 years ago

No doubt that it'd need a lot of testing. Thanks for the heads up. At least I can let know our theme users who are having this issue to stick to Version 4.4 or try to fix this manually.

@dd32
9 years ago

#6 @dd32
9 years ago

So this is actually quite a fun weird bug.

The fix for #35031 calls get_queried_object() which causes $wp_query->queried_object to get set, That being set then triggers a few rules in redirect_canonical() which specifically deal with page-on-front (which must've been broken previously?).

Ultimately it comes down to the fact that canonical doesn't know how to deal with paged posts when they're on the front-page, it specifically excludes is_front_page() - the problem being I can't see where it's supposed to be handled instead.

35344.2.diff makes it work for me, by properly creating the canonical URL.

It seems like it was probably a 4.4 regression that's caused $wp_query->queried_object to be false/unset at the time that redirect_canonical() gets called.. I'm not sure how/what happened there..

#7 @roymahfooz
9 years ago

#35348 was marked as a duplicate.

#8 @zoonini
9 years ago

I tested @dd32's patch and it worked to restore CPT pagination on my 4.4.1 install.

This CPT pagination issue was reported in the forums by several users running Espied with its static front page set to display Jetpack's portfolio custom post type and also affects other themes that use the portfolio CPT on the front page, like Illustratr. The patch worked well for both themes.

#9 @pento
9 years ago

#35353 was marked as a duplicate.

#10 @themewarrior
9 years ago

I can also confirm @dd32 is working and managed to fix the pagination issue

Version 0, edited 9 years ago by themewarrior (next)

#11 @dd32
9 years ago

I feel I should mention that crazy if in 35344.2.diff is built from the existing one there, plus the canonical rule far above which is matching to create the issue.

I'm certain it could probably be made a lot cleaner by using get_queried_object() and/or get_queried_object_id().

#12 @Uranbold
9 years ago

#35359 was marked as a duplicate.

#13 @northeme
9 years ago

I also confirm that both patches recommended by @dd32 and @pento work for me.

This ticket was mentioned in Slack in #core by dd32. View the logs.


9 years ago

#15 @dd32
9 years ago

#35365 was marked as a duplicate.

#16 follow-ups: @BackuPs
9 years ago

I can confirm changing the wp-includes/canonical.php lines 264-270 from this

<?php
                // Post Paging
                if ( is_singular() && ! is_front_page() && get_query_var('page') ) {
                        if ( !$redirect_url )
                                $redirect_url = get_permalink( get_queried_object_id() );
                        $redirect_url = trailingslashit( $redirect_url ) . user_trailingslashit( get_query_var( 'page' ), 'single_paged' );
                        $redirect['query'] = remove_query_arg( 'page', $redirect['query'] );
                }

to

<?php
          // Post Paging 
                if ( is_singular() && get_query_var('page') && ( !is_front_page() || ( isset($wp_query->queried_object) && 'page' == get_option('show_on_front') && $wp_query->queried_object->ID == get_option('page_on_front') ) ) ) { 
                                if ( !$redirect_url ) 
                                                $redirect_url = get_permalink( get_queried_object_id() ); 
                                $page = get_query_var( 'page' ); 
                                if ( is_front_page() ) { 
                                                $redirect_url = trailingslashit( $redirect_url ) . user_trailingslashit( "$wp_rewrite->pagination_base/$page", 'paged' ); 
                                } else { 
                                                $redirect_url = trailingslashit( $redirect_url ) . user_trailingslashit( $page, 'single_paged' ); 
                                } 
                                $redirect['query'] = remove_query_arg( 'page', $redirect['query'] ); 
                } 

fixes the issue.

I guess this will be part of the next update?

Thank you.

#17 @downstairsdev
9 years ago

I've also tested the patch and confirms it fixes this issue. Several users of Portfolio Press (theme in repo) have started to encounter this bug.

#18 @ia2ca
9 years ago

I applied the above fix to canonical.php and for us it did not fix the issue. Using InTouch template (ThemeForest).

#19 @azaozz
9 years ago

#35364 was marked as a duplicate.

#20 @dd32
9 years ago

#35369 was marked as a duplicate.

#21 @dd32
9 years ago

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

In 36237:

Canonical: Generate the correct canonical url for paged posts/pages when they're used as the page_on_front.

This fixes an issue where pages become inacessible on a front page post.

Fixes #35344 for trunk.

#22 @dd32
9 years ago

  • Component changed from Query to Canonical
  • Resolution fixed deleted
  • Status changed from closed to reopened

Re-opening for 4.4.2 work.

If you're here for the issue where capitalisation issues cause the front page to get stuck in an infinite loop, see #21602

#23 @pento
9 years ago

needs-unit-tests ;-)

#24 @dd32
9 years ago

needs-unit-tests ;-)

Hey, I didn't remove it, it still needs them :) (Just good luck working out why the existing unit test for it fails to catch that it was broken)

#25 @dd32
9 years ago

In 36238:

Tests: Add more paged canonical tests for page_on_front.

See #35344.

#26 @dd32
9 years ago

Actually it turns out that the unit tests kind of do cover this.

By changing from isset($wp_query->queried_object) to get_queried_object_id() it triggered a unit test failure, but that failure was then fixed by the main part of the patch.

[36238] adds a few more unit tests that cover the behaviour of [36237]

This ticket was mentioned in Slack in #core by asad. View the logs.


9 years ago

#28 in reply to: ↑ 16 @nikkoboy
9 years ago

  • Resolution set to worksforme
  • Status changed from reopened to closed

Replying to BackuPs: Changing the wp-includes/canonical.php lines 264-270 worked perfectly for me! I'm running WT Tera News on my blog, and I had this pagination issue since the 4.4.1 update. Deactivated all plugins at first to try to find the culprit (pagination problems have been often linked in my case, to the cache plugin I use or the Easy Digital Download plugin), but that didn't work.

Thanks to this workaround in the canonical.php file, pagination is back on the homepage.

Thank you for sharing!

I can confirm changing the wp-includes/canonical.php lines 264-270 from this

<?php
                // Post Paging
                if ( is_singular() && ! is_front_page() && get_query_var('page') ) {
                        if ( !$redirect_url )
                                $redirect_url = get_permalink( get_queried_object_id() );
                        $redirect_url = trailingslashit( $redirect_url ) . user_trailingslashit( get_query_var( 'page' ), 'single_paged' );
                        $redirect['query'] = remove_query_arg( 'page', $redirect['query'] );
                }

to

<?php
          // Post Paging 
                if ( is_singular() && get_query_var('page') && ( !is_front_page() || ( isset($wp_query->queried_object) && 'page' == get_option('show_on_front') && $wp_query->queried_object->ID == get_option('page_on_front') ) ) ) { 
                                if ( !$redirect_url ) 
                                                $redirect_url = get_permalink( get_queried_object_id() ); 
                                $page = get_query_var( 'page' ); 
                                if ( is_front_page() ) { 
                                                $redirect_url = trailingslashit( $redirect_url ) . user_trailingslashit( "$wp_rewrite->pagination_base/$page", 'paged' ); 
                                } else { 
                                                $redirect_url = trailingslashit( $redirect_url ) . user_trailingslashit( $page, 'single_paged' ); 
                                } 
                                $redirect['query'] = remove_query_arg( 'page', $redirect['query'] ); 
                } 

fixes the issue.

I guess this will be part of the next update?

Thank you.

Last edited 9 years ago by nikkoboy (previous) (diff)

#29 follow-ups: @contempoinc
9 years ago

  • Resolution worksforme deleted
  • Status changed from closed to reopened

Is this going to make it in 4.4.2? Its causing a ton of unneeded support tickets from my buyers.

Last edited 9 years ago by contempoinc (previous) (diff)

#30 in reply to: ↑ 29 @themebeans
9 years ago

Replying to contempoinc:

Is this going to make it in 4.4.2? Its causing a ton of unneeded support tickets from my buyers.

Totally hope so. I'm getting a whole lot as well.

#31 in reply to: ↑ 29 @cbaldelomar
9 years ago

Replying to contempoinc:

Is this going to make it in 4.4.2? Its causing a ton of unneeded support tickets from my buyers.

Yes, please release it under 4.4.2. I'm getting a lot as well.

#32 follow-up: @dd32
9 years ago

A fix for this will be included in 4.4.2. There is no ETA for release yet. +1's are not helpful right now, we understand it's an issue for many people.

#33 @dd32
9 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

In 36280:

Canonical / Query: Restore the is_404() check in wp_old_slug_redirect() which was removed in [34659].
This reverts part of [34659] due to excessive canonical problems it's caused in 4.4.x.

Fixes #35344, #21602 for the 4.4 branch.

#34 follow-ups: @dd32
9 years ago

For those still experiencing problems in 4.4, Can you please try to replace your wp-includes/query.php file with this one and let me know if that helps?

#35 @dd32
9 years ago

In 36281:

Canonical / Query: After [36280] remove the unit tests which are no longer supported for 4.4.
This also removes the is_feed() code to avoid confusion - only pages & embeds will be redirected.
See #35344

#36 @kraftbj
9 years ago

#35444 was marked as a duplicate.

#37 in reply to: ↑ 34 @ia2ca
9 years ago

Replying to dd32:

For those still experiencing problems in 4.4, Can you please try to replace your wp-includes/query.php file with this one and let me know if that helps?

This worked for me! The canonical.php fix earlier did not, so I reverted it back to the 4.4.1 version. This new query.php did correct the issue immediately.

#38 in reply to: ↑ 34 @cbaldelomar
9 years ago

Replying to dd32:

For those still experiencing problems in 4.4, Can you please try to replace your wp-includes/query.php file with this one and let me know if that helps?

Yes, pagination works on the homepage with the updated query.php file.

#39 in reply to: ↑ 34 @contempoinc
9 years ago

Replying to dd32:

For those still experiencing problems in 4.4, Can you please try to replace your wp-includes/query.php file with this one and let me know if that helps?

Yup, that definitely fixes the issue!

#40 in reply to: ↑ 34 @azizpbrimah
9 years ago

Replying to dd32:

For those still experiencing problems in 4.4, Can you please try to replace your wp-includes/query.php file with this one and let me know if that helps?

Fixed my issue. Thanks.

#41 @Clorith
9 years ago

#35455 was marked as a duplicate.

#42 in reply to: ↑ 34 ; follow-up: @Ipstenu
9 years ago

Replying to dd32:

For those still experiencing problems in 4.4, Can you please try to replace your wp-includes/query.php file with this one and let me know if that helps?

This fixed the front page issues, but it looks like I'm still having the same problem on category pages.

#43 in reply to: ↑ 32 @Netessov
9 years ago

Replying to dd32:

For those still experiencing problems in 4.4, Can you please try to replace your wp-includes/query.php file with this one and let me know if that helps?

I did it.
But it’s still not working.

Last edited 9 years ago by Netessov (previous) (diff)

#44 in reply to: ↑ 16 ; follow-up: @Netessov
9 years ago

Replying to BackuPs:

I can confirm changing the wp-includes/canonical.php lines 264-270 from this

<?php
                // Post Paging
                if ( is_singular() && ! is_front_page() && get_query_var('page') ) {
                        if ( !$redirect_url )
                                $redirect_url = get_permalink( get_queried_object_id() );
                        $redirect_url = trailingslashit( $redirect_url ) . user_trailingslashit( get_query_var( 'page' ), 'single_paged' );
                        $redirect['query'] = remove_query_arg( 'page', $redirect['query'] );
                }

to

<?php
          // Post Paging 
                if ( is_singular() && get_query_var('page') && ( !is_front_page() || ( isset($wp_query->queried_object) && 'page' == get_option('show_on_front') && $wp_query->queried_object->ID == get_option('page_on_front') ) ) ) { 
                                if ( !$redirect_url ) 
                                                $redirect_url = get_permalink( get_queried_object_id() ); 
                                $page = get_query_var( 'page' ); 
                                if ( is_front_page() ) { 
                                                $redirect_url = trailingslashit( $redirect_url ) . user_trailingslashit( "$wp_rewrite->pagination_base/$page", 'paged' ); 
                                } else { 
                                                $redirect_url = trailingslashit( $redirect_url ) . user_trailingslashit( $page, 'single_paged' ); 
                                } 
                                $redirect['query'] = remove_query_arg( 'page', $redirect['query'] ); 
                } 

fixes the issue.

I guess this will be part of the next update?

Thank you.

This helped me.
Thank you)

#45 in reply to: ↑ 44 @thetechnuttyuk
9 years ago

I also tested this, it fixes the problem completely, don't know if it breaks anything else yet, but it seems to be a good fix so far.

#46 in reply to: ↑ 16 @rolf deck
9 years ago

Replying to BackuPs:

I can confirm changing the wp-includes/canonical.php lines 264-270 from this

<?php
                // Post Paging
                if ( is_singular() && ! is_front_page() && get_query_var('page') ) {
                        if ( !$redirect_url )
                                $redirect_url = get_permalink( get_queried_object_id() );
                        $redirect_url = trailingslashit( $redirect_url ) . user_trailingslashit( get_query_var( 'page' ), 'single_paged' );
                        $redirect['query'] = remove_query_arg( 'page', $redirect['query'] );
                }

to

<?php
          // Post Paging 
                if ( is_singular() && get_query_var('page') && ( !is_front_page() || ( isset($wp_query->queried_object) && 'page' == get_option('show_on_front') && $wp_query->queried_object->ID == get_option('page_on_front') ) ) ) { 
                                if ( !$redirect_url ) 
                                                $redirect_url = get_permalink( get_queried_object_id() ); 
                                $page = get_query_var( 'page' ); 
                                if ( is_front_page() ) { 
                                                $redirect_url = trailingslashit( $redirect_url ) . user_trailingslashit( "$wp_rewrite->pagination_base/$page", 'paged' ); 
                                } else { 
                                                $redirect_url = trailingslashit( $redirect_url ) . user_trailingslashit( $page, 'single_paged' ); 
                                } 
                                $redirect['query'] = remove_query_arg( 'page', $redirect['query'] ); 
                } 

I have tested this and it fixes the problem form me, although I don't know if doesn´t breaks anything else...

#47 @faqeer
9 years ago

I tried this. https://core.trac.wordpress.org/ticket/35344#comment:16 by BackuPs
Not working for me.
How can i use patches by @dd32 and @pento ?

Why is that not working?

The strange thing is that, pagination in my theme is working fine on localhost, without editing any core files.

but on server when i uploaded my theme, and tested online then, its not working.

Any idea!

Last edited 9 years ago by faqeer (previous) (diff)

#48 follow-up: @dd32
9 years ago

@faqeer Follow the instructions here: https://core.trac.wordpress.org/ticket/35344#comment:34

Thanks to everyone who has +1'd various fixes here, please stop now :) The fix will be available in 4.4.2 when it's released.

#49 in reply to: ↑ 48 @faqeer
9 years ago

Replying to dd32:

@faqeer Follow the instructions here: https://core.trac.wordpress.org/ticket/35344#comment:34

Thanks to everyone who has +1'd various fixes here, please stop now :) The fix will be available in 4.4.2 when it's released.

Now it worked.
When will be 4.4.2 releasing.

Last edited 9 years ago by faqeer (previous) (diff)

#50 @swissspidy
9 years ago

#35543 was marked as a duplicate.

#51 in reply to: ↑ 16 @jakenelson09
9 years ago

Replying to BackuPs:

I can confirm changing the wp-includes/canonical.php lines 264-270 from this

<?php
                // Post Paging
                if ( is_singular() && ! is_front_page() && get_query_var('page') ) {
                        if ( !$redirect_url )
                                $redirect_url = get_permalink( get_queried_object_id() );
                        $redirect_url = trailingslashit( $redirect_url ) . user_trailingslashit( get_query_var( 'page' ), 'single_paged' );
                        $redirect['query'] = remove_query_arg( 'page', $redirect['query'] );
                }

to

<?php
          // Post Paging 
                if ( is_singular() && get_query_var('page') && ( !is_front_page() || ( isset($wp_query->queried_object) && 'page' == get_option('show_on_front') && $wp_query->queried_object->ID == get_option('page_on_front') ) ) ) { 
                                if ( !$redirect_url ) 
                                                $redirect_url = get_permalink( get_queried_object_id() ); 
                                $page = get_query_var( 'page' ); 
                                if ( is_front_page() ) { 
                                                $redirect_url = trailingslashit( $redirect_url ) . user_trailingslashit( "$wp_rewrite->pagination_base/$page", 'paged' ); 
                                } else { 
                                                $redirect_url = trailingslashit( $redirect_url ) . user_trailingslashit( $page, 'single_paged' ); 
                                } 
                                $redirect['query'] = remove_query_arg( 'page', $redirect['query'] ); 
                } 

fixes the issue.

I guess this will be part of the next update?

Thank you.

This also fixed the pagination issue I was having using the Ultimate Auction Pro plugin. Thanks!

#52 in reply to: ↑ 42 ; follow-up: @ZaneMatthew
9 years ago

Replying to Ipstenu:

Replying to dd32:

For those still experiencing problems in 4.4, Can you please try to replace your wp-includes/query.php file with this one and let me know if that helps?

This fixed the front page issues, but it looks like I'm still having the same problem on category pages.

Also experiencing this issue, and the fix is not working on category, or archive pages, i.e., any page that is not the front page.

Last edited 9 years ago by ZaneMatthew (previous) (diff)

#53 in reply to: ↑ 52 @dd32
9 years ago

Replying to ZaneMatthew:

Replying to Ipstenu:

Replying to dd32:

For those still experiencing problems in 4.4, Can you please try to replace your wp-includes/query.php file with this one and let me know if that helps?

This fixed the front page issues, but it looks like I'm still having the same problem on category pages.

Also experiencing this issue, and the fix is not working on category, or archive pages, i.e., any page that is not the front page.

I believe your issue may be different than the one covered here, potentially related to #35482. I've sent you an email to gather a few extra details.

#54 in reply to: ↑ 34 @chriscamplejohn
9 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Replying to dd32:

For those still experiencing problems in 4.4, Can you please try to replace your wp-includes/query.php file with this one and let me know if that helps?

I am now running with this query.php and 35344.2.diff but I still have the problem. /page/2 redirects to / and higher pages page/3, page/4 etc. do not redirect but show the front page.

#55 @punit5658
9 years ago

I tested it. I can reproduce the issue with Version 4.4.1 and after replacing the file, the issue is resolved.

#56 follow-up: @dd32
9 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

@chriscamplejohn I'll email you shortly to get a few more details about your site and figure out what problem you're experiencing.

#57 @RavanH
9 years ago

On a multisite with two sites on Illustratr with a Portfolio front page. Site 1 has Polylang running and (strangly) worked fine all along while site 2 without Polylang showed the pagination issue. Applying https://core.trac.wordpress.org/attachment/ticket/35344/35344.2.diff solved the issue on site 2 while site 1 kept working like before.

Last edited 9 years ago by RavanH (previous) (diff)

#58 @eomjerome
9 years ago

Partial fix for me with the update to canonical.php. Issue fixed on the home page but page 2 onwards now all point to /page/2/. I also don't see the 'recent posts' link allowing me to go forward again.

Using the new query.php file had no effect.

#59 in reply to: ↑ 56 @nimrodlflores
9 years ago

Replying to dd32:

@chriscamplejohn I'll email you shortly to get a few more details about your site and figure out what problem you're experiencing.

Experiencing the same thing with @chriscamplejohn. Already replaced my query.php file and applied the canonical.php patch and it only fixed the URL I see in the browser (meaning the redirect to home/front base page stopped) but the posts I get from query_posts() are still the same throughout /page/2/ ... /page/3/ ... and so on.

I see that the 'paged' (instead of 'page') query_var is being received by my page template but I think it's the query_posts() function that may not be handling properly the arguments I pass to it.

But on the other hand, isn't it also supposed to be the 'page' query_var that should be filled, instead of the 'paged'?

Last edited 9 years ago by nimrodlflores (previous) (diff)

#60 follow-up: @dd32
9 years ago

It seems to be that the remaining issues being experienced is this scenario, which is sounding like a combination of #35482 and this ticket.

  • Static page on front, no posts page defined
  • query_posts() or $wp_query = new WP_Query() used within the template to display the first page of posts
  • /page/2/ would then load page 2 of the posts index with no static page in sight.

The problem I face, is that /page/2/ should NOT be the archive of posts, it should be the second page of the static home page (You can paginate posts/pages using the <!--nextpage--> tag in your content).

#61 @dd32
9 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

This ticket was mentioned in Slack in #core by dd32. View the logs.


9 years ago

#63 @isaac2k2
9 years ago

None worked for me :(

I replaced the file but what it does is when I click on next page, it highlights that page but still shows page 1 content.

Before the change, it shows page 1 and highlights page 1 still even if i click on any other pages

Please help

#64 in reply to: ↑ 60 @nimrodlflores
9 years ago

Replying to dd32:

It seems to be that the remaining issues being experienced is this scenario, which is sounding like a combination of #35482 and this ticket.

  • Static page on front, no posts page defined
  • query_posts() or $wp_query = new WP_Query() used within the template to display the first page of posts
  • /page/2/ would then load page 2 of the posts index with no static page in sight.

The problem I face, is that /page/2/ should NOT be the archive of posts, it should be the second page of the static home page (You can paginate posts/pages using the <!--nextpage--> tag in your content).

It's more like this in my case:

  • Static page on front using a page template that lists:
    1. A limited number of custom posts and
    2. A paginated list of native wp blog posts
    (and yes, the page template in the front uses query_posts() to retrieve the CPT's and the native posts.)
  • Another separate page assigned as the blog posts page
  • /page/2/ would load the same set of CPT's and the 2nd page of the native posts index.

I'm not sure if I explained it clearly enough, but you can check the site here: http://philpropertyexpert.com

I ended up just rolling back to 4.3.2 for now just to get it working again. (4.4 has the same issue.) But before the rollback, /page/2/ and onwards would display the same 1st page of the native posts index.

#65 @keesiemeijer
9 years ago

#35623 was marked as a duplicate.

This ticket was mentioned in Slack in #bbpress by netweb. View the logs.


9 years ago

#67 @Clorith
9 years ago

#35646 was marked as a duplicate.

#68 @samuelsidler
9 years ago

  • Keywords fixed-major added
  • Resolution set to fixed
  • Status changed from reopened to closed

I'm re-closing this ticket. A lot of the initial issue was fixed and is fixed for those that are testing with 4.4.2 files. If you're still experiencing issues like those in comment 60 and on, please follow #35689 as we'll be working on the issues there.

#69 @laptophobo
9 years ago

Solution #34 (replacing the wp-includes/query.php file) worked for me (WP ver. 4.4.1). The other solution just made my blog go blank.

Thank you for this fix!

#70 follow-up: @isaac2k2
9 years ago

Well non worked for me, so had to revert back to WP v4.3.2

I will never upgrade to new release until after a month or 2

This ticket was mentioned in Slack in #bbpress by casiepa. View the logs.


9 years ago

#72 follow-up: @finomeno
9 years ago

Still having this problem on 4.4.2 with or without 35344.2.diff applied.
The query.php in 4.4.2 is way too different to apply 35344.diff.
In my case this bug only affects posts on a static front page, and instead of redirecting to the homepage it strips the page number from the URL and tries to load sitename.com/page/ which, in turn, returns a 404.
Following up in new bug #35847

#73 @SergeyBiryukov
9 years ago

In 37075:

Canonical / Query: Restore the is_404() check in wp_old_slug_redirect() which was removed in [34659].
This reverts part of [34659] due to excessive canonical problems it's caused in 4.4.x.

Remove the unit tests which are no longer supported.
This also removes the is_feed() code to avoid confusion - only pages & embeds will be redirected.

Merges [36280] and [36281] to trunk.

Props dd32.
See #21602, #35344.

#74 in reply to: ↑ 70 @PTLF
9 years ago

Replying to isaac2k2:

Well non worked for me, so had to revert back to WP v4.3.2

I will never upgrade to new release until after a month or 2

Hi,

is there an easy - maybe automated - way to revert back?
Thanks for your time
Lars

Last edited 9 years ago by PTLF (previous) (diff)

#75 in reply to: ↑ 72 @PTLF
9 years ago

Replying to finomeno:

Still having this problem on 4.4.2 with or without 35344.2.diff applied.
The query.php in 4.4.2 is way too different to apply 35344.diff.
In my case this bug only affects posts on a static front page, and instead of redirecting to the homepage it strips the page number from the URL and tries to load sitename.com/page/ which, in turn, returns a 404.
Following up in new bug #35847

Sounds like my problem. Did you solve this issue?
Thanks for a short relpy!
Lars

#76 @nimrodlflores
9 years ago

It has been more than 4 months since my last post, we're on 4.5.2 already but I still experience the same thing. I had to go back to 4.3.4 again. :(

#77 @swissspidy
9 years ago

@nimrodlflores Hey there, since this ticket was closed on a completed milestone, please open a new ticket(mentioning this one) or share your issues on #35847 if it sounds like the same bug.

Note: See TracTickets for help on using tickets.