Changeset 65543 in spip-zone

Timestamp:

Sep 5, 2012, 7:33:26 AM (7 years ago)

Author:

marcimat@…

Message:

Notices PHP sur #SET_PUSH et #SET_MERGE lors de l'utilisation de Formidable.

• corrections d'indéfinis et de array to string conversion

Location:

_plugins_/spip-bonux-3

Files:

• paquet.xml (modified) (1 diff)
• public/spip_bonux_balises.php (modified) (5 diffs)

_plugins_/spip-bonux-3/paquet.xml

@@ -2 +2 @@
         prefix="spip_bonux"
         categorie="outil"
-        version="3.0.2"
+        version="3.0.3"
         etat="stable"
         compatibilite="[3.0.0-dev;3.1.*]"

_plugins_/spip-bonux-3/public/spip_bonux_balises.php

@@ -14 +14 @@
  * #SET_PUSH{tableau,valeur}
  *
- * @param object $p : objet balise
- * @return ""
+ * @param Champ $p : objet balise
+ * @return Champ
+ *     La balise ne retourne rien ("")
 **/
 function balise_SET_PUSH_dist($p){
@@ -25 +26 @@
                 // on ajoute la valeur ensuite (sans passer par array_push)
                 $p->code = "vide((\$cle=$_nom)
-                        . (is_array(\$Pile['vars'][\$cle])?'':\$Pile['vars'][\$cle]=array())
-                        . (\$Pile['vars'][\$cle][]=$_valeur))";
+                        . vide((!isset(\$Pile['vars'][\$cle]) OR !is_array(\$Pile['vars'][\$cle]))?\$Pile['vars'][\$cle]=array():'')
+                        . vide(\$Pile['vars'][\$cle][]=$_valeur))";
         else
                 $p->code = "''";
@@ -43 +44 @@
  * #SET_MERGE{nom_tableau,#GET{tableau}}
  *
- * @param object $p : objet balise
- * @return ""
+ * @param Champ $p : objet balise
+ * @return Champ
+ *     La balise ne retourne rien ("")
 **/
 function balise_SET_MERGE_dist($p){
@@ -55 +57 @@
                 // si le tableau n'existe pas encore, on le cree
                 $p->code = "vide((\$cle=$_nom)
-                        . (is_array(\$Pile['vars'][\$cle])?'':\$Pile['vars'][\$cle]=array())
-                        . (is_array(\$new=$_t1)?'':\$new=array(\$new))
-                        . (\$Pile['vars'][\$cle] = array_merge(\$Pile['vars'][\$cle],\$new)))";
+                        . vide((!isset(\$Pile['vars'][\$cle]) OR !is_array(\$Pile['vars'][\$cle]))?\$Pile['vars'][\$cle]=array():'')
+                        . vide(is_array(\$new=$_t1)?'':\$new=array(\$new))
+                        . vide(\$Pile['vars'][\$cle] = array_merge(\$Pile['vars'][\$cle],\$new)))";
         elseif ($_nom AND $_t1 AND $_t2)
                 // 3 arguments : merge de $_t1 et $_t2 dans $_nom
                 // si le tableau n'existe pas encore, on le cree
                 $p->code = "vide((\$cle=$_nom)
-                        . (is_array(\$Pile['vars'][\$cle])?'':\$Pile['vars'][\$cle]=array())
-                        . (is_array(\$new1=$_t1)?'':\$new1=array(\$new1))
-                        . (is_array(\$new2=$_t2)?'':\$new2=array(\$new2))
-                        . (\$Pile['vars'][\$cle] = array_merge(\$new1,\$new2)))";
+                        . vide((!isset(\$Pile['vars'][\$cle]) OR !is_array(\$Pile['vars'][\$cle]))?\$Pile['vars'][\$cle]=array():'')
+                        . vide(is_array(\$new1=$_t1)?'':\$new1=array(\$new1))
+                        . vide(is_array(\$new2=$_t2)?'':\$new2=array(\$new2))
+                        . vide(\$Pile['vars'][\$cle] = array_merge(\$new1,\$new2)))";
         else
                 $p->code = "''";
@@ -76 +78 @@
  * Balise #COMPTEUR associee au critere compteur
  *
- * @param unknown_type $p
- * @return unknown
+ * @param Champ $p
+ * @return Champ
  */
 function balise_COMPTEUR_dist($p) {