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> ;)