WordPress.org

Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#5000 closed defect (bug) (fixed)

No input file specified on second page of search query results

Reported by: glenda Owned by:
Milestone: 2.3 Priority: normal
Severity: blocker Version: 2.3
Component: General Keywords:
Focuses: Cc:

Description

When using the search box:

Type phrase > Garners search results with multiple pages > Hit "Previous Entries" link at the bottom of first page's results > Second page's results for same search results in "No input file specified."

Click "Previous Entries" link for example in action:
http://www.agendacide.com/minutes/index.php?s=wordpress&searchbutton=Go%21

Which then results in the following:
http://www.agendacide.com/minutes/index.php/page/2/?s=wordpress&searchbutton=Go%21

This bug does not replicate itself on WordPress.com.

Attachments (1)

5000.diff (506 bytes) - added by ryan 7 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 glenda7 years ago

  • Summary changed from No input file specified off to No input file specified on second page of search query results

comment:2 glenda7 years ago

  • Component changed from General to WordPress.org
  • Owner changed from anonymous to matt

comment:3 glenda7 years ago

  • Component changed from WordPress.org to General
  • Owner changed from matt to anonymous

comment:4 ryan7 years ago

Can't replicate on my 2.3 testbed either. Have you let Matt near your site? ;-)

comment:5 matt7 years ago

  • Severity changed from normal to blocker

I can replicate on my blog on trunk:

http://photomatt.net/index.php?s=wordpress
http://photomatt.net/index.php/page/2/?s=wordpress

The difference is with my server setup (multiviews enabled) the index.php/arguments thing works.

However on many, like Dreamhost where Glenda is at, it will return the error above.

Of course if you do it without the index.php it works fine:

http://photomatt.net/?s=wordpress
http://photomatt.net/page/2/?s=wordpress

But countless themes have the search action include index.php.

comment:6 ryan7 years ago

That patch seems to do the trick. Needs more testing to see what side-affects it might have.

ryan7 years ago

comment:7 markjaquith7 years ago

Tested on:

  • PATH_INFO (/index.php/) permalink structure
  • mod_rewrite permalink structure
  • PATH_INFO (/index.php/) permalink structure with Page on Front
  • mod_rewrite permalink structure with Page on Front

So far so good.

comment:8 ryan7 years ago

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

(In [6144]) Always remove index.php from request before contructing page links. fixes #5000

Note: See TracTickets for help on using tickets.