From 84d7f85b67ba1333b4ca7fc8e0bf0ce0756c7d4c Mon Sep 17 00:00:00 2001
From: Cerdic <cedric@yterium.com>
Date: Tue, 5 Oct 2010 10:40:38 +0000
Subject: [PATCH] afficher l'email dans la liste des auteurs, c'est plus
 informatif que le site rarement utilise du coup le lien sur la messagerie
 avec son picto n'est present qu'en cas de messagerie active. Evacuation de
 code mort autour de l'activation de la messagerie, ce qui montre qu'il n'est
 plus possible de l'activer depuis SPIP 2.0 sauf pour ceux qui utilisent le
 plugin bandeau. Cela permet de relativiser l'usage de la messagerie sur les
 nouveaux sites (ou au contraire d'apprecier l'usage generalise du bandeau ?)

---
 ecrire/inc/auteur_infos.php        | 51 ------------------------------
 prive/listes/auteurs.html          |  8 ++---
 prive/listes/auteurs_fonctions.php | 10 +++---
 prive/listes/visiteurs.html        |  8 ++---
 4 files changed, 12 insertions(+), 65 deletions(-)

diff --git a/ecrire/inc/auteur_infos.php b/ecrire/inc/auteur_infos.php
index 2b268ebb9e..287ce1f8d2 100644
--- a/ecrire/inc/auteur_infos.php
+++ b/ecrire/inc/auteur_infos.php
@@ -86,57 +86,6 @@ function inc_auteur_infos_dist($auteur, $new, $echec, $edit, $id_article, $redir
 	return $corps;
 }
 
-// http://doc.spip.org/@afficher_erreurs_auteur
-function afficher_erreurs_auteur($echec) {
-	foreach (explode('@@@',$echec) as $e)
-		$corps .= '<p>' . _T($e) . "</p>\n";
-
-	$corps = debut_cadre_relief('', true)
-	.  "<span style='color: red; left-margin: 5px'>"
-	.  http_img_pack("warning-24.png", _T('info_avertissement'), "style='width: 48px; height: 48px; float: left; margin: 5px;'")
-	. $corps
-	.  _T('info_recommencer')
-	.  "</span>\n"
-	. fin_cadre_relief(true);
-
-	return $corps;
-}
-
-
-// http://doc.spip.org/@legender_auteur_saisir
-//
-// Apparaitre dans la liste des redacteurs connectes
-//
-
-// http://doc.spip.org/@apparait_auteur_infos
-function apparait_auteur_infos($id_auteur, $auteur) {
-
-	if ($auteur['imessage']=="non"){
-		$res = "<input type='radio' name='perso_activer_imessage' value='oui' id='perso_activer_imessage_on'>"
-		. " <label for='perso_activer_imessage_on'>"._T('bouton_radio_apparaitre_liste_redacteurs_connectes')."</label> "
-		. "<br />\n<input type='radio' name='perso_activer_imessage' value='non' checked='checked' id='perso_activer_imessage_off'>"
-		. " <b><label for='perso_activer_imessage_off'>"._T('bouton_radio_non_apparaitre_liste_redacteurs_connectes')."</label></b> ";
-	} else {
-		$res = "<input type='radio' name='perso_activer_imessage' value='oui' id='perso_activer_imessage_on' checked='checked'>"
-		. " <b><label for='perso_activer_imessage_on'>"
-		. _T('bouton_radio_apparaitre_liste_redacteurs_connectes')
-		. "</label></b> "
-		. "<br />\n<input type='radio' name='perso_activer_imessage' value='non' id='perso_activer_imessage_off'>"
-		. " <label for='perso_activer_imessage_off'>"
-		. _T('bouton_radio_non_apparaitre_liste_redacteurs_connectes')
-		. "</label> ";
-	}
-
-	return 
-		debut_cadre_enfonce("messagerie-24.png", true, "", _T('info_liste_redacteurs_connectes'))
-		. "\n<div>"
-		. _T('texte_auteur_messagerie')
-		. "</div>"
-		. $res
-		. fin_cadre_enfonce(true)
-		. "<br />\n";
-}
-
 
 // http://doc.spip.org/@legender_auteur_voir
 function legender_auteur_voir($auteur) {
diff --git a/prive/listes/auteurs.html b/prive/listes/auteurs.html
index 14e1bc9eb8..eb5c57dc59 100644
--- a/prive/listes/auteurs.html
+++ b/prive/listes/auteurs.html
@@ -24,9 +24,9 @@
 		[<tr><td colspan="5"><p class='pagination'>(#PAGINATION{prive})</p></td></tr>]
 		<tr class='first_row'>
 			<th class='statut'>[(#TRI{statut,#CHEMIN{images/admin-12.gif}|balise_img{<:afficher_objets:info_statut|attribut_html:>},ajax})]</th>
-			<th class='email'></th>
+			<th class='messagerie'></th>
 			<th class='nom'>[(#TRI{multi nom,<:info_nom:>,ajax})]</th>
-			<th class='site'>[(#TRI{url_site,<:info_site:>,ajax})]</th>
+			<th class='email'>[(#TRI{email,<:email:>,ajax})]</th>
 			<th class='nombre'>[(#TRI{compteur_articles,<:info_articles:>,ajax})]</th>
 		</tr>
 	</thead>
@@ -34,9 +34,9 @@
 	<BOUCLE_liste_aut(AUTEURS){tout}{id_auteur?}{where?}{statut?}{recherche?}{tri #ENV{par,multi nom},#GET{defaut_tri}}{pagination #ENV{nb,10} aut}{!compteur_articles_filtres #ENV{filtre_statut_articles,poubelle}}>
 		<tr class="[(#COMPTEUR_BOUCLE|alterner{row_odd,row_even})][ (#EXPOSE|unique)][ (#NOM**|extraire_multi|initiale|=={#ENV{i}}|?{on}|unique)]">
 			<td class='statut'>[(#ID_AUTEUR|puce_changement_statut{#STATUT,0,auteur})]</td>
-			<td class="mail">[<a href="(#ID_AUTEUR|auteur_lien_messagerie{#EN_LIGNE,#STATUT,#IMESSAGE,#EMAIL})">[(#CHEMIN{images/m_envoi.gif}|balise_img{<:info_envoyer_message_prive:>})]</a>]</td>
+			<td class="messagerie">[<a href="(#ID_AUTEUR|auteur_lien_messagerie{#EN_LIGNE,#STATUT,#IMESSAGE})">[(#CHEMIN{images/m_envoi.gif}|balise_img{<:info_envoyer_message_prive:>})]</a>]</td>
 			<td class='nom[ (#NOM|non)vide]'>[(#LOGO_AUTEUR|image_reduire{20,26})]<a href="[(#ID_AUTEUR|generer_url_entite{auteur})]"[ title="(#BIO*|couper{200}|attribut_html)"]>[(#RANG). ][(#NOM|sinon{<:texte_vide:>})]</a></td>
-			<td class='site'>[<a href='(#URL_SITE|traiter_lien_explicite)'>[(#NOM_SITE|sinon{#URL_SITE|traiter_lien_explicite|couper{30}})]</a>]</td>
+			<td class='email'>[<a href='mailto:(#EMAIL)'>[(#EMAIL|couper{30})]</a>]</td>
 			<td class='nombre'>[(#COMPTEUR_ARTICLES|singulier_ou_pluriel{info_1_article,info_nb_articles})][<br /><span>(#PIPELINE{'compter_contributions_auteur',#ARRAY{args,#ARRAY{id_auteur,#ID_AUTEUR},data,#ARRAY{}}}|implode{'<br />'})</span>]</td>
 		</tr>
 	</BOUCLE_liste_aut>
diff --git a/prive/listes/auteurs_fonctions.php b/prive/listes/auteurs_fonctions.php
index 1bc9fc7223..ae249de150 100644
--- a/prive/listes/auteurs_fonctions.php
+++ b/prive/listes/auteurs_fonctions.php
@@ -80,7 +80,8 @@ function afficher_initiale($url,$initiale,$compteur,$debut,$pas){
 		OR ($initiale!==$memo['initiale'])
 		){
 		$newcompt = intval(floor(($compteur-1)/$pas)*$pas);
-		#var_dump("$initiale:$newcompt");
+		// si fin de la pagination et une seule entree, ne pas l'afficher, ca ne sert a rien
+		if (!$initiale AND !$url AND !$memo['entree']) return "";
 		if ($memo){
 			$on = (($memo['compteur']<=$debut)
 				AND (
@@ -89,7 +90,7 @@ function afficher_initiale($url,$initiale,$compteur,$debut,$pas){
 			$res = lien_ou_expose($memo['url'],$memo['initiale'],$on,'lien_pagination');
 		}
 		if ($initiale)
-			$memo = array('initiale'=>$initiale,'url'=>parametre_url($url,'i',$initiale),'compteur'=>$newcompt);
+			$memo = array('entree'=>isset($memo['entree'])?$memo['entree']+1:0,'initiale'=>$initiale,'url'=>parametre_url($url,'i',$initiale),'compteur'=>$newcompt);
 	}
 	return $res;
 }
@@ -109,7 +110,7 @@ function afficher_initiale($url,$initiale,$compteur,$debut,$pas){
  * @param string $email
  * @return string
  */
-function auteur_lien_messagerie($id_auteur,$en_ligne,$statut,$imessage,$email){
+function auteur_lien_messagerie($id_auteur,$en_ligne,$statut,$imessage,$email=''){
 	static $time = null;
 	if (!in_array($statut, array('0minirezo', '1comite')))
 		return '';
@@ -127,9 +128,6 @@ function auteur_lien_messagerie($id_auteur,$en_ligne,$statut,$imessage,$email){
 	else
 		return '';
 
-	return "<a href='$href' title=\""
-	  .  _T('info_envoyer_message_prive')
-	  . "\" class='message'>&nbsp;</a>";
 }
 
 ?>
\ No newline at end of file
diff --git a/prive/listes/visiteurs.html b/prive/listes/visiteurs.html
index 1ea92b3195..05f27666c7 100644
--- a/prive/listes/visiteurs.html
+++ b/prive/listes/visiteurs.html
@@ -23,9 +23,9 @@
 		[<tr><td colspan="5"><p class='pagination'>(#PAGINATION{prive})</p></td></tr>]
 		<tr class='first_row'>
 			<th class='statut'>[(#TRI{statut,#CHEMIN{images/admin-12.gif}|balise_img{<:afficher_objets:info_statut|attribut_html:>},ajax})]</th>
-			<th class='email'></th>
+			<th class='messagerie'></th>
 			<th class='nom'>[(#TRI{multi nom,<:info_nom:>,ajax})]</th>
-			<th class='site'>[(#TRI{url_site,<:info_site:>,ajax})]</th>
+			<th class='email'>[(#TRI{email,<:email:>,ajax})]</th>
 			<th class='contributions'><:info_contributions:></th>
 		</tr>
 	</thead>
@@ -33,9 +33,9 @@
 	<BOUCLE_liste_aut(AUTEURS){tout}{id_auteur?}{where?}{statut?}{recherche?}{tri #ENV{par,multi nom},#GET{defaut_tri}}{pagination #ENV{nb,10} aut}{!compteur_articles_filtres #ENV{filtre_statut_articles,poubelle}}>
 		<tr class="[(#COMPTEUR_BOUCLE|alterner{row_odd,row_even})][ (#EXPOSE|unique)][ (#NOM**|extraire_multi|initiale|=={#ENV{i}}|?{on}|unique)]">
 			<td class='statut'>[(#ID_AUTEUR|puce_changement_statut{#STATUT,0,auteur})]</td>
-			<td class="mail">[<a href="(#ID_AUTEUR|auteur_lien_messagerie{#EN_LIGNE,#STATUT,#IMESSAGE,#EMAIL})">[(#CHEMIN{images/m_envoi.gif}|balise_img{<:info_envoyer_message_prive:>})]</a>]</td>
+			<td class="messagerie">[<a href="(#ID_AUTEUR|auteur_lien_messagerie{#EN_LIGNE,#STATUT,#IMESSAGE})">[(#CHEMIN{images/m_envoi.gif}|balise_img{<:info_envoyer_message_prive:>})]</a>]</td>
 			<td class='nom[ (#NOM|non)vide]'>[(#LOGO_AUTEUR|image_reduire{20,26})]<a href="[(#ID_AUTEUR|generer_url_entite{auteur})]"[ title="(#BIO*|couper{200}|attribut_html)"]>[(#RANG). ][(#NOM|sinon{<:texte_vide:>})]</a></td>
-			<td class='site'>[<a href='(#URL_SITE|traiter_lien_explicite)'>[(#NOM_SITE|sinon{#URL_SITE|traiter_lien_explicite|couper{30}})]</a>]</td>
+			<td class='email'>[<a href='mailto:(#EMAIL)'>[(#EMAIL|couper{30})]</a>]</td>
 			<td class='contributions'>[(#PIPELINE{'compter_contributions_auteur',#ARRAY{args,#ARRAY{id_auteur,#ID_AUTEUR},'data',#ARRAY{}}}|implode{'<br />'})]</td>
 		</tr>
 	</BOUCLE_liste_aut>
-- 
GitLab