spip / SPIP

prive/formulaires/editer_auteur.php

Braces +1 added lines, -2 removed lines

@@ -251,8 +251,7 @@
 	if (_request('login')) {
 		// on n'est jamais cense poster le name 'login'
 		$erreurs['login'] = _T('info_non_modifiable');
-	}
-	elseif (
+	} elseif (
 		($login = _request('new_login')) && $login !== sql_getfetsel('login', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur)
 	) {
 		// on verifie la meme chose que dans auteurs_edit_config()

Spacing +13 added lines, -13 removed lines

@@ -173,7 +173,7 @@ 
 		unset($erreurs['new_login']);
 	}
 
-	$auth_methode = sql_getfetsel('source', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur);
+	$auth_methode = sql_getfetsel('source', 'spip_auteurs', 'id_auteur='.(int) $id_auteur);
 	$auth_methode = ($auth_methode ?: 'spip');
 	include_spip('inc/auth');
 
@@ -189,7 +189,7 @@ 
 			!autoriser('modifier', 'auteur', $id_auteur, null, ['email' => '?'])
 			&& $GLOBALS['visiteur_session']['id_auteur'] == $id_auteur
 			&& !strlen(trim((string) $email))
-			&& $email != ($email_ancien = sql_getfetsel('email', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur))
+			&& $email != ($email_ancien = sql_getfetsel('email', 'spip_auteurs', 'id_auteur='.(int) $id_auteur))
 		) {
 			$erreurs['email'] = (($id_auteur == $GLOBALS['visiteur_session']['id_auteur']) ? _T('form_email_non_valide') : _T('form_prop_indiquer_email'));
 		} else {
@@ -204,7 +204,7 @@ 
 			#Nouvel auteur
 			if ((int) $id_auteur == 0) {
 				#Un auteur existe deja avec cette adresse ?
-				if (sql_countsel('spip_auteurs', 'email=' . sql_quote($email)) > 0) {
+				if (sql_countsel('spip_auteurs', 'email='.sql_quote($email)) > 0) {
 					$erreurs['email'] = _T('erreur_email_deja_existant');
 				}
 			} else {
@@ -212,12 +212,12 @@ 
 				if (
 					sql_countsel(
 						'spip_auteurs',
-						'email=' . sql_quote($email)
+						'email='.sql_quote($email)
 					) > 0
 					&& $id_auteur != ($id_auteur_ancien = sql_getfetsel(
 						'id_auteur',
 						'spip_auteurs',
-						'email=' . sql_quote($email)
+						'email='.sql_quote($email)
 					))
 				) {
 					$erreurs['email'] = _T('erreur_email_deja_existant');
@@ -238,7 +238,7 @@ 
 		&& (!str_contains((string) $url, ':')
 		&& strncasecmp((string) $url, 'www.', 4) === 0)
 	) {
-		$url = 'http://' . $url;
+		$url = 'http://'.$url;
 		set_request('url_site', $url);
 	}
 	// traiter les liens implicites avant de tester l'url
@@ -253,11 +253,11 @@ 
 		$erreurs['login'] = _T('info_non_modifiable');
 	}
 	elseif (
-		($login = _request('new_login')) && $login !== sql_getfetsel('login', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur)
+		($login = _request('new_login')) && $login !== sql_getfetsel('login', 'spip_auteurs', 'id_auteur='.(int) $id_auteur)
 	) {
 		// on verifie la meme chose que dans auteurs_edit_config()
 		if (
-			! auth_autoriser_modifier_login($auth_methode)
+			!auth_autoriser_modifier_login($auth_methode)
 			|| !autoriser('modifier', 'auteur', (int) $id_auteur, null, ['login' => true])
 			// legacy : ne pas risquer d'autoriser la modif login si fonction d'autorisation pas mise a jour et ne teste que l'option email
 			|| !autoriser('modifier', 'auteur', (int) $id_auteur, null, ['email' => true])
@@ -384,7 +384,7 @@ 
 		// et de revenir sur son profil
 		if (
 			$GLOBALS['visiteur_session']['id_auteur'] == $id_auteur
-			&& $email_nouveau != ($email_ancien = sql_getfetsel('email', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur))
+			&& $email_nouveau != ($email_ancien = sql_getfetsel('email', 'spip_auteurs', 'id_auteur='.(int) $id_auteur))
 		) {
 			$envoyer_mail = charger_fonction('envoyer_mail', 'inc');
 			$texte = _T(
@@ -423,12 +423,12 @@ 
 		if (isset($erreurs['message_ok'])) {
 			if (!isset($res['message_ok'])) { $res['message_ok'] = '';
 			}
-			$res['message_ok'] = trim($res['message_ok'] . ' ' . $erreurs['message_ok']);
+			$res['message_ok'] = trim($res['message_ok'].' '.$erreurs['message_ok']);
 		}
 		if (isset($erreurs['message_erreur']) && $erreurs['message_erreur']) {
 			if (!isset($res['message_erreur'])) { $res['message_erreur'] = '';
 			}
-			$res['message_erreur'] = trim($res['message_erreur'] . ' ' . $erreurs['message_erreur']);
+			$res['message_erreur'] = trim($res['message_erreur'].' '.$erreurs['message_erreur']);
 		}
 	}
 
@@ -455,7 +455,7 @@ 
 
 
 function auteur_reset_password($id_auteur, $erreurs = []) {
-	$auteur = sql_fetsel('*', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur);
+	$auteur = sql_fetsel('*', 'spip_auteurs', 'id_auteur='.(int) $id_auteur);
 	$config = auteurs_edit_config($auteur);
 
 	if ($config['edit_pass']) {
@@ -490,7 +490,7 @@ 
 		include_spip('action/editer_auteur');
 		auteur_modifier($id_auteur, $set);
 
-		$row = sql_fetsel('*', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur);
+		$row = sql_fetsel('*', 'spip_auteurs', 'id_auteur='.(int) $id_auteur);
 		include_spip('inc/filtres');
 		if (
 			$notifier

Indentation +369 added lines, -369 removed lines

@@ -7,7 +7,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/actions');
@@ -38,24 +38,24 @@ 
  *     Environnement du formulaire
  **/
 function formulaires_editer_auteur_charger_dist(
-	$id_auteur = 'new',
-	$retour = '',
-	$associer_objet = '',
-	$config_fonc = 'auteurs_edit_config',
-	$row = [],
-	$hidden = ''
+    $id_auteur = 'new',
+    $retour = '',
+    $associer_objet = '',
+    $config_fonc = 'auteurs_edit_config',
+    $row = [],
+    $hidden = ''
 ) {
-	$valeurs = formulaires_editer_objet_charger('auteur', $id_auteur, 0, 0, $retour, $config_fonc, $row, $hidden);
-	$valeurs['new_login'] = $valeurs['login'];
+    $valeurs = formulaires_editer_objet_charger('auteur', $id_auteur, 0, 0, $retour, $config_fonc, $row, $hidden);
+    $valeurs['new_login'] = $valeurs['login'];
 
-	// S'il n'y a pas la langue, on prend la langue du site
-	$valeurs['langue'] = $valeurs['langue'] ?: $GLOBALS['meta']['langue_site'];
+    // S'il n'y a pas la langue, on prend la langue du site
+    $valeurs['langue'] = $valeurs['langue'] ?: $GLOBALS['meta']['langue_site'];
 
-	if (!autoriser('modifier', 'auteur', (int) $id_auteur)) {
-		$valeurs['editable'] = '';
-	}
+    if (!autoriser('modifier', 'auteur', (int) $id_auteur)) {
+        $valeurs['editable'] = '';
+    }
 
-	return $valeurs;
+    return $valeurs;
 }
 
 /**
@@ -79,14 +79,14 @@ 
  *     Hash du formulaire
  */
 function formulaires_editer_auteur_identifier_dist(
-	$id_auteur = 'new',
-	$retour = '',
-	$associer_objet = '',
-	$config_fonc = 'auteurs_edit_config',
-	$row = [],
-	$hidden = ''
+    $id_auteur = 'new',
+    $retour = '',
+    $associer_objet = '',
+    $config_fonc = 'auteurs_edit_config',
+    $row = [],
+    $hidden = ''
 ) {
-	return serialize([(int) $id_auteur, $associer_objet]);
+    return serialize([(int) $id_auteur, $associer_objet]);
 }
 
 
@@ -100,27 +100,27 @@ 
  */
 function auteurs_edit_config(array $row): array {
 
-	$config = [];
-	$config['lignes'] = 8;
-	$config['langue'] = $GLOBALS['spip_lang'];
-
-	// pour instituer_auteur
-	$config['auteur'] = $row;
-
-	//$config['restreint'] = ($row['statut'] == 'publie');
-	$auth_methode = $row['source'];
-	include_spip('inc/auth');
-	$config['edit_login'] = (
-		auth_autoriser_modifier_login($auth_methode)
-		&& autoriser('modifier', 'auteur', $row['id_auteur'], null, ['login' => true])
-		&& autoriser('modifier', 'auteur', $row['id_auteur'], null, ['email' => true])
-	);
-	$config['edit_pass'] = (
-			auth_autoriser_modifier_pass($auth_methode)
-			&& autoriser('modifier', 'auteur', $row['id_auteur'])
-	);
-
-	return $config;
+    $config = [];
+    $config['lignes'] = 8;
+    $config['langue'] = $GLOBALS['spip_lang'];
+
+    // pour instituer_auteur
+    $config['auteur'] = $row;
+
+    //$config['restreint'] = ($row['statut'] == 'publie');
+    $auth_methode = $row['source'];
+    include_spip('inc/auth');
+    $config['edit_login'] = (
+        auth_autoriser_modifier_login($auth_methode)
+        && autoriser('modifier', 'auteur', $row['id_auteur'], null, ['login' => true])
+        && autoriser('modifier', 'auteur', $row['id_auteur'], null, ['email' => true])
+    );
+    $config['edit_pass'] = (
+            auth_autoriser_modifier_pass($auth_methode)
+            && autoriser('modifier', 'auteur', $row['id_auteur'])
+    );
+
+    return $config;
 }
 
 /**
@@ -150,145 +150,145 @@ 
  *     Erreurs des saisies
  **/
 function formulaires_editer_auteur_verifier_dist(
-	$id_auteur = 'new',
-	$retour = '',
-	$associer_objet = '',
-	$config_fonc = 'auteurs_edit_config',
-	$row = [],
-	$hidden = ''
+    $id_auteur = 'new',
+    $retour = '',
+    $associer_objet = '',
+    $config_fonc = 'auteurs_edit_config',
+    $row = [],
+    $hidden = ''
 ) {
-	// auto-renseigner le nom si il n'existe pas, sans couper
-	titre_automatique('nom', ['email', 'login'], 255);
-
-	$oblis = ['nom'];
-	// si on veut renvoyer des identifiants il faut un email et un login
-	if (_request('reset_password')) {
-		$oblis[] = 'email';
-		$oblis[] = 'new_login';
-	}
-	// mais il reste obligatoire si on a rien trouve
-	$erreurs = formulaires_editer_objet_verifier('auteur', $id_auteur, $oblis);
-	if (isset($erreurs['new_login'])) {
-		$erreurs['login'] = $erreurs['new_login'];
-		unset($erreurs['new_login']);
-	}
-
-	$auth_methode = sql_getfetsel('source', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur);
-	$auth_methode = ($auth_methode ?: 'spip');
-	include_spip('inc/auth');
-
-	if (!nom_acceptable(_request('nom'))) {
-		$erreurs['nom'] = _T('info_nom_pas_conforme');
-	}
-
-	if ($email = _request('email')) {
-		include_spip('inc/filtres');
-		include_spip('inc/autoriser');
-		// un redacteur qui modifie son email n'a pas le droit de le vider si il y en avait un
-		if (
-			!autoriser('modifier', 'auteur', $id_auteur, null, ['email' => '?'])
-			&& $GLOBALS['visiteur_session']['id_auteur'] == $id_auteur
-			&& !strlen(trim((string) $email))
-			&& $email != ($email_ancien = sql_getfetsel('email', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur))
-		) {
-			$erreurs['email'] = (($id_auteur == $GLOBALS['visiteur_session']['id_auteur']) ? _T('form_email_non_valide') : _T('form_prop_indiquer_email'));
-		} else {
-			if (!email_valide($email)) {
-				$erreurs['email'] = (($id_auteur == $GLOBALS['visiteur_session']['id_auteur']) ? _T('form_email_non_valide') : _T('form_prop_indiquer_email'));
-			}
-		}
-		# Ne pas autoriser d'avoir deux auteurs avec le même email
-		# cette fonctionalité nécessite que la base soit clean à l'activation : pas de
-		# doublon sur la requête select email,count(*) from spip_auteurs group by email ;
-		if (defined('_INTERDIRE_AUTEUR_MEME_EMAIL')) {
-			#Nouvel auteur
-			if ((int) $id_auteur == 0) {
-				#Un auteur existe deja avec cette adresse ?
-				if (sql_countsel('spip_auteurs', 'email=' . sql_quote($email)) > 0) {
-					$erreurs['email'] = _T('erreur_email_deja_existant');
-				}
-			} else {
-				#Un auteur existe deja avec cette adresse ? et n'est pas le user courant.
-				if (
-					sql_countsel(
-						'spip_auteurs',
-						'email=' . sql_quote($email)
-					) > 0
-					&& $id_auteur != ($id_auteur_ancien = sql_getfetsel(
-						'id_auteur',
-						'spip_auteurs',
-						'email=' . sql_quote($email)
-					))
-				) {
-					$erreurs['email'] = _T('erreur_email_deja_existant');
-				}
-			}
-		}
-	}
-
-	// quand c'est un auteur existant on fait le reset password ici
-	if (!(is_countable($erreurs) ? count($erreurs) : 0) && _request('reset_password') && (int) $id_auteur) {
-		return auteur_reset_password($id_auteur, $erreurs);
-	}
-
-	// corriger un cas si frequent : www.example.org sans le http:// qui precede
-	if (
-		($url = _request('url_site'))
-		&& !tester_url_absolue($url)
-		&& (!str_contains((string) $url, ':')
-		&& strncasecmp((string) $url, 'www.', 4) === 0)
-	) {
-		$url = 'http://' . $url;
-		set_request('url_site', $url);
-	}
-	// traiter les liens implicites avant de tester l'url
-	include_spip('inc/lien');
-	if (($url = calculer_url(_request('url_site'))) && !tester_url_absolue($url)) {
-		$erreurs['url_site'] = _T('info_url_site_pas_conforme');
-	}
-
-	$erreurs['message_erreur'] = '';
-	if (_request('login')) {
-		// on n'est jamais cense poster le name 'login'
-		$erreurs['login'] = _T('info_non_modifiable');
-	}
-	elseif (
-		($login = _request('new_login')) && $login !== sql_getfetsel('login', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur)
-	) {
-		// on verifie la meme chose que dans auteurs_edit_config()
-		if (
-			! auth_autoriser_modifier_login($auth_methode)
-			|| !autoriser('modifier', 'auteur', (int) $id_auteur, null, ['login' => true])
-			// legacy : ne pas risquer d'autoriser la modif login si fonction d'autorisation pas mise a jour et ne teste que l'option email
-			|| !autoriser('modifier', 'auteur', (int) $id_auteur, null, ['email' => true])
-		) {
-			$erreurs['login'] = _T('info_non_modifiable');
-		}
-	}
-
-	if (empty($erreurs['login'])) {
-		if ($err = auth_verifier_login($auth_methode, _request('new_login'), $id_auteur)) {
-			$erreurs['login'] = $err;
-			$erreurs['message_erreur'] .= $err;
-		} else {
-			// pass trop court ou confirmation non identique
-			if ($p = _request('new_pass')) {
-				if ($p != _request('new_pass2')) {
-					$erreurs['new_pass'] = _T('info_passes_identiques');
-					$erreurs['message_erreur'] .= _T('info_passes_identiques');
-				} elseif ($err = auth_verifier_pass($auth_methode, _request('new_login'), $p, $id_auteur)) {
-					$erreurs['new_pass'] = $err;
-					$erreurs['message_erreur'] .= $err;
-				}
-			}
-		}
-	}
-
-	if (!$erreurs['message_erreur']) {
-		unset($erreurs['message_erreur']);
-	}
-
-	return $erreurs;
+    // auto-renseigner le nom si il n'existe pas, sans couper
+    titre_automatique('nom', ['email', 'login'], 255);
+
+    $oblis = ['nom'];
+    // si on veut renvoyer des identifiants il faut un email et un login
+    if (_request('reset_password')) {
+        $oblis[] = 'email';
+        $oblis[] = 'new_login';
+    }
+    // mais il reste obligatoire si on a rien trouve
+    $erreurs = formulaires_editer_objet_verifier('auteur', $id_auteur, $oblis);
+    if (isset($erreurs['new_login'])) {
+        $erreurs['login'] = $erreurs['new_login'];
+        unset($erreurs['new_login']);
+    }
+
+    $auth_methode = sql_getfetsel('source', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur);
+    $auth_methode = ($auth_methode ?: 'spip');
+    include_spip('inc/auth');
+
+    if (!nom_acceptable(_request('nom'))) {
+        $erreurs['nom'] = _T('info_nom_pas_conforme');
+    }
+
+    if ($email = _request('email')) {
+        include_spip('inc/filtres');
+        include_spip('inc/autoriser');
+        // un redacteur qui modifie son email n'a pas le droit de le vider si il y en avait un
+        if (
+            !autoriser('modifier', 'auteur', $id_auteur, null, ['email' => '?'])
+            && $GLOBALS['visiteur_session']['id_auteur'] == $id_auteur
+            && !strlen(trim((string) $email))
+            && $email != ($email_ancien = sql_getfetsel('email', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur))
+        ) {
+            $erreurs['email'] = (($id_auteur == $GLOBALS['visiteur_session']['id_auteur']) ? _T('form_email_non_valide') : _T('form_prop_indiquer_email'));
+        } else {
+            if (!email_valide($email)) {
+                $erreurs['email'] = (($id_auteur == $GLOBALS['visiteur_session']['id_auteur']) ? _T('form_email_non_valide') : _T('form_prop_indiquer_email'));
+            }
+        }
+        # Ne pas autoriser d'avoir deux auteurs avec le même email
+        # cette fonctionalité nécessite que la base soit clean à l'activation : pas de
+        # doublon sur la requête select email,count(*) from spip_auteurs group by email ;
+        if (defined('_INTERDIRE_AUTEUR_MEME_EMAIL')) {
+            #Nouvel auteur
+            if ((int) $id_auteur == 0) {
+                #Un auteur existe deja avec cette adresse ?
+                if (sql_countsel('spip_auteurs', 'email=' . sql_quote($email)) > 0) {
+                    $erreurs['email'] = _T('erreur_email_deja_existant');
+                }
+            } else {
+                #Un auteur existe deja avec cette adresse ? et n'est pas le user courant.
+                if (
+                    sql_countsel(
+                        'spip_auteurs',
+                        'email=' . sql_quote($email)
+                    ) > 0
+                    && $id_auteur != ($id_auteur_ancien = sql_getfetsel(
+                        'id_auteur',
+                        'spip_auteurs',
+                        'email=' . sql_quote($email)
+                    ))
+                ) {
+                    $erreurs['email'] = _T('erreur_email_deja_existant');
+                }
+            }
+        }
+    }
+
+    // quand c'est un auteur existant on fait le reset password ici
+    if (!(is_countable($erreurs) ? count($erreurs) : 0) && _request('reset_password') && (int) $id_auteur) {
+        return auteur_reset_password($id_auteur, $erreurs);
+    }
+
+    // corriger un cas si frequent : www.example.org sans le http:// qui precede
+    if (
+        ($url = _request('url_site'))
+        && !tester_url_absolue($url)
+        && (!str_contains((string) $url, ':')
+        && strncasecmp((string) $url, 'www.', 4) === 0)
+    ) {
+        $url = 'http://' . $url;
+        set_request('url_site', $url);
+    }
+    // traiter les liens implicites avant de tester l'url
+    include_spip('inc/lien');
+    if (($url = calculer_url(_request('url_site'))) && !tester_url_absolue($url)) {
+        $erreurs['url_site'] = _T('info_url_site_pas_conforme');
+    }
+
+    $erreurs['message_erreur'] = '';
+    if (_request('login')) {
+        // on n'est jamais cense poster le name 'login'
+        $erreurs['login'] = _T('info_non_modifiable');
+    }
+    elseif (
+        ($login = _request('new_login')) && $login !== sql_getfetsel('login', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur)
+    ) {
+        // on verifie la meme chose que dans auteurs_edit_config()
+        if (
+            ! auth_autoriser_modifier_login($auth_methode)
+            || !autoriser('modifier', 'auteur', (int) $id_auteur, null, ['login' => true])
+            // legacy : ne pas risquer d'autoriser la modif login si fonction d'autorisation pas mise a jour et ne teste que l'option email
+            || !autoriser('modifier', 'auteur', (int) $id_auteur, null, ['email' => true])
+        ) {
+            $erreurs['login'] = _T('info_non_modifiable');
+        }
+    }
+
+    if (empty($erreurs['login'])) {
+        if ($err = auth_verifier_login($auth_methode, _request('new_login'), $id_auteur)) {
+            $erreurs['login'] = $err;
+            $erreurs['message_erreur'] .= $err;
+        } else {
+            // pass trop court ou confirmation non identique
+            if ($p = _request('new_pass')) {
+                if ($p != _request('new_pass2')) {
+                    $erreurs['new_pass'] = _T('info_passes_identiques');
+                    $erreurs['message_erreur'] .= _T('info_passes_identiques');
+                } elseif ($err = auth_verifier_pass($auth_methode, _request('new_login'), $p, $id_auteur)) {
+                    $erreurs['new_pass'] = $err;
+                    $erreurs['message_erreur'] .= $err;
+                }
+            }
+        }
+    }
+
+    if (!$erreurs['message_erreur']) {
+        unset($erreurs['message_erreur']);
+    }
+
+    return $erreurs;
 }
 
 
@@ -325,153 +325,153 @@ 
  *     Retour des traitements
  **/
 function formulaires_editer_auteur_traiter_dist(
-	$id_auteur = 'new',
-	$retour = '',
-	$associer_objet = '',
-	$config_fonc = 'auteurs_edit_config',
-	$row = [],
-	$hidden = ''
+    $id_auteur = 'new',
+    $retour = '',
+    $associer_objet = '',
+    $config_fonc = 'auteurs_edit_config',
+    $row = [],
+    $hidden = ''
 ) {
-	if (_request('saisie_webmestre') || _request('webmestre')) {
-		set_request('webmestre', _request('webmestre') ?: 'non');
-	}
-
-	// si il y a des modifs sensibles (statut, mot de passe), on refuse le traitement en ajax
-	// le formulaire ne peut être traité depuis une XMLHttpRequest
-	$prev = formulaires_editer_objet_charger('auteur', $id_auteur, 0, 0, $retour, $config_fonc, $row, $hidden);
-	if (
-		_request('new_pass') // nouveau mot de passe
-		|| empty($prev['statut']) // creation auteur
-		|| _request('email') && $prev['email'] !== _request('email') // modification email
-		|| _request('statut') === '0minirezo' && $prev['statut'] !== '0minirezo' // promotion 0minirezo
-		|| _request('statut') && (int) _request('statut') < (int) $prev['statut'] // promotion de statut
-		|| _request('webmestre') && _request('webmestre') !== 'non' && $prev['webmestre'] !== 'oui' // promotion webmestre
-	) {
-		refuser_traiter_formulaire_ajax();
-		// si on arrive là encore en ajax c'est pas OK, on genere une erreur
-		if (_AJAX || !empty($_SERVER['HTTP_X_REQUESTED_WITH'])) {
-			return [
-				'message_erreur' => _T('erreur_technique_ajaxform')
-			];
-		}
-	}
-
-	$id_objet = null;
-	$retour = parametre_url($retour, 'email_confirm', '');
-
-	if ($restreintes = _request('restreintes')) {
-		foreach ($restreintes as $k => $v) {
-			if (str_starts_with((string) $v, 'rubrique|')) {
-				$restreintes[$k] = substr((string) $v, 9);
-			}
-		}
-		set_request('restreintes', $restreintes);
-	}
-
-	set_request(
-		'email',
-		email_valide(_request('email'))
-	); // eviter d'enregistrer les cas qui sont acceptés par email_valide dans le verifier :
-	// "Marie@toto.com  " ou encore "Marie Toto <Marie@toto.com>"
-
-	include_spip('inc/autoriser');
-	if (!autoriser('modifier', 'auteur', $id_auteur, null, ['email' => '?'])) {
-		$email_nouveau = _request('email');
-		set_request('email'); // vider la saisie car l'auteur n'a pas le droit de modifier cet email
-		// mais si c'est son propre profil on lui envoie un email à l'adresse qu'il a indique
-		// pour qu'il confirme qu'il possede bien cette adresse
-		// son clic sur l'url du message permettre de confirmer le changement
-		// et de revenir sur son profil
-		if (
-			$GLOBALS['visiteur_session']['id_auteur'] == $id_auteur
-			&& $email_nouveau != ($email_ancien = sql_getfetsel('email', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur))
-		) {
-			$envoyer_mail = charger_fonction('envoyer_mail', 'inc');
-			$texte = _T(
-				'form_auteur_mail_confirmation',
-				[
-					'url' => generer_action_auteur(
-						'confirmer_email',
-						$email_nouveau,
-						parametre_url($retour, 'email_modif', 'ok')
-					)
-				]
-			);
-			$envoyer_mail($email_nouveau, _T('form_auteur_confirmation'), $texte);
-			set_request('email_confirm', $email_nouveau);
-			if ($email_ancien) {
-				$envoyer_mail(
-					$email_ancien,
-					_T('form_auteur_confirmation'),
-					_T('form_auteur_envoi_mail_confirmation', ['email' => $email_nouveau])
-				);
-			}
-			$retour = parametre_url($retour, 'email_confirm', $email_nouveau);
-		}
-	}
-
-	// Trafic de langue pour enregistrer la bonne
-	if ($langue = _request('langue')) {
-		set_request('lang', $langue);
-	}
-
-	$res = formulaires_editer_objet_traiter('auteur', $id_auteur, 0, 0, $retour, $config_fonc, $row, $hidden);
-
-	if (_request('reset_password') && !(int) $id_auteur && (int) $res['id_auteur']) {
-		$erreurs = [];
-		$erreurs = auteur_reset_password($res['id_auteur'], $erreurs);
-		if (isset($erreurs['message_ok'])) {
-			if (!isset($res['message_ok'])) { $res['message_ok'] = '';
-			}
-			$res['message_ok'] = trim($res['message_ok'] . ' ' . $erreurs['message_ok']);
-		}
-		if (isset($erreurs['message_erreur']) && $erreurs['message_erreur']) {
-			if (!isset($res['message_erreur'])) { $res['message_erreur'] = '';
-			}
-			$res['message_erreur'] = trim($res['message_erreur'] . ' ' . $erreurs['message_erreur']);
-		}
-	}
-
-	// Un lien auteur a prendre en compte ?
-	if ($associer_objet && ($id_auteur = $res['id_auteur'])) {
-		$objet = '';
-		if ((int) $associer_objet) {
-			$objet = 'article';
-			$id_objet = (int) $associer_objet;
-		} elseif (preg_match(',^\w+\|\d+$,', $associer_objet)) {
-			[$objet, $id_objet] = explode('|', $associer_objet);
-		}
-		if ((bool) $objet && (bool) $id_objet && autoriser('modifier', $objet, $id_objet)) {
-			include_spip('action/editer_auteur');
-			auteur_associer($id_auteur, [$objet => $id_objet]);
-			if (isset($res['redirect'])) {
-				$res['redirect'] = parametre_url($res['redirect'], 'id_lien_ajoute', $id_auteur, '&');
-			}
-		}
-	}
-
-	return $res;
+    if (_request('saisie_webmestre') || _request('webmestre')) {
+        set_request('webmestre', _request('webmestre') ?: 'non');
+    }
+
+    // si il y a des modifs sensibles (statut, mot de passe), on refuse le traitement en ajax
+    // le formulaire ne peut être traité depuis une XMLHttpRequest
+    $prev = formulaires_editer_objet_charger('auteur', $id_auteur, 0, 0, $retour, $config_fonc, $row, $hidden);
+    if (
+        _request('new_pass') // nouveau mot de passe
+        || empty($prev['statut']) // creation auteur
+        || _request('email') && $prev['email'] !== _request('email') // modification email
+        || _request('statut') === '0minirezo' && $prev['statut'] !== '0minirezo' // promotion 0minirezo
+        || _request('statut') && (int) _request('statut') < (int) $prev['statut'] // promotion de statut
+        || _request('webmestre') && _request('webmestre') !== 'non' && $prev['webmestre'] !== 'oui' // promotion webmestre
+    ) {
+        refuser_traiter_formulaire_ajax();
+        // si on arrive là encore en ajax c'est pas OK, on genere une erreur
+        if (_AJAX || !empty($_SERVER['HTTP_X_REQUESTED_WITH'])) {
+            return [
+                'message_erreur' => _T('erreur_technique_ajaxform')
+            ];
+        }
+    }
+
+    $id_objet = null;
+    $retour = parametre_url($retour, 'email_confirm', '');
+
+    if ($restreintes = _request('restreintes')) {
+        foreach ($restreintes as $k => $v) {
+            if (str_starts_with((string) $v, 'rubrique|')) {
+                $restreintes[$k] = substr((string) $v, 9);
+            }
+        }
+        set_request('restreintes', $restreintes);
+    }
+
+    set_request(
+        'email',
+        email_valide(_request('email'))
+    ); // eviter d'enregistrer les cas qui sont acceptés par email_valide dans le verifier :
+    // "Marie@toto.com  " ou encore "Marie Toto <Marie@toto.com>"
+
+    include_spip('inc/autoriser');
+    if (!autoriser('modifier', 'auteur', $id_auteur, null, ['email' => '?'])) {
+        $email_nouveau = _request('email');
+        set_request('email'); // vider la saisie car l'auteur n'a pas le droit de modifier cet email
+        // mais si c'est son propre profil on lui envoie un email à l'adresse qu'il a indique
+        // pour qu'il confirme qu'il possede bien cette adresse
+        // son clic sur l'url du message permettre de confirmer le changement
+        // et de revenir sur son profil
+        if (
+            $GLOBALS['visiteur_session']['id_auteur'] == $id_auteur
+            && $email_nouveau != ($email_ancien = sql_getfetsel('email', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur))
+        ) {
+            $envoyer_mail = charger_fonction('envoyer_mail', 'inc');
+            $texte = _T(
+                'form_auteur_mail_confirmation',
+                [
+                    'url' => generer_action_auteur(
+                        'confirmer_email',
+                        $email_nouveau,
+                        parametre_url($retour, 'email_modif', 'ok')
+                    )
+                ]
+            );
+            $envoyer_mail($email_nouveau, _T('form_auteur_confirmation'), $texte);
+            set_request('email_confirm', $email_nouveau);
+            if ($email_ancien) {
+                $envoyer_mail(
+                    $email_ancien,
+                    _T('form_auteur_confirmation'),
+                    _T('form_auteur_envoi_mail_confirmation', ['email' => $email_nouveau])
+                );
+            }
+            $retour = parametre_url($retour, 'email_confirm', $email_nouveau);
+        }
+    }
+
+    // Trafic de langue pour enregistrer la bonne
+    if ($langue = _request('langue')) {
+        set_request('lang', $langue);
+    }
+
+    $res = formulaires_editer_objet_traiter('auteur', $id_auteur, 0, 0, $retour, $config_fonc, $row, $hidden);
+
+    if (_request('reset_password') && !(int) $id_auteur && (int) $res['id_auteur']) {
+        $erreurs = [];
+        $erreurs = auteur_reset_password($res['id_auteur'], $erreurs);
+        if (isset($erreurs['message_ok'])) {
+            if (!isset($res['message_ok'])) { $res['message_ok'] = '';
+            }
+            $res['message_ok'] = trim($res['message_ok'] . ' ' . $erreurs['message_ok']);
+        }
+        if (isset($erreurs['message_erreur']) && $erreurs['message_erreur']) {
+            if (!isset($res['message_erreur'])) { $res['message_erreur'] = '';
+            }
+            $res['message_erreur'] = trim($res['message_erreur'] . ' ' . $erreurs['message_erreur']);
+        }
+    }
+
+    // Un lien auteur a prendre en compte ?
+    if ($associer_objet && ($id_auteur = $res['id_auteur'])) {
+        $objet = '';
+        if ((int) $associer_objet) {
+            $objet = 'article';
+            $id_objet = (int) $associer_objet;
+        } elseif (preg_match(',^\w+\|\d+$,', $associer_objet)) {
+            [$objet, $id_objet] = explode('|', $associer_objet);
+        }
+        if ((bool) $objet && (bool) $id_objet && autoriser('modifier', $objet, $id_objet)) {
+            include_spip('action/editer_auteur');
+            auteur_associer($id_auteur, [$objet => $id_objet]);
+            if (isset($res['redirect'])) {
+                $res['redirect'] = parametre_url($res['redirect'], 'id_lien_ajoute', $id_auteur, '&');
+            }
+        }
+    }
+
+    return $res;
 }
 
 
 function auteur_reset_password($id_auteur, $erreurs = []) {
-	$auteur = sql_fetsel('*', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur);
-	$config = auteurs_edit_config($auteur);
-
-	if ($config['edit_pass']) {
-		if ($email = auteur_regenerer_identifiants($id_auteur)) {
-			$erreurs['message_ok'] = _T('message_nouveaux_identifiants_ok', ['email' => $email]);
-			$erreurs['message_erreur'] = '';
-		} elseif ($email === false) {
-			$erreurs['message_erreur'] = _T('message_nouveaux_identifiants_echec_envoi');
-		} else {
-			$erreurs['message_erreur'] = _T('message_nouveaux_identifiants_echec');
-		}
-	} else {
-		$erreurs['message_erreur'] = _T('message_nouveaux_identifiants_echec');
-	}
-
-	return $erreurs;
+    $auteur = sql_fetsel('*', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur);
+    $config = auteurs_edit_config($auteur);
+
+    if ($config['edit_pass']) {
+        if ($email = auteur_regenerer_identifiants($id_auteur)) {
+            $erreurs['message_ok'] = _T('message_nouveaux_identifiants_ok', ['email' => $email]);
+            $erreurs['message_erreur'] = '';
+        } elseif ($email === false) {
+            $erreurs['message_erreur'] = _T('message_nouveaux_identifiants_echec_envoi');
+        } else {
+            $erreurs['message_erreur'] = _T('message_nouveaux_identifiants_echec');
+        }
+    } else {
+        $erreurs['message_erreur'] = _T('message_nouveaux_identifiants_echec');
+    }
+
+    return $erreurs;
 }
 
 /**
@@ -482,50 +482,50 @@ 
  * @return string
  */
 function auteur_regenerer_identifiants($id_auteur, $notifier = true, $contexte = []) {
-	if ($id_auteur) {
-		$set = [];
-		include_spip('inc/access');
-		$set['pass'] = creer_pass_aleatoire(max(_PASS_LONGUEUR_MINI, 16));
-
-		include_spip('action/editer_auteur');
-		auteur_modifier($id_auteur, $set);
-
-		$row = sql_fetsel('*', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur);
-		include_spip('inc/filtres');
-		if (
-			$notifier
-			&& $row['email']
-			&& email_valide($row['email'])
-			&& trouver_fond($fond = 'modeles/mail_nouveaux_identifiants')
-		) {
-			// envoyer l'email avec login/pass
-			$c = [
-				'id_auteur' => $id_auteur,
-				'nom' => $row['nom'],
-				'mode' => $row['statut'],
-				'email' => $row['email'],
-				'pass' => $set['pass'],
-			];
-			// on merge avec les champs fournit en appel, qui sont passes au modele de notification donc
-			$contexte = array_merge($contexte, $c);
-			// si pas de langue explicitement demandee, prendre celle de l'auteur si on la connait, ou a defaut celle du site
-			// plutot que celle de l'admin qui vient de cliquer sur le bouton
-			if (!isset($contexte['lang']) || !$contexte['lang']) {
-				$contexte['lang'] = isset($row['lang']) && $row['lang']
-					? $row['lang']
-					: $GLOBALS['meta']['langue_site'];
-			}
-			lang_select($contexte['lang']);
-			$message = recuperer_fond($fond, $contexte);
-			include_spip('inc/notifications');
-			notifications_envoyer_mails($row['email'], $message);
-			lang_select();
-
-			return $row['email'];
-		}
-
-		return false;
-	}
-
-	return '';
+    if ($id_auteur) {
+        $set = [];
+        include_spip('inc/access');
+        $set['pass'] = creer_pass_aleatoire(max(_PASS_LONGUEUR_MINI, 16));
+
+        include_spip('action/editer_auteur');
+        auteur_modifier($id_auteur, $set);
+
+        $row = sql_fetsel('*', 'spip_auteurs', 'id_auteur=' . (int) $id_auteur);
+        include_spip('inc/filtres');
+        if (
+            $notifier
+            && $row['email']
+            && email_valide($row['email'])
+            && trouver_fond($fond = 'modeles/mail_nouveaux_identifiants')
+        ) {
+            // envoyer l'email avec login/pass
+            $c = [
+                'id_auteur' => $id_auteur,
+                'nom' => $row['nom'],
+                'mode' => $row['statut'],
+                'email' => $row['email'],
+                'pass' => $set['pass'],
+            ];
+            // on merge avec les champs fournit en appel, qui sont passes au modele de notification donc
+            $contexte = array_merge($contexte, $c);
+            // si pas de langue explicitement demandee, prendre celle de l'auteur si on la connait, ou a defaut celle du site
+            // plutot que celle de l'admin qui vient de cliquer sur le bouton
+            if (!isset($contexte['lang']) || !$contexte['lang']) {
+                $contexte['lang'] = isset($row['lang']) && $row['lang']
+                    ? $row['lang']
+                    : $GLOBALS['meta']['langue_site'];
+            }
+            lang_select($contexte['lang']);
+            $message = recuperer_fond($fond, $contexte);
+            include_spip('inc/notifications');
+            notifications_envoyer_mails($row['email'], $message);
+            lang_select();
+
+            return $row['email'];
+        }
+
+        return false;
+    }
+
+    return '';
 }

prive/formulaires/configurer_visiteurs.php

Indentation +39 added lines, -39 removed lines

@@ -10,51 +10,51 @@
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 include_spip('inc/presentation');
 
 function formulaires_configurer_visiteurs_charger_dist() {
-	$valeurs = [];
-	if (avoir_visiteurs(false, false)) {
-		$valeurs['editable'] = false;
-	}
-
-	foreach (
-		[
-			'accepter_visiteurs'
-		] as $m
-	) {
-		$valeurs[$m] = $GLOBALS['meta'][$m];
-	}
-
-	return $valeurs;
+    $valeurs = [];
+    if (avoir_visiteurs(false, false)) {
+        $valeurs['editable'] = false;
+    }
+
+    foreach (
+        [
+            'accepter_visiteurs'
+        ] as $m
+    ) {
+        $valeurs[$m] = $GLOBALS['meta'][$m];
+    }
+
+    return $valeurs;
 }
 
 
 function formulaires_configurer_visiteurs_traiter_dist() {
-	$res = ['editable' => true];
-	// Modification du reglage accepter_inscriptions => vider le cache
-	// (pour repercuter la modif sur le panneau de login)
-	if (
-		(($i = _request('accepter_visiteurs'))
-		&& $i != $GLOBALS['meta']['accepter_visiteurs'])
-	) {
-		include_spip('inc/invalideur');
-		suivre_invalideur('1'); # tout effacer
-	}
-
-	foreach (
-		[
-			'accepter_visiteurs',
-		] as $m
-	) {
-		if (!is_null($v = _request($m))) {
-			ecrire_meta($m, $v == 'oui' ? 'oui' : 'non');
-		}
-	}
-
-	$res['message_ok'] = _T('config_info_enregistree');
-
-	return $res;
+    $res = ['editable' => true];
+    // Modification du reglage accepter_inscriptions => vider le cache
+    // (pour repercuter la modif sur le panneau de login)
+    if (
+        (($i = _request('accepter_visiteurs'))
+        && $i != $GLOBALS['meta']['accepter_visiteurs'])
+    ) {
+        include_spip('inc/invalideur');
+        suivre_invalideur('1'); # tout effacer
+    }
+
+    foreach (
+        [
+            'accepter_visiteurs',
+        ] as $m
+    ) {
+        if (!is_null($v = _request($m))) {
+            ecrire_meta($m, $v == 'oui' ? 'oui' : 'non');
+        }
+    }
+
+    $res['message_ok'] = _T('config_info_enregistree');
+
+    return $res;
 }

prive/formulaires/configurer_preferences_menus.php

Indentation +30 added lines, -30 removed lines

@@ -29,13 +29,13 @@ 
  *     Environnement du formulaire
  **/
 function formulaires_configurer_preferences_menus_charger_dist() {
-	// travailler sur des meta fraîches
-	include_spip('inc/meta');
-	lire_metas();
-	$valeurs = [];
-	$valeurs['activer_menudev'] = table_valeur($GLOBALS['visiteur_session'], 'prefs/activer_menudev', 'non');
-	$valeurs['menus_favoris'] = obtenir_menus_favoris();
-	return $valeurs;
+    // travailler sur des meta fraîches
+    include_spip('inc/meta');
+    lire_metas();
+    $valeurs = [];
+    $valeurs['activer_menudev'] = table_valeur($GLOBALS['visiteur_session'], 'prefs/activer_menudev', 'non');
+    $valeurs['menus_favoris'] = obtenir_menus_favoris();
+    return $valeurs;
 }
 
 /**
@@ -46,32 +46,32 @@ 
  **/
 function formulaires_configurer_preferences_menus_traiter_dist() {
 
-	$activer_menudev = _request('activer_menudev');
-	$menus_favoris = _request('menus_favoris');
-	$menus_favoris = array_filter($menus_favoris);
-	$menus_favoris = array_map('intval', $menus_favoris);
+    $activer_menudev = _request('activer_menudev');
+    $menus_favoris = _request('menus_favoris');
+    $menus_favoris = array_filter($menus_favoris);
+    $menus_favoris = array_map('intval', $menus_favoris);
 
-	if (_request('reset')) {
-		$menus_favoris = [];
-		set_request('menus_favoris', null);
-	}
+    if (_request('reset')) {
+        $menus_favoris = [];
+        set_request('menus_favoris', null);
+    }
 
-	// si le menu dev change, ou les menus favoris, on recharge toute la page.
-	if (
-		table_valeur($GLOBALS['visiteur_session'], 'prefs/activer_menudev') != $activer_menudev
-		|| $menus_favoris != obtenir_menus_favoris()
-	) {
-		refuser_traiter_formulaire_ajax();
+    // si le menu dev change, ou les menus favoris, on recharge toute la page.
+    if (
+        table_valeur($GLOBALS['visiteur_session'], 'prefs/activer_menudev') != $activer_menudev
+        || $menus_favoris != obtenir_menus_favoris()
+    ) {
+        refuser_traiter_formulaire_ajax();
 
-		$GLOBALS['visiteur_session']['prefs']['activer_menudev'] = $activer_menudev;
-		$GLOBALS['visiteur_session']['prefs']['menus_favoris'] = $menus_favoris;
+        $GLOBALS['visiteur_session']['prefs']['activer_menudev'] = $activer_menudev;
+        $GLOBALS['visiteur_session']['prefs']['menus_favoris'] = $menus_favoris;
 
-		if ((int) $GLOBALS['visiteur_session']['id_auteur']) {
-			include_spip('action/editer_auteur');
-			$c = ['prefs' => serialize($GLOBALS['visiteur_session']['prefs'])];
-			auteur_modifier($GLOBALS['visiteur_session']['id_auteur'], $c);
-		}
-	}
+        if ((int) $GLOBALS['visiteur_session']['id_auteur']) {
+            include_spip('action/editer_auteur');
+            $c = ['prefs' => serialize($GLOBALS['visiteur_session']['prefs'])];
+            auteur_modifier($GLOBALS['visiteur_session']['id_auteur'], $c);
+        }
+    }
 
-	return ['message_ok' => _T('config_info_enregistree'), 'editable' => true];
+    return ['message_ok' => _T('config_info_enregistree'), 'editable' => true];
 }

prive/formulaires/oubli.php

Spacing +2 added lines, -2 removed lines

@@ -35,7 +35,7 @@ 
 			[
 				'url_reset' => url_absolue(
 					generer_url_public('spip_pass', "$param=$cookie"),
-					$GLOBALS['meta']['adresse_site'] . '/'
+					$GLOBALS['meta']['adresse_site'].'/'
 				)
 			]
 		);
@@ -101,7 +101,7 @@ 
 
 		return [
 			$declaration,
-			sql_fetsel('id_auteur,statut,pass', 'spip_auteurs', "login<>'' AND email =" . sql_quote($declaration['mail']))
+			sql_fetsel('id_auteur,statut,pass', 'spip_auteurs', "login<>'' AND email =".sql_quote($declaration['mail']))
 		];
 	}
 }

Indentation +62 added lines, -62 removed lines

@@ -10,98 +10,98 @@
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 // chargement des valeurs par defaut des champs du formulaire
 function formulaires_oubli_charger_dist() {
-	return ['oubli' => '', 'nobot' => ''];
+    return ['oubli' => '', 'nobot' => ''];
 }
 
 // https://code.spip.net/@message_oubli
 function message_oubli($email, $param) {
-	$r = formulaires_oubli_mail($email);
-
-	if (is_array($r) && $r[1] && $r[1]['statut'] !== '5poubelle' && $r[1]['pass'] !== '') {
-		include_spip('inc/texte'); # pour corriger_typo
-
-		include_spip('action/inscrire_auteur');
-		$cookie = auteur_attribuer_jeton($r[1]['id_auteur']);
-
-		// l'url_reset doit etre une URL de confiance, on force donc un url_absolue sur adresse_site
-		include_spip('inc/filtres');
-		$msg = recuperer_fond(
-			'modeles/mail_oubli',
-			[
-				'url_reset' => url_absolue(
-					generer_url_public('spip_pass', "$param=$cookie"),
-					$GLOBALS['meta']['adresse_site'] . '/'
-				)
-			]
-		);
-		include_spip('inc/notifications');
-		notifications_envoyer_mails($email, $msg);
-	}
-
-	return _T('pass_recevoir_mail');
+    $r = formulaires_oubli_mail($email);
+
+    if (is_array($r) && $r[1] && $r[1]['statut'] !== '5poubelle' && $r[1]['pass'] !== '') {
+        include_spip('inc/texte'); # pour corriger_typo
+
+        include_spip('action/inscrire_auteur');
+        $cookie = auteur_attribuer_jeton($r[1]['id_auteur']);
+
+        // l'url_reset doit etre une URL de confiance, on force donc un url_absolue sur adresse_site
+        include_spip('inc/filtres');
+        $msg = recuperer_fond(
+            'modeles/mail_oubli',
+            [
+                'url_reset' => url_absolue(
+                    generer_url_public('spip_pass', "$param=$cookie"),
+                    $GLOBALS['meta']['adresse_site'] . '/'
+                )
+            ]
+        );
+        include_spip('inc/notifications');
+        notifications_envoyer_mails($email, $msg);
+    }
+
+    return _T('pass_recevoir_mail');
 }
 
 // la saisie a ete validee, on peut agir
 function formulaires_oubli_traiter_dist() {
 
-	$message = message_oubli(_request('oubli'), 'p');
+    $message = message_oubli(_request('oubli'), 'p');
 
-	return ['message_ok' => $message];
+    return ['message_ok' => $message];
 }
 
 
 // fonction qu'on peut redefinir pour filtrer les adresses mail
 // https://code.spip.net/@test_oubli
 function test_oubli_dist($email) {
-	include_spip('inc/filtres'); # pour email_valide()
-	if (!email_valide($email)) {
-		return _T('pass_erreur_non_valide', ['email_oubli' => spip_htmlspecialchars($email)]);
-	}
+    include_spip('inc/filtres'); # pour email_valide()
+    if (!email_valide($email)) {
+        return _T('pass_erreur_non_valide', ['email_oubli' => spip_htmlspecialchars($email)]);
+    }
 
-	return ['mail' => $email];
+    return ['mail' => $email];
 }
 
 function formulaires_oubli_verifier_dist() {
-	$erreurs = [];
+    $erreurs = [];
 
-	$email = (string) _request('oubli');
+    $email = (string) _request('oubli');
 
-	$r = formulaires_oubli_mail($email);
+    $r = formulaires_oubli_mail($email);
 
-	if (!is_array($r)) {
-		$erreurs['oubli'] = $r;
-	} else {
-		if (!$r[1]) {
-			spip_logger('oubli')->info("demande de reinitialisation de mot de passe pour $email non enregistre sur le site");
-		} elseif ($r[1]['statut'] == '5poubelle' || $r[1]['pass'] == '') {
-			spip_logger('oubli')->info("demande de reinitialisation de mot de passe pour $email sans acces (poubelle ou pass vide)");
-		}
-	}
+    if (!is_array($r)) {
+        $erreurs['oubli'] = $r;
+    } else {
+        if (!$r[1]) {
+            spip_logger('oubli')->info("demande de reinitialisation de mot de passe pour $email non enregistre sur le site");
+        } elseif ($r[1]['statut'] == '5poubelle' || $r[1]['pass'] == '') {
+            spip_logger('oubli')->info("demande de reinitialisation de mot de passe pour $email sans acces (poubelle ou pass vide)");
+        }
+    }
 
-	if (_request('nobot')) {
-		$erreurs['message_erreur'] = _T('pass_rien_a_faire_ici');
-	}
+    if (_request('nobot')) {
+        $erreurs['message_erreur'] = _T('pass_rien_a_faire_ici');
+    }
 
-	return $erreurs;
+    return $erreurs;
 }
 
 function formulaires_oubli_mail($email) {
-	$f = function_exists('test_oubli') ? 'test_oubli' : 'test_oubli_dist';
-	$declaration = $f($email);
-
-	if (!is_array($declaration)) {
-		return $declaration;
-	} else {
-		include_spip('base/abstract_sql');
-
-		return [
-			$declaration,
-			sql_fetsel('id_auteur,statut,pass', 'spip_auteurs', "login<>'' AND email =" . sql_quote($declaration['mail']))
-		];
-	}
+    $f = function_exists('test_oubli') ? 'test_oubli' : 'test_oubli_dist';
+    $declaration = $f($email);
+
+    if (!is_array($declaration)) {
+        return $declaration;
+    } else {
+        include_spip('base/abstract_sql');
+
+        return [
+            $declaration,
+            sql_fetsel('id_auteur,statut,pass', 'spip_auteurs', "login<>'' AND email =" . sql_quote($declaration['mail']))
+        ];
+    }
 }

prive/formulaires/rediriger_article.php

Spacing +3 added lines, -3 removed lines

@@ -20,7 +20,7 @@ 
 		return false;
 	}
 
-	$row = sql_fetsel('id_article,virtuel', 'spip_articles', 'id_article=' . (int) $id_article);
+	$row = sql_fetsel('id_article,virtuel', 'spip_articles', 'id_article='.(int) $id_article);
 	if (!$row['id_article']) {
 		return false;
 	}
@@ -47,7 +47,7 @@ 
 function formulaires_rediriger_article_verifier_dist($id_article, $retour = '') {
 	$erreurs = [];
 
-	if (($redirection = _request('redirection')) == $id_article || $redirection == 'art' . $id_article) {
+	if (($redirection = _request('redirection')) == $id_article || $redirection == 'art'.$id_article) {
 		$erreurs['redirection'] = _T('info_redirection_boucle');
 	}
 
@@ -67,7 +67,7 @@ 
 	$js = _AJAX ? '<script type="text/javascript">if (window.ajaxReload) ajaxReload("wysiwyg");</script>' : '';
 
 	return [
-		'message_ok' => ($url ? _T('info_redirection_activee') : _T('info_redirection_desactivee')) . $js,
+		'message_ok' => ($url ? _T('info_redirection_activee') : _T('info_redirection_desactivee')).$js,
 		'editable' => true
 	];
 }

Indentation +45 added lines, -45 removed lines

@@ -10,65 +10,65 @@
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 function formulaires_rediriger_article_charger_dist($id_article, $retour = '') {
 
-	include_spip('inc/autoriser');
-	if (!autoriser('modifier', 'article', $id_article)) {
-		return false;
-	}
-
-	$row = sql_fetsel('id_article,virtuel', 'spip_articles', 'id_article=' . (int) $id_article);
-	if (!$row['id_article']) {
-		return false;
-	}
-	include_spip('inc/lien');
-	$redirection = virtuel_redirige($row['virtuel']);
-
-	if (
-		!$redirection
-		&& $GLOBALS['meta']['articles_redirection'] != 'oui'
-	) {
-		return false;
-	}
-
-
-	include_spip('inc/texte');
-	include_spip('inc/filtres_ecrire');
-
-	return [
-		'redirection' => $redirection,
-		'id' => $id_article,
-		'_afficher_url' => ($redirection ? lien_article_virtuel($redirection) : ''),
-	];
+    include_spip('inc/autoriser');
+    if (!autoriser('modifier', 'article', $id_article)) {
+        return false;
+    }
+
+    $row = sql_fetsel('id_article,virtuel', 'spip_articles', 'id_article=' . (int) $id_article);
+    if (!$row['id_article']) {
+        return false;
+    }
+    include_spip('inc/lien');
+    $redirection = virtuel_redirige($row['virtuel']);
+
+    if (
+        !$redirection
+        && $GLOBALS['meta']['articles_redirection'] != 'oui'
+    ) {
+        return false;
+    }
+
+
+    include_spip('inc/texte');
+    include_spip('inc/filtres_ecrire');
+
+    return [
+        'redirection' => $redirection,
+        'id' => $id_article,
+        '_afficher_url' => ($redirection ? lien_article_virtuel($redirection) : ''),
+    ];
 }
 
 function formulaires_rediriger_article_verifier_dist($id_article, $retour = '') {
-	$erreurs = [];
+    $erreurs = [];
 
-	if (($redirection = _request('redirection')) == $id_article || $redirection == 'art' . $id_article) {
-		$erreurs['redirection'] = _T('info_redirection_boucle');
-	}
+    if (($redirection = _request('redirection')) == $id_article || $redirection == 'art' . $id_article) {
+        $erreurs['redirection'] = _T('info_redirection_boucle');
+    }
 
-	return $erreurs;
+    return $erreurs;
 }
 
 function formulaires_rediriger_article_traiter_dist($id_article, $retour = '') {
 
-	$url = preg_replace(',^\s*https?://$,i', '', rtrim((string) _request('redirection')));
-	if ($url) {
-		$url = corriger_caracteres($url);
-	}
+    $url = preg_replace(',^\s*https?://$,i', '', rtrim((string) _request('redirection')));
+    if ($url) {
+        $url = corriger_caracteres($url);
+    }
 
-	include_spip('action/editer_article');
-	article_modifier($id_article, ['virtuel' => $url]);
+    include_spip('action/editer_article');
+    article_modifier($id_article, ['virtuel' => $url]);
 
-	$js = _AJAX ? '<script type="text/javascript">if (window.ajaxReload) ajaxReload("wysiwyg");</script>' : '';
+    $js = _AJAX ? '<script type="text/javascript">if (window.ajaxReload) ajaxReload("wysiwyg");</script>' : '';
 
-	return [
-		'message_ok' => ($url ? _T('info_redirection_activee') : _T('info_redirection_desactivee')) . $js,
-		'editable' => true
-	];
+    return [
+        'message_ok' => ($url ? _T('info_redirection_activee') : _T('info_redirection_desactivee')) . $js,
+        'editable' => true
+    ];
 }

prive/formulaires/configurer_articles.php

Indentation +49 added lines, -49 removed lines

@@ -10,65 +10,65 @@
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 function formulaires_configurer_articles_charger_dist() {
-	$valeurs = [];
-	foreach (
-		[
-			'articles_surtitre',
-			'articles_soustitre',
-			'articles_descriptif',
-			'articles_chapeau',
-			'articles_texte',
-			'articles_ps',
-			'articles_redac',
-			'articles_urlref',
-			'post_dates',
-			'articles_redirection',
-		] as $m
-	) {
-		$valeurs[$m] = $GLOBALS['meta'][$m];
-	}
+    $valeurs = [];
+    foreach (
+        [
+            'articles_surtitre',
+            'articles_soustitre',
+            'articles_descriptif',
+            'articles_chapeau',
+            'articles_texte',
+            'articles_ps',
+            'articles_redac',
+            'articles_urlref',
+            'post_dates',
+            'articles_redirection',
+        ] as $m
+    ) {
+        $valeurs[$m] = $GLOBALS['meta'][$m];
+    }
 
-	return $valeurs;
+    return $valeurs;
 }
 
 
 function formulaires_configurer_articles_traiter_dist() {
-	$res = ['editable' => true];
-	$purger_skel = false;
-	// Purger les squelettes si un changement de meta les affecte
-	if (($i = _request('post_dates')) && $i != $GLOBALS['meta']['post_dates']) {
-		$purger_skel = true;
-	}
+    $res = ['editable' => true];
+    $purger_skel = false;
+    // Purger les squelettes si un changement de meta les affecte
+    if (($i = _request('post_dates')) && $i != $GLOBALS['meta']['post_dates']) {
+        $purger_skel = true;
+    }
 
-	foreach (
-		[
-			'articles_surtitre',
-			'articles_soustitre',
-			'articles_descriptif',
-			'articles_chapeau',
-			'articles_texte',
-			'articles_ps',
-			'articles_redac',
-			'articles_urlref',
-			'post_dates',
-			'articles_redirection',
-		] as $m
-	) {
-		if (!is_null($v = _request($m))) {
-			ecrire_meta($m, $v == 'oui' ? 'oui' : 'non');
-		}
-	}
+    foreach (
+        [
+            'articles_surtitre',
+            'articles_soustitre',
+            'articles_descriptif',
+            'articles_chapeau',
+            'articles_texte',
+            'articles_ps',
+            'articles_redac',
+            'articles_urlref',
+            'post_dates',
+            'articles_redirection',
+        ] as $m
+    ) {
+        if (!is_null($v = _request($m))) {
+            ecrire_meta($m, $v == 'oui' ? 'oui' : 'non');
+        }
+    }
 
-	if ($purger_skel) {
-		include_spip('inc/invalideur');
-		purger_repertoire(_DIR_SKELS);
-	}
+    if ($purger_skel) {
+        include_spip('inc/invalideur');
+        purger_repertoire(_DIR_SKELS);
+    }
 
-	$res['message_ok'] = _T('config_info_enregistree');
+    $res['message_ok'] = _T('config_info_enregistree');
 
-	return $res;
+    return $res;
 }

prive/objets/liste/auteurs_fonctions.php

Indentation +69 added lines, -69 removed lines

@@ -15,7 +15,7 @@ 
  * @package SPIP\Core\Fonctions
  **/
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -28,29 +28,29 @@ 
  * @param bool $left
  */
 function critere_compteur_articles_filtres_dist($idb, &$boucles, $crit, $left = false) {
-	$boucle = &$boucles[$idb];
+    $boucle = &$boucles[$idb];
 
-	$_statut = calculer_liste($crit->param[0], [], $boucles, $boucle->id_parent);
+    $_statut = calculer_liste($crit->param[0], [], $boucles, $boucle->id_parent);
 
-	$not = '';
-	if ($crit->not) {
-		$not = ", 'NOT'";
-	}
-	$boucle->from['LAA'] = 'spip_auteurs_liens';
-	$boucle->from_type['LAA'] = 'left';
-	$boucle->join['LAA'] = ["'auteurs'", "'id_auteur'", "'id_auteur'", "'LAA.objet=\'article\''"];
+    $not = '';
+    if ($crit->not) {
+        $not = ", 'NOT'";
+    }
+    $boucle->from['LAA'] = 'spip_auteurs_liens';
+    $boucle->from_type['LAA'] = 'left';
+    $boucle->join['LAA'] = ["'auteurs'", "'id_auteur'", "'id_auteur'", "'LAA.objet=\'article\''"];
 
-	$boucle->from['articles'] = 'spip_articles';
-	$boucle->from_type['articles'] = 'left';
-	$boucle->join['articles'] = [
-		"'LAA'",
-		"'id_article'",
-		"'id_objet'",
-		"'(articles.statut IS NULL OR '.sql_in_quote('articles.statut',[$_statut]$not).')'"
-	];
+    $boucle->from['articles'] = 'spip_articles';
+    $boucle->from_type['articles'] = 'left';
+    $boucle->join['articles'] = [
+        "'LAA'",
+        "'id_article'",
+        "'id_objet'",
+        "'(articles.statut IS NULL OR '.sql_in_quote('articles.statut',[$_statut]$not).')'"
+    ];
 
-	$boucle->select[] = 'COUNT(articles.id_article) AS compteur_articles';
-	$boucle->group[] = 'auteurs.id_auteur';
+    $boucle->select[] = 'COUNT(articles.id_article) AS compteur_articles';
+    $boucle->group[] = 'auteurs.id_auteur';
 }
 
 /**
@@ -68,7 +68,7 @@ 
  *     Pile complétée par le code à générer
  */
 function balise_COMPTEUR_ARTICLES_dist($p) {
-	return rindex_pile($p, 'compteur_articles', 'compteur_articles_filtres');
+    return rindex_pile($p, 'compteur_articles', 'compteur_articles_filtres');
 }
 
 
@@ -84,37 +84,37 @@ 
  * @return string
  */
 function afficher_initiale($url, $initiale, $compteur, $debut, $pas) {
-	static $memo = null;
-	static $res = [];
-	$out = '';
-	if (!$memo || !$initiale && !$url || $initiale !== $memo['initiale']) {
-		$newcompt = (int) (floor(($compteur - 1) / $pas) * $pas);
-		// si fin de la pagination et une seule entree, ne pas l'afficher, ca ne sert a rien
-		if (!$initiale && !$url && !$memo['compteur']) {
-			$memo = null;
-		}
-		if ($memo) {
-			$on = ($memo['compteur'] <= $debut && ($newcompt > $debut || $newcompt == $debut && $newcompt == $memo['compteur']));
-			$res[] = "<li class='pagination-item'>" . lien_ou_expose($memo['url'], $memo['initiale'], $on ? 'span.pagination-item-label' : '', 'pagination-item-label lien_pagination') . '</li>';
-		}
-		if ($initiale) {
-			$memo = [
-				'entree' => isset($memo['entree']) ? $memo['entree'] + 1 : 0,
-				'initiale' => $initiale,
-				'url' => parametre_url($url, 'i', $initiale),
-				'compteur' => $newcompt
-			];
-		}
-	}
-	if (!$initiale && !$url) {
-		if ((is_countable($res) ? count($res) : 0) > 1) {
-			$out = "<ul class='pagination-items'>" . implode(' ', $res) . '</ul>';
-		}
-		$memo = null;
-		$res = [];
-	}
+    static $memo = null;
+    static $res = [];
+    $out = '';
+    if (!$memo || !$initiale && !$url || $initiale !== $memo['initiale']) {
+        $newcompt = (int) (floor(($compteur - 1) / $pas) * $pas);
+        // si fin de la pagination et une seule entree, ne pas l'afficher, ca ne sert a rien
+        if (!$initiale && !$url && !$memo['compteur']) {
+            $memo = null;
+        }
+        if ($memo) {
+            $on = ($memo['compteur'] <= $debut && ($newcompt > $debut || $newcompt == $debut && $newcompt == $memo['compteur']));
+            $res[] = "<li class='pagination-item'>" . lien_ou_expose($memo['url'], $memo['initiale'], $on ? 'span.pagination-item-label' : '', 'pagination-item-label lien_pagination') . '</li>';
+        }
+        if ($initiale) {
+            $memo = [
+                'entree' => isset($memo['entree']) ? $memo['entree'] + 1 : 0,
+                'initiale' => $initiale,
+                'url' => parametre_url($url, 'i', $initiale),
+                'compteur' => $newcompt
+            ];
+        }
+    }
+    if (!$initiale && !$url) {
+        if ((is_countable($res) ? count($res) : 0) > 1) {
+            $out = "<ul class='pagination-items'>" . implode(' ', $res) . '</ul>';
+        }
+        $memo = null;
+        $res = [];
+    }
 
-	return $out;
+    return $out;
 }
 
 /**
@@ -133,24 +133,24 @@ 
  * @return string
  */
 function auteur_lien_messagerie($id_auteur, $en_ligne, $statut, $imessage, $email = '') {
-	static $time = null;
-	if (!in_array($statut, ['0minirezo', '1comite'])) {
-		return '';
-	}
+    static $time = null;
+    if (!in_array($statut, ['0minirezo', '1comite'])) {
+        return '';
+    }
 
-	if (is_null($time)) {
-		$time = time();
-	}
-	$parti = (($time - strtotime($en_ligne)) > 15 * 60);
+    if (is_null($time)) {
+        $time = time();
+    }
+    $parti = (($time - strtotime($en_ligne)) > 15 * 60);
 
-	if (
-		$imessage != 'non' && !$parti // historique : est-ce que ca a encore un sens de limiter vu qu'on a la notification par email ?
-		&& $GLOBALS['meta']['messagerie_agenda'] != 'non'
-	) {
-		return parametre_url(parametre_url(generer_url_ecrire('message_edit', 'new=oui'), 'to', $id_auteur), 'redirect', self());
-	} elseif (strlen($email) && autoriser('voir', 'auteur', $id_auteur)) {
-		return 'mailto:' . $email;
-	} else {
-		return '';
-	}
+    if (
+        $imessage != 'non' && !$parti // historique : est-ce que ca a encore un sens de limiter vu qu'on a la notification par email ?
+        && $GLOBALS['meta']['messagerie_agenda'] != 'non'
+    ) {
+        return parametre_url(parametre_url(generer_url_ecrire('message_edit', 'new=oui'), 'to', $id_auteur), 'redirect', self());
+    } elseif (strlen($email) && autoriser('voir', 'auteur', $id_auteur)) {
+        return 'mailto:' . $email;
+    } else {
+        return '';
+    }
 }

Spacing +3 added lines, -3 removed lines

@@ -95,7 +95,7 @@ 
 		}
 		if ($memo) {
 			$on = ($memo['compteur'] <= $debut && ($newcompt > $debut || $newcompt == $debut && $newcompt == $memo['compteur']));
-			$res[] = "<li class='pagination-item'>" . lien_ou_expose($memo['url'], $memo['initiale'], $on ? 'span.pagination-item-label' : '', 'pagination-item-label lien_pagination') . '</li>';
+			$res[] = "<li class='pagination-item'>".lien_ou_expose($memo['url'], $memo['initiale'], $on ? 'span.pagination-item-label' : '', 'pagination-item-label lien_pagination').'</li>';
 		}
 		if ($initiale) {
 			$memo = [
@@ -108,7 +108,7 @@ 
 	}
 	if (!$initiale && !$url) {
 		if ((is_countable($res) ? count($res) : 0) > 1) {
-			$out = "<ul class='pagination-items'>" . implode(' ', $res) . '</ul>';
+			$out = "<ul class='pagination-items'>".implode(' ', $res).'</ul>';
 		}
 		$memo = null;
 		$res = [];
@@ -149,7 +149,7 @@ 
 	) {
 		return parametre_url(parametre_url(generer_url_ecrire('message_edit', 'new=oui'), 'to', $id_auteur), 'redirect', self());
 	} elseif (strlen($email) && autoriser('voir', 'auteur', $id_auteur)) {
-		return 'mailto:' . $email;
+		return 'mailto:'.$email;
 	} else {
 		return '';
 	}

prive/objets/liste/objets-trad_fonctions.php

Indentation +14 added lines, -14 removed lines

@@ -10,24 +10,24 @@
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 function lister_traductions($id_trad, $objet) {
-	$table_objet_sql = table_objet_sql($objet);
-	$primary = id_table_objet($objet);
+    $table_objet_sql = table_objet_sql($objet);
+    $primary = id_table_objet($objet);
 
-	$select = "$primary as id,lang";
-	$where = 'id_trad=' . (int) $id_trad;
-	$trouver_table = charger_fonction('trouver_table', 'base');
-	$desc = $trouver_table($table_objet_sql);
-	if (isset($desc['field']['statut'])) {
-		$select .= ',statut';
-		$where .= ' AND statut!=' . sql_quote('poubelle');
-	}
+    $select = "$primary as id,lang";
+    $where = 'id_trad=' . (int) $id_trad;
+    $trouver_table = charger_fonction('trouver_table', 'base');
+    $desc = $trouver_table($table_objet_sql);
+    if (isset($desc['field']['statut'])) {
+        $select .= ',statut';
+        $where .= ' AND statut!=' . sql_quote('poubelle');
+    }
 
-	$rows = sql_allfetsel($select, $table_objet_sql, $where);
-	lang_select();
+    $rows = sql_allfetsel($select, $table_objet_sql, $where);
+    lang_select();
 
-	return $rows;
+    return $rows;
 }

Spacing +2 added lines, -2 removed lines

@@ -18,12 +18,12 @@
 	$primary = id_table_objet($objet);
 
 	$select = "$primary as id,lang";
-	$where = 'id_trad=' . (int) $id_trad;
+	$where = 'id_trad='.(int) $id_trad;
 	$trouver_table = charger_fonction('trouver_table', 'base');
 	$desc = $trouver_table($table_objet_sql);
 	if (isset($desc['field']['statut'])) {
 		$select .= ',statut';
-		$where .= ' AND statut!=' . sql_quote('poubelle');
+		$where .= ' AND statut!='.sql_quote('poubelle');
 	}
 
 	$rows = sql_allfetsel($select, $table_objet_sql, $where);

prive/echafaudage/hierarchie/objet_fonctions.php

Indentation +6 added lines, -6 removed lines

@@ -10,7 +10,7 @@ 
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -23,9 +23,9 @@ 
  */
 function deplacement_restreint($objet, $statut) {
 
-	return match ($objet) {
-		'rubrique' => !$GLOBALS['connect_toutes_rubriques'],
-		'article', 'site', 'syndic', 'breve' => $statut == 'publie',
-		default => $statut ? $statut == 'publie' : false,
-	};
+    return match ($objet) {
+        'rubrique' => !$GLOBALS['connect_toutes_rubriques'],
+        'article', 'site', 'syndic', 'breve' => $statut == 'publie',
+        default => $statut ? $statut == 'publie' : false,
+    };
 }