spip / SPIP

prive/formulaires/editer_liens.php

Indentation +352 added lines, -352 removed lines

@@ -16,7 +16,7 @@ 
  * @package SPIP\Core\Formulaires
  **/
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 
@@ -36,25 +36,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) and !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) and !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) and !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) and !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];
 }
 
 /**
@@ -80,93 +80,93 @@ 
  */
 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'];
-
-	list($table_source, $objet, $id_objet, $objet_lien) = determine_source_lien_objet($a, $b, $c);
-	if (!$table_source or !$objet or !$objet_lien or !$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 and autoriser('associer' . $table_source, $objet, $id_objet)
-		and autoriser('modifier', $objet, $id_objet));
-
-	if (
-		!$editable and !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 = _request('_oups');
-	if (unserialize(base64_decode($oups))) {
-		// on est bon, rien a faire
-	}
-	elseif (unserialize($oups)) {
-		// il faut encoder
-		$oups = base64_encode($oups);
-	}
-	else {
-		$oups = '';
-	}
-	$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' => $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'];
+
+    list($table_source, $objet, $id_objet, $objet_lien) = determine_source_lien_objet($a, $b, $c);
+    if (!$table_source or !$objet or !$objet_lien or !$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 and autoriser('associer' . $table_source, $objet, $id_objet)
+        and autoriser('modifier', $objet, $id_objet));
+
+    if (
+        !$editable and !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 = _request('_oups');
+    if (unserialize(base64_decode($oups))) {
+        // on est bon, rien a faire
+    }
+    elseif (unserialize($oups)) {
+        // il faut encoder
+        $oups = base64_encode($oups);
+    }
+    else {
+        $oups = '';
+    }
+    $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' => $oups,
+        'editable' => $editable,
+    ];
+
+    // les options non definies dans $valeurs sont passees telles quelles au formulaire html
+    $valeurs = array_merge($options, $valeurs);
+
+    return $valeurs;
 }
 
 /**
@@ -210,165 +210,165 @@ 
  * @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' => $editable ? true : false];
-	list($table_source, $objet, $id_objet, $objet_lien) = determine_source_lien_objet($a, $b, $c);
-	if (!$table_source or !$objet or !$objet_lien) {
-		return $res;
-	}
-
-
-	if (_request('tout_voir')) {
-		set_request('recherche', '');
-	}
-
-	include_spip('inc/autoriser');
-	if (autoriser('modifier', $objet, $id_objet)) {
-		// annuler les suppressions du coup d'avant !
-		if (
-			_request('annuler_oups')
-			and $oups = _request('_oups')
-			and $oups = base64_decode($oups)
-			and $oups = unserialize($oups)
-		) {
-			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
-			set_request('_oups');
-		}
-
-		$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);
-						list($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);
-						}
-					}
-				}
-			}
-			set_request('_oups', $oups ? base64_encode(serialize($oups)) : null);
-		}
-
-		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;
-						list($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) {
-				set_request('_oups');
-			}
-		}
-
-		if ($ordonner) {
-			include_spip('action/editer_liens');
-			foreach ($ordonner as $k => $rang_lien) {
-				if ($lien = lien_verifier_action($k, '')) {
-					list($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);
-					set_request('_oups');
-				}
-			}
-		}
-	}
-
-
-	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' => $editable ? true : false];
+    list($table_source, $objet, $id_objet, $objet_lien) = determine_source_lien_objet($a, $b, $c);
+    if (!$table_source or !$objet or !$objet_lien) {
+        return $res;
+    }
+
+
+    if (_request('tout_voir')) {
+        set_request('recherche', '');
+    }
+
+    include_spip('inc/autoriser');
+    if (autoriser('modifier', $objet, $id_objet)) {
+        // annuler les suppressions du coup d'avant !
+        if (
+            _request('annuler_oups')
+            and $oups = _request('_oups')
+            and $oups = base64_decode($oups)
+            and $oups = unserialize($oups)
+        ) {
+            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
+            set_request('_oups');
+        }
+
+        $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);
+                        list($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);
+                        }
+                    }
+                }
+            }
+            set_request('_oups', $oups ? base64_encode(serialize($oups)) : null);
+        }
+
+        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;
+                        list($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) {
+                set_request('_oups');
+            }
+        }
+
+        if ($ordonner) {
+            include_spip('action/editer_liens');
+            foreach ($ordonner as $k => $rang_lien) {
+                if ($lien = lien_verifier_action($k, '')) {
+                    list($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);
+                    set_request('_oups');
+                }
+            }
+        }
+    }
+
+
+    return $res;
 }
 
 
@@ -391,24 +391,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 and 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 and count(explode('-', $action)) == 4) {
+        $action .= '-';
+    }
+
+    return $action;
 }
 
 
@@ -424,66 +424,66 @@ 
  *   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);
-	list($objet1, , $objet2, , $role) = array_pad($defs, 5, null);
-	if ($objet_lien == $objet1) {
-		$colonne_role = roles_colonne($objet1, $objet2);
-	} else {
-		$colonne_role = 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 or !is_array($qualifier_lien)) {
-		return [];
-	}
-
-	// pas avec l'action complete (incluant le role)
-	$qualif = [];
-	if (
-		(!isset($qualifier_lien[$lien]) or !$qualif = $qualifier_lien[$lien])
-		and count($defs) == 5
-	) {
-		// on tente avec l'action sans le role
-		array_pop($defs);
-		$lien = implode('-', $defs);
-		if (!isset($qualifier_lien[$lien]) or !$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 (count($qualif)) {
-		$q = [];
-		foreach ($qualif as $att => $values) {
-			if (is_array($values)) {
-				$q[$att] = array_shift($qualif[$att]);
-				if (!count($qualif[$att])) {
-					unset($qualif[$att]);
-				}
-			} else {
-				$q[$att] = $values;
-				unset($qualif[$att]);
-			}
-		}
-		// pas de rôle vide
-		if (!$colonne_role or !isset($q[$colonne_role]) or $q[$colonne_role]) {
-			$qualifs[] = $q;
-		}
-	}
-
-	return $qualifs;
+    // un role est défini dans la liaison
+    $defs = explode('-', $lien);
+    list($objet1, , $objet2, , $role) = array_pad($defs, 5, null);
+    if ($objet_lien == $objet1) {
+        $colonne_role = roles_colonne($objet1, $objet2);
+    } else {
+        $colonne_role = 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 or !is_array($qualifier_lien)) {
+        return [];
+    }
+
+    // pas avec l'action complete (incluant le role)
+    $qualif = [];
+    if (
+        (!isset($qualifier_lien[$lien]) or !$qualif = $qualifier_lien[$lien])
+        and count($defs) == 5
+    ) {
+        // on tente avec l'action sans le role
+        array_pop($defs);
+        $lien = implode('-', $defs);
+        if (!isset($qualifier_lien[$lien]) or !$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 (count($qualif)) {
+        $q = [];
+        foreach ($qualif as $att => $values) {
+            if (is_array($values)) {
+                $q[$att] = array_shift($qualif[$att]);
+                if (!count($qualif[$att])) {
+                    unset($qualif[$att]);
+                }
+            } else {
+                $q[$att] = $values;
+                unset($qualif[$att]);
+            }
+        }
+        // pas de rôle vide
+        if (!$colonne_role or !isset($q[$colonne_role]) or $q[$colonne_role]) {
+            $qualifs[] = $q;
+        }
+    }
+
+    return $qualifs;
 }
 
 /**
@@ -502,12 +502,12 @@ 
  **/
 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) and 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) and count($qualifs)) {
+        foreach ($qualifs as $qualif) {
+            objet_associer([$objet_source => $ids], [$objet_lien => $idl], $qualif);
+        }
+    } else {
+        objet_associer([$objet_source => $ids], [$objet_lien => $idl]);
+    }
 }

Spacing +6 added lines, -6 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 and autoriser('associer' . $table_source, $objet, $id_objet)
+	$editable = ($editable and autoriser('associer'.$table_source, $objet, $id_objet)
 		and 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 = _request('_oups');
@@ -426,7 +426,7 @@ 
 function lien_retrouver_qualif($objet_lien, $lien) {
 	// un role est défini dans la liaison
 	$defs = explode('-', $lien);
-	list($objet1, , $objet2, , $role) = array_pad($defs, 5, null);
+	list($objet1,, $objet2,, $role) = array_pad($defs, 5, null);
 	if ($objet_lien == $objet1) {
 		$colonne_role = roles_colonne($objet1, $objet2);
 	} else {

Braces +2 added lines, -4 removed lines

@@ -133,12 +133,10 @@
 	$oups = _request('_oups');
 	if (unserialize(base64_decode($oups))) {
 		// on est bon, rien a faire
-	}
-	elseif (unserialize($oups)) {
+	} elseif (unserialize($oups)) {
 		// il faut encoder
 		$oups = base64_encode($oups);
-	}
-	else {
+	} else {
 		$oups = '';
 	}
 	$valeurs = [

prive/formulaires/editer_rubrique.php

Indentation +82 added lines, -82 removed lines

@@ -17,7 +17,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/actions');
@@ -47,30 +47,30 @@ 
  *     Environnement du formulaire
  **/
 function formulaires_editer_rubrique_charger_dist(
-	$id_rubrique = 'new',
-	$id_parent = 0,
-	$retour = '',
-	$lier_trad = 0,
-	$config_fonc = 'rubriques_edit_config',
-	$row = [],
-	$hidden = ''
+    $id_rubrique = 'new',
+    $id_parent = 0,
+    $retour = '',
+    $lier_trad = 0,
+    $config_fonc = 'rubriques_edit_config',
+    $row = [],
+    $hidden = ''
 ) {
-	$valeurs = formulaires_editer_objet_charger(
-		'rubrique',
-		$id_rubrique,
-		$id_parent,
-		$lier_trad,
-		$retour,
-		$config_fonc,
-		$row,
-		$hidden
-	);
-
-	if (intval($id_rubrique) and !autoriser('modifier', 'rubrique', intval($id_rubrique))) {
-		$valeurs['editable'] = '';
-	}
-
-	return $valeurs;
+    $valeurs = formulaires_editer_objet_charger(
+        'rubrique',
+        $id_rubrique,
+        $id_parent,
+        $lier_trad,
+        $retour,
+        $config_fonc,
+        $row,
+        $hidden
+    );
+
+    if (intval($id_rubrique) and !autoriser('modifier', 'rubrique', intval($id_rubrique))) {
+        $valeurs['editable'] = '';
+    }
+
+    return $valeurs;
 }
 
 /**
@@ -82,15 +82,15 @@ 
  *     Configuration pour le formulaire
  */
 function rubriques_edit_config($row) {
-	global $spip_lang;
+    global $spip_lang;
 
-	$config = $GLOBALS['meta'];
-	$config['lignes'] = 8;
-	$config['langue'] = $spip_lang;
+    $config = $GLOBALS['meta'];
+    $config['lignes'] = 8;
+    $config['langue'] = $spip_lang;
 
-	$config['restreint'] = (!$GLOBALS['connect_toutes_rubriques']);
+    $config['restreint'] = (!$GLOBALS['connect_toutes_rubriques']);
 
-	return $config;
+    return $config;
 }
 
 /**
@@ -115,15 +115,15 @@ 
  *     Hash du formulaire
  */
 function formulaires_editer_rubrique_identifier_dist(
-	$id_rubrique = 'new',
-	$id_parent = 0,
-	$retour = '',
-	$lier_trad = 0,
-	$config_fonc = 'rubriques_edit_config',
-	$row = [],
-	$hidden = ''
+    $id_rubrique = 'new',
+    $id_parent = 0,
+    $retour = '',
+    $lier_trad = 0,
+    $config_fonc = 'rubriques_edit_config',
+    $row = [],
+    $hidden = ''
 ) {
-	return serialize([intval($id_rubrique), $lier_trad]);
+    return serialize([intval($id_rubrique), $lier_trad]);
 }
 
 /**
@@ -149,34 +149,34 @@ 
  *     Erreurs du formulaire
  **/
 function formulaires_editer_rubrique_verifier_dist(
-	$id_rubrique = 'new',
-	$id_parent = 0,
-	$retour = '',
-	$lier_trad = 0,
-	$config_fonc = 'rubriques_edit_config',
-	$row = [],
-	$hidden = ''
+    $id_rubrique = 'new',
+    $id_parent = 0,
+    $retour = '',
+    $lier_trad = 0,
+    $config_fonc = 'rubriques_edit_config',
+    $row = [],
+    $hidden = ''
 ) {
-	// auto-renseigner le titre si il n'existe pas
-	titre_automatique('titre', ['descriptif', 'texte']);
-	// on ne demande pas le titre obligatoire : il sera rempli a la volee dans editer_rubrique si vide
-	$erreurs = formulaires_editer_objet_verifier('rubrique', $id_rubrique, []);
-
-	// s'assurer qu'on ne s'auto-designe pas comme parent !
-	if (
-		intval($id_rubrique)
-		and empty($erreurs['id_parent'])
-		and $id_parent = _request('id_parent')
-	) {
-		include_spip('inc/rubriques');
-		$branche = calcul_branche_in($id_rubrique);
-		$branche = explode(',', $branche);
-		if (in_array($id_parent, $branche)) {
-			$erreurs['id_parent'] = _L('Impossible de déplacer une rubrique dans sa propre branche, on tourne en rond !');
-		}
-	}
-
-	return $erreurs;
+    // auto-renseigner le titre si il n'existe pas
+    titre_automatique('titre', ['descriptif', 'texte']);
+    // on ne demande pas le titre obligatoire : il sera rempli a la volee dans editer_rubrique si vide
+    $erreurs = formulaires_editer_objet_verifier('rubrique', $id_rubrique, []);
+
+    // s'assurer qu'on ne s'auto-designe pas comme parent !
+    if (
+        intval($id_rubrique)
+        and empty($erreurs['id_parent'])
+        and $id_parent = _request('id_parent')
+    ) {
+        include_spip('inc/rubriques');
+        $branche = calcul_branche_in($id_rubrique);
+        $branche = explode(',', $branche);
+        if (in_array($id_parent, $branche)) {
+            $erreurs['id_parent'] = _L('Impossible de déplacer une rubrique dans sa propre branche, on tourne en rond !');
+        }
+    }
+
+    return $erreurs;
 }
 
 /**
@@ -202,22 +202,22 @@ 
  *     Retour des traitements
  **/
 function formulaires_editer_rubrique_traiter_dist(
-	$id_rubrique = 'new',
-	$id_parent = 0,
-	$retour = '',
-	$lier_trad = 0,
-	$config_fonc = 'rubriques_edit_config',
-	$row = [],
-	$hidden = ''
+    $id_rubrique = 'new',
+    $id_parent = 0,
+    $retour = '',
+    $lier_trad = 0,
+    $config_fonc = 'rubriques_edit_config',
+    $row = [],
+    $hidden = ''
 ) {
-	return formulaires_editer_objet_traiter(
-		'rubrique',
-		$id_rubrique,
-		$id_parent,
-		$lier_trad,
-		$retour,
-		$config_fonc,
-		$row,
-		$hidden
-	);
+    return formulaires_editer_objet_traiter(
+        'rubrique',
+        $id_rubrique,
+        $id_parent,
+        $lier_trad,
+        $retour,
+        $config_fonc,
+        $row,
+        $hidden
+    );
 }

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')
-		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);
@@ -68,7 +68,7 @@ 
 	.attr('name','noname');
 });
 /*]]>*/</script>";
-	}
+    }
 
-	return $res;
+    return $res;
 }

prive/formulaires/traduire.php

Indentation +114 added lines, -114 removed lines

@@ -17,7 +17,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/actions');
@@ -39,65 +39,65 @@ 
  *     Contexte à transmettre au squelette du formulaire sinon
  */
 function formulaires_traduire_charger_dist($objet, $id_objet, $retour = '', $traduire = true) {
-	if (!intval($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=' . intval($id_parent));
-	}
-
-	if (!$langue_parent) {
-		$langue_parent = $GLOBALS['meta']['langue_site'];
-	}
-	if (
-		$valeurs['editable']
-		and in_array(table_objet_sql($objet), explode(',', $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') and (_request('changer_lang') !== null
-		or _request('changer_id_trad') !== null));
-	$valeurs['_pipeline'] = ['traduire', ['type' => $objet, 'id' => $id_objet]];
-
-	return $valeurs;
+    if (!intval($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=' . intval($id_parent));
+    }
+
+    if (!$langue_parent) {
+        $langue_parent = $GLOBALS['meta']['langue_site'];
+    }
+    if (
+        $valeurs['editable']
+        and in_array(table_objet_sql($objet), explode(',', $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') and (_request('changer_lang') !== null
+        or _request('changer_id_trad') !== null));
+    $valeurs['_pipeline'] = ['traduire', ['type' => $objet, 'id' => $id_objet]];
+
+    return $valeurs;
 }
 
 /**
@@ -115,32 +115,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=" . intval($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=" . intval($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=" . intval($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=" . intval($id_trad))) {
+            $erreurs['id_trad'] = _L('Indiquez un contenu existant');
+        }
+    }
+
+    return $erreurs;
 }
 
 
@@ -159,32 +159,32 @@ 
  *     Retour des traitements
  */
 function formulaires_traduire_traiter_dist($objet, $id_objet, $retour = '', $traduire = true) {
-	$res = [];
-	if (!_request('annuler') and 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') and autoriser('changertraduction', $objet, $id_objet)) {
-		if ($id_trad = _request('id_trad') or _request('supprimer_trad')) {
-			$referencer_traduction = charger_fonction('referencer_traduction', 'action');
-			$referencer_traduction($objet, $id_objet, intval($id_trad)); // 0 si supprimer_trad
-		} elseif (
-			$new_id_trad = _request('changer_reference_trad')
-			and $new_id_trad = array_keys($new_id_trad)
-			and $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=" . intval($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') and 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') and autoriser('changertraduction', $objet, $id_objet)) {
+        if ($id_trad = _request('id_trad') or _request('supprimer_trad')) {
+            $referencer_traduction = charger_fonction('referencer_traduction', 'action');
+            $referencer_traduction($objet, $id_objet, intval($id_trad)); // 0 si supprimer_trad
+        } elseif (
+            $new_id_trad = _request('changer_reference_trad')
+            and $new_id_trad = array_keys($new_id_trad)
+            and $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=" . intval($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;
 }

Spacing +6 added lines, -6 removed lines

@@ -60,7 +60,7 @@ 
 		$id_parent = $valeurs['id_parent'];
 	}
 	if ($id_parent) {
-		$langue_parent = sql_getfetsel('lang', 'spip_rubriques', 'id_rubrique=' . intval($id_parent));
+		$langue_parent = sql_getfetsel('lang', 'spip_rubriques', 'id_rubrique='.intval($id_parent));
 	}
 
 	if (!$langue_parent) {
@@ -83,8 +83,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
@@ -130,12 +130,12 @@ 
 			sql_getfetsel(
 				'id_trad',
 				$table_objet_sql,
-				"$_id_table_objet=" . intval($id_objet)
+				"$_id_table_objet=".intval($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=" . intval($id_trad))) {
+		} elseif (!sql_getfetsel($_id_table_objet, $table_objet_sql, "$_id_table_objet=".intval($id_trad))) {
 			$erreurs['id_trad'] = _L('Indiquez un contenu existant');
 		}
 	}
@@ -175,7 +175,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=" . intval($id_objet))) {
+			if ($id_trad = sql_getfetsel('id_trad', $table_objet_sql, "$_id_table_objet=".intval($id_objet))) {
 				$referencer_traduction = charger_fonction('referencer_traduction', 'action');
 				$referencer_traduction($objet, $id_trad, $new_id_trad);
 			}

prive/formulaires/recherche_ecrire.php

Indentation +15 added lines, -15 removed lines

@@ -17,7 +17,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -33,19 +33,19 @@ 
  * @return array Environnement du formulaire
  **/
 function formulaires_recherche_ecrire_charger_dist($action = '', $class = '') {
-	if ($GLOBALS['spip_lang'] != $GLOBALS['meta']['langue_site']) {
-		$lang = $GLOBALS['spip_lang'];
-	} else {
-		$lang = '';
-	}
+    if ($GLOBALS['spip_lang'] != $GLOBALS['meta']['langue_site']) {
+        $lang = $GLOBALS['spip_lang'];
+    } else {
+        $lang = '';
+    }
 
-	return
-		[
-			'action' => ($action ? $action : generer_url_ecrire('recherche')),
-			# action specifique, ne passe pas par Verifier, ni Traiter
-			'recherche' => _request('recherche'),
-			'lang' => $lang,
-			'class' => $class,
-			'_id_champ' => 'rechercher_' . substr(md5($action . $class), 0, 4),
-		];
+    return
+        [
+            'action' => ($action ? $action : generer_url_ecrire('recherche')),
+            # action specifique, ne passe pas par Verifier, ni Traiter
+            'recherche' => _request('recherche'),
+            'lang' => $lang,
+            'class' => $class,
+            '_id_champ' => 'rechercher_' . substr(md5($action . $class), 0, 4),
+        ];
 }

Spacing +1 added lines, -1 removed lines

@@ -46,6 +46,6 @@
 			'recherche' => _request('recherche'),
 			'lang' => $lang,
 			'class' => $class,
-			'_id_champ' => 'rechercher_' . substr(md5($action . $class), 0, 4),
+			'_id_champ' => 'rechercher_'.substr(md5($action.$class), 0, 4),
 		];
 }

prive/formulaires/editer_article.php

Indentation +101 added lines, -101 removed lines

@@ -17,7 +17,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/actions');
@@ -46,33 +46,33 @@ 
  *     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
-	);
-
-	if (intval($id_article) and !autoriser('modifier', 'article', intval($id_article))) {
-		$valeurs['editable'] = '';
-	}
-
-	// il faut enlever l'id_rubrique car la saisie se fait sur id_parent
-	// et id_rubrique peut etre passe dans l'url comme rubrique parent initiale
-	// et sera perdue si elle est supposee saisie
-	return $valeurs;
+    $valeurs = formulaires_editer_objet_charger(
+        'article',
+        $id_article,
+        $id_rubrique,
+        $lier_trad,
+        $retour,
+        $config_fonc,
+        $row,
+        $hidden
+    );
+
+    if (intval($id_article) and !autoriser('modifier', 'article', intval($id_article))) {
+        $valeurs['editable'] = '';
+    }
+
+    // il faut enlever l'id_rubrique car la saisie se fait sur id_parent
+    // et id_rubrique peut etre passe dans l'url comme rubrique parent initiale
+    // et sera perdue si elle est supposee saisie
+    return $valeurs;
 }
 
 /**
@@ -97,15 +97,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([intval($id_article), $lier_trad]);
+    return serialize([intval($id_article), $lier_trad]);
 }
 
 /**
@@ -117,15 +117,15 @@ 
  *     Configuration pour le formulaire
  */
 function articles_edit_config($row) {
-	global $spip_lang;
+    global $spip_lang;
 
-	$config = $GLOBALS['meta'];
-	$config['lignes'] = 8;
-	$config['langue'] = $spip_lang;
+    $config = $GLOBALS['meta'];
+    $config['lignes'] = 8;
+    $config['langue'] = $spip_lang;
 
-	$config['restreint'] = ($row['statut'] == 'publie');
+    $config['restreint'] = ($row['statut'] == 'publie');
 
-	return $config;
+    return $config;
 }
 
 /**
@@ -151,43 +151,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') and !intval($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'])
-		and !autoriser('creerarticledans', 'rubrique', _request('id_parent'))
-	) {
-		$erreurs['id_parent'] = _T('info_creerdansrubrique_non_autorise');
-	}
-
-	return $erreurs;
+    // auto-renseigner le titre si il n'existe pas
+    titre_automatique('titre', ['descriptif', 'chapo', 'texte']);
+    if (!_request('id_parent') and !intval($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'])
+        and !autoriser('creerarticledans', 'rubrique', _request('id_parent'))
+    ) {
+        $erreurs['id_parent'] = _T('info_creerdansrubrique_non_autorise');
+    }
+
+    return $erreurs;
 }
 
 /**
@@ -213,29 +213,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');
-	}
-
-	return formulaires_editer_objet_traiter(
-		'article',
-		$id_article,
-		$id_rubrique,
-		$lier_trad,
-		$retour,
-		$config_fonc,
-		$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');
+    }
+
+    return formulaires_editer_objet_traiter(
+        'article',
+        $id_article,
+        $id_rubrique,
+        $lier_trad,
+        $retour,
+        $config_fonc,
+        $row,
+        $hidden
+    );
 }

prive/formulaires/configurer_previsualiseur.php

Indentation +11 added lines, -11 removed lines

@@ -11,28 +11,28 @@
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 include_spip('inc/presentation');
 
 function formulaires_configurer_previsualiseur_charger_dist() {
-	$valeurs = [];
-	$valeurs['preview'] = explode(',', $GLOBALS['meta']['preview']);
+    $valeurs = [];
+    $valeurs['preview'] = explode(',', $GLOBALS['meta']['preview']);
 
-	return $valeurs;
+    return $valeurs;
 }
 
 
 function formulaires_configurer_previsualiseur_traiter_dist() {
-	$res = ['editable' => true];
+    $res = ['editable' => true];
 
-	if ($i = _request('preview') and is_array($i)) {
-		$i = ',' . implode(',', $i) . ',';
-	}
+    if ($i = _request('preview') and 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;
 }

Spacing +1 added lines, -1 removed lines

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

prive/formulaires/rediriger_article.php

Indentation +45 added lines, -45 removed lines

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

Spacing +3 added lines, -3 removed lines

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

prive/formulaires/instituer_objet.php

Indentation +108 added lines, -108 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, ['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 = [
-		'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, ['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 = [
+        '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,35 +131,35 @@ 
  *     Tableau des erreurs
  */
 function formulaires_instituer_objet_verifier_dist($objet, $id_objet, $retour = '', $editable = true) {
-	$erreurs = [];
-	// 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, '', ['statut' => $statut])
-		) {
-			$erreurs['statut'] = _T('instituer_erreur_statut_non_autorise');
-		}
-	}
-
-	return $erreurs;
+    $erreurs = [];
+    // 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, '', ['statut' => $statut])
+        ) {
+            $erreurs['statut'] = _T('instituer_erreur_statut_non_autorise');
+        }
+    }
+
+    return $erreurs;
 }
 
 /**
@@ -178,25 +178,25 @@ 
  */
 function formulaires_instituer_objet_traiter_dist($objet, $id_objet, $retour = '', $editable = true) {
 
-	$c = ['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 = ['message_erreur' => $err];
-	} else {
-		$res = ['message_ok' => _T('info_modification_enregistree')];
-		if ($retour) {
-			$res['redirect'] = $retour;
-		}
-		set_request('statut');
-		set_request('date_posterieure');
-	}
-
-	return $res;
+    $c = ['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 = ['message_erreur' => $err];
+    } else {
+        $res = ['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']))