[SOLUCIONADO] Update que no Actualiza :S

Jose789 · #1 (**permalink**) 31/05/2013, 10:40

Hola pues miren este es el codigo pero no actualiza los nuevos datos que le pongo....

Código:

if(isset($_POST['save'])){
		if(strlen($_POST['mission']) > 32){
			$result = "&iexcl;Tu misi&oacute;n es demasiado larga!";
			$error = "1";
		}
			mysql_query("UPDATE users SET mission = mission = '".FilterText($_POST['mission'])."', mymusik = '".FilterText($_POST['mymusik'])."' WHERE id = '".$my_id."' LIMIT 1") or die(mysql_error());
			$result = "Ajustes actualizados con &eacute;xito.";
		}

y estos son los 2 input

Código:

<input type="text" name="mission" size="32" maxlength="32" value="<?php echo HoloText($user_row['mission']); ?>" id="avatarmotto" />

Código:

<input type="text" name="mymusik" size="32" value="<?php echo FilterText($user_row['mymusik']); ?>">

y este es el que hace el save

Código:

<input type="submit" value="Salvar cambios" name="save" class="submit" />

Alguien puede echarme una mano Gracias :)

gnzsoloyo · #2 (**permalink**) 31/05/2013, 10:43

Extraigamos el SQL:

Código SQL:

Ver originalUPDATE users 
SET mission = mission = '".FilterText($_POST['mission'])."', 
mymusik = '".FilterText($_POST['mymusik'])."' 
WHERE id = '".$my_id."' 
LIMIT 1

¿mission = mission = '".FilterText($_POST['mission'])."'?

¿Y qué se supone que hace eso?
tienes un "mission =" de mas...
Fuera de eso, debes verificar que los valores de las variables existan. Es muy usual que no esté llegando lo que se supone, por errores de código.
Es preferible que las valides antes de usarlas. Por simple precaución.

alexisverano · #3 (**permalink**) 31/05/2013, 10:44

A primera vista creo que tu problema esta aqui...

UPDATE users SET mission = mission = '".FilterText($_POST['mission'])."',.......

usalo de esta forma:

UPDATE users SET mission = '".FilterText($_POST['mission'])."',..........

Saludos,

Jose789 · #4 (**permalink**) 31/05/2013, 10:46

Cierto ya lo quite quedo asi

Código:

mysql_query("UPDATE users SET mission ='".FilterText($_POST['mission'])."', mymusik = '".FilterText($_POST['mymusik'])."' WHERE id = '".$my_id."' LIMIT 1")

Pero sigue sin Hacer Update :S

alexisverano · #5 (**permalink**) 31/05/2013, 10:47

Y donde llenas esta variable?

$my_id

Jose789 · #6 (**permalink**) 31/05/2013, 10:51

Cita:

Iniciado por alexisverano

Y donde llenas esta variable?

$my_id

el my_id se coje del core.php que ya lleva un requiere_once

alexisverano · #7 (**permalink**) 31/05/2013, 10:52

A ver hay cosas que no comprendo de tu linea de codigo...

Código PHP:

Ver originalmysql_query("UPDATE users SET mission ='".FilterText($_POST['mission'])."', mymusik = '".FilterText($_POST['mymusik'])."' WHERE id = '".$my_id."' LIMIT 1")

esto: que es? FilterText

yo normalmente pondria algo asi...

Código PHP:

Ver originalmysql_query("UPDATE users SET mission ='".$_POST['mission']."', mymusik = '".$_POST['mymusik']."' WHERE id = '".$my_id."' LIMIT 1")

tampoco entiendo porque usas LIMIT 1, si cuando tu ejecutas un UPDATE es SOLO actualiza segun los parametros del WHERE

gnzsoloyo · #8 (**permalink**) 31/05/2013, 10:54

¿Te molestaste en validar lo que llega como te dije?

Es es el error más común de todos. Incluso si estás convencido de tener todo OK. tienes que verificarlo..
Hace un par de días uno de los foristas insistía que estaba todo OK, y al final no había hecho en session_start(), por lo que la variable era nula... como yo le dije desde el principio.

Jose789 · #9 (**permalink**) 31/05/2013, 10:55

Cita:

Iniciado por alexisverano

A ver hay cosas que no comprendo de tu linea de codigo...

Código PHP:

Ver originalmysql_query("UPDATE users SET mission ='".FilterText($_POST['mission'])."', mymusik = '".FilterText($_POST['mymusik'])."' WHERE id = '".$my_id."' LIMIT 1")

esto: que es? FilterText

yo normalmente pondria algo asi...

Código PHP:

Ver originalmysql_query("UPDATE users SET mission ='".$_POST['mission']."', mymusik = '".$_POST['mymusik']."' WHERE id = '".$my_id."' LIMIT 1")

tampoco entiendo porque usas LIMIT 1, si cuando tu ejecutas un UPDATE es SOLO actualiza segun los parametros del WHERE

Te explico el filtertext es esto

Código:

function FilterText($str, $advanced=false) {
	if($advanced == true){ return mysql_real_escape_string($str); }
	$str = mysql_real_escape_string(htmlspecialchars($str));
	return $str;
}

Jose789 · #10 (**permalink**) 31/05/2013, 11:01

Cita:

Iniciado por gnzsoloyo

¿Te molestaste en validar lo que llega como te dije?

Es es el error más común de todos. Incluso si estás convencido de tener todo OK. tienes que verificarlo..
Hace un par de días uno de los foristas insistía que estaba todo OK, y al final no había hecho en session_start(), por lo que la variable era nula... como yo le dije desde el principio.

como? no soy muy experto en php... Si no te molestaría podrías explicarme

gnzsoloyo · #11 (**permalink**) 31/05/2013, 11:09

Es medio obvio: Muestra los valores en la pantalla.
Si no muestra nada, es que no hay nada.
Pero para estar más seguro, reemplaza esto:

Código PHP:

Ver originalmysql_query("UPDATE users SET mission = '".FilterText($_POST['mission'])."', mymusik = '".FilterText($_POST['mymusik'])."' WHERE id = '".$my_id."' LIMIT 1") or die(mysql_error());
$result = "Ajustes actualizados con &eacute;xito.";

Por esto:

Código PHP:

Ver original$query = "UPDATE users SET mission = '".FilterText($_POST['mission'])."', mymusik = '".FilterText($_POST['mymusik'])."' WHERE id = '".$my_id."' LIMIT 1";
echo $query;
$rslt = mysql_query($query) or die(mysql_error());
if($rslt)
{$result = "Modificacion realizada exitosamente";}

Luego copia y postea acá lo que muestre en la pantalla el echo.

Jose789 · #12 (**permalink**) 31/05/2013, 11:38

Cita:

Iniciado por gnzsoloyo

Es medio obvio: Muestra los valores en la pantalla.
Si no muestra nada, es que no hay nada.
Pero para estar más seguro, reemplaza esto:

Código PHP:

Ver originalmysql_query("UPDATE users SET mission = '".FilterText($_POST['mission'])."', mymusik = '".FilterText($_POST['mymusik'])."' WHERE id = '".$my_id."' LIMIT 1") or die(mysql_error());
$result = "Ajustes actualizados con &eacute;xito.";

Por esto:

Código PHP:

Ver original$query = "UPDATE users SET mission = '".FilterText($_POST['mission'])."', mymusik = '".FilterText($_POST['mymusik'])."' WHERE id = '".$my_id."' LIMIT 1";
echo $query;
$rslt = mysql_query($query) or die(mysql_error());
if($rslt)
{$result = "Modificacion realizada exitosamente";}

Luego copia y postea acá lo que muestre en la pantalla el echo.

Nada no muestra nada, y tampoco hace el Update... :S te dejo el codigo complento S: por que la verdad no se que puede ser

Código:

<?php

require_once('core.php');
require_once('includes/session.php');


if(isset($_POST['save'])){
		if(strlen($_POST['mission']) > 32){
			$result = "&iexcl;Tu misi&oacute;n es demasiado larga!";
			$error = "1";
		}
			$query = "UPDATE users SET mission = '".FilterText($_POST['mission'])."', mymusik = '".FilterText($_POST['mymusik'])."' WHERE id = '".$my_id."' LIMIT 1";
echo $query;
$rslt = mysql_query($query) or die(mysql_error());
if($rslt)
{$result = "Modificacion realizada exitosamente";}
		}

?>

<?php

if(!empty($result)){
	if($error == "1"){
		echo "<div class='rounded rounded-red'>";
	} else {
		echo "<div class='rounded rounded-green'>";
	}

	echo $result . "<br />
	</div><br />";
}

$user_sql = mysql_query("SELECT * FROM users WHERE id = '".$my_id."'");
$user_row = mysql_fetch_assoc($user_sql);

?>

<h3>Tu estado</h3>

<p>
Tu estado podr&aacute; ser visto por todos, &iexcl;pi&eacute;nsalo bien!
</p>

<p>
<span class="label">Estado:</span>
<input type="text" name="mission" size="32" maxlength="32" value="<?php echo HoloText($user_row['mission']); ?>" id="avatarmotto" />
</p>

<h3>Tu P&aacute;gina</h3>


<h3> M&uacute;sica</h3>
<p>
Escribe un link de youtube con tu canci&oacute;n favorita para escucharlo en la Home.<br><br><b>Nota:</b> El link debe ser diferente, ejemplo:<br><font color="green">http://youtube.com/</font><font color="red">watch?</font><font color="green">v</font><font color="red">=</font><font color="green">nciFjxOYe74</font> = <font color="green">http://youtube.com/v/nciFjxOYe74</font><br><br>
<label>
Link: <input type="text" name="mymusik" size="32" value="<?php echo FilterText($user_row['mymusik']); ?>">
</label></p>

<form>
<input type="submit" value="Salvar cambios" name="save" class="submit" />
</form>

gnzsoloyo · #13 (**permalink**) 31/05/2013, 11:47

Cita:

Nada no muestra nada, y tampoco hace el Update... :

Que no actualice nada, es irrelevante desde el momento en que no muestra nada.
Eso significa simplemente que eso no tiene lo que crees que tiene.

depura el proceso y fíjate por qué se pierden los datos o por qué no se genera la consulta como se supone que debe suceder.

Jose789 · #14 (**permalink**) 31/05/2013, 11:50

Cita:

Iniciado por gnzsoloyo

Que no actualice nada, es irrelevante desde el momento en que no muestra nada.
Eso significa simplemente que eso no tiene lo que crees que tiene.

depura el proceso y fíjate por qué se pierden los datos o por qué no se genera la consulta como se supone que debe suceder.

Mostrar si muestra... muestra lo que yo ya tengo puesto pero no actualiza con el nuevo que yo quiera poner S:_

gnzsoloyo · #15 (**permalink**) 31/05/2013, 12:07

No, no muestra.
Eso debería estar mostrando en pantalla la sintaxis de la consulta, y no lo está haciendo..

De hecho, es muy probable que ni siquiera esté entrando hasta allí.

Para asegurarnos, prueba poner eso así:

Código PHP:

Ver original<?php
 
require_once('core.php');
require_once('includes/session.php');
 
echo $_POST['save'];
echo strlen($_POST['mission']);
echo $_POST['mission'];
 
if(isset($_POST['save'])){
        if(strlen($_POST['mission']) > 32){
            $result = "&iexcl;Tu misi&oacute;n es demasiado larga!";
            $error = "1";
        }
            $query = "UPDATE users SET mission = '".FilterText($_POST['mission'])."', mymusik = '".FilterText($_POST['mymusik'])."' WHERE id = '".$my_id."' LIMIT 1";
echo $query;
$rslt = mysql_query($query) or die(mysql_error());
if($rslt)
{$result = "Modificacion realizada exitosamente";}
        }
 
?>

Cosa que muestro el valor de los componentes que están evaluandose en lso IF.

Jose789 · #16 (**permalink**) 31/05/2013, 12:24

Cita:

Iniciado por gnzsoloyo

No, no muestra.
Eso debería estar mostrando en pantalla la sintaxis de la consulta, y no lo está haciendo..

De hecho, es muy probable que ni siquiera esté entrando hasta allí.

Para asegurarnos, prueba poner eso así:

Código PHP:

Ver original<?php
 
require_once('core.php');
require_once('includes/session.php');
 
echo $_POST['save'];
echo strlen($_POST['mission']);
echo $_POST['mission'];
 
if(isset($_POST['save'])){
        if(strlen($_POST['mission']) > 32){
            $result = "&iexcl;Tu misi&oacute;n es demasiado larga!";
            $error = "1";
        }
            $query = "UPDATE users SET mission = '".FilterText($_POST['mission'])."', mymusik = '".FilterText($_POST['mymusik'])."' WHERE id = '".$my_id."' LIMIT 1";
echo $query;
$rslt = mysql_query($query) or die(mysql_error());
if($rslt)
{$result = "Modificacion realizada exitosamente";}
        }
 
?>

Cosa que muestro el valor de los componentes que están evaluandose en lso IF.

Nada igual no actualiza los nuevos datos y ahora me sale un 0 en la parte de arriba del todo...

gnzsoloyo · #17 (**permalink**) 31/05/2013, 12:32

Cita:

Nada igual no actualiza los nuevos datos y ahora me sale un 0 en la parte de arriba del todo...

Creo que no estás entendiendo lo que te estoy haciendo hacer...

No me importa que no actualice los datos, porque eso no es lo que estamos verificando.
Estamos tratando de determinar por qué no los actualiza, y eso sólo puede suceder porque el parámetro del WHERE no es válido.
ESO es lo que te estoy obligando a descubrir, y lo acabamos de hacer: Ese cero, significa que $_POST['save'] contiene cero, y eso significa FALSE.

¿Se entiende?

Ese UPDATE no actualiza, porque jamás llega a ejecutarse ni crearse.

Tan sencillo como eso.

Verifica cómo está entrando ese dato, porque ese dato no llega e invalida todo.

¿Vas entendiendo?

Jose789 · #18 (**permalink**) 31/05/2013, 12:37

Cita:

Iniciado por gnzsoloyo

Creo que no estás entendiendo lo que te estoy haciendo hacer...

No me importa que no actualice los datos, porque eso no es lo que estamos verificando.
Estamos tratando de determinar por qué no los actualiza, y eso sólo puede suceder porque el parámetro del WHERE no es válido.
ESO es lo que te estoy obligando a descubrir, y lo acabamos de hacer: Ese cero, significa que $_POST['save'] contiene cero, y eso significa FALSE.

¿Se entiende?

Ese UPDATE no actualiza, porque jamás llega a ejecutarse ni crearse.

Tan sencillo como eso.

Verifica cómo está entrando ese dato, porque ese dato no llega e invalida todo.

¿Vas entendiendo?

Que el where es Falso y nunca ejecuta lo de update

gnzsoloyo · #19 (**permalink**) 31/05/2013, 12:40

Exacto.

Verifica el ingreso de ese dato en tu formulario, porque jamás está llegando a ese script.
En alguna parte se pierde.

Jose789 · #20 (**permalink**) 31/05/2013, 12:47

Cita:

Iniciado por gnzsoloyo

Exacto.

Verifica el ingreso de ese dato en tu formulario, porque jamás está llegando a ese script.
En alguna parte se pierde.

Y si ago como un formulario y hago que el action se valla a otro archivo .php donde lleve el update... Entonces iria verdad

gnzsoloyo · #21 (**permalink**) 31/05/2013, 12:56

¿Me puedes explicar, al menos, de dónde sale ese valor?
¿En qué se origina?

Jose789 · #22 (**permalink**) 01/06/2013, 08:00

Cita:

Iniciado por gnzsoloyo

¿Me puedes explicar, al menos, de dónde sale ese valor?
¿En qué se origina?

Que valor el 0

gnzsoloyo · #23 (**permalink**) 01/06/2013, 11:13

A estas alturas creo que ya deberías haberte dado cuenta de que ese formulario no está funcionando como crees. ¿No te parece?
Para mí es más que evidente que si el parámetro "save" en el script está mostrando un valor cero (0), entonces está recibiendo un FALSE, y no ese supuesto TRUE que mencionas...

¿te queda claro?

Lo repito: Ese formulario no está funcionando bien, o como mínimo no envía el valor que tu supones.

Postea el código completo del formulario para verificar por qué no hace lo que se supone que debe hacer.

Jose789 · #24 (**permalink**) 01/06/2013, 14:06

Ya lo resolví solo puse <form method="post"> y al final </form> Y ya va... Pero Gracias por tu colaboración...