abrir popup desde boton en echo

[caballoloko]

hola que tal.

nesecito un poco de ayuda con este codigo que me esta matado ya que estoy tratando de abrir una ventana popup desde un formulario en echo pero no se me ejecuta la funcion onclick que he puesto en el boton

este es el codigo que estoy usando

Código HTML:

<title>estados</title>
<?
session_start();
if(!isset($_SESSION['k_username']))header("location: login.php");
?>
<script type="text/javascript">
function mostrar_popup()

{
	if(document.form1.Nombre.value!=0 && document.form1.hora.value!=0)
	{
		var pagina = "popup_mail.php?Nombre="+document.form1.Nombre.value+"&hora="+document.form1.hora.value;
		var opciones="toolbar=no, location=no, directories=no, status=no, menubar=no, scrollbars=yes, resizable=yes, width=700, height=350, top=125,left=350";
window.open(pagina,"",opciones);
	}
}
</script>
<FORM ACTION=estado.php METHOD=post>
  <img src="logo.png" width="232" height="49" />
  <table width=300 border=0 cellspacing=0 cellpadding=0>
    <tr>
      <td><p>Buscar RUT:</p></td>
    </tr>
    <tr>
      <td><input type=text name=ID></td>
    </tr>
    <tr>
      <td><input type=submit name=OK value=BUSCAR></td>
    </tr>
  </table>
</FORM>
<p>
  <?php 
/* 
primero vamos a formalizar el formulario 
en una funcion, asi podemos usarlo mas tarde 
con facilidad.
*/

// coneccion al servidor de bases de datos 
$dbh=mysql_connect ("localhost","root","1234") 
or die ('problema conectando porque :' . mysql_error()); 
// seleccionado la base de datos 
mysql_select_db ("tabla",$dbh); 

// recojemos las variables enviadas
$ID = $_POST['ID'];
$OK = $_POST['OK'];
$hora = $_POST['hora'];
$Nombre = $_POST['Nombre'];
$Estado = $_POST['Estado'];


if ($OK == "BUSCAR") { 
$q = "select * from tabla where RUT='$ID'"; 

// ejecutando el query select regresa un rowset 
$tabla = mysql_query($q, $dbh) or die ("problema con query") ; 

// regresando renglon con registro 
$reg = mysql_fetch_row($tabla); 


// construyendo forma dinamica 
echo "<FORM ACTION=diagnostico.php id=form1 METHOD=post>"; 
// recordar que strings se encadenan con . 
echo "Nombre:<INPUT TYPE=text NAME=Nombre value=$reg[2]><BR>"; 
echo "RUT:<INPUT TYPE=text NAME=RUT value=$reg[3]><BR>";
echo "Estado:<textarea NAME=Estado>$reg[11]</textarea><BR>"; 
echo "<input type=hidden name=ID value=$reg[0]>"; 
echo "<INPUT TYPE=submit NAME=OK VALUE=editar onclick=mostrar_popup()><BR>"; 
echo "</FORM>"; 
}

if ($OK == "editar") 
{ 
// preparando la instruccion sql 

mysql_query("INSERT INTO Diagnosticos(Nombre,Estado) 

VALUES('$Nombre','$RUT','$Estado')");

echo "nuevo estado aceptado";
}
?>
<p>&nbsp;</p> 

[andresdzphp]

Todos los atributos HTML deben ir entre comillas (y en minúsculas). No es recomendado imprimir código HTML dentro de PHP.

Código PHP:

<?php
//Código PHP
?>
FORMULARIO HTML <?php echo $variable; ?>
<?php
// más código PHP
?>
 

[caballoloko]

hola eso si lo se pero cuando ingreso los datos requeridos en el buscador automaticamente me sale los campos del formario echo con los datos en la base de datos si no lo isiera asi me saldrian los campos vacios

Iniciado por andresdzphp

Todos los atributos HTML deben ir entre comillas (y en minúsculas). No es recomendado imprimir código HTML dentro de PHP.

Código PHP:

<?php
//Código PHP
?>
FORMULARIO HTML <?php echo $variable; ?>
<?php
// más código PHP
?>
 

[andresdzphp]

Parece más un problema de HTML y javascript esto.

No puedes hacer esto?:

Código PHP:

Nombre: <input type="text" name="Nombre" value="<?php echo $reg[2]; ?>" />
 

o esto?:

Código HTML:

<input type="submit" name="OK" value="editar" onclick="mostrar_popup()" />
 

[caballoloko]

como deveria hacerlo entonces?

Iniciado por andresdzphp

Parece más un problema de HTML y javascript esto.

No puedes hacer esto:

Código HTML:

<input type="submit" name="OK" value="editar" onclick="mostrar_popup()" />
 

[andresdzphp]

Como te dije... Además de eso revisa tu función javascript:

Código Javascript:

if(document.form1.Nombre.value!=0 && document.form1.hora.value!=0) {
var pagina = "popup_mail.php?Nombre="+document.form1.Nombre.value+"&hora="+document.form1.hora.value;
 

Nisiquiera tienes un campo llamado hora y yo creo que mejor si usas

Código Javascript:

document.getElementById('campo_id').value
 

Tampoco le encuentro sentido a ese condicional. Arreglando eso te debe funcionar.

Saludos.