Changeset 106416 in spip-zone for _plugins_/lim

Timestamp:

Sep 25, 2017, 1:40:45 PM (4 years ago)

Author:

peetdu@…

Message:

Optimisation de code
+ début de prise en compte de l’objet Document si activé pour les rubriques
+ documentation plus claire
+ PSR

Location:

_plugins_/lim/branches/V1

Files:

• formulaires/configurer_lim_objets.html (modified) (1 diff)
• formulaires/configurer_lim_objets.php (modified) (1 diff)
• inc/lim_api.php (modified) (7 diffs)
• lang/paquet-lim_fr.php (modified) (1 diff)
• lim_pipelines.php (modified) (5 diffs)
• paquet.xml (modified) (2 diffs)

_plugins_/lim/branches/V1/formulaires/configurer_lim_objets.html

@@ -10 +10 @@
                 #ACTION_FORMULAIRE{#ENV{action}}
                 #SET{name,lim_objets}#SET{erreurs,#ENV**{erreurs}|table_valeur{#GET{name}}}
-                [(#SET{exclus, #REM|lim_objets_sans_rubrique})]
+                [(#SET{exclus, #REM|lim_objets_a_exclure})]
                 
                 #INCLURE{fond=formulaires/inc-choisir-objets,name=#GET{name},selected=#ENV**{#GET{name}},exclus=#GET{exclus}}

_plugins_/lim/branches/V1/formulaires/configurer_lim_objets.php

@@ -1 +1 @@
 <?php
 
-/***************************************************************************\
- *  SPIP, Systeme de publication pour l'internet                           *
- *                                                                         *
- *  Copyright (c) 2001-2014                                                *
- *  Arnaud Martin, Antoine Pitrou, Philippe Riviere, Emmanuel Saint-James  *
- *                                                                         *
- *  Ce programme est un logiciel libre distribue sous licence GNU/GPL.     *
- *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
-\***************************************************************************/
-
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+        return;
+}
 
 include_spip('inc/config');

_plugins_/lim/branches/V1/inc/lim_api.php

@@ -10 +10 @@
  */
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+        return;
+}
 
 /**
@@ -106 +108 @@
 
 /**
- * chercher les tables SPIP qui ne gèrent pas de rubrique, et donc non pertinentes dans la restriction par rubrique
- * gestion des tables historiques également : annuaire de site et brèves activés ?
+ * Construire la liste des objets à exclure
+ * les objets SPIP qui ne sont jamais listés dans rubrique, et donc non pertinents dans la restriction par rubrique.
+ *
+ * exception : pour les brèves et les sites, on vérifie qu'elles ont été activées
+ * exception : les documents si ceux-ci ont été activés dans les rubriques (menu Configuration -> Contenu du site -> paragraphe Documents joints)
  * 
  * @return array
  *      tableau des nom de tables SPIP à exclure (ex : spip_auteurs, spip_mots, etc.)
  */
-function lim_objets_sans_rubrique() {
+function lim_objets_a_exclure() {
         $exclus = array();
         $tables = lister_tables_objets_sql();
@@ -122 +127 @@
         // Exception pour les objets breves et sites : sont-ils activés
         if (lire_config('activer_breves') == 'non')
-                array_push($exclus,'spip_breves');
+                array_push($exclus, 'spip_breves');
         if (lire_config('activer_sites') == 'non')
-                array_push($exclus,'spip_syndic');
+                array_push($exclus, 'spip_syndic');
 
+        // Exception pour les documents (si ils ont été activés pour les rubriques)
+        $document_objet = lire_config('documents_objets');
+        if (strpos($document_objet, 'spip_rubriques')) {
+                $key = array_search('spip_documents', $exclus);
+                unset($exclus[$key]);
+        }
         return $exclus;
 }
 
 /**
- * récupérer le tableau des rubriques dans lesquelles il est possible d'editer un objet
+ * Récupérer la liste des rubriques dans lesquelles il est possible de créer l'objet demandé
  * 
  * @param string $type
@@ -140 +151 @@
         $tab_rubrique_objet = lire_config("lim_rubriques/$type");
 
-        // si aucune restriction on sort.
-        if (is_null($tab_rubrique_objet)) return $rubriques_choisies;
+        // si l'objet n'est pas dans la config, on sort
+        if (is_null($tab_rubrique_objet)) {
+                return $rubriques_choisies;
+        } 
 
         $res = sql_allfetsel('id_rubrique', 'spip_rubriques');
-        foreach ($res as $key => $value) {
-                $tab_rubriques[] = $value['id_rubrique'];
-        }
+        $tab_rubriques = array_column($res, 'id_rubrique');
         $rubriques_choisies = array_diff($tab_rubriques,$tab_rubrique_objet);
         return $rubriques_choisies;
@@ -158 +169 @@
  */
 function lim_type($tableau) {
-        if (!is_array($tableau))
+        if (!is_array($tableau)) {
                 return '';
+        }
 
         array_walk($tableau, 'lim_get_type');
@@ -166 +178 @@
 
 /**
+ * fonction callback pour lim_type
  * Changer les valeurs du tableau spip_articles -> article
  */
@@ -171 +184 @@
         $value = objet_type(table_objet($key));
 }
-
-/**
- * renvoyer le nombre de rubrique auxquelles est 
- */
-function lim_nombre_rubrique($objet) {
-        $value = objet_type(table_objet($key));
-}
-
-?>

_plugins_/lim/branches/V1/lang/paquet-lim_fr.php

@@ -13 +13 @@
         - <b>Logos</b> : si vous utiliser les logos, vous pourrez alors désactiver leur affichage au cas par cas (articles, auteurs, mots-clés, etc.),
 
-        - <b>Contenus par rubriques</b> : Choisir les rubriques où désactiver l\'édition d\'articles, brèves et sites référencés.',
+        - <b>Contenus par rubriques</b> : Choisir les rubriques où désactiver l\'édition d\'articles, brèves, sites référencés et autres objets éditoriaux.',
         'lim_nom' => 'Lim',
         'lim_slogan' => 'Alléger l\'espace privé de blocs et boutons inutiles dans votre projet',

_plugins_/lim/branches/V1/lim_pipelines.php

@@ -10 +10 @@
  */
 
-if (!defined('_ECRIRE_INC_VERSION')) return;
+if (!defined('_ECRIRE_INC_VERSION')) {
+        return;
+}
 
 include_spip('inc/lim_api');
@@ -42 +44 @@
 
 /**
- * gestion des contenus par rubrique : rediriger la creation d'un objet vers la bonne rubrique si celle-ci est pris en compte par LIM
+ * Gestion des contenus par rubrique : si pour un objet A la conf dans LIM ne laisse qu'un seule rubrique dans laquelle cet objet peut être éditer, rediriger l'enregistrement vers cette rubrique en renvoyant l'id_parent
  * ce traitement est rendu nécessaire par 
  * -> l'action de la fonction inc/lim_api.php -> inc_chercher_rubrique qui supprime l'affichage du sélecteur de rubrique si une seule rubrique
@@ -61 +63 @@
         $valid  = strpos($form, 'editer');
         // si ce n'est pas un formulaire d'édition 
-        //ou si la restriction par rubrique n'a pas été activée, on sort
-        if ($valid === false OR is_null(lire_config('lim_objets'))) return $flux;
-
+        // ou si la restriction par rubrique n'a pas été activée, on sort
+        if ($valid === false OR is_null(lire_config('lim_objets'))) {
+                return $flux;
+        }
 
         $type                           = substr($form, 7); // 'editer_objet' devient 'objet'
@@ -83 +86 @@
 
 /**
- * gestion des contenus par rubrique : vérifier si on à le droit de publier l'objet dans cette rubrique
+ * Gestion des contenus par rubrique : vérifier si on a le droit de publier l'objet dans cette rubrique
  * en fonction des rubriques décochées dans la page exec=configurer_lim_rubriques
  *
@@ -153 +156 @@
         return $flux;
 }
-
-?>

_plugins_/lim/branches/V1/paquet.xml

@@ -2 +2 @@
         prefix="lim"
         categorie="maintenance"
-        version="1.2.5"
+        version="1.2.6"
         etat="stable"
         compatibilite="[3.0.0;3.2.*]"
@@ -15 +15 @@
         <credit>Logo : LeChat</credit>
         <credit>Selecteur On/Off : Roland Chenel</credit>
-
 
         <licence>GNU/GPL</licence>