Dans la page des signatures, bien distinguer l'absence de signatures,...

Dans la page des signatures, bien distinguer l'absence de signatures, l'absence de pétitions (ou plutôt sa fermeture) et l'inexistence de l'article. Et indiquer le titre de la rubrique avec son lien, c'est parfois plus informatif que le titre de l'article associé à la pétition.

ecrire/exec/controle_petition.php

@@ -25,7 +25,6 @@ function exec_controle_petition_args($id_article, $type, $debut, $id_signature)
 {
 	include_spip('inc/presentation');
 
-	$titre =' ';
 	if (!preg_match('/^\w+$/',$type)) $type = 'public';
 	$statut='new';
 	$where = '';
@@ -35,16 +34,20 @@ function exec_controle_petition_args($id_article, $type, $debut, $id_signature)
 			$where = '(id_signature=' . sql_quote($id_signature) . ') AND ';
 	}
 	if ($id_article) {
-		if ($row = sql_fetsel("titre,statut", "spip_articles", "id_article=$id_article"));
-		if (!$row)
+		$row = sql_fetsel("titre,statut", "spip_articles", "id_article=$id_article");
+		if (!$row) {
 			$id_article = 0;
-		else {
+			$titre = '';
+		} else {
 			$titre = $row['titre'];
 			$statut = $row['statut'];	
 		}
-	}
+	} else 	$titre = '    '; // ok.
 
-	if (!(
+	if (!$titre) {
+		include_spip('inc/minipres');
+                echo minipres(_T('public:aucun_article'));
+	} elseif (!(
 		autoriser('modererpetition')
 		OR (
 			$id_article > 0
@@ -65,22 +68,15 @@ function exec_controle_petition_args($id_article, $type, $debut, $id_signature)
 
 		if (_request('var_ajaxcharset'))
 			ajax_retour($r);
-		else controle_petition_page($id_article, $debut, $type, $titre, $statut, $r);
+		else {
+		  $ong = controle_petition_onglet($id_article, $debut, $type);
+		  controle_petition_page($id_article, $titre, $ong, $statut, $r);
+		}
 	}
 }
 
-function controle_petition_page($id_article, $debut, $type, $titre, $statut, $r)
+function controle_petition_page($id_article, $titre,  $ong, $statut, $corps)
 {
-	$arg = ($id_article ? "id_article=$id_article&" :'');
-	$arg2 = ($debut ? "debut=$debut&" : '');
-	if ($type=='public') {
-	  $argp = $arg2;
-	  $argi = '';
-	} else {
-	  $argi = $arg2;
-	  $argp = '';
-	}
-
 	$commencer_page = charger_fonction('commencer_page', 'inc');
 	echo $commencer_page(_T('titre_page_controle_petition'), "forum", "suivi-petition");
 	echo debut_gauche('', true);
@@ -89,31 +85,46 @@ function controle_petition_page($id_article, $debut, $type, $titre, $statut, $r)
   
 	echo gros_titre(_T('titre_suivi_petition'),'', false);
 
-	echo debut_onglet();
-	echo onglet(_L('Signatures confirmées'), generer_url_ecrire('controle_petition', $argp . $arg . "type=public"), "public", $type=='public', "forum-public-24.gif");
-	echo onglet(_L('Signatures en attente de validation'), generer_url_ecrire('controle_petition', $argi . $arg .  "type=interne"), "interne", $type=='interne', "forum-interne-24.gif");
-	echo fin_onglet(), '<br /><br />';
-
-	if (!$titre)
-			echo _T('trad_article_inexistant');
-	else {
-		if ($id_article) {
-			echo  "<a href='",
-			  (($statut == 'publie') ? 
-			   generer_url_action('redirect', "id_article=$id_article") :
-			   generer_url_ecrire('articles', "id_article=$id_article")),
-			  "'>",
-			  typo($titre),
-			  "</a>",
-			  " <span class='arial1'>(",
-			  _T('info_numero_abbreviation'),
-			  $id_article,
-			  ")</span>";
-		}
-		$a = "editer_signature-" . $id_article;
+	echo $ong; 
 
-		echo  "<div id='", $a, "' class='serif2'>", $r, "</div>";
+	if ($id_article) {
+		echo  "<a href='",
+			(($statut == 'publie') ? 
+			generer_url_action('redirect', "id_article=$id_article") :
+			generer_url_ecrire('articles', "id_article=$id_article")),
+			"'>",
+			typo($titre),
+			"</a>",
+			" <span class='arial1'>(",
+			_T('info_numero_abbreviation'),
+			$id_article,
+			")</span>";
+		if (!sql_countsel('spip_petitions', "id_article=$id_article"))
+			echo '<br >', _L('petition close');
 	}
+	$a = "editer_signature-" . $id_article;
+
+	echo  "<br /><br /><div id='", $a, "' class='serif2'>", $corps, "</div>";
+
 	echo fin_gauche(), fin_page();
 }
+
+function controle_petition_onglet($id_article, $debut, $type)
+{
+	$arg = ($id_article ? "id_article=$id_article&" :'');
+	$arg2 = ($debut ? "debut=$debut&" : '');
+	if ($type=='public') {
+	  $argp = $arg2;
+	  $argi = '';
+	} else {
+	  $argi = $arg2;
+	  $argp = '';
+	}
+
+	return debut_onglet()
+	  . onglet(_L('Signatures confirm&eacute;es'), generer_url_ecrire('controle_petition', $argp . $arg . "type=public"), "public", $type=='public', "forum-public-24.gif")
+	.  onglet(_L('Signatures en attente de validation'), generer_url_ecrire('controle_petition', $argi . $arg .  "type=interne"), "interne", $type=='interne', "forum-interne-24.gif")
+	. fin_onglet()
+	. '<br />';
+}
 ?>

ecrire/inc/signatures.php

@@ -114,17 +114,20 @@ function signatures_edit($script, $id, $debut, $row, $type) {
 	$res .= '<br />' . message_de_signature($row);
 		
 	if (!$id) {
-			$r = sql_fetsel("titre, statut", "spip_articles", "id_article=$id_article");
-
-			$res .= "<span class='arial1' style='float: $spip_lang_right; color: black; padding-$spip_lang_left: 4px;'><b>"
-			. _T('info_numero_abbreviation')
-			. $id_article
-			. " </b></span><a href='"
-			  .  (($r['statut'] == 'publie') ? 
+			$r = sql_fetsel("titre, id_rubrique, statut", "spip_articles", "id_article=$id_article");
+			$id_rubrique = $r['id_rubrique'];
+			$publie = ($r['statut'] == 'publie');
+			$titre_a = $r['titre'];
+			$titre_r = supprimer_numero(sql_getfetsel("titre", "spip_rubriques", "id_rubrique=$id_rubrique"));
+		        $href = generer_url_ecrire('naviguer', "id_rubrique=" . $id_rubrique);
+			$res .= "<a style='float: $spip_lang_right; color: black; padding-$spip_lang_left: 4px;' href='$href' title='$id_rubrique'>"
+			. typo($titre_r)
+			. " </a><a title='$id_article' href='"
+			  .  ($publie ? 
 			      generer_url_action('redirect', "id_article=$id_article") :
 			      generer_url_ecrire('articles', "id_article=$id_article"))
 			  . "'>"
-			  . typo($r['titre'])
+			  . typo($titre_a)
 			  . "</a>";
 	}
 	$res .= "</td></tr></table>";