Changeset 106414 in spip-zone for _plugins_/lim


Ignore:
Timestamp:
Sep 25, 2017, 1:36:26 PM (4 years ago)
Author:
peetdu@…
Message:

Optimisation de code
+ documentation plus claire
+ PSR

Location:
_plugins_/lim/trunk
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • _plugins_/lim/trunk/formulaires/configurer_lim_objets.html

    r95028 r106414  
    1010                #ACTION_FORMULAIRE{#ENV{action}}
    1111                #SET{name,lim_objets}#SET{erreurs,#ENV**{erreurs}|table_valeur{#GET{name}}}
    12                 [(#SET{exclus, #REM|lim_objets_sans_rubrique})]
     12                [(#SET{exclus, #REM|lim_objets_a_exclure})]
    1313               
    1414                #INCLURE{fond=formulaires/inc-choisir-objets,name=#GET{name},selected=#ENV**{#GET{name}},exclus=#GET{exclus}}
  • _plugins_/lim/trunk/formulaires/configurer_lim_objets.php

    r93319 r106414  
    11<?php
    22
    3 /***************************************************************************\
    4  *  SPIP, Systeme de publication pour l'internet                           *
    5  *                                                                         *
    6  *  Copyright (c) 2001-2014                                                *
    7  *  Arnaud Martin, Antoine Pitrou, Philippe Riviere, Emmanuel Saint-James  *
    8  *                                                                         *
    9  *  Ce programme est un logiciel libre distribue sous licence GNU/GPL.     *
    10  *  Pour plus de details voir le fichier COPYING.txt ou l'aide en ligne.   *
    11 \***************************************************************************/
    12 
    13 if (!defined('_ECRIRE_INC_VERSION')) return;
     3if (!defined('_ECRIRE_INC_VERSION')) {
     4        return;
     5}
    146
    157include_spip('inc/config');
  • _plugins_/lim/trunk/inc/lim_api.php

    r96523 r106414  
    1010 */
    1111
    12 if (!defined('_ECRIRE_INC_VERSION')) return;
     12if (!defined('_ECRIRE_INC_VERSION')) {
     13        return;
     14}
    1315
    1416/**
     
    106108
    107109/**
    108  * chercher les tables SPIP qui ne gèrent pas de rubrique, et donc non pertinentes dans la restriction par rubrique
    109  * gestion des tables historiques également : annuaire de site et brèves activés ?
     110 * Construire la liste des objets à exclure
     111 * les objets SPIP qui ne sont jamais listés dans rubrique, et donc non pertinents dans la restriction par rubrique.
     112 *
     113 * exception : pour les brèves et les sites, on vérifie qu'elles ont été activées
     114 * exception : les documents si ceux-ci ont été activés dans les rubriques (menu Configuration -> Contenu du site -> paragraphe Documents joints)
    110115 *
    111116 * @return array
    112117 *      tableau des nom de tables SPIP à exclure (ex : spip_auteurs, spip_mots, etc.)
    113118 */
    114 function lim_objets_sans_rubrique() {
     119function lim_objets_a_exclure() {
    115120        $exclus = array();
    116121        $tables = lister_tables_objets_sql();
     
    122127        // Exception pour les objets breves et sites : sont-ils activés
    123128        if (lire_config('activer_breves') == 'non')
    124                 array_push($exclus,'spip_breves');
     129                array_push($exclus, 'spip_breves');
    125130        if (lire_config('activer_sites') == 'non')
    126                 array_push($exclus,'spip_syndic');
     131                array_push($exclus, 'spip_syndic');
    127132
     133        // Exception pour les documents (si ils ont été activés pour les rubriques)
     134        $document_objet = lire_config('documents_objets');
     135        if (strpos($document_objet, 'spip_rubriques')) {
     136                $key = array_search('spip_documents', $exclus);
     137                unset($exclus[$key]);
     138        }
    128139        return $exclus;
    129140}
    130141
    131142/**
    132  * récupérer le tableau des rubriques dans lesquelles il est possible d'editer un objet
     143 * Récupérer la liste des rubriques dans lesquelles il est possible de créer l'objet demandé
    133144 *
    134145 * @param string $type
     
    140151        $tab_rubrique_objet = lire_config("lim_rubriques/$type");
    141152
    142         // si aucune restriction on sort.
    143         if (is_null($tab_rubrique_objet)) return $rubriques_choisies;
     153        // si l'objet n'est pas dans la config, on sort
     154        if (is_null($tab_rubrique_objet)) {
     155                return $rubriques_choisies;
     156        }
    144157
    145158        $res = sql_allfetsel('id_rubrique', 'spip_rubriques');
    146         foreach ($res as $key => $value) {
    147                 $tab_rubriques[] = $value['id_rubrique'];
    148         }
     159        $tab_rubriques = array_column($res, 'id_rubrique');
    149160        $rubriques_choisies = array_diff($tab_rubriques,$tab_rubrique_objet);
    150161        return $rubriques_choisies;
     
    158169 */
    159170function lim_type($tableau) {
    160         if (!is_array($tableau))
     171        if (!is_array($tableau)) {
    161172                return '';
     173        }
    162174
    163175        array_walk($tableau, 'lim_get_type');
     
    166178
    167179/**
     180 * fonction callback pour lim_type
    168181 * Changer les valeurs du tableau spip_articles -> article
    169182 */
     
    171184        $value = objet_type(table_objet($key));
    172185}
    173 
    174 /**
    175  * renvoyer le nombre de rubrique auxquelles est
    176  */
    177 function lim_nombre_rubrique($objet) {
    178         $value = objet_type(table_objet($key));
    179 }
    180 
    181 ?>
Note: See TracChangeset for help on using the changeset viewer.