Problemas con una sentencia SQL

[jachavez]

Hola a todos estoy haciendo una funcion que me permita hacer busquedas de acuerdo al mes y al año de una fecha pero cuadno hago la sentencia sql me da error y no encuentro que puede ser alguno me puede dar una mano
este es el codigo:

Código PHP:

 function get_predicas($anio="", $mes="") 
    { 
      $strsql="select * from predicas where substr(fecha,4,2)=".$mes. " and  substr(fecha,7,4)=".$anio. " order by fecha"; 
      echo $strsql; 
      $result=mysql_query($strsql); 
    ?> 
        <table width="100%" border="0" cellspacing="0" cellpadding="0"> 
          <tr class="ts">  
            <td><div align="center">Id</div></td> 
            <td><div align="center">Fecha</div></td> 
            <td><div align="center">Pastor</div></td> 
            <td><div align="center">Predica</div></td> 
            <td><div align="center">Archivo</div></td> 
            <td><div align="center">Estado</div></td> 
            <td><div align="center">Opciones</div></td> 
          </tr> 
   <?  for ($i=1; $row = mysql_fetch_row ($result); $i++)  { ?> 
          <tr class="ts">  
            <td><?=$row[0]?></td> 
            <td><?=$row[1]?></td> 
            <td><?=$row[2]?></td> 
            <td><?=$row[3]?></td> 
            <td><?=$row[4]?></td> 
            <td><?=$row[5]?></td> 
            <td>Editar</td> 
          </tr> 
    <?    } ?> 
        </table> 
<? 
    }   

[Ayalnet]

No le has indicado si lo ordena accendente o descendente:
Prueba esto:
$strsql="select * from predicas where substr(fecha,4,2)=".$mes. " and substr(fecha,7,4)=".$anio. " order by fecha ASC";

o Bien:
$strsql="select * from predicas where substr(fecha,4,2)=".$mes. " and substr(fecha,7,4)=".$anio. " order by fecha DESC";

Un Saludo
Espero que te sirva

[nicolaspar]

substr??
Tenes:
SUBSTRING(str,pos)
SUBSTRING(str FROM pos)
SUBSTRING(str,pos,len)
SUBSTRING(str FROM pos FOR len)
SUBSTRING_INDEX(str,delim,count)

PD: por que usar :
<td><div align="center">Opciones</div></td> ??
eso es basura para el html, mejor hace:
<td align="center">Opciones</td> ;)