whats wrong?

[arkismad]

     if (isset($_GET['l'])) {          

         $link = $db->execute("SELECT * FROM links WHERE linkcode='{$_GET['l']}'");
         if (mysql_num_rows($link)>0) {
             $link = mysql_fetch_array($link, MYSQL_ASSOC);             

             $db->execute("UPDATE links SET visits=" . ++$link['visits'] . " WHERE linkcode='{$_GET['l']}'") ;
             header("Location: http://www.urlrewarder.com/ad.php" . urlencode ('$link')
             exit;     

         }
     }

can anyone see a problem with this?

[djjjozsi]

i changed som thing. but i think you made this wrong:
urlencode ('$link') <<<<<------- change the link into: $link["url"] , or just send toward the link ID only...

how adhout this:

arkismad;10891866 wrote:

     if (isset($_GET['l'])) {          

         $link = $db->execute("SELECT * FROM links WHERE linkcode='".mysql_real_escape_string($_GET['l'])."'   ");
         if (mysql_num_rows($link)>0) {
             $link = mysql_fetch_assoc($link);             

             $db->execute("UPDATE links SET visits=visits+1 WHERE linkcode='". mysql_real_escape_string($_GET['l'])."'   "  ) ;
             header("Location: http://www.urlrewarder.com/ad.php" . urlencode ('$link')
             exit;     

         }
     }

can anyone see a problem with this?

[arkismad]

well this code does as it should

   if (isset($_GET['l'])) {          

         $link = $db->execute("SELECT * FROM links WHERE linkcode='{$_GET['l']}'");
         if (mysql_num_rows($link)>0) {
             $link = mysql_fetch_array($link, MYSQL_ASSOC);             

             $db->execute("UPDATE links SET visits=" . ++$link['visits'] . " WHERE linkcode='{$_GET['l']}'") ;
             header("Location: http://www.urlrewarder.com/ad.php"); 
             exit;     

         }
     }

what i am trying to do is when it redirects to send the variable $link onto the page ad.php so i can use it further

[NogDog]

Probably the simplest thing would be to save it as session data, assuming the redirect is to the same domain?

Otherwise, if you need to send all the query result fields:

header("Location: http://www.urlrewarder.com/ad.php?link=" . urlencode (serialize($link));

Then on the ad.php page:

$link = unserialize($_GET['link']);

[arkismad]

could you show me what you mean by session data, i am very gratefull for your help so far

[Weedpacket]

See the [man]session[/man] reference.