Help! I've encoutered a problem using a 'add comment' php script

[pepsymax]

First i'll apologise as I know little about php, so bear with me if my explanation is not so good!

I've installed a php comment script (a script that allows my users to post a review on my php pages). I've installed it and it works.

However a little error occurs if I want the results to show up on more than one page. The problem is, the code is looking for a url ending with (.php). It then adds the code to forward to the next page directly after the php comment. My pages dont work as they end like this

index.php?&task=detail&id=7&mode=0&catid=45&navstart=0&search=*

The php comment in the code occurs too early on my pages, so the command to foward appears directly after php and forwards to the wrong page.

So! Instead of producing something like this...

index.php?&task=detail&id=7&mode=0&catid=45&navstart=0&search=*NEW_PAGE_CODE

It does this

index.php?NEW_PAGE_CODE

So I think the code tries to open the next page from (index.php) instead of from the end of the code (search*)

Now What I need is to rewrite the script so that it links after the (search=* ) comment.

Ok I think I've explained it... here is the code I have to change. Any ideas?

<?php

# configuration 
	$emptyMsg 	= " Be the first to add comment to this page.";
	$size 			= 40;	# size of input boxes 
	$cols 			= 53; # width of text areas 
	$rows				= 5;	# height of text areas 

$bgColor 		= "#eaeaea";

# input box length adjustment code 
	$ie = stristr($HTTP_USER_AGENT, "MSIE") && true;
	$op = stristr($HTTP_USER_AGENT, "Opera") && true;
	$n6 = stristr($HTTP_USER_AGENT, "Netscape6") && true;
	$ns = (!($ie)) && (!($n6));
	$size = fw($size);
	$cols = fc($cols);

function fw($w) {
  global $ns;
  if ($ns == 1) { $w = $w * 0.8; }
  return ceil($w);
}

function fc($w) {
  global $n6;
  if ($n6 == 1) { $w = $w * 0.7; } else { $w = $w * 0.75; }
  return ceil($w);
}

# FUNCTIONS ###################################################################

# Calculate page navigation tool and define the lower and upper record numbers
	function genNav ($listID, $items, $ipp, &$lowLimit, &$topLimit) {
	  $self = $_SERVER['PHP_SELF'];
		if (!isset($_GET['pID'])) { $pID = 0; } else { $pID = $_GET['pID']; }

	$str = "";
	$numberOfPages = ceil($items / $ipp);
	$lowLimit = $pID * $ipp;
	$topLimit = $lowLimit + $ipp;
	if ($topLimit > $items) { $topLimit = $items; }

	for ( $i=0; $i < $numberOfPages; $i++ ) {
		if ($i == $pID) {
			$str .= "<b>".($i+1)."</b> ";
		} else { $str .= "<a href=\"$self?pID=$i&listID=$listID#comment\">".($i+1)."</a> "; }
	}
	return $str;
}
?>

If you need it the whole script can be found http://www.greatnexus.com/_downloads/commenter.php

Thanks

pep

[TenFold]

you have to escape the & character..
just escape the part in your script that says this...

for ( $i=0; $i < $numberOfPages; $i++ ) {
            if ($i == $pID) {
                $str .= "<b>".($i+1)."</b> ";
            } else { $str .= "<a href=\"$self?pID=$i&listID=$listID#comment\">".($i+1)."</a> "; }
        } 

with this...

for ( $i=0; $i < $numberOfPages; $i++ ) {
            if ($i == $pID) {
                $str .= "<b>".($i+1)."</b> ";
            } else { $str .= "<a href=\"$self?pID=$i\&listID=$listID#comment\">".($i+1)."</a> "; }
        }

that might do it..

[pepsymax]

I played around with it, but I discovered that it was a another page that I needed to change (i think).

I made a mistake with the top post as the code posted by the script always starts with a '?'. It looks like the script seeks out a '?' then replaces it with with the new script path.

Here is the new php code that has to be changed. If you could help me ignore the '?', or make the script seek out the command 'search=*' and replace this instead of a '?', then it really might work!

<?php
# Change these variables as needed.
	global $relPath, $bgColor;
	$relPath	= "/components/com_bookmarks";

# No change below this line is needed.
	$baseRoot = $_SERVER['DOCUMENT_ROOT'].$relPath;

include_once("$baseRoot/includes/~config.php");
include_once("$baseRoot/includes/~interface.php");
include_once("$baseRoot/includes/~flatfile.php");
include_once("$baseRoot/includes/~getListNumber.php");
include_once("$baseRoot/includes/$template/~genBody.php");

$page	= $_SERVER["SCRIPT_NAME"];
$page	= ereg_replace(".PHPSESSID=[_a-zA-Z0-9]*","",$page);
if ($useQueryString) {
	$page .= "?".$_SERVER["QUERY_STRING"];
	$page	 = ereg_replace(".pID=[0-9]*","",$page);
	$page	 = ereg_replace(".listID=[0-9]*","",$page);
	$page  = ereg_replace("#comment","",$page);
	$page  = ereg_replace("\?$","",$page);
}
$gnBody = $emptyMsg;
$gnNav	= "";

# check if URL is in list.txt, if yes return list #
	$dataFile	=	"$baseRoot/data/list.txt";
	$makeNew	= false;
	$listID = ( !isset($_GET['listID'])
		? getListNumber ($dataFile, $page, $makeNew)
		: $_GET['listID']);

# Get & display entries 
  if ($listID) {
		$dataFile = "$baseRoot/data/$listID.txt";
		if (openFile ($dataFile, $dataArray, $error)) {
			$items = count($dataArray);
			rsort($dataArray);
			$gnNav 	= genNav ($listID, $items, $ipp, $lowLimit, $topLimit);
			$gnBody = genBody ($dataArray, $lowLimit, $topLimit, 1);
		}
	}

# Display commenter
	$PAGE_NAVIGATION	= $gnNav;
	$BODY		 					= $gnBody;
	$BASE_URL 				= $relPath;
	include_once("$baseRoot/includes/$template/~tpl_include.htm");
?>