From 61fa90b0ca0054f60d676963408e64e689b7ae75 Mon Sep 17 00:00:00 2001
From: Fil <fil@rezo.net>
Date: Tue, 1 Mar 2005 14:54:08 +0000
Subject: [PATCH] quelques broutilles
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* concernant #DEBUT_SURLIGNE et #FIN_SURLIGNE : annulation du patch précédent
  qui était buggué

* petits nettoyages logs, titre forum

* cohérence affichage ecrire/upload/  (is_readable)
---
 ecrire/inc_documents.php3                     |  6 ++++--
 ecrire/inc_getdocument.php3                   |  4 +++-
 ecrire/inc_lang.php3                          |  2 +-
 formulaires/inc-formulaire_ecrire_auteur.php3 |  2 +-
 formulaires/inc-formulaire_forum.php3         |  2 +-
 inc-balises.php3                              | 18 +++++-------------
 6 files changed, 15 insertions(+), 19 deletions(-)

diff --git a/ecrire/inc_documents.php3 b/ecrire/inc_documents.php3
index 6715868bb1..02aa677295 100644
--- a/ecrire/inc_documents.php3
+++ b/ecrire/inc_documents.php3
@@ -325,11 +325,13 @@ function fichiers_upload($dir) {
 	$d = opendir($dir);
 
 	while ($f = readdir($d)) {
-		if (is_file("$dir/$f") AND $f != 'remove.txt') {
+		if (is_file("$dir/$f") AND is_readable("$dir/$f")
+		AND $f != 'remove.txt') {
 			$fichiers[] = "$dir/$f";
 		}
 		else
-		if (is_dir("$dir/$f") AND $f != '.' AND $f != '..') {
+		if (is_dir("$dir/$f") AND is_readable("$dir/$f")
+		AND $f != '.' AND $f != '..') {
 			$fichiers_dir = fichiers_upload("$dir/$f");
 			while (list(,$f2) = each ($fichiers_dir))
 				$fichiers[] = $f2;
diff --git a/ecrire/inc_getdocument.php3 b/ecrire/inc_getdocument.php3
index 5b3c036d71..ec2c04aa6a 100644
--- a/ecrire/inc_getdocument.php3
+++ b/ecrire/inc_getdocument.php3
@@ -229,8 +229,10 @@ function ajouter_un_document ($source, $nom_envoye, $type_lien, $id_lien, $mode,
 
 	// Recopier le fichier a son emplacement definitif
 	$definitif = copier_document($ext, $nom_envoye, $source);
-	if (!$definitif)
+	if (!$definitif) {
+		spip_log("Impossible de copier_document($ext, $nom_envoye, $source)");
 		return;
+	}
 
 	// Quelques infos sur le fichier
 	if (!@file_exists($definitif)
diff --git a/ecrire/inc_lang.php3 b/ecrire/inc_lang.php3
index b086ea55eb..f7dc89b9ea 100644
--- a/ecrire/inc_lang.php3
+++ b/ecrire/inc_lang.php3
@@ -42,7 +42,7 @@ function charger_langue($lang, $module = 'spip') {
 			include_lang($fichier_lang);
 			$GLOBALS['i18n_'.$module.'_'.$lang]
 				= &$GLOBALS['i18n_'.$module.'_'.$l];
-			spip_log("module de langue : ${module}_$l.php3");
+			#spip_log("module de langue : ${module}_$l.php3");
 		}
 	}
 
diff --git a/formulaires/inc-formulaire_ecrire_auteur.php3 b/formulaires/inc-formulaire_ecrire_auteur.php3
index 4639ff0f09..787548e1c4 100644
--- a/formulaires/inc-formulaire_ecrire_auteur.php3
+++ b/formulaires/inc-formulaire_ecrire_auteur.php3
@@ -45,7 +45,7 @@ function balise_FORMULAIRE_ECRIRE_AUTEUR_dyn($id_auteur, $id_article, $mail) {
 
 	// id du formulaire (pour en avoir plusieurs sur une meme page)
 	$id = ($id_auteur ? '_'.$id_auteur : '_ar'.$id_article);
-spip_log("id formulaire = $id, "._request("valide".$id));
+	#spip_log("id formulaire = $id, "._request("valide".$id));
 	$sujet = _request('sujet_message_auteur'.$id);
 	$texte = _request('texte_message_auteur'.$id);
 	$adres = _request('email_message_auteur'.$id);
diff --git a/formulaires/inc-formulaire_forum.php3 b/formulaires/inc-formulaire_forum.php3
index c0562c94c2..9ec7efb8ad 100644
--- a/formulaires/inc-formulaire_forum.php3
+++ b/formulaires/inc-formulaire_forum.php3
@@ -382,7 +382,7 @@ function sql_recherche_donnees_forum ($idr, $idf, $ida, $idb, $ids) {
 
 	if ($r) {
 		list($titre) = spip_fetch_array(spip_query($r));
-		$titre = '> ' . supprimer_numero($titre);
+		$titre = '> ' . supprimer_numero(preg_replace('/^> /', '', $titre));
 	} else {
 		$titre = _T('forum_titre_erreur');
 		$table = '';
diff --git a/inc-balises.php3 b/inc-balises.php3
index b236be8e15..48cc72fde4 100644
--- a/inc-balises.php3
+++ b/inc-balises.php3
@@ -393,23 +393,15 @@ function balise_EMBED_DOCUMENT_dist($p) {
 // sinon elles seront remplacees par les fontions de inc_surligne
 // flag_pcre est juste une flag signalant que preg_match est dispo.
 
-function code_balise_surligne($p, $b)
-{
+function balise_DEBUT_SURLIGNE_dist($p) {
 	global $flag_pcre;
-	$p->code = (!$flag_pcre ? "''" :
-		    ('(!$GLOBALS["var_recherche"] ? "" : \'<' . 
-		     $b .
-		     'span class="spip_surligneconditionnel">\')'));
-	$p->statut = 'php';
+	$p->code = ($flag_pcre ? ('\'<span class="spip_surligneconditionnel">\'') : "''");
 	return $p;
 }
-
-function balise_DEBUT_SURLIGNE_dist($p) {
-	return code_balise_surligne($p, '');
-}
-
 function balise_FIN_SURLIGNE_dist($p) {
-	return code_balise_surligne($p, '/');
+	global $flag_pcre;
+	$p->code = ($flag_pcre ? ('\'</span class="spip_surligneconditionnel">\'') : "''");
+	return $p;
 }
 
 
-- 
GitLab