[SOLUCIONADO] UPDATEs vacios

sven_uri · #1 (**permalink**) 29/03/2015, 09:39

Buenas tardes a todos,

Soy novato con php y hasta ahora me he podido apañar yo solo pero he llegado en un punto que no veo que estoy haciendo mal. Puede que vosotros veáis lo que yo no he conseguido ver aun, ya me diréis.

Problema: Tengo un formulario que a través de POO me hace un update a una tabla. Resulta que realmente el update lo está haciendo pero sustituyendo todos los valores que le paso por formulario como vacíos. De manera que parece que realmente sí que está haciendo el update pero no como debería. No sé si es el orden en que tengo situadas las instrucciones.

Nota: Recupera correctamente los valores del producto que se selecciona. El problema solo esta cuando se hace el update.
Nota 2: No devuelve ningún error.

A mi parecer, creo que en algún momento se pierden los valores, pero no entiendo el por qué.

Gracias por vuestro tiempo : )

Código:

Código PHP:

Ver original<?php
$db = ASDatabase::getInstance();
 
    // Variable que envio para entrar en el producto pertinente
    
    $idProd= $_POST['form_idProd'];
 
$result = $db->select("SELECT intContador, strTipo FROM `tblcategorias` ORDER BY tblcategorias.strTipo ASC");
$editarProd = $db->select(
             "SELECT * FROM `tblprod` WHERE `intContador` = :id",
              array ("id" => $idProd)
          );
 
 
foreach($editarProd as $InfoProd) {   
    $id= $InfoProd['intContador'];
    $nom = $InfoProd['strProd']; 
    $tipus = $InfoProd['idTipo'];
    $url = $InfoProd['strUrl'];
    $fort = $InfoProd['intFuerte'];
    $imatge = $InfoProd['strImagen'];
    $desc = $InfoProd['strDescripcion'];
    $fecha = $InfoProd['fchFecha'];   
}
 
 
include 'templates/menu-izq-admin.php';
?>
 
              <div class="span9 profile-details-wrapper">
                    <!-- main content here -->   
<!------------------------------------ INICIO FORMULARIO -------------------------------------------->                
                    <form class="form-horizontal" id="form-editar" name="form-editar" action="prod-lista.php" method="post">
                        <fieldset>
                        
                        <!-- Form Name -->
                        <legend><?php echo "EDITAR: "; ?></legend>
                        
                        <!-- INICIO NOMBRE -->
                        <div class="control-group form-group">
                          <label class="control-label col-lg-4" for="nom">
                            <?php echo "Nombre"; ?>
                          </label>
                          <div class="controls col-lg-8">
                            <input id="new_nom" name="new_nom" type="text" placeholder="Introduce el nombre..." class="input-xlarge form-control" value="<?php echo $nom ?>">
                            
                          </div>
                        </div>
                        <!-- FIN NOMBRE -->
                         <!-- INICIO TIPO  -->
                        <div class="control-group form-group">
                          <label class="control-label col-lg-4" for="tipo">
                            <?php echo "Qué es?"; ?>
                          </label>
                          <div class="controls col-lg-8">
                           <select id="new_tipo" name="new_tipo" class="form-control" style="width: 100%;">
                                <option value="1" <?php if (!(strcmp(1, htmlentities($tipus, ENT_COMPAT, 'utf-8')))) {echo "SELECTED";} ?>>Servicio</option>
                                <option value="2" <?php if (!(strcmp(2, htmlentities($tipus, ENT_COMPAT, 'utf-8')))) {echo "SELECTED";} ?>>Producto</option>
                                <option value="3" <?php if (!(strcmp(3, htmlentities($tipus, ENT_COMPAT, 'utf-8')))) {echo "SELECTED";} ?>>Nada</option>
                           </select>
                          </div>
                        </div>
                        <!-- FIN TIPO  -->
                        <!-- URL YOUTUBE --> 
                        <div class="control-group form-group">
                          <label class="control-label col-lg-4" for="url">
                            <?php echo "URL de Youtube"; ?>
                          </label>
                          <div class="controls col-lg-8">
                            <input id="new_url" name="new_url" type="text" placeholder="Introduce la url del vídeo" class="input-xlarge form-control" value="<?php echo $url ?>">
                            
                          </div>
                        </div>
                        <!-- FIN URL YOUTUBE -->
                        <!-- INICIO FUERTE-->
                        <div class="control-group form-group">
                          <label class="control-label col-lg-4" for="fuerte">
                            <?php echo "Fuerte"; ?>
                          </label>
                          <div class="controls col-lg-8">
                           <select id="new_fuerte" name="new_fuerte" class="form-control" style="width: 100%;">
                                <option value="0" <?php if (!(strcmp(0, htmlentities($fort, ENT_COMPAT, 'utf-8')))) {echo "SELECTED";} ?>>0</option>
                                <option value="1" <?php if (!(strcmp(1, htmlentities($fort, ENT_COMPAT, 'utf-8')))) {echo "SELECTED";} ?>>1</option>
                                <option value="2" <?php if (!(strcmp(2, htmlentities($fort, ENT_COMPAT, 'utf-8')))) {echo "SELECTED";} ?>>2</option>                  
                           </select>
                          </div>
                        </div>
                        <!-- FIN FUERTE-->
                        <!-- INICIO IMAGEN -->                      
                        <div class="control-group form-group">
                          <label class="control-label col-lg-4" for="imagen">
                            <?php echo "Imagen"; ?>
                          </label>
                          <div class="controls col-lg-8">
                            <input id="new_imagen" name="new_imagen" type="text" placeholder="Selecciona una imagen" class="input-xlarge form-control" value="<?php echo $imatge ?>">  
                          </div>
                        </div>
                        <!-- FIN IMAGEN -->
                        <!-- DESCRIPCIÓN --> 
                        <div class="control-group form-group">
                          <label class="control-label col-lg-4" for="desc">
                            <?php echo "Descripción"; ?>
                          </label>
                          <div class="controls col-lg-8">
                          <textarea id="new_desc" name="new_desc"><?php echo $desc ?></textarea>                           
                          </div>
                        </div>
                        <!-- FIN DESCRIPCIÓN -->
                        <!-- Button -->
                        <div class="control-group form-group">
                          <label class="control-label col-lg-4" for="subir"></label>
                          <div class="controls col-lg-8">
                            <button id="subir" name="subir" class="btn btn-primary" type="submit">
                              <?php echo "Actualizar"; ?>
                            </button>
                            <a href="prod-lista.php">
                            <button id="atras" name="atras" class="btn btn-primary" type="button" style="background-color:red">
                              <?php echo "Volver atrás"; ?>
                            </button>
                          </a>                          </div>
                        </div>
                        <input type="hidden" name="MM_update" value="form-editar">
                        </fieldset>
                    </form>
<!------------------------------------ FIN FORMULARIO  -------------------------------------------->         
<?php
if ((isset($_POST["MM_update"])) && ($_POST["MM_update"] == "form-editar"))
{
$new_nom = $_POST['new_nom'];
$new_tipo = $_POST['new_tipo'];
$new_url = $_POST['new_url'];
$new_fuerte = $_POST['new_fuerte'];
$new_imagen = $_POST['new_imagen'];
$new_descripcion = $_POST['new_desc'];
 
 
$db->update(
     'tblprod',
     array ( "strNom" => $new_nom,
             "idTipo" => $new_tipo,
             "strUrl" => $new_url,
             "intFuerte" => $new_fuerte,
             "strImagen" => $new_imagen,
             "strDescripcion" => $new_descripcion ),
     "intContador = :id",
     array ("id" => $idProd)
);
}
?>

gnzsoloyo · #2 (**permalink**) 29/03/2015, 09:52

La razón del problema la estas dando tu mismo, sólo que debes analizar con precisiónb lo que sucede...

Cita:

Tengo un formulario que a través de POO me hace un update a una tabla.

Los formularios no realizan UPDATEs a las tablas de una BBDD. Se usan para enviar la información al script que lo realiza.

Cita:

Resulta que realmente el update lo está haciendo pero sustituyendo todos los valores que le paso por formulario como vacíos.

No confundas: El UPDATE se está ejecutando, pero lo que se ejecuta es la sentnecia SQL. Ahora bien si está reemplazando todo por vacíos, la razón sólo puede ser una: Los datos no están llegando al scrpt que crea la sentencia y la ejecuta.

Cita:

De manera que parece que realmente sí que está haciendo el update pero no como debería.

El UPDATE se está haciendo como debe. Lo que no estás haciendo en PHP es validar que los datos realmente hayan llegado antes de crear el UPDATE.

Verifica los datos que llegan. Esa parte del script parece que no la estás posteando...

sven_uri · #3 (**permalink**) 29/03/2015, 12:27

Buenas gnzsoloyo,

He revisado lo que me has comentado y los valores se envían correctamente. A la vez, he revisado que el id tenga valor (para que se cumpla la condición del where) y lo tiene.

Lo único que se me ocurre es que en mi sentencia del update ese $id no filtre bien. Para salir de dudas, he utilizado un id existente y se lo he marcado yo directamente (ej: 39) y no hace el update tampoco.

¿Alguna idea de dónde podría mirar?

gnzsoloyo · #4 (**permalink**) 29/03/2015, 12:53

Dices que se envían correctamente, pero ¿se reciben correctamente?
No es lo mismo...

Postea solamente el script que recibe los datos, y donde se crea la query. Lo que si puedo asegurarte es que no estas validando la toma de datos.

Ten en cuenta que si no los inserta y pone vacío, es que el dato no existe en el momento de construir el UPDATE. No hay otra opción.

sven_uri · #5 (**permalink**) 29/03/2015, 14:25

Los valores los recibo a partir de:

Código PHP:

Ver original//Esta variable me viene desde otra página. Lo único que hace es enviar el ID del // producto que quiero ver. He comprobado que llega con valor.
 
$idProd = $_POST['form_idProd'];
 
$editarProd = $db->select(
             "SELECT * FROM `tblprod` WHERE `intContador` = :id",
              array ("id" => $idProd)
          );
        
 
foreach($editarProd as $InfoProd) {   
    $id= $InfoProd['intContador'];
    $nom = $InfoProd['strCoctel']; 
    $tipus = $InfoProd['idTipo'];
    $url = $InfoProd['strUrl'];
    $fort = $InfoProd['intFuerte'];
    $imatge = $InfoProd['strImagen'];
    $desc = $InfoProd['strDescripcion'];
}

Más tarde hago el update de la siguiente forma:

Código PHP:

Ver originalif ((isset($_POST["MM_update"])) && ($_POST["MM_update"] == "form-editar"))
{
$new_nom = $_POST['new_nom'];
$new_tipo = $_POST['new_tipo'];
$new_url = $_POST['new_url'];
$new_fuerte = $_POST['new_fuerte'];
$new_imagen = $_POST['new_imagen'];
$new_descripcion = $_POST['new_desc'];
 
 
$db->update(
     'tblproducto',
     array ( "strProd" => $new_nom,
             "idTipo" => $new_tipo,
             "strUrl" => $new_url,
             "intFuerte" => $new_fuerte,
             "strImagen" => $new_imagen,
             "strDescripcion" => $new_descripcion ),
     "intContador = :id",
     array ("id" => $idProd)
     
     
);

Nota: Es curioso porque cuando hago el submit se envía todo lo que he escrito pero no llega a hacer el update. Creo saber por dónde van los problemas pero no veo por dónde seguir.

Gracias por tu tiempo gnzsoloyo : )

gnzsoloyo · #6 (**permalink**) 29/03/2015, 14:59

1) ¿En qué momento verificas que estos $_POST tengan realmente datos:

Código PHP:

  $new_nom = $_POST['new_nom'];
$new_tipo = $_POST['new_tipo'];
$new_url = $_POST['new_url'];
$new_fuerte = $_POST['new_fuerte'];
$new_imagen = $_POST['new_imagen'];
$new_descripcion = $_POST['new_desc'];

En ninguna parte. Tu estás asumiendo que esos valores de $_POST están llegando con datos, pero no lo verificas.

2)¿Dónde está el código de ese update()?:

Código PHP:

  $db->update(
     'tblproducto',
     array ( "strProd" => $new_nom,
             "idTipo" => $new_tipo,
             "strUrl" => $new_url,
             "intFuerte" => $new_fuerte,
             "strImagen" => $new_imagen,
             "strDescripcion" => $new_descripcion ),
     "intContador = :id",
     array ("id" => $idProd)

Voy a asumir que se trata de algún recurso como PrestaShop (http://doc.prestashop.com/display/PS...s-update%28%29)..
¿Es eso lo que usas?

sven_uri · #7 (**permalink**) 30/03/2015, 06:22

Buenos días gnzsoloyo,

He modificado un poco el código quedando de la siguiente forma:

Código PHP:

Ver original<?php
 
 
if(isset($_POST['boton']))
{
    
    //$new_nom = $_POST['new_nom'];
    //$new_tipo = $_POST['new_tipo'];
    //$new_url = $_POST['new_url'];
    //$new_fuerte = $_POST['new_fuerte'];
    //$new_imagen = $_POST['new_imagen'];
    //$new_descripcion = $_POST['new_desc'];
    
    if($_POST['new_nom'] == '')
    {
        echo "Error: No existe el nombre";  
    }
    else
    {
        $new_nom= $_POST['new_nom'];
    }
    if($_POST['new_tipo'] == '')
    {
        echo "Error: No existe el tipo";    
    }
    else
    {
        $new_tipo= $_POST['new_tipo'];
    }
    if($_POST['new_url'] == '')
    {
        echo "Error: No existe la URL"; 
    }
    else
    {
        $new_url= $_POST['new_url'];
    }
    if($_POST['new_fuerte'] == '')
    {
        echo "Error: No existe";    
    }
    else
    {
        $new_fuerte= $_POST['new_fuerte'];
    }
    if($_POST['new_imagen'] == '')
    {
        echo "Error: No existe la imagen";  
    }
    else
    {
        $new_imagen= $_POST['new_imagen'];
    }
    if($_POST['new_desc'] == '')
    {
        echo "Error: No existe la descripción"; 
    }
    else
    {
        $new_descripcion= $_POST['new_desc'];
    }
}
?>

Sobre la segunda pregunta, compre un script solucionando estos temas para hacérmelo a medida y a la vez seguir practicando. Pero no es ningún Gestor de Contenidos como prestashop.

Lo saqué de aquí: http://mstojanovic.net/as/documentat...tabase_queries

Me he leído la poca documentación que ponen a tu disposición pero no consigo sacarlo.

He creado varias páginas de prueba y los valores parece que se imprimen bien. Pero sigo sin ver por qué me sustituye mis campos por campos vacíos.

Saludos,

gnzsoloyo · #8 (**permalink**) 30/03/2015, 07:13

Y, seré curioso.... ¿qué resultado te dio la modificación?
La probaste?

COmo nota amdicional, ten en cuenta que cada una de esas validaciones debería al mismo tiempo impedir el update en caso de no cumplir.
¿Eso lo estás haciendo ahora?

Fuera de eso, según la decripcion de tu link, lo que hace es esto:

Cita:

UPDATE

Update method needs 4 parameters.

First one is database table that should be updated.

Second one is array where keys are names of columns that should be updated and values are new values for that column.

Third parameter is SQL WHERE query part. Remember that you need to use sql parameters for every variable that you want to pass to the query to prevent SQL Injection.

And fourth parameter is bind array with key => value pair for every sql parameter you have added into sql query.

So, if you want to update user's password, and you have $userId for that user, you can do it like this:

Código PHP:

Ver original$db->update(
    'as_users',
    array ( "password" => $newPassword ),
    "user_id = :id",
    array( "id" => $user_id )
    );

This is actually converted to

Código PHP:

Ver original"UPDATE `as_users` SET `password` = '$newPassword' WHERE `user_id` = '$user_id'"

but we use PDO prepared statements to prevent SQL injection.

En ese sentido, el metodo construye la query que ejecutará con el array, lo que en definitiva es interesante. Pero lo que se hace evidente es que no contiene una rutina devalidación interna de los parametroz que se pasa, sino que lo crea con lo que reciba.
Esto quiere decir que, como te indiqué, si no validas lo que entra, simplemente te pondrá valores vacíos en los campos...

sven_uri · #9 (**permalink**) 31/03/2015, 03:38

Solucionado!

Me fije en lo que me habías comentado gnzsoloyo, efectivamente era que no recibía bien el ID al hacer la transición desde el formulario a la página con la lista de productos.

Creía estar controlándolo bien pero no, siento que mi cabezonería te haya hecho perder tiempo, aun así gracias por insistir y por supuesto, por tu tiempo.

Me he dado cuenta haciéndolo sin PDO y haciéndolo con instrucciones mysqli paso a paso como las hormigas jejeje.

Ya me funciona tanto en PDO como con mysqli.

Pregunta de curiosidad: Por temas de seguridad, entre PDO e instrucciones mysqli preparadas, ¿cuál es más segura?

¡Saludos!