ReWriteRule Creating a PHP error!

[casbboy]

Ok, I have already posted how I have been having a problem with this script:

$newsid = $_GET['id']; 
$getnews = mysql_query("SELECT * FROM news WHERE newsid='$newsid' LIMIT 1",$conn); 
$newscheck = mysql_num_rows($getnews); 
$news = mysql_fetch_array($getnews); 

//update views  LOOK HERE 
if($newscheck == 1) { 
$views = $news['views']; 
$nextview= $views + 1; 
$addsql = mysql_query("UPDATE news SET views=$nextview WHERE newsid='$newsid'",$conn); 
} 
//LOOK ABOVE 

This was suppose to add one 'view' to the news article [counting views for popularity], but it was adding two.

I realized that a rewriterule was the problem.. here is the rule:

RewriteRule news/([0-9a-zA-Z#]+)/([0-9a-zA-Z#]+) /news.php?id=$1&title=$2

so:
news/9/mamalike
news.php?id=9&title=mamalike

When I run the file by doing the full path [no rewriterule] by news.php?, the update works perfect by adding '1'. When I do the rewritepath news/9/mamalike, the PHP error begins as it adds '2'.

What gives?

Thanks
Ryan

[Weedpacket]

Wouldn't

$newsid = (int)$_GET['id'];  

mysql_query("UPDATE news SET views=views+1 WHERE newsid='$newsid'",$conn); 

have been simpler all round? In the absence of a link to the other thread where you discuss this....

[casbboy]

I like that way better.

HOwever, the page is still getting run twice for some reason.

Well, I know the reason is the rewriterule posted in previous messege. Without the rule, and just doing file.php?id=10&title=try

it works just fine. With the rule, this starts happening.

With your script to update, it was still adding two each time the page was loaded.

Thanks
Ryan