nettoyage de code (problèmes potentiels signalés par l'outil "inspection du code") : (9b0e8c68) · Validations · Glop / spip

ecrire/action/charger_plugin.php

+4 −2

Numéro de ligne d'origine	Numéro de ligne de diff	Ligne de diff
		@@ -39,6 +39,7 @@ function action_charger_plugin_dist() {
		exit;
		}

		$zip = "";
		if ($arg == 'update_flux') {
		if (is_array($syndic_plug = @unserialize($GLOBALS['meta']['syndic_plug'])))
		foreach ($syndic_plug as $url => $c)
		@@ -153,6 +154,7 @@ function action_charger_plugin_dist() {
		// Reconnaitre un plugin par son fichier xml
		$get_infos = charger_fonction('get_infos','plugins');
		$infos = $get_infos($status['tmpname'], true, '');
		$nom = $description = $type = "";
		if ($infos) {
		$nom = $infos['nom'];
		$image = $infos['icon'];
		@@ -184,7 +186,7 @@ function action_charger_plugin_dist() {

		// l'icone ne peut pas etre dans tmp/ (lecture http oblige)
		// on la copie donc dans local/chargeur/
		if ($image) {
		if (isset($image) AND $image) {
		$dir = sous_repertoire(_DIR_VAR,'chargeur');
		@copy($status['tmpname'].'/'.$image, $image2 = $dir.basename($image));
		$retour = "<img src='".$image2."' style='float:right;' />"
		@@ -235,7 +237,7 @@ function action_charger_plugin_dist() {

		include_spip('exec/install'); // pour bouton_suivant()

		$texte = "<div style='text-align:$spip_lang_left;'>$texte</div>\n";
		$texte = "<div style='text-align:".$spip_lang_left.";'>$texte</div>\n";

		$redirect = rawurldecode(_request('redirect'));
		// par defaut on revient sur la page admin_plugin

+3 −1

Numéro de ligne d'origine	Numéro de ligne de diff	Ligne de diff
		@@ -14,7 +14,7 @@ if (!defined('_ECRIRE_INC_VERSION')) return;

		// http://doc.spip.org/@action_editer_article_dist
		function action_editer_article_dist($arg=null) {

		$err="";
		if (is_null($arg)){
		$securiser_action = charger_fonction('securiser_action', 'inc');
		$arg = $securiser_action();
		@@ -91,6 +91,8 @@ function insert_article($id_rubrique) {
		$id_secteur = $row['id_secteur'];
		$lang_rub = $row['lang'];

		$lang = "";
		$choisie = 'non';
		// La langue a la creation : si les liens de traduction sont autorises
		// dans les rubriques, on essaie avec la langue de l'auteur,
		// ou a defaut celle de la rubrique

+3 −19

Numéro de ligne d'origine	Numéro de ligne de diff	Ligne de diff
		@@ -42,26 +42,10 @@ function action_editer_auteur_dist($arg=null) {
		if ($id_auteur > 0)
		$err = auteurs_set($id_auteur);

		if ($redirect = _request('redirect')) {
		if ($err){
		$ret = ('&redirect=' . $redirect);
		spip_log("echec editeur auteur: " . join(' ',$echec));
		$echec = '&echec=' . join('@@@', $echec);
		$redirect = generer_url_ecrire('auteur',"id_auteur=$id_auteur$echec$ret",'&');
		}
		else
		$redirect = urldecode($redirect);
		if ($err)
		spip_log("echec editeur auteur: $err",_LOG_ERREUR);

		$redirect = parametre_url($redirect,'id_auteur', $id_auteur, '&');

		include_spip('inc/headers');
		redirige_par_entete($redirect);
		}
		else
		return array($id_auteur,$err);

		$redirect = _request('redirect');

		}

		function insert_auteur($source=null) {

+3 −2

Numéro de ligne d'origine	Numéro de ligne de diff	Ligne de diff
		@@ -28,6 +28,7 @@ function objet_associable($objet){
		$trouver_table = charger_fonction('trouver_table','base');
		$table_sql = table_objet_sql($objet);

		$l="";
		if ($primary = id_table_objet($objet)
		AND $trouver_table($l = $table_sql."_liens")
		AND !preg_match(',[^\w],',$primary)

+5 −0

Numéro de ligne d'origine	Numéro de ligne de diff	Ligne de diff
		@@ -91,6 +91,10 @@ function objets_set($objet, $id, $set=null) {
		$invalideur = "id='$objet/$id'";
		$indexation = true;
		}
		else {
		$invalideur = "";
		$indexation = false;
		}

		modifier_contenu($objet, $id,
		array(
		@@ -123,6 +127,7 @@ function insert_objet($objet, $id_parent=null) {
		if (!$desc OR !isset($desc['field']))
		return 0;

		$lang_rub = "";
		$champs = array();
		if (isset($desc['field']['id_rubrique'])){
		// Si id_rubrique vaut 0 ou n'est pas definie, creer l'objet