nettoyage et generalisation pour les logos, avec un pipeline libeller_logo

exec/iconifier ne sert plus

.gitattributes

@@ -117,7 +117,6 @@ ecrire/exec/documents_liste.php -text
 ecrire/exec/editer_auteurs.php -text
 ecrire/exec/fond.php -text
 ecrire/exec/fond_monobloc.php -text
-ecrire/exec/iconifier.php -text
 ecrire/exec/index.php -text
 ecrire/exec/info_plugin.php -text
 ecrire/exec/informer.php -text

ecrire/action/iconifier.php

@@ -12,7 +12,10 @@
 
 if (!defined("_ECRIRE_INC_VERSION")) return;
 
-// http://doc.spip.org/@action_iconifier_dist
+/**
+ * L'entree par l'action ne sert plus qu'a une retro compat eventuelle
+ * le #FORMULAIRE_EDITER_LOGO utilise action_spip_image_ajouter_dist
+ */
 function action_iconifier_dist()
 {
 	include_spip('inc/actions');

ecrire/core.xml

@@ -278,6 +278,7 @@
 	<pipeline><nom>lister_tables_noerase</nom><action></action></pipeline>
 	<pipeline><nom>lister_tables_noexport</nom><action></action></pipeline>
 	<pipeline><nom>lister_tables_noimport</nom><action></action></pipeline>
+	<pipeline><nom>libeller_logo</nom><action></action></pipeline>
 	<pipeline><nom>nettoyer_raccourcis_typo</nom><action></action></pipeline>
 	<pipeline><nom>notifications</nom><action></action></pipeline>
 	<pipeline><nom>objet_compte_enfants</nom><action></action></pipeline>

ecrire/exec/iconifier.php

-<?php
-
-/***************************************************************************\
- *  SPIP, Systeme de publication pour l'internet                           *
- *                                                                         *
- *  Copyright (c) 2001-2010                                                *
- *  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;
-
-include_spip('inc/presentation');
-
-// http://doc.spip.org/@exec_iconifier_dist
-function exec_iconifier_dist()
-{
-	$script = _request('script');
-	$iframe = _request('iframe');
-	$type =_request('type');
-	$id = intval(_request($type));
-	exec_iconifier_args($id, $type, $script, $iframe);
-}
-
-// http://doc.spip.org/@exec_iconifier_args
-function exec_iconifier_args($id, $type, $script, $iframe=false)
-{
-	global $connect_id_auteur, $connect_toutes_rubriques;;
-	if (!preg_match('/^\w+$/', "$type$script")) {
-		$droit = false;
-	} else {
-		if ($type == 'id_rubrique')
-			$droit = autoriser('publierdans','rubrique',$id);
-		elseif ($type == 'id_auteur')
-			$droit = (($id == $connect_id_auteur) OR $connect_toutes_rubriques);
-		elseif (in_array($type, array('id_mot', 'id_groupe')))
-			$droit = $connect_toutes_rubriques;
-		else {
-			$table=substr($type, 3) . (($type == 'id_syndic') ? '' : 's');
-			$row = sql_fetsel("id_rubrique, statut", "spip_$table", "$type=$id");
-			$droit = autoriser('publierdans','rubrique',$row['id_rubrique']);
-			if (!$droit AND  ($row['statut'] == 'prepa' OR $row['statut'] == 'prop' OR $row['statut'] == 'poubelle')) {
-			  $jointure = table_jointure('auteur', 'article');
-			  $droit = sql_fetsel("id_auteur", "spip_$jointure", "id_article=".sql_quote($id) . " AND id_auteur=$connect_id_auteur");
-			}
-		}
-	}
-
-	if (!$droit) {
-		include_spip('inc/minipres');
-		echo minipres();
-	} else {
-
-		$iconifier = charger_fonction('iconifier', 'inc');
-		$ret = $iconifier($type, $id, $script, $visible=true);
-	
-		if ($iframe!=='iframe') 
-			ajax_retour($ret);
-		else {
-			echo "<div class='upload_answer upload_document_added'>$ret</div>";
-		}
-	}
-}
-?>

ecrire/inc/iconifier.php

@@ -16,13 +16,10 @@ include_spip('inc/actions');
 
 // http://doc.spip.org/@inc_iconifier_dist
 function inc_iconifier_dist($objet, $id,  $script, $visible=false, $flag_modif=true) {
-	global $logo_libelles;
 	// compat avec anciens appels
-	if (substr($objet,0,3)=='id_')
-		$objet = substr($objet,3);
-
+	$objet = objet_type($objet);
+	
 	return recuperer_fond('prive/editer/logo',array('objet'=>$objet,'id_objet'=>$id,'editable'=>$flag_modif));
-
 }
 
 ?>

prive/formulaires/editer_logo.php

@@ -16,9 +16,8 @@ global $logo_libelles;
 $logo_libelles['article'] = _T('logo_article').aide ("logoart");
 $logo_libelles['auteur'] = _T('logo_auteur').aide ("logoart");
 $logo_libelles['breve'] = _T('logo_breve').aide ("breveslogo");
-$logo_libelles['syndic'] = _T('logo_site')." ".aide ("rublogo");
-$logo_libelles['mot'] = _T('logo_mot_cle').aide("breveslogo");
-$logo_libelles['groupe'] = _T('logo_groupe').aide("breveslogo");
+// utilise pour le logo du site, donc doit rester ici
+$logo_libelles['site'] = _T('logo_site')." ".aide ("rublogo");
 $logo_libelles['rubrique'] = _T('logo_rubrique')." ".aide ("rublogo");
 $logo_libelles['racine'] = _T('logo_standard_rubrique')." ".aide ("rublogo");
 
@@ -48,21 +47,20 @@ $logo_libelles['racine'] = _T('logo_standard_rubrique')." ".aide ("rublogo");
 function formulaires_editer_logo_charger_dist($objet, $id_objet, $retour='', $options=array()){
 	// pas dans une boucle ? formulaire pour le logo du site
 	// dans ce cas, il faut chercher un 'siteon0.ext'
-	if (!$objet) {
-		$objet = 'syndic';
-		$_id_objet = 'site';
-	}
-	else {
-		if ($objet=='site') $objet = 'syndic';
-		$_id_objet = id_table_objet($objet);
-	}
+	if (!$objet) $objet = 'site';
+
+	$objet = objet_type($objet);
+	$_id_objet = id_table_objet($objet);
 
 	if (!is_array($options))
 		$options = unserialize($options);
 
 	if (!isset ($options['titre'])) {
 		$img = balise_img(chemin_image('image-24.png'), "", 'cadre-icone');
-		$options['titre'] = $img . $GLOBALS['logo_libelles'][($id_objet OR $objet != 'rubrique') ? $objet : 'racine'];
+		$libelles = pipeline('libeller_logo',$GLOBALS['logo_libelles']);
+		$libelle = (($id_objet OR $objet != 'rubrique') ? $objet : 'racine');
+		$libelle = (isset($libelles[$libelle])?$libelles[$libelle]:$libelles['site']);
+		$options['titre'] = $img . $libelle;
 	}
 	if (!isset ($options['editable']))
 		$options['editable'] = autoriser('iconifier',$objet,$id_objet);
@@ -143,10 +141,10 @@ function formulaires_editer_logo_traiter_dist($objet, $id_objet, $retour=''){
 	
 	// pas dans une boucle ? formulaire pour le logo du site
 	// dans ce cas, il faut chercher un 'siteon0.ext'	
-	if (!$objet)
-		$_id_objet = 'site';
-	else
-		$_id_objet = id_table_objet($objet);
+	if (!$objet) $objet = 'site';
+
+	$objet = objet_type($objet);
+	$_id_objet = id_table_objet($objet);
 
 	// supprimer l'ancien logo puis copier le nouveau
 	include_spip('inc/chercher_logo');