You have an error in your SQL syntax

[Dundee]

Hola llevo todo el día con un formulario que llena un array y lo pasa a una función ,ahora que parece que he logrado pasarlo tengo otro problema.


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 '1' at line 1

Ese es el error que me tira esta consulta:

Código:

function update_articles_pedido($article) {

//return count($articles);
  $conn = db_connect();

for ($i=0;$i<count($article);$i++) {

 	$sSQL =mysql_query ( "UPDATE  pedidos_articulos   
             			SET unidades='$article[$i][1]'
			 	WHERE artid='$article[$i][0]' ");
							
	$result=mysql_query($sSQL)or die(mysql_error());
	mysql_free_result($sSQL);						  

	if (!$result)
	return 0;
	
	
	}
	
	return 1;
}

He probado también así pero lo mismo.

Código:

	$sSQL =mysql_query ( "UPDATE  pedidos_articulos   
             		SET unidades='".$article[$i][1]."'
			 WHERE artid='".$article[$i][0]."' " );

Gracias de antemano

[eits]

haz un

Código:

echo $sSQL =mysql_query ( "UPDATE  pedidos_articulos   
             			SET unidades='$article[$i][1]'
			 	WHERE artid='$article[$i][0]' ");

para ver si el query se esta formando correctamente, tambien debes de asegurarte que $article sea un array y que ademas tenga valores.
puedes probar con var_dump($article).
saludos.

[Dundee]

Iniciado por eits

haz un

Código:

echo $sSQL =mysql_query ( "UPDATE  pedidos_articulos   
             			SET unidades='$article[$i][1]'
			 	WHERE artid='$article[$i][0]' ");

para ver si el query se esta formando correctamente, tambien debes de asegurarte que $article sea un array y que ademas tenga valores.
puedes probar con var_dump($article).
saludos.

Si ya lo había comprobado antes haciendo return $article e invocando a la función , de todos modos el var_dump($article) me devuelve esto:

Código:

array(4) { [0]=>  array(2) { [0]=>  string(4) "1376" [1]=>  string(2) "22" } [1]=>  array(2) { [0]=>  string(4) "1383" [1]=>  string(1) "1" } [2]=>  array(2) { [0]=>  string(4) "1377" [1]=>  string(1) "5" } [3]=>  array(2) { [0]=>  string(4) "1374" [1]=>  string(1) "6" } }

También he comprobado que los campos "unidades" y "artid" existen.

Bueno ahora lo he simplificado con foreach en vez de for , pero no funciona el dichoso UPDATE , he comprobado como me decían antes que $article si tiene valores, y lo he hecho de este modo.

Código:

function update_articles_pedido($article,$ref) {
foreach ($article as $row) {
							 
 echo "primero ". $row[0]."<br>";
echo "segundo ". $row[1]."<br>";
							 
}
}

y da un resultado , por eso pasarlo si lo pasa correctamente a la función que hace el UPDATE, pero algo hay que no
he escrito bien y no consigo encontrar la respuesta en ningún sitio.

Un saludo

[eits]

prueba con esto

Código:

foreach($article as $row1){
    foreach($row1 as $row2){
         echo "primero ". $row2[0]."<br>";
         echo "segundo ". $row2[1]."<br>";
    }
}

saludos.

[Dundee]

Iniciado por eits

prueba con esto

Código:

foreach($article as $row1){
    foreach($row1 as $row2){
         echo "primero ". $row2[0]."<br>";
         echo "segundo ". $row2[1]."<br>";
    }
}

saludos.

Bueno eso con todos mis respetos no tiene mucho sentido , Porque no es tridimensional sino bidimensional, además ya he comprobado que el foreach funciona , por eso hago los echos .

Código:

foreach($article as $row){

         echo "primero ". $row[0]."<br>";
         echo "segundo ". $row[1]."<br>";
    }

Y se muestran los resultados. Lo que creo que faya el UPDATE.

2- hago echo al query y me devuelve 1 (osea que está bien formado ¿no? ).

3- el var_dump está más que comprobado , estoy seguro que $article es un array no solo por el var_dump sino también por el ejemplo este que me devuelve TODOS los valores del array bidimensional.

Código:

foreach($article as $row){

         echo "primero ". $row[0]."<br>";
         echo "segundo ". $row[1]."<br>";
    }

Más pruebas....

Si en vez de ese query con el array le pongo valores sigue dando el mismo error de sql


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 '1' at line 1

Código:

 $sSQL =mysql_query ( "UPDATE  pedidos_articulos   
          SET unidades='80'
	WHERE artid='1376' ");

Un saludo y gracias

[eits]

tu arreglo esta de esta forma segun veo por el var_dump

Código PHP:

  $article=array(0 => array(0 => "1376", 1 => "22"),
                1 => array(0 => "1383", 1 => "1"),
                2 => array(0 => "1377", 1 => "5"),
                3 => array(0 => "1374", 1 => "6")
        )  
   

ayer no pude checarlo porque ando con un buen de trabajo, pero si lo hubieras checado te hubieses dado cuenta que no estaba tan perdido solo era cuestion de moverle un poco
esto si funciona

Código PHP:

  foreach($article as $row1){
    foreach($row1 as $row2){
         echo $row2."<br>";
    }
}  
   

tambien tienes esta otra opción

Código PHP:

  foreach($article as $row1=>$k){
         echo "primero ". $k[0]."<br>";
         echo "segundo ". $k[1]."<br>";
}  
   

saludos.

[Dundee]

eits ya lo he solucuionado , era muy simple fíjate: había puesto dos veces
mysql_query() .

Un saludo y gracias por la ayuda

[masterojitos]

plop, jaja en vez de:
$sSQL =mysql_query ( "UPDATE pedidos_articulos SET unidades='$article[$i][1]' WHERE artid='$article[$i][0]' ");
$result=mysql_query($sSQL)or die(mysql_error());
era:
$sSQL = "UPDATE pedidos_articulos SET unidades='$article[$i][1]' WHERE artid='$article[$i][0]'";
$result=mysql_query($sSQL)or die(mysql_error());

[Dundee]

Iniciado por masterojitos

plop, jaja en vez de:
$sSQL =mysql_query ( "UPDATE pedidos_articulos SET unidades='$article[$i][1]' WHERE artid='$article[$i][0]' ");
$result=mysql_query($sSQL)or die(mysql_error());
era:
$sSQL = "UPDATE pedidos_articulos SET unidades='$article[$i][1]' WHERE artid='$article[$i][0]'";
$result=mysql_query($sSQL)or die(mysql_error());

Si ya se que es pa materme jejejeje
un saludo y gracias a tod@s