spip / SPIP

prive/objets/liste/objets-trad_fonctions.php

Indentation +14 added lines, -14 removed lines

@@ -11,24 +11,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=' . intval($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=' . intval($id_trad);
+    $trouver_table = charger_fonction('trouver_table','base');
+    $desc = $trouver_table($table_objet_sql);
+    if (isset($desc['field']['statut'])) {
+        $select .= ',statut';
+        $where .= ' AND statut!='.sql_quote('poubelle');
+    }
 
-	$rows = sql_allfetsel($select, $table_objet_sql, $where);
-	lang_select();
+    $rows = sql_allfetsel($select, $table_objet_sql, $where);
+    lang_select();
 
-	return $rows;
+    return $rows;
 }

Spacing +2 added lines, -2 removed lines

@@ -19,8 +19,8 @@
 	$primary = id_table_objet($objet);
 
 	$select = "$primary as id,lang";
-	$where = 'id_trad=' . intval($id_trad);
-	$trouver_table = charger_fonction('trouver_table','base');
+	$where = 'id_trad='.intval($id_trad);
+	$trouver_table = charger_fonction('trouver_table', 'base');
 	$desc = $trouver_table($table_objet_sql);
 	if (isset($desc['field']['statut'])) {
 		$select .= ',statut';

prive/objets/liste/auteurs_lies_fonctions.php

Indentation +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('prive/objets/liste/auteurs_fonctions');

prive/objets/liste/objets-en-edition_fonctions.php

Indentation +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/drapeau_edition');

prive/objets/liste/auteurs_fonctions.php

Spacing +1 added lines, -1 removed lines

@@ -154,7 +154,7 @@
 	) {
 		return parametre_url(parametre_url(generer_url_ecrire('message_edit', 'new=oui'), 'to', $id_auteur), 'redirect', self());
 	} elseif (strlen($email) and autoriser('voir', 'auteur', $id_auteur)) {
-		return 'mailto:' . $email;
+		return 'mailto:'.$email;
 	} else {
 		return '';
 	}

Indentation +74 added lines, -74 removed lines

@@ -16,7 +16,7 @@ 
  * @package SPIP\Core\Fonctions
  **/
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -29,29 +29,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], array(), $boucles, $boucle->id_parent);
+    $_statut = calculer_liste($crit->param[0], array(), $boucles, $boucle->id_parent);
 
-	$not = '';
-	if ($crit->not) {
-		$not = ", 'NOT'";
-	}
-	$boucle->from['LAA'] = 'spip_auteurs_liens';
-	$boucle->from_type['LAA'] = 'left';
-	$boucle->join['LAA'] = array("'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'] = array("'auteurs'", "'id_auteur'", "'id_auteur'", "'LAA.objet=\'article\''");
 
-	$boucle->from['articles'] = 'spip_articles';
-	$boucle->from_type['articles'] = 'left';
-	$boucle->join['articles'] = array(
-		"'LAA'",
-		"'id_article'",
-		"'id_objet'",
-		"'(articles.statut IS NULL OR '.sql_in('articles.statut',_q($_statut)$not).')'"
-	);
+    $boucle->from['articles'] = 'spip_articles';
+    $boucle->from_type['articles'] = 'left';
+    $boucle->join['articles'] = array(
+        "'LAA'",
+        "'id_article'",
+        "'id_objet'",
+        "'(articles.statut IS NULL OR '.sql_in('articles.statut',_q($_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';
 }
 
 /**
@@ -69,7 +69,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');
 }
 
 
@@ -85,43 +85,43 @@ 
  * @return string
  */
 function afficher_initiale($url, $initiale, $compteur, $debut, $pas) {
-	static $memo = null;
-	static $res = [];
-	$out = '';
-	if (!$memo
-		or (!$initiale and !$url)
-		or ($initiale !== $memo['initiale'])
-	) {
-		$newcompt = intval(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 and !$url and !$memo['compteur']) {
-			$memo = null;
-		}
-		if ($memo) {
-			$on = (($memo['compteur'] <= $debut)
-				and (
-					$newcompt > $debut or ($newcompt == $debut and $newcompt == $memo['compteur'])
-				));
-			$res[] = lien_ou_expose($memo['url'], $memo['initiale'], $on, 'lien_pagination');
-		}
-		if ($initiale) {
-			$memo = array(
-				'entree' => isset($memo['entree']) ? $memo['entree'] + 1 : 0,
-				'initiale' => $initiale,
-				'url' => parametre_url($url, 'i', $initiale),
-				'compteur' => $newcompt
-			);
-		}
-	}
-	if (!$initiale and !$url) {
-		if (count($res) > 1) {
-			$out = implode(' ', $res);
-		}
-		$memo = null;
-		$res = [];
-	}
+    static $memo = null;
+    static $res = [];
+    $out = '';
+    if (!$memo
+        or (!$initiale and !$url)
+        or ($initiale !== $memo['initiale'])
+    ) {
+        $newcompt = intval(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 and !$url and !$memo['compteur']) {
+            $memo = null;
+        }
+        if ($memo) {
+            $on = (($memo['compteur'] <= $debut)
+                and (
+                    $newcompt > $debut or ($newcompt == $debut and $newcompt == $memo['compteur'])
+                ));
+            $res[] = lien_ou_expose($memo['url'], $memo['initiale'], $on, 'lien_pagination');
+        }
+        if ($initiale) {
+            $memo = array(
+                'entree' => isset($memo['entree']) ? $memo['entree'] + 1 : 0,
+                'initiale' => $initiale,
+                'url' => parametre_url($url, 'i', $initiale),
+                'compteur' => $newcompt
+            );
+        }
+    }
+    if (!$initiale and !$url) {
+        if (count($res) > 1) {
+            $out = implode(' ', $res);
+        }
+        $memo = null;
+        $res = [];
+    }
 
-	return $out;
+    return $out;
 }
 
 /**
@@ -140,23 +140,23 @@ 
  * @return string
  */
 function auteur_lien_messagerie($id_auteur, $en_ligne, $statut, $imessage, $email = '') {
-	static $time = null;
-	if (!in_array($statut, array('0minirezo', '1comite'))) {
-		return '';
-	}
+    static $time = null;
+    if (!in_array($statut, array('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' and !$parti // historique : est-ce que ca a encore un sens de limiter vu qu'on a la notification par email ?
-		and $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) and autoriser('voir', 'auteur', $id_auteur)) {
-		return 'mailto:' . $email;
-	} else {
-		return '';
-	}
+    if ($imessage != 'non' and !$parti // historique : est-ce que ca a encore un sens de limiter vu qu'on a la notification par email ?
+        and $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) and autoriser('voir', 'auteur', $id_auteur)) {
+        return 'mailto:' . $email;
+    } else {
+        return '';
+    }
 }

prive/objets/liste/articles_fonctions.php

Indentation +27 added lines, -27 removed lines

@@ -11,40 +11,40 @@
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 function defaut_tri_defined($defaut) {
-	if (!defined('_TRI_ARTICLES_RUBRIQUE')) {
-		return $defaut;
-	}
+    if (!defined('_TRI_ARTICLES_RUBRIQUE')) {
+        return $defaut;
+    }
 
-	$sens = 1;
-	$tri = trim(_TRI_ARTICLES_RUBRIQUE);
-	$tri = explode(' ', $tri);
-	if (strncasecmp(end($tri), 'DESC', 4) == 0) {
-		$sens = -1;
-		array_pop($tri);
-	} elseif (strncasecmp(end($tri), 'ASC', 3) == 0) {
-		$sens = 1;
-		array_pop($tri);
-	}
-	$tri = implode(' ', $tri);
-	$tri = array($tri => $sens);
-	foreach ($defaut as $n => $s) {
-		if (!isset($tri[$n])) {
-			$tri[$n] = $s;
-		}
-	}
+    $sens = 1;
+    $tri = trim(_TRI_ARTICLES_RUBRIQUE);
+    $tri = explode(' ', $tri);
+    if (strncasecmp(end($tri), 'DESC', 4) == 0) {
+        $sens = -1;
+        array_pop($tri);
+    } elseif (strncasecmp(end($tri), 'ASC', 3) == 0) {
+        $sens = 1;
+        array_pop($tri);
+    }
+    $tri = implode(' ', $tri);
+    $tri = array($tri => $sens);
+    foreach ($defaut as $n => $s) {
+        if (!isset($tri[$n])) {
+            $tri[$n] = $s;
+        }
+    }
 
-	return $tri;
+    return $tri;
 }
 
 function defaut_tri_par($par, $defaut) {
-	if (!defined('_TRI_ARTICLES_RUBRIQUE')) {
-		return $par;
-	}
-	$par = array_keys($defaut);
+    if (!defined('_TRI_ARTICLES_RUBRIQUE')) {
+        return $par;
+    }
+    $par = array_keys($defaut);
 
-	return reset($par);
+    return reset($par);
 }

prive/formulaires/inc-instituer_auteur_fonctions.php

Indentation +25 added lines, -25 removed lines

@@ -6,7 +6,7 @@ 
  * @package SPIP\Core\Formulaires
  **/
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -23,31 +23,31 @@ 
  *     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')
-		and $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')
+        and $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);
@@ -66,7 +66,7 @@ 
 	.attr('name','noname');
 });
 /*]]>*/</script>";
-	}
+    }
 
-	return $res;
+    return $res;
 }

prive/formulaires/selecteur/generique_fonctions.php

Indentation +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/filtres_selecteur_generique');

prive/formulaires/configurer_preferences_menus.php

Indentation +30 added lines, -30 removed lines

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

prive/formulaires/instituer_objet.php

Indentation +106 added lines, -106 removed lines

@@ -17,7 +17,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/editer');
@@ -32,17 +32,17 @@ 
  * @return array
  */
 function lister_statuts_proposes($desc, $publiable = true) {
-	if (!isset($desc['statut_textes_instituer'])) {
-		return false;
-	}
+    if (!isset($desc['statut_textes_instituer'])) {
+        return false;
+    }
 
-	$l = $desc['statut_textes_instituer'];
-	if (!$publiable) {
-		unset($l['publie']);
-		unset($l['refuse']);
-	}
+    $l = $desc['statut_textes_instituer'];
+    if (!$publiable) {
+        unset($l['publie']);
+        unset($l['refuse']);
+    }
 
-	return $l;
+    return $l;
 }
 
 /**
@@ -64,54 +64,54 @@ 
  *     Environnement du formulaire ou false si aucun affichage à faire.
  */
 function formulaires_instituer_objet_charger_dist($objet, $id_objet, $retour = '', $editable = true) {
-	$editable = ($editable ? true : false);
-
-	$table = table_objet_sql($objet);
-	$desc = lister_tables_objets_sql($table);
-
-	if (!isset($desc['statut_textes_instituer'])) {
-		return false;
-	}
-
-	if (!autoriser('modifier', $objet, $id_objet)) {
-		$editable = false;
-	}
-
-	// charger le contenu de l'objet
-	// dont son champ statut
-	$v = formulaires_editer_objet_charger($objet, $id_objet, 0, 0, '', '');
-
-	$publiable = true;
-	$statuts = lister_statuts_proposes($desc);
-	// tester si on a le droit de publier, si un statut publie existe
-	if (isset($statuts['publie'])) {
-		if (!autoriser('instituer', $objet, $id_objet, null, array('statut' => 'publie'))) {
-			if ($v['statut'] == 'publie') {
-				$editable = false;
-			} else {
-				$publiable = false;
-			}
-		}
-	}
-	$statuts = lister_statuts_proposes($desc, $editable ? $publiable : true);
-	if (count($statuts) == 1 and isset($statuts[$v['statut']])) {
-		$editable = false;
-	}
-
-	$valeurs = array(
-		'editable' => $editable,
-		'statut' => $v['statut'],
-		'_objet' => $objet,
-		'_id_objet' => $id_objet,
-		'_statuts' => $statuts,
-		'_publiable' => $publiable,
-		'_label' => isset($desc['texte_changer_statut']) ? $desc['texte_changer_statut'] : 'texte_article_statut',
-		'_aide' => isset($desc['aide_changer_statut']) ? $desc['aide_changer_statut'] : '',
-		'_hidden' => "<input type='hidden' name='statut_old' value='" . $v['statut'] . "' />",
-	);
-
-	#if (!count($valeurs['statuts']))
-	return $valeurs;
+    $editable = ($editable ? true : false);
+
+    $table = table_objet_sql($objet);
+    $desc = lister_tables_objets_sql($table);
+
+    if (!isset($desc['statut_textes_instituer'])) {
+        return false;
+    }
+
+    if (!autoriser('modifier', $objet, $id_objet)) {
+        $editable = false;
+    }
+
+    // charger le contenu de l'objet
+    // dont son champ statut
+    $v = formulaires_editer_objet_charger($objet, $id_objet, 0, 0, '', '');
+
+    $publiable = true;
+    $statuts = lister_statuts_proposes($desc);
+    // tester si on a le droit de publier, si un statut publie existe
+    if (isset($statuts['publie'])) {
+        if (!autoriser('instituer', $objet, $id_objet, null, array('statut' => 'publie'))) {
+            if ($v['statut'] == 'publie') {
+                $editable = false;
+            } else {
+                $publiable = false;
+            }
+        }
+    }
+    $statuts = lister_statuts_proposes($desc, $editable ? $publiable : true);
+    if (count($statuts) == 1 and isset($statuts[$v['statut']])) {
+        $editable = false;
+    }
+
+    $valeurs = array(
+        'editable' => $editable,
+        'statut' => $v['statut'],
+        '_objet' => $objet,
+        '_id_objet' => $id_objet,
+        '_statuts' => $statuts,
+        '_publiable' => $publiable,
+        '_label' => isset($desc['texte_changer_statut']) ? $desc['texte_changer_statut'] : 'texte_article_statut',
+        '_aide' => isset($desc['aide_changer_statut']) ? $desc['aide_changer_statut'] : '',
+        '_hidden' => "<input type='hidden' name='statut_old' value='" . $v['statut'] . "' />",
+    );
+
+    #if (!count($valeurs['statuts']))
+    return $valeurs;
 }
 
 /**
@@ -131,33 +131,33 @@ 
  *     Tableau des erreurs
  */
 function formulaires_instituer_objet_verifier_dist($objet, $id_objet, $retour = '', $editable = true) {
-	$erreurs = array();
-	// charger le contenu de l'objet
-	// dont son champ statut
-	$v = formulaires_editer_objet_charger($objet, $id_objet, 0, 0, '', '');
-
-	if ($v['statut'] !== _request('statut_old')) {
-		$erreurs['statut'] = _T('instituer_erreur_statut_a_change');
-	} else {
-		$table = table_objet_sql($objet);
-		$desc = lister_tables_objets_sql($table);
-
-		$publiable = true;
-		if (isset($v['id_rubrique'])
-			and !autoriser('publierdans', 'rubrique', $v['id_rubrique'])
-		) {
-			$publiable = false;
-		}
-		$l = lister_statuts_proposes($desc, $publiable);
-		$statut = _request('statut');
-		if (!isset($l[$statut])
-			or !autoriser('instituer', $objet, $id_objet, '', array('statut' => $statut))
-		) {
-			$erreurs['statut'] = _T('instituer_erreur_statut_non_autorise');
-		}
-	}
-
-	return $erreurs;
+    $erreurs = array();
+    // charger le contenu de l'objet
+    // dont son champ statut
+    $v = formulaires_editer_objet_charger($objet, $id_objet, 0, 0, '', '');
+
+    if ($v['statut'] !== _request('statut_old')) {
+        $erreurs['statut'] = _T('instituer_erreur_statut_a_change');
+    } else {
+        $table = table_objet_sql($objet);
+        $desc = lister_tables_objets_sql($table);
+
+        $publiable = true;
+        if (isset($v['id_rubrique'])
+            and !autoriser('publierdans', 'rubrique', $v['id_rubrique'])
+        ) {
+            $publiable = false;
+        }
+        $l = lister_statuts_proposes($desc, $publiable);
+        $statut = _request('statut');
+        if (!isset($l[$statut])
+            or !autoriser('instituer', $objet, $id_objet, '', array('statut' => $statut))
+        ) {
+            $erreurs['statut'] = _T('instituer_erreur_statut_non_autorise');
+        }
+    }
+
+    return $erreurs;
 }
 
 /**
@@ -176,25 +176,25 @@ 
  */
 function formulaires_instituer_objet_traiter_dist($objet, $id_objet, $retour = '', $editable = true) {
 
-	$c = array('statut' => _request('statut'));
-	// si on a envoye une 'date_posterieure', l'enregistrer
-	// todo dans le HTML
-	if ($d = _request('date_posterieure')) {
-		$c['date'] = $d;
-	}
-
-
-	include_spip('action/editer_objet');
-	if ($err = objet_instituer($objet, $id_objet, $c)) {
-		$res = array('message_erreur' => $err);
-	} else {
-		$res = array('message_ok' => _T('info_modification_enregistree'));
-		if ($retour) {
-			$res['redirect'] = $retour;
-		}
-		set_request('statut');
-		set_request('date_posterieure');
-	}
-
-	return $res;
+    $c = array('statut' => _request('statut'));
+    // si on a envoye une 'date_posterieure', l'enregistrer
+    // todo dans le HTML
+    if ($d = _request('date_posterieure')) {
+        $c['date'] = $d;
+    }
+
+
+    include_spip('action/editer_objet');
+    if ($err = objet_instituer($objet, $id_objet, $c)) {
+        $res = array('message_erreur' => $err);
+    } else {
+        $res = array('message_ok' => _T('info_modification_enregistree'));
+        if ($retour) {
+            $res['redirect'] = $retour;
+        }
+        set_request('statut');
+        set_request('date_posterieure');
+    }
+
+    return $res;
 }

Spacing +1 added lines, -1 removed lines

@@ -107,7 +107,7 @@
 		'_publiable' => $publiable,
 		'_label' => isset($desc['texte_changer_statut']) ? $desc['texte_changer_statut'] : 'texte_article_statut',
 		'_aide' => isset($desc['aide_changer_statut']) ? $desc['aide_changer_statut'] : '',
-		'_hidden' => "<input type='hidden' name='statut_old' value='" . $v['statut'] . "' />",
+		'_hidden' => "<input type='hidden' name='statut_old' value='".$v['statut']."' />",
 	);
 
 	#if (!count($valeurs['statuts']))