Problema con consulta Update

[Fleon]

hola amigos, he estado buscando en el foro un problema similar aunque muchos hilos parecidos pero no encontré lo que ando buscando, tengo lo siguiente:

Código PHP:

function Save2_Article()
{
    require_once(dirname(__FILE__) .'/DbData.php');
    
    $titulo = $_POST['Titulo'];
    $seccion = $_POST['seccion'];
    $estado = $_POST['estado'];
    $autor = $_POST['Autor'];
    $cuerpo = $_POST['elm1'];
    $fecha = date('Y-m-d H:i:s');
    
    $query = "UPDATE articulos SET title = '$titulo', cat_id = '$seccion', st_id = '$estado', autor = '$autor', body = '$cuerpo', editado = '$fecha' WHERE id = ".$_REQUEST['id'];
    
    if (mysql_query($query))
    {
        Print "EL artículo se actuazó correctamente!";
    } else {
        print mysql_error();
    }
}
 

Y este el error que obtengo:

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

ahora, si la consulta la pongo así:

Código PHP:

$query = "UPDATE articulos SET title = $titulo, cat_id = $seccion, st_id = $estado, autor = $autor, body = $cuerpo, editado = $fecha WHERE id = ".$_REQUEST['id'];
 

este es el error que obtengo:

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 'Valle del Bravo, cat_id = 2, st_id = 2, autor = Fernando Leonor, body =

Bienv' at line 1

¿Cual es el problema?

Cabe aclarar que los campos body (contiene html) y editado son del tipo TEXT y DATETIME respectivamente.

Saludos.

[David]

¿Seguro que tienes datos en $_REQUEST['id']?

Haz un echo($query); para ver cómo queda la consulta al final.

P.S.: No uses directamente los datos del formulario en la consulta, usa mysql_real_escape_string() para filtrarlos primero (en caso de que sea texto)

[pilucho]

Esto te puedo ayudar

MIRA AQUI

[Fleon]

Si, $_REQUEST['id'] tiene datos, el echo me muestra los dato correctos:

UPDATE articulos SET title = El Valle del Bravo, cat_id = 2, st_id = 2, autor = Fernando Leonor, body =

Bienvenido a Orbe 74 y a esta sección Coordenada Tribal en donde encontraras información importante sobre destinos turísticos que podrán ser útiles para planear tus fines de semana o vacaciones, ya sea en compañía de la familia o el grupo de amigos.



En esta primera entrega te llevamos a Valle de Bravo:

\"\"

Valle de Bravo se encuentra a 145 kms. aproximadamente de la Ciudad de México y como dato cultural en tiempos prehispánicos Valle de Bravo ostentaba el nombre de “Temascaltepeque” palabra de origen náhuatl que significa “Cerro de los baños de vapor”.
Ya en el camino de entrada a Valle de Bravo podrás ir deleitando tu vista puesto que te encontraras en un sendero enmarcado por árboles de un verde intenso que al instante te hará sentir en armonía con la naturaleza, cuestión que como citadinos hemos dejado de lado.
Cuando te encuentres ubicado en este delicioso lugar de casas blancas con tejados rojos y calles angostas y empedradas, te recomiendo que en caso de llevar automóvil busques estacionamiento, ya resuelta esta situación, lo primero es y como todo buen turista hacer uso de tus piernas y trasladarte a la plaza de la independencia o mejor conocida como el zócalo de Valle de Bravo.

La plaza de la independencia o el zócalo es el mejor lugar para comenzar a conocer este magnífico lugar, empezando por La parroquia de San Francisco de Asís que se encuentra en este primer cuadro, además de que está rodeada por diversos comercios como restaurantes, cafés, tiendas de víveres y posadas. Aquí podrás disfrutar de un momento de relajación y tal vez una buena charla sentado en una de las bancas que rodean el kiosco acompañando el momento con una nieve del sabor de tu elección o alguno de los antojitos mexicanos que se encuentran a la venta alrededor de esta plaza.
, editado = 2010-02-03 21:59:07 WHERE id = 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 'Valle del Bravo, cat_id = 2, st_id = 2, autor = Fernando Leonor, body =

Bienv' at line 1

Saludos.

[David]

Debes usar las comillas, como en el primer ejemplo que pusiste.

Además, fíjate que muestra "WHERE id = ", significa que $_REQUEST['id'] no tiene datos en realidad.

[Fleon]

Iniciado por David

Debes usar las comillas, como en el primer ejemplo que pusiste.

Además, fíjate que muestra "WHERE id = ", significa que $_REQUEST['id'] no tiene datos en realidad.

Gracias David, lo acabo de ver, fíjate id lo saco del link que es así:

http://localhost/orbe/index.php?action=editarticle&id=3

he tratado de obtener el id y no lo logro

en la función lo coloqué así:

Código PHP:

function Save2_Article()
{
    require_once(dirname(__FILE__) .'/DbData.php');
    
    $titulo = $_POST['Titulo'];
    $seccion = $_POST['seccion'];
    $estado = $_POST['estado'];
    $autor = $_POST['Autor'];
    $cuerpo = $_POST['elm1'];
    $fecha = date('Y-m-d H:i:s');
        $id = $_GET['id'];
    
    $query = "UPDATE articulos SET title = '$titulo', cat_id = '$seccion', st_id = '$estado', autor = '$autor', body = '$cuerpo', editado = '$fecha' WHERE id = $id";
    
    if (mysql_query($query))
    {
        Print "EL artículo se actuazó correctamente!";
    } else {
        print $query;
        print mysql_error();
    }
}
 

he probado con $_REQUEST, $_GET y POST y no capto esa variable del link, cabe aclarar que el link está en un achivo diferente al formulario.

Saludos.

[David]

Deberías poder recoger esa variable si está presente en la página que procesa el formulario, ¿index.php es la página que procesa el formulario?

[Fleon]

Iniciado por David

Deberías poder recoger esa variable si está presente en la página que procesa el formulario, ¿index.php es la página que procesa el formulario?

No, no es index.php, el link es redireccionado, el que procesa el formulario es EditArticle.template.php, coloqué una variable en ella así:

$code = $_REQUEST['id'];

y en la función hice esto:

Código PHP:

function Save2_Article()
{
    require_once(dirname(__FILE__) .'/DbData.php');
    
    $titulo = $_POST['Titulo'];
    $seccion = $_POST['seccion'];
    $estado = $_POST['estado'];
    $autor = $_POST['Autor'];
    $cuerpo = $_POST['elm1'];
    $fecha = date('Y-m-d H:i:s');
    $code = $_GET['code'];
    
    $query = "UPDATE articulos SET title = '$titulo', cat_id = '$seccion', st_id = '$estado', autor = '$autor', body = '$cuerpo', editado = '$fecha' WHERE id = '$code'";
    
    if (mysql_query($query))
    {
        Print "EL artículo se actuazó correctamente!";
    } else {
        print mysql_error();
    }
}
 

me dice que se grabó bien, pero en realidad no actualiza nada

[David]

$code no existe en el ámbito de la función, debes usar $GLOBALS['code'] para acceder a la variable. De todos modos, asegúrate de que realmente tiene el valor que esperas.

[Fleon]

Iniciado por David

$code no existe en el ámbito de la función, debes usar $GLOBALS['code'] para acceder a la variable. De todos modos, asegúrate de que realmente tiene el valor que esperas.

A ver, ¿coloco $GLOBALS['code'] en la función o en EditArticle.template.php?, porque he tratado de todas las formas, fíjate, lo último que he hecho es mostrar el id en un textfield y me la muestra correctamente, el text field se llama aID y en la funcion la llamo asi:

Código PHP:

function Save2_Article()
{
    require_once(dirname(__FILE__) .'/DbData.php');
    
    $titulo = $_POST['Titulo'];
    $seccion = $_POST['seccion'];
    $estado = $_POST['estado'];
    $autor = $_POST['Autor'];
    $cuerpo = $_POST['elm1'];
    $fecha = date('Y-m-d H:i:s');
    $no = $_POST['aID'];
    
    $query = "UPDATE articulos SET title = '".$titulo."', cat_id = '".$seccion."', st_id = '".$estado."', autor = '".$autor."', body = '".$cuerpo."', editado = '".$fecha."' WHERE id = '".$no."'";
    
    if (mysql_query($query))
    {
        Print "EL artículo se actuazó correctamente!";
    } else {
        print mysql_error();
    }
}
 

ella me dice que grabó pero sigue sin actualizar, caray, esto me va volver loco

[David]

En lugar de usar el valor de retorno de mysql_query() para saber si el registro fue actualizado debes usar mysql_affected_rows:
http://www.php.net/manual/en/functio...ected-rows.php

Por cierto, no necesitas usar comillas en el id siendo este un campo numérico.

[Fleon]

Iniciado por David

En lugar de usar el valor de retorno de mysql_query() para saber si el registro fue actualizado debes usar mysql_affected_rows:
[url]http://www.php.net/manual/en/function.mysql-affected-rows.php[/url]

Por cierto, no necesitas usar comillas en el id siendo este un campo numérico.

Hola David, si le quito las comillas mira lo que obtengo:

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

[David]

La razón de eso error ya te expliqué. No estás recuperando ningún valor para el ID, y las comilllas no te van a solucionar el problema mientras no tengas el ID que quieres actualizar.

Todavía no entiendo, ¿qué página está procesando el formulario? (me refiero a la página, la URL que se muestra en el navegador, no el archivo en sí que se puede usar mediante include)

[Fleon]

Iniciado por David

La razón de eso error ya te expliqué. No estás recuperando ningún valor para el ID, y las comilllas no te van a solucionar el problema mientras no tengas el ID que quieres actualizar.

Todavía no entiendo, ¿qué página está procesando el formulario? (me refiero a la página, la URL que se muestra en el navegador, no el archivo en sí que se puede usar mediante include)

Tengo dos achivos Functions.php y EditArticle.template.php el formulario está en ésta última, o sea, esa es la que procesa el form, lo último que hice fue colocar el id en un Text Field, para así obviar obtener el valor desde el link, ahora, en Funtionc.php está la función que menciono más arriba, ahora trato de obtener el valor del Text Field que siempre tiene el id correcto, pero desde la funcion no logro capturar el valor del text field que se llama aID si lo demás lo captura, pos no lo entiendo

[GatorV]

Lo que pasa es que si ves estas llamando a index.php, si rediriges a EditArticle.template.php pierdes esos valores, tienes que pasarlos ya sea por GET o por una sesión antes de redirigir.

Saludos.

[Fleon]

Iniciado por GatorV

Lo que pasa es que si ves estas llamando a index.php, si rediriges a EditArticle.template.php pierdes esos valores, tienes que pasarlos ya sea por GET o por una sesión antes de redirigir.

Saludos.

Los valores no se pierden porque todos se cargan en textfields, lo raro es que solamente el Text Field del id no me lo jala si todos los demás sí, eso es lo que me tiene atónito

[Fleon]

Ya pude resolver, gracias al cielo, el asunto quedo así:

Código PHP:

function Save2_Article()
{
    require_once(dirname(__FILE__) .'/DbData.php');
    
    $titulo = $_POST['Titulo'];
    $seccion = $_POST['seccion'];
    $estado = $_POST['estado'];
    $autor = $_POST['Autor'];
    $cuerpo = $_POST['elm1'];
    $fecha = date('Y-m-d H:i:s');
    if (isset($_POST['code'])) $no = (int) $_POST['code'];
    
    $query = "UPDATE articulos SET title = '$titulo', cat_id = '$seccion', st_id = '$estado', autor = '$autor', body = '$cuerpo', editado = '$fecha' WHERE id = $no";
    
    if (mysql_query($query))
    {
        echo $query;
        Print "EL artículo se actuazó correctamente, se afectaron ".mysql_affected_rows()." articulos";
    } else {
        echo $query;
        print mysql_error();
    }
}
 

Gracias a todos por su tiempo y perdonen la molestia.

Saludos.