[SOLVED] Need help with a query

[Cardale]

I have what feels like a stupid questions, but it is starting to irritate me since I can't figure it out.  Can some one please point me in the right direction or tell me. I am getting a error with the delete query.  What could be the problem?

 

		$sql_1 = "SELECT * FROM sessions";

	$query_1 = $DB->query($sql_1) or die("c dont work");
	while($row = $DB->fetch_array($query_1)){
		$ip=$row["ip"];
  			$usertime=$row["time"];

		$user_time_calc = strtotime($usertime);

		if($timeout_calc >= $user_time_calc){ 
			$sql_2 = "DELETE FROM sessions WHERE ip=$ip";

			$DB->query($sql_2) or die("a dont work");
		}
  		}

[haku]

My magic ball tells me that the error is "you have screwed up somewhere".

 

Am I close?

[megaresp]

$sql_2 = "DELETE FROM sessions WHERE ip=$ip";

 

Hi Cardale

 

Try adding quotes around the variable $ip in the query, as follows...

 

$sql_2 = "DELETE FROM sessions WHERE ip='$ip'";

 

NOTE: I haven't tested this. My reasoning is the IP address is a string (to MySQL), so must appear in quotes within the query. For example, MySQL would fail name=Bob but pass name='Bob'

[Cardale]

$sql_2 = "DELETE FROM sessions WHERE ip=$ip";

 

Hi Cardale

 

Try adding quotes around the variable $ip in the query, as follows...

 

$sql_2 = "DELETE FROM sessions WHERE ip='$ip'";

 

NOTE: I haven't tested this. My reasoning is the IP address is a string (to MySQL), so must appear in quotes within the query. For example, MySQL would fail name=Bob but pass name='Bob'

 

haha Something tells me your magic ball was bought from the dollar store.

 

That was it man.  Good eye.