IIS fix for get_pagenum_link

[snakefoot]

The posts_nav_link doesn't work in categories, when trying to browse to the second page of a category using "Previous entries", then the link points to:

http://example.com/index.php/page/2/

Instead of pointing to:

http://example.com/index.php/category/page/2/

The fix for me was to modify get_pagenum_link located in template-functions-links.php (Both trunk and branch should get this rather easy fix)

function get_pagenum_link($pagenum = 1) {
	global $wp_rewrite;

	// IIS on Windows fix
	if (!$_SERVER['REQUEST_URI']) $_SERVER['REQUEST_URI'] = $_SERVER['SCRIPT_NAME'].$_SERVER['PATH_INFO'];

	$qstr = wp_specialchars($_SERVER['REQUEST_URI']);
...

[snakefoot]

Just discovered that my IIS didn't react to the IIS test ($_SERVER['REQUEST_URI'] returned "/index.php"). Now changed it according to this advice:

​http://www.xoops.org/modules/newbb/viewtopic.php?topic_id=29255

function get_pagenum_link($pagenum = 1) {
	global $wp_rewrite;

	if ( !isset($_SERVER['REQUEST_URI']) || empty($_SERVER['REQUEST_URI']) || $_SERVER['REQUEST_URI']==$_SERVER['PHP_SELF'])
		$_SERVER['REQUEST_URI'] = $_SERVER['SCRIPT_NAME'].$_SERVER['PATH_INFO'];

	$qstr = wp_specialchars($_SERVER['REQUEST_URI']);
...

[markjaquith]

Is this an issue for trunk? If so, it should have a 2.2 milestone.

[snakefoot]

Unless special updating of REQUEST_URI (if IIS) have been added to 2.1 or 2.2, then it is most likely an issue for the trunk as well (As the get_pagenum_link() is the same in the trunk).

Maybe one should consider creating a global function, which returns the expected URL independent of what PHP/IIS/Apache/Litestep version one is using, and then reuse the function all the locations where REQUEST_URI is used.

Wordpress has different places which tries to handle the REQUEST_URI behavior on IIS, but each implementation is different.

[snakefoot]

Discovered that wp_settings.php is the only one updating $_SERVER['REQUEST_URI'], so it is probably not a good idea to manipulate the $_SERVER['REQUEST_URI'] in get_pagenum_link, but instead just append PATH_INFO.

[rob1n]

PATH_INFO is already appended, I think.

[snakefoot]

With the #3930 solved, the get_pagenum_link() looks much prettier and by changing the following lines it should work with IIS (Notice the str_replace):

     if ( !$wp_rewrite->using_permalinks() ) { 
        $base = trailingslashit( get_bloginfo( 'home' ) ); 

        if ( $pagenum > 1 ) { 
            $result = add_query_arg( 'paged', $pagenum, $base . $request );  
        } else { 
            $result = $base . $request; 
        }
    } else {
        $request = str_replace('index.php/', '', $request);
        $request = str_replace('index.php', '', $request);
        
        $request = preg_replace( '|/?page/(.+)$|', '', $request);

Along with modifying REQUEST_URI in wp_settings.php (Will also make WP-Cache work out of the box):

     // Fix for IIS, which doesn't set REQUEST_URI
     if ( empty( $_SERVER['REQUEST_URI'] ) ) {
        $_SERVER['REQUEST_URI'] = $_SERVER['SCRIPT_NAME'] . $_SERVER['PATH_INFO'];

Must admit that I have only tested this on 2.0 (Exchanged the original get_pagenum_link with the new one in version 2.3). Have tested with permalink using index.php without ISAPI rewrite, and without index.php using ISAPI rewrite, and everything (pages, posts, categories, archives, search, paged) works as they should.

[snakefoot]

I guess the best solution would be to fix the REQUEST_URI completely. If one could detect that the permalink options is not set to use index.php, then don't include the SCRIPT_NAME in the REQUEST_URI, but just PATH_INFO.

[snakefoot]

Just made a plugin with the following code and now no modification is needed in the get_pagenum_link() (Still need the modification in wp_settings.php to make WP-Cache work)

function fix_request_uri()
{
	global $wp_rewrite;

	if ($wp_rewrite->using_index_permalinks())
	{
		$_SERVER['REQUEST_URI'] = $_SERVER['SCRIPT_NAME'] . $_SERVER['PATH_INFO'];
	}
	else
	{
		$_SERVER['REQUEST_URI'] = $_SERVER['PATH_INFO'];
	}

	// Append the query string if it exists and isn't null
	if (isset($_SERVER['QUERY_STRING']) && !empty($_SERVER['QUERY_STRING']))
	{
		$_SERVER['REQUEST_URI'] .= '?' . $_SERVER['QUERY_STRING'];
	}
}

add_action('init', 'fix_request_uri');

[snakefoot]

Corrected some bugs in the plugin:

function fix_request_uri()
{
	global $is_IIS;
	global $wp_rewrite;
	if (!$is_IIS)
		return;

	if ($wp_rewrite->using_index_permalinks())
	{
		$_SERVER['REQUEST_URI'] = $_SERVER['SCRIPT_NAME'] . $_SERVER['PATH_INFO'];
	}
	else if ($wp_rewrite->using_permalinks())
	{
		// SCRIPT_NAME includes script-path + script-filename (remove the filename) and append request-path PATH_INFO
		$_SERVER['REQUEST_URI'] = substr($_SERVER['SCRIPT_NAME'], 0, strrpos($_SERVER['SCRIPT_NAME'], '/')) . $_SERVER['PATH_INFO'];
	}
	else
	{
		$_SERVER['REQUEST_URI'] = $_SERVER['SCRIPT_NAME'];
	}

	// Append the query string if it exists and isn't null
	if (isset($_SERVER['QUERY_STRING']) && !empty($_SERVER['QUERY_STRING']))
	{
		$_SERVER['REQUEST_URI'] .= '?' . $_SERVER['QUERY_STRING'];
	}
}

[snakefoot]

REQUEST_URI patch for IIS

[snakefoot]

Added patch for wp-setting.php which fixes this ticket.

[markjaquith]

(In [5889]) Set REQUEST_URI for IIS in more situations. props snakefoot. fixes #3514