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> </p>