itsrien Posted November 18, 2010 Share Posted November 18, 2010 I'am using Adam's pagination script from: http://www.developphp.com/view_lesson.php?v=289 And I adjusted the code just a little, so it fitted my needs. Now I bumb into a problem, and I can't figure it out, how to let the script doing its job right. The adjusted pagination script pulls a list of links, from my database from a certain category. The pagination script is called from a link on another page, so that other categories can be choosen there. In the url of that link are the variables needed for the the pagination script to doing it's work. (see example bellow:) http://localhost/database/links-view.php?ID=73&pid=2&aid=1 The pagination script actually pulls data from my database, but only the first result page is shown, (only the 1e 10 result) when I try to click on the next button, to show the next 10 results, a empty page is shown. And the following error code appears: Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource in /var/www/database/links-view.php on line 79 The problem is that the variables ID, pid and aid are not posted/pass on to the rest of the script. My question is. How can I adjust the script so those variables are pass on/posted corectly, so the script can do it's work? Bellow the adjusted code of Adam's pagination script: <?php include "connect.php"; include "var.php"; $sql = mysql_query("SELECT * FROM linkpartners WHERE ID='$ID' AND pid='$pid' AND aid='$aid' ORDER BY lpid ASC"); //////////////////////////////////// Adam's Pagination Logic //////////////////////////////////////////////////////////////////////// $nr = mysql_num_rows($sql); // Get total of Num rows from the database query if (isset($_GET['pn'])) { // Get pn from URL vars if it is present $pn = preg_replace('#[^0-9]#i', '', $_GET['pn']); // filter everything but numbers for security(new) } else { // If the pn URL variable is not present force it to be value of page number 1 $pn = 1; } //This is where we set how many database items to show on each page $itemsPerPage = 10; // Get the value of the last page in the pagination result set $lastPage = ceil($nr / $itemsPerPage); // Be sure URL variable $pn(page number) is no lower than page 1 and no higher than $lastpage if ($pn < 1) { // If it is less than 1 $pn = 1; // force if to be 1 } else if ($pn > $lastPage) { // if it is greater than $lastpage $pn = $lastPage; // force it to be $lastpage's value } // This creates the numbers to click in between the next and back buttons // This section is explained well in the video that accompanies this script $centerPages = ""; $sub1 = $pn - 1; $sub2 = $pn - 2; $add1 = $pn + 1; $add2 = $pn + 2; if ($pn == 1) { $centerPages .= ' <span class="pagNumActive">' . $pn . '</span> '; $centerPages .= ' <a href="' . $_SERVER['PHP_SELF'] . '?pn=' . $add1 . '">' . $add1 . '</a> '; } else if ($pn == $lastPage) { $centerPages .= ' <a href="' . $_SERVER['PHP_SELF'] . '?pn=' . $sub1 . '">' . $sub1 . '</a> '; $centerPages .= ' <span class="pagNumActive">' . $pn . '</span> '; } else if ($pn > 2 && $pn < ($lastPage - 1)) { $centerPages .= ' <a href="' . $_SERVER['PHP_SELF'] . '?pn=' . $sub2 . '">' . $sub2 . '</a> '; $centerPages .= ' <a href="' . $_SERVER['PHP_SELF'] . '?pn=' . $sub1 . '">' . $sub1 . '</a> '; $centerPages .= ' <span class="pagNumActive">' . $pn . '</span> '; $centerPages .= ' <a href="' . $_SERVER['PHP_SELF'] . '?pn=' . $add1 . '">' . $add1 . '</a> '; $centerPages .= ' <a href="' . $_SERVER['PHP_SELF'] . '?pn=' . $add2 . '">' . $add2 . '</a> '; } else if ($pn > 1 && $pn < $lastPage) { $centerPages .= ' <a href="' . $_SERVER['PHP_SELF'] . '?pn=' . $sub1 . '">' . $sub1 . '</a> '; $centerPages .= ' <span class="pagNumActive">' . $pn . '</span> '; $centerPages .= ' <a href="' . $_SERVER['PHP_SELF'] . '?pn=' . $add1 . '">' . $add1 . '</a> '; } // This line sets the "LIMIT" range... the 2 values we place to choose a range of rows from database in our query $limit = 'LIMIT ' .($pn - 1) * $itemsPerPage .',' .$itemsPerPage; // Now we are going to run the same query as above but this time add $limit onto the end of the SQL syntax // $sql2 is what we will use to fuel our while loop statement below $sql2 = mysql_query("SELECT * FROM linkpartners WHERE ID='$ID' AND pid='$pid' AND aid='$aid' ORDER BY lpid ASC $limit"); //////////////////////////////// END Adam's Pagination Logic //////////////////////////////////////////////////////////////////////////////// ///////////////////////////////////// Adam's Pagination Display Setup ///////////////////////////////////////////////////////////////////// $paginationDisplay = ""; // Initialize the pagination output variable // This code runs only if the last page variable is ot equal to 1, if it is only 1 page we require no paginated links to display if ($lastPage != "1"){ // This shows the user what page they are on, and the total number of pages $paginationDisplay .= 'Page <strong>' . $pn . '</strong> of ' . $lastPage. ' '; // If we are not on page 1 we can place the Back button if ($pn != 1) { $previous = $pn - 1; $paginationDisplay .= ' <a href="' . $_SERVER['PHP_SELF'] . '?pn=' . $previous . '"> Back</a> '; } // Lay in the clickable numbers display here between the Back and Next links $paginationDisplay .= '<span class="paginationNumbers">' . $centerPages . '</span>'; // If we are not on the very last page we can place the Next button if ($pn != $lastPage) { $nextPage = $pn + 1; $paginationDisplay .= ' <a href="' . $_SERVER['PHP_SELF'] . '?pn=' . $nextPage . '"> Next</a> '; } } ///////////////////////////////////// END Adam's Pagination Display Setup /////////////////////////////////////////////////////////////////////////// // Build the Output Section Here $headerslist .= '<tr><td class="view-ctd">lpid</td> <td class="view-ctd">Url</td> </tr>'; $outputList = ''; while($row = mysql_fetch_array($sql2)){ $ID = $row["ID"]; $pid = $row["pid"]; $aid = $row["aid"]; $lpid = $row["lpid"]; $url = $row["url"]; $title = $row["title"]; $outputList .= '<tr><td class="view-dct">' . $lpid . '</td> <td class="view-dct"><a href="' . $url . '"target="_blank">' . $title . '</a></td> </tr>'; } // close while loop ?> <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd"> <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en"> <head> <title>Page Title</title> <meta http-equiv="content-type" content="text/html;charset=utf-8" /> <link rel="StyleSheet" type="text/css" media="screen" href="styles.css" /> <style type="text/css"> </style> </head> <body> <div style="margin-left:64px; margin-right:64px;"> <h2>Total Items: <?php echo $nr; ?></h2> </div> <div style="margin-left:58px; margin-right:58px; padding:6px; background-color:#FFF; border:#999 1px solid;"><?php echo $paginationDisplay; ?></div> <div style="margin-left:64px; margin-right:64px;"><table><?php print "$headerslist"; ?></div> <div style="margin-left:64px; margin-right:64px;"><?php print "$outputList"; ?></table></div> <div style="margin-left:58px; margin-right:58px; padding:6px; background-color:#FFF; border:#999 1px solid;"><?php echo $paginationDisplay; ?></div> </body> </html> Quote Link to comment https://forums.phpfreaks.com/topic/219060-php-pagination-and-postsession-variables-problem/ Share on other sites More sharing options...
upp Posted November 20, 2010 Share Posted November 20, 2010 i cant find where $pid and $aid are established in your script, and also on this part of the script: $paginationDisplay .= ' <a href="' . $_SERVER['PHP_SELF'] . '?pn=' . $nextPage . '"> Next</a> '; change that to: $paginationDisplay .= ' <a href="' . $_SERVER['PHP_SELF'] . '?pn=' . $nextPage . '&ID='. $id .'&aid='. $aid. '"> Next</a> '; so that id and aid are passing into the next page using $_GET Quote Link to comment https://forums.phpfreaks.com/topic/219060-php-pagination-and-postsession-variables-problem/#findComment-1137105 Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.