Make WordPress Core

Opened 4 years ago

Last modified 20 months ago

#15949 new defect (bug)

WP 3.0.x comments popup on pages returning 404 [including posible solution]

Reported by: odosuiadan Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 3.0
Component: Query Keywords: has-patch 3.2-early close
Focuses: Cc:


comments-popup does not work on pages (but still working on blog entries) since the 3.0.x upgrade.

i confirm bug and sollution on both: my own theme and on default them after adding comments_popup_link and comments_popup_script functions.

after a long time looking for a solution i found that the new "handle_404" method in class WP (wp-includes/classes.php line 475) is the responsable, substituing this method with an older one from wp 2.x brunch solves the problem.

looks like the new method "think" that there is no content when the index file is called with comments_popup parameter on a page id, so it returns a 404 status, as said the old one solves the problem.

Attachments (1)

15949.diff (592 bytes) - added by dd32 4 years ago.

Download all attachments as: .zip

Change History (5)

@dd324 years ago

comment:1 @dd324 years ago

  • Keywords has-patch 3.2-early added; comments popup 404 page id removed
  • Milestone changed from Awaiting Review to Future Release
  • Version set to 3.0

attachment 15949.diff added

This removes the post_type limit of comments_popup being a post, this seems to have caused a knock on effect which removes the need to modify handle_404 and canonical.

It appears that 3.0 has the same problem which this patch fixes, modifying handle_404() may cover the problem in 3.0, but it doesn't in 3.1.

As this isn't a regression from 3.0, and we're in a RC stage, I'm leaving this for 3.2 at this stage.

comment:2 @wonderboymusic20 months ago

  • Keywords close added

I don't see a bug here anymore - @dd32?

comment:3 @dd3220 months ago

What theme did you test with? It's hard to find any themes which still use the popup functions..

comment:4 @wonderboymusic20 months ago

I just used the functions to generate the link inside of 2013

Note: See TracTickets for help on using tickets.