Ticket #6460 (closed defect: fixed)

Opened 4 months ago

Last modified 12 hours ago

Feed PHP files show no posts when WP has it's own directory

Reported by: Viper007Bond Assigned to: westi
Priority: highest omg bbq Milestone: 2.6
Component: General Version: 2.5
Severity: critical Keywords: has-patch
Cc:

Description

I have all plugins disabled on my personal blog, but wp-rss2.php and it's siblings are all displaying no posts. They aren't blank, just no posts to display.

However, the pretty URLs work just fine (I've disabled my FeedBurner? mod_rewrite redirect rules for the moment).

Works: http://www.viper007bond.com/feed/
Nope: http://www.viper007bond.com/wordpress/wp-rss2.php

Works: http://www.viper007bond.com/feed/atom/
Nope: http://www.viper007bond.com/wordpress/wp-atom.php

The exact same thing is happening on another WordPress install on the same server too:

http://www.finalgear.com/wordpress/wp-rss2.php

It still has active plugins though.

phpinfo(): http://www.viper007bond.com/phpinfo.php
MySQL 5.0.45

Attachments

6460.diff (1.3 kB) - added by DD32 on 04/28/08 04:18:59.

Change History

03/30/08 04:10:13 changed by Viper007Bond

  • keywords set to needs-patch.
  • summary changed from All feed PHP files return no posts, pretty URLs do to Feed PHP files are blank when WP has it's own directory.

Okay, I got how to duplicate it! If siteurl != homeurl, it breaks, i.e. if WordPress is in it's own directory.

I made a nice and fresh new WordPress install and both /feed/ and /wp-rss2.php worked fine. Then I moved the WordPress files and /feed/ worked while /subdirectory/wp-rss2.php did not.

As to why this is happening is beyond me. I don't know the feed code well enough.

03/30/08 04:11:02 changed by DD32

Yup, Confirmed it with my blog having a HomeURL != SiteURL

http://dd32.id.au/wordpress/wp-rss2.php - doesnt work http://dd32.id.au/feed/ - works

03/30/08 04:15:09 changed by Viper007Bond

  • summary changed from Feed PHP files are blank when WP has it's own directory to Feed PHP files show no posts when WP has it's own directory.

03/30/08 11:40:06 changed by westi

  • owner changed from anonymous to westi.
  • status changed from new to assigned.

Does this exist in 2.3.3 ?

If so it may relate to [4551]

03/30/08 11:45:30 changed by Viper007Bond

No, it worked fine in 2.3.3.

03/30/08 12:56:12 changed by westi

Ok. It seems to work fine in my local test with permalinks disabled. Now to try with them enabled.

03/30/08 13:37:37 changed by westi

Definitely doesn't work with permalinks enabled.

It matches as a page lookup instead.

I have a fix which I will land in trunk for now and get some feedback/review before back porting to 2.5.1

03/30/08 13:42:07 changed by westi

(In [7574]) Make sure the rewrite rules for the feed files match when WordPress is installed in a subdirectory. See #6460 this is for trunk.

03/31/08 16:32:52 changed by lloydbudd

  • keywords changed from needs-patch to has-patch.

(follow-up: ↓ 11 ) 03/31/08 16:53:48 changed by markjaquith

It fixes the bug for me.

(in reply to: ↑ 10 ) 03/31/08 19:50:28 changed by westi

Replying to markjaquith:

It fixes the bug for me.

Cool. I'll merge this into branches/2.5 for 2.5.1

03/31/08 19:56:29 changed by westi

  • status changed from assigned to closed.
  • resolution set to fixed.

(In [7583]) Make sure the rewrite rules for the feed files match when WordPress is installed in a subdirectory. Fixes #6460 for 2.5.1.

04/02/08 13:27:33 changed by jsherk

I would like to try this patch and see if it fixes #6484

Where is the patch? What files were modified/changed?

I tried to figure it out, but don't know my way around trac well enough yet. Thanks

04/02/08 14:02:52 changed by Nazgul

There is no patch attached to this ticket. Westi made a patch and commited it directly, without adding it to trac.

But if you click on the [7583] there is a link at the bottom called "Unified Diff".

04/02/08 15:03:16 changed by jsherk

Okay thanks Nazgul. Tried it on #6484 but has no effect.

04/28/08 04:15:53 changed by DD32

  • status changed from closed to reopened.
  • resolution deleted.

[7583] has broken HomeURL == SiteURL, See [wp-testers] Feeds not publishing on 2.5.1

The issue is that the slash is not present when WP is installed in the current directory, I'll attach a patch, the Db version will need to be bumped. (Patch against trunk)

04/28/08 04:18:59 changed by DD32

  • attachment 6460.diff added.

04/28/08 04:24:31 changed by DD32

[7583] has broken HomeURL == SiteURL

By that, I mean:
HomeURL == SiteURL:

  • /feed/ = works
  • /?feed=rss2 = works
  • /wp-rss2.php = doesnt work.

04/28/08 06:17:01 changed by westi

  • status changed from reopened to closed.
  • resolution set to fixed.

(In [7841]) Make sure the direct feed files work when siteurl == homeurl. Fixes #6460 for trunk props DD32.

04/28/08 06:17:25 changed by westi

  • status changed from closed to reopened.
  • resolution deleted.
  • milestone changed from 2.5.1 to 2.5.2.

Re-open for 2.5.2

04/28/08 06:20:53 changed by westi

  • status changed from reopened to closed.
  • resolution set to fixed.

(In [7843]) Make sure the direct feed files work when siteurl == homeurl. Fixes #6460 for the 2.5 branch props DD32.

04/28/08 11:01:56 changed by MattyRob

  • status changed from closed to reopened.
  • resolution deleted.

I've applies the latest changes to my rewrite.php and version.php files and direct access to files such as wp-atom.php still fails to produce output.

HomeURL == SiteURL

  • /feed/ = works
  • /?feed=rss2 = works
  • /wp-rss2.php = doesnt work.

04/28/08 11:10:00 changed by DD32

I've applies the latest changes

Have you accessed the admin page to cause a database upgrade page to come up?

D

04/28/08 14:22:13 changed by MattyRob

  • status changed from reopened to closed.
  • resolution set to fixed.

@DD32,

My apologies, I had run the upgrade on one of my site having edited the files but I was looking at the feeds from a different side - full on ID10T issue.

07/15/08 16:24:07 changed by ryan

  • milestone changed from 2.5.2 to 2.9.

Milestone 2.5.2 deleted

07/19/08 22:21:46 changed by scribu

  • milestone deleted.

07/19/08 23:51:50 changed by westi

  • milestone set to 2.6.