L'installation est revue : la version_base 0.8 renomme les tables pour être conforme avec les dénominations SPIP - la recherche fonctionne un peu mieux - les jQuery Tooltips ne sont plus inclus via le pipeline 'insert_head_prive' (ils ne sont utiles que pour une page !) - et quelques corrections ... du coup, version 1.26, et hop

13 years ago · 4eb12b21b2
parent a5ae6c5272
commit 4eb12b21b2
9 changed files with 63 additions and 57 deletions
--- a/base/pubban_tables.php
+++ b/base/pubban_tables.php
@ -17,6 +17,8 @@ function pubban_declarer_tables_interfaces($interface){
 	// Table de jointure
 	$interface['tables_jointures']['publicites'][] = 'publicites';
 	$interface['tables_jointures']['bannieres'][] = 'bannieres';
+	$interface['tables_jointures']['bannieres_publicites'][] = 'publicites';
+	$interface['tables_jointures']['bannieres_publicites'][] = 'bannieres';
 	// Table des dates
 	$interface['table_date']['publicites'] = 'date_debut';
 	$interface['table_date']['publicites'] = 'date_fin';
--- a/base/pubban_upgrade.php
+++ b/base/pubban_upgrade.php
@ -10,13 +10,23 @@ if (!defined("_ECRIRE_INC_VERSION")) return;
 include_spip('base/pubban_chargeur');
 include_spip('base/abstract_sql');

+global $pubban_tables_set;
+$pubban_tables_set = array(
+	'spip_pubban_publicites'=>'spip_publicites',
+	'spip_pubban_emplacements'=>'spip_bannieres',
+	'spip_pubban_pub_empl'=>'spip_bannieres_publicites',
+	'spip_pubban_stats'=>'spip_pubban_stats',
+);
+
 function pubban_upgrade($nom_meta_base_version,$version_cible) {
 	include_spip('inc/meta');
 	include_spip('base/create');
-	$current_version = 0.0;
+	global $pubban_tables_set;
+	$current_version = '0.0'; // jamais installe
+	$version_new = '0.8'; // vieilles tables si < 0.8

 	// Si pas installe : on creer les tables et on insere les bannieres de test
-	if (!isset($GLOBALS['meta'][$nom_meta_base_version]) || $current_version=='0.0'){
+	if (!isset($GLOBALS['meta'][$nom_meta_base_version])){
 		creer_base();
 		foreach($GLOBALS['bannieres_site'] as $key => $value)
 			sql_insertq('spip_bannieres', $value, '');
@ -35,6 +45,19 @@ function pubban_upgrade($nom_meta_base_version,$version_cible) {
 		$current_version = $GLOBALS['meta'][$nom_meta_base_version],
 		$version_cible,"<")
 	){
+	
+		// Si version < 0.8, on renomme les tables ...
+		if (version_compare($current_version, $version_new, "<")) {
+			// ancienne=>nouvelle
+			foreach($pubban_tables_set as $old=>$new) {
+				if ($old!=$new)
+					$ok = sql_query("RENAME TABLE $old TO $new");
+			}
+			if ($ok)
+				spip_log("Plugin PUB BANNER - renommage des tables (version '$current_version' vers '$version_cible') OK");
+		}
+
+		// Puis on met a jour les champs
 		maj_tables(array(
 			'spip_publicites',
 			'spip_bannieres',
@ -46,12 +69,17 @@ function pubban_upgrade($nom_meta_base_version,$version_cible) {
 	}

 	// Sinon rien a faire
-	else spip_log("Plugin PUB BANNER - installation OK - rien a faire version_base idem");
+	else {
+		spip_log("Plugin PUB BANNER - installation OK - rien a faire version_base idem");
+	}
+	
+	ecrire_metas();
+	return true;
 }

 function pubban_vider_tables($nom_meta_base_version) {
 	include_spip('inc/meta');
-
+	global $pubban_tables_set;
 	$force = (defined('PUBBAN_FORCE_UNINSTALL') AND PUBBAN_FORCE_UNINSTALL==1) ? true : false;

 	// On verifie qu'il n'y ait pas de valeurs enregistrees
@ -61,20 +89,21 @@ function pubban_vider_tables($nom_meta_base_version) {

 	// Si ok, on efface
 	if($count_join==0 AND $count_stats==0){
-		$pubban_tables = 'spip_bannieres_publicites'
-			.','.'spip_publicites'
-			.','.'spip_bannieres'
-			.','.'spip_pubban_stats';
-		sql_drop_table($pubban_tables, true);
+		foreach($pubban_tables_set as $old=>$new) {
+			$ok = sql_drop_table($new, true);
+		}
 		effacer_meta('pubban_config');
 		effacer_meta($nom_meta_base_version);
-		spip_log("Plugin PUB BANNER - uninstall OK - sql_drop_table($pubban_tables) et metas effaces");
+		spip_log("Plugin PUB BANNER - uninstall OK - sql_drop_table() et metas effaces");
+		ecrire_metas();
 	}
 	// Sinon, on informe
 	else {
 		spip_log("Plugin PUB BANNER - uninstall pas possible car $count_join pubs et $count_stats statisqtiques en base ! - forcer l'effacement avec PUBBAN_FORCE_UNINSTALL=true dans 'pubban_options.php'");
+		ecrire_metas();
 		return false;
 	}
+
 	return true;
 }

--- a/exec/publicite_edit.php
+++ b/exec/publicite_edit.php
@ -20,7 +20,7 @@ function exec_publicite_edit_dist() {
 	$id_publicite = _request('id_publicite') ? _request('id_publicite') : 'new';

 	$retour = _request('retour') ? _request('retour') : (
-		($id_publicite == 'new') ? generer_url_ecrire("publicites_tous") : generer_url_ecrire("pubban_pub","id_publicite=$id_publicite")
+		($id_publicite == 'new') ? generer_url_ecrire("publicites_tous") : generer_url_ecrire("publicite_voir","id_publicite=$id_publicite")
 	);
 	$contexte = array(
 		'id_publicite' => $id_publicite,
--- a/exec/statistiques_bannieres.php
+++ b/exec/statistiques_bannieres.php
@ -29,7 +29,7 @@ function exec_statistiques_bannieres_dist() {

 	echo($commencer_page(_T('pubban:pubban')." - ".$titre_page, $rubrique, $sous_rubrique)), debut_gauche('', true),
 		debut_cadre_relief(find_in_path("img/ico-pubban.png"), true, "", "<br />"._T('pubban:intro_stats')),
-  		_T("pubban:intro_texte_stats"), _T("pubban:voir_page"), icone_horizontale(_T('pubban:page_infos'), generer_url_ecrire('pubban_info'), find_in_path("img/status-dock-24.png"), "rien.gif", false),
+  		_T("pubban:intro_texte_stats_banner"), _T("pubban:voir_page"), icone_horizontale(_T('pubban:page_infos'), generer_url_ecrire('pubban_info'), find_in_path("img/status-dock-24.png"), "rien.gif", false),
  		fin_cadre_relief(true), bloc_des_raccourcis($res),
 		creer_colonne_droite('', true), debut_droite('', true), gros_titre($titre_page, '', false),
 		"\n<div id='pubban_info_nojs' class='verdana2' style='text-align: justify;border:1px solid #404040;padding:1em;'><p>"
--- a/inc/pubban_prive.php
+++ b/inc/pubban_prive.php
@ -94,9 +94,9 @@ function pubban_search($str){
 	// Recherche dans les publicites
 	$ban_str = 'id_banniere=';
 	$ban_max = strlen($ban_str);
-	if(substr_count($str, $ban_str) != 0 AND is_numeric(substr($str, $ban_str))) {
-		$id_banniere = substr($str, $ban_str);
-		$pub = sql_select("id_publicite", 'spip_publicites', "id_banniere=".intval($id_banniere), '', '', '', '');
+	if(substr_count($str, $ban_str) != 0 AND is_numeric(substr($str, $ban_max))) {
+		$id_banniere = substr($str, $ban_max);
+		$pub = sql_select("id_publicite", 'spip_bannieres_publicites', "id_banniere=".intval($id_banniere), '', '', '', '');
 		while ($row = spip_fetch_array($pub)) {
 			$results['pub'][$i] = $row['id_publicite'];
 			$i++;
--- a/plugin.xml
+++ b/plugin.xml
@ -21,7 +21,7 @@ Cette documentation est disponible, si le plugin est actif sur votre site, sur l

 -* La page de statistiques du plugin est une adaptation libre du script '{{[VRG pub->http://vrgpub.frankdevelopper.com/]}}' d&#233;velopp&#233; par {{Vincent Roseberry}} ({sous licence GPL &#224; l'&#233;poque}).
 	</description>
-	<version>1.25</version>
+	<version>1.26</version>
 	<version_base>0.8</version_base>
 	<etat>dev</etat>
 	<icon>img/ico-pubban.png</icon>
@ -33,10 +33,6 @@ Cette documentation est disponible, si le plugin est actif sur votre site, sur l
 	<chemin dir='public' type='public' />
 	<chemin dir='prive' type='prive' />
 	<chemin dir='' />
-    <pipeline>
-        <nom>header_prive</nom>
-		<inclure>pubban_pipelines.php</inclure>
-    </pipeline>
 	<pipeline>
        <nom>declarer_tables_interfaces</nom>
        <inclure>base/pubban_tables.php</inclure>
@ -53,10 +49,6 @@ Cette documentation est disponible, si le plugin est actif sur votre site, sur l
 		<nom>taches_generales_cron</nom>
 		<inclure>pubban_pipelines.php</inclure>
 	</pipeline>	
-	<pipeline>
-		<nom>jquery_plugins</nom>
-		<inclure>pubban_pipelines.php</inclure>
-	</pipeline>	
 	<bouton id='pubban' parent='naviguer'>
 		<icone>img/ico-pubban-24.png</icone>
 		<titre>pubban:pubban_titre</titre>
--- a/prive/pubban_recherche_resultats.html
+++ b/prive/pubban_recherche_resultats.html
@ -6,6 +6,7 @@
 	<h4>&nbsp;#ENV{boucle_search}[ (#ENV{boucle_search}|singulier_ou_pluriel{pubban:result_match,pubban:results_match})]</h4>
 })]
 </div>
+
 <div class='titrem deplie' onmouseover="jQuery(this).depliant('#pubban_liste_pub');"><a href='#' onclick="return jQuery(this).depliant_clicancre('#pubban_liste_pub');" class='titremancre'></a><:pubban:search_results:></div>
 <div id='pubban_liste_pub' class='bloc_depliable blocdeplie'>
 <table border='0' cellpadding='3' cellspacing='0' width='100%' class='verdana1'>
@ -16,9 +17,7 @@
 	<th><:pubban:outils:></th>
 	</tr>

-<BOUCLE_pourpub(POUR){tableau #ENV{pub}}>
-<B_pubban>
-	<BOUCLE_pubban(PUBLICITES){id_publicite=#VALEUR}>
+<BOUCLE_pubban(PUBLICITES){id_publicite IN #ENV{pub}}>
 	<tr class='tr_liste'>
 	<td class="arial11">
 [(#STATUT|=={1inactif}|oui)
@ -67,13 +66,9 @@
 ]
 	</td>
 	</tr>
-	</BOUCLE_pubban>
-</B_pubban>
-</BOUCLE_pourpub>
+</BOUCLE_pubban>

-<BOUCLE_pouremp(POUR){tableau #ENV{emp}}>
-<B_pubban2>
-	<BOUCLE_pubban2(BANNIERES){id_banniere=#VALEUR}>
+<BOUCLE_pubban2(BANNIERES){id_banniere IN #ENV{emp}}>
 	<tr class='tr_liste'>
 	<td class="arial11">
 [(#STATUT|=={1inactif}|oui)
@ -119,9 +114,8 @@
 ]
 		</td>
 	</tr>
-	</BOUCLE_pubban2>
-</B_pubban2>
-</BOUCLE_pouremp>
+</BOUCLE_pubban2>
+
 </table>
 </div>
 </div>
--- a/prive/pubban_statistiques.html
+++ b/prive/pubban_statistiques.html
@ -1,4 +1,13 @@
 <div class='cadre-formulaire-editer'>
+<script type="text/javascript" src="#CHEMIN{javascripts/jquery.bgiframe.js}"></script>
+<script type="text/javascript" src="#CHEMIN{javascripts/jquery.dimensions.js}"></script>
+<script type="text/javascript" src="#CHEMIN{javascripts/jquery.tooltip.js}"></script>
+<link rel="stylesheet" type="text/css" href="#CHEMIN{javascripts/jquery.tooltip.css}" media='all' />
+<script type="text/javascript">
+$(document).ready(function(){
+	$('.pubban_tltp').tooltip({ track: true, showURL: false, delay: 0, left: -30, extraClass: 'pubban' });
+});
+</script>
 <script type="text/javascript" src="#CHEMIN{javascripts/wz_jsgraphics.js}"></script>
 <script type="text/javascript" src="#CHEMIN{javascripts/pie.js}"></script>
 <script type="text/javascript" src="#CHEMIN{javascripts/line.js}"></script>
@ -9,6 +18,6 @@
 	#FORMULAIRE_PUBBAN_STATISTIQUES
 ]
 <div class='formulaire_spip' style='padding: 0.4em; font-size: 0.8em'>
-<:pubban:licence_stats:>
+<:pubban:licence_stats|propre:>
 </div>
 </div>
--- a/pubban_pipelines.php
+++ b/pubban_pipelines.php
@ -17,24 +17,4 @@ function pubban_taches_generales_cron($taches_generales){
 	return $taches_generales;
 }

-function pubban_jquery_plugins($scripts){
-	$scripts[] = "javascripts/jquery.bgiframe.js";
-	$scripts[] = "javascripts/jquery.dimensions.js";
-	$scripts[] = "javascripts/jquery.tooltip.js";
-	return $scripts;
-}
-
-/**
- * Header des pages exec
- */
-function pubban_header_prive($texte) {
-	$texte .= "<link rel='stylesheet' type='text/css' href='".find_in_path("javascripts/jquery.tooltip.css")."' media='all' />"
-		."<script type=\"text/javascript\"><!--
-$(document).ready(function(){
-	$('.pubban_tltp').tooltip({ track: true, showURL: false, delay: 0, left: -30, extraClass: 'pubban' });
-});
-//--></script>";
-	return $texte;
-}
-
 ?>