update modifica todos los valores la tabla

[towi62]

aqui cuando no relampaguea, llovisna o llueve.
Ahora traigo otro problema, ahora lo que me falta es arreglar el update para que nada mas tome el valor que le corresponde y no modifique a los demas.

Este es el update que hago:



mysql_select_db ("base_datos",$conexion);//selecciono la base de datos y hago el update.
$sql = "UPDATE proveedores SET proveedor='".$_GET["proveedor"]."',calificacion='".$_GET["calificacion"]."'";
$respuesta= mysql_query($sql,$conexion) or die ("Problema con query");;

echo "Se han modificado los datos del proveedor";

[xIamAlex]

Te falta especificar la condición (WHERE)

Código PHP:

  mysql_select_db ("base_datos",$conexion);//selecciono la base de datos y hago el update. 
$sql = "UPDATE proveedores SET proveedor='".$_GET["proveedor"]."',calificacion='".$_GET["calificacion"]."' WHERE proveedor = 'Nombre_de_tu_proveedor'"; // modifica esta especificación 
$respuesta= mysql_query($sql,$conexion) or die ("Problema con query");; 
 
echo "Se han modificado los datos del proveedor";  
   

[kronoselft]

Mira en primer lugar, para que un UPDATE sea realmente funcional debes permitir que recorra todos los campos de la tabla a la cual deseas hacer la actualización porque segun sea necesario se actualizara el campo necesario, en segundo debes eliminar cualquier entrada de codigo malicioso, escapando caracteres especiales, una vez dicho lo anterior te comparto un ejemplo:

Código PHP:

//Creamos una funcion para escapar caracteres especiales evitando inyecciones
//public function comillas_inteligentes($valor) para cuando lo añades dentro de una clase y esta herede, pero para el caso suponemos que se hace desde la misma hoja php, por lo que solo le declaramos como function, ok...
function comillas_inteligentes($valor)
    {
        // Retirar las barras
        if (get_magic_quotes_gpc())
        {
            $valor = stripslashes($valor);
        }
    
        // Colocar comillas si no es entero
        if (!is_numeric($valor))
        {
            $valor = "'" . mysql_real_escape_string($valor) . "'";
        }
        return $valor;
    }
 
//Continuamos creando la consulta (UPDATE)
//Generando el update
$actualizar=sprintf("UPDATE proveedores SET proveedor=%s, calificacion=%s WHERE n_proveedor=%s)", 
                comillas_inteligentes($_POST["proveedor"]),
                comillas_inteligentes($_POST["calificacion"]),
                comillas_inteligentes($_POST["n_proveedor"]));
                    //Para que añada todos los campos en la tabla usuarios
                mysql_query($actualizar);
 

Como te habras fijado protejo el envio de mis datos y los filtro mediante la function comillas_inteligentes, bueno eso es solo un caso por ahi, hay varias formas de hacer lo mismo cada quien lo adapta segun le sirva, jejje, acto seguido haces la consulta y le pasas como parametro la function creada que filtrara todos los datos, para luego hacer el update donde sea necesario, espero te sirva, y cualquier cosa, no dudes en preguntar, que estamos para ayudar...

[gildus]

Holas,

Es bueno lo que menciona @kronoselft. Tambien deberias optar con librerias mas seguras que te ayudan mucho a ejecutar tu consulta en php:

http://www.php.net/manual/es/mysqli.prepare.php
http://www.php.net/manual/es/pdo.prepare.php

Saludos

[towi62]

Iniciado por kronoselft

Mira en primer lugar, para que un UPDATE sea realmente funcional debes permitir que recorra todos los campos de la tabla a la cual deseas hacer la actualización porque segun sea necesario se actualizara el campo necesario, en segundo debes eliminar cualquier entrada de codigo malicioso, escapando caracteres especiales, una vez dicho lo anterior te comparto un ejemplo:

Código PHP:

//Creamos una funcion para escapar caracteres especiales evitando inyecciones
//public function comillas_inteligentes($valor) para cuando lo añades dentro de una clase y esta herede, pero para el caso suponemos que se hace desde la misma hoja php, por lo que solo le declaramos como function, ok...
function comillas_inteligentes($valor)
    {
        // Retirar las barras
        if (get_magic_quotes_gpc())
        {
            $valor = stripslashes($valor);
        }
    
        // Colocar comillas si no es entero
        if (!is_numeric($valor))
        {
            $valor = "'" . mysql_real_escape_string($valor) . "'";
        }
        return $valor;
    }
 
//Continuamos creando la consulta (UPDATE)
//Generando el update
$actualizar=sprintf("UPDATE proveedores SET proveedor=%s, calificacion=%s WHERE n_proveedor=%s)", 
                comillas_inteligentes($_POST["proveedor"]),
                comillas_inteligentes($_POST["calificacion"]),
                comillas_inteligentes($_POST["n_proveedor"]));
                    //Para que añada todos los campos en la tabla usuarios
                mysql_query($actualizar);
 

Como te habras fijado protejo el envio de mis datos y los filtro mediante la function comillas_inteligentes, bueno eso es solo un caso por ahi, hay varias formas de hacer lo mismo cada quien lo adapta segun le sirva, jejje, acto seguido haces la consulta y le pasas como parametro la function creada que filtrara todos los datos, para luego hacer el update donde sea necesario, espero te sirva, y cualquier cosa, no dudes en preguntar, que estamos para ayudar...

Muchas gracias amigo, pero no me funciono la funcion

[towi62]

Nomas me dice que se ha modificado, pero no en mi tabla no hace ningun cambio

[megias]

Hola amigo
aqui falta la sentencia WHERE
$sql = "UPDATE proveedores SET proveedor='".$_GET["proveedor"]."',calificacion='".$_GET["calificacion"]."' WHERE (la condicion que quieras poner)";

pero para mas seguridad yo te recomiendo varias cosas

1 si siempre que hagas el UPDATE sera para modificar un solo registro de la tabla te recomiendo que alfinal de la consulta pongas LIMIT 1

2 Intenta evitar siempre el pasar directamente los datos utilizando variables globales. puesto que esta informacion puede ser manipulada por el usuario

por lo tanto te recomendaria hacer esto

$proveedor = htmlspecialchars($_GET['proveedor']);
$calificacion = htmlspecialchars($_GET['calificacion']);

entonces tu consulta quedaria asi


$sql = "UPDATE `proveedores` SET `proveedor` = $proveedor, `calificacion` = $calificacion WHERE `idproveedor` = $proveedorModificar LIMIT 1";

[gildus]

Mejor es ver cual es el codigo que esta usando, porque me imagino que siguio los pasos que se mencionaron, pero no se muestra que codigo en si esta usando, para asi ver cual es el problema o error.

Saludos

[towi62]

Iniciado por gildus

Mejor es ver cual es el codigo que esta usando, porque me imagino que siguio los pasos que se mencionaron, pero no se muestra que codigo en si esta usando, para asi ver cual es el problema o error.

Saludos

ya solucione el problema gracias

[megias]

lo solucionaste o te lo solucionamos?