Changeset 80169 in spip-zone

Timestamp:

Jan 21, 2014, 10:15:11 AM (7 years ago)

Author:

marcimat@…

Message:

• Nombreuses notices PHP en moins
• Changement de stratégie dans la fonction valeur_colonne_table_dist() : on ne calcule la partie where de la requête SQL que si c'est réellement utile, et pour ce faire, on sépare en 2 fonctions le calcul :

-- du nom de la table d'application et de sa/ses clés primaires
-- du where SQL

De la sorte, si un contrôleur calcule lui-même les valeurs de tous ses champs, aucune tentative erronée de créer un where adapté n'est réalisé, ce qui évite des notices PHP lorsque le controleur utilise des noms de champs ne correspondant pas à des colonne SQL.

Ceci permet entre autres de passer des #EDIT{mots_article-128} dans une boucle GROUPES_MOTS et en gérant un contrôleur adapté permettant de sélectionner les mots d'un certain groupe, liés/à lier à l'article 128.

Changement de version

Location:

_plugins_/crayons

Files:

• action/crayons_html.php (modified) (2 diffs)
• action/crayons_store.php (modified) (1 diff)
• inc/crayons.php (modified) (5 diffs)
• paquet.xml (modified) (1 diff)
• plugin.xml (modified) (1 diff)

_plugins_/crayons/action/crayons_html.php

@@ -179 +179 @@
         function Crayon($name, $texts = array(), $options = array(), $c=null) {
                 $this->name = $name;
-                list($this->type, $this->modele, $this->id) = explode('-', $this->name, 3);
+
+                list($this->type, $this->modele, $this->id) = array_pad(explode('-', $this->name, 3), 3, '');
                 list($this->distant,$this->table) = distant_table($this->type);
                 if (is_scalar($texts) || is_null($texts)) {
@@ -317 +318 @@
                                         . '" />'."\n";
                         }
+
                         if (is_array($spec) && isset($spec[$champ]['attrs'])) {
                                 foreach ($spec[$champ]['attrs'] as $attr=>$val) {

_plugins_/crayons/action/crayons_store.php

@@ -38 +38 @@
                                          * depuis jquery 1.5 (feature non documentee de jquery!)
                                          */
-                                        $content[$field] = is_array($_POST['content_'.$crayon.'_'.$field])?implode(',',$_POST['content_'.$crayon.'_'.$field]):$_POST['content_'.$crayon.'_'.$field];
+                                        if (isset($_POST['content_'.$crayon.'_'.$field])) {
+                                                $content[$field] = is_array($_POST['content_'.$crayon.'_'.$field])
+                                                        ?implode(',',$_POST['content_'.$crayon.'_'.$field])
+                                                        :$_POST['content_'.$crayon.'_'.$field];
+                                        } else {
+                                                $content[$field] = null;
+                                        }
                                 }
                         }

_plugins_/crayons/inc/crayons.php

@@ -382 +382 @@
         list($distant,$table) = distant_table($type);
         $nom_table = '';
-        if (!(($tabref = &crayons_get_table($table, $nom_table)) && ($brut = $tabref['field'][$col]))) {
-                return false;
+        if (!(($tabref = &crayons_get_table($table, $nom_table))
+                && isset($tabref['field'][$col])
+                && ($brut = $tabref['field'][$col]))) {
+                        return false;
         }
         $ana = explode(' ', $brut);
@@ -440 +442 @@
         return $ret;
 }
-//      var_dump(colonne_table('forum', 'id_syndic')); die();
+
+
+/**
+ * Obtient le nom de la table ainsi que sa ou ses clés primaires
+ *
+ * @param string $type
+ *     Table sur laquelle s'applique le crayon.
+ *     Ce type peut contenir le nom d'un connecteur distant tel que `{connect}__{table}`
+ *
+ * @return array|bool
+ *     - false si on ne trouve pas de table ou de table ayant de clé primaire
+ *     - liste :
+ *     - - nom de la table sql
+ *     - - tableau des noms de clés primaires
+**/
+function crayons_get_table_name_and_primary($type) {
+        static $types = array();
+        if (isset($types[$type])) {
+                return $types[$type];
+        }
+
+        $nom_table = '';
+        if ($tabref = &crayons_get_table($type, $nom_table)
+          and ($tabid = explode(',', $tabref['key']['PRIMARY KEY'])))
+        {
+                return $types[$type] = array($nom_table, $tabid);
+        }
+        spip_log('crayons: table ' . $type . ' inconnue');
+        return $types[$type] = false;
+}
+
 
 function table_where($type, $id, $where_en_tableau = false) {
-        list($distant,$table) = distant_table($type);
-        $nom_table = '';
-        if (!(($tabref = &crayons_get_table($type, $nom_table))
-                        && ($tabid = explode(',', $tabref['key']['PRIMARY KEY'])))) {
-                spip_log('crayons: table ' . $table . ' inconnue');
+
+
+        if (!$infos = crayons_get_table_name_and_primary($type)) {
                 return array(false, false);
         }
+
+        list($nom_table, $tabid) = $infos;
+
+
         if (is_scalar($id))
                 $id = explode('-', $id);
@@ -460 +494 @@
         // sinon sortie texte pour sql_query
         } else {
+
                 $where = $and = '';
                 foreach ($id as $idcol => $idval) {
@@ -471 +506 @@
 
 function valeur_colonne_table_dist($type, $col, $id) {
-        list($distant,$table) = distant_table($type);
-        list($nom_table, $where) = table_where($type, $id);
-
-        if (!$nom_table)
+
+        // Table introuvable ou sans clé primaire
+        if (!$infos = crayons_get_table_name_and_primary($type)) {
                 return false;
+        }
+        $table = reset($infos);
 
         $r = array();
@@ -488 +524 @@
 
         // valeurs SQL
-        if (count($col)
-        AND $s = spip_query(
-                        'SELECT `' . implode($col, '`, `') .
-                        '` FROM ' . $nom_table . ' WHERE ' . $where, $distant)
-        AND $t = sql_fetch($s))
-                $r = array_merge($r, $t);
+        if (count($col)) {
+                list($distant, $table)   = distant_table($type);
+                list($nom_table, $where) = table_where($type, $id);
+
+                if ($s = spip_query(
+                                'SELECT `' . implode($col, '`, `') .
+                                '` FROM ' . $nom_table . ' WHERE ' . $where, $distant)
+                        AND $t = sql_fetch($s)){
+                                $r = array_merge($r, $t);
+                }
+        }
 
         return $r;

_plugins_/crayons/paquet.xml

@@ -2 +2 @@
         prefix="crayons"
         categorie="edition"
-        version="1.16.6"
+        version="1.17.0"
         etat="stable"
         compatibilite="[1.9.0;3.*.*]"

_plugins_/crayons/plugin.xml

@@ -17 +17 @@
         </licence>
         <version>
-        1.16.6
+        1.17.0
         </version>
         <etat>