From f7b847c175ebdaffa21e1880ef4a4d187851a0eb Mon Sep 17 00:00:00 2001
From: Fil <fil@rezo.net>
Date: Sun, 30 Mar 2008 21:30:16 +0000
Subject: [PATCH] #1302 (baptiste): en cas d'echec on perd tout ; pas la peine
de tout casser, en fait
---
ecrire/action/editer_auteur.php | 78 ++++++++++++++++-----------------
1 file changed, 39 insertions(+), 39 deletions(-)
diff --git a/ecrire/action/editer_auteur.php b/ecrire/action/editer_auteur.php
index 2ba7aaa01e..d9df7ad365 100644
--- a/ecrire/action/editer_auteur.php
+++ b/ecrire/action/editer_auteur.php
@@ -144,9 +144,11 @@ function action_legender_auteur_post($statut, $nom, $email, $bio, $nom_site_aute
// les admins restreints ne peuvent modifier celui des autres admins
if (autoriser('modifier', 'auteur', $id_auteur, NULL, array('mail'=>1))) {
$email = trim($email);
- if ($email !='' AND !email_valide($email))
+ if ($email !='' AND !email_valide($email)) {
$echec[]= 'info_email_invalide';
- $c['email'] = $email;
+ } else {
+ $c['email'] = $email;
+ }
}
if ($visiteur_session['id_auteur'] == $id_auteur)
@@ -168,49 +170,47 @@ function action_legender_auteur_post($statut, $nom, $email, $bio, $nom_site_aute
$c['statut'] = $statut;
// l'entrer dans la base
- if (!$echec) {
-
- // Creer l'auteur ?
- if (!$id_auteur) {
- $id_auteur = sql_insertq("spip_auteurs", array('statut' => $statut));
-
- // recuperer l'eventuel logo charge avant la creation
- $id_hack = 0 - $GLOBALS['visiteur_session']['id_auteur'];
- $chercher_logo = charger_fonction('chercher_logo', 'inc');
- if (list($logo) = $chercher_logo($id_hack, 'id_auteur', 'on'))
- rename($logo, str_replace($id_hack, $id_auteur, $logo));
- if (list($logo) = $chercher_logo($id_hack, 'id_auteur', 'off'))
- rename($logo, str_replace($id_hack, $id_auteur, $logo));
- }
- // Restreindre avant de declarer l'auteur
- // (section critique sur les droits)
- if ($id_parent) {
- if (is_array($restreintes))
- $restreintes[] = $id_parent;
- else
- $restreintes = array($id_parent);
- }
- if (is_array($restreintes)
- AND autoriser('modifier', 'auteur', $id_auteur, NULL, array('restreint'=>$restreintes))) {
- sql_delete("spip_auteurs_rubriques", "id_auteur=".sql_quote($id_auteur));
- foreach (array_unique($restreintes) as $id_rub)
- if ($id_rub = intval($id_rub)) // si '0' on ignore
- sql_insertq('spip_auteurs_rubriques', array('id_auteur' => $id_auteur, 'id_rubrique'=>$id_rub));
- }
+ // Creer l'auteur ?
+ if (!$id_auteur) {
+ $id_auteur = sql_insertq("spip_auteurs", array('statut' => $statut));
+
+ // recuperer l'eventuel logo charge avant la creation
+ $id_hack = 0 - $GLOBALS['visiteur_session']['id_auteur'];
+ $chercher_logo = charger_fonction('chercher_logo', 'inc');
+ if (list($logo) = $chercher_logo($id_hack, 'id_auteur', 'on'))
+ rename($logo, str_replace($id_hack, $id_auteur, $logo));
+ if (list($logo) = $chercher_logo($id_hack, 'id_auteur', 'off'))
+ rename($logo, str_replace($id_hack, $id_auteur, $logo));
+ }
- include_spip('inc/modifier');
+ // Restreindre avant de declarer l'auteur
+ // (section critique sur les droits)
+ if ($id_parent) {
+ if (is_array($restreintes))
+ $restreintes[] = $id_parent;
+ else
+ $restreintes = array($id_parent);
+ }
+ if (is_array($restreintes)
+ AND autoriser('modifier', 'auteur', $id_auteur, NULL, array('restreint'=>$restreintes))) {
+ sql_delete("spip_auteurs_rubriques", "id_auteur=".sql_quote($id_auteur));
+ foreach (array_unique($restreintes) as $id_rub)
+ if ($id_rub = intval($id_rub)) // si '0' on ignore
+ sql_insertq('spip_auteurs_rubriques', array('id_auteur' => $id_auteur, 'id_rubrique'=>$id_rub));
+ }
- // Lier a un article
- if (($id_article = intval($lier_id_article))
- AND autoriser('modifier', 'article', $id_article)) {
- sql_insertq('spip_auteurs_articles', array('id_article' => $id_article, 'id_auteur' =>$id_auteur));
- }
+ include_spip('inc/modifier');
- // Modifier en base (declenche les notifications etc.)
- instituer_auteur($id_auteur, $c);
+ // Lier a un article
+ if (($id_article = intval($lier_id_article))
+ AND autoriser('modifier', 'article', $id_article)) {
+ sql_insertq('spip_auteurs_articles', array('id_article' => $id_article, 'id_auteur' =>$id_auteur));
}
+ // Modifier en base (declenche les notifications etc.)
+ instituer_auteur($id_auteur, $c);
+
return array($id_auteur, $echec);
}
--
GitLab