Pas sûr que
- $id_form = sql_insertq('spip_forms',array('titre'=>_q($titre)));
+ spip_query("INSERT INTO spip_forms (titre) VALUES ("._q($titre).")");
+ $id_form = mysql_insert_id();
soit franchement une bonne idée.
En effet, mysql_insert_id est une fonction de php propre à mysql, ce qui fait perdre la couche d'abstraction SQL de SPIP
Pas sûr que
- $id_form = sql_insertq('spip_forms',array('titre'=>_q($titre)));
+ spip_query("INSERT INTO spip_forms (titre) VALUES ("._q($titre).")");
+ $id_form = mysql_insert_id();
soit franchement une bonne idée.
En effet, mysql_insert_id est une fonction de php propre à mysql, ce qui
fait perdre la couche d'abstraction SQL de SPIP
Et j'avais même pas fait attention au spip_query !
Pour l'instant, du point de vue des requêtes, c'est une régression
je vois en effet beaucoup de retour en arrières dans ce commit :
- des $commencer_page qui sont redevenus debut_page() (obsolete)
- des sql_count qui sont redevenus spip_num_rows() (obsolete)
- la compat avec le plugin bando qui a sauté
- des boutons_block_invisible qui font leur retour (obsolete)
- des spip_fetch_array et spip_abstract_fetch qui font leur retour (obsoletes)
De toute évidence, si cette version apportait quelques corrections manquantes, elle était globalement bien moins avancée dans le portage.
Tu vas te prendre une tripotée de régressions et de bugs qui étaient résolus.
Je pense qu'il faut revenir à ta version précédente, et reprendre le merge plus proprement et calmement en prenant le temps de bien tout relire.
Cédric
Le 18 févr. 2010 à 10:26, RealET a écrit :
* RealET tapuscrivait, le 18/02/2010 10:20:
* jerome.escoffier@gmail.com tapuscrivait, le 18/02/2010 10:07:
Author: jerome.escoffier@gmail.com
Date: 2010-02-18 10:07:30 +0100 (Thu, 18 Feb 2010)
New Revision: 35297
Pas sûr que
- $id_form = sql_insertq('spip_forms',array('titre'=>_q($titre)));
+ spip_query("INSERT INTO spip_forms (titre) VALUES ("._q($titre).")");
+ $id_form = mysql_insert_id();
soit franchement une bonne idée.
En effet, mysql_insert_id est une fonction de php propre à mysql, ce qui
fait perdre la couche d'abstraction SQL de SPIP
Et j'avais même pas fait attention au spip_query !
Pour l'instant, du point de vue des requêtes, c'est une régression
