paso de variables

[bananas]

Hola,tengo un calendario con php y html y cuando haga click en el <tr><td> de un dia yame a la funcion de js valor donde le paso la fecha y el id del campo en el que quiero poner la fecha,les pongo el codigo:

Código PHP:

    <script>
    function valor(cadena,Id){
        opener.document.getElementById('Id').innerHTML=cadena;
        window.close();
    }
</script> 
  

Y donde paso las variables:

Código PHP:

  echo "<td onclick="valor($fecha,$nuevo_id)">$dia_actual</td>"; 
   

[Cluster]

Pero que problema tienes?

Tu has visto el código HTML que generas (con ese Javascript, en tu navegador: "ver código fuente") si es lo que esperas? ...

Un saludo,

[bananas]

El problema es que en la linea donde pongo:

Código PHP:

  echo "<td onclick="valor($fecha,$nuevo_id)">$dia_actual</td>"; 
   

me da un error y no me pasa los valores de $fecha y $nuevo_id a la funcion de js.

[merlin]

echo "<td onclick="valor($fecha,$nuevo_id)">$dia_actual</td>";


Tienes que por los puntos de suma de cadenas es decir:

echo "<td onclick=" . valor($fecha,$nuevo_id) . ">$dia_actual</td>";

[Cluster]

Te dá el error (que deberías haber puesto en su tiempo) por las comillas que usas . .no puedes mezclar comillas dobles o simple en una misma cadena que empezastes por " o ' .. debes "escapar" las comillas con \ delate:

Código PHP:

  echo "<td onclick=\"valor($fecha,$nuevo_id)\">$dia_actual</td>"; 
   

Un saludo,

[bananas]

Ni por esas,me sigue fallando y ya no se que hacer.
Cluster perdon por crear otro mensaje casi identico.

He modificado algunas cosas:

Código PHP:

  for ($i=0;$i<7;$i++){
                if ($i < $numero_dia){
                        //si el dia de la semana i es menor que el numero del primer dia de la semana no pongo nada en la celda
                        echo "<td></td>";
                } else {
                if (($i == 5) || ($i == 6)){
                                if ($dia_actual == $hoy){
                                    $fecha=$ano.'-'.$mes.'-'.$hoy;
                                    echo "<td bgcolor=#00FFFF onclick=\"valo($fecha,$nuevo_id)\" onMouseOver=this.style.background='#FF0000' onMouseOut=this.style.background='#00FFFF'>$dia_actual</td>";
                                }else{
                                    $fecha=$ano.'-'.$mes.'-'.$hoy;
                                    echo "<td bgcolor=#00FFFF onclick=\"valo($fecha,$nuevo_id)\" onMouseOver=this.style.background='#FF0000' onMouseOut=this.style.background='#00FFFF'>$dia_actual</td>";
                                }
                }else{
                                if ($dia_actual == $hoy){
                                    $fecha=$ano.'-'.$mes.'-'.$hoy;
                                    echo "<td bgcolor=#00CCFF onclick=\"valo($fecha,$nuevo_id)\" onMouseOver=this.style.background='#FF0000' onMouseOut=this.style.background='#00CCFF'>$dia_actual</td>";
                                }else{
                                    $fecha=$ano.'-'.$mes.'-'.$hoy;
                                    echo "<td bgcolor=#00CCFF onclick=\"valo($fecha,$nuevo_id)\" aling=center onMouseOver=this.style.background='#FF0000' onMouseOut=this.style.background='#00CCFF'>$dia_actual</td>";
                                }
                }
                        $dia_actual++;
                }
        }
        echo "</tr>";