spip / SPIP

prive/formulaires/configurer_preferences.php

Indentation +43 added lines, -43 removed lines

@@ -20,7 +20,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -30,27 +30,27 @@ 
  *     Environnement du formulaire
  **/
 function formulaires_configurer_preferences_charger_dist() {
-	// travailler sur des meta fraiches
-	include_spip('inc/meta');
-	lire_metas();
+    // travailler sur des meta fraiches
+    include_spip('inc/meta');
+    lire_metas();
 
-	$valeurs = [];
-	$valeurs['display_navigation'] = $GLOBALS['visiteur_session']['prefs']['display_navigation'] ?? 'navigation_avec_icones';
-	$valeurs['display'] = (isset($GLOBALS['visiteur_session']['prefs']['display']) && $GLOBALS['visiteur_session']['prefs']['display'] > 0) ? $GLOBALS['visiteur_session']['prefs']['display'] : 2;
-	$valeurs['couleur'] = (isset($GLOBALS['visiteur_session']['prefs']['couleur']) && $GLOBALS['visiteur_session']['prefs']['couleur'] > 0) ? $GLOBALS['visiteur_session']['prefs']['couleur'] : 1;
+    $valeurs = [];
+    $valeurs['display_navigation'] = $GLOBALS['visiteur_session']['prefs']['display_navigation'] ?? 'navigation_avec_icones';
+    $valeurs['display'] = (isset($GLOBALS['visiteur_session']['prefs']['display']) && $GLOBALS['visiteur_session']['prefs']['display'] > 0) ? $GLOBALS['visiteur_session']['prefs']['display'] : 2;
+    $valeurs['couleur'] = (isset($GLOBALS['visiteur_session']['prefs']['couleur']) && $GLOBALS['visiteur_session']['prefs']['couleur'] > 0) ? $GLOBALS['visiteur_session']['prefs']['couleur'] : 1;
 
-	$couleurs = charger_fonction('couleurs', 'inc');
-	$les_couleurs = $couleurs();
-	foreach ($les_couleurs as $k => $c) {
-		$valeurs['_couleurs_url'][$k] = generer_url_public('style_prive.css', 'ltr='
-			. $GLOBALS['spip_lang_left'] . '&'
-			. $couleurs($k));
-		$valeurs['couleurs'][$k] = $c;
-	}
+    $couleurs = charger_fonction('couleurs', 'inc');
+    $les_couleurs = $couleurs();
+    foreach ($les_couleurs as $k => $c) {
+        $valeurs['_couleurs_url'][$k] = generer_url_public('style_prive.css', 'ltr='
+            . $GLOBALS['spip_lang_left'] . '&'
+            . $couleurs($k));
+        $valeurs['couleurs'][$k] = $c;
+    }
 
-	$valeurs['imessage'] = $GLOBALS['visiteur_session']['imessage'];
+    $valeurs['imessage'] = $GLOBALS['visiteur_session']['imessage'];
 
-	return $valeurs;
+    return $valeurs;
 }
 
 /**
@@ -61,33 +61,33 @@ 
  **/
 function formulaires_configurer_preferences_traiter_dist() {
 
-	if ($couleur = _request('couleur')) {
-		$couleurs = charger_fonction('couleurs', 'inc');
-		$les_couleurs = $couleurs([], true);
-		if (isset($les_couleurs[$couleur])) {
-			$GLOBALS['visiteur_session']['prefs']['couleur'] = $couleur;
-		}
-	}
-	if ($display = (int) _request('display')) {
-		$GLOBALS['visiteur_session']['prefs']['display'] = $display;
-	}
-	if (
-		($display_navigation = _request('display_navigation'))
-		&& in_array($display_navigation, ['navigation_sans_icone', 'navigation_avec_icones'])
-	) {
-		$GLOBALS['visiteur_session']['prefs']['display_navigation'] = $display_navigation;
-	}
+    if ($couleur = _request('couleur')) {
+        $couleurs = charger_fonction('couleurs', 'inc');
+        $les_couleurs = $couleurs([], true);
+        if (isset($les_couleurs[$couleur])) {
+            $GLOBALS['visiteur_session']['prefs']['couleur'] = $couleur;
+        }
+    }
+    if ($display = (int) _request('display')) {
+        $GLOBALS['visiteur_session']['prefs']['display'] = $display;
+    }
+    if (
+        ($display_navigation = _request('display_navigation'))
+        && in_array($display_navigation, ['navigation_sans_icone', 'navigation_avec_icones'])
+    ) {
+        $GLOBALS['visiteur_session']['prefs']['display_navigation'] = $display_navigation;
+    }
 
-	if ((int) $GLOBALS['visiteur_session']['id_auteur']) {
-		include_spip('action/editer_auteur');
-		$c = ['prefs' => serialize($GLOBALS['visiteur_session']['prefs'])];
+    if ((int) $GLOBALS['visiteur_session']['id_auteur']) {
+        include_spip('action/editer_auteur');
+        $c = ['prefs' => serialize($GLOBALS['visiteur_session']['prefs'])];
 
-		if (($imessage = _request('imessage')) && in_array($imessage, ['oui', 'non'])) {
-			$c['imessage'] = $imessage;
-		}
+        if (($imessage = _request('imessage')) && in_array($imessage, ['oui', 'non'])) {
+            $c['imessage'] = $imessage;
+        }
 
-		auteur_modifier($GLOBALS['visiteur_session']['id_auteur'], $c);
-	}
+        auteur_modifier($GLOBALS['visiteur_session']['id_auteur'], $c);
+    }
 
-	return ['message_ok' => _T('config_info_enregistree'), 'editable' => true];
+    return ['message_ok' => _T('config_info_enregistree'), 'editable' => true];
 }

prive/formulaires/configurer_articles.php

Indentation +49 added lines, -49 removed lines

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

prive/objets/liste/auteurs_fonctions.php

Indentation +69 added lines, -69 removed lines

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

prive/objets/liste/objets-trad_fonctions.php

Indentation +14 added lines, -14 removed lines

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

prive/echafaudage/hierarchie/objet_fonctions.php

Indentation +6 added lines, -6 removed lines

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

ecrire/public/boucles.php

Indentation +24 added lines, -24 removed lines

@@ -16,7 +16,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 
@@ -31,7 +31,7 @@ 
  *     Code PHP compilé de la boucle
  **/
 function boucle_DEFAUT_dist($id_boucle, &$boucles) {
-	return calculer_boucle($id_boucle, $boucles);
+    return calculer_boucle($id_boucle, $boucles);
 }
 
 
@@ -50,7 +50,7 @@ 
  *     Code PHP compilé de la boucle
  **/
 function boucle_BOUCLE_dist($id_boucle, &$boucles) {
-	return calculer_boucle($id_boucle, $boucles);
+    return calculer_boucle($id_boucle, $boucles);
 }
 
 
@@ -80,30 +80,30 @@ 
  *     Code PHP compilé de la boucle
  **/
 function boucle_HIERARCHIE_dist($id_boucle, &$boucles) {
-	$boucle = &$boucles[$id_boucle];
-	$id_table = $boucle->id_table . '.id_rubrique';
+    $boucle = &$boucles[$id_boucle];
+    $id_table = $boucle->id_table . '.id_rubrique';
 
-	// Si la boucle mere est une boucle RUBRIQUES il faut ignorer la feuille
-	// sauf en presence du critere {tout} (vu par phraser_html)
-	// ou {id_article} qui positionne aussi le {tout}
+    // Si la boucle mere est une boucle RUBRIQUES il faut ignorer la feuille
+    // sauf en presence du critere {tout} (vu par phraser_html)
+    // ou {id_article} qui positionne aussi le {tout}
 
-	$boucle->hierarchie = 'if (!($id_rubrique = intval('
-		. calculer_argument_precedent($boucle->id_boucle, 'id_rubrique', $boucles)
-		. ")))\n\t\treturn '';\n\t"
-		. "include_spip('inc/rubriques');\n\t"
-		. '$hierarchie = calcul_hierarchie_in($id_rubrique,'
-		. (isset($boucle->modificateur['tout']) ? 'true' : 'false')
-		. ");\n\t"
-		. 'if (!$hierarchie) return "";' . "\n\t";
+    $boucle->hierarchie = 'if (!($id_rubrique = intval('
+        . calculer_argument_precedent($boucle->id_boucle, 'id_rubrique', $boucles)
+        . ")))\n\t\treturn '';\n\t"
+        . "include_spip('inc/rubriques');\n\t"
+        . '$hierarchie = calcul_hierarchie_in($id_rubrique,'
+        . (isset($boucle->modificateur['tout']) ? 'true' : 'false')
+        . ");\n\t"
+        . 'if (!$hierarchie) return "";' . "\n\t";
 
-	$boucle->where[] = ["'IN'", "'$id_table'", '"($hierarchie)"'];
+    $boucle->where[] = ["'IN'", "'$id_table'", '"($hierarchie)"'];
 
-	$order = "FIELD($id_table, \$hierarchie)";
-	if (!isset($boucle->default_order[0]) || $boucle->default_order[0] != ' DESC') {
-		$boucle->default_order[] = "\"$order\"";
-	} else {
-		$boucle->default_order[0] = "\"$order DESC\"";
-	}
+    $order = "FIELD($id_table, \$hierarchie)";
+    if (!isset($boucle->default_order[0]) || $boucle->default_order[0] != ' DESC') {
+        $boucle->default_order[] = "\"$order\"";
+    } else {
+        $boucle->default_order[0] = "\"$order DESC\"";
+    }
 
-	return calculer_boucle($id_boucle, $boucles);
+    return calculer_boucle($id_boucle, $boucles);
 }

ecrire/action/supprimer_rubrique.php

Indentation +38 added lines, -38 removed lines

@@ -16,7 +16,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/charsets');  # pour le nom de fichier
@@ -29,45 +29,45 @@ 
  */
 function action_supprimer_rubrique_dist($id_rubrique = null) {
 
-	if (is_null($id_rubrique)) {
-		$securiser_action = charger_fonction('securiser_action', 'inc');
-		$id_rubrique = $securiser_action();
-	}
+    if (is_null($id_rubrique)) {
+        $securiser_action = charger_fonction('securiser_action', 'inc');
+        $id_rubrique = $securiser_action();
+    }
 
-	if ((int) $id_rubrique) {
-		sql_delete('spip_rubriques', 'id_rubrique=' . (int) $id_rubrique);
-		// Les admin restreints qui n'administraient que cette rubrique
-		// deviennent redacteurs
-		// (il y a sans doute moyen de faire ca avec un having)
+    if ((int) $id_rubrique) {
+        sql_delete('spip_rubriques', 'id_rubrique=' . (int) $id_rubrique);
+        // Les admin restreints qui n'administraient que cette rubrique
+        // deviennent redacteurs
+        // (il y a sans doute moyen de faire ca avec un having)
 
-		$q = sql_select('id_auteur', 'spip_auteurs_liens', "objet='rubrique' AND id_objet=" . (int) $id_rubrique);
-		while ($r = sql_fetch($q)) {
-			$id_auteur = $r['id_auteur'];
-			// degrader avant de supprimer la restriction d'admin
-			// section critique sur les droits
-			$n = sql_countsel(
-				'spip_auteurs_liens',
-				"objet='rubrique' AND id_objet!=" . (int) $id_rubrique . ' AND id_auteur=' . (int) $id_auteur
-			);
-			if (!$n) {
-				include_spip('action/editer_auteur');
-				auteur_modifier($id_auteur, ['statut' => '1comite']);
-			}
-			sql_delete(
-				'spip_auteurs_liens',
-				"objet='rubrique' AND id_objet=" . (int) $id_rubrique . ' AND id_auteur=' . (int) $id_auteur
-			);
-		}
-		// menu_rubriques devra recalculer
-		effacer_meta('date_calcul_rubriques');
+        $q = sql_select('id_auteur', 'spip_auteurs_liens', "objet='rubrique' AND id_objet=" . (int) $id_rubrique);
+        while ($r = sql_fetch($q)) {
+            $id_auteur = $r['id_auteur'];
+            // degrader avant de supprimer la restriction d'admin
+            // section critique sur les droits
+            $n = sql_countsel(
+                'spip_auteurs_liens',
+                "objet='rubrique' AND id_objet!=" . (int) $id_rubrique . ' AND id_auteur=' . (int) $id_auteur
+            );
+            if (!$n) {
+                include_spip('action/editer_auteur');
+                auteur_modifier($id_auteur, ['statut' => '1comite']);
+            }
+            sql_delete(
+                'spip_auteurs_liens',
+                "objet='rubrique' AND id_objet=" . (int) $id_rubrique . ' AND id_auteur=' . (int) $id_auteur
+            );
+        }
+        // menu_rubriques devra recalculer
+        effacer_meta('date_calcul_rubriques');
 
-		// Une rubrique supprimable n'avait pas le statut "publie"
-		// donc rien de neuf pour la rubrique parente
-		include_spip('inc/rubriques');
-		calculer_langues_rubriques();
+        // Une rubrique supprimable n'avait pas le statut "publie"
+        // donc rien de neuf pour la rubrique parente
+        include_spip('inc/rubriques');
+        calculer_langues_rubriques();
 
-		// invalider les caches marques de cette rubrique
-		include_spip('inc/invalideur');
-		suivre_invalideur("id='rubrique/$id_rubrique'");
-	}
+        // invalider les caches marques de cette rubrique
+        include_spip('inc/invalideur');
+        suivre_invalideur("id='rubrique/$id_rubrique'");
+    }
 }

ecrire/urls/page.php

Indentation +31 added lines, -31 removed lines

@@ -10,7 +10,7 @@ 
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 defined('URLS_PAGE_EXEMPLE') || define('URLS_PAGE_EXEMPLE', 'spip.php?article12');
@@ -29,23 +29,23 @@ 
  */
 function urls_page_generer_url_objet_dist(int $id, string $objet, string $args = '', string $ancre = ''): string {
 
-	if ($generer_url_externe = charger_fonction_url($objet, 'defaut')) {
-		$url = $generer_url_externe($id, $args, $ancre);
-		// une url === null indique "je ne traite pas cette url, appliquez le calcul standard"
-		// une url vide est une url vide, ne rien faire de plus
-		if (!is_null($url)) {
-			return $url;
-		}
-	}
+    if ($generer_url_externe = charger_fonction_url($objet, 'defaut')) {
+        $url = $generer_url_externe($id, $args, $ancre);
+        // une url === null indique "je ne traite pas cette url, appliquez le calcul standard"
+        // une url vide est une url vide, ne rien faire de plus
+        if (!is_null($url)) {
+            return $url;
+        }
+    }
 
-	$url = \_debut_urls_page . $objet . \_separateur_urls_page
-		. $id . \_terminaison_urls_page;
+    $url = \_debut_urls_page . $objet . \_separateur_urls_page
+        . $id . \_terminaison_urls_page;
 
-	if ($args) {
-		$args = strpos($url, '?') ? "&$args" : "?$args";
-	}
+    if ($args) {
+        $args = strpos($url, '?') ? "&$args" : "?$args";
+    }
 
-	return _DIR_RACINE . $url . $args . ($ancre ? "#$ancre" : '');
+    return _DIR_RACINE . $url . $args . ($ancre ? "#$ancre" : '');
 }
 
 /**
@@ -61,27 +61,27 @@ 
  */
 function urls_page_decoder_url_dist(string $url, string $entite, array $contexte = []): array {
 
-	// traiter les injections du type domaine.org/spip.php/cestnimportequoi/ou/encore/plus/rubrique23
-	if ($GLOBALS['profondeur_url'] > 0 && $entite == 'sommaire') {
-		return [[], '404'];
-	}
+    // traiter les injections du type domaine.org/spip.php/cestnimportequoi/ou/encore/plus/rubrique23
+    if ($GLOBALS['profondeur_url'] > 0 && $entite == 'sommaire') {
+        return [[], '404'];
+    }
 
-	include_spip('inc/urls');
-	$r = nettoyer_url_page($url, $contexte);
-	if ($r) {
-		array_pop($r); // nettoyer_url_page renvoie un argument de plus inutile ici
-		return $r;
-	}
+    include_spip('inc/urls');
+    $r = nettoyer_url_page($url, $contexte);
+    if ($r) {
+        array_pop($r); // nettoyer_url_page renvoie un argument de plus inutile ici
+        return $r;
+    }
 
-	/*
+    /*
 	 * Le bloc qui suit sert a faciliter les transitions depuis
 	 * le mode 'urls-propres' vers les modes 'urls-standard' et 'url-html'
 	 * Il est inutile de le recopier si vous personnalisez vos URLs
 	 * et votre .htaccess
 	 */
-	// Si on est revenu en mode html, mais c'est une ancienne url_propre
-	// on ne redirige pas, on assume le nouveau contexte (si possible)
-	$url_propre = $url ?? $_SERVER['REDIRECT_url_propre'] ?? $_ENV['url_propre'] ?? '';
-	return urls_transition_retrouver_anciennes_url_propres($url_propre, $entite, $contexte);
-	/* Fin du bloc compatibilite url-propres */
+    // Si on est revenu en mode html, mais c'est une ancienne url_propre
+    // on ne redirige pas, on assume le nouveau contexte (si possible)
+    $url_propre = $url ?? $_SERVER['REDIRECT_url_propre'] ?? $_ENV['url_propre'] ?? '';
+    return urls_transition_retrouver_anciennes_url_propres($url_propre, $entite, $contexte);
+    /* Fin du bloc compatibilite url-propres */
 }

ecrire/balise/formulaire_ecrire_auteur.php

Indentation +38 added lines, -38 removed lines

@@ -16,7 +16,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('base/abstract_sql');
@@ -40,7 +40,7 @@ 
  *     Pile complétée du code compilé
  **/
 function balise_FORMULAIRE_ECRIRE_AUTEUR($p) {
-	return calculer_balise_dynamique($p, 'FORMULAIRE_ECRIRE_AUTEUR', ['id_auteur', 'id_article', 'email']);
+    return calculer_balise_dynamique($p, 'FORMULAIRE_ECRIRE_AUTEUR', ['id_auteur', 'id_article', 'email']);
 }
 
 /**
@@ -58,44 +58,44 @@ 
  *   - chaîne vide sinon (erreur ou non affichage).
  */
 function balise_FORMULAIRE_ECRIRE_AUTEUR_stat($args, $context_compil) {
-	include_spip('inc/filtres');
-	// Pas d'id_auteur ni d'id_article ? Erreur de contexte
-	$id = (int) $args[1];
-	if (!$args[0] && !$id) {
-		$msg = [
-			'zbug_champ_hors_motif',
-			[
-				'champ' => 'FORMULAIRE_ECRIRE_AUTEUR',
-				'motif' => 'AUTEURS/ARTICLES'
-			]
-		];
+    include_spip('inc/filtres');
+    // Pas d'id_auteur ni d'id_article ? Erreur de contexte
+    $id = (int) $args[1];
+    if (!$args[0] && !$id) {
+        $msg = [
+            'zbug_champ_hors_motif',
+            [
+                'champ' => 'FORMULAIRE_ECRIRE_AUTEUR',
+                'motif' => 'AUTEURS/ARTICLES'
+            ]
+        ];
 
-		erreur_squelette($msg, $context_compil);
+        erreur_squelette($msg, $context_compil);
 
-		return '';
-	}
-	// Si on est dans un contexte article,
-	// sortir tous les mails des auteurs de l'article
-	if (!$args[0] && $id) {
-		$r = '';
-		$s = sql_allfetsel(
-			'email',
-			'spip_auteurs AS A LEFT JOIN spip_auteurs_liens AS L ON (A.id_auteur=L.id_auteur AND L.objet=\'article\')',
-			"A.email != '' AND L.id_objet=$id"
-		);
-		foreach ($s as $row) {
-			if (email_valide($row['email'])) {
-				$r .= ', ' . $row['email'];
-			}
-		}
-		$args[2] = substr($r, 2);
-	}
+        return '';
+    }
+    // Si on est dans un contexte article,
+    // sortir tous les mails des auteurs de l'article
+    if (!$args[0] && $id) {
+        $r = '';
+        $s = sql_allfetsel(
+            'email',
+            'spip_auteurs AS A LEFT JOIN spip_auteurs_liens AS L ON (A.id_auteur=L.id_auteur AND L.objet=\'article\')',
+            "A.email != '' AND L.id_objet=$id"
+        );
+        foreach ($s as $row) {
+            if (email_valide($row['email'])) {
+                $r .= ', ' . $row['email'];
+            }
+        }
+        $args[2] = substr($r, 2);
+    }
 
-	// On ne peut pas ecrire a un auteur dont le mail n'est pas valide
-	if (!$args[2] || !email_valide($args[2])) {
-		return '';
-	}
+    // On ne peut pas ecrire a un auteur dont le mail n'est pas valide
+    if (!$args[2] || !email_valide($args[2])) {
+        return '';
+    }
 
-	// OK
-	return $args;
+    // OK
+    return $args;
 }