Hey everyone,

This is my first attempt at pagination. I seem to have everything working pretty well except for one slight problem.

Example: If I search for a PO Number containing the number 10. My query returns all the correct information along with the correct pagination links, however, Let's say I'm on page 6 of a current search. Now I type in a new term to search for such as PO Number containing '11', it returns the results - but, it automatically starts the new results on page 6 since that is the page I was on from the last search. I am still very much in the learning stage of php, and I don't know how to clear the $currentpage variable on a new search.

Here is code:

<?php session_start(); ?>
      <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
      <html xmlns="http://www.w3.org/1999/xhtml" lang="en" xml:lang="en">

                <title>Search For A PO</title>
<link href="style.css" rel="stylesheet" type="text/css" />

<div id="static">
<div class="static">
<a href="http://mysite/">Home</a> </div>

<div id="static2">
<div class="static2">
<a href="http://mysite/insert.php">Create A New PO</a> </div>

<div id="content">
<div class="contentbox">
        <h1>Search For A PO</h1>
         $type = $_REQUEST['type'];
         $term = $_REQUEST['term'];

            <form method="post" action="">
     <input type="hidden" name="action" value="search"/>
<table class="search">
            <tr><td> Search: <select name="type">
            <option value="po_num"<?php if(isset($_REQUEST['type']) && $_REQUEST['type']=='po_num'){print "selected";}?>>PO Number</option>
            <option value="date" <?php if(isset($_REQUEST['type']) && $_REQUEST['type']=='date'){print "selected";}?>>Date</option>
            <option value="vin_num"<?php if(isset($_REQUEST['type']) && $_REQUEST['type']=='vin_num'){print "selected";}?>>VIN Number</option>
<td>for: <input type="text" name="term" /><br /></td></tr>
           <tr><td> <input type="submit" name="submit" value="Search" /></td>
<td class="date">* date format: yyyymmdd</td></tr>
<hr width='100%'></hr>
  if ($term == '')
    echo '<h2>Please enter a value</h2>';
        if(!strcmp($_REQUEST['action'],"search"))  {
// database connection info
$conn = mysql_connect('localhost','xxx','xxx') or trigger_error("SQL", E_USER_ERROR);
$db = mysql_select_db('xxx',$conn) or trigger_error("SQL", E_USER_ERROR);

//safe strings
$safetype = mysql_real_escape_string($type);
$safeterm = trim(mysql_real_escape_string($term));

// find out how many rows are in the table
$sql = "SELECT COUNT(*) FROM parts_ord where $safetype like '%$safeterm%' AND date<>'0'";
$result = mysql_query($sql, $conn) or trigger_error("SQL", E_USER_ERROR);
$r = mysql_fetch_row($result);
$numrows = $r[0];

// number of rows to show per page
$rowsperpage = 2;
// find out total pages
$totalpages = ceil($numrows / $rowsperpage);

// get the current page or set a default
if (isset($_GET['currentpage']) && is_numeric($_GET['currentpage'])) {
   // cast var as int
   $currentpage = (int) $_GET['currentpage'];
} else {
   // default page num
   $currentpage = 1;
} // end if

// if current page is greater than total pages...
if ($currentpage > $totalpages) {
   // set current page to last page
   $currentpage = $totalpages;
} // end if
// if current page is less than first page...
if ($currentpage < 1) {
   // set current page to first page
   $currentpage = 1;
} // end if

// the offset of the list, based on current page
$offset = ($currentpage - 1) * $rowsperpage;

        $sql = "select * from parts_ord where $safetype like '%$safeterm%' AND date<>'0' ORDER BY date DESC LIMIT $offset, $rowsperpage";
        $result = mysql_query($sql, $conn) or trigger_error("SQL", E_USER_ERROR);

                echo '<h2>Search Results: '.$numrows.' matches found.</h2>';

                while ($row = mysql_fetch_assoc($result)){
echo "<br/><table class='results'>
<tr><td class='short'><B>PO Number:</B> </td><td>".$row['po_num']."</td></tr>
<tr><td><B>Vendor:</B> </td><td>".$row['vendor']."</td></tr>
<tr><td><B>Date:</B> </td><td>".$row['date']."</td></tr>
<tr><td><B>VIN Number:</B> </td><td>".$row['vin_num']."</td></tr>
<tr><td><B>Description:</B> </td><td>".$row['descr']."</td></tr>
<tr><td><B>Invoice Number:</B> </td><td>".$row['inv_num']."</td></tr>
<tr><td><B>Purchase Agent:</B> </td><td>".$row['agt']."</td></tr>
<tr><td colspan='2'><a href=\"edit.php/?id=".$row['po_num']."\">Edit</a></td></tr></table>";


if ($numrows==0){
/******  build the pagination links ******/
// range of num links to show
$range = 10;

// if not on page 1, don't show back links
if ($currentpage > 1) {
   // show << link to go back to page 1
   echo " <a href='{$_SERVER['PHP_SELF']}?currentpage=1&term=$term&type=$type&action=search'>First</a> ";
   // get previous page num
   $prevpage = $currentpage - 1;
   // show < link to go back to 1 page
   echo " <a href='{$_SERVER['PHP_SELF']}?currentpage=$prevpage&term=$term&type=$type&action=search'><</a> ";
} // end if

// loop to show links to range of pages around current page
for ($x = ($currentpage - $range); $x < (($currentpage + $range) + 1); $x++) {
   // if it's a valid page number...
   if (($x > 0) && ($x <= $totalpages)) {
      // if we're on current page...
      if ($x == $currentpage) {
         // 'highlight' it but don't make a link
         echo " [<b>$x</b>] ";
      // if not current page...
      } else {
         // make it a link
         echo " <a href='{$_SERVER['PHP_SELF']}?currentpage=$x&term=$term&type=$type&action=search'>$x</a> ";
      } // end else
   } // end if
} // end for

// if not on last page, show forward and last page links
if ($currentpage != $totalpages) {
   // get next page
   $nextpage = $currentpage + 1;
    // echo forward link for next page
   echo " <a href='{$_SERVER['PHP_SELF']}?currentpage=$nextpage&term=$term&type=$type&action=search'>></a> ";
   // echo forward link for lastpage
   echo " <a href='{$_SERVER['PHP_SELF']}?currentpage=$totalpages&term=$term&type=$type&action=search'>Last</a> ";
} // end if
/****** end build pagination links ******/
<hr width='100%'></hr>

Any Help is very much appreciated.

Recommended Answers

All 2 Replies

Member Avatar for diafol

I was wondering why you were using $_REQUEST. I know see that you're using POST for the form and GET for the querystring. $_REQUEST is a pig. You'd be better off changing the method of your form to GET as well, so you can use $_GET as opposed to $_REQUEST. This will also ensure that your form data is passed via querystring (may benefit your SEO??).

Here's how I'd do it:

<form method="GET" action="<?php echo $_SERVER['PHP_SELF'];">
     <input type="hidden" name="action" value="search" />
     [B]<input type="hidden" name="currentpage" value="1" />[/B]
Search: <select name="type">
            <option value="po_num"<?php if(isset($_GET['type']) && $_GET['type']=='po_num'){print "selected";}?>>PO Number</option>
            <option value="date" <?php if(isset($_GET['type']) &&  $_GET['type']=='date'){print "selected";}?>>Date</option>
            <option value="vin_num"<?php if(isset($_GET['type']) && $_GET['type']=='vin_num'){print "selected";}?>>VIN Number</option>
for: <input type="text" name="term" /> 
<input type="submit" name="submit" value="Search" /></td>

Although, I'd check user input before inserting it straight into the form. You could add the currentpage hidden thingy or just set the current page default to '1' if it is not in the querystring.

Your results should show up, with the pages listed below. If the form is sent again, it should then default to page 1 (should avoid a problem with 'stuck pages'). However, this doesn't solve your stuck pages wrt page links. I suggest you use sessions to get around this little problem.

Just place


at the top of the page, then after you check for your GET variables (querystring parameters):

CHECK presence and validity of $_GET, $_GET, $_GET, $_GET - if all OK call them $cp, $type, $term, $action for now:

if(isset($_SESSION['type']) && $_SESSION['type'] == $type && isset($_SESSION['term']) && $_SESSION['term'] == $term){
 //this is an old search - so the currentpage shouldn't need to be reset.  

 //this is a new search - so reset current page to 1 and fill session variables with the form (get) data:
  $_SESSION['term'] = $term;
  $_SESSION['type'] = $type;
  $cp = 1;

Something along those lines anyway

This helped a lot. I have added the hidden field and changed all REQUEST to GET. Not only does it work, it works a lot faster! Thanks again!

Be a part of the DaniWeb community

We're a friendly, industry-focused community of developers, IT pros, digital marketers, and technology enthusiasts meeting, networking, learning, and sharing knowledge.