spip / SPIP

prive/formulaires/editer_article.php

Indentation +93 added lines, -93 removed lines

@@ -16,7 +16,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/actions');
@@ -45,30 +45,30 @@ 
  *     Environnement du formulaire
  **/
 function formulaires_editer_article_charger_dist(
-	$id_article = 'new',
-	$id_rubrique = 0,
-	$retour = '',
-	$lier_trad = 0,
-	$config_fonc = 'articles_edit_config',
-	$row = [],
-	$hidden = ''
+    $id_article = 'new',
+    $id_rubrique = 0,
+    $retour = '',
+    $lier_trad = 0,
+    $config_fonc = 'articles_edit_config',
+    $row = [],
+    $hidden = ''
 ) {
-	$valeurs = formulaires_editer_objet_charger(
-		'article',
-		$id_article,
-		$id_rubrique,
-		$lier_trad,
-		$retour,
-		$config_fonc,
-		$row,
-		$hidden
-	);
+    $valeurs = formulaires_editer_objet_charger(
+        'article',
+        $id_article,
+        $id_rubrique,
+        $lier_trad,
+        $retour,
+        $config_fonc,
+        $row,
+        $hidden
+    );
 
-	if ((int) $id_article && !autoriser('modifier', 'article', (int) $id_article)) {
-		$valeurs['editable'] = '';
-	}
+    if ((int) $id_article && !autoriser('modifier', 'article', (int) $id_article)) {
+        $valeurs['editable'] = '';
+    }
 
-	return $valeurs;
+    return $valeurs;
 }
 
 /**
@@ -93,15 +93,15 @@ 
  *     Hash du formulaire
  */
 function formulaires_editer_article_identifier_dist(
-	$id_article = 'new',
-	$id_rubrique = 0,
-	$retour = '',
-	$lier_trad = 0,
-	$config_fonc = 'articles_edit_config',
-	$row = [],
-	$hidden = ''
+    $id_article = 'new',
+    $id_rubrique = 0,
+    $retour = '',
+    $lier_trad = 0,
+    $config_fonc = 'articles_edit_config',
+    $row = [],
+    $hidden = ''
 ) {
-	return serialize([(int) $id_article, $lier_trad]);
+    return serialize([(int) $id_article, $lier_trad]);
 }
 
 /**
@@ -113,11 +113,11 @@ 
  *     Configuration pour le formulaire
  */
 function articles_edit_config(array $row): array {
-	return [
-		'lignes' => 8,
-		'langue' => $GLOBALS['spip_lang'],
-		'restreint' => $row['statut'] === 'publie'
-	];
+    return [
+        'lignes' => 8,
+        'langue' => $GLOBALS['spip_lang'],
+        'restreint' => $row['statut'] === 'publie'
+    ];
 }
 
 /**
@@ -143,43 +143,43 @@ 
  *     Erreurs du formulaire
  **/
 function formulaires_editer_article_verifier_dist(
-	$id_article = 'new',
-	$id_rubrique = 0,
-	$retour = '',
-	$lier_trad = 0,
-	$config_fonc = 'articles_edit_config',
-	$row = [],
-	$hidden = ''
+    $id_article = 'new',
+    $id_rubrique = 0,
+    $retour = '',
+    $lier_trad = 0,
+    $config_fonc = 'articles_edit_config',
+    $row = [],
+    $hidden = ''
 ) {
-	// auto-renseigner le titre si il n'existe pas
-	titre_automatique('titre', ['descriptif', 'chapo', 'texte']);
-	if (!_request('id_parent') && !(int) $id_article) {
-		$valeurs = formulaires_editer_objet_charger(
-			'article',
-			$id_article,
-			$id_rubrique,
-			$lier_trad,
-			$retour,
-			$config_fonc,
-			$row,
-			$hidden
-		);
-		set_request('id_parent', $valeurs['id_parent']);
-	}
-	// on ne demande pas le titre obligatoire : il sera rempli a la volee dans editer_article si vide
-	$erreurs = formulaires_editer_objet_verifier('article', $id_article, ['id_parent']);
-	// si on utilise le formulaire dans le public
-	if (!function_exists('autoriser')) {
-		include_spip('inc/autoriser');
-	}
-	if (
-		!isset($erreurs['id_parent'])
-		&& !autoriser('creerarticledans', 'rubrique', _request('id_parent'))
-	) {
-		$erreurs['id_parent'] = _T('info_creerdansrubrique_non_autorise');
-	}
+    // auto-renseigner le titre si il n'existe pas
+    titre_automatique('titre', ['descriptif', 'chapo', 'texte']);
+    if (!_request('id_parent') && !(int) $id_article) {
+        $valeurs = formulaires_editer_objet_charger(
+            'article',
+            $id_article,
+            $id_rubrique,
+            $lier_trad,
+            $retour,
+            $config_fonc,
+            $row,
+            $hidden
+        );
+        set_request('id_parent', $valeurs['id_parent']);
+    }
+    // on ne demande pas le titre obligatoire : il sera rempli a la volee dans editer_article si vide
+    $erreurs = formulaires_editer_objet_verifier('article', $id_article, ['id_parent']);
+    // si on utilise le formulaire dans le public
+    if (!function_exists('autoriser')) {
+        include_spip('inc/autoriser');
+    }
+    if (
+        !isset($erreurs['id_parent'])
+        && !autoriser('creerarticledans', 'rubrique', _request('id_parent'))
+    ) {
+        $erreurs['id_parent'] = _T('info_creerdansrubrique_non_autorise');
+    }
 
-	return $erreurs;
+    return $erreurs;
 }
 
 /**
@@ -205,29 +205,29 @@ 
  *     Retours des traitements
  **/
 function formulaires_editer_article_traiter_dist(
-	$id_article = 'new',
-	$id_rubrique = 0,
-	$retour = '',
-	$lier_trad = 0,
-	$config_fonc = 'articles_edit_config',
-	$row = [],
-	$hidden = ''
+    $id_article = 'new',
+    $id_rubrique = 0,
+    $retour = '',
+    $lier_trad = 0,
+    $config_fonc = 'articles_edit_config',
+    $row = [],
+    $hidden = ''
 ) {
-	// ici on ignore changer_lang qui est poste en cas de trad,
-	// car l'heuristique du choix de la langue est pris en charge par article_inserer
-	// en fonction de la config du site et de la rubrique choisie
-	if (!$lier_trad) {
-		set_request('changer_lang');
-	}
+    // ici on ignore changer_lang qui est poste en cas de trad,
+    // car l'heuristique du choix de la langue est pris en charge par article_inserer
+    // en fonction de la config du site et de la rubrique choisie
+    if (!$lier_trad) {
+        set_request('changer_lang');
+    }
 
-	return formulaires_editer_objet_traiter(
-		'article',
-		$id_article,
-		$id_rubrique,
-		$lier_trad,
-		$retour,
-		$config_fonc,
-		$row,
-		$hidden
-	);
+    return formulaires_editer_objet_traiter(
+        'article',
+        $id_article,
+        $id_rubrique,
+        $lier_trad,
+        $retour,
+        $config_fonc,
+        $row,
+        $hidden
+    );
 }

prive/formulaires/editer_logo.php

Spacing +8 added lines, -8 removed lines

@@ -75,19 +75,19 @@ 
 		}
 		switch ($objet) {
 			case 'article':
-				$libelle .= ' ' . aider('logoart');
+				$libelle .= ' '.aider('logoart');
 				break;
 			case 'breve':
-				$libelle .= ' ' . aider('breveslogo');
+				$libelle .= ' '.aider('breveslogo');
 				break;
 			case 'rubrique':
-				$libelle .= ' ' . aider('rublogo');
+				$libelle .= ' '.aider('rublogo');
 				break;
 			default:
 				break;
 		}
 
-		$options['titre'] = $img . $libelle;
+		$options['titre'] = $img.$libelle;
 	}
 	if (!isset($options['editable'])) {
 		include_spip('inc/autoriser');
@@ -112,8 +112,8 @@ 
 	foreach ($etats as $etat) {
 		$logo = $chercher_logo($id_objet, $_id_objet, $etat);
 		if ($logo) {
-			$res['logo_' . $etat] = $logo[0];
-			$res['logo_id_' . $etat] = $logo[5]['id_document'] ?? '';
+			$res['logo_'.$etat] = $logo[0];
+			$res['logo_id_'.$etat] = $logo[5]['id_document'] ?? '';
 		}
 	}
 	// pas de logo_on -> pas de formulaire pour le survol
@@ -175,7 +175,7 @@ 
 					$extensions_possibles
 				)
 			) {
-				$erreurs['logo_' . $etat] = _L('Extension non reconnue');
+				$erreurs['logo_'.$etat] = _L('Extension non reconnue');
 			}
 		} elseif ($file && $file['error'] != 0 && isset($file['msg'])) {
 			$erreurs['message_erreur'] = $file['msg'];
@@ -260,7 +260,7 @@ 
 	include_spip('inc/documents');
 	$sources = [];
 	foreach (['on', 'off'] as $etat) {
-		$logo = 'logo_' . $etat;
+		$logo = 'logo_'.$etat;
 		if (isset($_FILES[$logo])) {
 			if ($_FILES[$logo]['error'] == 0) {
 				$sources[$etat] = $_FILES[$logo];

Indentation +157 added lines, -157 removed lines

@@ -29,7 +29,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 // utilise pour le logo du site, donc doit rester ici
@@ -47,88 +47,88 @@ 
  * @return array|false Variables d'environnement pour le fond
  */
 function formulaires_editer_logo_charger_dist($objet, $id_objet, $retour = '', $options = []) {
-	// pas dans une boucle ? formulaire pour le logo du site
-	// dans ce cas, il faut chercher un 'siteon0.ext'
-	if (!$objet) {
-		$objet = 'site';
-	}
+    // pas dans une boucle ? formulaire pour le logo du site
+    // dans ce cas, il faut chercher un 'siteon0.ext'
+    if (!$objet) {
+        $objet = 'site';
+    }
 
-	$objet = objet_type($objet);
-	$_id_objet = id_table_objet($objet);
+    $objet = objet_type($objet);
+    $_id_objet = id_table_objet($objet);
 
-	if (!is_array($options)) {
-		$options = unserialize($options);
-	}
-	$options = spip_sanitize_from_request($options, '*');
+    if (!is_array($options)) {
+        $options = unserialize($options);
+    }
+    $options = spip_sanitize_from_request($options, '*');
 
-	if (!isset($options['titre'])) {
-		$balise_img = chercher_filtre('balise_img');
-		$img = $balise_img(chemin_image('image-24.png'), '', 'cadre-icone');
-		$libelles = pipeline('libeller_logo', $GLOBALS['logo_libelles']);
-		$libelle = (($id_objet || $objet != 'rubrique') ? $objet : 'racine');
-		if (isset($libelles[$libelle])) {
-			$libelle = $libelles[$libelle];
-		} elseif ($libelle = objet_info($objet, 'texte_logo_objet')) {
-			$libelle = _T($libelle);
-		} else {
-			$libelle = _L('Logo');
-		}
-		switch ($objet) {
-			case 'article':
-				$libelle .= ' ' . aider('logoart');
-				break;
-			case 'breve':
-				$libelle .= ' ' . aider('breveslogo');
-				break;
-			case 'rubrique':
-				$libelle .= ' ' . aider('rublogo');
-				break;
-			default:
-				break;
-		}
+    if (!isset($options['titre'])) {
+        $balise_img = chercher_filtre('balise_img');
+        $img = $balise_img(chemin_image('image-24.png'), '', 'cadre-icone');
+        $libelles = pipeline('libeller_logo', $GLOBALS['logo_libelles']);
+        $libelle = (($id_objet || $objet != 'rubrique') ? $objet : 'racine');
+        if (isset($libelles[$libelle])) {
+            $libelle = $libelles[$libelle];
+        } elseif ($libelle = objet_info($objet, 'texte_logo_objet')) {
+            $libelle = _T($libelle);
+        } else {
+            $libelle = _L('Logo');
+        }
+        switch ($objet) {
+            case 'article':
+                $libelle .= ' ' . aider('logoart');
+                break;
+            case 'breve':
+                $libelle .= ' ' . aider('breveslogo');
+                break;
+            case 'rubrique':
+                $libelle .= ' ' . aider('rublogo');
+                break;
+            default:
+                break;
+        }
 
-		$options['titre'] = $img . $libelle;
-	}
-	if (!isset($options['editable'])) {
-		include_spip('inc/autoriser');
-		$options['editable'] = autoriser('iconifier', $objet, $id_objet);
-	}
+        $options['titre'] = $img . $libelle;
+    }
+    if (!isset($options['editable'])) {
+        include_spip('inc/autoriser');
+        $options['editable'] = autoriser('iconifier', $objet, $id_objet);
+    }
 
-	$res = [
-		'editable' => ($GLOBALS['meta']['activer_logos'] == 'oui' ? ' ' : '') && (!isset($options['editable']) || $options['editable']),
-		'logo_survol' => ($GLOBALS['meta']['activer_logos_survol'] == 'oui' ? ' ' : ''),
-		'objet' => $objet,
-		'id_objet' => $id_objet,
-		'_options' => $options,
-		'_show_upload_off' => '',
-	];
+    $res = [
+        'editable' => ($GLOBALS['meta']['activer_logos'] == 'oui' ? ' ' : '') && (!isset($options['editable']) || $options['editable']),
+        'logo_survol' => ($GLOBALS['meta']['activer_logos_survol'] == 'oui' ? ' ' : ''),
+        'objet' => $objet,
+        'id_objet' => $id_objet,
+        '_options' => $options,
+        '_show_upload_off' => '',
+    ];
 
-	// rechercher le logo de l'objet
-	// la fonction prend un parametre '_id_objet' etrange :
-	// le nom de la cle primaire (et non le nom de la table)
-	// ou directement le nom du raccourcis a chercher
-	$chercher_logo = charger_fonction('chercher_logo', 'inc');
-	$etats = $res['logo_survol'] ? ['on', 'off'] : ['on'];
-	foreach ($etats as $etat) {
-		$logo = $chercher_logo($id_objet, $_id_objet, $etat);
-		if ($logo) {
-			$res['logo_' . $etat] = $logo[0];
-			$res['logo_id_' . $etat] = $logo[5]['id_document'] ?? '';
-		}
-	}
-	// pas de logo_on -> pas de formulaire pour le survol
-	if (!isset($res['logo_on'])) {
-		$res['logo_survol'] = '';
-	} elseif (!isset($res['logo_off']) && _request('logo_up')) {
-		$res['_show_upload_off'] = ' ';
-	}
+    // rechercher le logo de l'objet
+    // la fonction prend un parametre '_id_objet' etrange :
+    // le nom de la cle primaire (et non le nom de la table)
+    // ou directement le nom du raccourcis a chercher
+    $chercher_logo = charger_fonction('chercher_logo', 'inc');
+    $etats = $res['logo_survol'] ? ['on', 'off'] : ['on'];
+    foreach ($etats as $etat) {
+        $logo = $chercher_logo($id_objet, $_id_objet, $etat);
+        if ($logo) {
+            $res['logo_' . $etat] = $logo[0];
+            $res['logo_id_' . $etat] = $logo[5]['id_document'] ?? '';
+        }
+    }
+    // pas de logo_on -> pas de formulaire pour le survol
+    if (!isset($res['logo_on'])) {
+        $res['logo_survol'] = '';
+    } elseif (!isset($res['logo_off']) && _request('logo_up')) {
+        $res['_show_upload_off'] = ' ';
+    }
 
-	// si le logo n'est pas editable et qu'il n'y en a pas, on affiche pas du tout le formulaire
-	if (!$res['editable'] && !isset($res['logo_off']) && !isset($res['logo_on'])) {
-		return false;
-	}
+    // si le logo n'est pas editable et qu'il n'y en a pas, on affiche pas du tout le formulaire
+    if (!$res['editable'] && !isset($res['logo_off']) && !isset($res['logo_on'])) {
+        return false;
+    }
 
-	return $res;
+    return $res;
 }
 
 /**
@@ -142,7 +142,7 @@ 
  * @return string              Hash du formulaire
  */
 function formulaires_editer_logo_identifier_dist($objet, $id_objet, $retour = '', $options = []) {
-	return serialize([$objet, $id_objet]);
+    return serialize([$objet, $id_objet]);
 }
 
 /**
@@ -158,31 +158,31 @@ 
  * @return array               Erreurs du formulaire
  */
 function formulaires_editer_logo_verifier_dist($objet, $id_objet, $retour = '', $options = []) {
-	$erreurs = [];
-	// verifier les extensions
-	$sources = formulaire_editer_logo_get_sources();
-	include_spip('action/editer_logo');
-	$extensions_possibles = $GLOBALS['formats_logos'];
-	if (in_array('jpg', $extensions_possibles)) {
-		$extensions_possibles[] = 'jpeg';
-	}
-	foreach ($sources as $etat => $file) {
-		// seulement si une reception correcte a eu lieu
-		if ($file && $file['error'] == 0) {
-			if (
-				!in_array(
-					strtolower(pathinfo((string) $file['name'], PATHINFO_EXTENSION)),
-					$extensions_possibles
-				)
-			) {
-				$erreurs['logo_' . $etat] = _L('Extension non reconnue');
-			}
-		} elseif ($file && $file['error'] != 0 && isset($file['msg'])) {
-			$erreurs['message_erreur'] = $file['msg'];
-		}
-	}
+    $erreurs = [];
+    // verifier les extensions
+    $sources = formulaire_editer_logo_get_sources();
+    include_spip('action/editer_logo');
+    $extensions_possibles = $GLOBALS['formats_logos'];
+    if (in_array('jpg', $extensions_possibles)) {
+        $extensions_possibles[] = 'jpeg';
+    }
+    foreach ($sources as $etat => $file) {
+        // seulement si une reception correcte a eu lieu
+        if ($file && $file['error'] == 0) {
+            if (
+                !in_array(
+                    strtolower(pathinfo((string) $file['name'], PATHINFO_EXTENSION)),
+                    $extensions_possibles
+                )
+            ) {
+                $erreurs['logo_' . $etat] = _L('Extension non reconnue');
+            }
+        } elseif ($file && $file['error'] != 0 && isset($file['msg'])) {
+            $erreurs['message_erreur'] = $file['msg'];
+        }
+    }
 
-	return $erreurs;
+    return $erreurs;
 }
 
 /**
@@ -198,47 +198,47 @@ 
  * @return array               Retour des traitements
  */
 function formulaires_editer_logo_traiter_dist($objet, $id_objet, $retour = '', $options = []) {
-	$res = ['editable' => ' '];
+    $res = ['editable' => ' '];
 
-	// pas dans une boucle ? formulaire pour le logo du site
-	// dans ce cas, il faut chercher un 'siteon0.ext'
-	if (!$objet) {
-		$objet = 'site';
-	}
+    // pas dans une boucle ? formulaire pour le logo du site
+    // dans ce cas, il faut chercher un 'siteon0.ext'
+    if (!$objet) {
+        $objet = 'site';
+    }
 
-	include_spip('action/editer_logo');
+    include_spip('action/editer_logo');
 
-	// effectuer la suppression si demandee d'un logo
-	$on = _request('supprimer_logo_on');
-	if ($on || _request('supprimer_logo_off')) {
-		logo_supprimer($objet, $id_objet, $on ? 'on' : 'off');
-		$res['message_ok'] = ''; // pas besoin de message : la validation est visuelle
-		set_request('logo_up', ' ');
-	} // sinon supprimer ancien logo puis copier le nouveau
-	else {
-		$sources = formulaire_editer_logo_get_sources();
-		foreach ($sources as $etat => $file) {
-			if ($file && $file['error'] == 0) {
-				if ($err = logo_modifier($objet, $id_objet, $etat, $file)) {
-					$res['message_erreur'] = $err;
-				} else {
-					$res['message_ok'] = '';
-				} // pas besoin de message : la validation est visuelle
-				set_request('logo_up', ' ');
-			}
-		}
-	}
+    // effectuer la suppression si demandee d'un logo
+    $on = _request('supprimer_logo_on');
+    if ($on || _request('supprimer_logo_off')) {
+        logo_supprimer($objet, $id_objet, $on ? 'on' : 'off');
+        $res['message_ok'] = ''; // pas besoin de message : la validation est visuelle
+        set_request('logo_up', ' ');
+    } // sinon supprimer ancien logo puis copier le nouveau
+    else {
+        $sources = formulaire_editer_logo_get_sources();
+        foreach ($sources as $etat => $file) {
+            if ($file && $file['error'] == 0) {
+                if ($err = logo_modifier($objet, $id_objet, $etat, $file)) {
+                    $res['message_erreur'] = $err;
+                } else {
+                    $res['message_ok'] = '';
+                } // pas besoin de message : la validation est visuelle
+                set_request('logo_up', ' ');
+            }
+        }
+    }
 
-	// Invalider les caches de l'objet
-	include_spip('inc/invalideur');
-	suivre_invalideur("id='$objet/$id_objet'");
+    // Invalider les caches de l'objet
+    include_spip('inc/invalideur');
+    suivre_invalideur("id='$objet/$id_objet'");
 
 
-	if ($retour) {
-		$res['redirect'] = $retour;
-	}
+    if ($retour) {
+        $res['redirect'] = $retour;
+    }
 
-	return $res;
+    return $res;
 }
 
 
@@ -250,29 +250,29 @@ 
  *     Sources des fichiers dans les clés `on` ou `off`
  */
 function formulaire_editer_logo_get_sources() {
-	if (!$_FILES) {
-		$_FILES = $GLOBALS['HTTP_POST_FILES'] ?? [];
-	}
-	if (!is_array($_FILES)) {
-		return [];
-	}
+    if (!$_FILES) {
+        $_FILES = $GLOBALS['HTTP_POST_FILES'] ?? [];
+    }
+    if (!is_array($_FILES)) {
+        return [];
+    }
 
-	include_spip('inc/documents');
-	$sources = [];
-	foreach (['on', 'off'] as $etat) {
-		$logo = 'logo_' . $etat;
-		if (isset($_FILES[$logo])) {
-			if ($_FILES[$logo]['error'] == 0) {
-				$sources[$etat] = $_FILES[$logo];
-			} elseif ($_FILES[$logo]['error'] != 0) {
-				$msg = check_upload_error($_FILES[$logo]['error'], false, true);
-				if ($msg && is_string($msg)) {
-					$sources[$etat] = $_FILES[$logo];
-					$sources[$etat]['msg'] = $msg;
-				}
-			}
-		}
-	}
+    include_spip('inc/documents');
+    $sources = [];
+    foreach (['on', 'off'] as $etat) {
+        $logo = 'logo_' . $etat;
+        if (isset($_FILES[$logo])) {
+            if ($_FILES[$logo]['error'] == 0) {
+                $sources[$etat] = $_FILES[$logo];
+            } elseif ($_FILES[$logo]['error'] != 0) {
+                $msg = check_upload_error($_FILES[$logo]['error'], false, true);
+                if ($msg && is_string($msg)) {
+                    $sources[$etat] = $_FILES[$logo];
+                    $sources[$etat]['msg'] = $msg;
+                }
+            }
+        }
+    }
 
-	return $sources;
+    return $sources;
 }

prive/formulaires/traduire.php

Spacing +6 added lines, -6 removed lines

@@ -59,7 +59,7 @@ 
 		$id_parent = $valeurs['id_parent'];
 	}
 	if ($id_parent) {
-		$langue_parent = sql_getfetsel('lang', 'spip_rubriques', 'id_rubrique=' . (int) $id_parent);
+		$langue_parent = sql_getfetsel('lang', 'spip_rubriques', 'id_rubrique='.(int) $id_parent);
 	}
 
 	if (!$langue_parent) {
@@ -82,8 +82,8 @@ 
 	$valeurs['_traduire'] = '';
 	if (isset($valeurs['id_trad'])) {
 		$valeurs['_traduire'] = ($traduire ? ' ' : '');
-		$valeurs['_vue_traductions'] = 'prive/objets/liste/' . (trouver_fond(
-			$f = table_objet($objet) . '-trad',
+		$valeurs['_vue_traductions'] = 'prive/objets/liste/'.(trouver_fond(
+			$f = table_objet($objet).'-trad',
 			'prive/objets/liste'
 		) ? $f : 'objets-trad');
 		// pour afficher la liste des trad sur la base de l'id_trad en base
@@ -129,12 +129,12 @@ 
 			sql_getfetsel(
 				'id_trad',
 				$table_objet_sql,
-				"$_id_table_objet=" . (int) $id_objet
+				"$_id_table_objet=".(int) $id_objet
 			)
 		) {
 			// ne devrait jamais arriver sauf concurence de saisie
 			$erreurs['id_trad'] = _L('Une traduction est deja referencee');
-		} elseif (!sql_getfetsel($_id_table_objet, $table_objet_sql, "$_id_table_objet=" . (int) $id_trad)) {
+		} elseif (!sql_getfetsel($_id_table_objet, $table_objet_sql, "$_id_table_objet=".(int) $id_trad)) {
 			$erreurs['id_trad'] = _L('Indiquez un contenu existant');
 		}
 	}
@@ -172,7 +172,7 @@ 
 		) {
 			$table_objet_sql = table_objet_sql($objet);
 			$_id_table_objet = id_table_objet($objet);
-			if ($id_trad = sql_getfetsel('id_trad', $table_objet_sql, "$_id_table_objet=" . (int) $id_objet)) {
+			if ($id_trad = sql_getfetsel('id_trad', $table_objet_sql, "$_id_table_objet=".(int) $id_objet)) {
 				$referencer_traduction = charger_fonction('referencer_traduction', 'action');
 				$referencer_traduction($objet, $id_trad, $new_id_trad);
 			}

Indentation +112 added lines, -112 removed lines

@@ -16,7 +16,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/actions');
@@ -38,65 +38,65 @@ 
  *     Contexte à transmettre au squelette du formulaire sinon
  */
 function formulaires_traduire_charger_dist($objet, $id_objet, $retour = '', $traduire = true) {
-	if (!(int) $id_objet) {
-		return false;
-	}
-	$valeurs = formulaires_editer_objet_charger($objet, $id_objet, null, 0, $retour, '');
-	// verifier que l'objet indique possede bien des champs id_trad et lang
-	// attention, charger renomme lang => langue pour ne pas perturber la langue d'affichage du squelette
-	if (!isset($valeurs['langue'])) {
-		return false;
-	}
-
-	$valeurs['editable'] = autoriser('changerlangue', $objet, $id_objet);
-	$valeurs['_langue'] = '';
-	$langue_parent = '';
-	$id_parent = '';
-	if (isset($valeurs['id_rubrique'])) {
-		$id_parent = $valeurs['id_rubrique'];
-	}
-	if (isset($valeurs['id_parent'])) {
-		$id_parent = $valeurs['id_parent'];
-	}
-	if ($id_parent) {
-		$langue_parent = sql_getfetsel('lang', 'spip_rubriques', 'id_rubrique=' . (int) $id_parent);
-	}
-
-	if (!$langue_parent) {
-		$langue_parent = $GLOBALS['meta']['langue_site'];
-	}
-	if (
-		$valeurs['editable']
-		&& in_array(table_objet_sql($objet), explode(',', (string) $GLOBALS['meta']['multi_objets']))
-	) {
-		$valeurs['_langue'] = $valeurs['langue'];
-	}
-	$valeurs['langue_parent'] = $langue_parent;
-
-	$valeurs['_objet'] = $objet;
-	$valeurs['_id_objet'] = $id_objet;
-	$valeurs['changer_lang'] = '';
-
-
-	$valeurs['_traduisible'] = autoriser('changertraduction', $objet, $id_objet);
-	$valeurs['_traduire'] = '';
-	if (isset($valeurs['id_trad'])) {
-		$valeurs['_traduire'] = ($traduire ? ' ' : '');
-		$valeurs['_vue_traductions'] = 'prive/objets/liste/' . (trouver_fond(
-			$f = table_objet($objet) . '-trad',
-			'prive/objets/liste'
-		) ? $f : 'objets-trad');
-		// pour afficher la liste des trad sur la base de l'id_trad en base
-		// independamment d'une saisie en cours sur id_trad
-		$valeurs['_lister_id_trad'] = $valeurs['id_trad'];
-		$valeurs['_id_parent'] = $id_parent;
-	}
-
-	$valeurs['_saisie_en_cours'] = (!_request('annuler') && (_request('changer_lang') !== null
-		|| _request('changer_id_trad') !== null));
-	$valeurs['_pipeline'] = ['traduire', ['type' => $objet, 'id' => $id_objet]];
-
-	return $valeurs;
+    if (!(int) $id_objet) {
+        return false;
+    }
+    $valeurs = formulaires_editer_objet_charger($objet, $id_objet, null, 0, $retour, '');
+    // verifier que l'objet indique possede bien des champs id_trad et lang
+    // attention, charger renomme lang => langue pour ne pas perturber la langue d'affichage du squelette
+    if (!isset($valeurs['langue'])) {
+        return false;
+    }
+
+    $valeurs['editable'] = autoriser('changerlangue', $objet, $id_objet);
+    $valeurs['_langue'] = '';
+    $langue_parent = '';
+    $id_parent = '';
+    if (isset($valeurs['id_rubrique'])) {
+        $id_parent = $valeurs['id_rubrique'];
+    }
+    if (isset($valeurs['id_parent'])) {
+        $id_parent = $valeurs['id_parent'];
+    }
+    if ($id_parent) {
+        $langue_parent = sql_getfetsel('lang', 'spip_rubriques', 'id_rubrique=' . (int) $id_parent);
+    }
+
+    if (!$langue_parent) {
+        $langue_parent = $GLOBALS['meta']['langue_site'];
+    }
+    if (
+        $valeurs['editable']
+        && in_array(table_objet_sql($objet), explode(',', (string) $GLOBALS['meta']['multi_objets']))
+    ) {
+        $valeurs['_langue'] = $valeurs['langue'];
+    }
+    $valeurs['langue_parent'] = $langue_parent;
+
+    $valeurs['_objet'] = $objet;
+    $valeurs['_id_objet'] = $id_objet;
+    $valeurs['changer_lang'] = '';
+
+
+    $valeurs['_traduisible'] = autoriser('changertraduction', $objet, $id_objet);
+    $valeurs['_traduire'] = '';
+    if (isset($valeurs['id_trad'])) {
+        $valeurs['_traduire'] = ($traduire ? ' ' : '');
+        $valeurs['_vue_traductions'] = 'prive/objets/liste/' . (trouver_fond(
+            $f = table_objet($objet) . '-trad',
+            'prive/objets/liste'
+        ) ? $f : 'objets-trad');
+        // pour afficher la liste des trad sur la base de l'id_trad en base
+        // independamment d'une saisie en cours sur id_trad
+        $valeurs['_lister_id_trad'] = $valeurs['id_trad'];
+        $valeurs['_id_parent'] = $id_parent;
+    }
+
+    $valeurs['_saisie_en_cours'] = (!_request('annuler') && (_request('changer_lang') !== null
+        || _request('changer_id_trad') !== null));
+    $valeurs['_pipeline'] = ['traduire', ['type' => $objet, 'id' => $id_objet]];
+
+    return $valeurs;
 }
 
 /**
@@ -114,32 +114,32 @@ 
  *     Erreurs des saisies
  */
 function formulaires_traduire_verifier_dist($objet, $id_objet, $retour = '', $traduire = true) {
-	$erreurs = [];
-
-	if (null !== _request('changer_lang')) {
-		$erreurs = formulaires_editer_objet_verifier($objet, $id_objet, ['changer_lang']);
-	}
-
-	// si id_trad fourni, verifier que cela ne conflicte pas avec un id_trad existant
-	// et que ca reference bien un objet existant
-	if ($id_trad = _request('id_trad')) {
-		$table_objet_sql = table_objet_sql($objet);
-		$_id_table_objet = id_table_objet($objet);
-		if (
-			sql_getfetsel(
-				'id_trad',
-				$table_objet_sql,
-				"$_id_table_objet=" . (int) $id_objet
-			)
-		) {
-			// ne devrait jamais arriver sauf concurence de saisie
-			$erreurs['id_trad'] = _L('Une traduction est deja referencee');
-		} elseif (!sql_getfetsel($_id_table_objet, $table_objet_sql, "$_id_table_objet=" . (int) $id_trad)) {
-			$erreurs['id_trad'] = _L('Indiquez un contenu existant');
-		}
-	}
-
-	return $erreurs;
+    $erreurs = [];
+
+    if (null !== _request('changer_lang')) {
+        $erreurs = formulaires_editer_objet_verifier($objet, $id_objet, ['changer_lang']);
+    }
+
+    // si id_trad fourni, verifier que cela ne conflicte pas avec un id_trad existant
+    // et que ca reference bien un objet existant
+    if ($id_trad = _request('id_trad')) {
+        $table_objet_sql = table_objet_sql($objet);
+        $_id_table_objet = id_table_objet($objet);
+        if (
+            sql_getfetsel(
+                'id_trad',
+                $table_objet_sql,
+                "$_id_table_objet=" . (int) $id_objet
+            )
+        ) {
+            // ne devrait jamais arriver sauf concurence de saisie
+            $erreurs['id_trad'] = _L('Une traduction est deja referencee');
+        } elseif (!sql_getfetsel($_id_table_objet, $table_objet_sql, "$_id_table_objet=" . (int) $id_trad)) {
+            $erreurs['id_trad'] = _L('Indiquez un contenu existant');
+        }
+    }
+
+    return $erreurs;
 }
 
 
@@ -158,30 +158,30 @@ 
  *     Retour des traitements
  */
 function formulaires_traduire_traiter_dist($objet, $id_objet, $retour = '', $traduire = true) {
-	$res = [];
-	if (!_request('annuler') && autoriser('changerlangue', $objet, $id_objet)) {
-		// action/editer_xxx doit traiter la modif de changer_lang
-		$res = formulaires_editer_objet_traiter($objet, $id_objet, 0, 0, $retour);
-	}
-	if (!_request('annuler') && autoriser('changertraduction', $objet, $id_objet)) {
-		if (($id_trad = _request('id_trad')) || _request('supprimer_trad')) {
-			$referencer_traduction = charger_fonction('referencer_traduction', 'action');
-			$referencer_traduction($objet, $id_objet, (int) $id_trad); // 0 si supprimer_trad
-		} elseif (
-			($new_id_trad = _request('changer_reference_trad')) && ($new_id_trad = array_keys($new_id_trad)) && ($new_id_trad = reset($new_id_trad))
-		) {
-			$table_objet_sql = table_objet_sql($objet);
-			$_id_table_objet = id_table_objet($objet);
-			if ($id_trad = sql_getfetsel('id_trad', $table_objet_sql, "$_id_table_objet=" . (int) $id_objet)) {
-				$referencer_traduction = charger_fonction('referencer_traduction', 'action');
-				$referencer_traduction($objet, $id_trad, $new_id_trad);
-			}
-		}
-	}
-	$res['editable'] = true;
-	if (!isset($res['message_erreur'])) {
-		set_request('annuler', 'annuler');
-	} // provoquer la fermeture du forumlaire
-
-	return $res;
+    $res = [];
+    if (!_request('annuler') && autoriser('changerlangue', $objet, $id_objet)) {
+        // action/editer_xxx doit traiter la modif de changer_lang
+        $res = formulaires_editer_objet_traiter($objet, $id_objet, 0, 0, $retour);
+    }
+    if (!_request('annuler') && autoriser('changertraduction', $objet, $id_objet)) {
+        if (($id_trad = _request('id_trad')) || _request('supprimer_trad')) {
+            $referencer_traduction = charger_fonction('referencer_traduction', 'action');
+            $referencer_traduction($objet, $id_objet, (int) $id_trad); // 0 si supprimer_trad
+        } elseif (
+            ($new_id_trad = _request('changer_reference_trad')) && ($new_id_trad = array_keys($new_id_trad)) && ($new_id_trad = reset($new_id_trad))
+        ) {
+            $table_objet_sql = table_objet_sql($objet);
+            $_id_table_objet = id_table_objet($objet);
+            if ($id_trad = sql_getfetsel('id_trad', $table_objet_sql, "$_id_table_objet=" . (int) $id_objet)) {
+                $referencer_traduction = charger_fonction('referencer_traduction', 'action');
+                $referencer_traduction($objet, $id_trad, $new_id_trad);
+            }
+        }
+    }
+    $res['editable'] = true;
+    if (!isset($res['message_erreur'])) {
+        set_request('annuler', 'annuler');
+    } // provoquer la fermeture du forumlaire
+
+    return $res;
 }

prive/formulaires/inc-instituer_auteur_fonctions.php

Indentation +26 added lines, -26 removed lines

@@ -7,7 +7,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -24,32 +24,32 @@ 
  *     Code HTML et javascript
  */
 function choisir_rubriques_admin_restreint(
-	$id_auteur,
-	$label = '',
-	$sel_css = '#liste_rubriques_restreintes',
-	$img_remove = ''
+    $id_auteur,
+    $label = '',
+    $sel_css = '#liste_rubriques_restreintes',
+    $img_remove = ''
 ) {
-	global $spip_lang;
-	$res = '';
-	// Ajouter une rubrique a un administrateur restreint
-	if (
-		($chercher_rubrique = charger_fonction('chercher_rubrique', 'inc'))
-		&& ($a = $chercher_rubrique(0, 'auteur', false))
-	) {
-		if ($img_remove) {
-			$img_remove = addslashes("<a href=\"#\" onclick=\"jQuery(this).parent().remove();return false;\" class=\"removelink\">$img_remove</a>");
-		}
+    global $spip_lang;
+    $res = '';
+    // Ajouter une rubrique a un administrateur restreint
+    if (
+        ($chercher_rubrique = charger_fonction('chercher_rubrique', 'inc'))
+        && ($a = $chercher_rubrique(0, 'auteur', false))
+    ) {
+        if ($img_remove) {
+            $img_remove = addslashes("<a href=\"#\" onclick=\"jQuery(this).parent().remove();return false;\" class=\"removelink\">$img_remove</a>");
+        }
 
-		$res =
-			"\n<div id='ajax_rubrique'>\n"
-			. "<label>$label</label>\n"
-			. "<input name='id_auteur' value='$id_auteur' type='hidden' />\n"
-			. $a
-			. "</div>\n"
+        $res =
+            "\n<div id='ajax_rubrique'>\n"
+            . "<label>$label</label>\n"
+            . "<input name='id_auteur' value='$id_auteur' type='hidden' />\n"
+            . $a
+            . "</div>\n"
 
-			// onchange = pour le menu
-			// l'evenement doit etre provoque a la main par le selecteur ajax
-			. "<script type='text/javascript'>/*<![CDATA[*/
+            // onchange = pour le menu
+            // l'evenement doit etre provoque a la main par le selecteur ajax
+            . "<script type='text/javascript'>/*<![CDATA[*/
 jQuery(function($){
 	$('#id_parent').on('change', function(){
 		var id_parent = parseInt(this.value);
@@ -68,7 +68,7 @@ 
 	.attr('name','noname');
 });
 /*]]>*/</script>";
-	}
+    }
 
-	return $res;
+    return $res;
 }

prive/formulaires/configurer_redacteurs.php

Indentation +35 added lines, -35 removed lines

@@ -10,46 +10,46 @@
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 function formulaires_configurer_redacteurs_charger_dist() {
-	$valeurs = [];
-	foreach (
-		[
-			'accepter_inscriptions',
-		] as $m
-	) {
-		$valeurs[$m] = $GLOBALS['meta'][$m];
-	}
-
-	return $valeurs;
+    $valeurs = [];
+    foreach (
+        [
+            'accepter_inscriptions',
+        ] as $m
+    ) {
+        $valeurs[$m] = $GLOBALS['meta'][$m];
+    }
+
+    return $valeurs;
 }
 
 
 function formulaires_configurer_redacteurs_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_inscriptions'))
-		&& $i != $GLOBALS['meta']['accepter_inscriptions'])
-	) {
-		include_spip('inc/invalideur');
-		suivre_invalideur('1'); # tout effacer
-	}
-
-	foreach (
-		[
-			'accepter_inscriptions',
-		] 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_inscriptions'))
+        && $i != $GLOBALS['meta']['accepter_inscriptions'])
+    ) {
+        include_spip('inc/invalideur');
+        suivre_invalideur('1'); # tout effacer
+    }
+
+    foreach (
+        [
+            'accepter_inscriptions',
+        ] 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_previsualiseur.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
 	$res = ['editable' => true];
 
 	if (($i = _request('preview')) && is_array($i)) {
-		$i = ',' . implode(',', $i) . ',';
+		$i = ','.implode(',', $i).',';
 	}
 
 	ecrire_meta('preview', $i);

Indentation +9 added lines, -9 removed lines

@@ -10,25 +10,25 @@
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 include_spip('inc/presentation');
 
 function formulaires_configurer_previsualiseur_charger_dist() {
-	return ['preview' => explode(',', (string) $GLOBALS['meta']['preview'])];
+    return ['preview' => explode(',', (string) $GLOBALS['meta']['preview'])];
 }
 
 
 function formulaires_configurer_previsualiseur_traiter_dist() {
-	$res = ['editable' => true];
+    $res = ['editable' => true];
 
-	if (($i = _request('preview')) && is_array($i)) {
-		$i = ',' . implode(',', $i) . ',';
-	}
+    if (($i = _request('preview')) && is_array($i)) {
+        $i = ',' . implode(',', $i) . ',';
+    }
 
-	ecrire_meta('preview', $i);
+    ecrire_meta('preview', $i);
 
-	$res['message_ok'] = _T('config_info_enregistree');
+    $res['message_ok'] = _T('config_info_enregistree');
 
-	return $res;
+    return $res;
 }

prive/formulaires/configurer_langue.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
 	foreach ($langues as $l) {
 		$res .= "<option value='$l'"
 			. ($l == $l_site ? " selected='selected'" : '')
-			. '>' . traduire_nom_langue($l) . "</option>\n";
+			. '>'.traduire_nom_langue($l)."</option>\n";
 	}
 
 	return [

Indentation +40 added lines, -40 removed lines

@@ -10,57 +10,57 @@
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 function formulaires_configurer_langue_charger_dist() {
-	$valeurs = [];
+    $valeurs = [];
 
-	include_spip('inc/lang');
-	$l_site = $GLOBALS['meta']['langue_site'];
-	$langue_site = traduire_nom_langue($l_site);
+    include_spip('inc/lang');
+    $l_site = $GLOBALS['meta']['langue_site'];
+    $langue_site = traduire_nom_langue($l_site);
 
-	$langues = explode(',', (string) $GLOBALS['meta']['langues_proposees']);
-	if (!in_array($l_site, $langues)) {
-		$langues[] = $l_site;
-	}
-	sort($langues);
+    $langues = explode(',', (string) $GLOBALS['meta']['langues_proposees']);
+    if (!in_array($l_site, $langues)) {
+        $langues[] = $l_site;
+    }
+    sort($langues);
 
-	$res = '';
-	foreach ($langues as $l) {
-		$res .= "<option value='$l'"
-			. ($l == $l_site ? " selected='selected'" : '')
-			. '>' . traduire_nom_langue($l) . "</option>\n";
-	}
+    $res = '';
+    foreach ($langues as $l) {
+        $res .= "<option value='$l'"
+            . ($l == $l_site ? " selected='selected'" : '')
+            . '>' . traduire_nom_langue($l) . "</option>\n";
+    }
 
-	return [
-		'_langues' => $res,
-		'_langue_site' => $langue_site,
-		'changer_langue_site' => '',
-	];
+    return [
+        '_langues' => $res,
+        '_langue_site' => $langue_site,
+        'changer_langue_site' => '',
+    ];
 }
 
 
 function formulaires_configurer_langue_traiter_dist() {
-	$res = ['editable' => true];
+    $res = ['editable' => true];
 
-	if ($lang = _request('changer_langue_site')) {
-		include_spip('inc/lang');
-		// verif que la langue demandee est licite
-		if (changer_langue($lang)) {
-			ecrire_meta('langue_site', $lang);
-			// le test a defait ca:
-			utiliser_langue_visiteur();
-			$res['message_ok'] = _T('config_info_enregistree');
-			include_spip('inc/rubriques');
-			calculer_langues_rubriques();
-		}
-		// le test a defait ca:
-		utiliser_langue_visiteur();
-	}
-	if (!$res['message_ok']) {
-		$res['message_erreur'] = _L('erreur');
-	}
+    if ($lang = _request('changer_langue_site')) {
+        include_spip('inc/lang');
+        // verif que la langue demandee est licite
+        if (changer_langue($lang)) {
+            ecrire_meta('langue_site', $lang);
+            // le test a defait ca:
+            utiliser_langue_visiteur();
+            $res['message_ok'] = _T('config_info_enregistree');
+            include_spip('inc/rubriques');
+            calculer_langues_rubriques();
+        }
+        // le test a defait ca:
+        utiliser_langue_visiteur();
+    }
+    if (!$res['message_ok']) {
+        $res['message_erreur'] = _L('erreur');
+    }
 
-	return $res;
+    return $res;
 }

prive/formulaires/editer_liens.php

Braces +1 added lines, -2 removed lines

@@ -545,8 +545,7 @@
 						|| empty($oups['oups_value'])
 					) {
 						$oups = null;
-					}
-					else {
+					} else {
 						$oups = $oups['oups_value'];
 						// controler le contenu
 						foreach ($oups as $k => $oup) {

Indentation +433 added lines, -433 removed lines

@@ -15,7 +15,7 @@ 
  * @package SPIP\Core\Formulaires
  **/
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 
@@ -35,25 +35,25 @@ 
  *   ($table_source,$objet,$id_objet,$objet_lien)
  */
 function determine_source_lien_objet($a, $b, $c) {
-	$table_source = $objet_lien = $objet = $id_objet = null;
-	// auteurs, article, 23 :
-	// associer des auteurs à l'article 23, sur la table pivot spip_auteurs_liens
-	if (is_numeric($c) && !is_numeric($b)) {
-		$table_source = table_objet($a);
-		$objet_lien = objet_type($a);
-		$objet = objet_type($b);
-		$id_objet = $c;
-	}
-	// article, 23, auteurs
-	// associer des auteurs à l'article 23, sur la table pivot spip_articles_liens
-	if (is_numeric($b) && !is_numeric($c)) {
-		$table_source = table_objet($c);
-		$objet_lien = objet_type($a);
-		$objet = objet_type($a);
-		$id_objet = $b;
-	}
-
-	return [$table_source, $objet, $id_objet, $objet_lien];
+    $table_source = $objet_lien = $objet = $id_objet = null;
+    // auteurs, article, 23 :
+    // associer des auteurs à l'article 23, sur la table pivot spip_auteurs_liens
+    if (is_numeric($c) && !is_numeric($b)) {
+        $table_source = table_objet($a);
+        $objet_lien = objet_type($a);
+        $objet = objet_type($b);
+        $id_objet = $c;
+    }
+    // article, 23, auteurs
+    // associer des auteurs à l'article 23, sur la table pivot spip_articles_liens
+    if (is_numeric($b) && !is_numeric($c)) {
+        $table_source = table_objet($c);
+        $objet_lien = objet_type($a);
+        $objet = objet_type($a);
+        $id_objet = $b;
+    }
+
+    return [$table_source, $objet, $id_objet, $objet_lien];
 }
 
 /**
@@ -79,87 +79,87 @@ 
  */
 function formulaires_editer_liens_charger_dist($a, $b, $c, $options = []) {
 
-	// compat avec ancienne signature ou le 4eme argument est $editable
-	if (!is_array($options)) {
-		$options = ['editable' => $options];
-	} elseif (!isset($options['editable'])) {
-		$options['editable'] = true;
-	}
-
-	$editable = $options['editable'];
-
-	[$table_source, $objet, $id_objet, $objet_lien] = determine_source_lien_objet($a, $b, $c);
-	if (!$table_source || !$objet || !$objet_lien || !$id_objet) {
-		return false;
-	}
-
-	$objet_source = objet_type($table_source);
-	$table_sql_source = table_objet_sql($objet_source);
-
-	// verifier existence de la table xxx_liens
-	include_spip('action/editer_liens');
-	if (!objet_associable($objet_lien)) {
-		return false;
-	}
-
-	// L'éditabilité :) est définie par un test permanent (par exemple "associermots") ET le 4ème argument
-	include_spip('inc/autoriser');
-	$editable = ($editable
-		&& autoriser('associer' . $table_source, $objet, $id_objet)
-		&& autoriser('modifier', $objet, $id_objet));
-
-	if (
-		!$editable && !count(objet_trouver_liens(
-			[$objet_lien => '*'],
-			[($objet_lien == $objet_source ? $objet : $objet_source) => $id_objet]
-		))
-	) {
-		return false;
-	}
-
-	// squelettes de vue et de d'association
-	// ils sont différents si des rôles sont définis.
-	$skel_vue = $table_source . '_lies';
-	$skel_ajout = $table_source . '_associer';
-
-	// description des roles
-	include_spip('inc/roles');
-	if ($roles = roles_presents($objet_source, $objet)) {
-		// on demande de nouveaux squelettes en conséquence
-		$skel_vue = $table_source . '_roles_lies';
-		$skel_ajout = $table_source . '_roles_associer';
-	}
-
-	$oups = '';
-	if ($editable) {
-		$oups = lien_gerer__oups('editer_liens', 'hash');
-	}
-	$valeurs = [
-		'id' => "$table_source-$objet-$id_objet-$objet_lien", // identifiant unique pour les id du form
-		'_vue_liee' => $skel_vue,
-		'_vue_ajout' => $skel_ajout,
-		'_objet_lien' => $objet_lien,
-		'id_lien_ajoute' => _request('id_lien_ajoute'),
-		'objet' => $objet,
-		'id_objet' => $id_objet,
-		'objet_source' => $objet_source,
-		'table_source' => $table_source,
-		'recherche' => '',
-		'visible' => 0,
-		'ajouter_lien' => '',
-		'supprimer_lien' => '',
-		'qualifier_lien' => '',
-		'ordonner_lien' => '',
-		'desordonner_liens' => '',
-		'_roles' => $roles, # description des roles
-		'_oups' => entites_html($oups),
-		'editable' => $editable,
-	];
-
-	// les options non definies dans $valeurs sont passees telles quelles au formulaire html
-	$valeurs = array_merge($options, $valeurs);
-
-	return $valeurs;
+    // compat avec ancienne signature ou le 4eme argument est $editable
+    if (!is_array($options)) {
+        $options = ['editable' => $options];
+    } elseif (!isset($options['editable'])) {
+        $options['editable'] = true;
+    }
+
+    $editable = $options['editable'];
+
+    [$table_source, $objet, $id_objet, $objet_lien] = determine_source_lien_objet($a, $b, $c);
+    if (!$table_source || !$objet || !$objet_lien || !$id_objet) {
+        return false;
+    }
+
+    $objet_source = objet_type($table_source);
+    $table_sql_source = table_objet_sql($objet_source);
+
+    // verifier existence de la table xxx_liens
+    include_spip('action/editer_liens');
+    if (!objet_associable($objet_lien)) {
+        return false;
+    }
+
+    // L'éditabilité :) est définie par un test permanent (par exemple "associermots") ET le 4ème argument
+    include_spip('inc/autoriser');
+    $editable = ($editable
+        && autoriser('associer' . $table_source, $objet, $id_objet)
+        && autoriser('modifier', $objet, $id_objet));
+
+    if (
+        !$editable && !count(objet_trouver_liens(
+            [$objet_lien => '*'],
+            [($objet_lien == $objet_source ? $objet : $objet_source) => $id_objet]
+        ))
+    ) {
+        return false;
+    }
+
+    // squelettes de vue et de d'association
+    // ils sont différents si des rôles sont définis.
+    $skel_vue = $table_source . '_lies';
+    $skel_ajout = $table_source . '_associer';
+
+    // description des roles
+    include_spip('inc/roles');
+    if ($roles = roles_presents($objet_source, $objet)) {
+        // on demande de nouveaux squelettes en conséquence
+        $skel_vue = $table_source . '_roles_lies';
+        $skel_ajout = $table_source . '_roles_associer';
+    }
+
+    $oups = '';
+    if ($editable) {
+        $oups = lien_gerer__oups('editer_liens', 'hash');
+    }
+    $valeurs = [
+        'id' => "$table_source-$objet-$id_objet-$objet_lien", // identifiant unique pour les id du form
+        '_vue_liee' => $skel_vue,
+        '_vue_ajout' => $skel_ajout,
+        '_objet_lien' => $objet_lien,
+        'id_lien_ajoute' => _request('id_lien_ajoute'),
+        'objet' => $objet,
+        'id_objet' => $id_objet,
+        'objet_source' => $objet_source,
+        'table_source' => $table_source,
+        'recherche' => '',
+        'visible' => 0,
+        'ajouter_lien' => '',
+        'supprimer_lien' => '',
+        'qualifier_lien' => '',
+        'ordonner_lien' => '',
+        'desordonner_liens' => '',
+        '_roles' => $roles, # description des roles
+        '_oups' => entites_html($oups),
+        'editable' => $editable,
+    ];
+
+    // les options non definies dans $valeurs sont passees telles quelles au formulaire html
+    $valeurs = array_merge($options, $valeurs);
+
+    return $valeurs;
 }
 
 /**
@@ -203,170 +203,170 @@ 
  * @return array
  */
 function formulaires_editer_liens_traiter_dist($a, $b, $c, $options = []) {
-	// compat avec ancienne signature ou le 4eme argument est $editable
-	if (!is_array($options)) {
-		$options = ['editable' => $options];
-	} elseif (!isset($options['editable'])) {
-		$options['editable'] = true;
-	}
-
-	$editable = $options['editable'];
-
-	$res = ['editable' => (bool) $editable];
-	[$table_source, $objet, $id_objet, $objet_lien] = determine_source_lien_objet($a, $b, $c);
-	if (!$table_source || !$objet || !$objet_lien) {
-		return $res;
-	}
-
-
-	if (_request('tout_voir')) {
-		set_request('recherche', '');
-	}
-
-	include_spip('inc/autoriser');
-	if (autoriser('modifier', $objet, $id_objet)) {
-		// recuperer le oups du coup d'avant pour le propager à charger() si on ne fait rien par exemple
-		lien_gerer__oups('editer_liens', 'request');
-
-		// annuler les suppressions du coup d'avant ?
-		if (
-			_request('annuler_oups')
-			&& ($oups = lien_gerer__oups('editer_liens', 'get'))
-		) {
-			if ($oups_objets = charger_fonction("editer_liens_oups_{$table_source}_{$objet}_{$objet_lien}", 'action', true)) {
-				$oups_objets($oups);
-			} else {
-				$objet_source = objet_type($table_source);
-				include_spip('action/editer_liens');
-				foreach ($oups as $oup) {
-					if ($objet_lien == $objet_source) {
-						objet_associer([$objet_source => $oup[$objet_source]], [$objet => $oup[$objet]], $oup);
-					} else {
-						objet_associer([$objet => $oup[$objet]], [$objet_source => $oup[$objet_source]], $oup);
-					}
-				}
-			}
-			# oups ne persiste que pour la derniere action, si suppression
-			lien_gerer__oups('editer_liens', 'reset');
-		}
-
-		$supprimer = _request('supprimer_lien');
-		$ajouter = _request('ajouter_lien');
-		$ordonner = _request('ordonner_lien');
-
-		if (_request('desordonner_liens')) {
-			include_spip('action/editer_liens');
-			objet_qualifier_liens([$objet_lien => '*'], [$objet => $id_objet], ['rang_lien' => 0]);
-		}
-
-		// il est possible de preciser dans une seule variable un remplacement :
-		// remplacer_lien[old][new]
-		if ($remplacer = _request('remplacer_lien')) {
-			foreach ($remplacer as $k => $v) {
-				if ($old = lien_verifier_action($k, '')) {
-					foreach (is_array($v) ? $v : [$v] as $kn => $vn) {
-						if ($new = lien_verifier_action($kn, $vn)) {
-							$supprimer[$old] = 'x';
-							$ajouter[$new] = '+';
-						}
-					}
-				}
-			}
-		}
-
-		if ($supprimer) {
-			if (
-				$supprimer_objets = charger_fonction(
-					"editer_liens_supprimer_{$table_source}_{$objet}_{$objet_lien}",
-					'action',
-					true
-				)
-			) {
-				$oups = $supprimer_objets($supprimer);
-			} else {
-				include_spip('action/editer_liens');
-				$oups = [];
-
-				foreach ($supprimer as $k => $v) {
-					if ($lien = lien_verifier_action($k, $v)) {
-						$lien = explode('-', $lien);
-						[$objet_source, $ids, $objet_lie, $idl, $role] = array_pad($lien, 5, null);
-						// appliquer une condition sur le rôle si défini ('*' pour tous les roles)
-						$cond = (is_null($role) ? [] : ['role' => $role]);
-						if ($objet_lien == $objet_source) {
-							$oups = array_merge(
-								$oups,
-								objet_trouver_liens([$objet_source => $ids], [$objet_lie => $idl], $cond)
-							);
-							objet_dissocier([$objet_source => $ids], [$objet_lie => $idl], $cond);
-						} else {
-							$oups = array_merge(
-								$oups,
-								objet_trouver_liens([$objet_lie => $idl], [$objet_source => $ids], $cond)
-							);
-							objet_dissocier([$objet_lie => $idl], [$objet_source => $ids], $cond);
-						}
-					}
-				}
-			}
-			if (!empty($oups)) {
-				lien_gerer__oups('editer_liens', 'set', $oups);
-			} else {
-				lien_gerer__oups('editer_liens', 'reset');
-			}
-		}
-
-		if ($ajouter) {
-			if (
-				$ajouter_objets = charger_fonction("editer_liens_ajouter_{$table_source}_{$objet}_{$objet_lien}", 'action', true)
-			) {
-				$ajout_ok = $ajouter_objets($ajouter);
-			} else {
-				$ajout_ok = false;
-				include_spip('action/editer_liens');
-				foreach ($ajouter as $k => $v) {
-					if ($lien = lien_verifier_action($k, $v)) {
-						$ajout_ok = true;
-						[$objet1, $ids, $objet2, $idl] = explode('-', $lien);
-						$qualifs = lien_retrouver_qualif($objet_lien, $lien);
-						if ($objet_lien == $objet1) {
-							lien_ajouter_liaisons($objet1, $ids, $objet2, $idl, $qualifs);
-						} else {
-							lien_ajouter_liaisons($objet2, $idl, $objet1, $ids, $qualifs);
-						}
-						set_request('id_lien_ajoute', $ids);
-					}
-				}
-			}
-			# oups ne persiste que pour la derniere action, si suppression
-			# une suppression suivie d'un ajout dans le meme hit est un remplacement
-			# non annulable !
-			if ($ajout_ok) {
-				lien_gerer__oups('editer_liens', 'reset');
-			}
-		}
-
-		if ($ordonner) {
-			include_spip('action/editer_liens');
-			foreach ($ordonner as $k => $rang_lien) {
-				if ($lien = lien_verifier_action($k, '')) {
-					[$objet1, $ids, $objet2, $idl] = explode('-', $lien);
-					$qualif = ['rang_lien' => $rang_lien];
-
-					if ($objet_lien == $objet1) {
-						objet_qualifier_liens([$objet1 => $ids], [$objet2 => $idl], $qualif);
-					} else {
-						objet_qualifier_liens([$objet2 => $idl], [$objet1 => $ids], $qualif);
-					}
-					set_request('id_lien_ajoute', $ids);
-					lien_gerer__oups('editer_liens', 'reset');
-				}
-			}
-		}
-	}
-
-
-	return $res;
+    // compat avec ancienne signature ou le 4eme argument est $editable
+    if (!is_array($options)) {
+        $options = ['editable' => $options];
+    } elseif (!isset($options['editable'])) {
+        $options['editable'] = true;
+    }
+
+    $editable = $options['editable'];
+
+    $res = ['editable' => (bool) $editable];
+    [$table_source, $objet, $id_objet, $objet_lien] = determine_source_lien_objet($a, $b, $c);
+    if (!$table_source || !$objet || !$objet_lien) {
+        return $res;
+    }
+
+
+    if (_request('tout_voir')) {
+        set_request('recherche', '');
+    }
+
+    include_spip('inc/autoriser');
+    if (autoriser('modifier', $objet, $id_objet)) {
+        // recuperer le oups du coup d'avant pour le propager à charger() si on ne fait rien par exemple
+        lien_gerer__oups('editer_liens', 'request');
+
+        // annuler les suppressions du coup d'avant ?
+        if (
+            _request('annuler_oups')
+            && ($oups = lien_gerer__oups('editer_liens', 'get'))
+        ) {
+            if ($oups_objets = charger_fonction("editer_liens_oups_{$table_source}_{$objet}_{$objet_lien}", 'action', true)) {
+                $oups_objets($oups);
+            } else {
+                $objet_source = objet_type($table_source);
+                include_spip('action/editer_liens');
+                foreach ($oups as $oup) {
+                    if ($objet_lien == $objet_source) {
+                        objet_associer([$objet_source => $oup[$objet_source]], [$objet => $oup[$objet]], $oup);
+                    } else {
+                        objet_associer([$objet => $oup[$objet]], [$objet_source => $oup[$objet_source]], $oup);
+                    }
+                }
+            }
+            # oups ne persiste que pour la derniere action, si suppression
+            lien_gerer__oups('editer_liens', 'reset');
+        }
+
+        $supprimer = _request('supprimer_lien');
+        $ajouter = _request('ajouter_lien');
+        $ordonner = _request('ordonner_lien');
+
+        if (_request('desordonner_liens')) {
+            include_spip('action/editer_liens');
+            objet_qualifier_liens([$objet_lien => '*'], [$objet => $id_objet], ['rang_lien' => 0]);
+        }
+
+        // il est possible de preciser dans une seule variable un remplacement :
+        // remplacer_lien[old][new]
+        if ($remplacer = _request('remplacer_lien')) {
+            foreach ($remplacer as $k => $v) {
+                if ($old = lien_verifier_action($k, '')) {
+                    foreach (is_array($v) ? $v : [$v] as $kn => $vn) {
+                        if ($new = lien_verifier_action($kn, $vn)) {
+                            $supprimer[$old] = 'x';
+                            $ajouter[$new] = '+';
+                        }
+                    }
+                }
+            }
+        }
+
+        if ($supprimer) {
+            if (
+                $supprimer_objets = charger_fonction(
+                    "editer_liens_supprimer_{$table_source}_{$objet}_{$objet_lien}",
+                    'action',
+                    true
+                )
+            ) {
+                $oups = $supprimer_objets($supprimer);
+            } else {
+                include_spip('action/editer_liens');
+                $oups = [];
+
+                foreach ($supprimer as $k => $v) {
+                    if ($lien = lien_verifier_action($k, $v)) {
+                        $lien = explode('-', $lien);
+                        [$objet_source, $ids, $objet_lie, $idl, $role] = array_pad($lien, 5, null);
+                        // appliquer une condition sur le rôle si défini ('*' pour tous les roles)
+                        $cond = (is_null($role) ? [] : ['role' => $role]);
+                        if ($objet_lien == $objet_source) {
+                            $oups = array_merge(
+                                $oups,
+                                objet_trouver_liens([$objet_source => $ids], [$objet_lie => $idl], $cond)
+                            );
+                            objet_dissocier([$objet_source => $ids], [$objet_lie => $idl], $cond);
+                        } else {
+                            $oups = array_merge(
+                                $oups,
+                                objet_trouver_liens([$objet_lie => $idl], [$objet_source => $ids], $cond)
+                            );
+                            objet_dissocier([$objet_lie => $idl], [$objet_source => $ids], $cond);
+                        }
+                    }
+                }
+            }
+            if (!empty($oups)) {
+                lien_gerer__oups('editer_liens', 'set', $oups);
+            } else {
+                lien_gerer__oups('editer_liens', 'reset');
+            }
+        }
+
+        if ($ajouter) {
+            if (
+                $ajouter_objets = charger_fonction("editer_liens_ajouter_{$table_source}_{$objet}_{$objet_lien}", 'action', true)
+            ) {
+                $ajout_ok = $ajouter_objets($ajouter);
+            } else {
+                $ajout_ok = false;
+                include_spip('action/editer_liens');
+                foreach ($ajouter as $k => $v) {
+                    if ($lien = lien_verifier_action($k, $v)) {
+                        $ajout_ok = true;
+                        [$objet1, $ids, $objet2, $idl] = explode('-', $lien);
+                        $qualifs = lien_retrouver_qualif($objet_lien, $lien);
+                        if ($objet_lien == $objet1) {
+                            lien_ajouter_liaisons($objet1, $ids, $objet2, $idl, $qualifs);
+                        } else {
+                            lien_ajouter_liaisons($objet2, $idl, $objet1, $ids, $qualifs);
+                        }
+                        set_request('id_lien_ajoute', $ids);
+                    }
+                }
+            }
+            # oups ne persiste que pour la derniere action, si suppression
+            # une suppression suivie d'un ajout dans le meme hit est un remplacement
+            # non annulable !
+            if ($ajout_ok) {
+                lien_gerer__oups('editer_liens', 'reset');
+            }
+        }
+
+        if ($ordonner) {
+            include_spip('action/editer_liens');
+            foreach ($ordonner as $k => $rang_lien) {
+                if ($lien = lien_verifier_action($k, '')) {
+                    [$objet1, $ids, $objet2, $idl] = explode('-', $lien);
+                    $qualif = ['rang_lien' => $rang_lien];
+
+                    if ($objet_lien == $objet1) {
+                        objet_qualifier_liens([$objet1 => $ids], [$objet2 => $idl], $qualif);
+                    } else {
+                        objet_qualifier_liens([$objet2 => $idl], [$objet1 => $ids], $qualif);
+                    }
+                    set_request('id_lien_ajoute', $ids);
+                    lien_gerer__oups('editer_liens', 'reset');
+                }
+            }
+        }
+    }
+
+
+    return $res;
 }
 
 
@@ -389,24 +389,24 @@ 
  * @return string Action demandée si trouvée, sinon ''
  */
 function lien_verifier_action($k, $v) {
-	$action = '';
-	if (preg_match(',^\w+-[\w*]+-[\w*]+-[\w*]+(-[\w*])?,', $k)) {
-		$action = $k;
-	}
-	if (preg_match(',^\w+-[\w*]+-[\w*]+-[\w*]+(-[\w*])?,', $v)) {
-		if (is_numeric($k)) {
-			$action = $v;
-		}
-		if (_request($k)) {
-			$action = $v;
-		}
-	}
-	// ajout un role null fictif (plus pratique) si pas défini
-	if ($action && count(explode('-', $action)) == 4) {
-		$action .= '-';
-	}
-
-	return $action;
+    $action = '';
+    if (preg_match(',^\w+-[\w*]+-[\w*]+-[\w*]+(-[\w*])?,', $k)) {
+        $action = $k;
+    }
+    if (preg_match(',^\w+-[\w*]+-[\w*]+-[\w*]+(-[\w*])?,', $v)) {
+        if (is_numeric($k)) {
+            $action = $v;
+        }
+        if (_request($k)) {
+            $action = $v;
+        }
+    }
+    // ajout un role null fictif (plus pratique) si pas défini
+    if ($action && count(explode('-', $action)) == 4) {
+        $action .= '-';
+    }
+
+    return $action;
 }
 
 
@@ -422,62 +422,62 @@ 
  *   Liste des qualifs pour chaque lien. Tableau vide s'il n'y en a pas.
  **/
 function lien_retrouver_qualif($objet_lien, $lien) {
-	// un role est défini dans la liaison
-	$defs = explode('-', $lien);
-	[$objet1, , $objet2, , $role] = array_pad($defs, 5, null);
-	$colonne_role = $objet_lien == $objet1 ? roles_colonne($objet1, $objet2) : roles_colonne($objet2, $objet1);
-
-	// cas ou le role est defini en 5e argument de l'action sur le lien (suppression, ajout rapide sans autre attribut)
-	if ($role) {
-		return [
-			// un seul lien avec ce role
-			[$colonne_role => $role]
-		];
-	}
-
-	// retrouver les rôles postés pour cette liaison, s'il y en a.
-	$qualifier_lien = _request('qualifier_lien');
-	if (!$qualifier_lien || !is_array($qualifier_lien)) {
-		return [];
-	}
-
-	// pas avec l'action complete (incluant le role)
-	$qualif = [];
-	if (
-		(!isset($qualifier_lien[$lien]) || !$qualif = $qualifier_lien[$lien])
-		&& count($defs) == 5
-	) {
-		// on tente avec l'action sans le role
-		array_pop($defs);
-		$lien = implode('-', $defs);
-		if (!isset($qualifier_lien[$lien]) || !$qualif = $qualifier_lien[$lien]) {
-			$qualif = [];
-		}
-	}
-
-	// $qualif de la forme array(role=>array(...),valeur=>array(...),....)
-	// on le reforme en array(array(role=>..,valeur=>..,..),array(role=>..,valeur=>..,..),...)
-	$qualifs = [];
-	while (is_countable($qualif) ? count($qualif) : 0) {
-		$q = [];
-		foreach ($qualif as $att => $values) {
-			if (is_array($values)) {
-				$q[$att] = array_shift($qualif[$att]);
-				if (!(is_countable($qualif[$att]) ? count($qualif[$att]) : 0)) {
-					unset($qualif[$att]);
-				}
-			} else {
-				$q[$att] = $values;
-				unset($qualif[$att]);
-			}
-		}
-		// pas de rôle vide
-		if (!$colonne_role || !isset($q[$colonne_role]) || $q[$colonne_role]) {
-			$qualifs[] = $q;
-		}
-	}
-
-	return $qualifs;
+    // un role est défini dans la liaison
+    $defs = explode('-', $lien);
+    [$objet1, , $objet2, , $role] = array_pad($defs, 5, null);
+    $colonne_role = $objet_lien == $objet1 ? roles_colonne($objet1, $objet2) : roles_colonne($objet2, $objet1);
+
+    // cas ou le role est defini en 5e argument de l'action sur le lien (suppression, ajout rapide sans autre attribut)
+    if ($role) {
+        return [
+            // un seul lien avec ce role
+            [$colonne_role => $role]
+        ];
+    }
+
+    // retrouver les rôles postés pour cette liaison, s'il y en a.
+    $qualifier_lien = _request('qualifier_lien');
+    if (!$qualifier_lien || !is_array($qualifier_lien)) {
+        return [];
+    }
+
+    // pas avec l'action complete (incluant le role)
+    $qualif = [];
+    if (
+        (!isset($qualifier_lien[$lien]) || !$qualif = $qualifier_lien[$lien])
+        && count($defs) == 5
+    ) {
+        // on tente avec l'action sans le role
+        array_pop($defs);
+        $lien = implode('-', $defs);
+        if (!isset($qualifier_lien[$lien]) || !$qualif = $qualifier_lien[$lien]) {
+            $qualif = [];
+        }
+    }
+
+    // $qualif de la forme array(role=>array(...),valeur=>array(...),....)
+    // on le reforme en array(array(role=>..,valeur=>..,..),array(role=>..,valeur=>..,..),...)
+    $qualifs = [];
+    while (is_countable($qualif) ? count($qualif) : 0) {
+        $q = [];
+        foreach ($qualif as $att => $values) {
+            if (is_array($values)) {
+                $q[$att] = array_shift($qualif[$att]);
+                if (!(is_countable($qualif[$att]) ? count($qualif[$att]) : 0)) {
+                    unset($qualif[$att]);
+                }
+            } else {
+                $q[$att] = $values;
+                unset($qualif[$att]);
+            }
+        }
+        // pas de rôle vide
+        if (!$colonne_role || !isset($q[$colonne_role]) || $q[$colonne_role]) {
+            $qualifs[] = $q;
+        }
+    }
+
+    return $qualifs;
 }
 
 /**
@@ -496,14 +496,14 @@ 
  **/
 function lien_ajouter_liaisons($objet_source, $ids, $objet_lien, $idl, $qualifs) {
 
-	// retrouver la colonne de roles s'il y en a a lier
-	if (is_array($qualifs) && count($qualifs)) {
-		foreach ($qualifs as $qualif) {
-			objet_associer([$objet_source => $ids], [$objet_lien => $idl], $qualif);
-		}
-	} else {
-		objet_associer([$objet_source => $ids], [$objet_lien => $idl]);
-	}
+    // retrouver la colonne de roles s'il y en a a lier
+    if (is_array($qualifs) && count($qualifs)) {
+        foreach ($qualifs as $qualif) {
+            objet_associer([$objet_source => $ids], [$objet_lien => $idl], $qualif);
+        }
+    } else {
+        objet_associer([$objet_source => $ids], [$objet_lien => $idl]);
+    }
 }
 
 
@@ -513,77 +513,77 @@ 
  * @return array|string|null
  */
 function lien_gerer__oups(string $form, string $action, ?array $valeur = null) {
-	static $_oups_value;
-
-	switch ($action) {
-		case 'reset':
-			$res = !empty($_oups_value);
-			$_oups_value = null;
-			return $res;
-
-		case 'get':
-			return $_oups_value ?: null;
-
-		case 'set':
-			$_oups_value = $valeur;
-			return true;
-
-		case 'request':
-			$_oups_value = null;
-			if ($oups = _request('_oups')) {
-				include_spip('inc/filtres');
-				// on accepte uniquement une valeur signée
-				if ($oups = decoder_contexte_ajax($oups, $form)) {
-					if (
-						!is_array($oups)
-						|| empty($oups['id_auteur'])
-						|| $oups['id_auteur'] !== $GLOBALS['visiteur_session']['id_auteur']
-						|| empty($oups['time'])
-						|| $oups['time'] < $_SERVER['REQUEST_TIME'] - 86400
-						|| empty($oups['args'])
-						|| $oups['args'] !== lien_gerer__oups_collecter_args($form, debug_backtrace(0, 5))
-						|| empty($oups['oups_value'])
-					) {
-						$oups = null;
-					}
-					else {
-						$oups = $oups['oups_value'];
-						// controler le contenu
-						foreach ($oups as $k => $oup) {
-							if (!is_array($oup)) {
-								unset($oups[$k]);
-							} else {
-								foreach ($oup as $champ => $valeur) {
-									if (!is_scalar($champ) || !is_scalar($valeur) || preg_match(',\W,', $champ)) {
-										unset($oups[$k][$champ]);
-									}
-								}
-								if (empty($oups[$k])) {
-									unset($oups[$k]);
-								}
-							}
-						}
-					}
-					$_oups_value = $oups;
-					return $_oups_value;
-				}
-			}
-			break;
-
-		case 'hash':
-			if (!$_oups_value) {
-				return '';
-			}
-
-			include_spip('inc/filtres');
-			$oups = [
-				'id_auteur' => $GLOBALS['visiteur_session']['id_auteur'] ?? 0,
-				'time' => strtotime(date('Y-m-d H:00:00')),
-				'args' => lien_gerer__oups_collecter_args($form, debug_backtrace(0, 5)),
-				'oups_value' => $_oups_value,
-			];
-			return encoder_contexte_ajax($oups, $form);
-	}
+    static $_oups_value;
+
+    switch ($action) {
+        case 'reset':
+            $res = !empty($_oups_value);
+            $_oups_value = null;
+            return $res;
+
+        case 'get':
+            return $_oups_value ?: null;
+
+        case 'set':
+            $_oups_value = $valeur;
+            return true;
+
+        case 'request':
+            $_oups_value = null;
+            if ($oups = _request('_oups')) {
+                include_spip('inc/filtres');
+                // on accepte uniquement une valeur signée
+                if ($oups = decoder_contexte_ajax($oups, $form)) {
+                    if (
+                        !is_array($oups)
+                        || empty($oups['id_auteur'])
+                        || $oups['id_auteur'] !== $GLOBALS['visiteur_session']['id_auteur']
+                        || empty($oups['time'])
+                        || $oups['time'] < $_SERVER['REQUEST_TIME'] - 86400
+                        || empty($oups['args'])
+                        || $oups['args'] !== lien_gerer__oups_collecter_args($form, debug_backtrace(0, 5))
+                        || empty($oups['oups_value'])
+                    ) {
+                        $oups = null;
+                    }
+                    else {
+                        $oups = $oups['oups_value'];
+                        // controler le contenu
+                        foreach ($oups as $k => $oup) {
+                            if (!is_array($oup)) {
+                                unset($oups[$k]);
+                            } else {
+                                foreach ($oup as $champ => $valeur) {
+                                    if (!is_scalar($champ) || !is_scalar($valeur) || preg_match(',\W,', $champ)) {
+                                        unset($oups[$k][$champ]);
+                                    }
+                                }
+                                if (empty($oups[$k])) {
+                                    unset($oups[$k]);
+                                }
+                            }
+                        }
+                    }
+                    $_oups_value = $oups;
+                    return $_oups_value;
+                }
+            }
+            break;
+
+        case 'hash':
+            if (!$_oups_value) {
+                return '';
+            }
+
+            include_spip('inc/filtres');
+            $oups = [
+                'id_auteur' => $GLOBALS['visiteur_session']['id_auteur'] ?? 0,
+                'time' => strtotime(date('Y-m-d H:00:00')),
+                'args' => lien_gerer__oups_collecter_args($form, debug_backtrace(0, 5)),
+                'oups_value' => $_oups_value,
+            ];
+            return encoder_contexte_ajax($oups, $form);
+    }
 }
 
 /**
@@ -593,19 +593,19 @@ 
  * @return string
  */
 function lien_gerer__oups_collecter_args($form, $trace) {
-	$args = '';
-	if (!empty($trace)) {
-		do {
-			$t = array_shift($trace);
-			$function = $t['function'] ?? '';
-			if (str_starts_with((string) $function, 'formulaires_' . $form)) {
-				if (isset($t['args'])) {
-					$args = json_encode($t['args'], JSON_THROW_ON_ERROR);
-				}
-				break;
-			}
-		}
-		while (count($trace));
-	}
-	return $args;
+    $args = '';
+    if (!empty($trace)) {
+        do {
+            $t = array_shift($trace);
+            $function = $t['function'] ?? '';
+            if (str_starts_with((string) $function, 'formulaires_' . $form)) {
+                if (isset($t['args'])) {
+                    $args = json_encode($t['args'], JSON_THROW_ON_ERROR);
+                }
+                break;
+            }
+        }
+        while (count($trace));
+    }
+    return $args;
 }

Spacing +7 added lines, -7 removed lines

@@ -105,7 +105,7 @@ 
 	// L'éditabilité :) est définie par un test permanent (par exemple "associermots") ET le 4ème argument
 	include_spip('inc/autoriser');
 	$editable = ($editable
-		&& autoriser('associer' . $table_source, $objet, $id_objet)
+		&& autoriser('associer'.$table_source, $objet, $id_objet)
 		&& autoriser('modifier', $objet, $id_objet));
 
 	if (
@@ -119,15 +119,15 @@ 
 
 	// squelettes de vue et de d'association
 	// ils sont différents si des rôles sont définis.
-	$skel_vue = $table_source . '_lies';
-	$skel_ajout = $table_source . '_associer';
+	$skel_vue = $table_source.'_lies';
+	$skel_ajout = $table_source.'_associer';
 
 	// description des roles
 	include_spip('inc/roles');
 	if ($roles = roles_presents($objet_source, $objet)) {
 		// on demande de nouveaux squelettes en conséquence
-		$skel_vue = $table_source . '_roles_lies';
-		$skel_ajout = $table_source . '_roles_associer';
+		$skel_vue = $table_source.'_roles_lies';
+		$skel_ajout = $table_source.'_roles_associer';
 	}
 
 	$oups = '';
@@ -424,7 +424,7 @@ 
 function lien_retrouver_qualif($objet_lien, $lien) {
 	// un role est défini dans la liaison
 	$defs = explode('-', $lien);
-	[$objet1, , $objet2, , $role] = array_pad($defs, 5, null);
+	[$objet1,, $objet2,, $role] = array_pad($defs, 5, null);
 	$colonne_role = $objet_lien == $objet1 ? roles_colonne($objet1, $objet2) : roles_colonne($objet2, $objet1);
 
 	// cas ou le role est defini en 5e argument de l'action sur le lien (suppression, ajout rapide sans autre attribut)
@@ -598,7 +598,7 @@ 
 		do {
 			$t = array_shift($trace);
 			$function = $t['function'] ?? '';
-			if (str_starts_with((string) $function, 'formulaires_' . $form)) {
+			if (str_starts_with((string) $function, 'formulaires_'.$form)) {
 				if (isset($t['args'])) {
 					$args = json_encode($t['args'], JSON_THROW_ON_ERROR);
 				}

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 '';
 }