spip / SPIP

ecrire/inc/bandeau.php

Indentation +149 added lines, -149 removed lines

@@ -18,7 +18,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/boutons');
@@ -38,33 +38,33 @@ 
  *     contexte
  **/
 function definir_barre_contexte($contexte = null) {
-	if (is_null($contexte)) {
-		$contexte = $_GET;
-	} elseif (is_string($contexte)) {
-		$contexte = unserialize($contexte);
-	}
-	if (!isset($contexte['id_rubrique']) && isset($contexte['exec'])) {
-		if (!function_exists('trouver_objet_exec')) {
-			include_spip('inc/pipelines_ecrire');
-		}
-		if ($e = trouver_objet_exec($contexte['exec'])) {
-			$_id = $e['id_table_objet'];
-			if (isset($contexte[$_id]) && ($id = (int) $contexte[$_id])) {
-				$table = $e['table_objet_sql'];
-				$row = sql_fetsel('*', $table, "$_id=" . (int) $id);
-				if (isset($row['id_rubrique'])) {
-					$contexte['id_rubrique'] = $row['id_rubrique'];
-					if (isset($row['id_secteur'])) {
-						$contexte['id_secteur'] = $row['id_secteur'];
-					}
-				} elseif (isset($row['id_groupe'])) {
-					// TODO supprimer ce bloc quand https://core.spip.net/issues/3844 sera réalisé
-					$contexte['id_groupe'] = $row['id_groupe'];
-				}
-			}
-		}
-	}
-	return $contexte;
+    if (is_null($contexte)) {
+        $contexte = $_GET;
+    } elseif (is_string($contexte)) {
+        $contexte = unserialize($contexte);
+    }
+    if (!isset($contexte['id_rubrique']) && isset($contexte['exec'])) {
+        if (!function_exists('trouver_objet_exec')) {
+            include_spip('inc/pipelines_ecrire');
+        }
+        if ($e = trouver_objet_exec($contexte['exec'])) {
+            $_id = $e['id_table_objet'];
+            if (isset($contexte[$_id]) && ($id = (int) $contexte[$_id])) {
+                $table = $e['table_objet_sql'];
+                $row = sql_fetsel('*', $table, "$_id=" . (int) $id);
+                if (isset($row['id_rubrique'])) {
+                    $contexte['id_rubrique'] = $row['id_rubrique'];
+                    if (isset($row['id_secteur'])) {
+                        $contexte['id_secteur'] = $row['id_secteur'];
+                    }
+                } elseif (isset($row['id_groupe'])) {
+                    // TODO supprimer ce bloc quand https://core.spip.net/issues/3844 sera réalisé
+                    $contexte['id_groupe'] = $row['id_groupe'];
+                }
+            }
+        }
+    }
+    return $contexte;
 }
 
 /**
@@ -80,89 +80,89 @@ 
  * @return array
  */
 function definir_barre_boutons($contexte = [], $icones = true, $autorise = true) {
-	include_spip('inc/autoriser');
-	$boutons_admin = [];
+    include_spip('inc/autoriser');
+    $boutons_admin = [];
 
-	// les boutons du core, issus de ecrire/paquet.xml
-	$liste_boutons = [];
+    // les boutons du core, issus de ecrire/paquet.xml
+    $liste_boutons = [];
 
-	// ajouter les boutons issus des plugin via paquet.xml
-	if (
-		function_exists('boutons_plugins')
-		&& is_array($liste_boutons_plugins = boutons_plugins())
-	) {
-		$liste_boutons = &$liste_boutons_plugins;
-	}
+    // ajouter les boutons issus des plugin via paquet.xml
+    if (
+        function_exists('boutons_plugins')
+        && is_array($liste_boutons_plugins = boutons_plugins())
+    ) {
+        $liste_boutons = &$liste_boutons_plugins;
+    }
 
-	foreach ($liste_boutons as $id => $infos) {
-		$parent = '';
-		// les boutons principaux ne sont pas soumis a autorisation
-		if (
-			!isset($infos['parent'])
-			|| !($parent = $infos['parent'])
-			|| !$autorise
-			|| autoriser('menu', "_$id", 0, null, ['contexte' => $contexte])
-		) {
-			if (
-				$parent
-				&& ($parent = preg_replace(',^bando_,', 'menu_', $parent))
-				&& isset($boutons_admin[$parent])
-			) {
-				$position = (isset($infos['position']) && strlen($infos['position'])) ? (int) $infos['position'] : count($boutons_admin[$parent]->sousmenu);
-				if ($position < 0) {
-					$position = count($boutons_admin[$parent]->sousmenu) + 1 + $position;
-				}
-				$boutons_admin[$parent]->sousmenu = array_slice($boutons_admin[$parent]->sousmenu, 0, $position)
-					+ [
-						$id => new Bouton(
-							($icones && !empty($infos['icone'])) ? find_in_theme($infos['icone']) : '',  // icone
-							$infos['titre'],  // titre
-							(isset($infos['action']) && $infos['action']) ? $infos['action'] : null,
-							(isset($infos['parametres']) && $infos['parametres']) ? $infos['parametres'] : null
-						)
-					]
-					+ array_slice($boutons_admin[$parent]->sousmenu, $position, 100);
-			}
-			if (
-				!$parent
-				// provisoire, eviter les vieux boutons
-				&& !in_array($id, ['forum', 'statistiques_visites'])
-				&& (!$autorise || autoriser('menugrandeentree', "_$id", 0, null, ['contexte' => $contexte]))
-			) {
-				$position = (isset($infos['position']) && $infos['position']) ? $infos['position'] : count($boutons_admin);
-				$boutons_admin = array_slice($boutons_admin, 0, $position)
-					+ [
-						$id => new Bouton(
-							($icones && isset($infos['icone']) && $infos['icone']) ? find_in_theme($infos['icone']) : '',  // icone
-							$infos['titre'],  // titre
-							(isset($infos['action']) && $infos['action']) ? $infos['action'] : null,
-							(isset($infos['parametres']) && $infos['parametres']) ? $infos['parametres'] : null
-						)
-					]
-					+ array_slice($boutons_admin, $position, 100);
-			}
-		}
-	}
-	$boutons_admin = pipeline('ajouter_menus', $boutons_admin);
+    foreach ($liste_boutons as $id => $infos) {
+        $parent = '';
+        // les boutons principaux ne sont pas soumis a autorisation
+        if (
+            !isset($infos['parent'])
+            || !($parent = $infos['parent'])
+            || !$autorise
+            || autoriser('menu', "_$id", 0, null, ['contexte' => $contexte])
+        ) {
+            if (
+                $parent
+                && ($parent = preg_replace(',^bando_,', 'menu_', $parent))
+                && isset($boutons_admin[$parent])
+            ) {
+                $position = (isset($infos['position']) && strlen($infos['position'])) ? (int) $infos['position'] : count($boutons_admin[$parent]->sousmenu);
+                if ($position < 0) {
+                    $position = count($boutons_admin[$parent]->sousmenu) + 1 + $position;
+                }
+                $boutons_admin[$parent]->sousmenu = array_slice($boutons_admin[$parent]->sousmenu, 0, $position)
+                    + [
+                        $id => new Bouton(
+                            ($icones && !empty($infos['icone'])) ? find_in_theme($infos['icone']) : '',  // icone
+                            $infos['titre'],  // titre
+                            (isset($infos['action']) && $infos['action']) ? $infos['action'] : null,
+                            (isset($infos['parametres']) && $infos['parametres']) ? $infos['parametres'] : null
+                        )
+                    ]
+                    + array_slice($boutons_admin[$parent]->sousmenu, $position, 100);
+            }
+            if (
+                !$parent
+                // provisoire, eviter les vieux boutons
+                && !in_array($id, ['forum', 'statistiques_visites'])
+                && (!$autorise || autoriser('menugrandeentree', "_$id", 0, null, ['contexte' => $contexte]))
+            ) {
+                $position = (isset($infos['position']) && $infos['position']) ? $infos['position'] : count($boutons_admin);
+                $boutons_admin = array_slice($boutons_admin, 0, $position)
+                    + [
+                        $id => new Bouton(
+                            ($icones && isset($infos['icone']) && $infos['icone']) ? find_in_theme($infos['icone']) : '',  // icone
+                            $infos['titre'],  // titre
+                            (isset($infos['action']) && $infos['action']) ? $infos['action'] : null,
+                            (isset($infos['parametres']) && $infos['parametres']) ? $infos['parametres'] : null
+                        )
+                    ]
+                    + array_slice($boutons_admin, $position, 100);
+            }
+        }
+    }
+    $boutons_admin = pipeline('ajouter_menus', $boutons_admin);
 
-	// définir les favoris et positions d’origine
-	if ($boutons_admin) {
-		$menus_favoris = obtenir_menus_favoris();
-		$i = 1;
-		foreach ($boutons_admin as $key => $menu) {
-			$menu->favori = (int) table_valeur($menus_favoris, $key, false);
-			$menu->position = $i++;
-			if ($menu->sousmenu) {
-				$j = 1;
-				foreach ($menu->sousmenu as $key => $bouton) {
-					$bouton->favori = (int) table_valeur($menus_favoris, $key, false);
-					$bouton->position = $j++;
-				}
-			}
-		}
-	}
+    // définir les favoris et positions d’origine
+    if ($boutons_admin) {
+        $menus_favoris = obtenir_menus_favoris();
+        $i = 1;
+        foreach ($boutons_admin as $key => $menu) {
+            $menu->favori = (int) table_valeur($menus_favoris, $key, false);
+            $menu->position = $i++;
+            if ($menu->sousmenu) {
+                $j = 1;
+                foreach ($menu->sousmenu as $key => $bouton) {
+                    $bouton->favori = (int) table_valeur($menus_favoris, $key, false);
+                    $bouton->position = $j++;
+                }
+            }
+        }
+    }
 
-	return $boutons_admin;
+    return $boutons_admin;
 }
 
 /**
@@ -174,22 +174,22 @@ 
  * @return Bouton[]
  */
 function trier_boutons_enfants_par_alpha($menus, $avec_favoris = false) {
-	foreach ($menus as $menu) {
-		if ($menu->sousmenu) {
-			$libelles = $isfavoris = $favoris = [];
-			foreach ($menu->sousmenu as $key => $item) {
-				$libelles[$key] = strtolower(translitteration(_T($item->libelle)));
-				$isfavoris[$key] = (bool) $item->favori;
-				$favoris[$key] = $item->favori;
-			}
-			if ($avec_favoris) {
-				array_multisort($isfavoris, SORT_DESC, $favoris, SORT_ASC, $libelles, SORT_ASC, $menu->sousmenu);
-			} else {
-				array_multisort($libelles, SORT_ASC, $menu->sousmenu);
-			}
-		}
-	}
-	return $menus;
+    foreach ($menus as $menu) {
+        if ($menu->sousmenu) {
+            $libelles = $isfavoris = $favoris = [];
+            foreach ($menu->sousmenu as $key => $item) {
+                $libelles[$key] = strtolower(translitteration(_T($item->libelle)));
+                $isfavoris[$key] = (bool) $item->favori;
+                $favoris[$key] = $item->favori;
+            }
+            if ($avec_favoris) {
+                array_multisort($isfavoris, SORT_DESC, $favoris, SORT_ASC, $libelles, SORT_ASC, $menu->sousmenu);
+            } else {
+                array_multisort($libelles, SORT_ASC, $menu->sousmenu);
+            }
+        }
+    }
+    return $menus;
 }
 
 /**
@@ -200,7 +200,7 @@ 
  * @return Bouton[]
  */
 function trier_boutons_enfants_par_favoris_alpha($menus) {
-	return trier_boutons_enfants_par_alpha($menus, true);
+    return trier_boutons_enfants_par_alpha($menus, true);
 }
 
 
@@ -213,23 +213,23 @@ 
  * @return string
  */
 function bandeau_creer_url($url, $args = '', $contexte = null) {
-	if (!preg_match(',[\/\?],', $url)) {
-		$url = generer_url_ecrire($url, $args, true);
-		// recuperer les parametres du contexte demande par l'url sous la forme
-		// &truc=@machin@
-		// @machin@ etant remplace par _request('machin')
-		$url = str_replace('&amp;', '&', $url);
-		while (preg_match(',[&?]([a-z_]+)=@([a-z_]+)@,i', $url, $matches)) {
-			if ($matches[2] == 'id_secteur' && !isset($contexte['id_secteur']) && isset($contexte['id_rubrique'])) {
-				$contexte['id_secteur'] = sql_getfetsel('id_secteur', 'spip_rubriques', 'id_rubrique=' . (int) $contexte['id_rubrique']);
-			}
-			$val = _request($matches[2], $contexte);
-			$url = parametre_url($url, $matches[1], $val ?: '', '&');
-		}
-		$url = str_replace('&', '&amp;', $url);
-	}
+    if (!preg_match(',[\/\?],', $url)) {
+        $url = generer_url_ecrire($url, $args, true);
+        // recuperer les parametres du contexte demande par l'url sous la forme
+        // &truc=@machin@
+        // @machin@ etant remplace par _request('machin')
+        $url = str_replace('&amp;', '&', $url);
+        while (preg_match(',[&?]([a-z_]+)=@([a-z_]+)@,i', $url, $matches)) {
+            if ($matches[2] == 'id_secteur' && !isset($contexte['id_secteur']) && isset($contexte['id_rubrique'])) {
+                $contexte['id_secteur'] = sql_getfetsel('id_secteur', 'spip_rubriques', 'id_rubrique=' . (int) $contexte['id_rubrique']);
+            }
+            $val = _request($matches[2], $contexte);
+            $url = parametre_url($url, $matches[1], $val ?: '', '&');
+        }
+        $url = str_replace('&', '&amp;', $url);
+    }
 
-	return $url;
+    return $url;
 }
 
 /**
@@ -239,7 +239,7 @@ 
  *     Code HTML du bandeau
  */
 function inc_bandeau_dist() {
-	return recuperer_fond('prive/squelettes/inclure/barre-nav', $_GET);
+    return recuperer_fond('prive/squelettes/inclure/barre-nav', $_GET);
 }
 
 
@@ -248,13 +248,13 @@ 
  * @return array
  */
 function obtenir_menus_favoris() {
-	if (
-		isset($GLOBALS['visiteur_session']['prefs']['menus_favoris'])
-		&& is_array($GLOBALS['visiteur_session']['prefs']['menus_favoris'])
-		&& $GLOBALS['visiteur_session']['prefs']['menus_favoris']
-	) {
-		return $GLOBALS['visiteur_session']['prefs']['menus_favoris'];
-	}
-	$definir_menus_favoris = charger_fonction('definir_menus_favoris', 'inc');
-	return $definir_menus_favoris();
+    if (
+        isset($GLOBALS['visiteur_session']['prefs']['menus_favoris'])
+        && is_array($GLOBALS['visiteur_session']['prefs']['menus_favoris'])
+        && $GLOBALS['visiteur_session']['prefs']['menus_favoris']
+    ) {
+        return $GLOBALS['visiteur_session']['prefs']['menus_favoris'];
+    }
+    $definir_menus_favoris = charger_fonction('definir_menus_favoris', 'inc');
+    return $definir_menus_favoris();
 }

Spacing +6 added lines, -6 removed lines

@@ -51,7 +51,7 @@ 
 			$_id = $e['id_table_objet'];
 			if (isset($contexte[$_id]) && ($id = (int) $contexte[$_id])) {
 				$table = $e['table_objet_sql'];
-				$row = sql_fetsel('*', $table, "$_id=" . (int) $id);
+				$row = sql_fetsel('*', $table, "$_id=".(int) $id);
 				if (isset($row['id_rubrique'])) {
 					$contexte['id_rubrique'] = $row['id_rubrique'];
 					if (isset($row['id_secteur'])) {
@@ -115,8 +115,8 @@ 
 				$boutons_admin[$parent]->sousmenu = array_slice($boutons_admin[$parent]->sousmenu, 0, $position)
 					+ [
 						$id => new Bouton(
-							($icones && !empty($infos['icone'])) ? find_in_theme($infos['icone']) : '',  // icone
-							$infos['titre'],  // titre
+							($icones && !empty($infos['icone'])) ? find_in_theme($infos['icone']) : '', // icone
+							$infos['titre'], // titre
 							(isset($infos['action']) && $infos['action']) ? $infos['action'] : null,
 							(isset($infos['parametres']) && $infos['parametres']) ? $infos['parametres'] : null
 						)
@@ -133,8 +133,8 @@ 
 				$boutons_admin = array_slice($boutons_admin, 0, $position)
 					+ [
 						$id => new Bouton(
-							($icones && isset($infos['icone']) && $infos['icone']) ? find_in_theme($infos['icone']) : '',  // icone
-							$infos['titre'],  // titre
+							($icones && isset($infos['icone']) && $infos['icone']) ? find_in_theme($infos['icone']) : '', // icone
+							$infos['titre'], // titre
 							(isset($infos['action']) && $infos['action']) ? $infos['action'] : null,
 							(isset($infos['parametres']) && $infos['parametres']) ? $infos['parametres'] : null
 						)
@@ -221,7 +221,7 @@ 
 		$url = str_replace('&', '&', $url);
 		while (preg_match(',[&?]([a-z_]+)=@([a-z_]+)@,i', $url, $matches)) {
 			if ($matches[2] == 'id_secteur' && !isset($contexte['id_secteur']) && isset($contexte['id_rubrique'])) {
-				$contexte['id_secteur'] = sql_getfetsel('id_secteur', 'spip_rubriques', 'id_rubrique=' . (int) $contexte['id_rubrique']);
+				$contexte['id_secteur'] = sql_getfetsel('id_secteur', 'spip_rubriques', 'id_rubrique='.(int) $contexte['id_rubrique']);
 			}
 			$val = _request($matches[2], $contexte);
 			$url = parametre_url($url, $matches[1], $val ?: '', '&');

ecrire/inc/cvt_autosave.php

Indentation +73 added lines, -73 removed lines

@@ -16,7 +16,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -26,7 +26,7 @@ 
  * @return string
  */
 function autosave_clean_value($val) {
-	return stripslashes(urldecode($val));
+    return stripslashes(urldecode($val));
 }
 
 /**
@@ -37,58 +37,58 @@ 
  * @return array
  */
 function cvtautosave_formulaire_charger($flux) {
-	if (
-		is_array($flux['data'])
-		&& isset($flux['data']['_autosave_id'])
-		&& ($cle_autosave = $flux['data']['_autosave_id'])
-	) {
-		$form = $flux['args']['form'];
-		$je_suis_poste = $flux['args']['je_suis_poste'];
+    if (
+        is_array($flux['data'])
+        && isset($flux['data']['_autosave_id'])
+        && ($cle_autosave = $flux['data']['_autosave_id'])
+    ) {
+        $form = $flux['args']['form'];
+        $je_suis_poste = $flux['args']['je_suis_poste'];
 
-		$cle_autosave = serialize($cle_autosave);
-		$cle_autosave = $form . '_' . md5($cle_autosave);
+        $cle_autosave = serialize($cle_autosave);
+        $cle_autosave = $form . '_' . md5($cle_autosave);
 
-		// si on a un backup en session et qu'on est au premier chargement, non poste
-		// on restitue les donnees
-		if (
-			isset($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave])
-			&& !$je_suis_poste
-		) {
-			parse_str($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave], $vars);
-			foreach ($vars as $key => $val) {
-				if (isset($flux['data'][$key])) {
-					$flux['data'][$key] = (is_string($val) ? autosave_clean_value($val) : array_map(
-						'autosave_clean_value',
-						$val
-					));
-				}
-			}
-		}
+        // si on a un backup en session et qu'on est au premier chargement, non poste
+        // on restitue les donnees
+        if (
+            isset($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave])
+            && !$je_suis_poste
+        ) {
+            parse_str($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave], $vars);
+            foreach ($vars as $key => $val) {
+                if (isset($flux['data'][$key])) {
+                    $flux['data'][$key] = (is_string($val) ? autosave_clean_value($val) : array_map(
+                        'autosave_clean_value',
+                        $val
+                    ));
+                }
+            }
+        }
 
-		// si on est dans le charger() qui suit le traiter(), l'autosave a normalement ete vide
-		// mais si il y a plusieurs sessions il peut y avoir concurrence et un retour de l'autosave
-		if ($je_suis_poste && _request('autosave') === $cle_autosave && function_exists('terminer_actualiser_sessions')) {
-			terminer_actualiser_sessions();
-			// et verifions si jamais l'autosave a fait un come back, dans ce cas on le revide
-			if (isset($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave])) {
-				session_set('session_autosave_' . $cle_autosave, null);
-				// en court sleep pour etre certain que la concurrence est finie
-				sleep(1);
-				terminer_actualiser_sessions();
-			}
-		}
+        // si on est dans le charger() qui suit le traiter(), l'autosave a normalement ete vide
+        // mais si il y a plusieurs sessions il peut y avoir concurrence et un retour de l'autosave
+        if ($je_suis_poste && _request('autosave') === $cle_autosave && function_exists('terminer_actualiser_sessions')) {
+            terminer_actualiser_sessions();
+            // et verifions si jamais l'autosave a fait un come back, dans ce cas on le revide
+            if (isset($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave])) {
+                session_set('session_autosave_' . $cle_autosave, null);
+                // en court sleep pour etre certain que la concurrence est finie
+                sleep(1);
+                terminer_actualiser_sessions();
+            }
+        }
 
 
-		/**
-		 * Envoyer le input hidden et le bout de js qui l'utilisera
-		 */
-		$flux['data']['_hidden'] .= "<input type='hidden' name='autosave' class='autosaveactive' value='$cle_autosave' />"
-			. '<script type="text/javascript">/*<![CDATA[*/if (window.jQuery) jQuery(function(){
+        /**
+         * Envoyer le input hidden et le bout de js qui l'utilisera
+         */
+        $flux['data']['_hidden'] .= "<input type='hidden' name='autosave' class='autosaveactive' value='$cle_autosave' />"
+            . '<script type="text/javascript">/*<![CDATA[*/if (window.jQuery) jQuery(function(){
 		  $("input.autosaveactive").closest("form:not(.autosaveon)").autosave({url:"' . $GLOBALS['meta']['adresse_site'] . '/"}).addClass("autosaveon");
 			});/*]]>*/</script>';
-	}
+    }
 
-	return $flux;
+    return $flux;
 }
 
 /**
@@ -102,33 +102,33 @@ 
  * @return array
  */
 function cvtautosave_formulaire_traiter($flux) {
-	// si on poste 'autosave' c'est qu'on n'a plus besoin de sauvegarder :
-	// on elimine les donnees de la session
-	if ($cle_autosave = _request('autosave')) {
-		include_spip('inc/session');
-		session_set('session_autosave_' . $cle_autosave, null);
-	}
+    // si on poste 'autosave' c'est qu'on n'a plus besoin de sauvegarder :
+    // on elimine les donnees de la session
+    if ($cle_autosave = _request('autosave')) {
+        include_spip('inc/session');
+        session_set('session_autosave_' . $cle_autosave, null);
+    }
 
-	if (isset($GLOBALS['visiteur_session']) && $GLOBALS['visiteur_session']) {
-		// delai par defaut avant purge d'un backup de form : 72H
-		if (!defined('_AUTOSAVE_GB_DELAY')) {
-			define('_AUTOSAVE_GB_DELAY', 72 * 3600);
-		}
-		$time_too_old = time() - _AUTOSAVE_GB_DELAY;
-		// purger aussi toutes les vieilles autosave
-		$session = $GLOBALS['visiteur_session'];
-		foreach ($session as $k => $v) {
-			if (str_starts_with($k, 'session_autosave_')) {
-				$timestamp = 0;
-				if (preg_match(',&__timestamp=(\d+)$,', $v, $m)) {
-					$timestamp = (int) $m[1];
-				}
-				if ($timestamp < $time_too_old) {
-					session_set($k, null);
-				}
-			}
-		}
-	}
+    if (isset($GLOBALS['visiteur_session']) && $GLOBALS['visiteur_session']) {
+        // delai par defaut avant purge d'un backup de form : 72H
+        if (!defined('_AUTOSAVE_GB_DELAY')) {
+            define('_AUTOSAVE_GB_DELAY', 72 * 3600);
+        }
+        $time_too_old = time() - _AUTOSAVE_GB_DELAY;
+        // purger aussi toutes les vieilles autosave
+        $session = $GLOBALS['visiteur_session'];
+        foreach ($session as $k => $v) {
+            if (str_starts_with($k, 'session_autosave_')) {
+                $timestamp = 0;
+                if (preg_match(',&__timestamp=(\d+)$,', $v, $m)) {
+                    $timestamp = (int) $m[1];
+                }
+                if ($timestamp < $time_too_old) {
+                    session_set($k, null);
+                }
+            }
+        }
+    }
 
-	return $flux;
+    return $flux;
 }

Spacing +7 added lines, -7 removed lines

@@ -46,15 +46,15 @@ 
 		$je_suis_poste = $flux['args']['je_suis_poste'];
 
 		$cle_autosave = serialize($cle_autosave);
-		$cle_autosave = $form . '_' . md5($cle_autosave);
+		$cle_autosave = $form.'_'.md5($cle_autosave);
 
 		// si on a un backup en session et qu'on est au premier chargement, non poste
 		// on restitue les donnees
 		if (
-			isset($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave])
+			isset($GLOBALS['visiteur_session']['session_autosave_'.$cle_autosave])
 			&& !$je_suis_poste
 		) {
-			parse_str($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave], $vars);
+			parse_str($GLOBALS['visiteur_session']['session_autosave_'.$cle_autosave], $vars);
 			foreach ($vars as $key => $val) {
 				if (isset($flux['data'][$key])) {
 					$flux['data'][$key] = (is_string($val) ? autosave_clean_value($val) : array_map(
@@ -70,8 +70,8 @@ 
 		if ($je_suis_poste && _request('autosave') === $cle_autosave && function_exists('terminer_actualiser_sessions')) {
 			terminer_actualiser_sessions();
 			// et verifions si jamais l'autosave a fait un come back, dans ce cas on le revide
-			if (isset($GLOBALS['visiteur_session']['session_autosave_' . $cle_autosave])) {
-				session_set('session_autosave_' . $cle_autosave, null);
+			if (isset($GLOBALS['visiteur_session']['session_autosave_'.$cle_autosave])) {
+				session_set('session_autosave_'.$cle_autosave, null);
 				// en court sleep pour etre certain que la concurrence est finie
 				sleep(1);
 				terminer_actualiser_sessions();
@@ -84,7 +84,7 @@ 
 		 */
 		$flux['data']['_hidden'] .= "<input type='hidden' name='autosave' class='autosaveactive' value='$cle_autosave' />"
 			. '<script type="text/javascript">/*<![CDATA[*/if (window.jQuery) jQuery(function(){
-		  $("input.autosaveactive").closest("form:not(.autosaveon)").autosave({url:"' . $GLOBALS['meta']['adresse_site'] . '/"}).addClass("autosaveon");
+		  $("input.autosaveactive").closest("form:not(.autosaveon)").autosave({url:"' . $GLOBALS['meta']['adresse_site'].'/"}).addClass("autosaveon");
 			});/*]]>*/</script>';
 	}
 
@@ -106,7 +106,7 @@ 
 	// on elimine les donnees de la session
 	if ($cle_autosave = _request('autosave')) {
 		include_spip('inc/session');
-		session_set('session_autosave_' . $cle_autosave, null);
+		session_set('session_autosave_'.$cle_autosave, null);
 	}
 
 	if (isset($GLOBALS['visiteur_session']) && $GLOBALS['visiteur_session']) {

ecrire/inc/cookie.php

Indentation +93 added lines, -93 removed lines

@@ -16,7 +16,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 
@@ -51,64 +51,64 @@ 
  *     cookie sécurisé ou non ?
  **/
 function spip_setcookie($name = '', $value = '', $options = []) {
-	static $to_secure_list = ['spip_session'];
-	if (defined('_COOKIE_SECURE_LIST') && is_array(_COOKIE_SECURE_LIST)) {
-		$to_secure_list = array_merge($to_secure_list, _COOKIE_SECURE_LIST);
-	}
-
-	if (!is_array($options)) {
-		// anciens paramètres :
-		# spip_setcookie($name = '', $value = '', $expire = 0, $path = 'AUTO', $domain = '', $secure = '')
-		$opt = func_get_args();
-		$opt = array_slice($opt, 2);
-		$options = []; # /!\ après le func_get_args (sinon $opt[0] référence la nouvelle valeur de $options !);
-		if (isset($opt[0])) {
-			$options['expires'] = $opt[0];
-		}
-		if (isset($opt[1])) {
-			$options['path'] = $opt[1];
-		}
-		if (isset($opt[2])) {
-			$options['domain'] = $opt[2];
-		}
-		if (isset($opt[3])) {
-			$options['secure'] = $opt[3];
-		}
-	}
-
-	// expires
-	if (!isset($options['expires'])) {
-		$options['expires'] = 0;
-	}
-	if (!isset($options['path']) || $options['path'] === 'AUTO') {
-		$options['path'] = defined('_COOKIE_PATH') ? _COOKIE_PATH : preg_replace(',^\w+://[^/]*,', '', url_de_base());
-	}
-	if (empty($options['domain']) && defined('_COOKIE_DOMAIN') && _COOKIE_DOMAIN) {
-		$options['domain'] = _COOKIE_DOMAIN;
-	}
-	if (in_array($name, $to_secure_list)) {
-		if (empty($options['secure']) && defined('_COOKIE_SECURE') && _COOKIE_SECURE) {
-			$options['secure'] = true;
-		}
-		if (empty($options['httponly'])) {
-			$options['httponly'] = true;
-		}
-	}
-	if (empty($options['samesite'])) {
-		$options['samesite'] = 'Lax';
-	}
-
-	// in fine renommer le prefixe si besoin
-	if (str_starts_with($name, 'spip_')) {
-		$name = $GLOBALS['cookie_prefix'] . '_' . substr($name, 5);
-	}
-
-	#spip_log("cookie('$name', '$value', " . json_encode($options, true) . ")", "cookies");
-	$a = @setcookie($name, $value, $options);
-
-	spip_cookie_envoye(true);
-
-	return $a;
+    static $to_secure_list = ['spip_session'];
+    if (defined('_COOKIE_SECURE_LIST') && is_array(_COOKIE_SECURE_LIST)) {
+        $to_secure_list = array_merge($to_secure_list, _COOKIE_SECURE_LIST);
+    }
+
+    if (!is_array($options)) {
+        // anciens paramètres :
+        # spip_setcookie($name = '', $value = '', $expire = 0, $path = 'AUTO', $domain = '', $secure = '')
+        $opt = func_get_args();
+        $opt = array_slice($opt, 2);
+        $options = []; # /!\ après le func_get_args (sinon $opt[0] référence la nouvelle valeur de $options !);
+        if (isset($opt[0])) {
+            $options['expires'] = $opt[0];
+        }
+        if (isset($opt[1])) {
+            $options['path'] = $opt[1];
+        }
+        if (isset($opt[2])) {
+            $options['domain'] = $opt[2];
+        }
+        if (isset($opt[3])) {
+            $options['secure'] = $opt[3];
+        }
+    }
+
+    // expires
+    if (!isset($options['expires'])) {
+        $options['expires'] = 0;
+    }
+    if (!isset($options['path']) || $options['path'] === 'AUTO') {
+        $options['path'] = defined('_COOKIE_PATH') ? _COOKIE_PATH : preg_replace(',^\w+://[^/]*,', '', url_de_base());
+    }
+    if (empty($options['domain']) && defined('_COOKIE_DOMAIN') && _COOKIE_DOMAIN) {
+        $options['domain'] = _COOKIE_DOMAIN;
+    }
+    if (in_array($name, $to_secure_list)) {
+        if (empty($options['secure']) && defined('_COOKIE_SECURE') && _COOKIE_SECURE) {
+            $options['secure'] = true;
+        }
+        if (empty($options['httponly'])) {
+            $options['httponly'] = true;
+        }
+    }
+    if (empty($options['samesite'])) {
+        $options['samesite'] = 'Lax';
+    }
+
+    // in fine renommer le prefixe si besoin
+    if (str_starts_with($name, 'spip_')) {
+        $name = $GLOBALS['cookie_prefix'] . '_' . substr($name, 5);
+    }
+
+    #spip_log("cookie('$name', '$value', " . json_encode($options, true) . ")", "cookies");
+    $a = @setcookie($name, $value, $options);
+
+    spip_cookie_envoye(true);
+
+    return $a;
 }
 
 /**
@@ -124,12 +124,12 @@ 
  * @return bool
  **/
 function spip_cookie_envoye($set = '') {
-	static $envoye = false;
-	if ($set) {
-		$envoye = true;
-	}
+    static $envoye = false;
+    if ($set) {
+        $envoye = true;
+    }
 
-	return $envoye;
+    return $envoye;
 }
 
 /**
@@ -148,21 +148,21 @@ 
  *     Préfixe des cookies de SPIP
  **/
 function recuperer_cookies_spip($cookie_prefix) {
-	$prefix_long = strlen($cookie_prefix);
-
-	foreach (array_keys($_COOKIE) as $name) {
-		if (str_starts_with($name, 'spip_') && substr($name, 0, $prefix_long) != $cookie_prefix) {
-			unset($_COOKIE[$name]);
-			unset($GLOBALS[$name]);
-		}
-	}
-	foreach ($_COOKIE as $name => $value) {
-		if (substr($name, 0, $prefix_long) == $cookie_prefix) {
-			$spipname = preg_replace('/^' . $cookie_prefix . '_/', 'spip_', $name);
-			$_COOKIE[$spipname] = $value;
-			$GLOBALS[$spipname] = $value;
-		}
-	}
+    $prefix_long = strlen($cookie_prefix);
+
+    foreach (array_keys($_COOKIE) as $name) {
+        if (str_starts_with($name, 'spip_') && substr($name, 0, $prefix_long) != $cookie_prefix) {
+            unset($_COOKIE[$name]);
+            unset($GLOBALS[$name]);
+        }
+    }
+    foreach ($_COOKIE as $name => $value) {
+        if (substr($name, 0, $prefix_long) == $cookie_prefix) {
+            $spipname = preg_replace('/^' . $cookie_prefix . '_/', 'spip_', $name);
+            $_COOKIE[$spipname] = $value;
+            $GLOBALS[$spipname] = $value;
+        }
+    }
 }
 
 
@@ -181,18 +181,18 @@ 
  *
  **/
 function exec_test_ajax_dist() {
-	switch (_request('js')) {
-		// on est appele par <noscript>
-		case -1:
-			spip_setcookie('spip_accepte_ajax', -1);
-			include_spip('inc/headers');
-			redirige_par_entete(chemin_image('erreur-xx.svg'));
-			break;
-
-		// ou par ajax
-		case 1:
-		default:
-			spip_setcookie('spip_accepte_ajax', 1);
-			break;
-	}
+    switch (_request('js')) {
+        // on est appele par <noscript>
+        case -1:
+            spip_setcookie('spip_accepte_ajax', -1);
+            include_spip('inc/headers');
+            redirige_par_entete(chemin_image('erreur-xx.svg'));
+            break;
+
+        // ou par ajax
+        case 1:
+        default:
+            spip_setcookie('spip_accepte_ajax', 1);
+            break;
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -100,7 +100,7 @@ 
 
 	// in fine renommer le prefixe si besoin
 	if (str_starts_with($name, 'spip_')) {
-		$name = $GLOBALS['cookie_prefix'] . '_' . substr($name, 5);
+		$name = $GLOBALS['cookie_prefix'].'_'.substr($name, 5);
 	}
 
 	#spip_log("cookie('$name', '$value', " . json_encode($options, true) . ")", "cookies");
@@ -158,7 +158,7 @@ 
 	}
 	foreach ($_COOKIE as $name => $value) {
 		if (substr($name, 0, $prefix_long) == $cookie_prefix) {
-			$spipname = preg_replace('/^' . $cookie_prefix . '_/', 'spip_', $name);
+			$spipname = preg_replace('/^'.$cookie_prefix.'_/', 'spip_', $name);
 			$_COOKIE[$spipname] = $value;
 			$GLOBALS[$spipname] = $value;
 		}

ecrire/inc/selectionner.php

Indentation +108 added lines, -108 removed lines

@@ -16,7 +16,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/filtres');
@@ -45,56 +45,56 @@ 
  **/
 function inc_selectionner_dist($sel, $idom = '', $exclus = 0, $aff_racine = false, $recur = true, $do = 'aff') {
 
-	if ($recur) {
-		$recur = mini_hier($sel);
-	} else {
-		$sel = 0;
-	}
-
-	if ($aff_racine) {
-		$info = generer_url_ecrire('informer', "type=rubrique&rac=$idom&do=$do&id=");
-		$idom3 = $idom . '_selection';
-
-		$onClick = "jQuery(this).parent().addClass('on');jQuery('#choix_parent_principal .on').removeClass('on'); aff_selection(0, '$idom3', '$info', event);return false;";
-
-		$ondbClick = strtr(
-			str_replace(
-				"'",
-				'’',
-				str_replace(
-					'"',
-					'"',
-					textebrut(_T('info_racine_site'))
-				)
-			),
-			"\n\r",
-			'  '
-		);
-
-		$js_func = $do . '_selection_titre';
-		$ondbClick = "$js_func('$ondbClick',0,'selection_rubrique','id_parent');";
-
-		$aff_racine = "<div class='petit-item petite-racine item'>"
-			. "<a href='#'"
-			. 'onclick="'
-			. $onClick
-			. "\"\nondbclick=\""
-			. $ondbClick
-			. $onClick
-			. '">'
-			. _T('info_racine_site')
-			. '</a></div>';
-	}
-
-	$url_init = generer_url_ecrire('plonger', "rac=$idom&exclus=$exclus&id=0&col=1&do=$do");
-
-	$plonger = charger_fonction('plonger', 'inc');
-	$plonger_r = $plonger($sel, $idom, $recur, 1, $exclus, $do);
-
-	// url completee par la fonction JS onkeypress_rechercher
-	$url = generer_url_ecrire('rechercher', "exclus=$exclus&rac=$idom&do=$do&type=");
-
-	return construire_selectionner_hierarchie($idom, $plonger_r, $aff_racine, $url, 'id_parent', $url_init);
+    if ($recur) {
+        $recur = mini_hier($sel);
+    } else {
+        $sel = 0;
+    }
+
+    if ($aff_racine) {
+        $info = generer_url_ecrire('informer', "type=rubrique&rac=$idom&do=$do&id=");
+        $idom3 = $idom . '_selection';
+
+        $onClick = "jQuery(this).parent().addClass('on');jQuery('#choix_parent_principal .on').removeClass('on'); aff_selection(0, '$idom3', '$info', event);return false;";
+
+        $ondbClick = strtr(
+            str_replace(
+                "'",
+                '&#8217;',
+                str_replace(
+                    '"',
+                    '&#34;',
+                    textebrut(_T('info_racine_site'))
+                )
+            ),
+            "\n\r",
+            '  '
+        );
+
+        $js_func = $do . '_selection_titre';
+        $ondbClick = "$js_func('$ondbClick',0,'selection_rubrique','id_parent');";
+
+        $aff_racine = "<div class='petit-item petite-racine item'>"
+            . "<a href='#'"
+            . 'onclick="'
+            . $onClick
+            . "\"\nondbclick=\""
+            . $ondbClick
+            . $onClick
+            . '">'
+            . _T('info_racine_site')
+            . '</a></div>';
+    }
+
+    $url_init = generer_url_ecrire('plonger', "rac=$idom&exclus=$exclus&id=0&col=1&do=$do");
+
+    $plonger = charger_fonction('plonger', 'inc');
+    $plonger_r = $plonger($sel, $idom, $recur, 1, $exclus, $do);
+
+    // url completee par la fonction JS onkeypress_rechercher
+    $url = generer_url_ecrire('rechercher', "exclus=$exclus&rac=$idom&do=$do&type=");
+
+    return construire_selectionner_hierarchie($idom, $plonger_r, $aff_racine, $url, 'id_parent', $url_init);
 }
 
 /**
@@ -112,58 +112,58 @@ 
  **/
 function construire_selectionner_hierarchie($idom, $liste, $racine, $url, $name, $url_init = '') {
 
-	$idom1 = $idom . '_champ_recherche';
-	$idom2 = $idom . '_principal';
-	$idom3 = $idom . '_selection';
-	$idom4 = $idom . '_col_1';
-	$idom5 = 'img_' . $idom4;
-	$idom6 = $idom . '_fonc';
-
-	return "<div id='$idom'>"
-	. "<a id='$idom6' style='visibility: hidden;'"
-	. ($url_init ? "\nhref='$url_init'" : '')
-	. '></a>'
-	. "<div class='recherche_rapide_parent formulaire_recherche'>"
-	. http_img_pack(
-		'loader.svg',
-		'',
-		"class='loader' style='visibility: hidden;float:" . $GLOBALS['spip_lang_right'] . "' id='$idom5'"
-	)
-	. ''
-	. "<input style='width: 10em;float:" . $GLOBALS['spip_lang_right'] . ";' type='text' class='text search' id='$idom1' placeholder='" . _T('info_rechercher') . "'"
-	// eliminer Return car il provoque la soumission (balise unique)
-	// et eliminer Tab pour la navigation au clavier
-	// ce serait encore mieux de ne le faire que s'il y a encore plusieurs
-	// resultats retournes par la recherche
-	. "\nonkeypress=\"k=event.keyCode;if (k==13 || k==3 || k==9){return false;}\""
-	// lancer la recherche apres le filtrage ci-dessus sauf sur le tab (navigation au clavier)
-	. "\nonkeyup=\"if(event.keyCode==9){return false;};return onkey_rechercher(this.value,"
-	// la destination de la recherche
-	. "'$idom4'"
+    $idom1 = $idom . '_champ_recherche';
+    $idom2 = $idom . '_principal';
+    $idom3 = $idom . '_selection';
+    $idom4 = $idom . '_col_1';
+    $idom5 = 'img_' . $idom4;
+    $idom6 = $idom . '_fonc';
+
+    return "<div id='$idom'>"
+    . "<a id='$idom6' style='visibility: hidden;'"
+    . ($url_init ? "\nhref='$url_init'" : '')
+    . '></a>'
+    . "<div class='recherche_rapide_parent formulaire_recherche'>"
+    . http_img_pack(
+        'loader.svg',
+        '',
+        "class='loader' style='visibility: hidden;float:" . $GLOBALS['spip_lang_right'] . "' id='$idom5'"
+    )
+    . ''
+    . "<input style='width: 10em;float:" . $GLOBALS['spip_lang_right'] . ";' type='text' class='text search' id='$idom1' placeholder='" . _T('info_rechercher') . "'"
+    // eliminer Return car il provoque la soumission (balise unique)
+    // et eliminer Tab pour la navigation au clavier
+    // ce serait encore mieux de ne le faire que s'il y a encore plusieurs
+    // resultats retournes par la recherche
+    . "\nonkeypress=\"k=event.keyCode;if (k==13 || k==3 || k==9){return false;}\""
+    // lancer la recherche apres le filtrage ci-dessus sauf sur le tab (navigation au clavier)
+    . "\nonkeyup=\"if(event.keyCode==9){return false;};return onkey_rechercher(this.value,"
+    // la destination de la recherche
+    . "'$idom4'"
 #	. "this.parentNode.parentNode.parentNode.parentNode.nextSibling.firstChild.id"
-	. ",'"
-	// l'url effectuant la recherche
-	. $url
-	. "',"
-	// le noeud contenant un gif anime
-	// . "'idom5'"
-	. 'this.parentNode.previousSibling.firstChild'
-	. ",'"
-	// la valeur de l'attribut Name a remplir
-	. $name
-	. "','"
-	// noeud invisible memorisant l'URL initiale (pour re-initialisation)
-	. $idom6
-	. "')\""
-	. ' />'
-	. "\n</div>"
-	. ($racine ? "<div>$racine</div>" : '')
-	. "<div id='"
-	. $idom2
-	. "'><div id='$idom4'"
-	. " class=''>"
-	. $liste
-	. "</div></div>\n<div id='$idom3'></div></div>\n";
+    . ",'"
+    // l'url effectuant la recherche
+    . $url
+    . "',"
+    // le noeud contenant un gif anime
+    // . "'idom5'"
+    . 'this.parentNode.previousSibling.firstChild'
+    . ",'"
+    // la valeur de l'attribut Name a remplir
+    . $name
+    . "','"
+    // noeud invisible memorisant l'URL initiale (pour re-initialisation)
+    . $idom6
+    . "')\""
+    . ' />'
+    . "\n</div>"
+    . ($racine ? "<div>$racine</div>" : '')
+    . "<div id='"
+    . $idom2
+    . "'><div id='$idom4'"
+    . " class=''>"
+    . $liste
+    . "</div></div>\n<div id='$idom3'></div></div>\n";
 }
 
 /**
@@ -175,11 +175,11 @@ 
  **/
 function mini_hier($id_rubrique) {
 
-	$liste = $id_rubrique;
-	$id_rubrique = (int) $id_rubrique;
-	while ($id_rubrique = sql_getfetsel('id_parent', 'spip_rubriques', 'id_rubrique = ' . $id_rubrique)) {
-		$liste = $id_rubrique . ",$liste";
-	}
+    $liste = $id_rubrique;
+    $id_rubrique = (int) $id_rubrique;
+    while ($id_rubrique = sql_getfetsel('id_parent', 'spip_rubriques', 'id_rubrique = ' . $id_rubrique)) {
+        $liste = $id_rubrique . ",$liste";
+    }
 
-	return explode(',', "0,$liste");
+    return explode(',', "0,$liste");
 }

Spacing +12 added lines, -12 removed lines

@@ -53,7 +53,7 @@ 
 
 	if ($aff_racine) {
 		$info = generer_url_ecrire('informer', "type=rubrique&rac=$idom&do=$do&id=");
-		$idom3 = $idom . '_selection';
+		$idom3 = $idom.'_selection';
 
 		$onClick = "jQuery(this).parent().addClass('on');jQuery('#choix_parent_principal .on').removeClass('on'); aff_selection(0, '$idom3', '$info', event);return false;";
 
@@ -71,7 +71,7 @@ 
 			'  '
 		);
 
-		$js_func = $do . '_selection_titre';
+		$js_func = $do.'_selection_titre';
 		$ondbClick = "$js_func('$ondbClick',0,'selection_rubrique','id_parent');";
 
 		$aff_racine = "<div class='petit-item petite-racine item'>"
@@ -112,12 +112,12 @@ 
  **/
 function construire_selectionner_hierarchie($idom, $liste, $racine, $url, $name, $url_init = '') {
 
-	$idom1 = $idom . '_champ_recherche';
-	$idom2 = $idom . '_principal';
-	$idom3 = $idom . '_selection';
-	$idom4 = $idom . '_col_1';
-	$idom5 = 'img_' . $idom4;
-	$idom6 = $idom . '_fonc';
+	$idom1 = $idom.'_champ_recherche';
+	$idom2 = $idom.'_principal';
+	$idom3 = $idom.'_selection';
+	$idom4 = $idom.'_col_1';
+	$idom5 = 'img_'.$idom4;
+	$idom6 = $idom.'_fonc';
 
 	return "<div id='$idom'>"
 	. "<a id='$idom6' style='visibility: hidden;'"
@@ -127,10 +127,10 @@ 
 	. http_img_pack(
 		'loader.svg',
 		'',
-		"class='loader' style='visibility: hidden;float:" . $GLOBALS['spip_lang_right'] . "' id='$idom5'"
+		"class='loader' style='visibility: hidden;float:".$GLOBALS['spip_lang_right']."' id='$idom5'"
 	)
 	. ''
-	. "<input style='width: 10em;float:" . $GLOBALS['spip_lang_right'] . ";' type='text' class='text search' id='$idom1' placeholder='" . _T('info_rechercher') . "'"
+	. "<input style='width: 10em;float:".$GLOBALS['spip_lang_right'].";' type='text' class='text search' id='$idom1' placeholder='"._T('info_rechercher')."'"
 	// eliminer Return car il provoque la soumission (balise unique)
 	// et eliminer Tab pour la navigation au clavier
 	// ce serait encore mieux de ne le faire que s'il y a encore plusieurs
@@ -177,8 +177,8 @@ 
 
 	$liste = $id_rubrique;
 	$id_rubrique = (int) $id_rubrique;
-	while ($id_rubrique = sql_getfetsel('id_parent', 'spip_rubriques', 'id_rubrique = ' . $id_rubrique)) {
-		$liste = $id_rubrique . ",$liste";
+	while ($id_rubrique = sql_getfetsel('id_parent', 'spip_rubriques', 'id_rubrique = '.$id_rubrique)) {
+		$liste = $id_rubrique.",$liste";
 	}
 
 	return explode(',', "0,$liste");

ecrire/inc/simplexml_to_array.php

Indentation +62 added lines, -62 removed lines

@@ -10,7 +10,7 @@ 
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 
@@ -22,12 +22,12 @@ 
  * @return array
  */
 function inc_simplexml_to_array_dist($u, $utiliser_namespace = false) {
-	// decoder la chaine en SimpleXML si pas deja fait
-	if (is_string($u)) {
-		$u = simplexml_load_string($u);
-	}
+    // decoder la chaine en SimpleXML si pas deja fait
+    if (is_string($u)) {
+        $u = simplexml_load_string($u);
+    }
 
-	return ['root' => @xmlObjToArr($u, $utiliser_namespace)];
+    return ['root' => @xmlObjToArr($u, $utiliser_namespace)];
 }
 
 
@@ -42,67 +42,67 @@ 
  **/
 function xmlObjToArr($obj, $utiliser_namespace = false) {
 
-	$namespace = [];
-	$tableau = [];
+    $namespace = [];
+    $tableau = [];
 
-	// Cette fonction getDocNamespaces() est longue sur de gros xml. On permet donc
-	// de l'activer ou pas suivant le contenu supposé du XML
-	if (is_object($obj)) {
-		if (is_array($utiliser_namespace)) {
-			$namespace = $utiliser_namespace;
-		} else {
-			if ($utiliser_namespace) {
-				$namespace = $obj->getDocNamespaces(true);
-			}
-			$namespace[null] = null;
-		}
+    // Cette fonction getDocNamespaces() est longue sur de gros xml. On permet donc
+    // de l'activer ou pas suivant le contenu supposé du XML
+    if (is_object($obj)) {
+        if (is_array($utiliser_namespace)) {
+            $namespace = $utiliser_namespace;
+        } else {
+            if ($utiliser_namespace) {
+                $namespace = $obj->getDocNamespaces(true);
+            }
+            $namespace[null] = null;
+        }
 
-		$name = strtolower((string)$obj->getName());
-		$text = trim((string)$obj);
-		if (strlen($text) <= 0) {
-			$text = null;
-		}
+        $name = strtolower((string)$obj->getName());
+        $text = trim((string)$obj);
+        if (strlen($text) <= 0) {
+            $text = null;
+        }
 
-		$children = [];
-		$attributes = [];
+        $children = [];
+        $attributes = [];
 
-		// get info for all namespaces
-		foreach (array_keys($namespace) as $ns) {
-			// attributes
-			$objAttributes = $obj->attributes($ns, true);
-			foreach ($objAttributes as $attributeName => $attributeValue) {
-				$attribName = strtolower(trim((string)$attributeName));
-				$attribVal = trim((string)$attributeValue);
-				if (!empty($ns)) {
-					$attribName = $ns . ':' . $attribName;
-				}
-				$attributes[$attribName] = $attribVal;
-			}
+        // get info for all namespaces
+        foreach (array_keys($namespace) as $ns) {
+            // attributes
+            $objAttributes = $obj->attributes($ns, true);
+            foreach ($objAttributes as $attributeName => $attributeValue) {
+                $attribName = strtolower(trim((string)$attributeName));
+                $attribVal = trim((string)$attributeValue);
+                if (!empty($ns)) {
+                    $attribName = $ns . ':' . $attribName;
+                }
+                $attributes[$attribName] = $attribVal;
+            }
 
-			// children
-			$objChildren = $obj->children($ns, true);
-			foreach ($objChildren as $childName => $child) {
-				$childName = strtolower((string)$childName);
-				if (!empty($ns)) {
-					$childName = $ns . ':' . $childName;
-				}
-				$children[$childName][] = xmlObjToArr($child, $namespace);
-			}
-		}
+            // children
+            $objChildren = $obj->children($ns, true);
+            foreach ($objChildren as $childName => $child) {
+                $childName = strtolower((string)$childName);
+                if (!empty($ns)) {
+                    $childName = $ns . ':' . $childName;
+                }
+                $children[$childName][] = xmlObjToArr($child, $namespace);
+            }
+        }
 
-		$tableau = [
-			'name' => $name,
-		];
-		if ($text) {
-			$tableau['text'] = $text;
-		}
-		if ($attributes) {
-			$tableau['attributes'] = $attributes;
-		}
-		if ($children) {
-			$tableau['children'] = $children;
-		}
-	}
+        $tableau = [
+            'name' => $name,
+        ];
+        if ($text) {
+            $tableau['text'] = $text;
+        }
+        if ($attributes) {
+            $tableau['attributes'] = $attributes;
+        }
+        if ($children) {
+            $tableau['children'] = $children;
+        }
+    }
 
-	return $tableau;
+    return $tableau;
 }

ecrire/inc/icone_renommer.php

Indentation +53 added lines, -53 removed lines

@@ -10,71 +10,71 @@
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/boutons');
 include_spip('base/objets');
 
 function inc_icone_renommer_dist($fond, $fonction) {
-	$size = 24;
-	if (
-		preg_match('/(?:-(\d{1,3}))?([.](gif|png|svg))?$/i', $fond, $match)
-		&& (isset($match[0]) && $match[0] || isset($match[1]) && $match[1])
-	) {
-		if (isset($match[1]) && $match[1]) {
-			$size = $match[1];
-		}
-		$type = substr($fond, 0, -strlen($match[0]));
-		if (!isset($match[2]) || !$match[2]) {
-			$fond .= '.png';
-		}
-	} else {
-		$type = $fond;
-		$fond .= '.png';
-	}
+    $size = 24;
+    if (
+        preg_match('/(?:-(\d{1,3}))?([.](gif|png|svg))?$/i', $fond, $match)
+        && (isset($match[0]) && $match[0] || isset($match[1]) && $match[1])
+    ) {
+        if (isset($match[1]) && $match[1]) {
+            $size = $match[1];
+        }
+        $type = substr($fond, 0, -strlen($match[0]));
+        if (!isset($match[2]) || !$match[2]) {
+            $fond .= '.png';
+        }
+    } else {
+        $type = $fond;
+        $fond .= '.png';
+    }
 
-	$rtl = false;
-	if (preg_match(',[-_]rtl$,i', $type, $match)) {
-		$rtl = true;
-		$type = substr($type, 0, -strlen($match[0]));
-	}
+    $rtl = false;
+    if (preg_match(',[-_]rtl$,i', $type, $match)) {
+        $rtl = true;
+        $type = substr($type, 0, -strlen($match[0]));
+    }
 
-	// objet_type garde invariant tout ce qui ne commence par par id_, spip_
-	// et ne finit pas par un s, sauf si c'est une exception declaree
-	$type = objet_type($type, false);
+    // objet_type garde invariant tout ce qui ne commence par par id_, spip_
+    // et ne finit pas par un s, sauf si c'est une exception declaree
+    $type = objet_type($type, false);
 
-	$dir = 'images/';
-	$f = "$type-$size.png";
-	if ($icone = find_in_theme($dir . $f)) {
-		$dir = dirname($icone);
-		$fond = $icone;
+    $dir = 'images/';
+    $f = "$type-$size.png";
+    if ($icone = find_in_theme($dir . $f)) {
+        $dir = dirname($icone);
+        $fond = $icone;
 
-		if (
-			$rtl
-			&& ($fr = $dir . '/' . str_replace("$type-", "$type-rtl-", basename($icone)))
-			&& file_exists($fr)
-		) {
-			$fond = $fr;
-		}
+        if (
+            $rtl
+            && ($fr = $dir . '/' . str_replace("$type-", "$type-rtl-", basename($icone)))
+            && file_exists($fr)
+        ) {
+            $fond = $fr;
+        }
 
-		$action = $fonction;
-		if ($action == 'supprimer.gif') {
-			$action = 'del';
-		} elseif ($action == 'creer.gif') {
-			$action = 'new';
-		} elseif ($action == 'edit.gif') {
-			$action = 'edit';
-		}
+        $action = $fonction;
+        if ($action == 'supprimer.gif') {
+            $action = 'del';
+        } elseif ($action == 'creer.gif') {
+            $action = 'new';
+        } elseif ($action == 'edit.gif') {
+            $action = 'edit';
+        }
 
-		$fonction = '';
-		if (in_array($action, ['add','del', 'new', 'edit', 'config'])) {
-			$fonction = $action;
-		}
+        $fonction = '';
+        if (in_array($action, ['add','del', 'new', 'edit', 'config'])) {
+            $fonction = $action;
+        }
 
-		// c'est bon !
-		return [$fond, $fonction];
-	}
+        // c'est bon !
+        return [$fond, $fonction];
+    }
 
-	return [$fond, $fonction];
+    return [$fond, $fonction];
 }

Spacing +3 added lines, -3 removed lines

@@ -46,13 +46,13 @@ 
 
 	$dir = 'images/';
 	$f = "$type-$size.png";
-	if ($icone = find_in_theme($dir . $f)) {
+	if ($icone = find_in_theme($dir.$f)) {
 		$dir = dirname($icone);
 		$fond = $icone;
 
 		if (
 			$rtl
-			&& ($fr = $dir . '/' . str_replace("$type-", "$type-rtl-", basename($icone)))
+			&& ($fr = $dir.'/'.str_replace("$type-", "$type-rtl-", basename($icone)))
 			&& file_exists($fr)
 		) {
 			$fond = $fr;
@@ -68,7 +68,7 @@ 
 		}
 
 		$fonction = '';
-		if (in_array($action, ['add','del', 'new', 'edit', 'config'])) {
+		if (in_array($action, ['add', 'del', 'new', 'edit', 'config'])) {
 			$fonction = $action;
 		}
 

ecrire/inc/modifier.php

Indentation +281 added lines, -281 removed lines

@@ -16,7 +16,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -38,28 +38,28 @@ 
  *     Tableau des champs et valeurs collectées
  */
 function collecter_requests($include_list, $exclude_list = [], $set = null, $tous = false) {
-	$c = $set;
-	if (!$c) {
-		$c = [];
-		foreach ($include_list as $champ) {
-			// on ne collecte que les champs reellement envoyes par defaut.
-			// le cas d'un envoi de valeur NULL peut du coup poser probleme.
-			$val = _request($champ);
-			if ($tous or $val !== null) {
-				$c[$champ] = $val;
-			}
-		}
-		// on ajoute toujours la lang en saisie possible
-		// meme si pas prevu au depart pour l'objet concerne
-		if ($l = _request('changer_lang')) {
-			$c['lang'] = $l;
-		}
-	}
-	foreach ($exclude_list as $champ) {
-		unset($c[$champ]);
-	}
-
-	return $c;
+    $c = $set;
+    if (!$c) {
+        $c = [];
+        foreach ($include_list as $champ) {
+            // on ne collecte que les champs reellement envoyes par defaut.
+            // le cas d'un envoi de valeur NULL peut du coup poser probleme.
+            $val = _request($champ);
+            if ($tous or $val !== null) {
+                $c[$champ] = $val;
+            }
+        }
+        // on ajoute toujours la lang en saisie possible
+        // meme si pas prevu au depart pour l'objet concerne
+        if ($l = _request('changer_lang')) {
+            $c['lang'] = $l;
+        }
+    }
+    foreach ($exclude_list as $champ) {
+        unset($c[$champ]);
+    }
+
+    return $c;
 }
 
 /**
@@ -96,262 +96,262 @@ 
  *     - chaîne : texte d'un message d'erreur
  */
 function objet_modifier_champs($objet, $id_objet, $options, $c = null, $serveur = '') {
-	if (!$id_objet = intval($id_objet)) {
-		spip_log('Erreur $id_objet non defini', 'warn');
-
-		return _T('erreur_technique_enregistrement_impossible');
-	}
-
-	include_spip('inc/filtres');
-
-	$table_objet = table_objet($objet, $serveur);
-	$spip_table_objet = table_objet_sql($objet, $serveur);
-	$id_table_objet = id_table_objet($objet, $serveur);
-	$trouver_table = charger_fonction('trouver_table', 'base');
-	$desc = $trouver_table($spip_table_objet, $serveur);
-
-	// Appels incomplets (sans $c)
-	if (!is_array($c)) {
-		spip_log('erreur appel objet_modifier_champs(' . $objet . '), manque $c');
-
-		return _T('erreur_technique_enregistrement_impossible');
-	}
-
-	// Securite : certaines variables ne sont jamais acceptees ici
-	// car elles ne relevent pas de autoriser(xxx, modifier) ;
-	// il faut passer par instituer_XX()
-	// TODO: faut-il passer ces variables interdites
-	// dans un fichier de description separe ?
-	unset($c['statut']);
-	unset($c['id_parent']);
-	unset($c['id_rubrique']);
-	unset($c['id_secteur']);
-
-	// Gerer les champs non vides
-	if (isset($options['nonvide']) and is_array($options['nonvide'])) {
-		foreach ($options['nonvide'] as $champ => $sinon) {
-			if (isset($c[$champ]) and $c[$champ] === '') {
-				$c[$champ] = $sinon;
-			}
-		}
-	}
-
-	// N'accepter que les champs qui existent dans la table
-	$champs = array_intersect_key($c, $desc['field']);
-	// et dont la valeur n'est pas null
-	$champs = array_filter($champs, static function ($var) {
-		return $var !== null;
-	});
-	// TODO: ici aussi on peut valider les contenus
-	// en fonction du type
-
-	// Nettoyer les valeurs
-	$champs = array_map('corriger_caracteres', $champs);
-
-	// On récupère l'état avant toute modification
-	$row = sql_fetsel('*', $spip_table_objet, $id_table_objet . '=' . $id_objet);
-
-	// Envoyer aux plugins
-	$champs = pipeline(
-		'pre_edition',
-		[
-			'args' => [
-				'table' => $spip_table_objet, // compatibilite
-				'table_objet' => $table_objet,
-				'spip_table_objet' => $spip_table_objet,
-				'desc' => $desc,
-				'type' => $objet,
-				'id_objet' => $id_objet,
-				'data' => $options['data'] ?? null,
-				'champs' => $options['champs'] ?? [], // [doc] c'est quoi ?
-				'champs_anciens' => $row, // état du contenu avant modif
-				'serveur' => $serveur,
-				'action' => $options['action'] ?? 'modifier'
-			],
-			'data' => $champs
-		]
-	);
-
-	if (!$champs) {
-		return false;
-	}
-
-
-	// marquer le fait que l'objet est travaille par toto a telle date
-	include_spip('inc/config');
-	if (lire_config('articles_modif', 'non') !== 'non') {
-		include_spip('inc/drapeau_edition');
-		signale_edition($id_objet, $GLOBALS['visiteur_session'], $objet);
-	}
-
-	// Verifier si les mises a jour sont pertinentes, datees, en conflit etc
-	include_spip('inc/editer');
-	if (!isset($options['data']) or is_null($options['data'])) {
-		$options['data'] = &$_POST;
-	}
-	$conflits = controler_md5($champs, $options['data'], $objet, $id_objet, $serveur);
-	// cas hypothetique : normalement inc/editer verifie en amont le conflit edition
-	// et gere l'interface
-	// ici on ne renvoie donc qu'un messsage d'erreur, au cas ou on y arrive quand meme
-	if ($conflits) {
-		return _T('titre_conflit_edition');
-	}
-
-	if ($champs) {
-		// cas particulier de la langue : passer par instituer_langue_objet
-		if (isset($champs['lang'])) {
-			if ($changer_lang = $champs['lang']) {
-				$id_rubrique = 0;
-				if (isset($desc['field']['id_rubrique'])) {
-					$parent = ($objet == 'rubrique') ? 'id_parent' : 'id_rubrique';
-					$id_rubrique = sql_getfetsel($parent, $spip_table_objet, "$id_table_objet=" . intval($id_objet));
-				}
-				$instituer_langue_objet = charger_fonction('instituer_langue_objet', 'action');
-				$champs['lang'] = $instituer_langue_objet($objet, $id_objet, $id_rubrique, $changer_lang, $serveur);
-			}
-			// on laisse 'lang' dans $champs,
-			// ca permet de passer dans le pipeline post_edition et de journaliser
-			// et ca ne gene pas qu'on refasse un sql_updateq dessus apres l'avoir
-			// deja pris en compte
-		}
-
-		// la modif peut avoir lieu
-
-		// faut-il ajouter date_modif ?
-		if (
-			!empty($options['date_modif'])
-			and !isset($champs[$options['date_modif']])
-		) {
-			$champs[$options['date_modif']] = date('Y-m-d H:i:s');
-		}
-
-		// allez on commit la modif
-		sql_updateq($spip_table_objet, $champs, "$id_table_objet=" . intval($id_objet), [], $serveur);
-
-		// on verifie si elle est bien passee
-		$moof = sql_fetsel(
-			array_keys($champs),
-			$spip_table_objet,
-			"$id_table_objet=" . intval($id_objet),
-			[],
-			[],
-			'',
-			[],
-			$serveur
-		);
-		// si difference entre les champs, reperer les champs mal enregistres
-		if ($moof != $champs) {
-			$liste = [];
-			foreach ($moof as $k => $v) {
-				if (
-					$v !== $champs[$k]
-					// ne pas alerter si le champ est numerique est que les valeurs sont equivalentes
-					and (!is_numeric($v) or intval($v) !== intval($champs[$k]))
-					// ne pas alerter si le champ est date, qu'on a envoye une valeur vide et qu'on recupere une date nulle
-					and (strlen($champs[$k]) or !in_array($v, ['0000-00-00 00:00:00', '0000-00-00']))
-				) {
-					$liste[] = $k;
-					$conflits[$k]['post'] = $champs[$k];
-					$conflits[$k]['save'] = $v;
-
-					// cas specifique MySQL+emoji : si l'un est la
-					// conversion utf8_noplanes de l'autre alors c'est OK
-					if (defined('_MYSQL_NOPLANES') && _MYSQL_NOPLANES) {
-						include_spip('inc/charsets');
-						if ($v == utf8_noplanes($champs[$k])) {
-							array_pop($liste);
-						}
-					}
-				}
-			}
-			// si un champ n'a pas ete correctement enregistre, loger et retourner une erreur
-			// c'est un cas exceptionnel
-			if (count($liste)) {
-				spip_log(
-					"Erreur enregistrement en base $objet/$id_objet champs :" . var_export($conflits, true),
-					'modifier.' . _LOG_CRITIQUE
-				);
-
-				return _T(
-					'erreur_technique_enregistrement_champs',
-					['champs' => "<i>'" . implode("'</i>,<i>'", $liste) . "'</i>"]
-				);
-			}
-		}
-
-		// Invalider les caches
-		if (isset($options['invalideur']) and $options['invalideur']) {
-			include_spip('inc/invalideur');
-			if (is_array($options['invalideur'])) {
-				array_map('suivre_invalideur', $options['invalideur']);
-			} else {
-				suivre_invalideur($options['invalideur']);
-			}
-		}
-
-		// Notifications, gestion des revisions...
-		// en standard, appelle |nouvelle_revision ci-dessous
-		pipeline(
-			'post_edition',
-			[
-				'args' => [
-					'table' => $spip_table_objet,
-					'table_objet' => $table_objet,
-					'spip_table_objet' => $spip_table_objet,
-					'desc' => $desc,
-					'type' => $objet,
-					'id_objet' => $id_objet,
-					'champs' => $options['champs'] ?? [], // [doc] kesako ?
-					'champs_anciens' => $row, // état du contenu avant modif
-					'serveur' => $serveur,
-					'action' => $options['action'] ?? 'modifier'
-				],
-				'data' => $champs
-			]
-		);
-	}
-
-	// Appeler une notification
-	if ($notifications = charger_fonction('notifications', 'inc')) {
-		$notifications(
-			"{$objet}_modifier",
-			$id_objet,
-			[
-				'champs' => $champs,
-			]
-		);
-		$notifications(
-			'objet_modifier',
-			$id_objet,
-			[
-				'objet' => $objet,
-				'id_objet' => $id_objet,
-				'champs' => $champs,
-			]
-		);
-	}
-
-	// journaliser l'affaire
-	// message a affiner :-)
-	include_spip('inc/filtres_mini');
-	$qui = '';
-	if (!empty($GLOBALS['visiteur_session']['id_auteur'])) {
-		$qui .= ' #id_auteur:' . $GLOBALS['visiteur_session']['id_auteur'] . '#';
-	}
-	if (!empty($GLOBALS['visiteur_session']['nom'])) {
-		$qui .= ' #nom:' . $GLOBALS['visiteur_session']['nom'] . '#';
-	}
-	if ($qui == '') {
-		$qui = '#ip:' . $GLOBALS['ip'] . '#';
-	}
-	journal(_L($qui . ' a édité ' . $objet . ' ' . $id_objet . ' (' . join(
-		'+',
-		array_diff(array_keys($champs), ['date_modif'])
-	) . ')'), [
-		'faire' => 'modifier',
-		'quoi' => $objet,
-		'id' => $id_objet
-	]);
-
-	return '';
+    if (!$id_objet = intval($id_objet)) {
+        spip_log('Erreur $id_objet non defini', 'warn');
+
+        return _T('erreur_technique_enregistrement_impossible');
+    }
+
+    include_spip('inc/filtres');
+
+    $table_objet = table_objet($objet, $serveur);
+    $spip_table_objet = table_objet_sql($objet, $serveur);
+    $id_table_objet = id_table_objet($objet, $serveur);
+    $trouver_table = charger_fonction('trouver_table', 'base');
+    $desc = $trouver_table($spip_table_objet, $serveur);
+
+    // Appels incomplets (sans $c)
+    if (!is_array($c)) {
+        spip_log('erreur appel objet_modifier_champs(' . $objet . '), manque $c');
+
+        return _T('erreur_technique_enregistrement_impossible');
+    }
+
+    // Securite : certaines variables ne sont jamais acceptees ici
+    // car elles ne relevent pas de autoriser(xxx, modifier) ;
+    // il faut passer par instituer_XX()
+    // TODO: faut-il passer ces variables interdites
+    // dans un fichier de description separe ?
+    unset($c['statut']);
+    unset($c['id_parent']);
+    unset($c['id_rubrique']);
+    unset($c['id_secteur']);
+
+    // Gerer les champs non vides
+    if (isset($options['nonvide']) and is_array($options['nonvide'])) {
+        foreach ($options['nonvide'] as $champ => $sinon) {
+            if (isset($c[$champ]) and $c[$champ] === '') {
+                $c[$champ] = $sinon;
+            }
+        }
+    }
+
+    // N'accepter que les champs qui existent dans la table
+    $champs = array_intersect_key($c, $desc['field']);
+    // et dont la valeur n'est pas null
+    $champs = array_filter($champs, static function ($var) {
+        return $var !== null;
+    });
+    // TODO: ici aussi on peut valider les contenus
+    // en fonction du type
+
+    // Nettoyer les valeurs
+    $champs = array_map('corriger_caracteres', $champs);
+
+    // On récupère l'état avant toute modification
+    $row = sql_fetsel('*', $spip_table_objet, $id_table_objet . '=' . $id_objet);
+
+    // Envoyer aux plugins
+    $champs = pipeline(
+        'pre_edition',
+        [
+            'args' => [
+                'table' => $spip_table_objet, // compatibilite
+                'table_objet' => $table_objet,
+                'spip_table_objet' => $spip_table_objet,
+                'desc' => $desc,
+                'type' => $objet,
+                'id_objet' => $id_objet,
+                'data' => $options['data'] ?? null,
+                'champs' => $options['champs'] ?? [], // [doc] c'est quoi ?
+                'champs_anciens' => $row, // état du contenu avant modif
+                'serveur' => $serveur,
+                'action' => $options['action'] ?? 'modifier'
+            ],
+            'data' => $champs
+        ]
+    );
+
+    if (!$champs) {
+        return false;
+    }
+
+
+    // marquer le fait que l'objet est travaille par toto a telle date
+    include_spip('inc/config');
+    if (lire_config('articles_modif', 'non') !== 'non') {
+        include_spip('inc/drapeau_edition');
+        signale_edition($id_objet, $GLOBALS['visiteur_session'], $objet);
+    }
+
+    // Verifier si les mises a jour sont pertinentes, datees, en conflit etc
+    include_spip('inc/editer');
+    if (!isset($options['data']) or is_null($options['data'])) {
+        $options['data'] = &$_POST;
+    }
+    $conflits = controler_md5($champs, $options['data'], $objet, $id_objet, $serveur);
+    // cas hypothetique : normalement inc/editer verifie en amont le conflit edition
+    // et gere l'interface
+    // ici on ne renvoie donc qu'un messsage d'erreur, au cas ou on y arrive quand meme
+    if ($conflits) {
+        return _T('titre_conflit_edition');
+    }
+
+    if ($champs) {
+        // cas particulier de la langue : passer par instituer_langue_objet
+        if (isset($champs['lang'])) {
+            if ($changer_lang = $champs['lang']) {
+                $id_rubrique = 0;
+                if (isset($desc['field']['id_rubrique'])) {
+                    $parent = ($objet == 'rubrique') ? 'id_parent' : 'id_rubrique';
+                    $id_rubrique = sql_getfetsel($parent, $spip_table_objet, "$id_table_objet=" . intval($id_objet));
+                }
+                $instituer_langue_objet = charger_fonction('instituer_langue_objet', 'action');
+                $champs['lang'] = $instituer_langue_objet($objet, $id_objet, $id_rubrique, $changer_lang, $serveur);
+            }
+            // on laisse 'lang' dans $champs,
+            // ca permet de passer dans le pipeline post_edition et de journaliser
+            // et ca ne gene pas qu'on refasse un sql_updateq dessus apres l'avoir
+            // deja pris en compte
+        }
+
+        // la modif peut avoir lieu
+
+        // faut-il ajouter date_modif ?
+        if (
+            !empty($options['date_modif'])
+            and !isset($champs[$options['date_modif']])
+        ) {
+            $champs[$options['date_modif']] = date('Y-m-d H:i:s');
+        }
+
+        // allez on commit la modif
+        sql_updateq($spip_table_objet, $champs, "$id_table_objet=" . intval($id_objet), [], $serveur);
+
+        // on verifie si elle est bien passee
+        $moof = sql_fetsel(
+            array_keys($champs),
+            $spip_table_objet,
+            "$id_table_objet=" . intval($id_objet),
+            [],
+            [],
+            '',
+            [],
+            $serveur
+        );
+        // si difference entre les champs, reperer les champs mal enregistres
+        if ($moof != $champs) {
+            $liste = [];
+            foreach ($moof as $k => $v) {
+                if (
+                    $v !== $champs[$k]
+                    // ne pas alerter si le champ est numerique est que les valeurs sont equivalentes
+                    and (!is_numeric($v) or intval($v) !== intval($champs[$k]))
+                    // ne pas alerter si le champ est date, qu'on a envoye une valeur vide et qu'on recupere une date nulle
+                    and (strlen($champs[$k]) or !in_array($v, ['0000-00-00 00:00:00', '0000-00-00']))
+                ) {
+                    $liste[] = $k;
+                    $conflits[$k]['post'] = $champs[$k];
+                    $conflits[$k]['save'] = $v;
+
+                    // cas specifique MySQL+emoji : si l'un est la
+                    // conversion utf8_noplanes de l'autre alors c'est OK
+                    if (defined('_MYSQL_NOPLANES') && _MYSQL_NOPLANES) {
+                        include_spip('inc/charsets');
+                        if ($v == utf8_noplanes($champs[$k])) {
+                            array_pop($liste);
+                        }
+                    }
+                }
+            }
+            // si un champ n'a pas ete correctement enregistre, loger et retourner une erreur
+            // c'est un cas exceptionnel
+            if (count($liste)) {
+                spip_log(
+                    "Erreur enregistrement en base $objet/$id_objet champs :" . var_export($conflits, true),
+                    'modifier.' . _LOG_CRITIQUE
+                );
+
+                return _T(
+                    'erreur_technique_enregistrement_champs',
+                    ['champs' => "<i>'" . implode("'</i>,<i>'", $liste) . "'</i>"]
+                );
+            }
+        }
+
+        // Invalider les caches
+        if (isset($options['invalideur']) and $options['invalideur']) {
+            include_spip('inc/invalideur');
+            if (is_array($options['invalideur'])) {
+                array_map('suivre_invalideur', $options['invalideur']);
+            } else {
+                suivre_invalideur($options['invalideur']);
+            }
+        }
+
+        // Notifications, gestion des revisions...
+        // en standard, appelle |nouvelle_revision ci-dessous
+        pipeline(
+            'post_edition',
+            [
+                'args' => [
+                    'table' => $spip_table_objet,
+                    'table_objet' => $table_objet,
+                    'spip_table_objet' => $spip_table_objet,
+                    'desc' => $desc,
+                    'type' => $objet,
+                    'id_objet' => $id_objet,
+                    'champs' => $options['champs'] ?? [], // [doc] kesako ?
+                    'champs_anciens' => $row, // état du contenu avant modif
+                    'serveur' => $serveur,
+                    'action' => $options['action'] ?? 'modifier'
+                ],
+                'data' => $champs
+            ]
+        );
+    }
+
+    // Appeler une notification
+    if ($notifications = charger_fonction('notifications', 'inc')) {
+        $notifications(
+            "{$objet}_modifier",
+            $id_objet,
+            [
+                'champs' => $champs,
+            ]
+        );
+        $notifications(
+            'objet_modifier',
+            $id_objet,
+            [
+                'objet' => $objet,
+                'id_objet' => $id_objet,
+                'champs' => $champs,
+            ]
+        );
+    }
+
+    // journaliser l'affaire
+    // message a affiner :-)
+    include_spip('inc/filtres_mini');
+    $qui = '';
+    if (!empty($GLOBALS['visiteur_session']['id_auteur'])) {
+        $qui .= ' #id_auteur:' . $GLOBALS['visiteur_session']['id_auteur'] . '#';
+    }
+    if (!empty($GLOBALS['visiteur_session']['nom'])) {
+        $qui .= ' #nom:' . $GLOBALS['visiteur_session']['nom'] . '#';
+    }
+    if ($qui == '') {
+        $qui = '#ip:' . $GLOBALS['ip'] . '#';
+    }
+    journal(_L($qui . ' a édité ' . $objet . ' ' . $id_objet . ' (' . join(
+        '+',
+        array_diff(array_keys($champs), ['date_modif'])
+    ) . ')'), [
+        'faire' => 'modifier',
+        'quoi' => $objet,
+        'id' => $id_objet
+    ]);
+
+    return '';
 }

ecrire/inc/chercher_rubrique.php

Indentation +189 added lines, -189 removed lines

@@ -17,16 +17,16 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 if (!defined('_SPIP_SELECT_RUBRIQUES')) {
-	/**
-	 * @var int Nombre de rubriques maximum du sélecteur de rubriques.
-	 * Au delà, on bascule sur un sélecteur ajax.
-	 * mettre 100000 pour desactiver ajax
-	 */
-	define('_SPIP_SELECT_RUBRIQUES', 20);
+    /**
+     * @var int Nombre de rubriques maximum du sélecteur de rubriques.
+     * Au delà, on bascule sur un sélecteur ajax.
+     * mettre 100000 pour desactiver ajax
+     */
+    define('_SPIP_SELECT_RUBRIQUES', 20);
 }
 
 /**
@@ -54,22 +54,22 @@ 
  *     Code HTML du sélecteur
  **/
 function inc_chercher_rubrique_dist($id_rubrique, $type, $restreint, $idem = 0, $do = 'aff') {
-	if (sql_countsel('spip_rubriques') < 1) {
-		return '';
-	}
-
-	// Mode sans Ajax :
-	// - soit parce que le cookie ajax n'est pas la
-	// - soit parce qu'il y a peu de rubriques
-	if (
-		_SPIP_AJAX < 1
-		|| $type == 'breve'
-		|| sql_countsel('spip_rubriques') < _SPIP_SELECT_RUBRIQUES
-	) {
-		return selecteur_rubrique_html($id_rubrique, $type, $restreint, $idem);
-	} else {
-		return selecteur_rubrique_ajax($id_rubrique, $type, $restreint, $idem, $do);
-	}
+    if (sql_countsel('spip_rubriques') < 1) {
+        return '';
+    }
+
+    // Mode sans Ajax :
+    // - soit parce que le cookie ajax n'est pas la
+    // - soit parce qu'il y a peu de rubriques
+    if (
+        _SPIP_AJAX < 1
+        || $type == 'breve'
+        || sql_countsel('spip_rubriques') < _SPIP_SELECT_RUBRIQUES
+    ) {
+        return selecteur_rubrique_html($id_rubrique, $type, $restreint, $idem);
+    } else {
+        return selecteur_rubrique_ajax($id_rubrique, $type, $restreint, $idem, $do);
+    }
 }
 
 // compatibilite pour extensions qui utilisaient l'ancien nom
@@ -86,17 +86,17 @@ 
  **/
 function style_menu_rubriques($i) {
 
-	$espace = '';
-	$style = '';
-	for ($count = 1; $count <= $i; $count++) {
-		$espace .= '&nbsp;&nbsp;&nbsp;&nbsp;';
-	}
-	if ($i == 1) {
-		$espace = '';
-	}
-	$class = "niveau_$i";
-
-	return [$class, $style, $espace];
+    $espace = '';
+    $style = '';
+    for ($count = 1; $count <= $i; $count++) {
+        $espace .= '&nbsp;&nbsp;&nbsp;&nbsp;';
+    }
+    if ($i == 1) {
+        $espace = '';
+    }
+    $class = "niveau_$i";
+
+    return [$class, $style, $espace];
 }
 
 /**
@@ -120,57 +120,57 @@ 
  *     Code HTML du sélecteur
  **/
 function sous_menu_rubriques($id_rubrique, $root, $niv, &$data, &$enfants, $exclus, $restreint, $type) {
-	static $decalage_secteur;
-
-	// Si on a demande l'exclusion ne pas descendre dans la rubrique courante
-	if ($exclus > 0 && $root == $exclus) {
-		return '';
-	}
-
-	// en fonction du niveau faire un affichage plus ou moins kikoo
-
-	// selected ?
-	$selected = ($root == $id_rubrique) ? ' selected="selected"' : '';
-
-	// le style en fonction de la profondeur
-	[$class, $style, $espace] = style_menu_rubriques($niv);
-
-	$class .= ' selec_rub';
-
-	// creer l'<option> pour la rubrique $root
-
-	if (isset($data[$root])) { # pas de racine sauf pour les rubriques
-	$r = "<option$selected value='$root' class='$class' style='$style'>$espace"
-			. $data[$root]
-			. '</option>' . "\n";
-	} else {
-		$r = '';
-	}
-
-	// et le sous-menu pour ses enfants
-	$sous = '';
-	if (isset($enfants[$root])) {
-		foreach ($enfants[$root] as $sousrub) {
-			$sous .= sous_menu_rubriques(
-				$id_rubrique,
-				$sousrub,
-				$niv + 1,
-				$data,
-				$enfants,
-				$exclus,
-				$restreint,
-				$type
-			);
-		}
-	}
-
-	// si l'objet a deplacer est publie, verifier qu'on a acces aux rubriques
-	if ($restreint && $root != $id_rubrique && !autoriser('publierdans', 'rubrique', $root)) {
-		return $sous;
-	}
-
-	// et voila le travail
-	return $r . $sous;
+    static $decalage_secteur;
+
+    // Si on a demande l'exclusion ne pas descendre dans la rubrique courante
+    if ($exclus > 0 && $root == $exclus) {
+        return '';
+    }
+
+    // en fonction du niveau faire un affichage plus ou moins kikoo
+
+    // selected ?
+    $selected = ($root == $id_rubrique) ? ' selected="selected"' : '';
+
+    // le style en fonction de la profondeur
+    [$class, $style, $espace] = style_menu_rubriques($niv);
+
+    $class .= ' selec_rub';
+
+    // creer l'<option> pour la rubrique $root
+
+    if (isset($data[$root])) { # pas de racine sauf pour les rubriques
+    $r = "<option$selected value='$root' class='$class' style='$style'>$espace"
+            . $data[$root]
+            . '</option>' . "\n";
+    } else {
+        $r = '';
+    }
+
+    // et le sous-menu pour ses enfants
+    $sous = '';
+    if (isset($enfants[$root])) {
+        foreach ($enfants[$root] as $sousrub) {
+            $sous .= sous_menu_rubriques(
+                $id_rubrique,
+                $sousrub,
+                $niv + 1,
+                $data,
+                $enfants,
+                $exclus,
+                $restreint,
+                $type
+            );
+        }
+    }
+
+    // si l'objet a deplacer est publie, verifier qu'on a acces aux rubriques
+    if ($restreint && $root != $id_rubrique && !autoriser('publierdans', 'rubrique', $root)) {
+        return $sous;
+    }
+
+    // et voila le travail
+    return $r . $sous;
 }
 
 /**
@@ -191,75 +191,75 @@ 
  *     Code HTML du sélecteur
  **/
 function selecteur_rubrique_html($id_rubrique, $type, $restreint, $idem = 0) {
-	$enfants = [];
-	$data = [];
-	if ($type == 'rubrique' && autoriser('publierdans', 'rubrique', 0)) {
-		$data[0] = _T('info_racine_site');
-	}
-	# premier choix = neant
-	# si auteur (rubriques restreintes)
-	# ou si creation avec id_rubrique=0
-	elseif ($type == 'auteur' || !$id_rubrique) {
-		$data[0] = '&nbsp;';
-	}
-
-	//
-	// creer une structure contenant toute l'arborescence
-	//
-
-	include_spip('base/abstract_sql');
-	$q = sql_select(
-		'id_rubrique, id_parent, titre, statut, lang, langue_choisie',
-		'spip_rubriques',
-		($type == 'breve' ? ' id_parent=0 ' : ''),
-		'',
-		'0+titre,titre'
-	);
-	while ($r = sql_fetch($q)) {
-		if (autoriser('voir', 'rubrique', $r['id_rubrique'])) {
-			// titre largeur maxi a 50
-			$titre = couper(supprimer_tags(typo($r['titre'])) . ' ', 50);
-			if (
-				$GLOBALS['meta']['multi_rubriques'] == 'oui'
-				&& ($r['langue_choisie'] == 'oui' || $r['id_parent'] == 0)
-			) {
-				$titre .= ' [' . traduire_nom_langue($r['lang']) . ']';
-			}
-			$data[$r['id_rubrique']] = $titre;
-			$enfants[$r['id_parent']][] = $r['id_rubrique'];
-			if ($id_rubrique == $r['id_rubrique']) {
-				$id_parent = $r['id_parent'];
-			}
-		}
-	}
-
-	// si une seule rubrique comme choix possible,
-	// inutile de mettre le selecteur sur un choix vide par defaut
-	// sauf si le selecteur s'adresse a une rubrique puisque on peut la mettre a la racine dans ce cas
-	if (
-		count($data) == 2
-		&& isset($data[0])
-		&& !in_array($type, ['auteur', 'rubrique'])
-		&& !$id_rubrique
-	) {
-		unset($data[0]);
-	}
-
-
-	$opt = sous_menu_rubriques($id_rubrique, 0, 0, $data, $enfants, $idem, $restreint, $type);
-	$att = " id='id_parent' name='id_parent'\nclass='selecteur_parent verdana1'";
-
-	if (preg_match(',^<option[^<>]*value=.(\d*).[^<>]*>([^<]*)</option>$,', $opt, $r)) {
-		$r = "<input$att type='hidden' value='" . $r[1] . "' />" . $r[2];
-	} else {
-		$r = '<select' . $att . " size='1'>\n$opt</select>\n";
-	}
-
-	# message pour neuneus (a supprimer ?)
+    $enfants = [];
+    $data = [];
+    if ($type == 'rubrique' && autoriser('publierdans', 'rubrique', 0)) {
+        $data[0] = _T('info_racine_site');
+    }
+    # premier choix = neant
+    # si auteur (rubriques restreintes)
+    # ou si creation avec id_rubrique=0
+    elseif ($type == 'auteur' || !$id_rubrique) {
+        $data[0] = '&nbsp;';
+    }
+
+    //
+    // creer une structure contenant toute l'arborescence
+    //
+
+    include_spip('base/abstract_sql');
+    $q = sql_select(
+        'id_rubrique, id_parent, titre, statut, lang, langue_choisie',
+        'spip_rubriques',
+        ($type == 'breve' ? ' id_parent=0 ' : ''),
+        '',
+        '0+titre,titre'
+    );
+    while ($r = sql_fetch($q)) {
+        if (autoriser('voir', 'rubrique', $r['id_rubrique'])) {
+            // titre largeur maxi a 50
+            $titre = couper(supprimer_tags(typo($r['titre'])) . ' ', 50);
+            if (
+                $GLOBALS['meta']['multi_rubriques'] == 'oui'
+                && ($r['langue_choisie'] == 'oui' || $r['id_parent'] == 0)
+            ) {
+                $titre .= ' [' . traduire_nom_langue($r['lang']) . ']';
+            }
+            $data[$r['id_rubrique']] = $titre;
+            $enfants[$r['id_parent']][] = $r['id_rubrique'];
+            if ($id_rubrique == $r['id_rubrique']) {
+                $id_parent = $r['id_parent'];
+            }
+        }
+    }
+
+    // si une seule rubrique comme choix possible,
+    // inutile de mettre le selecteur sur un choix vide par defaut
+    // sauf si le selecteur s'adresse a une rubrique puisque on peut la mettre a la racine dans ce cas
+    if (
+        count($data) == 2
+        && isset($data[0])
+        && !in_array($type, ['auteur', 'rubrique'])
+        && !$id_rubrique
+    ) {
+        unset($data[0]);
+    }
+
+
+    $opt = sous_menu_rubriques($id_rubrique, 0, 0, $data, $enfants, $idem, $restreint, $type);
+    $att = " id='id_parent' name='id_parent'\nclass='selecteur_parent verdana1'";
+
+    if (preg_match(',^<option[^<>]*value=.(\d*).[^<>]*>([^<]*)</option>$,', $opt, $r)) {
+        $r = "<input$att type='hidden' value='" . $r[1] . "' />" . $r[2];
+    } else {
+        $r = '<select' . $att . " size='1'>\n$opt</select>\n";
+    }
+
+    # message pour neuneus (a supprimer ?)
 #	if ($type != 'auteur' AND $type != 'breve')
 #		$r .= "\n<br />"._T('texte_rappel_selection_champs');
 
-	return $r;
+    return $r;
 }
 
 /**
@@ -293,22 +293,22 @@ 
  */
 function selecteur_rubrique_ajax($id_rubrique, $type, $restreint, $idem = 0, $do = 'aff') {
 
-	if ($id_rubrique) {
-		$titre = sql_getfetsel('titre', 'spip_rubriques', 'id_rubrique=' . (int) $id_rubrique);
-	} else {
-		$titre = $type == 'auteur' ? '&nbsp;' : _T('info_racine_site');
-	}
+    if ($id_rubrique) {
+        $titre = sql_getfetsel('titre', 'spip_rubriques', 'id_rubrique=' . (int) $id_rubrique);
+    } else {
+        $titre = $type == 'auteur' ? '&nbsp;' : _T('info_racine_site');
+    }
 
-	$titre = str_replace('&amp;', '&', entites_html(textebrut(typo($titre))));
-	$init = " disabled='disabled' type='text' value=\"" . $titre . "\"\nstyle='width:300px;'";
+    $titre = str_replace('&amp;', '&', entites_html(textebrut(typo($titre))));
+    $init = " disabled='disabled' type='text' value=\"" . $titre . "\"\nstyle='width:300px;'";
 
-	$url = generer_url_ecrire('selectionner', "id=$id_rubrique&type=$type&do=$do"
-		. ($idem ? "&exclus=$idem" : '')
-		. ($restreint ? '' : '&racine=oui')
-		. (isset($GLOBALS['var_profile']) ? '&var_profile=1' : ''));
+    $url = generer_url_ecrire('selectionner', "id=$id_rubrique&type=$type&do=$do"
+        . ($idem ? "&exclus=$idem" : '')
+        . ($restreint ? '' : '&racine=oui')
+        . (isset($GLOBALS['var_profile']) ? '&var_profile=1' : ''));
 
 
-	return construire_selecteur($url, '', 'selection_rubrique', 'id_parent', $init, $id_rubrique);
+    return construire_selecteur($url, '', 'selection_rubrique', 'id_parent', $init, $id_rubrique);
 }
 
 /**
@@ -338,30 +338,30 @@ 
  *     Code HTML du sélecteur de rubrique AJAX
  **/
 function construire_selecteur($url, $js, $idom, $name, $init = '', $id = 0) {
-	$icone = (str_contains($idom, 'auteur')) ? 'auteur-24.png' : 'rechercher-20.png';
-	// si icone de recherche on embed le svg
-	$balise = ($icone === 'rechercher-20.png' ? chercher_filtre('balise_svg') : chercher_filtre('balise_img'));
-	$img_icone = $balise(chemin_image($icone), _T('titre_image_selecteur'));
-
-	return
-		"<div class='rubrique_actuelle'><a href='#' class='rubrique-search' role='button' style='display:inline-flex;vertical-align:middle;' onclick=\""
-		. $js
-		. " jQuery(this).toggleClass('toggled'); "
-		. "return charger_node_url_si_vide('"
-		. $url
-		. "', this.parentNode.nextSibling, this.nextSibling,'',event)\" title='" . attribut_html(_T('titre_image_selecteur')) . "'>"
-		. $img_icone
-		. "</a><img src='"
-		. chemin_image('loader.svg')
-		. "' class='loader' id='img_"
-		. $idom
-		. "'\nstyle='visibility: hidden;' alt='*' />"
-		. "<input id='titreparent' name='titreparent' class='text'"
-		. $init
-		. ' />'
-		. "<input type='hidden' id='$name' name='$name' value='"
-		. $id
-		. "' /><div class='nettoyeur'></div></div><div id='"
-		. $idom
-		. "'\nstyle='display: none;'></div>";
+    $icone = (str_contains($idom, 'auteur')) ? 'auteur-24.png' : 'rechercher-20.png';
+    // si icone de recherche on embed le svg
+    $balise = ($icone === 'rechercher-20.png' ? chercher_filtre('balise_svg') : chercher_filtre('balise_img'));
+    $img_icone = $balise(chemin_image($icone), _T('titre_image_selecteur'));
+
+    return
+        "<div class='rubrique_actuelle'><a href='#' class='rubrique-search' role='button' style='display:inline-flex;vertical-align:middle;' onclick=\""
+        . $js
+        . " jQuery(this).toggleClass('toggled'); "
+        . "return charger_node_url_si_vide('"
+        . $url
+        . "', this.parentNode.nextSibling, this.nextSibling,'',event)\" title='" . attribut_html(_T('titre_image_selecteur')) . "'>"
+        . $img_icone
+        . "</a><img src='"
+        . chemin_image('loader.svg')
+        . "' class='loader' id='img_"
+        . $idom
+        . "'\nstyle='visibility: hidden;' alt='*' />"
+        . "<input id='titreparent' name='titreparent' class='text'"
+        . $init
+        . ' />'
+        . "<input type='hidden' id='$name' name='$name' value='"
+        . $id
+        . "' /><div class='nettoyeur'></div></div><div id='"
+        . $idom
+        . "'\nstyle='display: none;'></div>";
 }

Spacing +9 added lines, -9 removed lines

@@ -142,7 +142,7 @@ 
 	if (isset($data[$root])) { # pas de racine sauf pour les rubriques
 	$r = "<option$selected value='$root' class='$class' style='$style'>$espace"
 			. $data[$root]
-			. '</option>' . "\n";
+			. '</option>'."\n";
 	} else {
 		$r = '';
 	}
@@ -170,7 +170,7 @@ 
 	}
 
 	// et voila le travail
-	return $r . $sous;
+	return $r.$sous;
 }
 
 /**
@@ -218,12 +218,12 @@ 
 	while ($r = sql_fetch($q)) {
 		if (autoriser('voir', 'rubrique', $r['id_rubrique'])) {
 			// titre largeur maxi a 50
-			$titre = couper(supprimer_tags(typo($r['titre'])) . ' ', 50);
+			$titre = couper(supprimer_tags(typo($r['titre'])).' ', 50);
 			if (
 				$GLOBALS['meta']['multi_rubriques'] == 'oui'
 				&& ($r['langue_choisie'] == 'oui' || $r['id_parent'] == 0)
 			) {
-				$titre .= ' [' . traduire_nom_langue($r['lang']) . ']';
+				$titre .= ' ['.traduire_nom_langue($r['lang']).']';
 			}
 			$data[$r['id_rubrique']] = $titre;
 			$enfants[$r['id_parent']][] = $r['id_rubrique'];
@@ -250,9 +250,9 @@ 
 	$att = " id='id_parent' name='id_parent'\nclass='selecteur_parent verdana1'";
 
 	if (preg_match(',^<option[^<>]*value=.(\d*).[^<>]*>([^<]*)</option>$,', $opt, $r)) {
-		$r = "<input$att type='hidden' value='" . $r[1] . "' />" . $r[2];
+		$r = "<input$att type='hidden' value='".$r[1]."' />".$r[2];
 	} else {
-		$r = '<select' . $att . " size='1'>\n$opt</select>\n";
+		$r = '<select'.$att." size='1'>\n$opt</select>\n";
 	}
 
 	# message pour neuneus (a supprimer ?)
@@ -294,13 +294,13 @@ 
 function selecteur_rubrique_ajax($id_rubrique, $type, $restreint, $idem = 0, $do = 'aff') {
 
 	if ($id_rubrique) {
-		$titre = sql_getfetsel('titre', 'spip_rubriques', 'id_rubrique=' . (int) $id_rubrique);
+		$titre = sql_getfetsel('titre', 'spip_rubriques', 'id_rubrique='.(int) $id_rubrique);
 	} else {
 		$titre = $type == 'auteur' ? '&nbsp;' : _T('info_racine_site');
 	}
 
 	$titre = str_replace('&amp;', '&', entites_html(textebrut(typo($titre))));
-	$init = " disabled='disabled' type='text' value=\"" . $titre . "\"\nstyle='width:300px;'";
+	$init = " disabled='disabled' type='text' value=\"".$titre."\"\nstyle='width:300px;'";
 
 	$url = generer_url_ecrire('selectionner', "id=$id_rubrique&type=$type&do=$do"
 		. ($idem ? "&exclus=$idem" : '')
@@ -349,7 +349,7 @@ 
 		. " jQuery(this).toggleClass('toggled'); "
 		. "return charger_node_url_si_vide('"
 		. $url
-		. "', this.parentNode.nextSibling, this.nextSibling,'',event)\" title='" . attribut_html(_T('titre_image_selecteur')) . "'>"
+		. "', this.parentNode.nextSibling, this.nextSibling,'',event)\" title='".attribut_html(_T('titre_image_selecteur'))."'>"
 		. $img_icone
 		. "</a><img src='"
 		. chemin_image('loader.svg')

ecrire/inc/completer_traduction.php

Indentation +38 added lines, -38 removed lines

@@ -16,7 +16,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -28,51 +28,51 @@ 
  * @return string Erreur éventuelle
  */
 function inc_completer_traduction_dist($objet, $id_objet, $id_trad) {
-	// dupliquer tous les liens sauf les auteurs : le nouvel auteur est celui qui traduit
-	// cf API editer_liens
-	include_spip('action/editer_liens');
-	objet_dupliquer_liens($objet, $id_trad, $id_objet, null, ['auteur']);
-	$_id_table = id_table_objet($objet);
+    // dupliquer tous les liens sauf les auteurs : le nouvel auteur est celui qui traduit
+    // cf API editer_liens
+    include_spip('action/editer_liens');
+    objet_dupliquer_liens($objet, $id_trad, $id_objet, null, ['auteur']);
+    $_id_table = id_table_objet($objet);
 
-	// recuperer le logo
-	$chercher_logo = charger_fonction('chercher_logo', 'inc');
-	include_spip('action/editer_logo');
-	foreach (['on', 'off'] as $etat) {
-		$logo = $chercher_logo($id_trad, $_id_table, $etat);
-		if ($logo && ($file = reset($logo))) {
-			logo_modifier($objet, $id_objet, $etat, $file);
-		}
-	}
+    // recuperer le logo
+    $chercher_logo = charger_fonction('chercher_logo', 'inc');
+    include_spip('action/editer_logo');
+    foreach (['on', 'off'] as $etat) {
+        $logo = $chercher_logo($id_trad, $_id_table, $etat);
+        if ($logo && ($file = reset($logo))) {
+            logo_modifier($objet, $id_objet, $etat, $file);
+        }
+    }
 
-	// dupliquer certains champs
-	$trouver_table = charger_fonction('trouver_table', 'base');
-	$desc = $trouver_table(table_objet_sql($objet));
-	$champs = $set = [];
+    // dupliquer certains champs
+    $trouver_table = charger_fonction('trouver_table', 'base');
+    $desc = $trouver_table(table_objet_sql($objet));
+    $champs = $set = [];
 
-	// un éventuel champ 'virtuel' (redirections)
-	if (!empty($desc['field']['virtuel'])) {
-		$champs[] = 'virtuel';
-	}
+    // un éventuel champ 'virtuel' (redirections)
+    if (!empty($desc['field']['virtuel'])) {
+        $champs[] = 'virtuel';
+    }
 
-	$data = sql_fetsel('*', $desc['table'], $_id_table . '=' . (int) $id_trad);
+    $data = sql_fetsel('*', $desc['table'], $_id_table . '=' . (int) $id_trad);
 
-	foreach ($champs as $c) {
-		$set[$c] = $data[$c];
-	}
+    foreach ($champs as $c) {
+        $set[$c] = $data[$c];
+    }
 
-	/*
+    /*
 	 * Le pipeline 'pre_edition' sera appelé avec l'action 'completer_traduction'.
 	 * Des plugins pourront ainsi compléter les champs d'un objet traduit lors d'une nouvelle traduction.
 	 */
-	$err = objet_modifier_champs(
-		$objet,
-		$id_objet,
-		[
-			'data' => $data,
-			'action' => 'completer_traduction',
-		],
-		$set
-	);
+    $err = objet_modifier_champs(
+        $objet,
+        $id_objet,
+        [
+            'data' => $data,
+            'action' => 'completer_traduction',
+        ],
+        $set
+    );
 
-	return $err;
+    return $err;
 }

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
 		$champs[] = 'virtuel';
 	}
 
-	$data = sql_fetsel('*', $desc['table'], $_id_table . '=' . (int) $id_trad);
+	$data = sql_fetsel('*', $desc['table'], $_id_table.'='.(int) $id_trad);
 
 	foreach ($champs as $c) {
 		$set[$c] = $data[$c];