Valider 1678ad70 rédigé par esj's avatar esj
Parcourir les fichiers

Erreur de stratégie dans l'activation des plugins. Ce n'est pas parce que le... 

Erreur de stratégie dans l'activation des plugins. Ce n'est pas parce que le serveur SQL est indisponible, et donc ne permet pas de sauver la meta "plugin", qu'il faut renoncer à activer ceux-ci. Cette stratégie empêche un plugin de surcharger les premières étapes de l'installation, et peut avoir des effets pervers si le serveur est indisponible provisoirement (par exemple des accès restreints qui ne seraient plus activés).

Après nettoyage du code, il apparaît que la raison pour laquelle on abandonnait tout si le serveur SQL était indisponible était que la fonction effacer_meta, disymétriquement à ecrire_meta, ne se protégait pas de cette situation au lieu de faire le travail a minima (affectation de la globale PHP quand même et retour à l'appelant). On aligne donc cette fonction sur ecrire_meta, ce qui permet de créer les fichiers tmp/*plug* etc en faisant sauter le test de disponibilité du serveur dans inc/plugin.
parent 74c2f6d7

ecrire/inc/meta.php

+1 −1
Numéro de ligne d'origine Numéro de ligne de diff Ligne de diff
@@ -101,7 +101,7 @@ function effacer_meta($nom, $table='meta') { 
	static $touch = array();

	$antidate = time() - (_META_CACHE_TIME<<4);

	if (!isset($touch[$table])) {touch_meta($antidate, $table);}

	sql_delete('spip_' . $table, "nom='$nom'");

	sql_delete('spip_' . $table, "nom='$nom'",'','continue');

	unset($GLOBALS[$table][$nom]);

	if (!isset($touch[$table])) {touch_meta($antidate, $table); $touch[$table] = false;}

}

ecrire/inc/plugin.php

+42 −27
Numéro de ligne d'origine Numéro de ligne de diff Ligne de diff
@@ -323,17 +323,14 @@ function liste_chemin_plugin_actifs($dir_plugins=_DIR_PLUGINS){ 
	return $liste;

}



// Activateur/desactivateur de plugins

// retourne la liste des plugins actifs, sous forme de chaine serialisee

// http://doc.spip.org/@ecrire_plugin_actifs

function ecrire_plugin_actifs($plugin,$pipe_recherche=false,$operation='raz') {

	static $liste_pipe_manquants=array();



	// creer le repertoire cache/ si necessaire ! (installation notamment)

	sous_repertoire(_DIR_CACHE, '', false,true);



	$liste_fichier_verif = array();

	if (($pipe_recherche)&&(!in_array($pipe_recherche,$liste_pipe_manquants)))

		$liste_pipe_manquants[]=$pipe_recherche;



	if ($operation!='raz'){

		$plugin_actifs = liste_chemin_plugin_actifs();

		$plugin_liste = liste_plugin_files();
@@ -347,17 +344,34 @@ function ecrire_plugin_actifs($plugin,$pipe_recherche=false,$operation='raz') { 
	// recharger le xml des plugins a activer

	list($plugin_valides,$ordre,$infos) = liste_plugin_valides($plugin,true);



	ecrire_meta('plugin',serialize($plugin_valides));

    // Calcul des metas

    $plugin_s = serialize($plugin_valides);

    $plugin_header = array();

    foreach($plugin_valides as $p=>$r){

            $plugin_header[]= $p.($r['version']?"(".$r['version'].")":"");

    }

    $plugin_header = strtolower(substr(implode(",",$plugin_header), 0,900));



    // Mettre a jour les meta

    // Les ecrire_meta() doivent en principe aussi initialiser la valeur a vide

    // si elle n'existe pas

    // risque en php5 a cause du typage ou de null (verifier dans la doc php)

    

    effacer_meta('message_crash_plugins'); // baisser ce flag !

	$plugin_header_info = array();

	foreach($plugin_valides as $p=>$resume){

		$plugin_header_info[]= $p.($resume['version']?"(".$resume['version'].")":"");

    ecrire_meta('plugin', $plugin_s);

    ecrire_meta('plugin_header', $plugin_header);

    ecrire_plugin_actifs2($plugin_valides, $ordre, $infos, $pipe_recherche);

    return $plugin_s;

}

	ecrire_meta('plugin_header',substr(strtolower(implode(",",$plugin_header_info)),0,900));



	$start_file = "<"."?php\nif (defined('_ECRIRE_INC_VERSION')) {\n";

	$end_file = "}\n?".">";

function ecrire_plugin_actifs2($plugin_valides, $ordre, $infos, $pipe_recherche)

{	

	static $liste_pipe_manquants=array();



	if (($pipe_recherche)&&(!in_array($pipe_recherche,$liste_pipe_manquants)))

		$liste_pipe_manquants[]=$pipe_recherche;



	$liste_fichier_verif = array();

	if (is_array($infos)){

		// construire tableaux de boutons et onglets

		$liste_boutons = array();
@@ -446,7 +460,7 @@ function ecrire_plugin_actifs($plugin,$pipe_recherche=false,$operation='raz') { 
			$s .= "if (!function_exists('boutons_plugins')){function boutons_plugins(){return unserialize('".str_replace("'","\'",serialize($liste_boutons))."');}}\n";

			$s .= "if (!function_exists('onglets_plugins')){function onglets_plugins(){return unserialize('".str_replace("'","\'",serialize($liste_onglets))."');}}\n";

		}

		ecrire_fichier($fileconf, $start_file . $splugs . $s . $end_file);

		ecrire_fichier_actif($fileconf, "$splugs$s");

	}



	if (is_array($infos)){
@@ -529,8 +543,6 @@ function pipeline_precompile(){ 
	global $spip_pipeline, $spip_matrice;

	$liste_fichier_verif = array();



	$start_file = "<"."?php\nif (defined('_ECRIRE_INC_VERSION')) {\n";

	$end_file = "}\n?".">";

	$content = "";

	foreach($spip_pipeline as $action=>$pipeline){

		$s_inc = "";
@@ -564,29 +576,32 @@ function pipeline_precompile(){ 
		$content .= $s_call;

		$content .= "return \$val;\n}\n\n";

	}

	ecrire_fichier(_CACHE_PIPELINES, $start_file . $content . $end_file);

	ecrire_fichier_actif(_CACHE_PIPELINES, $content);

	return $liste_fichier_verif;

}



function ecrire_fichier_actif($nom, $corps)

{

    $corps = '<'."?php\nif (defined('_ECRIRE_INC_VERSION')) {\n$corps}\n?".'>';

    ecrire_fichier($nom, $corps);

}



// pas sur que ca serve...

// http://doc.spip.org/@liste_plugin_inactifs

function liste_plugin_inactifs(){

	return array_diff (liste_plugin_files(),liste_chemin_plugin_actifs());

}



// mise a jour du meta en fonction de l'etat du repertoire

// Les  ecrire_meta() doivent en principe aussi initialiser la valeur a vide

// si elle n'existe pas

// risque de pb en php5 a cause du typage ou de null (verifier dans la doc php)

// mise a jour des fichiers rendant actifs les plugins choisis,

// en fonction du contenu de leurs repertoires

function actualise_plugins_actifs($pipe_recherche = false){

	if (!spip_connect()) return false;

	$plugin_actifs = liste_chemin_plugin_actifs();

	$plugin_liste = liste_plugin_files();

	$plugin_new = array_intersect($plugin_actifs,$plugin_liste);

	$actifs_avant = $GLOBALS['meta']['plugin'];

	ecrire_plugin_actifs($plugin_new,$pipe_recherche);

	$avant = isset($GLOBALS['meta']['plugin']) ? $GLOBALS['meta']['plugin'] : '';

    $apres = ecrire_plugin_actifs($plugin_new,$pipe_recherche);

	// retourner -1 si la liste des plugins actifs a change

	return (strcmp($GLOBALS['meta']['plugin'],$actifs_avant)==0) ? 1 : -1;

	return (strcmp($apres,$avant)==0) ? 1 : -1;

}



// http://doc.spip.org/@spip_plugin_install

Forge communautaire SPIP | Charte d'utilisation | Signaler un problème sur ce site