Me dice que complete el formulario cuando ya lo he completado del todo

Distriker · #1 (**permalink**) 22/12/2009, 18:18

Buenas, resulta ser de que estoy haciendo un foro para una página web y para evitar que se publiquen mensajes sin datos en algunos campos. Mi problema es que aunque rellene todos los campos siempre me dice que faltan campos, y luego lo raro es que el mensaje es publicado, que es lo que mas me mosquea.

Pero bueno, antes de comenzar a dejaros mi código me gustaría comentaros aspectos de mi foro que pienso que son importantes.

Le he creado un editor WYSING para mayor usabilidad del foro, así los usuarios podrán moverse mejor. Ya os dejaré el script también por si os hace falta.

No tiene sistema de usuarios, de momento.

Y bueno, creo que ya puedo dejaros el código, aunque si necesitáis mas información decídmelo y os la daré.

formulario.html

Código HTML:

Ver original<div width="90%" border="0">
<form name="formulario" action="agregar.php" method="post">
<input type="hidden" name="identificador" value="<?php echo $id ; ?>">
Autor: <input type="text" name="autor" maxlength="20"> <br />
Titulo: <input type="text" name="titulo" value="<?php echo $titulo ; ?>"> <br />
<form id="form1" name="form1" method="post" action="">
<input type="button" name=negrita" value="Negrita" onclick="formato('bold')" />
<input type="button" name="sursiva" value="Cursiva" onclick="formato('italic')" />
<input type="button" name="subrayado" value="Subrayado" onclick="formato('underline')" />
<input type="button" name="eliminar-formato" value="Eliminar formato" onclick="formato('RemoveFormat')" />
<input type="button" name="insertar-link" value="Insertar link" onclick="insertarEnlace()" />
<input type="button" name="eliminar-link" value="Eliminar link" onclick="formato('Unlink')" />
<input type="button" name="insertar-imagen" value="Insertar imagen" onclick="insertarImagen()" />
<input type="button" name="deshacer-acción" value="Deshacer acción" onclick="formato('undo')" />
<input type="button" name="rehacer-acción " value="Rehacer acción" onclick="formato('redo')" />
<input type="button" name="insertar-html" value="Insertar texto mediante HTML" onclick="inHTML()" />
<br />
<iframe name="mensaje" id="editor" width="100%" height="300" style="border:1px solid #000;"><?php echo $mensaje ; ?></iframe>
<input type="submit" name="Submit" value="Enviar mensaje">
</form> 
</div>

agregar.php

Código PHP:

  <?php
require ('configuracion.php') ;
$autor = $_POST ["autor"] ;
$titulo = $_POST ["titulo"] ;
$mensaje = $_POST ["mensaje"] ;
$identificador = $_POST ["identificador"] ;

if ($autor == "" or $titulo == "" or $mensaje == "") {
  echo 'Complete el formulario' ;
}

$sql = "INSERT INTO temas (autor, titulo, mensaje, identificador, fecha, ultimarespuesta) " ;
$sql.= "VALUES ('$autor', '$titulo', '$mensaje', '$identificador', NOW(),NOW())" ;
$rs = mysql_query ($sql, $conectar) or die("Error al grabar el mensaje: ".mysql_error) ;
$ult_id = mysql_insert_id ($conectar) ;

if (!empty($identificador))
{
$sql = "UPDATE foro SET respuestas=respuestas+1, ultimarespuesta=NOW()" ;
$sql.= " WHERE id = '$identificador'" ;
$rs = mysql_query ($sql, $conectar) ;
Header ("Location: foro.php?id=$identificador#ult_id") ;
exit () ;
}
Header ("Location: index.php") ;
?>

header.html

Código JAVASCRIPT:

Ver original<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<title>Foro de Soporte de IMGBeta</title>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css">
</style>
<script>
var editor;
function $(id){ 
    return document.getElementById(id) ;
}
function formato(f){
    editor.execCommand(f, false, null) ;
}
function rev(t)    {
    return t.split("<").join("&lt;").split(">").join("&gt;").split("\"").join("&quot;") ;
}
function insertarEnlace(){
    var u ;
    if(!(u=prompt('Ingresar URL:','http://')))return ;
    editor.execCommand("CreateLink",false,u) ;
}
function insertarImagen(){
    var u ;
    if(!(u=prompt('Ingresar URL de la imagen:','http://')))return ;
    editor.body.focus() ;
    editor.execCommand("InsertImage",false,u) ;
}
 
function inHTML(){
    var u,u2 ;
    if(!(u=prompt('Ingresar texto escrito en HTML:','')))return ;
    
        try{
            editor.execCommand("inserthtml",false,u) ;
        }catch(e){
            try{
                editor.body.focus() ;
                u2=editor.selection.createRange() ;
                u2.pasteHTML(u) ;
            }catch(E){
                alert('No es valido') ;
            }
        }
}
 
window.onload=function(){
    
    editor=$('editor').contentDocument || $('editor').contentWindow.document ;
    editor.designMode='on' ;
}
</script>
</head>
<body bgcolor="#9999cc" text="#FFFFFF" link="#FFFFFF">
<div width="90%" border="0" align="center">
<div bgcolor="#4b557d"><h2>Foro de soporte de IMGBeta</h2>
<div align="right">[ <a href="index.php">Inicio</a>] - <a href="respuesta.php">Nuevo tema</a> ] </div> </div>
<br />
<br />

Le pongo el coloreado de JavaScript para poder apreciar el script.

index.php

Código PHP:

 
<html>
<head>
<link rel="stylesheet" href="template/classic/css/estilo.css" type="text/css" media="all">
<?php
require ('configuracion.php') ;
require ('funciones.php') ;
require ('template/classic/header.html') ;
$sql = "SELECT id, autor, titulo, fecha, respuestas, ultimarespuesta " ;
$sql.= "FROM temas WHERE identificador=0 ORDER BY ultimarespuesta DESC " ;
$rs = mysql_query ($sql, $conectar) or die(mysql_error()) ;
if(mysql_num_rows($rs)>0)
{
$template = implode ("", file("template/classic/temas.html")) ;
require ('template/classic/titulos.html') ;
while ($row = mysql_fetch_assoc($rs)) 
{
$color = ($color == ""?"#999999":"") ;
$row ["color"] = $color ;
mostrartemplate ($template, $row) ;
}
}
require ('template/classic/footer.html') ;
?>

foro.php

Código PHP:

  <?php
require ('configuracion.php') ;
require ('funciones.php') ;
$id = $_GET ["id"] ;
if (empty($id)) Header ("Location: index.php") ;

$sql = "SELECT id, autor, titulo, mensaje, " ;
$sql.= "DATE_FORMAT(fecha, '%d/%m/%Y %H:%i:%s') as enviado FROM temas " ;
$sql.= "WHERE id='$id' OR identificador='$id' ORDER BY fecha ASC " ;
$rs = mysql_query ($sql, $conectar) ;
require ('template/classic/header.html') ;
if (mysql_num_rows($rs)>0) 

require ('template/classic/titulos_post.html') ;
$template = implode ("", file ('template/classic/post.html')) ;
while ($row = mysql_fetch_assoc ($rs))
{
$color = ($color== ""?"#5b69a6":"") ;
$row ["color"] = $color ;
$row ["mensaje"] = nl2br ($row["mensaje"]) ;
$row ["mensaje"] = parsearTags($row["mensaje"]) ;
mostrartemplate ($template, $row) ;
}
require ('template/classic/footer.html') ;
?>

Esto es todo lo que creo que debo de dejaros, espero que podáis ayudarme.

Muchas gracias de antemano.

Saludos

jackson666 · #2 (**permalink**) 22/12/2009, 19:12

Asi nomas veo esto (en agregar.php):

Código PHP:

  [...]

/** aca estas preguntando si la variable contiene 
  * una cadena vacia adentro (cosa que no esta  
  * del todo bien) para eso esta empty() o isset().
  * Te lo corregi: */

if (empty($autor) or empty($titulo) or empty($mensaje)) {
  echo 'Complete el formulario' ;

/** ademas si te fijas las lineas que estan fuera 
  * del if, siempre se ejecutan, debido a que no hay un else 
  * Proba asi: */
}else{

$sql = "INSERT INTO temas (autor, titulo, mensaje, identificador, fecha, ultimarespuesta) " ;
$sql.= "VALUES ('$autor', '$titulo', '$mensaje', '$identificador', NOW(),NOW())" ;
$rs = mysql_query ($sql, $conectar) or die("Error al grabar el mensaje: ".mysql_error) ;
$ult_id = mysql_insert_id ($conectar) ;

}

[...]

Distriker · #3 (**permalink**) 23/12/2009, 05:35

Buenas Jackson, vale, no sabía que se podía tener como error esa cadena así, gracias por decirlo. Vale, ahora tengo el empty en vez de las cadenas, vacias, por lo que me quedó así:

Código PHP:

  if (empty ($autor) or empty ($titulo) or empty ($mensaje)) { 
  echo 'Complete el formulario' ;

Pero tengo un problema, el error continua, sigue diciéndome que rellene todo el formulario, pero una cosa si se solucionó, aunque te mande ese mensaje ya no se graba en la BD, por lo que gracias Jackson, me acabas de resolver un problema, gracias.

Ahora tengo que averiguar por que sigue diciéndome esto cuando ya he rellenado todo y aun así me sigue diciendo eso. Por una vez pensé que iba a ser el iframe que no tenía la variable correcta, pero no, no es el iframe.

Voy a ello

Por cierto, el código entero queda así:

Código PHP:

  <?php 
require ('configuracion.php') ; 
$autor = $_POST ["autor"] ; 
$titulo = $_POST ["titulo"] ; 
$mensaje = $_POST ["mensaje"] ; 
$identificador = $_POST ["identificador"] ; 
 
if (empty ($autor) or empty ($titulo) or empty ($mensaje)) { 
  echo 'Complete el formulario' ; 
 
} else { 
 
$sql = "INSERT INTO temas (autor, titulo, mensaje, identificador, fecha, ultimarespuesta) " ; 
$sql.= "VALUES ('$autor', '$titulo', '$mensaje', '$identificador', NOW(),NOW())" ; 
$rs = mysql_query ($sql, $conectar) or die("Error al grabar el mensaje: ".mysql_error) ; 
$ult_id = mysql_insert_id ($conectar) ; 
} 
 
if (!empty($identificador)) 
{ 
$sql = "UPDATE foro SET respuestas=respuestas+1, ultimarespuesta=NOW()" ; 
$sql.= " WHERE id = '$identificador'" ; 
$rs = mysql_query ($sql, $conectar) ; 
Header ("Location: foro.php?id=$identificador#ult_id") ; 
exit () ; 
} 
?>

Saludos

jackson666 · #4 (**permalink**) 23/12/2009, 14:13

Y si te dice eso es porque alguna de las variables esta vacia.... Hace esto:

Código PHP:

  <?php 
$autor = $_POST ["autor"] ; 
$titulo = $_POST ["titulo"] ; 
$mensaje = $_POST ["mensaje"] ; 
$identificador = $_POST ["identificador"] ; 

echo $autor."<br />";
echo $titulo."<br />";
echo $mensaje."<br />";
echo $identificador."<br />";

/** si alguna te imprime un valor en blanco
  * seguramente tengas un problema con
  * los nombres de tus cajas de texto */
?>

darkasecas · #5 (**permalink**) 23/12/2009, 15:12

-Despues de abrir el form, abres otro form.
-Estas intentando enviar un iframe como campo del formulario? Hasta donde se no se puede hacer eso.

Distriker · #6 (**permalink**) 28/12/2009, 17:45

Cita:

Iniciado por darkasecas

-Despues de abrir el form, abres otro form.
-Estas intentando enviar un iframe como campo del formulario? Hasta donde se no se puede hacer eso.

Lo del form está corregido.

Lo del Iframe, si ya, pero es que hasta donde yo he podido averiguar, tampoco es valido mi editor si le pongo un text-area o un input con el atributo text, por lo que he tenido que dejar el iframe hasta averiguar que diablos podría usar.

¿A alguien se le ocurre algo?

Saludos

zaetoner · #7 (**permalink**) 28/12/2009, 19:22

Probre tu script localmente y me funciona, creo que deberias revisar que los nombres en tu base de datos coinsidan con los del script

Distriker · #8 (**permalink**) 29/12/2009, 04:47

¿Te funciona enviando datos con el iframe o con el text-area?

¿Te manda el mensaje como si todos los campos estubiesen correctos?

Saludos

Edito.

Finalmente se queda con el textarea, pero tengo que buscar algo para que sirva mi script.

El codigo de agregar.php se queda así:

Código PHP:

Ver original<?php
require ('configuracion.php') ;
$autor = $_POST ["autor"] ;
$titulo = $_POST ["titulo"] ;
$mensaje = $_POST ["mensaje"] ;
$identificador = $_POST ["identificador"] ;
 
if (empty ($autor) or empty ($titulo) or empty ($mensaje)) {
  echo 'Complete el formulario' ;
 
} else {
 
$sql = "INSERT INTO temas (autor, titulo, mensaje, identificador, fecha, ultimarespuesta) " ;
$sql.= "VALUES ('$autor', '$titulo', '$mensaje', '$identificador', NOW(),NOW())" ;
$rs = mysql_query ($sql, $conectar) or die("Error al grabar el mensaje: ".mysql_error) ;
$ult_id = mysql_insert_id ($conectar) ;
}
 
if (!empty($identificador))
{
$sql = "UPDATE foro SET respuestas=respuestas+1, ultimarespuesta=NOW()" ;
$sql.= " WHERE id = '$identificador'" ;
$rs = mysql_query ($sql, $conectar) ;
Header ("Location: foro.php?id=$identificador#ult_id") ;
exit () ;
}
?>

Saludos