AJAX no envía los datos (vía GET) pero me dice que si

[Ueki]

¡Hola!
Soy bastante novato en AJAX, así que quizás sea una tontería el error, pero no consigo encontrarlo.
Son dos cosas, uno de enviar datos y otro de eliminarlo, los pongo por separado y explico cada cosa.

El objeto AJAX lo tengo definido de la siguiente forma:

Código Javascript:

function objetoAjax(){
    var xmlhttp=false;
    try {
        xmlhttp = new ActiveXObject("Msxml2.XMLHTTP");
    } catch (e) {
        try {
           xmlhttp = new ActiveXObject("Microsoft.XMLHTTP");
        } catch (E) {
            xmlhttp = false;
        }
    }
 
    if (!xmlhttp && typeof XMLHttpRequest!='undefined') {
        xmlhttp = new XMLHttpRequest();
    }
    return xmlhttp;
}
 

Enviar datos vía GET con AJAX
Tengo un código que, debería enviarme datos a la BD, pero no lo hace, aunque me salta el mensaje de confirmación de que si que lo ha hecho. Pongo el código .js y el .php.

La conexión con la BD está hecha en otro documento, ya que este se usa en otro por medio de un include().

Código PHP:

<?php
if($_GET['act'] == "denunciar")
{
    $idcoment = mysql_escape_string($_GET['id']);
    $autor = mysql_escape_string($_GET['autor']);
    if(empty($idcoment))
    {
        echo '<script type="text/javascript">alert("No hay ninguna ID."); history.back();</script>';
    }
    else
    {
        mysql_query("INSERT INTO `denunciacoment` (`idcoment`, `autor`) VALUES ('$idcoment','$autor')");
    }
}
?>
 

Código Javascript:

// Denunciar comentario con AJAX //
function denunciarcom(id, autor){
    //usaremos un cuadro de confirmacion    
    var denunciar = confirm("¿Está seguro de que desea denunciar el comentario? Si lo hacee, el equipo de la web se encargará de revisarlo y tomar las medidas necesarias.");
    if ( denunciar ) {
        //instanciamos el objetoAjax
        ajax=objetoAjax();
        //uso del medotod GET
        //indicamos el archivo que realizará el proceso de denuncia
        //junto con un valor que representa el id del comentario
        ajax.open("GET", "/comentarios.php?act=denunciar&id="+id+"&autor="+autor);
        ajax.onreadystatechange=function() {
            if (ajax.readyState==4) {
                //mostrar resultados en esta capa
                alert("Comentario denunciado");
            }
        }
        //como hacemos uso del metodo GET
        //colocamos null
        ajax.send(null)
    }
}
// Denunciar comentario con AJAX //
 

Borrar datos vía GET con AJAX
En este caso, el problema es igual, solo que en lugar de enviar datos, necesito que los borre.

Código PHP:

$sqldelete = "DELETE FROM `comentarios` WHERE `id` = '".$_GET['id']."'";
        if(mysql_query($sqldelete))
        {
            ?>
            <script type="text/javascript">alert("Comentario eliminado"); window.location = "/admin/comentarios.php";</script>
            <?php
        }
        else
        {
            ?>
            <script type="text/javascript">alert("Comentario no eliminado"); window.location = "/admin/comentarios.php";</script>
            <?php
        }
 

Código Javascript:

// Eliminar comentarios con AJAX //
function deletecom(id){
    //usaremos un cuadro de confirmacion    
    var eliminar = confirm("De verdad desea eliminar este comentario?");
    if ( eliminar ) {
        //instanciamos el objetoAjax
        ajax=objetoAjax();
        //uso del medotod GET
        //indicamos el archivo que realizará el proceso de eliminación
        //junto con un valor que representa el id del comentario
        ajax.open("GET", "/admin/comentarios.php?act=delete&id="+id);
        ajax.onreadystatechange=function() {
            if (ajax.readyState==4) {
                //mostrar resultados en esta capa
                alert("Comentario eliminado");
            }
        }
        //como hacemos uso del metodo GET
        //colocamos null
        ajax.send(null)
    }
}
// Eliminar comentarios con AJAX //
 

Muchísimas gracias a quien pueda ayudarme, sé que es algo largo de leer.

[Ueki]

Siento escribir otro post, pero me he dado cuenta de que creo saber donde está el fallo, pero no sé arreglarlo. (Hablo de la eliminación, aunque lo otro quizás sea lo mismo).

He añadido esto
if(ajax.open("GET", "/admin/comentarios.php?act=delete&id="+id)) alert("Si entro"); else alert("No entro");

Debajo del ajax.open, y siempre me devuelve No entro.

¿Alguien sabría arreglarlo? Muchas gracias

[zerokilled]

buenas,
¿cómo has de determinar que los datos se enviaron? pregunto porque al final del título dices "pero me dice que si". revisa la consola de error del navegador para determinar si hay algún error obvio en javascript. de momento, no le veo nada extraño al menos en el script del lado cliente. te recomiendo que realices pruebas simples sin la base de datos. es decir, determinar si el servidor recibe los datos y luego devolver una confirmación de parte del servidor. esta confirmación te llegará a la propiedad responseText del objeto ajax. si todo marcha bien, entonces el problema debe ser en el servidor.

Iniciado por Ueki

He añadido esto
if(ajax.open("GET", "/admin/comentarios.php?act=delete&id="+id)) alert("Si entro"); else alert("No entro");

el método open siempre devuelve undefined. cuando undefined es convertido a boolean equivale a false. por tanto, el intérprete nunca entrará al bloque de la condición if.

[Ueki]

Con lo de "y me dice que si" me refiero a que if (ajax.readyState==4) {
//mostrar resultados en este alert
alert("Comentario eliminado con éxito");
}

Me dice "Comentario eliminado con éxito". En el caso del eliminado. "Comentario denunciado" en el caso de denunciacom().

Por lo demás, en la consola no me marca ningún error.

Lo que puse en mi segundo comentario puede ayudar en algo? Es que me di cuenta de que ahí falla algo.

Muchísimas gracias :D

[zerokilled]

ahí no falla nada, simplemente es inútil utilizar el valor devuelto por open como condición en la expresión if. tal como comente anteriormente, open siempre devuelve undefined el cual es equivalente a false.