Posted by Petr Vileta on 09/11/06 14:48

Jeff Gardner wrote:
> One wrote:
>> Jeff Gardner wrote:
>>> Greetings:
>>>
>>> I have an UPDATE query (php 5.1.6/mysql 5.0.24a on apache 2.2) that
>>> appears to execute with no errors (php,mysql, or apache) but the
>>> data in the "UPDATED" table doesn't change. I've checked
>>> privileges for connecting account and that isn't an issue. Maybe
>>> it's my query: $editO = "
>>> UPDATE organization
>>> SET
>>> orgname = '$name',
>>> office = '$off',
>>> address1 = '$add1',
>>> address2 = '$add2',
>>> city = '$city',
>>> state = '$state',
>>> country = '$country',
>>> zip = '$ZIP',
>>> phone1 = '$phone1',
>>> phone2 = '$phone2',
>>> fax = '$fax',
>>> email = '$email',
>>> website = '$website',
>>> notes = '$notes',
>>> lasteditdate = '$lastedit',
>>> publish = '$pub',
>>> updatedby = '$uname'
>>> WHERE org_id = '$orgid'";
>>>
>>> include_once 'dsn.php';
>>> $res=mysql_query($editO)
>>> or die('Query Failed: ' . mysql_error());
>>>
>>> I've shuffled it around to get rid of whitespace to no effect. Any
>>> advice is appreciated.
>>
>> Can you echo out the statment being processed and post it as well ?
>>
> It's from a form that is pre-populated with existing data:
>
> $org = trim($_POST['name']);
> $off = trim($_POST['off']);
> $ad1 = trim($_POST['add1']);
> $ad2 = trim($_POST['add2']);
> $city = trim($_POST['city']);
> $state = trim($_POST['state']);
> $country = trim($_POST['country']);
> $zip = trim($_POST['ZIP']);
> $phone1 = trim($_POST['phone1']);
> $phone2 = trim($_POST['phone2']);
> $fax = trim($_POST['fax']);
> $email = trim($_POST['email']);
> $website = trim($_POST['website']);
> $cleanNotes = htmlspecialchars($_POST['notes']);
> $notes = trim($cleanNotes);
> $lastedit = date ('D \t\h\e jS \of M Y \@ h:i:s a T');
> $pub = trim($_POST['pub']);
Isn't some "must be escaped" character in some $_POST[] field? Some stupid
characters are:
" ' ; \ @

--
Petr Vileta, Czech republic
(My server rejects all messages from Yahoo and Hotmail. Send me your mail
from another non-spammer site please.)

[Back to original message]