Pagination links not working

[maximus83]

I'm a novice with php and mysql so please be kind. The problem is I'm calling a database which has various states open, closed, pending etc, all this works fine.

The problem is when I click next on my paging links the page just reloads to the start page with all states visible and the first 6 rows visible.

My code seems to work fine as the url of next, prev page as a different start number for the start of the database query eg (http:/wordpress/?page_id=1072?s=6&np=3).

I was wondering if this could be to with wordpress?

help much appreciated.

$dbc = mysql_connect (DB_HOST, DB_USER, DB_PASSWORD) ;
mysql_select_db (DB_NAME) ;

/*state control buttons */

echo "<div id=\"shownew\"><table class=\"showhide\"><tr><td><p><form method=\"post\" action=\"".htmlentities($_SERVER['REQUEST_URI'])."\"><INPUT TYPE=\"submit\" name=\"showopen\" VALUE=\"\" class=\"show_open\"></form></p></td>";
echo "<td><p><form method=\"post\" action=\"".htmlentities($_SERVER['REQUEST_URI'])."\"><INPUT TYPE=\"submit\" name=\"showpending\" VALUE=\"\" class=\"show_pending\"></form></p></td>";
echo "<td><p><form method=\"post\" action=\"".htmlentities($_SERVER['REQUEST_URI'])."\"><INPUT TYPE=\"submit\" name=\"showclosed\" VALUE=\"\" class=\"show_closed\"></form></p></td>";
echo "<td><p><form method=\"post\" action=\"".htmlentities($_SERVER['REQUEST_URI'])."\"><INPUT TYPE=\"submit\" name=\"show_all\" VALUE=\"\" class=\"show_all\"></form></p></td></tr></table></div>";


$self = 'http://' . $_SERVER['HTTP_HOST'] . $_SERVER['PHP_SELF'] ;
/* Status set code */

if (isset($_POST['close'])) {
    $close_row = $_POST['close_row'];
    $close = "UPDATE support_users SET status = 'Closed', date_altered = NOW() WHERE ticket_number=$close_row";
    $closeresult = @mysql_query ($close);
    $viewstate = $_POST['viewstate'];
}

if (isset($_POST['open'])) {
    $close_row = $_POST['close_row'];
    $close = "UPDATE support_users SET status = 'Open', date_altered = NOW() WHERE ticket_number=$close_row";
    $closeresult = @mysql_query ($close);
    $viewstate = $_POST['viewstate'];
}

if (isset($_POST['pending'])) {
    $close_row = $_POST['close_row'];
    $close = "UPDATE support_users SET status = 'Pending', date_altered = NOW() WHERE ticket_number=$close_row";
    $closeresult = @mysql_query ($close);
    $viewstate = $_POST['viewstate'];
}

if (isset($_POST['pending_ami'])) {
    $close_row = $_POST['close_row'];
    $close = "UPDATE support_users SET status = 'Pending AMI', date_altered = NOW() WHERE ticket_number=$close_row";
    $closeresult = @mysql_query ($close);
    $viewstate = $_POST['viewstate'];
}

if (isset($_POST['pending_arp'])) {
    $close_row = $_POST['close_row'];
    $close = "UPDATE support_users SET status = 'Pending ARP', date_altered = NOW() WHERE ticket_number=$close_row";
    $closeresult = @mysql_query ($close);
    $viewstate = $_POST['viewstate'];
}

/* View state set code */

if (isset($_POST['showopen'])) {
    $viewstate='open';
}

if (isset($_POST['showpending'])) {
    $viewstate='pending';
}


if (isset($_POST['showclosed'])) {
    $viewstate='closed';
}

if (isset($_POST['show_all'])) {
    $viewstate='all';
}


$display = 6;

if (isset($_GET['np'])) {
    $num_pages = $_GET['np'];

} else {

    if ($viewstate=='open') {   
    $query = "SELECT ticket_number, first_name, surname, email, product, retailer, DATE_FORMAT(dop, '%d %M %Y') AS dop, message, address, DATE_FORMAT(created, '%d %M %Y %r') AS created, status FROM support_dev WHERE status='Open' ORDER BY ticket_number ASC";
    $query_result = mysql_query ($query); 
    $num_records = @mysql_num_rows ($query_result);

    if ($num_records > $display) {
$num_pages = ceil ($num_records/$display);
} else {
    $num_pages = 1;
    }
}


elseif ($viewstate=='pending') {
    $query = "SELECT ticket_number, first_name, surname, email, product, retailer, DATE_FORMAT(dop, '%d %M %Y') AS dop, message, address, DATE_FORMAT(created, '%d %M %Y %r') AS created, status FROM support_dev WHERE status LIKE 'Pending%' ORDER BY ticket_number ASC";
    $query_result = mysql_query ($query); 
    $num_records = @mysql_num_rows ($query_result);

    if ($num_records > $display) {
$num_pages = ceil ($num_records/$display);
} else {
    $num_pages = 1;
}
    }

elseif ($viewstate=='closed') {
        $query = "SELECT ticket_number, first_name, surname, email, product, retailer, DATE_FORMAT(dop, '%d %M %Y') AS dop, message, address, DATE_FORMAT(created, '%d %M %Y %r') AS created, status FROM support_dev WHERE status='Closed' ORDER BY ticket_number ASC";
        $query_result = mysql_query ($query); 
        $num_records = @mysql_num_rows ($query_result);

    if ($num_records > $display) {
$num_pages = ceil ($num_records/$display);
} else {
    $num_pages = 1;
    }

}
else {
        $query = "SELECT ticket_number, first_name, surname, email, product, retailer, DATE_FORMAT(dop, '%d %M %Y') AS dop, message, address, DATE_FORMAT(created, '%d %M %Y %r') AS created, status FROM support_dev ORDER BY ticket_number ASC";
        $query_result = mysql_query ($query); 
        $num_records = @mysql_num_rows ($query_result);

    if ($num_records > $display) {
$num_pages = ceil ($num_records/$display);
} else {
    $num_pages = 1;
    }
}
}


if (isset($_GET['s'])) {
    $start = $_GET['s'];
} else {
    $start = 0;
}

$query = "SELECT ticket_number, first_name, surname, email, product, retailer, DATE_FORMAT(dop, '%d %M %Y') AS dop, message, address, DATE_FORMAT(created, '%d %M %Y %r') AS created, status FROM support_dev ORDER BY ticket_number ASC LIMIT $start, $display";
if ($viewstate=='open') {

    $query = "SELECT ticket_number, first_name, surname, email, product, retailer, DATE_FORMAT(dop, '%d %M %Y') AS dop, message, address, DATE_FORMAT(created, '%d %M %Y %r') AS created, status FROM support_dev WHERE status='Open' ORDER BY ticket_number ASC LIMIT $start, $display";
    }
        elseif ($viewstate=='pending') {
    $query = "SELECT ticket_number, first_name, surname, email, product, retailer, DATE_FORMAT(dop, '%d %M %Y') AS dop, message, address, DATE_FORMAT(created, '%d %M %Y %r') AS created, status FROM support_dev WHERE status LIKE 'Pending%' ORDER BY ticket_number ASC LIMIT $start, $display";
    }
    elseif ($viewstate=='closed') {
    $query = "SELECT ticket_number, first_name, surname, email, product, retailer, DATE_FORMAT(dop, '%d %M %Y') AS dop, message, address, DATE_FORMAT(created, '%d %M %Y %r') AS created, status FROM support_dev WHERE status='Closed' ORDER BY ticket_number ASC LIMIT $start, $display";}
    $result = @mysql_query ($query);
    $num = mysql_num_rows ($result);

if ($num > 0) {

if ($num_pages > 1) {
    echo '<p>';
    $current_page = ($start/$display) + 1;

    if ($current_page != 1) {
        echo '<a href="?page_id=1072?s=' . ($start - $display) . '&np=' . $num_pages . '">Previous</a> ';
    }
    for ($i = 1; $i <= $num_pages; $i++) {
        if ($i != $current_page) {
            echo '<a href="?page_id=1072?s=' . (($display * ($i - 1))) . '&np=' . $num_pages . '">' .$i .'</a> ';
        } else {
            echo $i . ' ';
        }
    }
    if ($current_page !=$num_pages) {
        echo '<a href="' . $query_result . '?s=' . ($start + $display) . '&np=' . $num_pages . '">Next</a> ';
    }
    echo '</p><br />'; 

}


 if ($result) {
     echo '
<div id="supviewwrapperheader">
<div id="supviewticket"><p>Ticket</p></div> 
<div id="supviewfirst"><p>First Name</p></div>
<div id="supviewlast"><p>Last Name</p></div>
<div id="supviewemail"><p>Email</p></div>
<div id="supviewproduct"><p>Product</p></div>
<div id="supviewretailer"><p>Retailer</p></div>
<div id="supviewdop"><p>D.O.P.</p></div>
<div id="supviewmessage"><p>Message</p></div>
<div id="supviewaddress"><p>Address</p></div>
<div id="supviewcreated"><p>Date created</p></div>
<div id="supviewstatus"><p>Status</p></div>
<div id="supviewbuttons"><p></p></div>
</div>
';

while ($row = mysql_fetch_array($result, MYSQL_NUM)) {
    echo "
    <div id=\"supviewwrapper\" class=\"".($row[10])."\">
    <div id=\"supviewticket\"><p><a name=\"a".$row[0]."\"></a>$row[0]</p></div>
    <div id=\"supviewfirst\"><p>$row[1]</p></div>
    <div id=\"supviewlast\"><p>$row[2]</p></div>
    <div id=\"supviewemail\"><p>$row[3]</p></div>
    <div id=\"supviewproduct\"><p>$row[4]</p></div>
    <div id=\"supviewretailer\"><p>$row[5]</p></div>
    <div id=\"supviewdop\"><p>$row[6]</p></div>
    <div id=\"supviewmessage\"><p>$row[7]</p></div>
    <div id=\"supviewaddress\"><p>$row[8]</p></div>
    <div id=\"supviewcreated\"><p>$row[9]</p></div>
    <div id=\"supviewstatus\"><p>$row[10]</p></div>
    <div id=\"supviewbuttons\"><p><form method=\"post\" action=\"".htmlentities($_SERVER['REQUEST_URI'])."#a".$row[0]."\"><input type=\"hidden\" name=\"close_row\" value=\"".($row[0])."\" /><input type=\"hidden\" name=\"viewstate\" value=\"".$viewstate."\" /><INPUT TYPE=\"submit\" name=\"open\" VALUE=\"\" class=\"submit_open\"><br /><INPUT TYPE=\"submit\" name=\"pending\" VALUE=\"\" class=\"submit_pending\"><br /><INPUT TYPE=\"submit\" name=\"pending_ami\" VALUE=\"\" class=\"submit_pendingami\"><br /><INPUT TYPE=\"submit\" name=\"pending_arp\" VALUE=\"\" class=\"submit_pendingarp\"><br /><INPUT TYPE=\"submit\" name=\"close\" VALUE=\"\" class=\"submit_closed\"></form></p></div></div>";
}
 }
mysql_free_result ($result);

 }  


if (is_null($viewstate)) {
    $viewstate='all';
}



?>

[tomhath]

It looks to me that $viewstate is only set on a POST. When you click on a link the GET defaults to the initial state.

[maximus83]

Hi Tom,

thanks for that, any ideas how to make sure the GET doesn't default to the initial state. Like I said I'm a new to PHP .

[johanafm]

It's rather an issue of HTML, even if you have to provide the actual pagination number with PHP. A get request takes its parameter values in the so called query string, i.e. everything after the ? in the url. These parameters are name=value pairs and are separated by the ampersand character &. However, since & marks the start of a character entity reference in (X)HTML, you have to output & for it to become an actual &.
You can look at how the value is set in the post request, i.e. from within the form, then in the same way output

<a href="http://example.com/file.php?pagination=value">link text</a>

[maximus83]

'<a href="'?page_id=1072??s=' . ($start + $display) . '&amp;np=' . $num_pages . '">Next</a> ';

This is my url link for the next set of records, what is my issue? this is processed as :

?page_id=1072?s=6&np=3 in the browser, the 3 is for start and np for total number of pages, I'm still lost as to why I dont get anything other than the first set of results, do I need something else in my url?

[johanafm]

You should indent your code so that you, and others, can see what blocks of code belongs to what if or else statements.

if (isset($_GET['np'])) {
    $num_pages = $_GET['np'];

} else {

    if ($viewstate=='open') {   
    $query = "SELECT ticket_number, first_name, surname, email, product, retailer, DATE_FORMAT(dop, '%d %M %Y') AS dop, message, address, DATE_FORMAT(created, '%d %M %Y %r') AS created, status FROM support_dev WHERE status='Open' ORDER BY ticket_number ASC";
    $query_result = mysql_query ($query); 
    $num_records = @mysql_num_rows ($query_result);

    if ($num_records > $display) {
$num_pages = ceil ($num_records/$display);
} else {
    $num_pages = 1;
    }
}


elseif ($viewstate=='pending') {
    $query = "SELECT ticket_number, first_name, surname, email, product, retailer, DATE_FORMAT(dop, '%d %M %Y') AS dop, message, address, DATE_FORMAT(created, '%d %M %Y %r') AS created, status FROM support_dev WHERE status LIKE 'Pending%' ORDER BY ticket_number ASC";
    $query_result = mysql_query ($query); 
    $num_records = @mysql_num_rows ($query_result);

    if ($num_records > $display) {
$num_pages = ceil ($num_records/$display);
} else {
    $num_pages = 1;
}
    }

elseif ($viewstate=='closed') {
        $query = "SELECT ticket_number, first_name, surname, email, product, retailer, DATE_FORMAT(dop, '%d %M %Y') AS dop, message, address, DATE_FORMAT(created, '%d %M %Y %r') AS created, status FROM support_dev WHERE status='Closed' ORDER BY ticket_number ASC";
        $query_result = mysql_query ($query); 
        $num_records = @mysql_num_rows ($query_result);

    if ($num_records > $display) {
$num_pages = ceil ($num_records/$display);
} else {
    $num_pages = 1;
    }

}
else {
        $query = "SELECT ticket_number, first_name, surname, email, product, retailer, DATE_FORMAT(dop, '%d %M %Y') AS dop, message, address, DATE_FORMAT(created, '%d %M %Y %r') AS created, status FROM support_dev ORDER BY ticket_number ASC";
        $query_result = mysql_query ($query); 
        $num_records = @mysql_num_rows ($query_result);

    if ($num_records > $display) {
$num_pages = ceil ($num_records/$display);
} else {
    $num_pages = 1;
    }
}
}

In your html code you seem to claim that np is the number of pages and s is the current one, but np is 3 and s is 6. So out of 3 pages, you are currently on page 6.

Also, in the posted code you never make use of $_GET['s'] and there is no limit clause in either of the queries.

[maximus83]

s is the starting row so page 2 would start on at row 6 page on row 0 etc.

there is a limit clause in my code if you view my query results at the bottom:

 $display = $start+6;

	if (isset($_GET['np'])) {
$num_pages = $_GET['np'];
} else {
	$where = "1"; // default
	switch ($viewstate) {
		case "open": $where = "status='Open'"; break;
		case "pending": $where = "status LIKE 'Pending%'"; break;
		case "closed": $where = "status='Closed'"; break;
	}
	$query = "SELECT ticket_number, first_name, surname, email, product, retailer, DATE_FORMAT(dop, '%d %M %Y') AS dop,
		message, address, DATE_FORMAT(created, '%d %M %Y %r') AS created, status FROM support_dev WHERE $where ORDER BY ticket_number ASC";
	$query_result = mysql_query ($query); 
	$num_records = @mysql_num_rows ($query_result);
	if ($num_records > $display) {
		$num_pages = ceil ($num_records/$display);
	} else {
		$num_pages = 1;
	}
}


if (isset($_GET['s'])) {
	$start = $_GET['s'];
} else {
	$start = $_GET['$num_pages']*6;

}

$query = "SELECT ticket_number, first_name, surname, email, product, retailer, DATE_FORMAT(dop, '%d %M %Y') AS dop, message, address, DATE_FORMAT(created, '%d %M %Y %r') AS created, status FROM support_dev ORDER BY ticket_number ASC LIMIT $start, $display";
if ($viewstate=='open') {

	$query = "SELECT ticket_number, first_name, surname, email, product, retailer, DATE_FORMAT(dop, '%d %M %Y') AS dop, message, address, DATE_FORMAT(created, '%d %M %Y %r') AS created, status FROM support_dev WHERE status='Open' ORDER BY ticket_number ASC LIMIT $start, $display";
	}
		elseif ($viewstate=='pending') {
	$query = "SELECT ticket_number, first_name, surname, email, product, retailer, DATE_FORMAT(dop, '%d %M %Y') AS dop, message, address, DATE_FORMAT(created, '%d %M %Y %r') AS created, status FROM support_dev WHERE status LIKE 'Pending%' ORDER BY ticket_number ASC LIMIT $start, $display";
	}
	elseif ($viewstate=='closed') {
	$query = "SELECT ticket_number, first_name, surname, email, product, retailer, DATE_FORMAT(dop, '%d %M %Y') AS dop, message, address, DATE_FORMAT(created, '%d %M %Y %r') AS created, status FROM support_dev WHERE status='Closed' ORDER BY ticket_number ASC LIMIT $start, $display";}
	$result = @mysql_query ($query);
	$num = mysql_num_rows ($result);

if ($num > 0) {

if ($num_pages > 1) {
	echo '<p>';
	$current_page = ($start/$display) + 1;

	if ($current_page != 1) {
		echo '<a href="?page_id=1072?s=' . ($start - $display) . '&np=' . $num_pages . '">Previous</a> ';
	}
	for ($i = 1; $i <= $num_pages; $i++) {
		if ($i != $current_page) {
			echo '<a href="?page_id=1072?s=' . (($display * ($i - 1))) . '&np=' . $num_pages . '">' .$i .'</a> ';
		} else {
			echo $i . ' ';
		}
	}
	if ($current_page != $num_pages) {
		echo '<a href="?page_id=1072?start=' . ($start + $display) . '&amp;np=' . $num_pages . '">Next</a> ';
	}
	echo '</p><br />'; 

}


 if ($result) {
	 echo '
<div id="supviewwrapperheader">
<div id="supviewticket"><p>Ticket</p></div> 
<div id="supviewfirst"><p>First Name</p></div>
<div id="supviewlast"><p>Last Name</p></div>
<div id="supviewemail"><p>Email</p></div>
<div id="supviewproduct"><p>Product</p></div>
<div id="supviewretailer"><p>Retailer</p></div>
<div id="supviewdop"><p>D.O.P.</p></div>
<div id="supviewmessage"><p>Message</p></div>
<div id="supviewaddress"><p>Address</p></div>
<div id="supviewcreated"><p>Date created</p></div>
<div id="supviewstatus"><p>Status</p></div>
<div id="supviewbuttons"><p></p></div>
</div>
';

while ($row = mysql_fetch_array($result, MYSQL_NUM)) {
	echo "
	<div id=\"supviewwrapper\" class=\"".($row[10])."\">
	<div id=\"supviewticket\"><p><a name=\"a".$row[0]."\"></a>$row[0]</p></div>
	<div id=\"supviewfirst\"><p>$row[1]</p></div>
	<div id=\"supviewlast\"><p>$row[2]</p></div>
	<div id=\"supviewemail\"><p>$row[3]</p></div>
	<div id=\"supviewproduct\"><p>$row[4]</p></div>
	<div id=\"supviewretailer\"><p>$row[5]</p></div>
	<div id=\"supviewdop\"><p>$row[6]</p></div>
	<div id=\"supviewmessage\"><p>$row[7]</p></div>
	<div id=\"supviewaddress\"><p>$row[8]</p></div>
	<div id=\"supviewcreated\"><p>$row[9]</p></div>
	<div id=\"supviewstatus\"><p>$row[10]</p></div>
	<div id=\"supviewbuttons\"><p><form method=\"post\" action=\"".htmlentities($_SERVER['REQUEST_URI'])."#a".$row[0]."\"><input type=\"hidden\" name=\"close_row\" value=\"".($row[0])."\" /><input type=\"hidden\" name=\"viewstate\" value=\"".$viewstate."\" /><INPUT TYPE=\"submit\" name=\"open\" VALUE=\"\" class=\"submit_open\"><br /><INPUT TYPE=\"submit\" name=\"pending\" VALUE=\"\" class=\"submit_pending\"><br /><INPUT TYPE=\"submit\" name=\"pending_ami\" VALUE=\"\" class=\"submit_pendingami\"><br /><INPUT TYPE=\"submit\" name=\"pending_arp\" VALUE=\"\" class=\"submit_pendingarp\"><br /><INPUT TYPE=\"submit\" name=\"close\" VALUE=\"\" class=\"submit_closed\"></form></p></div></div>";
}
 }

mysql_free_result ($result);

 }	


if (is_null($viewstate)) {
	$viewstate='all';
}



?>

[bradgrafelman]

Why are you outputting links like:

?page_id=1072?s=123&np=456

Such a link would create $GET['page_id'] with a value of '1072?s=123' and $GET['np'] with a value of '456'.

[maximus83]

bradgrafelman;10985408 wrote:

Why are you outputting links like:

?page_id=1072?s=123&np=456

Such a link would create $GET['page_id'] with a value of '1072?s=123' and $GET['np'] with a value of '456'.

i'm not my out putted link = http://www.xxxxxxxxxxxx.com/wordpress/?page_id=1072?s=6&np=3

the pge id is the wordpress generated link the s is for start row in this case 6 so this wouldd be page 6 for results 7 -12 np is for number of pages in this case 3 pages as i have 15 results.

Any idea why the first set of results for pages 1 is being reloaded instead of loading page 2 and onwards.

[bradgrafelman]

maximus83;10985410 wrote:

i'm not my out putted link = http://www.xxxxxxxxxxxx.com/wordpress/?page_id=1072?s=6&np=3

Yes, that URL exhibits the same problem as I noted above. I said "like" meaning that I was omitting the beginning part of the full URL and focusing on the query string itself, which is where the problem is.

[tomhath]

I think Brad is correct. Shouldn't the URL be:

?page_id=1072&s=123&np=456

Instead of:

?page_id=1072?s=123&np=456

[johanafm]

Yes it should. And IF there was supposed to be a question mark, after the question mark that delimites the start of the query string, it should be urlencoded and thus expressed as '%3F'.

Apart from that, there are several other things I believe needs fixing in the current code

# The number of items to display per page depends on the start value?
# See the query below: LIMIT $start, $display
# which means that $display is the number of records retrieved
$display = $start+6;

if (isset($_GET['np']))
{
	$num_pages = $_GET['np'];
}
else
{
	# Personally, I'd change the $where part to
    $where = ''; // default
    switch ($viewstate) {
        case "open": $where = "WHERE status='Open'"; break;
        case "pending": $where = "WHERE status LIKE 'Pending%'"; break;
        case "closed": $where = "WHERE status='Closed'"; break;
    }
    # since this will give you a query without "WHERE 1" when you don't need a where clause

$query_result = mysql_query ($query);
# Error supression operator! Always a sign that something should be done differently
#    $num_records = @mysql_num_rows ($query_result);
	if (!$query_result)
	{
		error_log(sprintf('%s (%d):%s%s', mysql_error(), mysql_errno(), PHP_EOL, $query));
		# also inform user that the page cannot be displayed right now
		# They should try again later, or contact support if the problem persists.

	$num_pages = 0;
}
else
{
	if ($num_records > $display)
	{
		$num_pages = ceil ($num_records/$display);
	}
	else
	{
		$num_pages = 1;
	}
}
}


if (isset($_GET['s']))
{
    $start = $_GET['s'];
}
else
{
	/* This will read exactly: $_GET['$num_pages']
	 * without any substitution of $num_pages. You are in other words referring to
	 * an array element containing with string literal that starts with a $-sign followed
	 * by num_pages.
	 * If you had error reporting turned on and checked your error log, you would instantly
	 * become aware of this. While developping, you might also want to turn on display_errors.
	 * All of this is done in php.ini:
	 *		- error_reporting:	What errors to report
	 *		- log_errors:		What file to log errors to
	 *		- display_errors:	Where to display errors such as stdout or stderr
	 */
    $start = $_GET['$num_pages']*6;
}
/* Assuming you were actually referring to $_GET["$num_pages"], or better yet
 * $_GET[$num_pages], you'd take the value of the array element whose key
 * is whatever number is in $num_pages, for example $_GET[4];
 * And if it was supposed to be $num_pages, rather than $_GET[$num_pages]
 * you'd start at or most likely beyond the number of records.
 * Let's say you have 10 records and 6 per page giving 2 pages, then this last case would mean
 * $start = 6 * 2 = 12 which is beyond the 10 existing records.
 */


# $start should be in the range [0, N-1], N = number of records
# $start should also be a multiple of $display
# $display should be the number of records per page
$query = "... LIMIT $start, $display";

[maximus83]

thanks for help and insights guys , still no luck with getting my 2nd, 3rd pages of results to work though. This is driving me nuts.

[maximus83]

I've found that if i set start to row 6 ($start=6 , instead of $start=0) or any other number in the middle of the rows record and then click on the link for page 1 thus start becomes 0 the link works correctly.

http://www.xxxxxx.com/wordpress/?page_id=1072&s=0&np=3 - this works to display page 1

this doesn't work I just get a 404 page
http://www.xxxxxx.com/wordpress/?page_id=1072&s=6&np=3 can't understand why only s=0 works in the url string any ideas?