Re: [Devel] r490 - branches/generic

[Ploc <plub2010 AT acampado.net>]

Bonjour,

Sur ma lancée, j'ai continué à corriger les problèmes qui apparaissaient
sur les flux rss de l'agenda du libre.

Il y avait en effet un problème sur l'affichage du flux rss pour les noms
de villes qui contiennent des apostrophes ("'"). En effet, ces caractères
spéciaux pour mysql n'étaient pas échappés et cela pourrissait la requête
sql, qui échouait, puis le flux rss :

#   <rdf:li
rdf:resource="http://agenda.survie.org/dev/showevent.php?id=9"/>
#  </rdf:Seq>
# </items>
# </channel>
#  
#  
# Probleme dans la syntaxe de select longitude, latitude from cities
where name='l'Isle d'Abeau' : 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 'Isle d'Abeau'' at line 1
#  <item rdf:about="http://agenda.survie.org/dev/showevent.php?id=11";>
#  <title>l&#039;Isle d&#039;Abeau : concert de Eric Toulis, du lundi 15
novembre 2010 au samedi 27 novembre 2010</title>

Le problème a été corrigé en échappant les caractères spéciaux du nom des
villes dans la fonction get_city_coordinates en utilisant la fonction php
mysql_real_escape_string.

Corrigé dans la branche generic.

Ploc


On Tue,  9 Nov 2010 20:17:07 +0100 (CET), svn AT agendadulibre.org wrote:
> Author: ploc
> Date: Tue Nov  9 20:17:06 2010
> New Revision: 490
> 
> Log:
> Fixing bug where rss feed was outputing mysql error when city name was
> containing characters such as [']. The fix consist in escaping mysql
> parameters using the mysql_real_escape_string php function in the
> get_city_coordinates agenda function.
> 
> Modified:
>    branches/generic/rss.php
> 
> Modified: branches/generic/rss.php
>
==============================================================================
> --- branches/generic/rss.php  Tue Nov  9 20:06:04 2010        (r489)
> +++ branches/generic/rss.php  Tue Nov  9 20:17:06 2010        (r490)
> @@ -68,7 +68,7 @@
>  
>  function get_city_coordinates ($db, $city)
>  {
> -  $sql = "select longitude, latitude from cities where name='" . $city
.
> "'";
> +  $sql = "select longitude, latitude from cities where name='" .
> mysql_real_escape_string($city, $db) . "'";
>    $ret = $db->query($sql);
>  
>    if (mysql_num_rows($ret) == 1)
> _______________________________________________
> Devel mailing list
> Devel AT agendadulibre.org
> http://agendadulibre.org/cgi-bin/mailman/listinfo/devel