[SOLUCIONADO] Error SQL syntaxis al realizar una búsqueda

[diegesis]

¡Buenas tardes!

Tengo un error de sintaxis que soy incapaz de descifrar...¡Espero vuestra ayuda!
Mientras no se ejecuta la consulta no hay errores, pero una vez que se hace, me aparece este error en un archivo:
You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '2013''' at line 1

Y este otro, en otro archivo:
You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'ANULADO'' order by ' clientes.provincia,usuarios.nombre,acciones.fcreac ion ' l' at line 1

¿Alguien puede echarme una mano?

[jonni09lo]

Son errores de sintaxis de sql, lo mas seguro sea que estas creando los sql de una manera incorrecta, pero sin el código que exponga la manera como lo haces la verdad bien difícil sera adivinar...

Saludos

[informacionsys]

hola

muestra la consulta que estas ejecutando

[diegesis]

Este es el aviso:

Error al Ejecutar la Consulta: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'ANULADO'' order by 'clientes.provincia,usuarios.nombre,acciones.fcrea cion' lim' at line 1

Haga donde haga la consulta en el formulario (si pongo más variables el aviso de error es más largo oviamente) me dice que es error de sintaxis. Os paso parte del archivo donde puede estar el error:

Código PHP:

<?php
 
 
 
 
 
 
 
 
 
 
 
 
 
//  echo $_POST["anadir"];
  $filtros='';
 
  $querys='';
  if(isset($_GET['estado'])) {
    if ( $_GET['estado']=='ANULADO'){
    $filtros.="accion_alumno.estado='ANULADO' AND ";
    $querys.="estado=ANULADO&";
    }elseif($_GET['estado']=='IMPAGADO'){
    $filtros.="accion_alumno.estado<>'ANULADO' AND accion_alumno.IdAccion in (select det_fra_emp.IdAccion from facturas_empresas,det_fra_emp where facturas_empresas.IdFactura=det_fra_emp.IdFactura and (facturas_empresas.estado='PTE. PAGO' or facturas_empresas.estado LIKE 'IMPAGADO%')) AND ";
    $querys.="estado=IMPAGADO&";
}elseif($_GET['estado']=='PAGADO'){
    $filtros.="accion_alumno.estado<>'ANULADO' AND accion_alumno.IdAccion in (select det_fra_emp.IdAccion from facturas_empresas,det_fra_emp where facturas_empresas.IdFactura=det_fra_emp.IdFactura and facturas_empresas.estado='PAGADO' ) AND ";
    $querys.="estado=PAGADO&";
 
    }elseif($_GET['estado']==''){
    $filtros.="accion_alumno.estado<>'ANULADO' AND ";
    $querys.="estado=&";
    }
}
  foreach($_GET as $indice => $valor) {
  if ( $indice!='pag' && $indice!='orden' && $indice!='estado'){
    if ($valor!='') {
    $querys.="$indice=$valor&";
    if ($indice=='fcreacion2'){
    $valor=guardafecha($valor);
        $filtros.="acciones.fcreacion between '$valor' and  '".sumaDia($valor,6)."' AND ";  
        }else{
        if ($indice=='ffin2'){
        $valor=guardafecha($valor);
        $filtros.="acciones.ffin between '$valor' and  '".sumaDia($valor,6)."' AND ";
        }else{  
    if ($indice=='fcreacion'){
 
        $filtros.="month(acciones.$indice)='$valor' AND ";  
        }else{
        if ($indice=='anofcreacion'){
 
        $filtros.="year(acciones.fcreacion)='$valor' AND "; 
        }else{
        if ($indice=='ffin'){
        
        $filtros.="month(acciones.$indice)='$valor' AND ";
        }else{
        if ($indice=='anoffin'){
        
        $filtros.="year(acciones.ffin)='$valor' AND ";
        }else{      
        if ($indice=='IdUsuario'){
        $filtros.="(clientes.$indice='$valor' or usuarios.IdParent='$valor') and ";
        }else{ 
        if ($indice=='provincia'){
        $filtros.="clientes.$indice='$valor' AND ";
        
        }elseif($indice=='tiporegalo'){
        $filtros.="accion_alumno.$indice='$valor' AND ";
        }elseif($indice=='IdArea'){
            if($valor=='ELEARNING'){
            $filtros.=" (cursos.IdArea=14 or cursos.IdArea=15 or cursos.IdArea=16) and ";
            }elseif($valor=='A DISTANCIA'){
            $filtros.=" (cursos.IdArea<>14 and cursos.IdArea<>15 and  cursos.IdArea<>16 and  cursos.IdArea<>12) and ";
            }else{
            
            $filtros.="cursos.$indice='$valor' AND ";
            }
        
        }else{
            
        $filtros.="cursos.$indice like '%$valor%' AND ";
        }
        }
        }
        }
        }   
        }
        } 
        }               
    
    }
    }
  }
?>
 
<table width='905' border='0' cellpadding='3'> <tr class='filaimencabezados'>
  <td width="3%">&nbsp;</td>
    <td width="6%">Provincia</td>
    <td width="11%">Delegado</td>
    <td width="8%">Comercial</td>
  <td width="7%">Cliente</td>
  <td width='5%'>Precio</td>
  <td width='9%'>Fecha Venta</td>    <td width="16%">Denominación</td>
  <td width="9%">Compromiso</td>
  <td width="9%">Test</td>
  <td colspan="2">Estado</td>
</tr>
<?php
    
    
     if ($filtros!='') {
  if (!isset($_GET['pag'])){
   $pag = 1;
   }else{
   $pag=$_GET['pag'];
   } 
     if (!isset($_GET['orden']) or empty($_GET['orden'])){
   $orden = "clientes.provincia,usuarios.nombre,acciones.fcreacion";
   }else{
   $orden=$_GET['orden'];
   } 
    $filtros=substr($filtros,0,strlen($filtros)-5); //se quita ultimo -AND-
    
    
 
    $result = mysql_query("SELECT COUNT(acciones.IdAccion)as total ,sum(cursos.precio) as ventas from accion_alumno,acciones, cursos,alumnos,clientes,usuarios where clientes.IdUsuario=usuarios.IdUsuario and clientes.IdCliente=alumnos.IdCliente and accion_alumno.IdAlumno=alumnos.IdAlumno and accion_alumno.IdAccion=acciones.IdAccion and acciones.IdCurso=cursos.IdCurso   and '$filtros'", $conexion); 
$row = @mysql_fetch_array($result);
$total=$row['total'];
$ventasA=$row['ventas'];
$tampag = 100;
 
 
$reg1 = ($pag-1) * $tampag;
 
 
 
 
 
  
    $consulta="select  cursos.*, alumnos.nombre, alumnos.ap1, alumnos.ap2,clientes.entidad,accion_alumno.*,usuarios.nombre as com, clientes.provincia, acciones.*, accion_alumno.estado as estado1,accion_alumno.regalo as regalo2,usuarios.ap1 as com1 ,usuarios.IdParent,usuarios.IdPerfil from accion_alumno,acciones, cursos,alumnos,clientes,usuarios where clientes.IdUsuario=usuarios.IdUsuario and clientes.IdCliente=alumnos.IdCliente and accion_alumno.IdAlumno=alumnos.IdAlumno and accion_alumno.IdAccion=acciones.IdAccion and acciones.IdCurso=cursos.IdCurso  and '$filtros'  order by '$orden'  limit $reg1,$tampag";
    $result=mysql_query($consulta) or die ("Error al Ejecutar la Consulta: ".mysql_error());
 

[dashtrash]

Código PHP:

and '$filtros'
 

Seguro que todo el contenido de $filtros va entre comillas?

[diegesis]

Gracias por responder, lo primero.

No entiendo lo de que si todo el contenido de $filtros va entre comillas...Te puedo añadir que en el servidor con el que trabajaba antes funcionaba perfectamente, no así en este. ¿Alguna idea?

[neurotoxina]

Iniciado por diegesis

Gracias por responder, lo primero.

No entiendo lo de que si todo el contenido de $filtros va entre comillas...Te puedo añadir que en el servidor con el que trabajaba antes funcionaba perfectamente, no así en este. ¿Alguna idea?

Mmmmm... prueba volver a tu servidor o cambiar a un par mas. Si vez que la mano no va, es un error de codigo. Ami me paso que en una casa me andaba el codigo y en otra casa no o.O .

[diegesis]

Se que es un error de código...¡Pero no lo ubico! No se porqué me aparece porque aparentemente el código es impoluto...Sugerencias por favor...

[rodno]

Buenas,

Imprime la consulta, para ver como quedó conformada despues de agregarle todas las variables,

Así tendras una mejor vision de como es que se está ejecutando tu consulta,

Saludos

[gnzsoloyo]

Normalmente errores de esas características se producen al crear dinámicamente las consultas sin tener cuidado en cómo queda redactado el SQL al final. Ten en cuenta que el SQL es un lenguaje independiente de PHP, con sus propias reglas y sintaxis.
Lo más probable es que a nivel de PHP, el error sea invisible porque sólo se manifiesta una vez creada la consulta en SQL.

Captura la sentencia que se ejecuta (un echo sería suficiente), y posteala aquí.

[dashtrash]

Vuelvo con lo que dije antes.Lo que hay en $filtros no va entre comillas.
En $filtros estás metiendo cosas como:

Código PHP:

$filtros.="accion_alumno.estado<>'ANULADO' AND accion_alumno.IdAccion in (select det_fra_emp.IdAccion from facturas_empresas,det_fra_emp where facturas_empresas.IdFactura=det_fra_emp.IdFactura and (facturas_empresas.estado='PTE. PAGO' or facturas_empresas.estado LIKE 'IMPAGADO%')) AND ";
 

Y al final del código, estás haciendo

Código PHP:

... and '$filtros'" ...
 

Con lo que todo el contenido de $filtros va a ir entrecomillado.Y eso no es lo que quieres.
Quita las comillas alrededor de $filtros.

Y, en general, es mucho mejor que en vez de

Código PHP:

$result = mysql_query("SELECT COUNT(acciones.IdAccion)as total.....
 

Escribas:

Código PHP:

$query="SELECT COUNT(acciones.IdAccion)as total.....";
 $result = mysql_query($query)
 

Asi , si tienes un problema con la query, haciendo echo $query la puedes examinar.

[diegesis]

Quitadas las comillas de $filtros y seguimos igual.

Cuando selecciono todas las variables del formulario este es el error:
Error al Ejecutar la Consulta: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'ANULADO' AND cursos.ref like '%01%' AND cursos.denominacion like '%cursoprueba%'' at line 1

...

[gnzsoloyo]

Por favor, ¿puedes capturar la sentencia que se envia a ejecución y postearla acá?
Sin ver lo que MySQL recibe realmente, es imposible saber exactamente en qué punto se genera el problema.

[diegesis]

http://www.miweb.com/INTRANET/ventas.php?ref=&denominacion=&horas=&tiporegalo=&I dArea=&provincia=&estado=&ffin=&anoffin=&fcreacion =&anofcreacion=&IdUsuario=&ffin2=&fcreacion2=&orde n=

¿Esto es lo que necesitas? No se si te he entendido...

[gnzsoloyo]

No.
Estoy hablando de la consulta en SQL, que es lo que MySQL recibe.
Más allá de que la creación dinámica de las sentencias puede plantear problemas, la principal fuente de error que veo, luego de estructurar el SQL se ven errores de construcción (simples, pero evidentes):

Código SQL:

SELECT 
    COUNT(acciones.IdAccion)AS total ,
    SUM(cursos.precio) AS ventas 
FROM 
    accion_alumno,
    acciones, 
    cursos,
    alumnos,
    clientes,
    usuarios 
WHERE 
    clientes.IdUsuario=usuarios.IdUsuario 
    AND clientes.IdCliente=alumnos.IdCliente 
    AND accion_alumno.IdAlumno=alumnos.IdAlumno 
    AND accion_alumno.IdAccion=acciones.IdAccion 
    AND acciones.IdCurso=cursos.IdCurso   
    AND '$filtros'    -- Esto está mal
 

Esto también tiene errores:

Código SQL:

SELECT  
    cursos.*, 
    alumnos.nombre, 
    alumnos.ap1, 
    alumnos.ap2,
    clientes.entidad,
    accion_alumno.
    *,usuarios.nombre AS com, 
    clientes.provincia, 
    acciones.*, 
    accion_alumno.estado AS estado1,
    accion_alumno.regalo AS regalo2,
    usuarios.ap1 AS com1 ,
    usuarios.IdParent,
    usuarios.IdPerfil 
FROM 
    accion_alumno,
    acciones, 
    cursos,
    alumnos,
    clientes,
    usuarios 
WHERE     
    clientes.IdUsuario=usuarios.IdUsuario 
    AND clientes.IdCliente=alumnos.IdCliente 
    AND accion_alumno.IdAlumno=alumnos.IdAlumno 
    AND accion_alumno.IdAccion=acciones.IdAccion 
    AND acciones.IdCurso=cursos.IdCurso  
    AND '$filtros'  -- Esto esta mal. 
ORDER BY '$orden'      -- Esto está mal.
LIMIT $reg1,$tampag
 

En principio el error que se ve es simple: lo que agregas luego como condiciones adicionales queda encerrado entre apóstrofos y es interpretado por MySQL como una cadena de texto.
Si además de eso, dentro de esa cadena de texto hay apóstrofos, se produce una interrupción multiple de la misma, pudiendo quedar mal construida.
es decir, el resultado podría ser sencillamente algo como:

Código SQL:

AND 'usuario.id='3256' AND alumno.alimno_id='245''
 

Como puedes ver, la estructura SQL queda completamente mal construida.

Además, estás poniendo entre apostrofos también lo que va en el ORDER BY, lo que s un error, porque esa cláusula espera recibir o nombres de campos, o bien la posicion de la columna a usar de ordenamiento... niguna de las cosa que en realida recibe,.

Obviamente todo eso sería muy fácil de ver al instante, si al menos hicieras lo que te pedí: Postear la consulta creada por el script y que envías a ejecutar... Cosa que no has hecho.

[jonni09lo]

gnzsoloyo también pensé exactamente lo mismo, pero estaba esperando hasta que el susodicho haga lo que se le pidió, ademas dashtrash le dijo como hacerlo...

diegesis si lo que comento el colega arriba no te sirvió, has esto (explicación para dummies?)

Código PHP:

$query = "SELECT COUNT(acciones.IdAccion)as total ,sum(cursos.precio) as ventas from accion_alumno,acciones, cursos,alumnos,clientes,usuarios where clientes.IdUsuario=usuarios.IdUsuario and clientes.IdCliente=alumnos.IdCliente and accion_alumno.IdAlumno=alumnos.IdAlumno and accion_alumno.IdAccion=acciones.IdAccion and acciones.IdCurso=cursos.IdCurso   and '$filtros'";
echo $query; //ESTE ES UNO DE LOS VALORES QUE SE ESTAN PIDIENDO
$result = mysql_query($query, $conexion); 
$row = @mysql_fetch_array($result);
$total=$row['total'];
$ventasA=$row['ventas'];
$tampag = 100;
$reg1 = ($pag-1) * $tampag;
$consulta="select  cursos.*, alumnos.nombre, alumnos.ap1, alumnos.ap2,clientes.entidad,accion_alumno.*,usuarios.nombre as com, clientes.provincia, acciones.*, accion_alumno.estado as estado1,accion_alumno.regalo as regalo2,usuarios.ap1 as com1 ,usuarios.IdParent,usuarios.IdPerfil from accion_alumno,acciones, cursos,alumnos,clientes,usuarios where clientes.IdUsuario=usuarios.IdUsuario and clientes.IdCliente=alumnos.IdCliente and accion_alumno.IdAlumno=alumnos.IdAlumno and accion_alumno.IdAccion=acciones.IdAccion and acciones.IdCurso=cursos.IdCurso  and '$filtros'  order by '$orden'  limit $reg1,$tampag";
echo $consulta; //ESTE ES EL OTRO DE LOS VALORES QUE SE ESTAN PIDIENDO
$result=mysql_query($consulta) or die ("Error al Ejecutar la Consulta: ".mysql_error());
 

Saludos

PD: En caso de que no te funcione y no hagas esto, por mi caso me doy por vencido con usted...

[diegesis]

Gracias por contestar y por evidente que te parezca, tengo tal lío que no me estoy enterando de nada...Voy a probar todo lo que me dices aunque como he puesto en el post anteior, todas las comillas de $filtros y $orden ya las había quitado y sigue dando el mismo error.

Gracias igualmente

[gnzsoloyo]

Pregunta: ¿Puedes capturar la consulta creada en el script o no?
¿Sabes hacerlo?
¿Lo has intentado?

Ya es la tercera vez que te lo pido.

¿Puedes, por favor intentarlo?

Entiende que sin ver exactamente cómo está escrita la consulta que MySQL recibe, no es posible saber en qué punto se genera el error y a qué error se refiere.
MySQL te devuelve el último segmento interpretable de la consulta, pero eso no significa que allí esté el error. Puede ser mucho antes o mucho después. Sin ver el SQL real que le llega, no podemos saberlo. No hacemos magia.

[jonni09lo]

Iniciado por gnzsoloyo

Pregunta: ¿Puedes capturar la consulta creada en el script o no?
¿Sabes hacerlo?

El comentario que expuse arriba le deja como hacerlo en caso de que no sepa hacerlo...

Ademas no se si aquí nos expresamos tan mal que haya que pedir tanto las cosas...

Saludos

[dashtrash]

diegesis...Si no te estás enterando de nada, mal vamos...Yo no sé explicarme mejor.
Si lo que esperas es simplemente que se te diga qué cosa poner o quitar, tú hacerlo tal cual se te diga, y volver a postear "no me funciona", también mal vamos.
Sea lo que sea lo que te está pasando, es algo cuyo tiempo de arreglo es aproximadamente de 10 segundos.Lo que me extraña es que estés manipulando una query grande...y las cosas tan extraordinariamente simples que se te piden..no las entiendas..

Qué parte, exactamente de HAZ UN ECHO DE LA QUERY es dificil de entender??

[rodno]

Buenas,

Lo que hemos intendo decirte todos es que hagas esto:

Código PHP:

  echo $query; // Simplemente esto 
   

Saludos

[diegesis]

Gracias por todo. ¡Lo hemos solucionado!

Gracias por todos los aportes...Al hacer el echo, nos hemos dado cuenta del error. Eran las comillas de $filtros, que entre tanto ir ir venir, nunca terminábamos de quitarlas en todos los sitios...Pero gracas otra vez y perdonar a la dummie...