"le panneau de plugin en action c'est mieux"

.gitattributes

@@ -317,6 +317,7 @@ dist/vignettes/xls.png -text
 dist/vignettes/xml.png -text
 dist/vignettes/zip.png -text
 dist/win_width.htc -text
+ecrire/action/activer_plugins.php -text
 ecrire/action/autoriser.php -text
 ecrire/action/converser.php -text
 ecrire/action/dater.php -text

ecrire/action/activer_plugins.php

+<?php
+
+/***************************************************************************\
+ *  SPIP, Systeme de publication pour l'internet                           *
+ *                                                                         *
+ *  Copyright (c) 2001-2007                                                *
+ *  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;
+
+// mise a jour des donnees si envoi via formulaire
+// http://doc.spip.org/@enregistre_modif_plugin
+function enregistre_modif_plugin(){
+	include_spip('inc/plugin');
+  // recuperer les plugins dans l'ordre des $_POST
+  $test = array();
+	foreach(liste_plugin_files() as $file){
+	  $test["statusplug_$file"] = $file;
+	}
+	// gerer les noms de repertoires qui ont un espace
+	// sachant qu'ils vont arriver dans le $_POST avec un _ a la place
+	// mais qu'il faut pas se melanger si jamais deux repertoire existent et ne different
+	// que par un espace et un underscore
+	foreach($test as $postvar=>$file){
+		$alt_postvar = str_replace(" ","_",$postvar); // les espaces deviennent des _
+		$alt_postvar = str_replace(".","_",$postvar); // les points deviennent des _
+		if (!isset($test[$alt_postvar]))
+	  	$test[$alt_postvar] = $file;
+	}
+	$plugin=array();
+	foreach($_POST as $choix=>$val){
+		if (isset($test[$choix])&&$val=='O')
+			$plugin[]=$test[$choix];
+	}
+	global $connect_id_auteur, $connect_login;
+	spip_log("Changement des plugins actifs par auteur id=$connect_id_auteur :".implode(',',$plugin));
+	ecrire_plugin_actifs($plugin);
+	ecrire_metas();
+}
+
+function action_activer_plugins_dist() {
+
+	$securiser_action = charger_fonction('securiser_action', 'inc');
+	$arg = $securiser_action();
+
+	enregistre_modif_plugin();
+	
+	if ($redirect = _request('redirect')){
+		include_spip('inc/headers');
+		$redirect = str_replace('&amp;','&',$redirect);
+		redirige_par_entete($redirect);
+	}
+}
+
+?>

ecrire/exec/admin_plugin.php

@@ -34,19 +34,10 @@ function exec_admin_plugin() {
 		exit;
 	}
 
-	// mise a jour des donnees si envoi via formulaire
-	// sinon fait une passe de verif sur les plugin
-	if (_request('changer_plugin')=='oui'){
-		enregistre_modif_plugin();
-		// pour la peine, un redirige, 
-		// que les plugin charges soient coherent avec la liste
-		include_spip('inc/headers');
-		redirige_par_entete(generer_url_ecrire(_request('exec')));
-	}
-	else
-		verif_plugin();
+	verif_plugin();
 	if (isset($_GET['surligne']))
 		$surligne = $_GET['surligne'];
+
 	global $couleur_claire;
 	$commencer_page = charger_fonction('commencer_page', 'inc');
 	echo $commencer_page(_T('icone_admin_plugin'), "configuration", "plugin");
@@ -121,11 +112,11 @@ EOF;
 	echo "</style>\n";
 	echo "<br/><br/><br/>";
 	
-	gros_titre(_T('icone_admin_plugin'));
+	echo gros_titre(_T('icone_admin_plugin'),'',false);
 	// barre_onglets("configuration", "plugin"); // a creer dynamiquement en fonction des plugin charges qui utilisent une page admin ?
 	
-	debut_gauche();
-	debut_boite_info();
+	echo debut_gauche('plugin',true);
+	echo debut_boite_info(true);
 	$s = "";
 	$s .= _T('info_gauche_admin_tech');
 	$s .= "<p><span class='dev'>&nbsp;</span>"._T('plugin_etat_developpement')."</p>";
@@ -133,11 +124,11 @@ EOF;
 	$s .= "<p><span class='stable'>&nbsp;</span>"._T('plugin_etat_stable')."</p>";
 	$s .= "<p><span class='experimental'>&nbsp;</span>"._T('plugin_etat_experimental')."</p>";
 	echo $s;
-	fin_boite_info();
+	echo fin_boite_info(true);
 
-	debut_droite();
+	echo debut_droite('plugin',true);
 
-	debut_cadre_relief();
+	echo debut_cadre_relief('',true);
 
 	global $couleur_foncee;
 	echo "<table border='0' cellspacing='0' cellpadding='5' width='100%'>";
@@ -147,14 +138,15 @@ EOF;
 	echo "<tr><td class='serif' colspan='4'>";
 	echo _T('texte_presente_plugin');
 
-	echo generer_url_post_ecrire(_request('exec'));
+	$action = generer_action_auteur('activer_plugins','activer',generer_url_ecrire("admin_plugin"));
+	echo "<form action='$action' method='post' >";
+	echo form_hidden($action);
+	echo "<div style='text-align:$spip_lang_right'>";
+	echo "<input type='submit' name='Valider' value='"._T('bouton_valider')."' class='fondo' />";
+	echo "</div>";
 
 	affiche_arbre_plugins(liste_plugin_files(),liste_chemin_plugin_actifs());
 
-	echo "\n<input type='hidden' name='id_auteur' value='$connect_id_auteur' />";
-	echo "\n<input type='hidden' name='hash' value='" . calculer_action_auteur("valide_plugin") . "' />";
-	echo "\n<input type='hidden' name='changer_plugin' value='oui' />";
-
 	echo "\n<p />";
 
 	echo "<div style='text-align:$spip_lang_right'>";

ecrire/inc/plugin.php

@@ -233,37 +233,6 @@ function verif_plugin($pipe_recherche = false){
 	ecrire_metas();
 }
 
-// mise a jour des donnees si envoi via formulaire
-// http://doc.spip.org/@enregistre_modif_plugin
-function enregistre_modif_plugin(){
-  // recuperer les plugins dans l'ordre des $_POST
-  $test = array();
-	foreach(liste_plugin_files() as $file){
-	  $test["statusplug_$file"] = $file;
-	}
-	// gerer les noms de repertoires qui ont un espace
-	// sachant qu'ils vont arriver dans le $_POST avec un _ a la place
-	// mais qu'il faut pas se melanger si jamais deux repertoire existent et ne different
-	// que par un espace et un underscore
-	foreach($test as $postvar=>$file){
-		$alt_postvar = str_replace(" ","_",$postvar); // les espaces deviennent des _
-		$alt_postvar = str_replace(".","_",$postvar); // les points deviennent des _
-		if (!isset($test[$alt_postvar]))
-	  	$test[$alt_postvar] = $file;
-	}
-	$plugin=array();
-	if (!isset($_POST['desactive_tous'])){
-		foreach($_POST as $choix=>$val){
-			if (isset($test[$choix])&&$val=='O')
-				$plugin[]=$test[$choix];
-		}
-	}
-	global $connect_id_auteur, $connect_login;
-	spip_log("Changement des plugins actifs par auteur id=$connect_id_auteur :".implode(',',$plugin));
-	ecrire_plugin_actifs($plugin);
-	ecrire_metas();
-}
-
 // http://doc.spip.org/@ordonne_plugin
 function ordonne_plugin(){
 	$liste = liste_chemin_plugin_actifs();