Hi everyone forgive me I am a little new to php but I can get by....

I am trying to implement a pagination script into my sub cat page in a shop however I ahve most of it done but cannot seem to get the pageination to show can someone please put my back on tracks I am getting confused.

I have included my code so you can see thanks in advanced.

<?php



$shopcatid = $_GET['id'];
$shopsubcatid = $_GET['s'];


	// How many adjacent pages should be shown on each side?
	$adjacents = 3;
	
	/* 
	   First get total number of rows in data table. 
	   If you have a WHERE clause in your query, make sure you mirror it here.
	*/
	$query = "SELECT COUNT(*) as num FROM shopcatid = '" . $shopcatid . "' AND shopsubcatid = '" . $shopsubcatid . "'";
	$total_pages = mysql_fetch_array(mysql_query($query));
	$total_pages = $total_pages[num];
	
	/* Setup vars for query. */
	$targetpage = "sub-category.php"; 	//your file name  (the name of this file)
	$limit = 9; 								//how many items to show per page
	$page = $_GET['page'];
	if($page) 
		$start = ($page - 1) * $limit; 			//first item to display on this page
	else
		$start = 0;								//if no page var is given, set start to 0
	
	/* Get data. */
	$query = "SELECT * FROM shop_products WHERE shopcatid = '" . $shopcatid . "' AND shopsubcatid = '" . $shopsubcatid . "' LIMIT $start, $limit";
	$result = mysql_query($query) or die( "An error has occured: " .mysql_error (). ":" .mysql_errno ());
	
	/* Setup page vars for display. */
	if ($page == 0) $page = 1;					//if no page var is given, default to 1.
	$prev = $page - 1;							//previous page is page - 1
	$next = $page + 1;							//next page is page + 1
	$lastpage = ceil($total_pages/$limit);		//lastpage is = total pages / items per page, rounded up.
	$lpm1 = $lastpage - 1;						//last page minus 1
	
	/* 
		Now we apply our rules and draw the pagination object. 
		We're actually saving the code to a variable in case we want to draw it more than once.
	*/
	$pagination = "";
	if($lastpage > 1)
	{	
		$pagination .= "<div class=\"pagination\">";
		//previous button
		if ($page > 1) 
			$pagination.= "<a href=\"$targetpage?page=$prev\">« previous</a>";
		else
			$pagination.= "<span class=\"disabled\">« previous</span>";	
		
		//pages	
		if ($lastpage < 7 + ($adjacents * 2))	//not enough pages to bother breaking it up
		{	
			for ($counter = 1; $counter <= $lastpage; $counter++)
			{
				if ($counter == $page)
					$pagination.= "<span class=\"current\">$counter</span>";
				else
					$pagination.= "<a href=\"$targetpage?page=$counter\">$counter</a>";					
			}
		}
		elseif($lastpage > 5 + ($adjacents * 2))	//enough pages to hide some
		{
			//close to beginning; only hide later pages
			if($page < 1 + ($adjacents * 2))		
			{
				for ($counter = 1; $counter < 4 + ($adjacents * 2); $counter++)
				{
					if ($counter == $page)
						$pagination.= "<span class=\"current\">$counter</span>";
					else
						$pagination.= "<a href=\"$targetpage?page=$counter\">$counter</a>";					
				}
				$pagination.= "...";
				$pagination.= "<a href=\"$targetpage?page=$lpm1\">$lpm1</a>";
				$pagination.= "<a href=\"$targetpage?page=$lastpage\">$lastpage</a>";		
			}
			//in middle; hide some front and some back
			elseif($lastpage - ($adjacents * 2) > $page && $page > ($adjacents * 2))
			{
				$pagination.= "<a href=\"$targetpage?page=1\">1</a>";
				$pagination.= "<a href=\"$targetpage?page=2\">2</a>";
				$pagination.= "...";
				for ($counter = $page - $adjacents; $counter <= $page + $adjacents; $counter++)
				{
					if ($counter == $page)
						$pagination.= "<span class=\"current\">$counter</span>";
					else
						$pagination.= "<a href=\"$targetpage?page=$counter\">$counter</a>";					
				}
				$pagination.= "...";
				$pagination.= "<a href=\"$targetpage?page=$lpm1\">$lpm1</a>";
				$pagination.= "<a href=\"$targetpage?page=$lastpage\">$lastpage</a>";		
			}
			//close to end; only hide early pages
			else
			{
				$pagination.= "<a href=\"$targetpage?page=1\">1</a>";
				$pagination.= "<a href=\"$targetpage?page=2\">2</a>";
				$pagination.= "...";
				for ($counter = $lastpage - (2 + ($adjacents * 2)); $counter <= $lastpage; $counter++)
				{
					if ($counter == $page)
						$pagination.= "<span class=\"current\">$counter</span>";
					else
						$pagination.= "<a href=\"$targetpage?page=$counter\">$counter</a>";					
				}
			}
		}
		
		//next button
		if ($page < $counter - 1) 
			$pagination.= "<a href=\"$targetpage?page=$next\">next »</a>";
		else
			$pagination.= "<span class=\"disabled\">next »</span>";
		$pagination.= "</div>\n";		
	}
?>

	<?php
		while($row = mysql_fetch_array($result))
		{
	
		// Get an image if we can
		$queryi = "SELECT filename FROM images WHERE productid = '" . $row['productid'] . "'";
		$resulti = mysql_query($queryi) or die( "An error has occured: " .mysql_error (). ":" .mysql_errno ());
		$tweener = mysql_fetch_row($resulti);
		$filename = $tweener[0];
		
		if ($filename != "")
		{
			// Resize thumbs ready for display
			
			$image = '<img src="' . $root . 'admin/uploads/thumb_' . $filename . '" alt="' . $row['productname'] . '" height="100" />';
		}
		else
		{
			$image = '<img src="' . $root . 'assets/no-image.jpg" alt="No Image Available" />';
		}
		
		echo '
		<div class="thumb" style="margin-left:-10px;">
			<div class="img"><a href="' . $root . 'shop/product.php?id=' . $row['productid'] . '">' . $image . '</a></div>
			<a href="' . $root . 'shop/product.php?id=' . $row['productid'] . '" style="color:#903;">' . $row['productname'] . '</a>
		</div>';
	
		}
	?>
    
    
    
<?php echo $pagination?>
Member Avatar for diafol

So with which bit do you need help. Dumping a bag of code and then saying "it doesn't work" ain't enough. Where does it go wrong? Have you put in a bunch of temporary echoes to watch the values of variables?

Pls. make it more clear

Try changing

$query = "SELECT COUNT(*) as num FROM shopcatid = '" . $shopcatid . "' AND shopsubcatid = '" . $shopsubcatid . "'";
$total_pages = mysql_fetch_array(mysql_query($query));
$total_pages = $total_pages[num];

to this

$query = "SELECT COUNT(*) FROM shopcatid = '" . $shopcatid . "' AND shopsubcatid = '" . $shopsubcatid . "'";
$total_pages = mysql_fetch_row(mysql_query($query));
$total_pages = $total_pages[0];

Just my humble opinion, why not remove the entire pagination codes out of your codes and make it a function. This way your page is only left with the database query and the results.

Yes sorry perhaps you are correct people... when I click the link to the next page it does not show the next 9 products it appears to be just blank..

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.