Changeset 117941 in spip-zone

Timestamp:

Sep 24, 2019, 11:02:25 PM (13 months ago)

Author:

Charles Razack

Message:

Pétouilles : un peu de rangement dans le fichier d'administration + < INCLURE> au lieu de #INCLURE dans le form de config

Location:

_plugins_/statistiques_objets/trunk

Files:

• . (modified) (2 props)
• formulaires/inc-statistiques_objets.html (modified) (1 diff)
• paquet.xml (modified) (1 diff)
• statsobjets_administrations.php (modified) (2 diffs)

_plugins_/statistiques_objets/trunk/formulaires/inc-statistiques_objets.html

@@ -1 @@
-#SET{name,objets}#SET{erreurs,#ENV**{erreurs/#GET{name}}}
+#SET{name,objets}
+#SET{erreurs,#ENV**{erreurs/#GET{name}}}
 #SET{div,#REM|spip_version|floatval|>={3.1}|?{div,li}}
-<[(#GET{div}) ]class="editer editer_[(#GET{name})][ (#GET{obli})][ (#GET{erreurs}|oui)erreur]">
+<[(#GET{div}) ]class="editer configurer_[(#GET{name})][ (#GET{obli})][ (#GET{erreurs}|oui)erreur]">
         <span class="explication"><:statistiques:cfg_champ_objets_explication:></span>
-        <label><:statistiques:cfg_champ_objets_label:></label>[
-        <span class='erreur_message'>(#GET{erreurs})</span>
-        ]
-        #INCLURE{fond=formulaires/inc-choisir-objets,name=#GET{name},selected=#ENV**{#GET{name}}}
+        <label><:statistiques:cfg_champ_objets_label:></label>
+        [<span class='erreur_message'>(#GET{erreurs})</span>]
+        <INCLURE{fond=formulaires/inc-choisir-objets, name=#GET{name}, selected=#ENV**{#GET{name}}}>
 </[(#GET{div})]>

_plugins_/statistiques_objets/trunk/paquet.xml

@@ -2 +2 @@
         prefix="statsobjets"
         categorie="statistique"
-        version="1.1.1"
+        version="1.1.2"
         etat="test"
         compatibilite="[3.0.0;3.2.*]"

_plugins_/statistiques_objets/trunk/statsobjets_administrations.php

@@ -31 +31 @@
         $maj = array();
 
-        // Ajout de 2 tables pour prendre en compte les statistiques des objets éditoriaux
-        // Ajout des colonnes popularite, visites et referers sur tous les objets
+        // Ajout de 2 tables pour prendre en compte les statistiques des objets éditoriaux.
+        // Ajout des colonnes popularite, visites et referers sur tous les objets.
         $maj['create'] = array(
                 array('maj_tables', array('spip_visites_objets', 'spip_referers_objets')),
-                array('statsobjets_check_upgrade'),
+                array('statsobjets_maj_create'),
         );
-        // Si les stats sont activées, on prend en compte les articles
-        if (lire_config('activer_statistiques') == 'oui'){
-                $maj['create'][] = array('ecrire_config', 'activer_statistiques_objets', array('spip_articles'));
-        }
 
         include_spip('base/upgrade');
         maj_plugin($nom_meta_base_version, $version_cible, $maj);
-
-}
-
-
-/**
- * Une fonction pour vérifier que les champs nécessaires sont bien sur tous les objets,
- * appelée lorsqu'on enregistre la configuration
- * (cas d'un nouvel objet ajouté apres l'install du plugin)
- *
- * @return void
- */
-function statsobjets_check_upgrade(){
-
-        include_spip('base/objets');
-
-        $tables_objets = array_keys(lister_tables_objets_sql());
-        $trouver_table = charger_fonction('trouver_table','base');
-        foreach($tables_objets as $table){
-                $desc = $trouver_table($table);
-                if (!isset($desc['field']['popularite'])){
-                        sql_alter("TABLE $table ADD popularite DOUBLE DEFAULT '0' NOT NULL");
-                }
-                if (!isset($desc['field']['visites'])) {
-                        sql_alter("TABLE $table ADD visites integer DEFAULT '0' NOT NULL");
-                }
-                if (!isset($desc['field']['referers'])) {
-                        sql_alter("TABLE $table ADD referers integer DEFAULT '0' NOT NULL");
-                }
-        }
 
 }
@@ -107 +74 @@
         effacer_meta($nom_meta_base_version);
 }
+
+
+/**
+ * Fonction privée pour l'installation
+ * @return void
+ */
+function statsobjets_maj_create() {
+        // Si les stats sont activées, on prend en compte les articles
+        include_spip('inc/config');
+        if (lire_config('activer_statistiques') == 'oui') {
+                ecrire_config('activer_statistiques_objets', array('spip_articles'));
+        }
+        // vérifier la présence des champs sur toutes les tables
+        statsobjets_check_upgrade();
+}
+
+
+/**
+ * Vérifier que les champs nécessaires sont présents sur tous les objets : popularite, visites, referers
+ * Fonction appelée également lorsqu'on enregistre la configuration
+ * (cas d'un nouvel objet ajouté apres l'install du plugin)
+ *
+ * @return void
+ */
+function statsobjets_check_upgrade() {
+
+        include_spip('base/objets');
+
+        $tables_objets = array_keys(lister_tables_objets_sql());
+        $trouver_table = charger_fonction('trouver_table','base');
+        foreach($tables_objets as $table){
+                $desc = $trouver_table($table);
+                if (!isset($desc['field']['popularite'])){
+                        sql_alter("TABLE $table ADD popularite DOUBLE DEFAULT '0' NOT NULL");
+                }
+                if (!isset($desc['field']['visites'])) {
+                        sql_alter("TABLE $table ADD visites integer DEFAULT '0' NOT NULL");
+                }
+                if (!isset($desc['field']['referers'])) {
+                        sql_alter("TABLE $table ADD referers integer DEFAULT '0' NOT NULL");
+                }
+        }
+
+}