il semblerait que la fonction sql_error() ait changé de comportement entre SPIP 2.0 et SPIP 2.1 lorsqu'elle est appelée suite à une requête qui s'est bien passé:
- en 2.0 la fonction se comportait comme son homologue en PHP mysql_error(): si pas d'erreur dans la requête précédant l'appel de la fonction => retour vide
- en 2.1 en revanche si pas d'erreur => la fonction retourne *la requête SQL*
Du coup ça casse la compatibilité pour ceux qui utilisaient cette fonction pour faire des contrôles d'erreur avec une syntaxe du style:
if (sql_error() != '')
die((_T('median:erreur_install_groupe_technique ')).sql_error());
Alors certes il ne faut pas rester bloqué sur un fonctionnement existant, évoluer avec les versions, etc... si on ne veut pas fossiliser le truc (et passer pour un vieux c***) mais là je trouve ça vraiment *pas gentil* pour les pôv codeurs de plugins qui prenaient bien soin de gérer "proprement" les erreurs dans leurs requêtes pour être compatible avec tous les types de bases de données et qui s'en retrouvent fort mal récompensés en devant se faire le rechercher/remplacer sur tous leurs plugins pour passer à une syntaxe du style:
if (sql_errno() > '0') ...
Bref, amha c'est une sale rupture de compatibilité pour un bénéfice bien hypothétique... et qui donne un comportement très atypique pour une fonction de contrôle d'erreur
PS: une recherche sur plus d'une 100aine de plugins de la zone montre que la majorité des codeurs de plugins (tous sauf moi en gros) utilisent mysql_error() dans leurs contrôles d'erreurs... Comme quoi il vaudrait mieux coder "salement" pour être tranquille... y'a vraiment pas de morale chez SPIP!