2

Describo mi problema: el objetivo es integrar el uso de ajax con código php. Tengo un php llamado país en el que hay un formulario desplegable y seguirá las siguientes instrucciones al seleccionar un valor llamará a la función ajax, la función ajax llamará a otro php:

El otro php leerá de la base de datos paises de la tabla provincias que contenga las provincias del pais seleccionado y se nos mostrará un desplegable con dichas provincias.

Mi error es el siguiente: Notice: undefined index pais in leerbd.php en la línea 26 ($query = "SELECT * FROM provincias where pais = '".$_GET['pais']."'";)`

pais.php:

<?php




include 'leerbd.php';


?>

<html>
    <head>
        <meta charset="utf-8">
        <title></title>
    </head>
    <body>
    <section>

    <form action="" method="post" align="center">
            Pais: <select name="pais" id="pais" onchange="Provincias()"> <!-- en el onchange le ponemos el nombre de la funcion-->
            <option value="">[Seleccione el pais]</option> 
            <option value="es">España</option>
            <option value="pt">Portugal</option>
        </select><br />
        <div id="provincias"></div>  
    </form>





    </section>

    <script type="text/javascript">
    /*función para crear el objeto de AJAX que permite la conexión*/

        function getHTTPObject(){
            if (window.ActiveXObject)
                return new ActiveXObject("Microsoft.XMLHTTP");
            else if (window.XMLHttpRequest)
                return new XMLHttpRequest();
            else {
                alert("No soportado"); 
                return null;
            }

        }



        function Provincias(){
            var pais = document.getElementById('pais').value; //recogemos el pais
            httpObject = getHTTPObject(); 
            if (httpObject != null) {
                httpObject.open("GET", "leerbd.php?pais=" + pais ,true); 
                httpObject.onreadystatechange = function()
                {
                    if(httpObject.readyState == 4  && httpObject.status == 200) 
                    /*cuando acabe de cargar leerbd.php /*mostramos el contenido generado en suma.php */
                    {

                         document.getElementById('provincias').innerHTML = httpObject.responseText;
                    }
                }
                httpObject.send(null);
            }
        } 



    </script>



</body>
</html>

leerbd.php:

<?php

//Conectando y seleccionando la base de datos

$conexion = mysqli_connect('localhost', 'root', '') or die('No pudo conectarse: ' .mysqli_error());


echo 'Conexion correcta </br>';



mysqli_select_db($conexion, 'paises') or die('No se pudo seleccionar la base de datos');
    echo 'Provincia : <select id="provincia" name="provincia">';

    $query = "SELECT * FROM provincias where pais =  '".$_GET['pais']."'";

    //echo $query;
    $result = mysqli_query($conexion, $query) or die('Consulta fallida: ' .mysqli_error());
    if($result){
        echo 'llego </br>';
        //si tenemos registros
        $row_cnt = mysqli_num_rows($result);
        if($row_cnt !== 0) {
            echo 'llego 2';
            while($row = mysqli_fetch_array($result)) {
                echo '<option value="'.$row['id'].'">'.$row['provincia'].'</option>';
            }   
        } else {
            echo 'No se han encontrado registros';
        }
    }


    echo '</select>';
    mysqli_close($conexion);


?>

`

13
  • Creo que en el leerbd.php, en vez de $_GET['pais'] debería ser un $_POST['pais'], ya que en esa página estás posteando ese valor no? Ahora no tengo el ordenador delante para hacer el código pero creo que puede ser eso
    – Vera Canet
    Commented el 29 may. 2018 a las 8:54
  • Eso no es así, la petición ajax tiene especificada que la comunicación es GET y los parámetros que le pasa son mediante la url, lo cual es GET indiscutiblemente, no obstante sí que sería una buena práctica cambiar la comunicación de GET a POST. Commented el 29 may. 2018 a las 8:56
  • PD: También es siempre una buena práctica (ya que tienes en el select una opción con valor vacío) comprobar antes de realizar la petición ajax si el valor del input tiene longitud mayor a 0, vamos : if (pais.length > 0){[...]}. Commented el 29 may. 2018 a las 8:59
  • has probado a hacer un console.log(pais) en js para comprobar captas el país como debes? Commented el 29 may. 2018 a las 9:01
  • no comprendo por qué haces un include leerbd.php Commented el 29 may. 2018 a las 9:03

2 respuestas 2

0

Tu problema es el siguiente:

<?php




include 'leerbd.php';


?>

Ya que eso va a cargar el archivo php en tu página al cargar esta. Para hacer la petición ajax no necesitas importar el archivo, solo tienes que especificar bien cual será la ruta a la que quieres llamar, al hacer el include, dentro del leerbd.php estás esperando un parámetro $_GET['pais'], el cual al principio no existe, eso siempre te va a petar. Espero que te haya servido :)

9
  • PD: puedes cambiar el pais.php por un pais.html sin ningún tipo de problema. El include 'leerbd.php'; eliminalo directamente. Commented el 29 may. 2018 a las 9:09
  • el include está al principio de pais.php
    – Dani Serra
    Commented el 29 may. 2018 a las 9:51
  • @A.Cedano metete en la edición, y podrás ver el código completo, yo tengo una edición propuesta arreglandolo, stackoverflow se bugea a veces y omite parte del texto, lo reconoce como código de la página o algo de eso y no lo muestra en el cuadro de la pregunta Commented el 29 may. 2018 a las 10:25
  • Vale, no he dicho nada, ya se ha aceptado la edición, deberías poder verlo ya Commented el 29 may. 2018 a las 10:26
  • Ahora sí lo veo. De todos modos, si envía .send(null) el servidor no recibirá nada. Borro comentario anterior. Tienes razón, ese include no tiene ningún sentido.
    – A. Cedano
    Commented el 29 may. 2018 a las 10:47
0

Dani, el problema principal de tu código es que tú estás enviando nada al hacer esto:

httpObject.send(null);1

Prueba de esta manera:

function Provincias(){
        var pais = document.getElementById('pais').value; //recogemos el pais
        httpObject = getHTTPObject(); 
        if (httpObject != null) {
            httpObject.open("POST", "leerbd.php", true); 
            httpObject.setRequestHeader("Content-type", "application/x-www-form-urlencoded");
            httpObject.send("pais="+pais);

            httpObject.onreadystatechange = function()
            {
                if(httpObject.readyState == 4  && httpObject.status == 200) 
                /*cuando acabe de cargar leerbd.php /*mostramos el contenido generado en suma.php */
                {

                     document.getElementById('provincias').innerHTML = httpObject.responseText;
                }
            }
        }
    }

NOTA SOBRE LA SEGURIDAD

El código en leerbd.php es vulnerable a ataques de Inyección SQL. Si te interesa corregir ese error, una vez solucionado el problema original planteado, lo comentas para sugerirte un código corrigiendo ese fallo.


1 Curiosamente, la documentación de MDN en Castellano muestra ejemplos erroneos sobre el método send, indicando que a este método se le puede enviar null, lo cual no tendría ningún sentido. El código entre la versión en inglés (correcto) y la versión en castellano (incorrecto), es totalmente distinto en cuanto al uso de send.

3
  • Los parámetros los pasa en la url de la petición, sí se lo está pasando. Commented el 29 may. 2018 a las 10:22
  • @enriquemanzanogil el método que envía es send. Si a ese método tú le pones .send(null) simplemente estás anulando todo lo anterior. Es como si tú tuvieras una función cualquiera que recibe parámetros, tú construyes los parámetros bien antes de usar la función, pero al final le envías null, lo que la función recibirá será null, no los parámetros que construiste bien antes de hacer la llamada. Aquí, el servidor recibirá lo que le envíes con send.
    – A. Cedano
    Commented el 29 may. 2018 a las 10:40
  • Ahhhhmmm, va, de todas formas, el include del principio también está mal puesto, el fallo del Undefined Index viene de ahí casi seguro, simplemente porque está intentando cargar el archivo al cargar la página Commented el 29 may. 2018 a las 10:42

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.