|
|
Posted by Miguel Cruz on 08/07/06 06:17
"mantrid" <ian.dandav@virgin.net> wrote:
> Im having problems working out why the following code does not work. I dont
> think its the sql as the error occurs on the first update which ever one is
> put there ($q1 or $q2). Ive swapped then around to test this.
> Help greatly appreciated
>
> The error is Unknown column 'A' in 'field list'
> but there is no field 'A'. im thinking that the 'A' may be first letter of
> the word Array, as if it is using this as a variable, but i dont know how.
>
>
> $commindexarray=$_POST['commindex'];
> $catarray=addslashes($_POST['category']);
> $commarray=addslashes($_POST['comm']);
> $availarray=$_POST['avail'];
> $delarray=$_POST['del'];
>
> if($delarray==''){
> for ($i = 0; $i < count($commindexarray); $i++){
> $currtime=date('YmdHis');
> $q2 ="UPDATE usercomms SET typeid={$catarray[$i]} WHERE userid=
> ".$_SESSION['userid']." AND commindex ={$commindexarray[$i]}";//
> $updateusercomm = mysql_query($q2) or die('<br><span
> class=RedWarning>Sorry, there was a problem updating. Try
> again.<br><br>Error - 1 '.count($delarray). mysql_error().'</span>');
> $q="UPDATE comments SET typeid={$catarray[$i]}, comment='{$commarray[$i]}',
> available={$availarray[$i]}, timestp='$currtime', globalavail=0 WHERE
> commindex={$commindexarray[$i]}";//
> $updatecomm = mysql_query($q) or die('<br><span class=RedWarning>Sorry,
> there was a problem updating some records. Try again.<br><br>Error - 2
> '.count($delarray).mysql_error().'</span>');
>
> }
> }
PHP is not Perl. If you call addslashes() on an array, it treats it as a
string, which means it has the value "Array". Then, when $i == 0 and you
look at $catarray[$i], you get the first character, which is 'A'.
You need to iterate through your arrays and call addslashes on each
element individually. The most efficient way would be to move the
addslashes (or perhaps mysql_real_escape_string instead) inside the $i
for loop that you already have:
for ($i = 0; $i < count($commindexarray); $i++)
{
$cat = intval($_POST['category'][$i]);
$comm = mysql_real_escape_string($_POST['comm'][$i]);
$q = "update usercomms
set type = {$cat},
comm = {$comm}
where whatever";
}
miguel
--
Photos from 40 countries on 5 continents: http://travel.u.nu
Latest photos: Malaysia; Thailand; Singapore; Spain; Morocco
Airports of the world: http://airport.u.nu
[Back to original message]
|