Ayuda con ajax

[vesvello]

Necesito que, haciendo click en el enlace me ejecute en segundo plano favoritos.php pero sin recarar la pagina... esto es lo que tengo pero no funciona... alguien sabria donde esta el error que pudiera guiarme?

Código PHP:

 <a href="JavaScript:void(0);" onclick="anadirFavoritos(<?php echo $arrAd["codigo"];?>)">A&ntilde;adir a favoritos</a>  

Código:

function anadirFavoritos(elId) {
	var contenedor;
	contenedor = document.getElementById('favoritos');
       
	ajax=nuevoAjax();
	ajax.open("GET", "anadirfavoritos.php?codigo="+elId ,true);
	
	
	
	ajax.onreadystatechange=function() {
		if (ajax.readyState==4) {
			contenedor.innerHTML = ajax.responseText
		}
	}
	 ajax.send(null)
}

y en anadirfavoritos.php:

Código PHP:

  if($_GET["codigo"] != "")
{
$_SESSION[strfavoritos].= " codigo LIKE '%".get_param("codigo")."%' OR";
$strfavoritos = $_SESSION[strfavoritos];
}  
   

pero claro, no funciona.. alguien sabria decirme donde esta el error?

[jackson666]

El ajax aparentemente esta bien, no se que abra en nuevoAjax() pero si devolves un objeto XMLHttpRequest esta bien...

Porque no pones un simple echo "hola"; en tu php para ver si esta entrando en el?

Te recomendaria que te instales el firebug en el firefox y veas, tal vez tengas un error de sintaxis chico...

Proba cerrando con punto y coma las siguientes lineas:

Código Javascript:

contenedor.innerHTML = ajax.responseText
 

Código Javascript:

ajax.send(null)
 

Ah y otra cosa... no es "JavaScript", es "javascript" (en minuscula).
Fijate de tener definido el atributo type en el tag <script>

[caricatos]

Hola:

No, no... el punto y coma (";") no tiene nada que ver... el retorno de carro es un separador de sentencias válido...

Tal vez la parte php... veo que falta entrecomillar los índices de la variable $_SESSION

No sé porque usas un tag "a" si no enlazas... si es por la mano, se puede usar cualquier tag y poner la mano con estilos.

Saludos

[vesvello]

Gracias por las sugerencias, pero el error estaba aqui:

Código:

onclick="anadirFavoritos(<?php echo $arrAd["codigo"];?>)"

Falatabas las ' '.
Asi es como debiera estar:

Código:

onclick="anadirFavoritos('<?php echo $arrAd["codigo"];?>')"

y ya funciona.

Iniciado por caricatos

No sé porque usas un tag "a" si no enlazas... si es por la mano, se puede usar cualquier tag y poner la mano con estilos.
Saludos

que tag sugeririas tu? Yo pense que onclick solo iba con determinados tipos de tags. Pero se aceptan sugerencias.

[caricatos]

Hola:

onclick funciona en cualquier tag visible... sobre los tags que te pueda recomendar... el que le pueda dar al script su contenido semántico, o tal vez un simple span.

Saludos