spip / SPIP

ecrire/inc/boutons.php

Spacing +3 added lines, -3 removed lines

@@ -102,8 +102,8 @@ 
 			&& autoriser('onglet', "_$id")
 		) {
 			$onglets[$id] = new Bouton(
-				isset($infos['icone']) ? find_in_theme($infos['icone']) : '',  // icone
-				$infos['titre'],  // titre
+				isset($infos['icone']) ? find_in_theme($infos['icone']) : '', // icone
+				$infos['titre'], // titre
 				(isset($infos['action']) and $infos['action'])
 					? generer_url_ecrire(
 						$infos['action'],
@@ -142,5 +142,5 @@ 
 		$res .= onglet(_T($onglet->libelle), $url, $exec, $ongletCourant, $onglet->icone);
 	}
 
-	return !$res ? '' : (debut_onglet($class) . $res . fin_onglet());
+	return !$res ? '' : (debut_onglet($class).$res.fin_onglet());
 }

Indentation +33 added lines, -33 removed lines

@@ -19,7 +19,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -35,35 +35,35 @@ 
  */
 function definir_barre_onglets($script) {
 
-	$onglets = [];
-	$liste_onglets = [];
+    $onglets = [];
+    $liste_onglets = [];
 
-	// ajouter les onglets issus des plugin via paquet.xml
-	if (function_exists('onglets_plugins')) {
-		$liste_onglets = onglets_plugins();
-	}
+    // ajouter les onglets issus des plugin via paquet.xml
+    if (function_exists('onglets_plugins')) {
+        $liste_onglets = onglets_plugins();
+    }
 
 
-	foreach ($liste_onglets as $id => $infos) {
-		if (
-			($parent = $infos['parent'])
-			&& $parent == $script
-			&& autoriser('onglet', "_$id")
-		) {
-			$onglets[$id] = new Bouton(
-				isset($infos['icone']) ? find_in_theme($infos['icone']) : '',  // icone
-				$infos['titre'],  // titre
-				(isset($infos['action']) and $infos['action'])
-					? generer_url_ecrire(
-						$infos['action'],
-						(isset($infos['parametres']) and $infos['parametres']) ? $infos['parametres'] : ''
-					)
-					: null
-			);
-		}
-	}
+    foreach ($liste_onglets as $id => $infos) {
+        if (
+            ($parent = $infos['parent'])
+            && $parent == $script
+            && autoriser('onglet', "_$id")
+        ) {
+            $onglets[$id] = new Bouton(
+                isset($infos['icone']) ? find_in_theme($infos['icone']) : '',  // icone
+                $infos['titre'],  // titre
+                (isset($infos['action']) and $infos['action'])
+                    ? generer_url_ecrire(
+                        $infos['action'],
+                        (isset($infos['parametres']) and $infos['parametres']) ? $infos['parametres'] : ''
+                    )
+                    : null
+            );
+        }
+    }
 
-	return pipeline('ajouter_onglets', ['data' => $onglets, 'args' => $script]);
+    return pipeline('ajouter_onglets', ['data' => $onglets, 'args' => $script]);
 }
 
 /**
@@ -81,14 +81,14 @@ 
  * @return string
  */
 function barre_onglets($rubrique, $ongletCourant, $class = 'barre_onglet') {
-	include_spip('inc/presentation');
+    include_spip('inc/presentation');
 
-	$res = '';
+    $res = '';
 
-	foreach (definir_barre_onglets($rubrique) as $exec => $onglet) {
-		$url = $onglet->url ?: generer_url_ecrire($exec);
-		$res .= onglet(_T($onglet->libelle), $url, $exec, $ongletCourant, $onglet->icone);
-	}
+    foreach (definir_barre_onglets($rubrique) as $exec => $onglet) {
+        $url = $onglet->url ?: generer_url_ecrire($exec);
+        $res .= onglet(_T($onglet->libelle), $url, $exec, $ongletCourant, $onglet->icone);
+    }
 
-	return !$res ? '' : (debut_onglet($class) . $res . fin_onglet());
+    return !$res ? '' : (debut_onglet($class) . $res . fin_onglet());
 }

ecrire/inc/prepare_recherche.php

Spacing +4 added lines, -4 removed lines

@@ -82,12 +82,12 @@ 
 	$where_resultat_recent = sql_date_proche('maj', (0 - ($delai_fraicheur + 100)), ' SECOND');
 	if (!isset($cache[$serveur][$table][$recherche])) {
 		$hash_serv = ($serveur ? substr(md5($serveur), 0, 16) : '');
-		$hash = substr(md5($recherche . $table), 0, 16);
-		$where = "(resultats.recherche='$hash' AND resultats.table_objet=" . sql_quote($table) . " AND resultats.serveur='$hash_serv')";
+		$hash = substr(md5($recherche.$table), 0, 16);
+		$where = "(resultats.recherche='$hash' AND resultats.table_objet=".sql_quote($table)." AND resultats.serveur='$hash_serv')";
 		$row = sql_fetsel(
 			'recherche',
 			'spip_resultats AS resultats',
-			$where . " AND $where_resultat_recent",
+			$where." AND $where_resultat_recent",
 			'',
 			'',
 			'0,1'
@@ -196,7 +196,7 @@ 
 		}
 
 		foreach ($listes_ids as $p => $ids) {
-			$select .= "+$p*(" .
+			$select .= "+$p*(".
 				sql_in("$table.$primary", $ids, '', $serveur)
 				. ') ';
 		}

Indentation +139 added lines, -139 removed lines

@@ -16,12 +16,12 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/rechercher');
 if (!defined('_DELAI_CACHE_resultats')) {
-	define('_DELAI_CACHE_resultats', 600);
+    define('_DELAI_CACHE_resultats', 600);
 }
 
 /**
@@ -48,126 +48,126 @@ 
  * @return array
  */
 function inc_prepare_recherche_dist(
-	$recherche,
-	$table = 'articles',
-	$cond = false,
-	$serveur = '',
-	$modificateurs = [],
-	$primary = ''
+    $recherche,
+    $table = 'articles',
+    $cond = false,
+    $serveur = '',
+    $modificateurs = [],
+    $primary = ''
 ) {
-	$where = null;
-	$rows = null;
-	static $cache = [];
-	$delai_fraicheur = min(
-		\_DELAI_CACHE_resultats,
-		time() - ($GLOBALS['meta']['derniere_modif'] ?? 0)
-	);
-
-	// si recherche n'est pas dans le contexte, on va prendre en globals
-	// ca permet de faire des inclure simple.
-	if (!isset($recherche) && isset($GLOBALS['recherche'])) {
-		$recherche = $GLOBALS['recherche'];
-	}
-
-	// traiter le cas {recherche?}
-	if ($cond && !strlen($recherche)) {
-		return [
-			'0 as points' /* as points */, /* where */
-			''
-		];
-	}
-
-
-	$rechercher = false;
-
-	$where_resultat_recent = sql_date_proche('maj', (0 - ($delai_fraicheur + 100)), ' SECOND');
-	if (!isset($cache[$serveur][$table][$recherche])) {
-		$hash_serv = ($serveur ? substr(md5($serveur), 0, 16) : '');
-		$hash = substr(md5($recherche . $table), 0, 16);
-		$where = "(resultats.recherche='$hash' AND resultats.table_objet=" . sql_quote($table) . " AND resultats.serveur='$hash_serv')";
-		$row = sql_fetsel(
-			'recherche',
-			'spip_resultats AS resultats',
-			$where . " AND $where_resultat_recent",
-			'',
-			'',
-			'0,1'
-		);
-		if (!$row || (defined('_VAR_MODE') && _VAR_MODE == 'recalcul')) {
-			$rechercher = true;
-		}
-	}
-
-	// si on n'a pas encore traite les donnees dans une boucle precedente
-	if ($rechercher) {
-		//$tables = liste_des_champs();
-		$x = objet_type($table);
-		$points = recherche_en_base(
-			$recherche,
-			$x,
-			[
-				'score' => true,
-				'toutvoir' => true,
-				'jointures' => true
-			],
-			$serveur
-		);
-		// pas de résultat, pas de point
-		$points = $points[$x] ?? [];
-
-		// permettre aux plugins de modifier le resultat
-		$points = pipeline('prepare_recherche', [
-			'args' => [
-				'type' => $x,
-				'recherche' => $recherche,
-				'serveur' => $serveur,
-				'modificateurs' => $modificateurs
-			],
-			'data' => $points
-		]);
-
-		// supprimer les anciens resultats de cette recherche
-		// et les resultats trop vieux avec une marge
-		// pas de AS resultats dans un delete (mysql)
-		$whered = str_replace(
-			['resultats.recherche', 'resultats.table_objet', 'resultats.serveur'],
-			['recherche', 'table_objet', 'serveur'],
-			$where
-		);
-
-		sql_delete(
-			'spip_resultats',
-			"NOT($where_resultat_recent) OR ($whered)"
-		);
-
-		// inserer les resultats dans la table de cache des resultats
-		if (is_countable($points) ? count($points) : 0) {
-			$tab_couples = [];
-			foreach ($points as $id => $p) {
-				$tab_couples[] = [
-					'recherche' => $hash,
-					'id' => $id,
-					'points' => $p['score'],
-					'table_objet' => $table,
-					'serveur' => $hash_serv,
-				];
-			}
-			sql_insertq_multi('spip_resultats', $tab_couples, []);
-		}
-	}
-
-	if (!isset($cache[$serveur][$table][$recherche])) {
-		if (!$serveur) {
-			$cache[$serveur][$table][$recherche] = ['resultats.points AS points', $where];
-		} else {
-			if (sql_countsel('spip_resultats as resultats', $where)) {
-				$rows = sql_allfetsel('resultats.id,resultats.points', 'spip_resultats as resultats', $where);
-			}
-			$cache[$serveur][$table][$recherche] = generer_select_where_explicites($table, $primary, $rows, $serveur);
-		}
-	}
-
-	return $cache[$serveur][$table][$recherche];
+    $where = null;
+    $rows = null;
+    static $cache = [];
+    $delai_fraicheur = min(
+        \_DELAI_CACHE_resultats,
+        time() - ($GLOBALS['meta']['derniere_modif'] ?? 0)
+    );
+
+    // si recherche n'est pas dans le contexte, on va prendre en globals
+    // ca permet de faire des inclure simple.
+    if (!isset($recherche) && isset($GLOBALS['recherche'])) {
+        $recherche = $GLOBALS['recherche'];
+    }
+
+    // traiter le cas {recherche?}
+    if ($cond && !strlen($recherche)) {
+        return [
+            '0 as points' /* as points */, /* where */
+            ''
+        ];
+    }
+
+
+    $rechercher = false;
+
+    $where_resultat_recent = sql_date_proche('maj', (0 - ($delai_fraicheur + 100)), ' SECOND');
+    if (!isset($cache[$serveur][$table][$recherche])) {
+        $hash_serv = ($serveur ? substr(md5($serveur), 0, 16) : '');
+        $hash = substr(md5($recherche . $table), 0, 16);
+        $where = "(resultats.recherche='$hash' AND resultats.table_objet=" . sql_quote($table) . " AND resultats.serveur='$hash_serv')";
+        $row = sql_fetsel(
+            'recherche',
+            'spip_resultats AS resultats',
+            $where . " AND $where_resultat_recent",
+            '',
+            '',
+            '0,1'
+        );
+        if (!$row || (defined('_VAR_MODE') && _VAR_MODE == 'recalcul')) {
+            $rechercher = true;
+        }
+    }
+
+    // si on n'a pas encore traite les donnees dans une boucle precedente
+    if ($rechercher) {
+        //$tables = liste_des_champs();
+        $x = objet_type($table);
+        $points = recherche_en_base(
+            $recherche,
+            $x,
+            [
+                'score' => true,
+                'toutvoir' => true,
+                'jointures' => true
+            ],
+            $serveur
+        );
+        // pas de résultat, pas de point
+        $points = $points[$x] ?? [];
+
+        // permettre aux plugins de modifier le resultat
+        $points = pipeline('prepare_recherche', [
+            'args' => [
+                'type' => $x,
+                'recherche' => $recherche,
+                'serveur' => $serveur,
+                'modificateurs' => $modificateurs
+            ],
+            'data' => $points
+        ]);
+
+        // supprimer les anciens resultats de cette recherche
+        // et les resultats trop vieux avec une marge
+        // pas de AS resultats dans un delete (mysql)
+        $whered = str_replace(
+            ['resultats.recherche', 'resultats.table_objet', 'resultats.serveur'],
+            ['recherche', 'table_objet', 'serveur'],
+            $where
+        );
+
+        sql_delete(
+            'spip_resultats',
+            "NOT($where_resultat_recent) OR ($whered)"
+        );
+
+        // inserer les resultats dans la table de cache des resultats
+        if (is_countable($points) ? count($points) : 0) {
+            $tab_couples = [];
+            foreach ($points as $id => $p) {
+                $tab_couples[] = [
+                    'recherche' => $hash,
+                    'id' => $id,
+                    'points' => $p['score'],
+                    'table_objet' => $table,
+                    'serveur' => $hash_serv,
+                ];
+            }
+            sql_insertq_multi('spip_resultats', $tab_couples, []);
+        }
+    }
+
+    if (!isset($cache[$serveur][$table][$recherche])) {
+        if (!$serveur) {
+            $cache[$serveur][$table][$recherche] = ['resultats.points AS points', $where];
+        } else {
+            if (sql_countsel('spip_resultats as resultats', $where)) {
+                $rows = sql_allfetsel('resultats.id,resultats.points', 'spip_resultats as resultats', $where);
+            }
+            $cache[$serveur][$table][$recherche] = generer_select_where_explicites($table, $primary, $rows, $serveur);
+        }
+    }
+
+    return $cache[$serveur][$table][$recherche];
 }
 
 
@@ -183,22 +183,22 @@ 
  * @return array
  */
 function generer_select_where_explicites($table, $primary, $rows, $serveur) {
-	# calculer le {id_article IN()} et le {... as points}
-	if ($rows === []) {
-		return ["''", '0=1'];
-	} else {
-		$listes_ids = [];
-		$select = '0';
-		foreach ($rows as $r) {
-			$listes_ids[$r['points']][] = $r['id'];
-		}
-
-		foreach ($listes_ids as $p => $ids) {
-			$select .= "+$p*(" .
-				sql_in("$table.$primary", $ids, '', $serveur)
-				. ') ';
-		}
-
-		return ["$select AS points ", sql_in("$table.$primary", array_map('reset', $rows), '', $serveur)];
-	}
+    # calculer le {id_article IN()} et le {... as points}
+    if ($rows === []) {
+        return ["''", '0=1'];
+    } else {
+        $listes_ids = [];
+        $select = '0';
+        foreach ($rows as $r) {
+            $listes_ids[$r['points']][] = $r['id'];
+        }
+
+        foreach ($listes_ids as $p => $ids) {
+            $select .= "+$p*(" .
+                sql_in("$table.$primary", $ids, '', $serveur)
+                . ') ';
+        }
+
+        return ["$select AS points ", sql_in("$table.$primary", array_map('reset', $rows), '', $serveur)];
+    }
 }

ecrire/inc/lister_objets.php

Indentation +21 added lines, -21 removed lines

@@ -17,7 +17,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 
@@ -40,29 +40,29 @@ 
  *     Code HTML de la liste
  */
 function inc_lister_objets_dist($vue, $contexte = [], $force = false) {
-	$res = ''; // debug
-	if (!is_array($contexte)) {
-		return _L('$contexte doit etre un tableau dans inc/lister_objets');
-	}
+    $res = ''; // debug
+    if (!is_array($contexte)) {
+        return _L('$contexte doit etre un tableau dans inc/lister_objets');
+    }
 
-	$fond = "prive/objets/liste/$vue";
-	if (!find_in_path($fond . '.' . _EXTENSION_SQUELETTES)) {
-		// traiter les cas particuliers
-		include_spip('base/connect_sql');
-		$vue = table_objet($vue);
-		$fond = "prive/objets/liste/$vue";
-		if (!find_in_path($fond . '.' . _EXTENSION_SQUELETTES)) {
-			return _L("vue $vue introuvable pour lister les objets");
-		}
-	}
+    $fond = "prive/objets/liste/$vue";
+    if (!find_in_path($fond . '.' . _EXTENSION_SQUELETTES)) {
+        // traiter les cas particuliers
+        include_spip('base/connect_sql');
+        $vue = table_objet($vue);
+        $fond = "prive/objets/liste/$vue";
+        if (!find_in_path($fond . '.' . _EXTENSION_SQUELETTES)) {
+            return _L("vue $vue introuvable pour lister les objets");
+        }
+    }
 
 
-	$contexte['sinon'] = ($force ? $contexte['titre'] : '');
+    $contexte['sinon'] = ($force ? $contexte['titre'] : '');
 
-	$res = recuperer_fond($fond, $contexte, ['ajax' => true]);
-	if (_request('var_liste')) {
-		echo var_export($contexte, true);
-	}
+    $res = recuperer_fond($fond, $contexte, ['ajax' => true]);
+    if (_request('var_liste')) {
+        echo var_export($contexte, true);
+    }
 
-	return $res;
+    return $res;
 }

Spacing +2 added lines, -2 removed lines

@@ -46,12 +46,12 @@
 	}
 
 	$fond = "prive/objets/liste/$vue";
-	if (!find_in_path($fond . '.' . _EXTENSION_SQUELETTES)) {
+	if (!find_in_path($fond.'.'._EXTENSION_SQUELETTES)) {
 		// traiter les cas particuliers
 		include_spip('base/connect_sql');
 		$vue = table_objet($vue);
 		$fond = "prive/objets/liste/$vue";
-		if (!find_in_path($fond . '.' . _EXTENSION_SQUELETTES)) {
+		if (!find_in_path($fond.'.'._EXTENSION_SQUELETTES)) {
 			return _L("vue $vue introuvable pour lister les objets");
 		}
 	}

ecrire/inc/pipelines_ecrire.php

Spacing +5 added lines, -5 removed lines

@@ -93,7 +93,7 @@ 
 
 		$titre = '<title>['
 			. $nom_site_spip
-			. '] ' . $match
+			. '] '.$match
 			. '</title>';
 
 		$texte = substr_replace($texte, $titre, $p + 6, 0);
@@ -155,7 +155,7 @@ 
 			$flux['data']['texte'] = pipeline(
 				'affiche_droite',
 				['args' => $flux['args']['contexte'], 'data' => $flux['data']['texte']]
-			) . liste_objets_bloques(
+			).liste_objets_bloques(
 				$exec,
 				$flux['args']['contexte']
 			);
@@ -190,7 +190,7 @@ 
 				$flux['data']['texte'] = str_replace('<!--affiche_milieu-->', recuperer_fond(
 					'prive/objets/editer/traductions',
 					['objet' => $objet, 'id_objet' => $id, 'espace_prive' => 1]
-				) . '<!--affiche_milieu-->', $flux['data']['texte']);
+				).'<!--affiche_milieu-->', $flux['data']['texte']);
 				$flux['data']['texte'] = pipeline('afficher_fiche_objet', [
 					'args' => [
 						'contexte' => $flux['args']['contexte'],
@@ -258,7 +258,7 @@ 
 		$full_marqueur = "$ouvrir$marqueur$fermer";
 
 		// Le marqueur est absent : on l'ajoute avant l'élément indiqué
-		if ($marqueur_pos  === false) {
+		if ($marqueur_pos === false) {
 			$texte = preg_replace(
 				",$inserer_avant,",
 				"$full_marqueur\\0",
@@ -305,7 +305,7 @@ 
 		}
 	}
 	if ($res) {
-		$flux['data'] = $res . $flux['data'];
+		$flux['data'] = $res.$flux['data'];
 	}
 
 	return $flux;

Indentation +229 added lines, -229 removed lines

@@ -15,7 +15,7 @@ 
  * @package SPIP\Core\Pipelines
  **/
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 
@@ -34,40 +34,40 @@ 
  * @return string          Contenu complété des scripts javascripts, dont jQuery
  **/
 function f_jQuery_prive($texte) {
-	$x = '';
-	$jquery_plugins = pipeline(
-		'jquery_plugins',
-		[
-			'prive/javascript/jquery.js',
-			'prive/javascript/jquery.form.js',
-			'prive/javascript/jquery.autosave.js',
-			'prive/javascript/jquery.placeholder-label.js',
-			'prive/javascript/ajaxCallback.js',
-			'prive/javascript/js.cookie.js',
-			'prive/javascript/spip_barre.js',
-		]
-	);
-	foreach (array_unique($jquery_plugins) as $script) {
-		$script = supprimer_timestamp($script);
-		if (
-			(str_starts_with($script, _DIR_VAR) and file_exists($script))
-			or $script = find_in_path($script)
-		) {
-			$script = timestamp($script);
-			$x .= "\n<script src=\"$script\" type=\"text/javascript\"></script>\n";
-		}
-	}
-	// inserer avant le premier script externe ou a la fin
-	if (
-		preg_match(',<script[^><]*src=,', $texte, $match)
-		and $p = strpos($texte, (string) $match[0])
-	) {
-		$texte = substr_replace($texte, $x, $p, 0);
-	} else {
-		$texte .= $x;
-	}
+    $x = '';
+    $jquery_plugins = pipeline(
+        'jquery_plugins',
+        [
+            'prive/javascript/jquery.js',
+            'prive/javascript/jquery.form.js',
+            'prive/javascript/jquery.autosave.js',
+            'prive/javascript/jquery.placeholder-label.js',
+            'prive/javascript/ajaxCallback.js',
+            'prive/javascript/js.cookie.js',
+            'prive/javascript/spip_barre.js',
+        ]
+    );
+    foreach (array_unique($jquery_plugins) as $script) {
+        $script = supprimer_timestamp($script);
+        if (
+            (str_starts_with($script, _DIR_VAR) and file_exists($script))
+            or $script = find_in_path($script)
+        ) {
+            $script = timestamp($script);
+            $x .= "\n<script src=\"$script\" type=\"text/javascript\"></script>\n";
+        }
+    }
+    // inserer avant le premier script externe ou a la fin
+    if (
+        preg_match(',<script[^><]*src=,', $texte, $match)
+        and $p = strpos($texte, (string) $match[0])
+    ) {
+        $texte = substr_replace($texte, $x, $p, 0);
+    } else {
+        $texte .= $x;
+    }
 
-	return $texte;
+    return $texte;
 }
 
 
@@ -80,42 +80,42 @@ 
  * @return string
  */
 function affichage_final_prive_title_auto($texte) {
-	if (
-		strpos($texte, '<title>') === false
-		and
-		(preg_match(',<h1[^>]*>(.+)</h1>,Uims', $texte, $match)
-			or preg_match(',<h[23][^>]*>(.+)</h[23]>,Uims', $texte, $match))
-		and $match = textebrut(trim($match[1]))
-		and ($p = strpos($texte, '<head>')) !== false
-	) {
-		if (!$nom_site_spip = textebrut(typo($GLOBALS['meta']['nom_site']))) {
-			$nom_site_spip = _T('info_mon_site_spip');
-		}
+    if (
+        strpos($texte, '<title>') === false
+        and
+        (preg_match(',<h1[^>]*>(.+)</h1>,Uims', $texte, $match)
+            or preg_match(',<h[23][^>]*>(.+)</h[23]>,Uims', $texte, $match))
+        and $match = textebrut(trim($match[1]))
+        and ($p = strpos($texte, '<head>')) !== false
+    ) {
+        if (!$nom_site_spip = textebrut(typo($GLOBALS['meta']['nom_site']))) {
+            $nom_site_spip = _T('info_mon_site_spip');
+        }
 
-		$titre = '<title>['
-			. $nom_site_spip
-			. '] ' . $match
-			. '</title>';
+        $titre = '<title>['
+            . $nom_site_spip
+            . '] ' . $match
+            . '</title>';
 
-		$texte = substr_replace($texte, $titre, $p + 6, 0);
-	}
+        $texte = substr_replace($texte, $titre, $p + 6, 0);
+    }
 
-	return $texte;
+    return $texte;
 }
 
 
 // Fonction standard pour le pipeline 'boite_infos'
 function f_boite_infos($flux) {
-	$args = $flux['args'];
-	$type = $args['type'];
-	unset($args['row']);
-	if (!trouver_fond($type, 'prive/objets/infos/')) {
-		$type = 'objet';
-	}
-	$args['espace_prive'] = 1;
-	$flux['data'] .= recuperer_fond("prive/objets/infos/$type", $args);
+    $args = $flux['args'];
+    $type = $args['type'];
+    unset($args['row']);
+    if (!trouver_fond($type, 'prive/objets/infos/')) {
+        $type = 'objet';
+    }
+    $args['espace_prive'] = 1;
+    $flux['data'] .= recuperer_fond("prive/objets/infos/$type", $args);
 
-	return $flux;
+    return $flux;
 }
 
 
@@ -134,97 +134,97 @@ 
  * @return array Données du pipeline
  */
 function f_afficher_blocs_ecrire($flux) {
-	static $o = [];
-	if (is_string($fond = $flux['args']['fond'])) {
-		$exec = $flux['args']['contexte']['exec'] ?? _request('exec');
-		if (!isset($o[$exec])) {
-			$o[$exec] = trouver_objet_exec($exec);
-		}
-		// cas particulier
-		if ($exec == 'infos_perso') {
-			$flux['args']['contexte']['id_auteur'] = $GLOBALS['visiteur_session']['id_auteur'];
-		}
-		$typepage = ($flux['args']['contexte']['type-page'] ?? $exec);
-		if ($fond == "prive/squelettes/navigation/$typepage") {
-			$flux['data']['texte'] = pipeline(
-				'affiche_gauche',
-				['args' => $flux['args']['contexte'], 'data' => $flux['data']['texte']]
-			);
-		} elseif ($fond == "prive/squelettes/extra/$typepage") {
-			include_spip('inc/presentation_mini');
-			$flux['data']['texte'] = pipeline(
-				'affiche_droite',
-				['args' => $flux['args']['contexte'], 'data' => $flux['data']['texte']]
-			) . liste_objets_bloques(
-				$exec,
-				$flux['args']['contexte']
-			);
-		} elseif ($fond == "prive/squelettes/hierarchie/$typepage" and $o[$exec]) {
-			// id non defini sur les formulaire de nouveaux objets
-			$id = isset($flux['args']['contexte'][$o[$exec]['id_table_objet']]) ? intval($flux['args']['contexte'][$o[$exec]['id_table_objet']]) : 0;
-			$flux['data']['texte'] = pipeline(
-				'affiche_hierarchie',
-				['args' => ['objet' => $o[$exec]['type'], 'id_objet' => $id], 'data' => $flux['data']['texte']]
-			);
-		} elseif ($fond == "prive/squelettes/contenu/$typepage") {
-			// Préparation du marqueur affiche_milieu
-			// Si c'est la page d'un objet pas en édition, on l'encapsule dans un div
-			$est_page_objet = !empty($o[$exec]['type']);
-			$est_en_edition = (isset($o[$exec]['edition']) and $o[$exec]['edition'] === true);
-			$encapsuler_milieu = ($est_page_objet and !$est_en_edition);
-			$flux['data']['texte'] = afficher_blocs_ecrire_preparer_marqueur(
-				$flux['data']['texte'],
-				'<!--affiche_milieu-->',
-				'<div id=["\']wysiwyg',
-				$encapsuler_milieu ? '<div class="affiche_milieu">' : '',
-				$encapsuler_milieu ? '</div>' : ''
-			);
-			if (
-				$o[$exec]
-				and $objet = $o[$exec]['type']
-				and $o[$exec]['edition'] == false
-				and isset($flux['args']['contexte'][$o[$exec]['id_table_objet']])
-				and $id = intval($flux['args']['contexte'][$o[$exec]['id_table_objet']])
-			) {
-				// inserer le formulaire de traduction
-				$flux['data']['texte'] = str_replace('<!--affiche_milieu-->', recuperer_fond(
-					'prive/objets/editer/traductions',
-					['objet' => $objet, 'id_objet' => $id, 'espace_prive' => 1]
-				) . '<!--affiche_milieu-->', $flux['data']['texte']);
-				$flux['data']['texte'] = pipeline('afficher_fiche_objet', [
-					'args' => [
-						'contexte' => $flux['args']['contexte'],
-						'type' => $objet,
-						'id' => $id
-					],
-					'data' => $flux['data']['texte']
-				]);
-			}
-			$flux['data']['texte'] = pipeline(
-				'affiche_milieu',
-				['args' => $flux['args']['contexte'], 'data' => $flux['data']['texte']]
-			);
-		} elseif ($fond == 'prive/squelettes/inclure/pied') {
-			$flux['data']['texte'] = pipeline(
-				'affiche_pied',
-				['args' => $flux['args']['contexte'], 'data' => $flux['data']['texte']]
-			);
-		} elseif (
-			strncmp($fond, 'prive/objets/contenu/', 21) == 0
-			and $objet = basename($fond)
-			and $objet == substr($fond, 21)
-			and isset($o[$objet])
-			and $o[$objet]
-		) {
-			$id = intval($flux['args']['contexte'][$o[$exec]['id_table_objet']]);
-			$flux['data']['texte'] = pipeline('afficher_contenu_objet', [
-				'args' => ['type' => $objet, 'id_objet' => $id, 'contexte' => $flux['args']['contexte']],
-				'data' => $flux['data']['texte']
-			]);
-		}
-	}
+    static $o = [];
+    if (is_string($fond = $flux['args']['fond'])) {
+        $exec = $flux['args']['contexte']['exec'] ?? _request('exec');
+        if (!isset($o[$exec])) {
+            $o[$exec] = trouver_objet_exec($exec);
+        }
+        // cas particulier
+        if ($exec == 'infos_perso') {
+            $flux['args']['contexte']['id_auteur'] = $GLOBALS['visiteur_session']['id_auteur'];
+        }
+        $typepage = ($flux['args']['contexte']['type-page'] ?? $exec);
+        if ($fond == "prive/squelettes/navigation/$typepage") {
+            $flux['data']['texte'] = pipeline(
+                'affiche_gauche',
+                ['args' => $flux['args']['contexte'], 'data' => $flux['data']['texte']]
+            );
+        } elseif ($fond == "prive/squelettes/extra/$typepage") {
+            include_spip('inc/presentation_mini');
+            $flux['data']['texte'] = pipeline(
+                'affiche_droite',
+                ['args' => $flux['args']['contexte'], 'data' => $flux['data']['texte']]
+            ) . liste_objets_bloques(
+                $exec,
+                $flux['args']['contexte']
+            );
+        } elseif ($fond == "prive/squelettes/hierarchie/$typepage" and $o[$exec]) {
+            // id non defini sur les formulaire de nouveaux objets
+            $id = isset($flux['args']['contexte'][$o[$exec]['id_table_objet']]) ? intval($flux['args']['contexte'][$o[$exec]['id_table_objet']]) : 0;
+            $flux['data']['texte'] = pipeline(
+                'affiche_hierarchie',
+                ['args' => ['objet' => $o[$exec]['type'], 'id_objet' => $id], 'data' => $flux['data']['texte']]
+            );
+        } elseif ($fond == "prive/squelettes/contenu/$typepage") {
+            // Préparation du marqueur affiche_milieu
+            // Si c'est la page d'un objet pas en édition, on l'encapsule dans un div
+            $est_page_objet = !empty($o[$exec]['type']);
+            $est_en_edition = (isset($o[$exec]['edition']) and $o[$exec]['edition'] === true);
+            $encapsuler_milieu = ($est_page_objet and !$est_en_edition);
+            $flux['data']['texte'] = afficher_blocs_ecrire_preparer_marqueur(
+                $flux['data']['texte'],
+                '<!--affiche_milieu-->',
+                '<div id=["\']wysiwyg',
+                $encapsuler_milieu ? '<div class="affiche_milieu">' : '',
+                $encapsuler_milieu ? '</div>' : ''
+            );
+            if (
+                $o[$exec]
+                and $objet = $o[$exec]['type']
+                and $o[$exec]['edition'] == false
+                and isset($flux['args']['contexte'][$o[$exec]['id_table_objet']])
+                and $id = intval($flux['args']['contexte'][$o[$exec]['id_table_objet']])
+            ) {
+                // inserer le formulaire de traduction
+                $flux['data']['texte'] = str_replace('<!--affiche_milieu-->', recuperer_fond(
+                    'prive/objets/editer/traductions',
+                    ['objet' => $objet, 'id_objet' => $id, 'espace_prive' => 1]
+                ) . '<!--affiche_milieu-->', $flux['data']['texte']);
+                $flux['data']['texte'] = pipeline('afficher_fiche_objet', [
+                    'args' => [
+                        'contexte' => $flux['args']['contexte'],
+                        'type' => $objet,
+                        'id' => $id
+                    ],
+                    'data' => $flux['data']['texte']
+                ]);
+            }
+            $flux['data']['texte'] = pipeline(
+                'affiche_milieu',
+                ['args' => $flux['args']['contexte'], 'data' => $flux['data']['texte']]
+            );
+        } elseif ($fond == 'prive/squelettes/inclure/pied') {
+            $flux['data']['texte'] = pipeline(
+                'affiche_pied',
+                ['args' => $flux['args']['contexte'], 'data' => $flux['data']['texte']]
+            );
+        } elseif (
+            strncmp($fond, 'prive/objets/contenu/', 21) == 0
+            and $objet = basename($fond)
+            and $objet == substr($fond, 21)
+            and isset($o[$objet])
+            and $o[$objet]
+        ) {
+            $id = intval($flux['args']['contexte'][$o[$exec]['id_table_objet']]);
+            $flux['data']['texte'] = pipeline('afficher_contenu_objet', [
+                'args' => ['type' => $objet, 'id_objet' => $id, 'contexte' => $flux['args']['contexte']],
+                'data' => $flux['data']['texte']
+            ]);
+        }
+    }
 
-	return $flux;
+    return $flux;
 }
 
 /**
@@ -252,36 +252,36 @@ 
  */
 function afficher_blocs_ecrire_preparer_marqueur(?string $texte, string $marqueur, string $inserer_avant, string $ouvrir = '', string $fermer = ''): ?string {
 
-	if ($texte) {
-		$encapsuler = (($ouvrir and $fermer) ? true : false);
-		$marqueur_pos = strpos($texte, $marqueur);
-		$full_marqueur = "$ouvrir$marqueur$fermer";
+    if ($texte) {
+        $encapsuler = (($ouvrir and $fermer) ? true : false);
+        $marqueur_pos = strpos($texte, $marqueur);
+        $full_marqueur = "$ouvrir$marqueur$fermer";
 
-		// Le marqueur est absent : on l'ajoute avant l'élément indiqué
-		if ($marqueur_pos  === false) {
-			$texte = preg_replace(
-				",$inserer_avant,",
-				"$full_marqueur\\0",
-				$texte
-			);
-		// Le marqueur est présent mais pas encapsulé : on ajoute les balises ouvrantes et fermantes.
-		// Pour vérifier, on prend le texte précédent et on regarde si ça correspond à la balise ouvrante.
-		// Il ne faut donc aucun espace blanc en trop.
-		} elseif (
-			$marqueur_pos !== false
-			and $encapsuler
-			and substr($texte, $marqueur_pos - strlen($ouvrir), strlen($ouvrir)) !== $ouvrir
-		) {
-			$texte = substr_replace(
-				$texte,
-				$full_marqueur,
-				$marqueur_pos,
-				strlen($marqueur)
-			);
-		}
-	}
+        // Le marqueur est absent : on l'ajoute avant l'élément indiqué
+        if ($marqueur_pos  === false) {
+            $texte = preg_replace(
+                ",$inserer_avant,",
+                "$full_marqueur\\0",
+                $texte
+            );
+        // Le marqueur est présent mais pas encapsulé : on ajoute les balises ouvrantes et fermantes.
+        // Pour vérifier, on prend le texte précédent et on regarde si ça correspond à la balise ouvrante.
+        // Il ne faut donc aucun espace blanc en trop.
+        } elseif (
+            $marqueur_pos !== false
+            and $encapsuler
+            and substr($texte, $marqueur_pos - strlen($ouvrir), strlen($ouvrir)) !== $ouvrir
+        ) {
+            $texte = substr_replace(
+                $texte,
+                $full_marqueur,
+                $marqueur_pos,
+                strlen($marqueur)
+            );
+        }
+    }
 
-	return $texte;
+    return $texte;
 }
 
 /**
@@ -292,23 +292,23 @@ 
  * @return string
  */
 function f_queue_affiche_milieu($flux) {
-	$args = $flux['args'];
-	$res = '';
-	foreach ($args as $key => $arg) {
-		if (preg_match(',^id_,', $key) and is_numeric($arg) and $arg = intval($arg)) {
-			$objet = preg_replace(',^id_,', '', $key);
-			$res .= recuperer_fond(
-				'modeles/object_jobs_list',
-				['id_objet' => $arg, 'objet' => $objet, 'espace_prive' => 1],
-				['ajax' => true]
-			);
-		}
-	}
-	if ($res) {
-		$flux['data'] = $res . $flux['data'];
-	}
+    $args = $flux['args'];
+    $res = '';
+    foreach ($args as $key => $arg) {
+        if (preg_match(',^id_,', $key) and is_numeric($arg) and $arg = intval($arg)) {
+            $objet = preg_replace(',^id_,', '', $key);
+            $res .= recuperer_fond(
+                'modeles/object_jobs_list',
+                ['id_objet' => $arg, 'objet' => $objet, 'espace_prive' => 1],
+                ['ajax' => true]
+            );
+        }
+    }
+    if ($res) {
+        $flux['data'] = $res . $flux['data'];
+    }
 
-	return $flux;
+    return $flux;
 }
 
 /**
@@ -322,39 +322,39 @@ 
  * @return array|bool
  */
 function trouver_objet_exec(?string $exec) {
-	static $objet_exec = [];
-	if (!$exec) {
-		return false;
-	}
-	// cas particulier
-	if ($exec === 'infos_perso') {
-		$exec = 'auteur';
-		set_request('id_auteur', $GLOBALS['visiteur_session']['id_auteur']);
-	}
-	if (!isset($objet_exec[$exec])) {
-		$objet_exec[$exec] = false;
-		$infos = lister_tables_objets_sql();
-		foreach ($infos as $t => $info) {
-			if ($exec === $info['url_edit'] and $info['editable']) {
-				return $objet_exec[$exec] = [
-					'edition' => $exec == $info['url_voir'] ? '' : true,
-					'table_objet_sql' => $t,
-					'table' => $info['table_objet'],
-					'type' => $info['type'],
-					'id_table_objet' => id_table_objet($info['type'])
-				];
-			}
-			if ($exec === $info['url_voir']) {
-				return $objet_exec[$exec] = [
-					'edition' => false,
-					'table_objet_sql' => $t,
-					'table' => $info['table_objet'],
-					'type' => $info['type'],
-					'id_table_objet' => id_table_objet($info['type'])
-				];
-			}
-		}
-	}
+    static $objet_exec = [];
+    if (!$exec) {
+        return false;
+    }
+    // cas particulier
+    if ($exec === 'infos_perso') {
+        $exec = 'auteur';
+        set_request('id_auteur', $GLOBALS['visiteur_session']['id_auteur']);
+    }
+    if (!isset($objet_exec[$exec])) {
+        $objet_exec[$exec] = false;
+        $infos = lister_tables_objets_sql();
+        foreach ($infos as $t => $info) {
+            if ($exec === $info['url_edit'] and $info['editable']) {
+                return $objet_exec[$exec] = [
+                    'edition' => $exec == $info['url_voir'] ? '' : true,
+                    'table_objet_sql' => $t,
+                    'table' => $info['table_objet'],
+                    'type' => $info['type'],
+                    'id_table_objet' => id_table_objet($info['type'])
+                ];
+            }
+            if ($exec === $info['url_voir']) {
+                return $objet_exec[$exec] = [
+                    'edition' => false,
+                    'table_objet_sql' => $t,
+                    'table' => $info['table_objet'],
+                    'type' => $info['type'],
+                    'id_table_objet' => id_table_objet($info['type'])
+                ];
+            }
+        }
+    }
 
-	return $objet_exec[$exec];
+    return $objet_exec[$exec];
 }

ecrire/inc/admin.php

Spacing +11 added lines, -11 removed lines

@@ -55,7 +55,7 @@ 
 		if ($res) {
 			return $res;
 		}
-		spip_log("meta: $script " . print_r($_POST, true));
+		spip_log("meta: $script ".print_r($_POST, true));
 		ecrire_meta($script, serialize($_POST));
 	}
 
@@ -124,7 +124,7 @@ 
 	}
 	// on pourrait statuer automatiquement les webmestres a l'init d'une action auth par ftp ... ?
 
-	spip_log("admin $pref" . ($valeur ? ' (reprise)' : ' (init)'), $journal);
+	spip_log("admin $pref".($valeur ? ' (reprise)' : ' (init)'), $journal);
 
 	return '';
 }
@@ -142,7 +142,7 @@ 
 	if (autoriser('configurer')) {
 		return _DIR_TMP;
 	} else {
-		return _DIR_TRANSFERT . $GLOBALS['visiteur_session']['login'] . '/';
+		return _DIR_TRANSFERT.$GLOBALS['visiteur_session']['login'].'/';
 	}
 }
 
@@ -160,8 +160,8 @@ 
  *     Nom du fichier
  **/
 function fichier_admin($action, $pref = 'admin_') {
-	return $pref .
-	substr(md5($action . (time() & ~2047) . $GLOBALS['visiteur_session']['login']), 0, 10);
+	return $pref.
+	substr(md5($action.(time() & ~2047).$GLOBALS['visiteur_session']['login']), 0, 10);
 }
 
 /**
@@ -194,7 +194,7 @@ 
 	} else {
 		$dir = dir_admin();
 		$signal = fichier_admin($script);
-		if (@file_exists($dir . $signal)) {
+		if (@file_exists($dir.$signal)) {
 			spip_log("Action admin: $action");
 
 			return '';
@@ -212,7 +212,7 @@ 
 
 				return '';
 			}
-			$corps .= '<input type="hidden" name="validation_admin" value="' . $signal . '" />';
+			$corps .= '<input type="hidden" name="validation_admin" value="'.$signal.'" />';
 			$suivant = _T('bouton_valider');
 			$js = '';
 		} else {
@@ -227,7 +227,7 @@ 
 				. "</legend>\n<label for='fichier'>"
 				. _T('info_creer_repertoire')
 				. "</label>\n"
-				. "<span id='signal' class='formo'>" . $signal . '</span>'
+				. "<span id='signal' class='formo'>".$signal.'</span>'
 				. "<input type='hidden' id='fichier' name='fichier' value='"
 				. $signal
 				. "' />"
@@ -264,7 +264,7 @@ 
  *     Nom de l'action (en base) qui a été exécutée
  **/
 function fin_admin($action) {
-	$signal = dir_admin() . fichier_admin($action);
+	$signal = dir_admin().fichier_admin($action);
 	spip_unlink($signal);
 	if ($action != 'delete_all') {
 		effacer_meta($action);
@@ -292,8 +292,8 @@ 
 	include_spip('inc/filtres');
 	foreach (array_merge($_POST, $_GET) as $n => $c) {
 		if (!in_array($n, ['fichier', 'exec', 'validation_admin']) and !is_array($c)) {
-			$suite .= "\n<input type='hidden' name='" . spip_htmlspecialchars($n) . "' value='" .
-				entites_html($c) .
+			$suite .= "\n<input type='hidden' name='".spip_htmlspecialchars($n)."' value='".
+				entites_html($c).
 				"'  />";
 		}
 	}

Indentation +129 added lines, -129 removed lines

@@ -17,7 +17,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -45,29 +45,29 @@ 
  *     sinon code HTML de la page après le traitement effectué.
  **/
 function inc_admin_dist($script, $titre, $comment = '', $anonymous = false) {
-	$reprise = true;
-	if (
-		!isset($GLOBALS['meta'][$script])
-		or !isset($GLOBALS['meta']['admin'])
-	) {
-		$reprise = false;
-		$res = debut_admin($script, $titre, $comment);
-		if ($res) {
-			return $res;
-		}
-		spip_log("meta: $script " . print_r($_POST, true));
-		ecrire_meta($script, serialize($_POST));
-	}
+    $reprise = true;
+    if (
+        !isset($GLOBALS['meta'][$script])
+        or !isset($GLOBALS['meta']['admin'])
+    ) {
+        $reprise = false;
+        $res = debut_admin($script, $titre, $comment);
+        if ($res) {
+            return $res;
+        }
+        spip_log("meta: $script " . print_r($_POST, true));
+        ecrire_meta($script, serialize($_POST));
+    }
 
-	$res = admin_verifie_session($script, $anonymous);
-	if ($res) {
-		return $res;
-	}
-	$base = charger_fonction($script, 'base');
-	$base($titre, $reprise);
-	fin_admin($script);
+    $res = admin_verifie_session($script, $anonymous);
+    if ($res) {
+        return $res;
+    }
+    $base = charger_fonction($script, 'base');
+    $base($titre, $reprise);
+    fin_admin($script);
 
-	return '';
+    return '';
 }
 
 /**
@@ -98,35 +98,35 @@ 
  *     Code HTML si message d'erreur, '' sinon;
  */
 function admin_verifie_session($script, $anonymous = false) {
-	include_spip('base/abstract_sql');
-	$pref = sprintf('_%d_', $GLOBALS['visiteur_session']['id_auteur']);
-	$signal = fichier_admin($script, "$script$pref");
-	$valeur = sql_getfetsel('valeur', 'spip_meta', "nom='admin'");
-	if ($valeur === null) {
-		ecrire_meta('admin', $signal, 'non');
-	} else {
-		if (!$anonymous and ($valeur != $signal)) {
-			if (
-				!preg_match('/^(.*)_(\d+)_/', $GLOBALS['meta']['admin'], $l)
-				or intval($l[2]) != $GLOBALS['visiteur_session']['id_auteur']
-			) {
-				include_spip('inc/minipres');
-				spip_log("refus de lancer $script, priorite a $valeur");
+    include_spip('base/abstract_sql');
+    $pref = sprintf('_%d_', $GLOBALS['visiteur_session']['id_auteur']);
+    $signal = fichier_admin($script, "$script$pref");
+    $valeur = sql_getfetsel('valeur', 'spip_meta', "nom='admin'");
+    if ($valeur === null) {
+        ecrire_meta('admin', $signal, 'non');
+    } else {
+        if (!$anonymous and ($valeur != $signal)) {
+            if (
+                !preg_match('/^(.*)_(\d+)_/', $GLOBALS['meta']['admin'], $l)
+                or intval($l[2]) != $GLOBALS['visiteur_session']['id_auteur']
+            ) {
+                include_spip('inc/minipres');
+                spip_log("refus de lancer $script, priorite a $valeur");
 
-				return minipres(_T('info_travaux_texte'), '', ['status' => 503]);
-			}
-		}
-	}
-	$journal = 'spip';
-	if (autoriser('configurer')) {
-		// c'est une action webmestre, soit par ftp soit par statut webmestre
-		$journal = 'webmestre';
-	}
-	// on pourrait statuer automatiquement les webmestres a l'init d'une action auth par ftp ... ?
+                return minipres(_T('info_travaux_texte'), '', ['status' => 503]);
+            }
+        }
+    }
+    $journal = 'spip';
+    if (autoriser('configurer')) {
+        // c'est une action webmestre, soit par ftp soit par statut webmestre
+        $journal = 'webmestre';
+    }
+    // on pourrait statuer automatiquement les webmestres a l'init d'une action auth par ftp ... ?
 
-	spip_log("admin $pref" . ($valeur ? ' (reprise)' : ' (init)'), $journal);
+    spip_log("admin $pref" . ($valeur ? ' (reprise)' : ' (init)'), $journal);
 
-	return '';
+    return '';
 }
 
 /**
@@ -139,11 +139,11 @@ 
  *     Chemin du répertoire.
  **/
 function dir_admin() {
-	if (autoriser('configurer')) {
-		return _DIR_TMP;
-	} else {
-		return _DIR_TRANSFERT . $GLOBALS['visiteur_session']['login'] . '/';
-	}
+    if (autoriser('configurer')) {
+        return _DIR_TMP;
+    } else {
+        return _DIR_TRANSFERT . $GLOBALS['visiteur_session']['login'] . '/';
+    }
 }
 
 /**
@@ -160,8 +160,8 @@ 
  *     Nom du fichier
  **/
 function fichier_admin($action, $pref = 'admin_') {
-	return $pref .
-	substr(md5($action . (time() & ~2047) . $GLOBALS['visiteur_session']['login']), 0, 10);
+    return $pref .
+    substr(md5($action . (time() & ~2047) . $GLOBALS['visiteur_session']['login']), 0, 10);
 }
 
 /**
@@ -187,73 +187,73 @@ 
  *     sinon chaîne vide si déjà fait.
  **/
 function debut_admin($script, $action = '', $corps = '') {
-	if ((!$action) || !(autoriser('webmestre') or autoriser('chargerftp'))) {
-		include_spip('inc/minipres');
+    if ((!$action) || !(autoriser('webmestre') or autoriser('chargerftp'))) {
+        include_spip('inc/minipres');
 
-		return minipres();
-	} else {
-		$dir = dir_admin();
-		$signal = fichier_admin($script);
-		if (@file_exists($dir . $signal)) {
-			spip_log("Action admin: $action");
+        return minipres();
+    } else {
+        $dir = dir_admin();
+        $signal = fichier_admin($script);
+        if (@file_exists($dir . $signal)) {
+            spip_log("Action admin: $action");
 
-			return '';
-		}
-		include_spip('inc/minipres');
+            return '';
+        }
+        include_spip('inc/minipres');
 
-		// Si on est un super-admin, un bouton de validation suffit
-		// sauf dans les cas destroy
-		if (
-			(autoriser('webmestre') or $script === 'repair')
-			and $script != 'delete_all'
-		) {
-			if (_request('validation_admin') == $signal) {
-				spip_log("Action super-admin: $action");
+        // Si on est un super-admin, un bouton de validation suffit
+        // sauf dans les cas destroy
+        if (
+            (autoriser('webmestre') or $script === 'repair')
+            and $script != 'delete_all'
+        ) {
+            if (_request('validation_admin') == $signal) {
+                spip_log("Action super-admin: $action");
 
-				return '';
-			}
-			$corps .= '<input type="hidden" name="validation_admin" value="' . $signal . '" />';
-			$suivant = _T('bouton_valider');
-			$js = '';
-		} else {
-			// cet appel permet d'assurer un copier-coller du nom du repertoire a creer dans tmp (esj)
-			// l'insertion du script a cet endroit n'est pas xhtml licite
-			// mais evite de l'embarquer dans toutes les pages minipres
-			$corps .= http_script('', 'spip_barre.js');
+                return '';
+            }
+            $corps .= '<input type="hidden" name="validation_admin" value="' . $signal . '" />';
+            $suivant = _T('bouton_valider');
+            $js = '';
+        } else {
+            // cet appel permet d'assurer un copier-coller du nom du repertoire a creer dans tmp (esj)
+            // l'insertion du script a cet endroit n'est pas xhtml licite
+            // mais evite de l'embarquer dans toutes les pages minipres
+            $corps .= http_script('', 'spip_barre.js');
 
-			$corps .= '<fieldset><legend>'
-				. _T('info_authentification_ftp')
-				. aider('ftp_auth')
-				. "</legend>\n<label for='fichier'>"
-				. _T('info_creer_repertoire')
-				. "</label>\n"
-				. "<span id='signal' class='formo'>" . $signal . '</span>'
-				. "<input type='hidden' id='fichier' name='fichier' value='"
-				. $signal
-				. "' />"
-				. _T('info_creer_repertoire_2', ['repertoire' => joli_repertoire($dir)])
-				. '</fieldset>';
+            $corps .= '<fieldset><legend>'
+                . _T('info_authentification_ftp')
+                . aider('ftp_auth')
+                . "</legend>\n<label for='fichier'>"
+                . _T('info_creer_repertoire')
+                . "</label>\n"
+                . "<span id='signal' class='formo'>" . $signal . '</span>'
+                . "<input type='hidden' id='fichier' name='fichier' value='"
+                . $signal
+                . "' />"
+                . _T('info_creer_repertoire_2', ['repertoire' => joli_repertoire($dir)])
+                . '</fieldset>';
 
-			$suivant = _T('bouton_recharger_page');
+            $suivant = _T('bouton_recharger_page');
 
-			// code volontairement tordu:
-			// provoquer la copie dans le presse papier du nom du repertoire
-			// en remettant a vide le champ pour que ca marche aussi en cas
-			// de JavaScript inactif.
-			$js = " onload='var range=document.createRange(); var signal = document.getElementById(\"signal\"); var userSelection = window.getSelection(); range.setStart(signal,0); range.setEnd(signal,1); userSelection.addRange(range);'";
-		}
+            // code volontairement tordu:
+            // provoquer la copie dans le presse papier du nom du repertoire
+            // en remettant a vide le champ pour que ca marche aussi en cas
+            // de JavaScript inactif.
+            $js = " onload='var range=document.createRange(); var signal = document.getElementById(\"signal\"); var userSelection = window.getSelection(); range.setStart(signal,0); range.setEnd(signal,1); userSelection.addRange(range);'";
+        }
 
-		// admin/xxx correspond
-		// a exec/base_xxx de preference
-		// et exec/xxx sinon (compat)
-		if (tester_url_ecrire("base_$script")) {
-			$script = "base_$script";
-		}
-		$form = copy_request($script, $corps, $suivant);
-		$info_action = _T('info_action', ['action' => "$action"]);
+        // admin/xxx correspond
+        // a exec/base_xxx de preference
+        // et exec/xxx sinon (compat)
+        if (tester_url_ecrire("base_$script")) {
+            $script = "base_$script";
+        }
+        $form = copy_request($script, $corps, $suivant);
+        $info_action = _T('info_action', ['action' => "$action"]);
 
-		return minipres($info_action, $form, ['onload' => $js]);
-	}
+        return minipres($info_action, $form, ['onload' => $js]);
+    }
 }
 
 /**
@@ -264,13 +264,13 @@ 
  *     Nom de l'action (en base) qui a été exécutée
  **/
 function fin_admin($action) {
-	$signal = dir_admin() . fichier_admin($action);
-	spip_unlink($signal);
-	if ($action != 'delete_all') {
-		effacer_meta($action);
-		effacer_meta('admin');
-		spip_log("efface les meta admin et $action ");
-	}
+    $signal = dir_admin() . fichier_admin($action);
+    spip_unlink($signal);
+    if ($action != 'delete_all') {
+        effacer_meta($action);
+        effacer_meta('admin');
+        spip_log("efface les meta admin et $action ");
+    }
 }
 
 /**
@@ -289,14 +289,14 @@ 
  *     Code HTML du formulaire
  **/
 function copy_request($script, $suite, $submit = '') {
-	include_spip('inc/filtres');
-	foreach (array_merge($_POST, $_GET) as $n => $c) {
-		if (!in_array($n, ['fichier', 'exec', 'validation_admin']) and !is_array($c)) {
-			$suite .= "\n<input type='hidden' name='" . spip_htmlspecialchars($n) . "' value='" .
-				entites_html($c) .
-				"'  />";
-		}
-	}
+    include_spip('inc/filtres');
+    foreach (array_merge($_POST, $_GET) as $n => $c) {
+        if (!in_array($n, ['fichier', 'exec', 'validation_admin']) and !is_array($c)) {
+            $suite .= "\n<input type='hidden' name='" . spip_htmlspecialchars($n) . "' value='" .
+                entites_html($c) .
+                "'  />";
+        }
+    }
 
-	return generer_form_ecrire($script, $suite, '', $submit);
+    return generer_form_ecrire($script, $suite, '', $submit);
 }

ecrire/inc/flock.php

Spacing +18 added lines, -18 removed lines

@@ -122,7 +122,7 @@ 
 		$contenu = @gzfile($fichier);
 	}
 
-	return is_array($contenu) ? join('', $contenu) : (string)$contenu;
+	return is_array($contenu) ? join('', $contenu) : (string) $contenu;
 }
 
 
@@ -315,9 +315,9 @@ 
  */
 function ecrire_fichier_securise($fichier, $contenu, $ecrire_quand_meme = false, $truncate = true) {
 	if (substr($fichier, -4) !== '.php') {
-		spip_log('Erreur de programmation: ' . $fichier . ' doit finir par .php');
+		spip_log('Erreur de programmation: '.$fichier.' doit finir par .php');
 	}
-	$contenu = '<' . "?php die ('Acces interdit'); ?" . ">\n" . $contenu;
+	$contenu = '<'."?php die ('Acces interdit'); ?".">\n".$contenu;
 
 	return ecrire_fichier($fichier, $contenu, $ecrire_quand_meme, $truncate);
 }
@@ -330,7 +330,7 @@ 
  * @return bool
  */
 function ecrire_fichier_calcule_si_modifie($fichier, $contenu, $force = false, $use_copy = false) {
-	$fichier_tmp = $fichier . '.last';
+	$fichier_tmp = $fichier.'.last';
 	if (!ecrire_fichier($fichier_tmp, $contenu, true)) {
 		return false;
 	}
@@ -370,7 +370,7 @@ 
  */
 function lire_fichier_securise($fichier, &$contenu, $options = []) {
 	if ($res = lire_fichier($fichier, $contenu, $options)) {
-		$contenu = substr($contenu, strlen('<' . "?php die ('Acces interdit'); ?" . ">\n"));
+		$contenu = substr($contenu, strlen('<'."?php die ('Acces interdit'); ?".">\n"));
 	}
 
 	return $res;
@@ -540,7 +540,7 @@ 
 				$wait = 0;
 			}
 		}
-		spip_log('Probleme de configuration opcache.revalidate_freq ' . $duree . 's : on attend ' . $wait . 's', _LOG_INFO_IMPORTANTE);
+		spip_log('Probleme de configuration opcache.revalidate_freq '.$duree.'s : on attend '.$wait.'s', _LOG_INFO_IMPORTANTE);
 		if ($wait) {
 			sleep($duree + 1);
 		}
@@ -568,9 +568,9 @@ 
 		if ($item == '.' || $item == '..') {
 			continue;
 		}
-		if (!supprimer_repertoire($dir . '/' . $item)) {
-			@chmod($dir . '/' . $item, 0777);
-			if (!supprimer_repertoire($dir . '/' . $item)) {
+		if (!supprimer_repertoire($dir.'/'.$item)) {
+			@chmod($dir.'/'.$item, 0777);
+			if (!supprimer_repertoire($dir.'/'.$item)) {
 				return false;
 			}
 		};
@@ -615,7 +615,7 @@ 
 	if (!strlen($subdir)) {
 		$n = strrpos($base, '/');
 		if ($n === false) {
-			return $nobase ? '' : ($base . '/');
+			return $nobase ? '' : ($base.'/');
 		}
 		$subdir = substr($base, $n + 1);
 		$base = substr($base, 0, $n + 1);
@@ -625,14 +625,14 @@ 
 	}
 
 	$baseaff = $nobase ? '' : $base;
-	if (isset($dirs[$base . $subdir])) {
-		return $baseaff . $dirs[$base . $subdir];
+	if (isset($dirs[$base.$subdir])) {
+		return $baseaff.$dirs[$base.$subdir];
 	}
 
-	$path = $base . $subdir; # $path = 'IMG/distant/pdf' ou 'IMG/distant_pdf'
+	$path = $base.$subdir; # $path = 'IMG/distant/pdf' ou 'IMG/distant_pdf'
 
 	if (file_exists("$path/.ok")) {
-		return $baseaff . ($dirs[$base . $subdir] = "$subdir/");
+		return $baseaff.($dirs[$base.$subdir] = "$subdir/");
 	}
 
 	@mkdir($path, _SPIP_CHMOD);
@@ -642,7 +642,7 @@ 
 		@touch("$path/.ok");
 		spip_log("creation $base$subdir/");
 
-		return $baseaff . ($dirs[$base . $subdir] = "$subdir/");
+		return $baseaff.($dirs[$base.$subdir] = "$subdir/");
 	}
 
 	// en cas d'echec c'est peut etre tout simplement que le disque est plein :
@@ -652,10 +652,10 @@ 
 		return '';
 	}
 	if (!_DIR_RESTREINT) {
-		$base = preg_replace(',^' . _DIR_RACINE . ',', '', $base);
+		$base = preg_replace(',^'._DIR_RACINE.',', '', $base);
 	}
 	$base .= $subdir;
-	raler_fichier($base . '/.ok');
+	raler_fichier($base.'/.ok');
 }
 
 
@@ -728,7 +728,7 @@ 
 								$maxfiles - $nbfiles,
 								$recurs
 							);
-							$fichiers = array_merge((array)$beginning, (array)$end);
+							$fichiers = array_merge((array) $beginning, (array) $end);
 							$nbfiles = count($fichiers);
 						}
 					}

Braces +1 added lines, -2 removed lines

@@ -341,8 +341,7 @@
 	) {
 		if ($use_copy) {
 			@copy($fichier_tmp, $fichier);
-		}
-		else {
+		} else {
 			@rename($fichier_tmp, $fichier);
 		}
 		// eviter que PHP ne reserve le vieux timestamp

Indentation +414 added lines, -414 removed lines

@@ -16,12 +16,12 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 if (!defined('_TEST_FILE_EXISTS')) {
-	/** Permettre d'éviter des tests file_exists sur certains hébergeurs */
-	define('_TEST_FILE_EXISTS', preg_match(',(online|free)[.]fr$,', $_ENV['HTTP_HOST'] ?? ''));
+    /** Permettre d'éviter des tests file_exists sur certains hébergeurs */
+    define('_TEST_FILE_EXISTS', preg_match(',(online|free)[.]fr$,', $_ENV['HTTP_HOST'] ?? ''));
 }
 
 #define('_SPIP_LOCK_MODE',0); // ne pas utiliser de lock (deconseille)
@@ -29,7 +29,7 @@ 
 #define('_SPIP_LOCK_MODE',2); // utiliser le nfslock de spip
 
 if (_SPIP_LOCK_MODE == 2) {
-	include_spip('inc/nfslock');
+    include_spip('inc/nfslock');
 }
 
 $GLOBALS['liste_verrous'] = [];
@@ -52,24 +52,24 @@ 
  *     Ressource sur le fichier ouvert, sinon false.
  **/
 function spip_fopen_lock($fichier, $mode, $verrou) {
-	if (_SPIP_LOCK_MODE == 1) {
-		if ($fl = @fopen($fichier, $mode)) {
-			// verrou
-			@flock($fl, $verrou);
-		}
-
-		return $fl;
-	} elseif (_SPIP_LOCK_MODE == 2) {
-		if (($verrou = spip_nfslock($fichier)) && ($fl = @fopen($fichier, $mode))) {
-			$GLOBALS['liste_verrous'][$fl] = [$fichier, $verrou];
-
-			return $fl;
-		} else {
-			return false;
-		}
-	}
-
-	return @fopen($fichier, $mode);
+    if (_SPIP_LOCK_MODE == 1) {
+        if ($fl = @fopen($fichier, $mode)) {
+            // verrou
+            @flock($fl, $verrou);
+        }
+
+        return $fl;
+    } elseif (_SPIP_LOCK_MODE == 2) {
+        if (($verrou = spip_nfslock($fichier)) && ($fl = @fopen($fichier, $mode))) {
+            $GLOBALS['liste_verrous'][$fl] = [$fichier, $verrou];
+
+            return $fl;
+        } else {
+            return false;
+        }
+    }
+
+    return @fopen($fichier, $mode);
 }
 
 /**
@@ -84,14 +84,14 @@ 
  *     true si succès, false sinon.
  **/
 function spip_fclose_unlock($handle) {
-	if (_SPIP_LOCK_MODE == 1) {
-		@flock($handle, LOCK_UN);
-	} elseif (_SPIP_LOCK_MODE == 2) {
-		spip_nfsunlock(reset($GLOBALS['liste_verrous'][$handle]), end($GLOBALS['liste_verrous'][$handle]));
-		unset($GLOBALS['liste_verrous'][$handle]);
-	}
-
-	return @fclose($handle);
+    if (_SPIP_LOCK_MODE == 1) {
+        @flock($handle, LOCK_UN);
+    } elseif (_SPIP_LOCK_MODE == 2) {
+        spip_nfsunlock(reset($GLOBALS['liste_verrous'][$handle]), end($GLOBALS['liste_verrous'][$handle]));
+        unset($GLOBALS['liste_verrous'][$handle]);
+    }
+
+    return @fclose($handle);
 }
 
 
@@ -105,23 +105,23 @@ 
  *     Contenu du fichier
  **/
 function spip_file_get_contents($fichier) {
-	if (substr($fichier, -3) != '.gz') {
-		if (function_exists('file_get_contents')) {
-			// quand on est sous windows on ne sait pas si file_get_contents marche
-			// on essaye : si ca retourne du contenu alors c'est bon
-			// sinon on fait un file() pour avoir le coeur net
-			$contenu = @file_get_contents($fichier);
-			if (!$contenu and _OS_SERVEUR == 'windows') {
-				$contenu = @file($fichier);
-			}
-		} else {
-			$contenu = @file($fichier);
-		}
-	} else {
-		$contenu = @gzfile($fichier);
-	}
-
-	return is_array($contenu) ? join('', $contenu) : (string)$contenu;
+    if (substr($fichier, -3) != '.gz') {
+        if (function_exists('file_get_contents')) {
+            // quand on est sous windows on ne sait pas si file_get_contents marche
+            // on essaye : si ca retourne du contenu alors c'est bon
+            // sinon on fait un file() pour avoir le coeur net
+            $contenu = @file_get_contents($fichier);
+            if (!$contenu and _OS_SERVEUR == 'windows') {
+                $contenu = @file($fichier);
+            }
+        } else {
+            $contenu = @file($fichier);
+        }
+    } else {
+        $contenu = @gzfile($fichier);
+    }
+
+    return is_array($contenu) ? join('', $contenu) : (string)$contenu;
 }
 
 
@@ -146,48 +146,48 @@ 
  *     true si l'opération a réussie, false sinon.
  **/
 function lire_fichier($fichier, &$contenu, $options = []) {
-	$contenu = '';
-	// inutile car si le fichier n'existe pas, le lock va renvoyer false juste apres
-	// economisons donc les acces disque, sauf chez free qui rale pour un rien
-	if (_TEST_FILE_EXISTS and !@file_exists($fichier)) {
-		return false;
-	}
-
-	#spip_timer('lire_fichier');
-
-	// pas de @ sur spip_fopen_lock qui est silencieux de toute facon
-	if ($fl = spip_fopen_lock($fichier, 'r', LOCK_SH)) {
-		// lire le fichier avant tout
-		$contenu = spip_file_get_contents($fichier);
-
-		// le fichier a-t-il ete supprime par le locker ?
-		// on ne verifie que si la tentative de lecture a echoue
-		// pour discriminer un contenu vide d'un fichier absent
-		// et eviter un acces disque
-		if (!$contenu and !@file_exists($fichier)) {
-			spip_fclose_unlock($fl);
-
-			return false;
-		}
-
-		// liberer le verrou
-		spip_fclose_unlock($fl);
-
-		// Verifications
-		$ok = true;
-		if (isset($options['phpcheck']) and $options['phpcheck'] == 'oui') {
-			$ok &= (preg_match(",[?]>\n?$,", $contenu));
-		}
-
-		#spip_log("$fread $fichier ".spip_timer('lire_fichier'));
-		if (!$ok) {
-			spip_log("echec lecture $fichier");
-		}
-
-		return $ok;
-	}
-
-	return false;
+    $contenu = '';
+    // inutile car si le fichier n'existe pas, le lock va renvoyer false juste apres
+    // economisons donc les acces disque, sauf chez free qui rale pour un rien
+    if (_TEST_FILE_EXISTS and !@file_exists($fichier)) {
+        return false;
+    }
+
+    #spip_timer('lire_fichier');
+
+    // pas de @ sur spip_fopen_lock qui est silencieux de toute facon
+    if ($fl = spip_fopen_lock($fichier, 'r', LOCK_SH)) {
+        // lire le fichier avant tout
+        $contenu = spip_file_get_contents($fichier);
+
+        // le fichier a-t-il ete supprime par le locker ?
+        // on ne verifie que si la tentative de lecture a echoue
+        // pour discriminer un contenu vide d'un fichier absent
+        // et eviter un acces disque
+        if (!$contenu and !@file_exists($fichier)) {
+            spip_fclose_unlock($fl);
+
+            return false;
+        }
+
+        // liberer le verrou
+        spip_fclose_unlock($fl);
+
+        // Verifications
+        $ok = true;
+        if (isset($options['phpcheck']) and $options['phpcheck'] == 'oui') {
+            $ok &= (preg_match(",[?]>\n?$,", $contenu));
+        }
+
+        #spip_log("$fread $fichier ".spip_timer('lire_fichier'));
+        if (!$ok) {
+            spip_log("echec lecture $fichier");
+        }
+
+        return $ok;
+    }
+
+    return false;
 }
 
 
@@ -215,92 +215,92 @@ 
  **/
 function ecrire_fichier($fichier, $contenu, $ignorer_echec = false, $truncate = true) {
 
-	#spip_timer('ecrire_fichier');
-
-	// verrouiller le fichier destination
-	if ($fp = spip_fopen_lock($fichier, 'a', LOCK_EX)) {
-		// ecrire les donnees, compressees le cas echeant
-		// (on ouvre un nouveau pointeur sur le fichier, ce qui a l'avantage
-		// de le recreer si le locker qui nous precede l'avait supprime...)
-		if (substr($fichier, -3) == '.gz') {
-			$contenu = gzencode($contenu);
-		}
-		// si c'est une ecriture avec troncation , on fait plutot une ecriture complete a cote suivie unlink+rename
-		// pour etre sur d'avoir une operation atomique
-		// y compris en NFS : http://www.ietf.org/rfc/rfc1094.txt
-		// sauf sous wintruc ou ca ne marche pas
-		$ok = false;
-		if ($truncate and _OS_SERVEUR != 'windows') {
-			if (!function_exists('creer_uniqid')) {
-				include_spip('inc/acces');
-			}
-			$id = creer_uniqid();
-			// on ouvre un pointeur sur un fichier temporaire en ecriture +raz
-			if ($fp2 = spip_fopen_lock("$fichier.$id", 'w', LOCK_EX)) {
-				$s = @fputs($fp2, $contenu, $a = strlen($contenu));
-				$ok = ($s == $a);
-				spip_fclose_unlock($fp2);
-				spip_fclose_unlock($fp);
-				$fp = null;
-				// unlink direct et pas spip_unlink car on avait deja le verrou
-				// a priori pas besoin car rename ecrase la cible
-				// @unlink($fichier);
-				// le rename aussitot, atomique quand on est pas sous windows
-				// au pire on arrive en second en cas de concourance, et le rename echoue
-				// --> on a la version de l'autre process qui doit etre identique
-				@rename("$fichier.$id", $fichier);
-				// precaution en cas d'echec du rename
-				if (!_TEST_FILE_EXISTS or @file_exists("$fichier.$id")) {
-					@unlink("$fichier.$id");
-				}
-				if ($ok) {
-					$ok = file_exists($fichier);
-				}
-			} else // echec mais penser a fermer ..
-			{
-				spip_fclose_unlock($fp);
-				$fp = null;
-			}
-		}
-		// sinon ou si methode precedente a echoueee
-		// on se rabat sur la methode ancienne
-		if (!$ok and !is_null($fp)) {
-			// ici on est en ajout ou sous windows, cas desespere
-			if ($truncate) {
-				@ftruncate($fp, 0);
-			}
-			$s = @fputs($fp, $contenu, $a = strlen($contenu));
-
-			$ok = ($s == $a);
-			spip_fclose_unlock($fp);
-		}
-		// on tente une dernière fois file_put_contents
-		if (!$ok) {
-			$l = file_put_contents($fichier, $contenu, $truncate ? LOCK_EX : LOCK_EX | FILE_APPEND);
-			$ok = ($l === strlen($contenu));
-		}
-
-		// liberer le verrou et fermer le fichier
-		@chmod($fichier, _SPIP_CHMOD & 0666);
-		if ($ok) {
-			if (strpos($fichier, '.php') !== false) {
-				spip_clear_opcode_cache(realpath($fichier));
-			}
-
-			return $ok;
-		}
-	}
-
-	if (!$ignorer_echec) {
-		include_spip('inc/autoriser');
-		if (autoriser('chargerftp')) {
-			raler_fichier($fichier);
-		}
-		spip_unlink($fichier);
-	}
-	spip_log("Ecriture fichier $fichier impossible", _LOG_INFO_IMPORTANTE);
-
-	return false;
+    #spip_timer('ecrire_fichier');
+
+    // verrouiller le fichier destination
+    if ($fp = spip_fopen_lock($fichier, 'a', LOCK_EX)) {
+        // ecrire les donnees, compressees le cas echeant
+        // (on ouvre un nouveau pointeur sur le fichier, ce qui a l'avantage
+        // de le recreer si le locker qui nous precede l'avait supprime...)
+        if (substr($fichier, -3) == '.gz') {
+            $contenu = gzencode($contenu);
+        }
+        // si c'est une ecriture avec troncation , on fait plutot une ecriture complete a cote suivie unlink+rename
+        // pour etre sur d'avoir une operation atomique
+        // y compris en NFS : http://www.ietf.org/rfc/rfc1094.txt
+        // sauf sous wintruc ou ca ne marche pas
+        $ok = false;
+        if ($truncate and _OS_SERVEUR != 'windows') {
+            if (!function_exists('creer_uniqid')) {
+                include_spip('inc/acces');
+            }
+            $id = creer_uniqid();
+            // on ouvre un pointeur sur un fichier temporaire en ecriture +raz
+            if ($fp2 = spip_fopen_lock("$fichier.$id", 'w', LOCK_EX)) {
+                $s = @fputs($fp2, $contenu, $a = strlen($contenu));
+                $ok = ($s == $a);
+                spip_fclose_unlock($fp2);
+                spip_fclose_unlock($fp);
+                $fp = null;
+                // unlink direct et pas spip_unlink car on avait deja le verrou
+                // a priori pas besoin car rename ecrase la cible
+                // @unlink($fichier);
+                // le rename aussitot, atomique quand on est pas sous windows
+                // au pire on arrive en second en cas de concourance, et le rename echoue
+                // --> on a la version de l'autre process qui doit etre identique
+                @rename("$fichier.$id", $fichier);
+                // precaution en cas d'echec du rename
+                if (!_TEST_FILE_EXISTS or @file_exists("$fichier.$id")) {
+                    @unlink("$fichier.$id");
+                }
+                if ($ok) {
+                    $ok = file_exists($fichier);
+                }
+            } else // echec mais penser a fermer ..
+            {
+                spip_fclose_unlock($fp);
+                $fp = null;
+            }
+        }
+        // sinon ou si methode precedente a echoueee
+        // on se rabat sur la methode ancienne
+        if (!$ok and !is_null($fp)) {
+            // ici on est en ajout ou sous windows, cas desespere
+            if ($truncate) {
+                @ftruncate($fp, 0);
+            }
+            $s = @fputs($fp, $contenu, $a = strlen($contenu));
+
+            $ok = ($s == $a);
+            spip_fclose_unlock($fp);
+        }
+        // on tente une dernière fois file_put_contents
+        if (!$ok) {
+            $l = file_put_contents($fichier, $contenu, $truncate ? LOCK_EX : LOCK_EX | FILE_APPEND);
+            $ok = ($l === strlen($contenu));
+        }
+
+        // liberer le verrou et fermer le fichier
+        @chmod($fichier, _SPIP_CHMOD & 0666);
+        if ($ok) {
+            if (strpos($fichier, '.php') !== false) {
+                spip_clear_opcode_cache(realpath($fichier));
+            }
+
+            return $ok;
+        }
+    }
+
+    if (!$ignorer_echec) {
+        include_spip('inc/autoriser');
+        if (autoriser('chargerftp')) {
+            raler_fichier($fichier);
+        }
+        spip_unlink($fichier);
+    }
+    spip_log("Ecriture fichier $fichier impossible", _LOG_INFO_IMPORTANTE);
+
+    return false;
 }
 
 /**
@@ -320,12 +320,12 @@ 
  *     Écriture avec troncation ?
  */
 function ecrire_fichier_securise($fichier, $contenu, $ecrire_quand_meme = false, $truncate = true) {
-	if (substr($fichier, -4) !== '.php') {
-		spip_log('Erreur de programmation: ' . $fichier . ' doit finir par .php');
-	}
-	$contenu = '<' . "?php die ('Acces interdit'); ?" . ">\n" . $contenu;
+    if (substr($fichier, -4) !== '.php') {
+        spip_log('Erreur de programmation: ' . $fichier . ' doit finir par .php');
+    }
+    $contenu = '<' . "?php die ('Acces interdit'); ?" . ">\n" . $contenu;
 
-	return ecrire_fichier($fichier, $contenu, $ecrire_quand_meme, $truncate);
+    return ecrire_fichier($fichier, $contenu, $ecrire_quand_meme, $truncate);
 }
 
 
@@ -336,25 +336,25 @@ 
  * @return bool
  */
 function ecrire_fichier_calcule_si_modifie($fichier, $contenu, $force = false, $use_copy = false) {
-	$fichier_tmp = $fichier . '.last';
-	if (!ecrire_fichier($fichier_tmp, $contenu, true)) {
-		return false;
-	}
-	if (
-		$force
-		or !file_exists($fichier)
-		or md5_file($fichier) != md5_file($fichier_tmp)
-	) {
-		if ($use_copy) {
-			@copy($fichier_tmp, $fichier);
-		}
-		else {
-			@rename($fichier_tmp, $fichier);
-		}
-		// eviter que PHP ne reserve le vieux timestamp
-		clearstatcache(true, $fichier);
-	}
-	return true;
+    $fichier_tmp = $fichier . '.last';
+    if (!ecrire_fichier($fichier_tmp, $contenu, true)) {
+        return false;
+    }
+    if (
+        $force
+        or !file_exists($fichier)
+        or md5_file($fichier) != md5_file($fichier_tmp)
+    ) {
+        if ($use_copy) {
+            @copy($fichier_tmp, $fichier);
+        }
+        else {
+            @rename($fichier_tmp, $fichier);
+        }
+        // eviter que PHP ne reserve le vieux timestamp
+        clearstatcache(true, $fichier);
+    }
+    return true;
 }
 
 
@@ -375,11 +375,11 @@ 
  *     true si l'opération a réussie, false sinon.
  */
 function lire_fichier_securise($fichier, &$contenu, $options = []) {
-	if ($res = lire_fichier($fichier, $contenu, $options)) {
-		$contenu = substr($contenu, strlen('<' . "?php die ('Acces interdit'); ?" . ">\n"));
-	}
+    if ($res = lire_fichier($fichier, $contenu, $options)) {
+        $contenu = substr($contenu, strlen('<' . "?php die ('Acces interdit'); ?" . ">\n"));
+    }
 
-	return $res;
+    return $res;
 }
 
 /**
@@ -394,25 +394,25 @@ 
  *     Chemin du fichier
  **/
 function raler_fichier($fichier) {
-	if (!defined('_SPIP_ECRIRE_SCRIPT')) {
-		spip_initialisation_suite();
-	}
-	include_spip('inc/minipres');
-	$dir = dirname($fichier);
-	http_response_code(401);
-	echo minipres(_T('texte_inc_meta_2'), "<h4 style='color: red'>"
-		. _T('texte_inc_meta_1', ['fichier' => $fichier])
-		. " <a href='"
-		. generer_url_ecrire('install', "etape=chmod&test_dir=$dir")
-		. "'>"
-		. _T('texte_inc_meta_2')
-		. '</a> '
-		. _T(
-			'texte_inc_meta_3',
-			['repertoire' => joli_repertoire($dir)]
-		)
-		. "</h4>\n");
-	exit;
+    if (!defined('_SPIP_ECRIRE_SCRIPT')) {
+        spip_initialisation_suite();
+    }
+    include_spip('inc/minipres');
+    $dir = dirname($fichier);
+    http_response_code(401);
+    echo minipres(_T('texte_inc_meta_2'), "<h4 style='color: red'>"
+        . _T('texte_inc_meta_1', ['fichier' => $fichier])
+        . " <a href='"
+        . generer_url_ecrire('install', "etape=chmod&test_dir=$dir")
+        . "'>"
+        . _T('texte_inc_meta_2')
+        . '</a> '
+        . _T(
+            'texte_inc_meta_3',
+            ['repertoire' => joli_repertoire($dir)]
+        )
+        . "</h4>\n");
+    exit;
 }
 
 
@@ -427,14 +427,14 @@ 
  *     - true si récent, false sinon
  */
 function jeune_fichier($fichier, $n) {
-	if (!file_exists($fichier)) {
-		return false;
-	}
-	if (!$c = @filemtime($fichier)) {
-		return false;
-	}
-
-	return (time() - $n <= $c);
+    if (!file_exists($fichier)) {
+        return false;
+    }
+    if (!$c = @filemtime($fichier)) {
+        return false;
+    }
+
+    return (time() - $n <= $c);
 }
 
 /**
@@ -449,22 +449,22 @@ 
  *     - false si on n'arrive pas poser le verrou ou si la suppression échoue
  */
 function supprimer_fichier($fichier, $lock = true) {
-	if (!@file_exists($fichier)) {
-		return true;
-	}
-
-	if ($lock) {
-		// verrouiller le fichier destination
-		if (!$fp = spip_fopen_lock($fichier, 'a', LOCK_EX)) {
-			return false;
-		}
-
-		// liberer le verrou
-		spip_fclose_unlock($fp);
-	}
-
-	// supprimer
-	return @unlink($fichier);
+    if (!@file_exists($fichier)) {
+        return true;
+    }
+
+    if ($lock) {
+        // verrouiller le fichier destination
+        if (!$fp = spip_fopen_lock($fichier, 'a', LOCK_EX)) {
+            return false;
+        }
+
+        // liberer le verrou
+        spip_fclose_unlock($fp);
+    }
+
+    // supprimer
+    return @unlink($fichier);
 }
 
 /**
@@ -474,12 +474,12 @@ 
  *     Chemin du fichier
  */
 function spip_unlink($f) {
-	if (!is_dir($f)) {
-		supprimer_fichier($f, false);
-	} else {
-		@unlink("$f/.ok");
-		@rmdir($f);
-	}
+    if (!is_dir($f)) {
+        supprimer_fichier($f, false);
+    } else {
+        @unlink("$f/.ok");
+        @rmdir($f);
+    }
 }
 
 /**
@@ -493,26 +493,26 @@ 
  *   The absolute path of the PHP file to invalidate.
  */
 function spip_clear_opcode_cache($filepath) {
-	clearstatcache(true, $filepath);
-
-	// Zend OPcache
-	if (function_exists('opcache_invalidate')) {
-		$invalidate = @opcache_invalidate($filepath, true);
-		// si l'invalidation a echoue lever un flag
-		if (!$invalidate and !defined('_spip_attend_invalidation_opcode_cache')) {
-			define('_spip_attend_invalidation_opcode_cache', true);
-		}
-	} elseif (!defined('_spip_attend_invalidation_opcode_cache')) {
-		// n'agira que si opcache est effectivement actif (il semble qu'on a pas toujours la fonction opcache_invalidate)
-		define('_spip_attend_invalidation_opcode_cache', true);
-	}
-	// APC.
-	if (function_exists('apc_delete_file')) {
-		// apc_delete_file() throws a PHP warning in case the specified file was
-		// not compiled yet.
-		// @see http://php.net/apc-delete-file
-		@apc_delete_file($filepath);
-	}
+    clearstatcache(true, $filepath);
+
+    // Zend OPcache
+    if (function_exists('opcache_invalidate')) {
+        $invalidate = @opcache_invalidate($filepath, true);
+        // si l'invalidation a echoue lever un flag
+        if (!$invalidate and !defined('_spip_attend_invalidation_opcode_cache')) {
+            define('_spip_attend_invalidation_opcode_cache', true);
+        }
+    } elseif (!defined('_spip_attend_invalidation_opcode_cache')) {
+        // n'agira que si opcache est effectivement actif (il semble qu'on a pas toujours la fonction opcache_invalidate)
+        define('_spip_attend_invalidation_opcode_cache', true);
+    }
+    // APC.
+    if (function_exists('apc_delete_file')) {
+        // apc_delete_file() throws a PHP warning in case the specified file was
+        // not compiled yet.
+        // @see http://php.net/apc-delete-file
+        @apc_delete_file($filepath);
+    }
 }
 
 /**
@@ -535,25 +535,25 @@ 
  *
  */
 function spip_attend_invalidation_opcode_cache($timestamp = null) {
-	if (
-		function_exists('opcache_get_configuration')
-		and @ini_get('opcache.enable')
-		and @ini_get('opcache.validate_timestamps')
-		and ($duree = intval(@ini_get('opcache.revalidate_freq')) or $duree = 2)
-		and defined('_spip_attend_invalidation_opcode_cache') // des invalidations ont echouees
-	) {
-		$wait = $duree + 1;
-		if ($timestamp) {
-			$wait -= (time() - $timestamp);
-			if ($wait < 0) {
-				$wait = 0;
-			}
-		}
-		spip_log('Probleme de configuration opcache.revalidate_freq ' . $duree . 's : on attend ' . $wait . 's', _LOG_INFO_IMPORTANTE);
-		if ($wait) {
-			sleep($duree + 1);
-		}
-	}
+    if (
+        function_exists('opcache_get_configuration')
+        and @ini_get('opcache.enable')
+        and @ini_get('opcache.validate_timestamps')
+        and ($duree = intval(@ini_get('opcache.revalidate_freq')) or $duree = 2)
+        and defined('_spip_attend_invalidation_opcode_cache') // des invalidations ont echouees
+    ) {
+        $wait = $duree + 1;
+        if ($timestamp) {
+            $wait -= (time() - $timestamp);
+            if ($wait < 0) {
+                $wait = 0;
+            }
+        }
+        spip_log('Probleme de configuration opcache.revalidate_freq ' . $duree . 's : on attend ' . $wait . 's', _LOG_INFO_IMPORTANTE);
+        if ($wait) {
+            sleep($duree + 1);
+        }
+    }
 }
 
 
@@ -566,26 +566,26 @@ 
  * @return bool Suppression reussie.
  */
 function supprimer_repertoire($dir) {
-	if (!file_exists($dir)) {
-		return true;
-	}
-	if (!is_dir($dir) || is_link($dir)) {
-		return @unlink($dir);
-	}
-
-	foreach (scandir($dir) as $item) {
-		if ($item == '.' || $item == '..') {
-			continue;
-		}
-		if (!supprimer_repertoire($dir . '/' . $item)) {
-			@chmod($dir . '/' . $item, 0777);
-			if (!supprimer_repertoire($dir . '/' . $item)) {
-				return false;
-			}
-		};
-	}
-
-	return @rmdir($dir);
+    if (!file_exists($dir)) {
+        return true;
+    }
+    if (!is_dir($dir) || is_link($dir)) {
+        return @unlink($dir);
+    }
+
+    foreach (scandir($dir) as $item) {
+        if ($item == '.' || $item == '..') {
+            continue;
+        }
+        if (!supprimer_repertoire($dir . '/' . $item)) {
+            @chmod($dir . '/' . $item, 0777);
+            if (!supprimer_repertoire($dir . '/' . $item)) {
+                return false;
+            }
+        };
+    }
+
+    return @rmdir($dir);
 }
 
 
@@ -614,57 +614,57 @@ 
  *     Chemin du répertoire créé.
  **/
 function sous_repertoire($base, $subdir = '', $nobase = false, $tantpis = false) {
-	static $dirs = [];
-
-	$base = str_replace('//', '/', $base);
-
-	# suppr le dernier caractere si c'est un /
-	$base = rtrim($base, '/');
-
-	if (!strlen($subdir)) {
-		$n = strrpos($base, '/');
-		if ($n === false) {
-			return $nobase ? '' : ($base . '/');
-		}
-		$subdir = substr($base, $n + 1);
-		$base = substr($base, 0, $n + 1);
-	} else {
-		$base .= '/';
-		$subdir = str_replace('/', '', $subdir);
-	}
-
-	$baseaff = $nobase ? '' : $base;
-	if (isset($dirs[$base . $subdir])) {
-		return $baseaff . $dirs[$base . $subdir];
-	}
-
-	$path = $base . $subdir; # $path = 'IMG/distant/pdf' ou 'IMG/distant_pdf'
-
-	if (file_exists("$path/.ok")) {
-		return $baseaff . ($dirs[$base . $subdir] = "$subdir/");
-	}
-
-	@mkdir($path, _SPIP_CHMOD);
-	@chmod($path, _SPIP_CHMOD);
-
-	if (is_dir($path) && is_writable($path)) {
-		@touch("$path/.ok");
-		spip_log("creation $base$subdir/");
-
-		return $baseaff . ($dirs[$base . $subdir] = "$subdir/");
-	}
-
-	// en cas d'echec c'est peut etre tout simplement que le disque est plein :
-	// l'inode du fichier dir_test existe, mais impossible d'y mettre du contenu
-	spip_log("echec creation $base{$subdir}");
-	if ($tantpis) {
-		return '';
-	}
-	if (!_DIR_RESTREINT) {
-		$base = preg_replace(',^' . _DIR_RACINE . ',', '', $base);
-	}
-	$base .= $subdir;
-	raler_fichier($base . '/.ok');
+    static $dirs = [];
+
+    $base = str_replace('//', '/', $base);
+
+    # suppr le dernier caractere si c'est un /
+    $base = rtrim($base, '/');
+
+    if (!strlen($subdir)) {
+        $n = strrpos($base, '/');
+        if ($n === false) {
+            return $nobase ? '' : ($base . '/');
+        }
+        $subdir = substr($base, $n + 1);
+        $base = substr($base, 0, $n + 1);
+    } else {
+        $base .= '/';
+        $subdir = str_replace('/', '', $subdir);
+    }
+
+    $baseaff = $nobase ? '' : $base;
+    if (isset($dirs[$base . $subdir])) {
+        return $baseaff . $dirs[$base . $subdir];
+    }
+
+    $path = $base . $subdir; # $path = 'IMG/distant/pdf' ou 'IMG/distant_pdf'
+
+    if (file_exists("$path/.ok")) {
+        return $baseaff . ($dirs[$base . $subdir] = "$subdir/");
+    }
+
+    @mkdir($path, _SPIP_CHMOD);
+    @chmod($path, _SPIP_CHMOD);
+
+    if (is_dir($path) && is_writable($path)) {
+        @touch("$path/.ok");
+        spip_log("creation $base$subdir/");
+
+        return $baseaff . ($dirs[$base . $subdir] = "$subdir/");
+    }
+
+    // en cas d'echec c'est peut etre tout simplement que le disque est plein :
+    // l'inode du fichier dir_test existe, mais impossible d'y mettre du contenu
+    spip_log("echec creation $base{$subdir}");
+    if ($tantpis) {
+        return '';
+    }
+    if (!_DIR_RESTREINT) {
+        $base = preg_replace(',^' . _DIR_RACINE . ',', '', $base);
+    }
+    $base .= $subdir;
+    raler_fichier($base . '/.ok');
 }
 
 
@@ -697,56 +697,56 @@ 
  *     Chemins des fichiers trouvés.
  **/
 function preg_files($dir, $pattern = -1 /* AUTO */, $maxfiles = 10000, $recurs = []) {
-	$nbfiles = 0;
-	if ($pattern == -1) {
-		$pattern = '';
-	}
-	$fichiers = [];
-	// revenir au repertoire racine si on a recu dossier/truc
-	// pour regarder dossier/truc/ ne pas oublier le / final
-	$dir = preg_replace(',/[^/]*$,', '', $dir);
-	if ($dir == '') {
-		$dir = '.';
-	}
-
-	if (@is_dir($dir) and is_readable($dir) and $d = opendir($dir)) {
-		while (($f = readdir($d)) !== false && ($nbfiles < $maxfiles)) {
-			if (
-				$f[0] != '.' # ignorer . .. .svn etc
-				and $f != 'CVS'
-				and $f != 'remove.txt'
-				and is_readable($f = "$dir/$f")
-			) {
-				if (is_file($f)) {
-					if (!$pattern or preg_match(";$pattern;iS", $f)) {
-						$fichiers[] = $f;
-						$nbfiles++;
-					}
-				} else {
-					if (is_dir($f) and is_array($recurs)) {
-						$rp = @realpath($f);
-						if (!is_string($rp) or !strlen($rp)) {
-							$rp = $f;
-						} # realpath n'est peut etre pas autorise
-						if (!isset($recurs[$rp])) {
-							$recurs[$rp] = true;
-							$beginning = $fichiers;
-							$end = preg_files(
-								"$f/",
-								$pattern,
-								$maxfiles - $nbfiles,
-								$recurs
-							);
-							$fichiers = array_merge((array)$beginning, (array)$end);
-							$nbfiles = count($fichiers);
-						}
-					}
-				}
-			}
-		}
-		closedir($d);
-	}
-	sort($fichiers);
-
-	return $fichiers;
+    $nbfiles = 0;
+    if ($pattern == -1) {
+        $pattern = '';
+    }
+    $fichiers = [];
+    // revenir au repertoire racine si on a recu dossier/truc
+    // pour regarder dossier/truc/ ne pas oublier le / final
+    $dir = preg_replace(',/[^/]*$,', '', $dir);
+    if ($dir == '') {
+        $dir = '.';
+    }
+
+    if (@is_dir($dir) and is_readable($dir) and $d = opendir($dir)) {
+        while (($f = readdir($d)) !== false && ($nbfiles < $maxfiles)) {
+            if (
+                $f[0] != '.' # ignorer . .. .svn etc
+                and $f != 'CVS'
+                and $f != 'remove.txt'
+                and is_readable($f = "$dir/$f")
+            ) {
+                if (is_file($f)) {
+                    if (!$pattern or preg_match(";$pattern;iS", $f)) {
+                        $fichiers[] = $f;
+                        $nbfiles++;
+                    }
+                } else {
+                    if (is_dir($f) and is_array($recurs)) {
+                        $rp = @realpath($f);
+                        if (!is_string($rp) or !strlen($rp)) {
+                            $rp = $f;
+                        } # realpath n'est peut etre pas autorise
+                        if (!isset($recurs[$rp])) {
+                            $recurs[$rp] = true;
+                            $beginning = $fichiers;
+                            $end = preg_files(
+                                "$f/",
+                                $pattern,
+                                $maxfiles - $nbfiles,
+                                $recurs
+                            );
+                            $fichiers = array_merge((array)$beginning, (array)$end);
+                            $nbfiles = count($fichiers);
+                        }
+                    }
+                }
+            }
+        }
+        closedir($d);
+    }
+    sort($fichiers);
+
+    return $fichiers;
 }

ecrire/inc/couleurs.php

Indentation +63 added lines, -63 removed lines

@@ -17,7 +17,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -38,70 +38,70 @@ 
  * @return array|string
  */
 function inc_couleurs_dist($choix = null, $ajouter = false) {
-	static $couleurs_spip = [
-		// Violet soutenu
-		9 => ['couleur_theme' => '#9a6ef2'],
-		// Violet rosé
-		4 => ['couleur_theme' => '#c464cb'],
-		// Rose interface SPIP
-		2 => ['couleur_theme' =>  '#F02364'],
-		// Rouge
-		8 => ['couleur_theme' => '#ff4524'],
-		// Orange
-		3 => ['couleur_theme' => '#c97500'],
-		// Vert SPIP
-		1 => ['couleur_theme' => '#9dba00'],
-		// Vert Troglo
-		7 => ['couleur_theme' => '#419a2c'],
-		// Bleu-vert
-		12 => ['couleur_theme' => '#269681'],
-		//  Bleu pastel
-		5 => ['couleur_theme' => '#3190ae'],
-		//  Bleu Kermesse
-		11 => ['couleur_theme' => '#288bdd'],
-		//  Gris bleuté
-		6 => ['couleur_theme' => '#7d90a2'],
-		//  Gris
-		10 => ['couleur_theme' => '#909090'],
-	];
+    static $couleurs_spip = [
+        // Violet soutenu
+        9 => ['couleur_theme' => '#9a6ef2'],
+        // Violet rosé
+        4 => ['couleur_theme' => '#c464cb'],
+        // Rose interface SPIP
+        2 => ['couleur_theme' =>  '#F02364'],
+        // Rouge
+        8 => ['couleur_theme' => '#ff4524'],
+        // Orange
+        3 => ['couleur_theme' => '#c97500'],
+        // Vert SPIP
+        1 => ['couleur_theme' => '#9dba00'],
+        // Vert Troglo
+        7 => ['couleur_theme' => '#419a2c'],
+        // Bleu-vert
+        12 => ['couleur_theme' => '#269681'],
+        //  Bleu pastel
+        5 => ['couleur_theme' => '#3190ae'],
+        //  Bleu Kermesse
+        11 => ['couleur_theme' => '#288bdd'],
+        //  Gris bleuté
+        6 => ['couleur_theme' => '#7d90a2'],
+        //  Gris
+        10 => ['couleur_theme' => '#909090'],
+    ];
 
-	if (is_numeric($choix)) {
-		$c = $couleurs_spip[$choix];
-		// compat < SPIP 3.3
-		include_spip('inc/filtres_images_mini');
-		$c['couleur_foncee'] = $c['couleur_theme'];
-		$c['couleur_claire'] = '#' . couleur_eclaircir($c['couleur_theme'], .5);
+    if (is_numeric($choix)) {
+        $c = $couleurs_spip[$choix];
+        // compat < SPIP 3.3
+        include_spip('inc/filtres_images_mini');
+        $c['couleur_foncee'] = $c['couleur_theme'];
+        $c['couleur_claire'] = '#' . couleur_eclaircir($c['couleur_theme'], .5);
 
-		return
-			'couleur_theme=' . substr($c['couleur_theme'], 1)
-			// compat < SPIP 3.3
-			. '&couleur_claire=' . substr($c['couleur_claire'], 1)
-			. '&couleur_foncee=' . substr($c['couleur_foncee'], 1);
-	} else {
-		if (is_array($choix)) {
-			// compat < SPIP 3.3
-			$compat_spip_33 = function ($c) {
-				if (!isset($c['couleur_theme'])) {
-					$c['couleur_theme'] = $c['couleur_foncee'];
-					unset($c['couleur_foncee']);
-					unset($c['couleur_claire']);
-					unset($c['couleur_lien']);
-					unset($c['couleur_lien_off']);
-				}
-				return $c;
-			};
-			if ($ajouter) {
-				foreach ($choix as $c) {
-					$couleurs_spip[] = $compat_spip_33($c);
-				}
+        return
+            'couleur_theme=' . substr($c['couleur_theme'], 1)
+            // compat < SPIP 3.3
+            . '&couleur_claire=' . substr($c['couleur_claire'], 1)
+            . '&couleur_foncee=' . substr($c['couleur_foncee'], 1);
+    } else {
+        if (is_array($choix)) {
+            // compat < SPIP 3.3
+            $compat_spip_33 = function ($c) {
+                if (!isset($c['couleur_theme'])) {
+                    $c['couleur_theme'] = $c['couleur_foncee'];
+                    unset($c['couleur_foncee']);
+                    unset($c['couleur_claire']);
+                    unset($c['couleur_lien']);
+                    unset($c['couleur_lien_off']);
+                }
+                return $c;
+            };
+            if ($ajouter) {
+                foreach ($choix as $c) {
+                    $couleurs_spip[] = $compat_spip_33($c);
+                }
 
-				return $couleurs_spip;
-			} else {
-				$choix = array_map($compat_spip_33, $choix);
-				return $couleurs_spip = $choix;
-			}
-		}
-	}
+                return $couleurs_spip;
+            } else {
+                $choix = array_map($compat_spip_33, $choix);
+                return $couleurs_spip = $choix;
+            }
+        }
+    }
 
-	return $couleurs_spip;
+    return $couleurs_spip;
 }

Spacing +5 added lines, -5 removed lines

@@ -70,17 +70,17 @@
 		// compat < SPIP 3.3
 		include_spip('inc/filtres_images_mini');
 		$c['couleur_foncee'] = $c['couleur_theme'];
-		$c['couleur_claire'] = '#' . couleur_eclaircir($c['couleur_theme'], .5);
+		$c['couleur_claire'] = '#'.couleur_eclaircir($c['couleur_theme'], .5);
 
 		return
-			'couleur_theme=' . substr($c['couleur_theme'], 1)
+			'couleur_theme='.substr($c['couleur_theme'], 1)
 			// compat < SPIP 3.3
-			. '&couleur_claire=' . substr($c['couleur_claire'], 1)
-			. '&couleur_foncee=' . substr($c['couleur_foncee'], 1);
+			. '&couleur_claire='.substr($c['couleur_claire'], 1)
+			. '&couleur_foncee='.substr($c['couleur_foncee'], 1);
 	} else {
 		if (is_array($choix)) {
 			// compat < SPIP 3.3
-			$compat_spip_33 = function ($c) {
+			$compat_spip_33 = function($c) {
 				if (!isset($c['couleur_theme'])) {
 					$c['couleur_theme'] = $c['couleur_foncee'];
 					unset($c['couleur_foncee']);

ecrire/inc/iconifier.php

Indentation +7 added lines, -7 removed lines

@@ -17,7 +17,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/actions');
@@ -36,11 +36,11 @@ 
  *     - ou tableau d'information sur le squelette.
  */
 function inc_iconifier_dist($objet, $id, $script, $visible = false, $flag_modif = true) {
-	// compat avec anciens appels
-	$objet = objet_type($objet);
+    // compat avec anciens appels
+    $objet = objet_type($objet);
 
-	return recuperer_fond(
-		'prive/objets/editer/logo',
-		['objet' => $objet, 'id_objet' => $id, 'editable' => $flag_modif]
-	);
+    return recuperer_fond(
+        'prive/objets/editer/logo',
+        ['objet' => $objet, 'id_objet' => $id, 'editable' => $flag_modif]
+    );
 }

ecrire/inc/xml.php

Spacing +1 added lines, -1 removed lines

@@ -137,7 +137,7 @@
 					$p !== false
 					and ($morceau = substr($txt, $d, $p - $d))
 					and (($nopen += preg_match_all(
-						'{<' . preg_quote($closing_tag) . '(\s*>|\s[^>]*[^/>]>)}is',
+						'{<'.preg_quote($closing_tag).'(\s*>|\s[^>]*[^/>]>)}is',
 						$morceau,
 						$matches,
 						PREG_SET_ORDER

Indentation +157 added lines, -157 removed lines

@@ -17,7 +17,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 
@@ -47,24 +47,24 @@ 
  *     - false si l'arbre xml ne peut être créé ou est vide
  **/
 function spip_xml_load($fichier, $strict = true, $clean = true, $taille_max = 1_048_576, $datas = '', $profondeur = -1) {
-	$contenu = '';
-	if (tester_url_absolue($fichier)) {
-		include_spip('inc/distant');
-		$contenu = recuperer_url($fichier, ['taille_max' => $taille_max, 'datas' => $datas]);
-		$contenu = $contenu['page'] ?? '';
-	} else {
-		lire_fichier($fichier, $contenu);
-	}
-	$arbre = [];
-	if ($contenu) {
-		$arbre = spip_xml_parse($contenu, $strict, $clean, $profondeur);
-	}
+    $contenu = '';
+    if (tester_url_absolue($fichier)) {
+        include_spip('inc/distant');
+        $contenu = recuperer_url($fichier, ['taille_max' => $taille_max, 'datas' => $datas]);
+        $contenu = $contenu['page'] ?? '';
+    } else {
+        lire_fichier($fichier, $contenu);
+    }
+    $arbre = [];
+    if ($contenu) {
+        $arbre = spip_xml_parse($contenu, $strict, $clean, $profondeur);
+    }
 
-	return count($arbre) ? $arbre : false;
+    return count($arbre) ? $arbre : false;
 }
 
 if (!defined('_SPIP_XML_TAG_SPLIT')) {
-	define('_SPIP_XML_TAG_SPLIT', '{<([^:>][^>]*?)>}sS');
+    define('_SPIP_XML_TAG_SPLIT', '{<([^:>][^>]*?)>}sS');
 }
 
 /**
@@ -83,150 +83,150 @@ 
  *     - false si l'arbre xml ne peut être créé ou est vide
  **/
 function spip_xml_parse(&$texte, $strict = true, $clean = true, $profondeur = -1) {
-	$out = [];
-	// enlever les commentaires
-	$charset = 'AUTO';
-	if ($clean === true) {
-		if (preg_match(",<\?xml\s(.*?)encoding=['\"]?(.*?)['\"]?(\s(.*))?\?>,im", $texte, $regs)) {
-			$charset = $regs[2];
-		}
-		$texte = preg_replace(',<!--(.*?)-->,is', '', $texte);
-		$texte = preg_replace(',<\?(.*?)\?>,is', '', $texte);
-		include_spip('inc/charsets');
-		$clean = $charset;
-		//$texte = importer_charset($texte,$charset);
-	}
-	if (is_string($clean)) {
-		$charset = $clean;
-	}
-	$txt = $texte;
+    $out = [];
+    // enlever les commentaires
+    $charset = 'AUTO';
+    if ($clean === true) {
+        if (preg_match(",<\?xml\s(.*?)encoding=['\"]?(.*?)['\"]?(\s(.*))?\?>,im", $texte, $regs)) {
+            $charset = $regs[2];
+        }
+        $texte = preg_replace(',<!--(.*?)-->,is', '', $texte);
+        $texte = preg_replace(',<\?(.*?)\?>,is', '', $texte);
+        include_spip('inc/charsets');
+        $clean = $charset;
+        //$texte = importer_charset($texte,$charset);
+    }
+    if (is_string($clean)) {
+        $charset = $clean;
+    }
+    $txt = $texte;
 
-	// tant qu'il y a des tags
-	$chars = preg_split(_SPIP_XML_TAG_SPLIT, $txt, 2, PREG_SPLIT_DELIM_CAPTURE);
-	while ((is_countable($chars) ? count($chars) : 0) >= 2) {
-		// tag ouvrant
-		//$chars = preg_split("{<([^>]*?)>}s",$txt,2,PREG_SPLIT_DELIM_CAPTURE);
+    // tant qu'il y a des tags
+    $chars = preg_split(_SPIP_XML_TAG_SPLIT, $txt, 2, PREG_SPLIT_DELIM_CAPTURE);
+    while ((is_countable($chars) ? count($chars) : 0) >= 2) {
+        // tag ouvrant
+        //$chars = preg_split("{<([^>]*?)>}s",$txt,2,PREG_SPLIT_DELIM_CAPTURE);
 
-		// $before doit etre vide ou des espaces uniquements!
-		$before = trim($chars[0]);
+        // $before doit etre vide ou des espaces uniquements!
+        $before = trim($chars[0]);
 
-		if (strlen($before) > 0) {
-			return importer_charset($texte, $charset);
-		}//$texte; // before non vide, donc on est dans du texte
+        if (strlen($before) > 0) {
+            return importer_charset($texte, $charset);
+        }//$texte; // before non vide, donc on est dans du texte
 
-		$tag = rtrim($chars[1]);
-		$txt = $chars[2];
+        $tag = rtrim($chars[1]);
+        $txt = $chars[2];
 
-		if (strncmp($tag, '![CDATA[', 8) == 0) {
-			return importer_charset($texte, $charset);
-		}//$texte;
-		if (substr($tag, -1) == '/') { // self closing tag
-			$tag = rtrim(substr($tag, 0, strlen($tag) - 1));
-			$out[$tag][] = '';
-		} else {
-			$closing_tag = preg_split(",\s|\t|\n|\r,", trim($tag));
-			$closing_tag = reset($closing_tag);
-			// tag fermant
-			$ncclos = strlen("</$closing_tag>");
-			$p = strpos($txt, (string) "</$closing_tag>");
-			if ($p !== false and (strpos($txt, '<') < $p)) {
-				$nclose = 0;
-				$nopen = 0;
-				$d = 0;
-				while (
-					$p !== false
-					and ($morceau = substr($txt, $d, $p - $d))
-					and (($nopen += preg_match_all(
-						'{<' . preg_quote($closing_tag) . '(\s*>|\s[^>]*[^/>]>)}is',
-						$morceau,
-						$matches,
-						PREG_SET_ORDER
-					)) > $nclose)
-				) {
-					$nclose++;
-					$d = $p + $ncclos;
-					$p = strpos($txt, (string) "</$closing_tag>", $d);
-				}
-			}
-			if ($p === false) {
-				if ($strict) {
-					$out[$tag][] = "erreur : tag fermant $tag manquant::$txt";
+        if (strncmp($tag, '![CDATA[', 8) == 0) {
+            return importer_charset($texte, $charset);
+        }//$texte;
+        if (substr($tag, -1) == '/') { // self closing tag
+            $tag = rtrim(substr($tag, 0, strlen($tag) - 1));
+            $out[$tag][] = '';
+        } else {
+            $closing_tag = preg_split(",\s|\t|\n|\r,", trim($tag));
+            $closing_tag = reset($closing_tag);
+            // tag fermant
+            $ncclos = strlen("</$closing_tag>");
+            $p = strpos($txt, (string) "</$closing_tag>");
+            if ($p !== false and (strpos($txt, '<') < $p)) {
+                $nclose = 0;
+                $nopen = 0;
+                $d = 0;
+                while (
+                    $p !== false
+                    and ($morceau = substr($txt, $d, $p - $d))
+                    and (($nopen += preg_match_all(
+                        '{<' . preg_quote($closing_tag) . '(\s*>|\s[^>]*[^/>]>)}is',
+                        $morceau,
+                        $matches,
+                        PREG_SET_ORDER
+                    )) > $nclose)
+                ) {
+                    $nclose++;
+                    $d = $p + $ncclos;
+                    $p = strpos($txt, (string) "</$closing_tag>", $d);
+                }
+            }
+            if ($p === false) {
+                if ($strict) {
+                    $out[$tag][] = "erreur : tag fermant $tag manquant::$txt";
 
-					return $out;
-				} else {
-					return importer_charset($texte, $charset);
-				}//$texte // un tag qui constitue du texte a reporter dans $before
-			}
-			$content = substr($txt, 0, $p);
-			$txt = substr($txt, $p + $ncclos);
-			if ($profondeur == 0 or !str_contains($content, '<')) { // eviter une recursion si pas utile
-			$out[$tag][] = importer_charset($content, $charset);
-			}//$content;
-			else {
-				$out[$tag][] = spip_xml_parse($content, $strict, $clean, $profondeur - 1);
-			}
-		}
-		$chars = preg_split(_SPIP_XML_TAG_SPLIT, $txt, 2, PREG_SPLIT_DELIM_CAPTURE);
-	}
-	if (count($out) && (strlen(trim($txt)) == 0)) {
-		return $out;
-	} else {
-		return importer_charset($texte, $charset);
-	}//$texte;
+                    return $out;
+                } else {
+                    return importer_charset($texte, $charset);
+                }//$texte // un tag qui constitue du texte a reporter dans $before
+            }
+            $content = substr($txt, 0, $p);
+            $txt = substr($txt, $p + $ncclos);
+            if ($profondeur == 0 or !str_contains($content, '<')) { // eviter une recursion si pas utile
+            $out[$tag][] = importer_charset($content, $charset);
+            }//$content;
+            else {
+                $out[$tag][] = spip_xml_parse($content, $strict, $clean, $profondeur - 1);
+            }
+        }
+        $chars = preg_split(_SPIP_XML_TAG_SPLIT, $txt, 2, PREG_SPLIT_DELIM_CAPTURE);
+    }
+    if (count($out) && (strlen(trim($txt)) == 0)) {
+        return $out;
+    } else {
+        return importer_charset($texte, $charset);
+    }//$texte;
 }
 
 function spip_xml_aplatit($arbre, $separateur = ' ') {
-	$s = '';
-	if (is_array($arbre)) {
-		foreach ($arbre as $tag => $feuille) {
-			if (is_array($feuille)) {
-				if ($tag !== intval($tag)) {
-					$f = spip_xml_aplatit($feuille, $separateur);
-					if (strlen($f)) {
-						$tagf = explode(' ', $tag);
-						$tagf = $tagf[0];
-						$s .= "<$tag>$f</$tagf>";
-					} else {
-						$s .= "<$tag />";
-					}
-				} else {
-					$s .= spip_xml_aplatit($feuille);
-				}
-				$s .= $separateur;
-			} else {
-				$s .= "$feuille$separateur";
-			}
-		}
-	}
+    $s = '';
+    if (is_array($arbre)) {
+        foreach ($arbre as $tag => $feuille) {
+            if (is_array($feuille)) {
+                if ($tag !== intval($tag)) {
+                    $f = spip_xml_aplatit($feuille, $separateur);
+                    if (strlen($f)) {
+                        $tagf = explode(' ', $tag);
+                        $tagf = $tagf[0];
+                        $s .= "<$tag>$f</$tagf>";
+                    } else {
+                        $s .= "<$tag />";
+                    }
+                } else {
+                    $s .= spip_xml_aplatit($feuille);
+                }
+                $s .= $separateur;
+            } else {
+                $s .= "$feuille$separateur";
+            }
+        }
+    }
 
-	return strlen($separateur) ? substr($s, 0, -strlen($separateur)) : $s;
+    return strlen($separateur) ? substr($s, 0, -strlen($separateur)) : $s;
 }
 
 function spip_xml_tagname($tag) {
-	if (preg_match(',^([a-z][\w:]*),i', $tag, $reg)) {
-		return $reg[1];
-	}
+    if (preg_match(',^([a-z][\w:]*),i', $tag, $reg)) {
+        return $reg[1];
+    }
 
-	return '';
+    return '';
 }
 
 function spip_xml_decompose_tag($tag) {
-	$tagname = spip_xml_tagname($tag);
-	$liste = [];
-	$tag = ltrim(strpbrk($tag, " \n\t"));
-	$p = strpos($tag, '=');
-	while ($p !== false) {
-		$attr = trim(substr($tag, 0, $p));
-		$tag = ltrim(substr($tag, $p + 1));
-		$quote = $tag[0];
-		$p = strpos($tag, $quote, 1);
-		$cont = substr($tag, 1, $p - 1);
-		$liste[$attr] = $cont;
-		$tag = substr($tag, $p + 1);
-		$p = strpos($tag, '=');
-	}
+    $tagname = spip_xml_tagname($tag);
+    $liste = [];
+    $tag = ltrim(strpbrk($tag, " \n\t"));
+    $p = strpos($tag, '=');
+    while ($p !== false) {
+        $attr = trim(substr($tag, 0, $p));
+        $tag = ltrim(substr($tag, $p + 1));
+        $quote = $tag[0];
+        $p = strpos($tag, $quote, 1);
+        $cont = substr($tag, 1, $p - 1);
+        $liste[$attr] = $cont;
+        $tag = substr($tag, $p + 1);
+        $p = strpos($tag, '=');
+    }
 
-	return [$tagname, $liste];
+    return [$tagname, $liste];
 }
 
 /**
@@ -249,21 +249,21 @@ 
  *     false si aucun élément ne valide l'expression régulière, true sinon.
  **/
 function spip_xml_match_nodes($regexp, &$arbre, &$matches, $init = true) {
-	if ($init) {
-		$matches = [];
-	}
-	if (is_array($arbre) && count($arbre)) {
-		foreach (array_keys($arbre) as $tag) {
-			if (preg_match($regexp, $tag)) {
-				$matches[$tag] = &$arbre[$tag];
-			}
-			if (is_array($arbre[$tag])) {
-				foreach (array_keys($arbre[$tag]) as $occurences) {
-					spip_xml_match_nodes($regexp, $arbre[$tag][$occurences], $matches, false);
-				}
-			}
-		}
-	}
+    if ($init) {
+        $matches = [];
+    }
+    if (is_array($arbre) && count($arbre)) {
+        foreach (array_keys($arbre) as $tag) {
+            if (preg_match($regexp, $tag)) {
+                $matches[$tag] = &$arbre[$tag];
+            }
+            if (is_array($arbre[$tag])) {
+                foreach (array_keys($arbre[$tag]) as $occurences) {
+                    spip_xml_match_nodes($regexp, $arbre[$tag][$occurences], $matches, false);
+                }
+            }
+        }
+    }
 
-	return (is_countable($matches) ? count($matches) : 0);
+    return (is_countable($matches) ? count($matches) : 0);
 }