Quelques modifications en prévisions du necessite svpbase.

Cette version ne doit pas être encore utilisée

action/actualiser_stats.php

@@ -21,10 +21,13 @@ function action_actualiser_stats_dist(){
 	// Actualisation des statistiques d'utilisation des plugins en provenance de 
 	// stats.spip.org
 	// On verife tout de meme qu'il y a au moins un depot
-	if (($mode === 'tout') AND sql_countsel('spip_depots')) {
+	if (
+		($mode === 'tout')
+		and sql_countsel('spip_depots')
+	) {
 		include_spip('inc/svp_statistiquer');
 		svp_actualiser_stats();
 		// On consigne l'action
-		spip_log("MODULE STATS - ACTION ACTUALISER STATS (manuel)", 'svp_actions.' . _LOG_INFO);
+		spip_log("MODULE STATS - ACTION ACTUALISER STATS (manuel)", 'svpbase.' . _LOG_INFO);
 	}
 }

genie/svp_actualiser_stats.php

@@ -8,7 +8,7 @@ function genie_svp_actualiser_stats_dist ($last) {
 	if (sql_countsel('spip_depots')) {
 		include_spip('inc/svp_statistiquer');
 		svp_actualiser_stats();
-		spip_log("MODULE STATS - ACTION ACTUALISER STATS (automatique)", 'svp_actions.' . _LOG_INFO);
+		spip_log("MODULE STATS - ACTION ACTUALISER STATS (automatique)", 'svpbase.' . _LOG_INFO);
 	}
 
 	return 1;

inc/svp_statistiquer.php

@@ -1,7 +1,8 @@
 <?php
 
-if (!defined("_ECRIRE_INC_VERSION")) return;
-
+if (!defined('_ECRIRE_INC_VERSION')) {
+	return;
+}
 // - Adresse de la page fournissant les statistiques par json
 if (!defined('_SVP_SOURCE_STATS')) {
 	define('_SVP_SOURCE_STATS', 'https://stats.spip.net/spip.php?page=stats.json');
@@ -11,11 +12,11 @@ if (!defined('_SVP_SOURCE_STATS')) {
 
 /**
  * Actualisation des statistiques des plugins presents dans la base.
- * @return boolean
+ *
+ * @return bool
  */
-
 function svp_actualiser_stats() {
-	
+
 	// Initialisation du retour
 	$retour = true;
 
@@ -29,17 +30,19 @@ function svp_actualiser_stats() {
 	) {
 		foreach ($stats as $_stat) {
 			$prefixe = strtoupper($_stat->nom);
-			if ($id_plugin = sql_getfetsel('id_plugin', 'spip_plugins', array('prefixe='. sql_quote($prefixe)))) {
+			if ($id_plugin = sql_getfetsel('id_plugin', 'spip_plugins', array('prefixe=' . sql_quote($prefixe)))) {
 				// Si le plugin est bien dans la base on peut lui mettre a jour ses statistiques
-				sql_updateq('spip_plugins',
-							array('nbr_sites'=> $_stat->sites, 'popularite'=> floatval(trim($_stat->pourcentage, '%'))),
-							'id_plugin=' . intval($id_plugin));
+				sql_updateq(
+					'spip_plugins',
+					array('nbr_sites'=> $_stat->sites, 'popularite'=> floatval(trim($_stat->pourcentage, '%'))),
+					'id_plugin=' . intval($id_plugin)
+				);
 			}
 		}
 	} else {
 		// On ne fait que loger l'erreur car celle-ci n'a pas d'incidence sur le comportement
 		// de SVP
-		spip_log('MODULE STATS - Réponse du serveur incorrecte ou mal formée. Les statistiques ne seront pas mises à jour', 'svp_actions.' . _LOG_ERREUR);
+		spip_log('MODULE STATS - Réponse du serveur incorrecte ou mal formée. Les statistiques ne seront pas mises à jour', 'svpbase.' . _LOG_ERREUR);
 		$retour = false;
 	}
 
@@ -59,7 +62,7 @@ function svp_actualiser_stats() {
 		) {
 			foreach ($stats as $_stat) {
 				$prefixe = strtoupper($_stat->nom);
-				$where = array('prefixe='. sql_quote($prefixe), 'branche_spip=' . sql_quote($_branche));
+				$where = array('prefixe=' . sql_quote($prefixe), 'branche_spip=' . sql_quote($_branche));
 				// Suivant que l'enregistrement du plugin pour la branche donnée existe ou pas, on met à jour
 				// ou on insère les statistiques.
 				$historique = array();
@@ -94,10 +97,10 @@ function svp_actualiser_stats() {
 		} else {
 			// On ne fait que loger l'erreur car celle-ci n'a pas d'incidence sur le comportement
 			// de SVP
-			spip_log("MODULE STATS - Réponse du serveur incorrecte ou mal formée. Les statistiques de la branche ${_branche} ne seront pas mises à jour", 'svp_actions.' . _LOG_ERREUR);
+			spip_log("MODULE STATS - Réponse du serveur incorrecte ou mal formée. Les statistiques de la branche ${_branche} ne seront pas mises à jour", 'svpbase.' . _LOG_ERREUR);
 			$retour = false;
 		}
 	}
-	
+
 	return $retour;
 }

paquet.xml

@@ -15,9 +15,9 @@
 
 	<licence lien="http://www.gnu.org/licenses/gpl-3.0.html">GNU/GPL</licence>
 
-	<pipeline nom="declarer_tables_interfaces" inclure="base/svpstats_declarer.php" />
-	<pipeline nom="declarer_tables_principales" inclure="base/svpstats_declarer.php" />
-	<pipeline nom="declarer_tables_auxiliaires" inclure="base/svpstats_declarer.php" />
+	<pipeline nom="declarer_tables_interfaces" inclure="base/svpstats_declarations.php" />
+	<pipeline nom="declarer_tables_principales" inclure="base/svpstats_declarations.php" />
+	<pipeline nom="declarer_tables_auxiliaires" inclure="base/svpstats_declarations.php" />
 	<pipeline nom="taches_generales_cron" inclure="genie/svpstats_taches_generales_cron.php" />
 	<pipeline nom="affiche_gauche" inclure="svpstats_pipelines.php" />
 

prive/squelettes/inclure/info_stats.html

@@ -1,5 +1,9 @@
 [(#BOITE_OUVRIR{'', info})]
 	#SET{periode, #VAL|svpstats_periode_actualisation_stats}
 	[(#GET{periode}|oui)<:svpstats:info_boite_statistiques{periode=#GET{periode}}:>]
-	[(#BOUTON_ACTION{<:svpstats:bouton_actualiser_stats:>, #URL_ACTION_AUTEUR{actualiser_stats, tout, #SELF}, '', '', <:svpstats:bulle_actualiser_stats:>})]
+	<BOUCLE_depot_existe(DEPOTS) {id_depot>0}{0,1}>
+	[(#AUTORISER{ajouter, depot, #ID_DEPOT}|oui)
+		[(#BOUTON_ACTION{<:svpstats:bouton_actualiser_stats:>, #URL_ACTION_AUTEUR{actualiser_stats, tout, #SELF}, '', '', <:svpstats:bulle_actualiser_stats:>})]
+	]
+	</BOUCLE_depot_existe>
 #BOITE_FERMER

svpstats_administrations.php

@@ -19,16 +19,22 @@ function svpstats_upgrade($nom_meta_base_version, $version_cible){
 	include_spip('base/upgrade');
 	maj_plugin($nom_meta_base_version, $version_cible, $maj);
 
-	spip_log('MODULE STATS - INSTALLATION BDD', 'svp_actions.' . _LOG_INFO);
+	spip_log('MODULE STATS - INSTALLATION BDD', 'svpbase.' . _LOG_INFO);
 }
 
 function svpstats_vider_tables($nom_meta_base_version) {
+
+	// Supprimer les colonnes ajoutées à la table spip_plugins
 	sql_alter("TABLE spip_plugins DROP COLUMN nbr_sites");
 	sql_alter("TABLE spip_plugins DROP COLUMN popularite");
+
+	// Supprimer la table de stats ajoutée
 	sql_drop_table("spip_plugins_stats");
+
+	// Supprimer la meta du schéma du plugin
 	effacer_meta($nom_meta_base_version);
 
-	spip_log('MODULE STATS - DESINSTALLATION BDD', 'svp_actions.' . _LOG_INFO);
+	spip_log('MODULE STATS - DESINSTALLATION BDD', 'svpbase.' . _LOG_INFO);
 }
 
 /**

svpstats_pipelines.php

@@ -10,7 +10,7 @@ function svpstats_affiche_gauche($flux){
 
 	if ($flux['args']['exec'] == 'depots') {
 		if (
-			($resultats = sql_select('*', 'spip_depots'))
+			(sql_select('id_depot', 'spip_depots'))
 			and ($texte = recuperer_fond('prive/squelettes/inclure/info_stats', array()))
 		) {
 			if ($pos = strpos($flux['data'],'<!--affiche_gauche_milieu-->')) {