Changeset 109580 in spip-zone

Timestamp:

Mar 18, 2018, 10:40:03 AM (3 years ago)

Author:

bystrano@…

Message:

Évite des erreurs MySQL 1146

Dans le pipeline declarer_objets_sql, il ne faut surtout pas utiliser de
fonctions qui appellent lister_objets_sql() - comme p.ex. table_objet() -
puisque ça revient à appeler le pipeline depuis le pipeline…

Merci à Placido pour le signalement !

Location:

_plugins_/logos_roles/trunk

Files:

• base/logos_roles.php (modified) (2 diffs)
• logos_roles_fonctions.php (modified) (3 diffs)

_plugins_/logos_roles/trunk/base/logos_roles.php

@@ -139 +139 @@
 function logos_roles_declarer_tables_objets_sql($tables) {
 
+        $noms_tables = array_filter(array_keys($tables), 'is_string');
+
         include_spip('logos_roles_fonctions');
         include_spip('base/objets');
 
-        $roles_logos = lister_roles_logos();
+        $roles_logos = lister_roles_logos(null, null, $noms_tables);
 
         if (is_array($roles_logos)) {
@@ -151 +153 @@
                         $nouveaux_roles_titres[$role] = $options['label'];
                         foreach ($options['objets'] as $objet) {
-                                $nouveaux_roles_objets[table_objet($objet)][] = $role;
+                                // Une ré-implémentation naïve de la fonction table_objet(), qu'on n'a
+                                // pas le droit d'utiliser ici, parce qu'elle appelle indirectement ce
+                                // pipeline.
+                                foreach ($noms_tables as $nom_table) {
+                                        if (preg_match("/$objet/", $nom_table) === 1) {
+                                                $table_objet = $nom_table;
+                                                $nouveaux_roles_objets[$table_objet][] = $role;
+                                                break;
+                                        }
+                                }
                         }
                 }

_plugins_/logos_roles/trunk/logos_roles_fonctions.php

@@ -27 +27 @@
  *
  * @param string|null $objet : Un nom d'objet auquel se restreindre. La fonction
- *        ne retourne alors que les rôles de logos que l'on peut attribuer à cet
- *        objet.
+ *     ne retourne alors que les rôles de logos que l'on peut attribuer à cet
+ *     objet.
  * @param string|null $role : Un rôle auquel se restreindre. On accepte `on` ou
- * `off` pour la rétro-compatibilité
+ *     `off` pour la rétro-compatibilité
+ * @param string|null $tous_les_objets : Une liste des objets éditoriaux à
+ *     prendre en compte. Ça n'est nécessaire que dans le cas où l'on appelle
+ *     cette fonction à un moment où l'API lister_tables_objets_sql n'est pas
+ *     encore disponible, notamment dans le pipeline declarer_tables_objets_sql.
+ *     Dans les autres cas on utilise l'API pour trouver les tables, pas besoin
+ *     de les spécifier.
  *
  * @return array : Retourne le tableau décrivant les rôles de logos. Si on a
- * passé un paramètre rôle, on retourne directement la définition plutôt qu'une
- * liste avec un seul rôle.
- */
-function lister_roles_logos($objet = null, $role = null) {
+ *     passé un paramètre rôle, on retourne directement la définition plutôt
+ *     qu'une liste avec un seul rôle.
+ */
+function lister_roles_logos($objet = null, $role = null, $tous_les_objets = null) {
 
         if ($role === 'on') {
@@ -44 +50 @@
         }
 
-        $tous_les_objets = array_map('table_objet', array_keys(lister_tables_objets_sql()));
+        if (! $tous_les_objets) {
+                $tous_les_objets = array_map(
+                        'table_objet',
+                        array_filter(array_keys(lister_tables_objets_sql()))
+                );
+        }
 
         // Logos par défaut
@@ -71 +82 @@
 
         /* Filtrer par objet */
-        if ($objet = table_objet($objet)) {
+        if ($objet and $objet = table_objet($objet)) {
                 $roles_logos_objet = array();
                 foreach ($roles_logos as $cle_role => $options) {