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/configurer_avertisseur.php

Indentation +25 added lines, -25 removed lines

@@ -11,37 +11,37 @@
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 include_spip('inc/presentation');
 
 function formulaires_configurer_avertisseur_charger_dist() {
-	$valeurs = [];
-	foreach (
-		[
-		'articles_modif',
-		] as $m
-	) {
-		$valeurs[$m] = $GLOBALS['meta'][$m];
-	}
-
-	return $valeurs;
+    $valeurs = [];
+    foreach (
+        [
+        'articles_modif',
+        ] as $m
+    ) {
+        $valeurs[$m] = $GLOBALS['meta'][$m];
+    }
+
+    return $valeurs;
 }
 
 
 function formulaires_configurer_avertisseur_traiter_dist() {
-	$res = ['editable' => true];
-	foreach (
-		[
-		'articles_modif',
-		] as $m
-	) {
-		if (!is_null($v = _request($m))) {
-			ecrire_meta($m, $v == 'oui' ? 'oui' : 'non');
-		}
-	}
-
-	$res['message_ok'] = _T('config_info_enregistree');
-
-	return $res;
+    $res = ['editable' => true];
+    foreach (
+        [
+        'articles_modif',
+        ] as $m
+    ) {
+        if (!is_null($v = _request($m))) {
+            ecrire_meta($m, $v == 'oui' ? 'oui' : 'non');
+        }
+    }
+
+    $res['message_ok'] = _T('config_info_enregistree');
+
+    return $res;
 }

prive/formulaires/configurer_visiteurs.php

Indentation +39 added lines, -39 removed lines

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

prive/formulaires/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/configurer_metas.php

Indentation +63 added lines, -63 removed lines

@@ -11,68 +11,68 @@ 
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 function formulaires_configurer_metas_charger_dist($form) {
-	$f = charger_fonction('charger', "formulaires/$form", true);
-	if ($f) {
-		return $f($form);
-	} else {
-		$infos = formulaires_configurer_metas_infos($form);
-		if (!is_array($infos)) {
-			return $infos;
-		}
+    $f = charger_fonction('charger', "formulaires/$form", true);
+    if ($f) {
+        return $f($form);
+    } else {
+        $infos = formulaires_configurer_metas_infos($form);
+        if (!is_array($infos)) {
+            return $infos;
+        }
 
-		return $GLOBALS[$infos['meta']];
-	}
+        return $GLOBALS[$infos['meta']];
+    }
 }
 
 function formulaires_configurer_metas_verifier_dist($form) {
-	$f = charger_fonction('verifier', "formulaires/$form", true);
+    $f = charger_fonction('verifier', "formulaires/$form", true);
 
-	return $f ? $f($form) : [];
+    return $f ? $f($form) : [];
 }
 
 function formulaires_configurer_metas_traiter_dist($form) {
-	$f = charger_fonction('traiter', "formulaires/$form", true);
-	if ($f) {
-		return $f($form);
-	} else {
-		$infos = formulaires_configurer_metas_infos($form);
-		if (!is_array($infos)) {
-			return $infos;
-		} // fait ci-dessus en fait
-		$vars = formulaires_configurer_metas_recense($infos['path'], PREG_PATTERN_ORDER);
-		$meta = $infos['meta'];
-		foreach (array_unique($vars[2]) as $k) {
-			$v = _request($k);
-			ecrire_meta($k, is_array($v) ? serialise($v) : $v, 'oui', $meta);
-		}
+    $f = charger_fonction('traiter', "formulaires/$form", true);
+    if ($f) {
+        return $f($form);
+    } else {
+        $infos = formulaires_configurer_metas_infos($form);
+        if (!is_array($infos)) {
+            return $infos;
+        } // fait ci-dessus en fait
+        $vars = formulaires_configurer_metas_recense($infos['path'], PREG_PATTERN_ORDER);
+        $meta = $infos['meta'];
+        foreach (array_unique($vars[2]) as $k) {
+            $v = _request($k);
+            ecrire_meta($k, is_array($v) ? serialise($v) : $v, 'oui', $meta);
+        }
 
-		return !isset($infos['prefix']) ? []
-			: ['redirect' => generer_url_ecrire($infos['prefix'])];
-	}
+        return !isset($infos['prefix']) ? []
+            : ['redirect' => generer_url_ecrire($infos['prefix'])];
+    }
 }
 
 // version amelioree de la RegExp de cfg_formulaire.
 define(
-	'_EXTRAIRE_SAISIES',
-	'#<(select|textarea|input)[^>]*\sname=["\'](\w+)(\[\w*\])?["\'](?: class=["\']([^\'"]*)["\'])?( multiple=)?[^>]*?>#ims'
+    '_EXTRAIRE_SAISIES',
+    '#<(select|textarea|input)[^>]*\sname=["\'](\w+)(\[\w*\])?["\'](?: class=["\']([^\'"]*)["\'])?( multiple=)?[^>]*?>#ims'
 );
 
 // determiner la liste des noms des saisies d'un formulaire
 // (a refaire avec SAX)
 function formulaires_configurer_metas_recense($form, $opt = '') {
-	if (!$opt) {
-		$opt = PREG_SET_ORDER;
-	}
-	$f = file_get_contents($form);
-	if ($f and preg_match_all(_EXTRAIRE_SAISIES, $f, $r, $opt)) {
-		return $r;
-	} else {
-		return [];
-	}
+    if (!$opt) {
+        $opt = PREG_SET_ORDER;
+    }
+    $f = file_get_contents($form);
+    if ($f and preg_match_all(_EXTRAIRE_SAISIES, $f, $r, $opt)) {
+        return $r;
+    } else {
+        return [];
+    }
 }
 
 // Repertoires potentiels des plugins, ce serait bien d'avoir ça ailleurs
@@ -86,27 +86,27 @@ 
 // Si ce n'est pas un plugin, dire qu'il faut prendre la table std des meta.
 function formulaires_configurer_metas_infos($form) {
 
-	$path = find_in_path($form . '.' . _EXTENSION_SQUELETTES, 'formulaires/');
-	if (!$path) {
-		return '';
-	} // cas traite en amont normalement.
-	if (!preg_match(_EXTRAIRE_PLUGIN, $path, $m)) {
-		return ['path' => $path, 'meta' => 'meta'];
-	}
-	$plugin = $m[2];
-	$get_infos = charger_fonction('get_infos', 'plugins');
-	$infos = $get_infos($plugin, false, $m[1]);
-	if (!is_array($infos)) {
-		return _T('erreur_plugin_nom_manquant');
-	}
-	if (isset($infos['erreur'])) {
-		return $infos['erreur'][0];
-	}
-	$prefix = $infos['prefix'];
-	$infos['path'] = $path;
-	if (!isset($infos['meta'])) {
-		$infos['meta'] = ($prefix . '_metas');
-	}
+    $path = find_in_path($form . '.' . _EXTENSION_SQUELETTES, 'formulaires/');
+    if (!$path) {
+        return '';
+    } // cas traite en amont normalement.
+    if (!preg_match(_EXTRAIRE_PLUGIN, $path, $m)) {
+        return ['path' => $path, 'meta' => 'meta'];
+    }
+    $plugin = $m[2];
+    $get_infos = charger_fonction('get_infos', 'plugins');
+    $infos = $get_infos($plugin, false, $m[1]);
+    if (!is_array($infos)) {
+        return _T('erreur_plugin_nom_manquant');
+    }
+    if (isset($infos['erreur'])) {
+        return $infos['erreur'][0];
+    }
+    $prefix = $infos['prefix'];
+    $infos['path'] = $path;
+    if (!isset($infos['meta'])) {
+        $infos['meta'] = ($prefix . '_metas');
+    }
 
-	return $infos;
+    return $infos;
 }

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/configurer_articles.php

Indentation +49 added lines, -49 removed lines

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