From 370c52664a798615edca1608444769b34e3eb0bb Mon Sep 17 00:00:00 2001
From: "Committo,Ergo:sum" <esj@rezo.net>
Date: Mon, 13 Nov 2006 22:09:45 +0000
Subject: [PATCH] =?UTF-8?q?Validation=20XML=20de=20breves=5Fedit,=20et=20p?=
=?UTF-8?q?etits=20compl=C3=A9ments=20[7876].?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
---
ecrire/exec/articles.php | 7 +++--
ecrire/exec/auteur_infos.php | 8 ++---
ecrire/exec/breves_edit.php | 51 +++++++++++++++-----------------
ecrire/inc/chercher_rubrique.php | 2 +-
ecrire/inc/forum.php | 18 +++++------
ecrire/public/stats.php | 6 ++--
6 files changed, 45 insertions(+), 47 deletions(-)
diff --git a/ecrire/exec/articles.php b/ecrire/exec/articles.php
index a4384d29ca..a1b4e7dd90 100644
--- a/ecrire/exec/articles.php
+++ b/ecrire/exec/articles.php
@@ -97,9 +97,10 @@ function articles_affiche($id_article, $row, $cherche_auteur, $ids, $cherche_mot
$editer_auteurs = charger_fonction('editer_auteurs', 'inc');
$referencer_traduction = charger_fonction('referencer_traduction', 'inc');
- if ($flag_editable AND ($spip_display != 4))
+ if ($flag_editable AND ($spip_display != 4)) {
$iconifier = charger_fonction('iconifier', 'inc');
- else $iconifier = '';
+ $icone = $iconifier('id_article', $id_article,'articles','iconifier');
+ } else $icone = '';
$instituer_article = charger_fonction('instituer_article', 'inc');
@@ -107,7 +108,7 @@ function articles_affiche($id_article, $row, $cherche_auteur, $ids, $cherche_mot
. boite_info_articles($id_article, $statut_article, $visites, $id_version)
- . (!$iconifier ? '' : $iconifier('id_article', $id_article,'articles','iconifier'))
+ . $icone
. boites_de_config_articles($id_article)
. boite_article_virtuel($id_article, $virtuel, $flag_editable)
diff --git a/ecrire/exec/auteur_infos.php b/ecrire/exec/auteur_infos.php
index 50cfb2deb6..d3ad7a31a5 100644
--- a/ecrire/exec/auteur_infos.php
+++ b/ecrire/exec/auteur_infos.php
@@ -47,7 +47,7 @@ function exec_auteur_infos_dist()
$legender_auteur = charger_fonction('legender_auteur', 'inc');
$legender_auteur_r = $legender_auteur($id_auteur, $auteur, $initial, $echec, $redirect);
- if (_request('var_ajaxcharset')) ajax_retour($legender_auteur);
+ if (_request('var_ajaxcharset')) ajax_retour($legender_auteur_r);
if ($connect_id_auteur == $id_auteur) {
$commencer_page = charger_fonction('commencer_page', 'inc');
@@ -75,8 +75,8 @@ function exec_auteur_infos_dist()
if (statut_modifiable_auteur($id_auteur, $auteur) AND ($spip_display != 4)) {
$iconifier = charger_fonction('iconifier', 'inc');
- $iconifier = $iconifier('id_auteur', $id_auteur, 'auteur_infos','iconifier');
- } else $iconifier ='';
+ $icone = $iconifier('id_auteur', $id_auteur, 'auteur_infos','iconifier');
+ } else $icone ='';
creer_colonne_droite();
echo pipeline('affiche_droite',
@@ -86,7 +86,7 @@ function exec_auteur_infos_dist()
'data'=>'')
);
- echo $iconifier,
+ echo $icone,
debut_droite();
diff --git a/ecrire/exec/breves_edit.php b/ecrire/exec/breves_edit.php
index 49a1508c63..10288056b7 100644
--- a/ecrire/exec/breves_edit.php
+++ b/ecrire/exec/breves_edit.php
@@ -86,17 +86,17 @@ debut_cadre_formulaire();
if ($new != "oui") {
echo "\n<table cellpadding=0 cellspacing=0 border=0 width='100%'>";
- echo "<tr width='100%'>";
+ echo "\n<tr width='100%'>";
echo "<td>";
icone(_T('icone_retour'), generer_url_ecrire("breves_voir","id_breve=$id_breve"), "breve-24.gif", "rien.gif");
echo "</td>";
- echo "<td>", http_img_pack("rien.gif", ' ', "width='10'"), "</td>\n";
+ echo "\n<td>", http_img_pack("rien.gif", ' ', "width='10'"), "</td>\n";
echo "<td width='100%'>";
echo _T('info_modifier_breve');
gros_titre($titre);
echo "</td></tr></table>";
- echo "<p>";
+ echo "\n<p>";
}
@@ -107,11 +107,11 @@ if ($connect_statut=="0minirezo" OR $statut=="prop" OR $new == "oui") {
$lien_titre = entites_html($lien_titre);
$form = _T('entree_titre_obligatoire')
- . "<INPUT TYPE='text' CLASS='formo' NAME='titre' VALUE=\"$titre\" SIZE='40' $onfocus>"
+ . "<input type='text' class='formo' name='titre' value=\"$titre\" size='40' $onfocus />"
/// Dans la rubrique....
- . "<INPUT TYPE='Hidden' NAME='id_rubrique_old' VALUE=\"$id_rubrique\"><p />";
+ . "<input type='hidden' name='id_rubrique_old' value=\"$id_rubrique\" /><p />";
if ($id_rubrique == 0) $logo_parent = "racine-site-24.gif";
else {
@@ -147,19 +147,18 @@ if ($connect_statut=="0minirezo" OR $statut=="prop" OR $new == "oui") {
if ($spip_ecran == "large") $rows = 28;
else $rows = 15;
- $form .= "<p /><B>"._T('entree_texte_breve')."</B><BR>"
+ $form .= "<p /><b>"._T('entree_texte_breve')."</b><br />\n"
. afficher_barre('document.formulaire.texte')
- . "<TEXTAREA NAME='texte' ".$GLOBALS['browser_caret']." ROWS='$rows' CLASS='formo' COLS='40' wrap='soft'>"
+ . "<textarea name='texte' ".$GLOBALS['browser_caret']." rows='$rows' class='formo' cols='40'>"
. entites_html($texte)
- . "</TEXTAREA><P>\n"
-
-
- . _T('entree_liens_sites').aide ("breveslien")."<BR>"
- . _T('info_titre')."<BR>"
- . "<INPUT TYPE='text' CLASS='forml' NAME='lien_titre' VALUE=\"$lien_titre\" SIZE='40'><BR>"
-
- . _T('info_url')."<BR>"
- . "<INPUT TYPE='text' CLASS='forml' NAME='lien_url' VALUE=\"$lien_url\" SIZE='40'><P>";
+ . "</textarea><p />\n"
+ . _T('entree_liens_sites')
+ . aide ("breveslien")
+ . "<br />\n"
+ . _T('info_titre')."<br />\n"
+ . "<input type='text' class='forml' name='lien_titre' value=\"$lien_titre\" size='40' /><br />\n"
+ . _T('info_url')."<br />\n"
+ . "<input type='text' class='forml' name='lien_url' value=\"$lien_url\" size='40' /><p />";
if ($GLOBALS['champs_extra']) {
include_spip('inc/extra');
@@ -168,28 +167,26 @@ if ($connect_statut=="0minirezo" OR $statut=="prop" OR $new == "oui") {
if ($connect_statut=="0minirezo" AND acces_rubrique($id_rubrique)) {
$form .= debut_cadre_relief('', true)
- . "<B>"._T('entree_breve_publiee')."</B>\n"
-
- . "<SELECT NAME='statut' SIZE=1 CLASS='fondl'>\n"
- . "<OPTION".mySel("prop",$statut)." style='background-color: white'>"._T('item_breve_proposee')."\n"
- . "<OPTION".mySel("refuse",$statut). http_style_background('rayures-sup.gif'). ">"._T('item_breve_refusee')."\n"
- . "<OPTION".mySel("publie",$statut)." style='background-color: #B4E8C5'>"._T('item_breve_validee')."\n"
-
- . "</SELECT>".aide ("brevesstatut")."<P>\n"
+ . "<b>"._T('entree_breve_publiee')."</b>\n"
+ . "<select name='statut' size='1' class='fondl'>\n"
+ . "<option".mySel("prop",$statut)." style='background-color: white'>"._T('item_breve_proposee')."</option>\n"
+ . "<option".mySel("refuse",$statut). http_style_background('rayures-sup.gif'). ">"._T('item_breve_refusee')."</option>\n"
+ . "<option".mySel("publie",$statut)." style='background-color: #B4E8C5'>"._T('item_breve_validee')."</option>\n"
+ . "</select>".aide ("brevesstatut")."<p />\n"
. fin_cadre_relief(true);
}
- $form .= "<P ALIGN='right'><INPUT TYPE='submit' NAME='Valider' VALUE='"._T('bouton_enregistrer')."' CLASS='fondo'>";
+ $form .= "<p align='right'><input type='submit' value='"._T('bouton_enregistrer')."' class='fondo' /></p>";
echo generer_action_auteur('editer_breve',
$new ? $new : $id_breve,
- $redirect = generer_url_ecrire('breves_voir'),
+ generer_url_ecrire('breves_voir'),
$form,
" method='post' name='formulaire'"
);
}
else
- echo "<H2>"._T('info_page_interdite')."</H2>";
+ echo "<h2>"._T('info_page_interdite')."</h2>";
fin_cadre_formulaire();
echo fin_page();
diff --git a/ecrire/inc/chercher_rubrique.php b/ecrire/inc/chercher_rubrique.php
index e2cce0915f..b9feaa9d3e 100644
--- a/ecrire/inc/chercher_rubrique.php
+++ b/ecrire/inc/chercher_rubrique.php
@@ -107,7 +107,7 @@ function sous_menu_rubriques($id_rubrique, $root, $niv, &$data, &$enfants, $excl
// en fonction du niveau faire un affichage plus ou moins kikoo
// selected ?
- $selected = ($root == $id_rubrique) ? ' selected' : '';
+ $selected = ($root == $id_rubrique) ? ' selected="selected"' : '';
// class='selec_rub' sauf pour contourner le bug MSIE / MacOs 9.0
if (!($browser_name == "MSIE" AND floor($browser_version) == "5"))
diff --git a/ecrire/inc/forum.php b/ecrire/inc/forum.php
index 441419e31c..2719f38d12 100644
--- a/ecrire/inc/forum.php
+++ b/ecrire/inc/forum.php
@@ -61,14 +61,14 @@ function boutons_controle_forum($id_forum, $forum_stat, $forum_id_auteur=0, $ref
$logo = "forum-interne-24.gif";
$valider = false;
$valider_repondre = false;
- $supprimer = 'privoff';
+ $suppression = 'privoff';
break;
# forum des administrateurs
case "privadmin":
$logo = "forum-admin-24.gif";
$valider = false;
$valider_repondre = false;
- $supprimer = false;
+ $suppression = false;
break;
# forum de l'espace prive, supprime (non revalidable,
# d'ailleurs on ne sait plus a quel type de forum il appartenait)
@@ -76,14 +76,14 @@ function boutons_controle_forum($id_forum, $forum_stat, $forum_id_auteur=0, $ref
$logo = "forum-interne-24.gif";
$valider = false;
$valider_repondre = false;
- $supprimer = false;
+ $suppression = false;
break;
# forum general de l'espace prive
case "privrac":
$logo = "forum-interne-24.gif";
$valider = false;
$valider_repondre = false;
- $supprimer = 'privoff';
+ $suppression = 'privoff';
break;
# forum publie sur le site public
@@ -91,14 +91,14 @@ function boutons_controle_forum($id_forum, $forum_stat, $forum_id_auteur=0, $ref
$logo = "forum-public-24.gif";
$valider = false;
$valider_repondre = false;
- $supprimer = 'off';
+ $suppression = 'off';
break;
# forum supprime sur le site public
case "off":
$logo = "forum-public-24.gif";
$valider = 'publie';
$valider_repondre = false;
- $supprimer = false;
+ $suppression = false;
$controle = "<br /><font color='red'><b>"._T('info_message_supprime')." $forum_ip</b></font>";
if($forum_id_auteur)
$controle .= " - <a href='" . generer_url_ecrire('auteurs_edit', "id_auteur=$forum_id_auteur") .
@@ -109,15 +109,15 @@ function boutons_controle_forum($id_forum, $forum_stat, $forum_id_auteur=0, $ref
$logo = "forum-public-24.gif";
$valider = 'publie';
$valider_repondre = true;
- $supprimer = 'off';
+ $suppression = 'off';
break;
default:
return;
}
$lien = str_replace('&', '&', self()) . "#id$id_forum";
- if ($supprimer)
- $controle .= icone(_T('icone_supprimer_message'), generer_action_auteur('instituer_forum',"$id_forum-$supprimer", _DIR_RESTREINT_ABS . $lien),
+ if ($suppression)
+ $controle .= icone(_T('icone_supprimer_message'), generer_action_auteur('instituer_forum',"$id_forum-$suppression", _DIR_RESTREINT_ABS . $lien),
$logo,
"supprimer.gif", 'right', 'non');
diff --git a/ecrire/public/stats.php b/ecrire/public/stats.php
index ff7e593d2e..624e1afa7c 100644
--- a/ecrire/public/stats.php
+++ b/ecrire/public/stats.php
@@ -59,8 +59,8 @@ function public_stats_dist() {
// 1. Chercher s'il existe deja une session pour ce numero IP.
$content = array();
- $session = sous_repertoire(_DIR_TMP, 'visites') . $client_id;
- if (lire_fichier($session, $content))
+ $fichier = sous_repertoire(_DIR_TMP, 'visites') . $client_id;
+ if (lire_fichier($fichier, $content))
$content = @unserialize($content);
// 2. Plafonner le nombre de hits pris en compte pour un IP (robots etc.)
@@ -89,7 +89,7 @@ function public_stats_dist() {
$content[$log_type]++;
else $content[$log_type] = 1; // bienvenue au club
- ecrire_fichier($session, serialize($content));
+ ecrire_fichier($fichier, serialize($content));
}
}
--
GitLab