Odd "Older Entries" link problem

[jamesisin]

You can find an in depth discussion of the issue here:

​http://wordpress.org/support/topic/232369

In short, at the bottom of the main pages are links for Older/Newer posts which should look something like this:

​http://www.soundunreason.com/InkWell/?paged=2

However, I have discovered a defect in how these links are being created by WP. All other links generated by WP appear to be unaffected and function as expected.

Here is the scenario.

As you can see from the example above, the WP installation URL includes two capital letters (I and W). This is true of the container and the server is set to ignore case so all is well. However, if we substitute the lowercase letters thus:

​http://www.soundunreason.com/inkwell

the resulting links generated by WP become:

​http://www.soundunreason.com/InkWell/inkwell/index.php?paged=2

Clearly WP is adding in a superfluous folder hierarchy (and the unnecessary index.php file specification). This breaks the link. (The index.php does not effect the matter, but I include it because it is superfluous.) If one removes the additional inkwell (either case) the URL functions as expected.

No other link created by WP within the site is thus effected (or at least none I could find). There must be something about the code which creates these two (Older/Newer) links which is causing this problem.

I have done quite a bit of testing to ensure that the problem was not being caused by something else. I have disabled all plugins, for instance, to no avail.

I am happy to continue testing if that will be useful to you. Please let me know how I can assist.

James

[jamesisin]

Just stopping in to see if there was any progress or if I can offer any assistance.

[Denis-de-Bernardy]

Possibly related: #8038

[Denis-de-Bernardy]

moving to future pending patch

[janeforshort]

Changing to future release. If someone wants this to be put into a specific release for review, add a patch.

[hakre]

Is this confirmed against 2.8 anyway? Maybe the bug doesn't exists any longer. I made a comment to the forums thread as well asking if this applies to the default theme having all plugins deactivated as well.

[dd32]

I think this affects CGI-based systems.. Really not sure of that though.. would need to test it out.

[jamesisin]

Actually it has moved in the exact wrong direction. In 3.0.5 the URL constructed under the above conditions looks like this:

​http://www.soundunreason.com/InkWell/index.php/inkwell/index.php/page/2/

So WP is now adding yet another additional and superfluous layer in the folder hierarchy.

(I tried moving to 3.1 but it borked my blog and I haven't sorted it out as yet.)

[SergeyBiryukov]

You shouldn't change the version specified at the ticket creation. It is used to track when the bug was initially noticed/introduced.

[jakbob]

Hi. I've done some digging, the problem seams to appear when the Site URL is set to say lowercase, but the user tries to access the site with uppercase. Since many servers are case-insensitive it will work, but WordPress wont recognize the uppercase part as part of the Site URL and will treat it like a query, which is wrong.
My patch makes WordPress match the url case-insensitively, which should remove the problem.

[hakre]

-1 - HTTP URI paths are case-sensitive as well as many file-systems, especially server preferred ones.

[SergeyBiryukov]

Related: #13164, #16200

[SergeyBiryukov]

Refreshed after [21174] (#18034 was a related bug).

Moving for review along with #13164.

[georgestephanis]

+1 for Sergey's patch.

[kovshenin]

8847.2.patch​ works for me.

[nacin]

In [22298]:

When stripping off a pathinfo home root (/index.php/) do it with case insensitivity. props SergeyBiryukov. fixes #13164. see #8847.

[nacin]

Would it be possible to have test coverage for [22298]? (This is the only reason I've left this ticket open.)

[wonderboymusic]

Added a unit test - it passes but requires this ticket to be marked fixed to run. Someone with commit on UT: commit the patch, por favor?

[SergeyBiryukov]

Replying to wonderboymusic:

Added a unit test - it passes but requires this ticket to be marked fixed to run.

Skipped tests for open tickets can be run by specifying the ticket number as a group:

phpunit --group 8847

[SergeyBiryukov]

​[UT1114]