Problema con update de perfil!

den_22 · #1 (**permalink**) 12/03/2010, 17:41

Hola ! estoy con un problema les comento, estoy haciendo un perfil de un usuario, en el cual hay un apartado de modificar perfil. Obtengo los datos de la bd sin ningun problema, pero hay otros que deben ser modificados y posteriormente guardados.

Este es el problema, que al apretar el "editar perfil" los datos no se actualizan en la bd.

El codigo del update_cuenta.php es asi:

Código PHP:

Ver original<?
 
include('connect.php'); //incluimos el connect.php que contiene los datos de la conexi&oacute;n a la db
include('function.php');
include('getdata.php')
 
$firstname=$_POST['firstname']; 
$lastname=$_POST['lastname'];
 
 
//introduccimos la modificacion hecha por el usuario
 
 
 
mysql_query("
UPDATE prueba
SET firstname='$firstname',
 lastname='$lastname',
 
WHERE usuario='$_SESSION[usuario]' ");
echo '<div class="Estilo7">Datos Guardados con éxito</div>';
 
 
 
 
 
?>

Y el formulario desde donde el usuario hace la modificacion:

Código PHP:

Ver original<form action="update_cuenta.php" method="post">
  <table width="330" border="0">
    <tr>
      <td>Nombre(s):</td>
      <td><input type="text" name="firstname"  id="firstname" value=" <?php echo $query[firstname];?> " /></td>
    </tr>
    <tr>
      <td>Apellido:</td>
      <td><input type="text" name="firstname"  id="firstname" value=" <?php echo $query[lastname];?> " /></td>
    </tr>
        <div id="container_submit"> <input name="Submit" type="submit" class="titulo" value="EDITAR PERFIL">   </div>
  </form>

Cual puede ser el problema por el cual no se hace la actualizacion?

Gracias, den_22

rpv · #2 (**permalink**) 12/03/2010, 18:18

es un error de sql mi estimado, en tu code, la línea 18 contiene una coma adicional que hace que tu query falle.

gastonemen · #3 (**permalink**) 12/03/2010, 18:22

y si pruebas con un

Código PHP:

Ver originalif(isset($_REQUEST['submit'])){
$firstname=$_POST['firstname']; 
$lastname=$_POST['lastname'];  //introduccimos la modificacion hecha por el usuario   
 
mysql_query("UPDATE pruebaSET firstname='$firstname', lastname='$lastname' WHERE usuario='$_SESSION[usuario]' ");
 
echo '<div class="Estilo7">Datos Guardados con éxito</div>';
}
[

y lo otro....

Código PHP:

Ver original<form action="update_cuenta.php" method="post">
  <table width="330" border="0">
    <tr>
      <td>Nombre(s):</td>
      <td><input type="text" name="firstname"  id="firstname" value=" <?php echo $query[firstname];?> " /></td>
    </tr>
    <tr>
      <td>Apellido:</td>
      <td><input type="text" name="firstname"  id="firstname" value=" <?php echo $query[lastname];?> " /></td>
    </tr>
        <div id="container_submit"> <input name="Submit" type="submit" class="titulo" value="EDITAR PERFIL">   </div>
  </form>

tienes 2 veces "firstname" y nunca tienes a "lastname"

lo otro usar el echo"que salio???".$firstname=$_POST['firstname']; para ver que toma...

bueno en eso te puedo acotar.

suerte ! D:

den_22 · #4 (**permalink**) 12/03/2010, 20:09

No le encuentro la solución, probé con lo que me dijeron. gastonemen hice lo que dijiste, pero lo toma como undefined.

Para hacerlo mas completo al post, y dar mas informacion para que me puedan ayudar,les comento que el modificarperfil.php se maneja con tres archivos el connect.php, el function.php, y el getdata.php

Estos son:

function.php:

Código PHP:

Ver original<?php
 
 
 
 
 
 
 
function getfname($usuario)
{
    $query = mysql_query("select * from prueba where usuario='$usuario'");
    $getname= mysql_fetch_array($query);
    $firstname= $getname["firstname"];
    return $firstname;
}
 
function getlname($usuario)
{
    $query =mysql_query("select * from prueba where usuario='$usuario' ");
    $getname= mysql_fetch_array($query);
    $lastname=$getname["lastname"];
    return $lastname;
}
 function getgender ($usuario)
{
    $query=mysql_query("select * from prueba where usuario='$usuario' ");
    $getname=mysql_fetch_array($query);
    $gender=$getname["sexo"];
    return $gender;
}
function getestadocivil ($usuario)
{
    $query= mysql_query("select * from prueba where usuario='$usuario'");
    $getname=mysql_fetch_array($query);
    $estadocivil=$getname["estadocivil"];
    return $estadocivil;
}
 
?>

El getdata.php:

Código PHP:

Ver original<?php
 
include ("connect.php");
 
$query=mysql_query("select * from prueba where usuario='$usuario'");
$getusuario=mysql_fetch_array($query);
$firstname=getfname('firstname');
$lastname=getlname('lastname');
$gender=getgender('sexo');
?>

Esto lo que hace es mostrar los datos de la bd, en un formulario, que es el que quiero modificar con el update_cuenta.php.
Existe otra forma de hacerlo a partir de esos archivos?

den_22 · #5 (**permalink**) 13/03/2010, 07:35

Hola, como puedo hacer para definir una funcion que llame al action de un formulario
Algo asi if ($action == "save"), como defino $action?

eulloa · #6 (**permalink**) 13/03/2010, 08:02

De lo primero ya te lo dijeron arriba, por si no lo viste es la línea de la consulta update

Código PHP:

Ver originalmysql_query("
UPDATE prueba
SET firstname='$firstname',
lastname='$lastname'
WHERE usuario='$_SESSION[usuario]' ");

La consulta tuya tenía una coma antes del where
De lo segundo, por más q miré no entendí. ¿Te podrías explicar un poco mejor?

den_22 · #7 (**permalink**) 13/03/2010, 08:11

A ver tengo un perfil de usuarios, en éste hay un apartado para modificar perfil.
El modificarperfil.php, muestra los datos de registro de la bd, y también más campos que hay que llenar. Lo que yo quiero hacer es que en este formulario los datos que se introduzcan se actualicen, pero es lo que no puedo lograr. Igual sacandole la "," no hay forma.

Entonces el modificarperfil.php:

Código PHP:

Ver original<?php
 
session_name('pLogin');
session_set_cookie_params(2*7*24*60*60);
session_start();
 
 
 
 
$usuario = $_SESSION['usuario'];
 
include("function.php");
include("getdata.php");
include("connect.php"); //incluimos el connect.php que contiene los datos de la conexión a la db y la sesión
 
if(!isset($usuario)){
 
echo 'No se ha seleccionado ningun usuario.'; 
 
}else{ 
 
$query=mysql_query("SELECT * FROM prueba WHERE usuario='$usuario' ");
 
if(mysql_num_rows($query)>0){
 
$query=mysql_fetch_array($query);
//todo comprobado, ahora solo falta mostrar los datos
echo 'Bienvenid@ <b>'.$query['usuario'].'</b><br>'; 
 
}else{
 
echo 'El usuario seleccionado no existe';
 
}
}
?>

El connect.php:

Código PHP:

Ver original<?php
 
if(!defined('INCLUDE_CHECK')) {
}else{
echo 'No estas autorizado a ejecutar este archivo directamente';
}
 
 
/* Database config */
 
$db_host        = 'localhost';
$db_user        = 'usr';
$db_pass        = 'xxxx';
$db_database    = 'prueba'; 
 
/* End config */
 
 
 
$link = mysql_connect($db_host,$db_user,$db_pass) or die('Imposible establecer la conexión a la base de datos');
 
mysql_select_db($db_database,$link);
mysql_query("SET names UTF8");
 
?>

El function.php:

Código PHP:

Ver original<?php
 
 
 
 
 
 
 
function getfname($usuario)
{
    $query = mysql_query("select * from prueba where usuario='$usuario'");
    $getname= mysql_fetch_array($query);
    $firstname= $getname["firstname"];
    return $firstname;
}
 
function getlname($usuario)
{
    $query =mysql_query("select * from prueba where usuario='$usuario' ");
    $getname= mysql_fetch_array($query);
    $lastname=$getname["lastname"];
    return $lastname;
}
 function getgender ($usuario)
{
    $query=mysql_query("select * from prueba where usuario='$usuario' ");
    $getname=mysql_fetch_array($query);
    $gender=$getname["sexo"];
    return $gender;
}
function getestadocivil ($usuario)
{
    $query= mysql_query("select * from prueba where usuario='$usuario'");
    $getname=mysql_fetch_array($query);
    $estadocivil=$getname["estadocivil"];
    return $estadocivil;
}
 
?>

El getdata.php:

Código PHP:

Ver original<?php
 
include ("connect.php");
 
$query=mysql_query("select * from prueba where usuario='$usuario'");
$getusuario=mysql_fetch_array($query);
$firstname=getfname('firstname');
$lastname=getlname('lastname');
$gender=getgender('sexo');
?>

El form del modificarperfil.php:

Código PHP:

Ver original<form method="post" action="updatecuenta.php" >
  <table width="330" border="0">
    <tr>
      <td width="213">Nombre de usuario:</td>
      <td width="101"><input type="text" name="usuario"  id="usuario" value=" <?php echo $query[usuario];?> " /></td>
    </tr>
    <tr>
      <td>Nombre(s):</td>
      <td><input type="text" name="firstname"  id="firstname" value=" <?php echo $query[firstname];?> " /></td>
    </tr>
    <tr>
      <td>Apellido:</td>
      <td><input type="text" name="lastname"  id="lastname" value=" <?php echo $query[lastname];?> " /></td>
    </tr>
</table>
</form>

eulloa · #8 (**permalink**) 13/03/2010, 08:22

Ahora no veo por ninguna parte una consulta UPDATE. ¿Es todo el code q tienes?.
No debes postear todo el code, mejor el q crees q está fallando junto con una explicación de lo q está pasando, me refiero al update_cuenta del principio, ¿cuando dices falla es q salta algún error o solo q no actualiza nada de la BD?

den_22 · #9 (**permalink**) 13/03/2010, 08:27

Disculpá me olvidé de postear el update_cuenta.php, claro el tema es que no me actualiza nada de la bd. Diciendome el notice de "undefined index:firstname" y el de "undefined index:lastname".

update_cuenta.php:

Código PHP:

Ver original<?
 
include('connect.php'); //incluimos el connect.php que contiene los datos de la conexi&oacute;n a la db
include('function.php');
include('getdata.php');
 
$firstname=$_POST['firstname']; 
$lastname=$_POST['lastname'];
 
 
//introduccimos la modificacion hecha por el usuario
 
 
mysql_query("
UPDATE prueba
SET firstname='$firstname',
lastname='$lastname'
WHERE usuario='$_SESSION[usuario]' "); 
 
echo '<div class="Estilo7">Datos Guardados con éxito</div>';
 
 
 
 
?>

eulloa · #10 (**permalink**) 13/03/2010, 08:30

EN el form HTML tienes el action updatecuenta.php y tu archivo se llama update_cuenta.php. Arréglalo y vuelve a probar

den_22 · #11 (**permalink**) 13/03/2010, 08:39

Ya lo arreglé pero nada.

eulloa · #12 (**permalink**) 13/03/2010, 08:47

Prueba a agregar en la consulta
#
mysql_query("UPDATE prueba SET firstname='$firstname', lastname='$lastname' WHERE usuario='$_SESSION[usuario]' ") or die (mysql_error());

den_22 · #13 (**permalink**) 13/03/2010, 09:00

No, tira los mismos errores.

Por ejemplo estaba pensando en hacer el codigo en el mismo php del modificarperfil.php.

Código PHP:

Ver original<?php
 
 
 
if ($action == "submit")
{
    mysql_query("update prueba set firstname='$firstname',lastname='$lastname' where usuario=$usuario") or mysql_error();   
}
 
?>

y en el form:

<form method=post action="#">

y el boton <input type="hidden" name="action" value="submit">

pero el tema es que no sé como tendria que declarar la funcion $action

den_22 · #14 (**permalink**) 13/03/2010, 09:54

Alguien tiene una idea de como podria hacer?. No puedo hacer que ande.

darkasecas · #15 (**permalink**) 13/03/2010, 10:02

En ese caso, $action no seria una funcion, sino una variable, y si la mandas como un campo del formlario se define igual que los otros campos, recibiendola de $_POST

den_22 · #16 (**permalink**) 13/03/2010, 10:19

A ver digamos que lo pongo asi, quito lo de update_cuenta.php.

modificarperfil.php:

Código PHP:

Ver original<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<?php
error_reporting(E_ALL ^ E_NOTICE);
session_name('pLogin');
session_set_cookie_params(2*7*24*60*60);
session_start();
 
 
 
 
$usuario = $_SESSION['usuario'];
 
include("function.php");
include("getdata.php");
include("connect.php"); //incluimos el connect.php que contiene los datos de la conexión a la db y la sesión
 
 
 
 
 
if(!isset($usuario)){
 
echo 'No se ha seleccionado ningun usuario.'; 
 
}else{ 
 
$query=mysql_query("SELECT * FROM prueba WHERE usuario='$usuario' ");
 
if(mysql_num_rows($query)>0){
 
$query=mysql_fetch_array($query);
//todo comprobado, ahora solo falta mostrar los datos
echo 'Bienvenid@ <b>'.$query['usuario'].'</b><br>'; 
 
}else{
 
echo 'El usuario seleccionado no existe';
 
}
}
?>
<?php
 
 
 
if ($action == "submit")
{
    mysql_query("update prueba set firstname='$firstname',lastname='$lastname' where usuario=$usuario") or mysql_error();   
}
 
?>
 
 
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<title>Modificar perfil|Deemo|</title>
<link href="modificarperfil.css" rel="stylesheet" type="text/css" />
<style type="text/css">
<!--
body {
    background-image: url(../images/fondito.png);
    background-repeat: repeat;
}
 
 
-->
</style>
</head>
 
<body>
<div id="container_logo"></div>
<div id="container_modify"><div id="cont_first"> 
<div id="container_tabla">
<form method="post"  action="#" >
  <table width="330" border="0">
    <tr>
      <td width="213">Nombre de usuario:</td>
      <td width="101"><input type="text" name="usuario"  id="usuario" value=" <?php echo $query[usuario];?> " /></td>
    </tr>
    <tr>
      <td>Nombre(s):</td>
      <td><input type="text" name="firstname"  id="firstname" value=" <?php echo $query[firstname];  ?> " /></td>
    </tr>
    <tr>
      <td>Apellido:</td>
      <td><input type="text" name="lastname"  id="lastname" value=" <?php echo $query[lastname];?> " /></td>
    </tr>
    
  </table>
    <div id="container_submit"> <input type="hidden" name="action"  value="submit"><input type=image height=23 alt=Save width=87 src="../images/boton_submit.gif"  border=0 name=submit>  </div>
  </form>
 
</div>
 
</div>
</div>
 
 
 
 
 
</body>
</html>

Estaria bien asi?, porque no le veo el cambio

darkasecas · #17 (**permalink**) 13/03/2010, 11:05

Lo unico que veo es que no estas asignando las variables de $_POST

den_22 · #18 (**permalink**) 13/03/2010, 11:07

En donde deberia asignarlas?

den_22 · #19 (**permalink**) 13/03/2010, 11:45

Alguien tiene una idea de como poder hacerlo?, el tema es que no me sale :(

darkasecas · #20 (**permalink**) 13/03/2010, 12:51

Duh, pues en donde las vayas a usar :/
Asigna $action antes de hacer la comparacion con "submit"
y las otras antes de hacer la consulta en la bd donde las utilizas :