|
|
Posted by Chris Hope on 07/09/05 01:21
Kimmo Laine wrote:
> "Chris Hope" <blackhole@electrictoolbox.com> wrote in message
> news:dalhce$u6f$1@lust.ihug.co.nz...
>> roN @ work wrote:
>>
>>> roN @ work wrote:
>>>> Hi,
>>>> I got following Code and get Below Error, why? :(
>>>> [Code]
>>>> function reportSpam($email_address_and_relay)
>>>> {
>>>> $connection=mysql_connect(db_server,db_user,db_passwort);
>>>> mysql_select_db(db_name);
>>>>
>>>> $date = date("y-m-d");
>>>> $tmp=explode("*",$email_address_and_relay);
>>>> $address = $tmp[0];
>>>> $relay = $tmp[1];
>>>> $query = "insert into blacklist (`Date`,`Address`, `Relay`) values
>>>> ('$date','$address', $relay)";
>>>>
>>>> mysql_query($query) or die(mysql_error());
>>>> mysql_close($connection);
>>>> echo "reported ".$email_address."<br>\n";
>>>> }
>>>> [/Code]
>>>> [Error]
>>>> You have an error in your SQL syntax. Check the manual that
>>>> corresponds to your MySQL server version for the right syntax to
>>>> use near '.160.97)' at line 2
>>>> [/Error]
>>>> In Variable $email_address_and_relay there is something like
>>>> test@example.com*255.127.53.49 I'd like to explode that and save
>>>> the e-mail address and the relay IP in diffrent cols in the same
>>>> row. shouldn't it work? :(
>>>> table blacklist looks like this:
>>>> Date - date
>>>> Address - varchar(100)
>>>> Relay - varchar(50)
>>>> do I have to convert $relay to an integer before I'm gonna save it
>>>> to the DB? Yes? How?
>>>
>>> Sorry, I meant convert it to a String for sure....
>>
>> Read my other post. You don't need to convert it to a string as it
>> should already be a string; you just need to put quotes around the
>> value ie
>>
>> $query = "insert into blacklist (`Date`,`Address`, `Relay`) values
>> ('$date','$address', $'relay')";
>> ^ ^
>
> And I'm sure Chris meant '$relay' instead of $'relay', but I thought
> I'd mention it...
Yep :)
--
Chris Hope | www.electrictoolbox.com | www.linuxcdmall.com
Navigation:
[Reply to this message]
|