Skip to main content
3 de 3
se eliminaron 6 caracteres en el cuerpo
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

Como bien indica @sal, debes usar comillas para valores de tipo cadena en consultas SQL.

Ahora bien, el tema de la seguridad es importante, pasar una consulta como la tuya directamente abriría la puerta a ataques de inyección SQL, lo cual supone un gravísimo problema de seguridad. Para evitarlo, lo mejor es usar consultas preparadas.

Por otro lado, no deberías usar affected_rows para consultas del tipo SELECT, esa propiedad es para consultas del tipo DELETE, INSERT o UPDATE. Es importante que leas la documentación antes de empezar a trabajar con una herramienta, más aún si se trata del manejo de datos.

Dado que quieres obtener un JSON como respuesta final, podrías usar el método fetch_object(), que te ayudará a escribir el código de forma más simple y natural sin tener que recurrir a trucos hechos a mano como esto $json=substr(trim($json),0,-1); o esto $json=$json."]}"; Pero mysqli en algunas versiones depende de otro controlador que no viene instalado en todos los entornos PHP para funcionar sin demasiadas complicaciones. Por eso no usaremos fetch_object(), pero sí trabajaremos convirtiendo el array de filas con json_decode(json_encode($rows)).

Por último, no programes de forma ingenua. Verifica todo aquello que podría ser nulo o vacío antes de proceder. Sólo así tendrás un código maduro, que debe contemplar el manejo de errores.

Corrigiendo todo lo dicho, podrías escribir tu código así:

<?php

if($_SERVER['REQUEST_METHOD']=="GET"){
    $Marca = $_GET ['Marca_Vehiculo'] ?? NULL;
    if ($Marca) {
        require("conexion1.php");
        $conexion = retornarConexion();
        if ($conexion) {
            $query="SELECT N_Filas,Tipo_Vehiculo,Marca_Vehiculo,Modelo_Vehiculo,Activo FROM vehiculo1 WHERE Marca_Vehiculo = ? and Activo = 1 
    Order By N_Filas asc";
            if ($stmt=$conexion->prepare($query)) {
                $stmt->bind_param("s", $Marca);
                $stmt->execute();
                $stmt->bind_result($Filas, $Tipo, $MarcaV, $Modelo, $Activo);
                $rows=array();
                while($row=$stmt->fetch()){
                    $rows[]=array('N_Filas'=>$Filas,'Tipo_Vehiculo'=>$Tipo,'Marca_Vehiculo'=>$MarcaV,'Modelo_Vehiculo'=>$Modelo,'Activo'=>$Activo);
                }
                $stmt->close();
                if ($rows) {
                    $data=array('data'=>json_decode(json_encode($rows)));
                } else {
                    $data=array('error'=>'No filas encontradas');
                }
            } else {
                $data=array('error'=>'Error en consulta SQL');
            }
            $conexion->close();
        } else {
            $data=array('error'=>'No hay conexión');
        }
    } else {
        $data=array('error'=>'No se postearon datos');
    }
} else {
    $data=array('error'=>'Método no permitido');
}
header('Content-type:application/json;charset=utf-8');
echo json_encode($data);
?>
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257