spip / SPIP

ecrire/inc/filtres_selecteur_generique.php

Spacing +4 added lines, -4 removed lines

@@ -159,7 +159,7 @@ 
 		$type = reset($objets);
 		$type = objet_type($type);
 		$id = (int) $ref;
-		$ref = $type . $ref;
+		$ref = $type.$ref;
 	}
 	else {
 		// Si la référence ne correspond à rien, c'est fini
@@ -167,7 +167,7 @@ 
 			return json_export(false);
 		}
 		// Sinon on récupère les infos utiles
-		[$type, , $id, , , , ] = array_pad($match, 7, null);
+		[$type,, $id,,,,] = array_pad($match, 7, null);
 
 		// On regarde si le type trouvé fait partie des objets sélectionnables
 		if (!in_array(table_objet($type), $objets)) {
@@ -213,8 +213,8 @@ 
 		$types = (is_array($types) ? array_filter($types) : []);
 
 		// recuperer tous les freres et soeurs de la rubrique visee
-		$id_parent = sql_getfetsel('id_parent', 'spip_rubriques', 'id_rubrique=' . (int) $id_rubrique);
-		$fratrie = sql_allfetsel('id_rubrique', 'spip_rubriques', 'id_parent=' . (int) $id_parent);
+		$id_parent = sql_getfetsel('id_parent', 'spip_rubriques', 'id_rubrique='.(int) $id_rubrique);
+		$fratrie = sql_allfetsel('id_rubrique', 'spip_rubriques', 'id_parent='.(int) $id_parent);
 		$fratrie = array_column($fratrie, 'id_rubrique');
 		$has = sql_allfetsel('DISTINCT id_parent', 'spip_rubriques', sql_in('id_parent', $fratrie));
 		$has = array_column($has, 'id_parent');

Braces +1 added lines, -2 removed lines

@@ -160,8 +160,7 @@
 		$type = objet_type($type);
 		$id = (int) $ref;
 		$ref = $type . $ref;
-	}
-	else {
+	} else {
 		// Si la référence ne correspond à rien, c'est fini
 		if (!($match = typer_raccourci($ref))) {
 			return json_export(false);

Indentation +146 added lines, -146 removed lines

@@ -7,7 +7,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -28,49 +28,49 @@ 
  *     - afficher : tableau des objets à afficher (mais pas forcément sélectionnables)
  */
 function selecteur_lister_objets($includelist = [], $excludelist = []) {
-	static $liste_selecteurs, $liste_parents;
-
-	if (!$liste_selecteurs) {
-		$liste_selecteurs = find_all_in_path('formulaires/selecteur/', 'hierarchie-[\w]+[.]html$');
-	}
-	$objets_selectionner = [];
-	foreach ($liste_selecteurs as $fichier => $chemin) {
-		$objets_selectionner[] = preg_replace('/^hierarchie-([\w]+)[.]html$/', '$1', (string) $fichier);
-	}
-
-	// S'il y a une whitelist on ne garde que ce qui est dedans
-	if (!empty($includelist)) {
-		$includelist = array_map('table_objet', $includelist);
-		$objets_selectionner = array_intersect($objets_selectionner, $includelist);
-	}
-	// On supprime ce qui est dans la liste d’exclusion
-	$excludelist = array_map('table_objet', $excludelist);
-	// On enlève toujours la racine
-	$excludelist[] = 'racine';
-	$objets_selectionner = array_diff($objets_selectionner, $excludelist);
-
-	// Ensuite on cherche ce qu'on doit afficher : au moins ceux qu'on peut sélectionner
-	$objets_afficher = $objets_selectionner;
-
-	// Il faut alors chercher d'éventuels parents obligatoires en plus :
-	// lister-trucs-bidules.html => on doit afficher des "trucs" pour trouver des "bidules"
-	if (!$liste_parents) {
-		$liste_parents = find_all_in_path('formulaires/selecteur/', 'lister-[\w]+-[\w]+[.]html$');
-	}
-	foreach ($liste_parents as $fichier => $chemin) {
-		preg_match('/^lister-([\w]+)-([\w]+)[.]html$/', (string) $fichier, $captures);
-		$parent = $captures[1];
-		$type = $captures[2];
-		// Si le type fait partie de ce qu'on doit afficher alors on ajoute aussi le parent à l'affichage
-		if (in_array($type, $objets_afficher)) {
-			$objets_afficher[] = $parent;
-		}
-	}
-
-	return [
-		'selectionner' => array_unique($objets_selectionner),
-		'afficher' => array_unique($objets_afficher),
-	];
+    static $liste_selecteurs, $liste_parents;
+
+    if (!$liste_selecteurs) {
+        $liste_selecteurs = find_all_in_path('formulaires/selecteur/', 'hierarchie-[\w]+[.]html$');
+    }
+    $objets_selectionner = [];
+    foreach ($liste_selecteurs as $fichier => $chemin) {
+        $objets_selectionner[] = preg_replace('/^hierarchie-([\w]+)[.]html$/', '$1', (string) $fichier);
+    }
+
+    // S'il y a une whitelist on ne garde que ce qui est dedans
+    if (!empty($includelist)) {
+        $includelist = array_map('table_objet', $includelist);
+        $objets_selectionner = array_intersect($objets_selectionner, $includelist);
+    }
+    // On supprime ce qui est dans la liste d’exclusion
+    $excludelist = array_map('table_objet', $excludelist);
+    // On enlève toujours la racine
+    $excludelist[] = 'racine';
+    $objets_selectionner = array_diff($objets_selectionner, $excludelist);
+
+    // Ensuite on cherche ce qu'on doit afficher : au moins ceux qu'on peut sélectionner
+    $objets_afficher = $objets_selectionner;
+
+    // Il faut alors chercher d'éventuels parents obligatoires en plus :
+    // lister-trucs-bidules.html => on doit afficher des "trucs" pour trouver des "bidules"
+    if (!$liste_parents) {
+        $liste_parents = find_all_in_path('formulaires/selecteur/', 'lister-[\w]+-[\w]+[.]html$');
+    }
+    foreach ($liste_parents as $fichier => $chemin) {
+        preg_match('/^lister-([\w]+)-([\w]+)[.]html$/', (string) $fichier, $captures);
+        $parent = $captures[1];
+        $type = $captures[2];
+        // Si le type fait partie de ce qu'on doit afficher alors on ajoute aussi le parent à l'affichage
+        if (in_array($type, $objets_afficher)) {
+            $objets_afficher[] = $parent;
+        }
+    }
+
+    return [
+        'selectionner' => array_unique($objets_selectionner),
+        'afficher' => array_unique($objets_afficher),
+    ];
 }
 
 /**
@@ -97,31 +97,31 @@ 
  *     liste des couples (objets => id_objet) ou liste des identifiants d'un type d'objet.
  **/
 function picker_selected($selected, $type = '') {
-	$select = [];
-	$type = preg_replace(',\W,', '', $type);
-
-	if ($selected && !is_array($selected)) {
-		$selected = explode(',', $selected);
-	}
-
-	if (is_array($selected)) {
-		foreach ($selected as $value) {
-			// Si c'est le bon format déjà
-			if (preg_match('/^([\w]+)[|](\d+)$/', (string) $value, $captures)) {
-				$objet = $captures[1];
-				$id_objet = (int) $captures[2];
-
-				// Si on cherche un type et que c'est le bon, on renvoit un tableau que d'identifiants
-				if (is_string($type) && $type == $objet && ($id_objet || in_array($objet, ['racine', 'rubrique']))) {
-					$select[] = $id_objet;
-				} elseif (!$type && ($id_objet || in_array($objet, ['racine', 'rubrique']))) {
-					$select[] = ['objet' => $objet, 'id_objet' => $id_objet];
-				}
-			}
-		}
-	}
-
-	return $select;
+    $select = [];
+    $type = preg_replace(',\W,', '', $type);
+
+    if ($selected && !is_array($selected)) {
+        $selected = explode(',', $selected);
+    }
+
+    if (is_array($selected)) {
+        foreach ($selected as $value) {
+            // Si c'est le bon format déjà
+            if (preg_match('/^([\w]+)[|](\d+)$/', (string) $value, $captures)) {
+                $objet = $captures[1];
+                $id_objet = (int) $captures[2];
+
+                // Si on cherche un type et que c'est le bon, on renvoit un tableau que d'identifiants
+                if (is_string($type) && $type == $objet && ($id_objet || in_array($objet, ['racine', 'rubrique']))) {
+                    $select[] = $id_objet;
+                } elseif (!$type && ($id_objet || in_array($objet, ['racine', 'rubrique']))) {
+                    $select[] = ['objet' => $objet, 'id_objet' => $id_objet];
+                }
+            }
+        }
+    }
+
+    return $select;
 }
 
 /**
@@ -138,53 +138,53 @@ 
  *     Booléen indiquant si les articles sont sélectionnables
  */
 function picker_identifie_id_rapide($ref, $rubriques_ou_objets = false, $articles = false) {
-	include_spip('inc/json');
-	include_spip('inc/lien');
-
-	// On construit un tableau des objets sélectionnables suivant les paramètres
-	$objets = [];
-	if ($rubriques_ou_objets && is_array($rubriques_ou_objets)) {
-		$objets = $rubriques_ou_objets;
-	} else {
-		if ($rubriques_ou_objets) {
-			$objets[] = 'rubriques';
-		}
-		if ($articles) {
-			$objets[] = 'articles';
-		}
-	}
-
-	// si id numerique et un seul objet possible, pas d'ambiguite
-	if (is_numeric($ref) && count($objets) === 1) {
-		$type = reset($objets);
-		$type = objet_type($type);
-		$id = (int) $ref;
-		$ref = $type . $ref;
-	}
-	else {
-		// Si la référence ne correspond à rien, c'est fini
-		if (!($match = typer_raccourci($ref))) {
-			return json_export(false);
-		}
-		// Sinon on récupère les infos utiles
-		[$type, , $id, , , , ] = array_pad($match, 7, null);
-
-		// On regarde si le type trouvé fait partie des objets sélectionnables
-		if (!in_array(table_objet($type), $objets)) {
-			return json_export(false);
-		}
-	}
-
-	// Maintenant que tout est bon, on cherche les informations sur cet objet
-	include_spip('inc/filtres');
-	if (!$titre = generer_objet_info($id, $type, 'titre')) {
-		return json_export(false);
-	}
-
-	// On simplifie le texte
-	$titre = attribut_html($titre);
-
-	return json_export(['type' => $type, 'id' => "$type|$id", 'titre' => $titre]);
+    include_spip('inc/json');
+    include_spip('inc/lien');
+
+    // On construit un tableau des objets sélectionnables suivant les paramètres
+    $objets = [];
+    if ($rubriques_ou_objets && is_array($rubriques_ou_objets)) {
+        $objets = $rubriques_ou_objets;
+    } else {
+        if ($rubriques_ou_objets) {
+            $objets[] = 'rubriques';
+        }
+        if ($articles) {
+            $objets[] = 'articles';
+        }
+    }
+
+    // si id numerique et un seul objet possible, pas d'ambiguite
+    if (is_numeric($ref) && count($objets) === 1) {
+        $type = reset($objets);
+        $type = objet_type($type);
+        $id = (int) $ref;
+        $ref = $type . $ref;
+    }
+    else {
+        // Si la référence ne correspond à rien, c'est fini
+        if (!($match = typer_raccourci($ref))) {
+            return json_export(false);
+        }
+        // Sinon on récupère les infos utiles
+        [$type, , $id, , , , ] = array_pad($match, 7, null);
+
+        // On regarde si le type trouvé fait partie des objets sélectionnables
+        if (!in_array(table_objet($type), $objets)) {
+            return json_export(false);
+        }
+    }
+
+    // Maintenant que tout est bon, on cherche les informations sur cet objet
+    include_spip('inc/filtres');
+    if (!$titre = generer_objet_info($id, $type, 'titre')) {
+        return json_export(false);
+    }
+
+    // On simplifie le texte
+    $titre = attribut_html($titre);
+
+    return json_export(['type' => $type, 'id' => "$type|$id", 'titre' => $titre]);
 }
 
 /**
@@ -207,34 +207,34 @@ 
  *     Comme le filtre `oui` : espace (` `) si rubrique à afficher, chaîne vide sinon.
  */
 function test_enfants_rubrique($id_rubrique, $types = []) {
-	static $has_child = [];
-
-	if (!isset($has_child[$id_rubrique])) {
-		$types = (is_array($types) ? array_filter($types) : []);
-
-		// recuperer tous les freres et soeurs de la rubrique visee
-		$id_parent = sql_getfetsel('id_parent', 'spip_rubriques', 'id_rubrique=' . (int) $id_rubrique);
-		$fratrie = sql_allfetsel('id_rubrique', 'spip_rubriques', 'id_parent=' . (int) $id_parent);
-		$fratrie = array_column($fratrie, 'id_rubrique');
-		$has = sql_allfetsel('DISTINCT id_parent', 'spip_rubriques', sql_in('id_parent', $fratrie));
-		$has = array_column($has, 'id_parent');
-		$fratrie = array_diff($fratrie, $has);
-
-		while (count($fratrie) && is_array($types) && count($types)) {
-			$type = array_shift($types);
-			$h = sql_allfetsel('DISTINCT id_rubrique', table_objet_sql($type), sql_in('id_rubrique', $fratrie));
-			$h = array_column($h, 'id_rubrique');
-			$has = [...$has, ...$h];
-			$fratrie = array_diff($fratrie, $h);
-		}
-
-		if ($has !== []) {
-			$has_child += array_combine($has, array_pad([], count($has), true));
-		}
-		if ($fratrie !== []) {
-			$has_child += array_combine($fratrie, array_pad([], count($fratrie), false));
-		}
-	}
-
-	return $has_child[$id_rubrique] ? ' ' : '';
+    static $has_child = [];
+
+    if (!isset($has_child[$id_rubrique])) {
+        $types = (is_array($types) ? array_filter($types) : []);
+
+        // recuperer tous les freres et soeurs de la rubrique visee
+        $id_parent = sql_getfetsel('id_parent', 'spip_rubriques', 'id_rubrique=' . (int) $id_rubrique);
+        $fratrie = sql_allfetsel('id_rubrique', 'spip_rubriques', 'id_parent=' . (int) $id_parent);
+        $fratrie = array_column($fratrie, 'id_rubrique');
+        $has = sql_allfetsel('DISTINCT id_parent', 'spip_rubriques', sql_in('id_parent', $fratrie));
+        $has = array_column($has, 'id_parent');
+        $fratrie = array_diff($fratrie, $has);
+
+        while (count($fratrie) && is_array($types) && count($types)) {
+            $type = array_shift($types);
+            $h = sql_allfetsel('DISTINCT id_rubrique', table_objet_sql($type), sql_in('id_rubrique', $fratrie));
+            $h = array_column($h, 'id_rubrique');
+            $has = [...$has, ...$h];
+            $fratrie = array_diff($fratrie, $h);
+        }
+
+        if ($has !== []) {
+            $has_child += array_combine($has, array_pad([], count($has), true));
+        }
+        if ($fratrie !== []) {
+            $has_child += array_combine($fratrie, array_pad([], count($fratrie), false));
+        }
+    }
+
+    return $has_child[$id_rubrique] ? ' ' : '';
 }

ecrire/inc/genie.php

Indentation +89 added lines, -89 removed lines

@@ -10,7 +10,7 @@ 
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -74,28 +74,28 @@ 
  * @return
  **/
 function inc_genie_dist($taches = []) {
-	include_spip('inc/queue');
-
-	if (_request('exec') == 'job_queue') {
-		return false;
-	}
-
-	$force_jobs = [];
-	// l'ancienne facon de lancer une tache cron immediatement
-	// etait de la passer en parametre a ing_genie_dist
-	// on reroute en ajoutant simplement le job a la queue, ASAP
-	foreach ($taches as $function => $period) {
-		$force_jobs[] = queue_add_job(
-			$function,
-			_T('tache_cron_asap', ['function' => $function]),
-			[time() - abs($period)],
-			'genie/'
-		);
-	}
-
-	// et on passe la main a la gestion de la queue !
-	// en forcant eventuellement les jobs ajoute a l'instant
-	return queue_schedule(count($force_jobs) ? $force_jobs : null);
+    include_spip('inc/queue');
+
+    if (_request('exec') == 'job_queue') {
+        return false;
+    }
+
+    $force_jobs = [];
+    // l'ancienne facon de lancer une tache cron immediatement
+    // etait de la passer en parametre a ing_genie_dist
+    // on reroute en ajoutant simplement le job a la queue, ASAP
+    foreach ($taches as $function => $period) {
+        $force_jobs[] = queue_add_job(
+            $function,
+            _T('tache_cron_asap', ['function' => $function]),
+            [time() - abs($period)],
+            'genie/'
+        );
+    }
+
+    // et on passe la main a la gestion de la queue !
+    // en forcant eventuellement les jobs ajoute a l'instant
+    return queue_schedule(count($force_jobs) ? $force_jobs : null);
 }
 
 //
@@ -108,34 +108,34 @@ 
 //
 function taches_generales($taches_generales = []) {
 
-	// verifier que toutes les taches cron sont planifiees
-	// c'est une tache cron !
-	$taches_generales['queue_watch'] = 3600 * 24;
+    // verifier que toutes les taches cron sont planifiees
+    // c'est une tache cron !
+    $taches_generales['queue_watch'] = 3600 * 24;
 
-	// MAJ des rubriques publiques (cas de la publication post-datee)
-	// est fait au coup par coup a present
-	//	$taches_generales['rubriques'] = 3600;
+    // MAJ des rubriques publiques (cas de la publication post-datee)
+    // est fait au coup par coup a present
+    //	$taches_generales['rubriques'] = 3600;
 
-	// Optimisation de la base
-	$taches_generales['optimiser'] = 3600 * 48;
+    // Optimisation de la base
+    $taches_generales['optimiser'] = 3600 * 48;
 
-	// nouveautes
-	if (
-		isset($GLOBALS['meta']['adresse_neuf'])
-		&& $GLOBALS['meta']['adresse_neuf']
-		&& $GLOBALS['meta']['jours_neuf']
-		&& $GLOBALS['meta']['quoi_de_neuf'] == 'oui'
-	) {
-		$taches_generales['mail'] = 3600 * 24 * $GLOBALS['meta']['jours_neuf'];
-	}
+    // nouveautes
+    if (
+        isset($GLOBALS['meta']['adresse_neuf'])
+        && $GLOBALS['meta']['adresse_neuf']
+        && $GLOBALS['meta']['jours_neuf']
+        && $GLOBALS['meta']['quoi_de_neuf'] == 'oui'
+    ) {
+        $taches_generales['mail'] = 3600 * 24 * $GLOBALS['meta']['jours_neuf'];
+    }
 
-	// maintenance (ajax, verifications diverses)
-	$taches_generales['maintenance'] = 3600 * 2;
+    // maintenance (ajax, verifications diverses)
+    $taches_generales['maintenance'] = 3600 * 2;
 
-	// verifier si une mise a jour de spip est disponible (2 fois par semaine suffit largement)
-	$taches_generales['mise_a_jour'] = 3 * 24 * 3600;
+    // verifier si une mise a jour de spip est disponible (2 fois par semaine suffit largement)
+    $taches_generales['mise_a_jour'] = 3 * 24 * 3600;
 
-	return pipeline('taches_generales_cron', $taches_generales);
+    return pipeline('taches_generales_cron', $taches_generales);
 }
 
 /**
@@ -151,22 +151,22 @@ 
  * @return int
  */
 function genie_queue_watch_dist() {
-	static $deja_la = false;
-	if ($deja_la) {
-		return;
-	} // re-entrance si l'insertion des jobs echoue (pas de table spip_jobs a l'upgrade par exemple)
-	$deja_la = true;
-	$taches = taches_generales();
-	$programmees = sql_allfetsel('fonction', 'spip_jobs', sql_in('fonction', array_keys($taches)));
-	$programmees = array_column($programmees, 'fonction');
-	foreach ($taches as $tache => $periode) {
-		if (!in_array($tache, $programmees)) {
-			queue_genie_replan_job($tache, $periode, time() - round(random_int(1, $periode)), 0);
-		}
-	}
-	$deja_la = false;
-
-	return 1;
+    static $deja_la = false;
+    if ($deja_la) {
+        return;
+    } // re-entrance si l'insertion des jobs echoue (pas de table spip_jobs a l'upgrade par exemple)
+    $deja_la = true;
+    $taches = taches_generales();
+    $programmees = sql_allfetsel('fonction', 'spip_jobs', sql_in('fonction', array_keys($taches)));
+    $programmees = array_column($programmees, 'fonction');
+    foreach ($taches as $tache => $periode) {
+        if (!in_array($tache, $programmees)) {
+            queue_genie_replan_job($tache, $periode, time() - round(random_int(1, $periode)), 0);
+        }
+    }
+    $deja_la = false;
+
+    return 1;
 }
 
 /**
@@ -187,32 +187,32 @@ 
  * @return void
  */
 function queue_genie_replan_job($function, $period, $last = 0, $time = null, $priority = 0) {
-	static $done = [];
-	if (isset($done[$function])) {
-		return;
-	}
-	$done[$function] = true;
-	if (is_null($time)) {
-		$time = time();
-		if ($last) {
-			$time = max($last + $period, $time);
-		}
-	}
-	if (!$last) {
-		$last = $time - $period;
-	}
-	spip_log("replan_job $function $period $last $time $priority", 'queue');
-	include_spip('inc/queue');
-	// on replanifie un job cron
-	// uniquement si il n'y en a pas deja un avec le meme nom
-	// independament de l'argument
-	queue_add_job(
-		$function,
-		_T('tache_cron_secondes', ['function' => $function, 'nb' => $period]),
-		[$last],
-		'genie/',
-		'function_only',
-		$time,
-		$priority
-	);
+    static $done = [];
+    if (isset($done[$function])) {
+        return;
+    }
+    $done[$function] = true;
+    if (is_null($time)) {
+        $time = time();
+        if ($last) {
+            $time = max($last + $period, $time);
+        }
+    }
+    if (!$last) {
+        $last = $time - $period;
+    }
+    spip_log("replan_job $function $period $last $time $priority", 'queue');
+    include_spip('inc/queue');
+    // on replanifie un job cron
+    // uniquement si il n'y en a pas deja un avec le meme nom
+    // independament de l'argument
+    queue_add_job(
+        $function,
+        _T('tache_cron_secondes', ['function' => $function, 'nb' => $period]),
+        [$last],
+        'genie/',
+        'function_only',
+        $time,
+        $priority
+    );
 }

ecrire/inc/roles.php

Indentation +182 added lines, -182 removed lines

@@ -27,7 +27,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 
@@ -47,55 +47,55 @@ 
  *     array : description des roles applicables dans 3 index : colonne, titres, roles
  **/
 function roles_presents($objet, $objet_destination = '') {
-	$desc = lister_tables_objets_sql(table_objet_sql($objet));
-
-	// pas de liste de roles, on sort
-	if (!isset($desc['roles_titres']) || !($titres = $desc['roles_titres'])) {
-		return false;
-	}
-
-	// on vérifie que la table de liaison existe
-	include_spip('action/editer_liens');
-	if (!$lien = objet_associable($objet)) {
-		return false;
-	}
-
-	// on cherche ensuite si la colonne existe bien dans la table de liaison (par défaut 'role')
-	$colonne = $desc['roles_colonne'] ?? 'role';
-	$trouver_table = charger_fonction('trouver_table', 'base');
-	[, $table_lien] = $lien;
-	$desc_lien = $trouver_table($table_lien);
-	if (!isset($desc_lien['field'][$colonne])) {
-		return false;
-	}
-
-	// sur quoi peuvent s'appliquer nos rôles
-	if (!$application = $desc['roles_objets']) {
-		return false;
-	}
-
-	// destination presente, on restreint si possible
-	if ($objet_destination) {
-		$objet_destination = table_objet($objet_destination);
-
-		// pour l'objet
-		if (isset($application[$objet_destination])) {
-			$application = $application[$objet_destination];
-			// sinon pour tous les objets
-		} elseif (isset($application['*'])) {
-			$application = $application['*'];
-		} // sinon tant pis
-		else {
-			return false;
-		}
-	}
-
-	// tout est ok
-	return [
-		'titres' => $titres,
-		'roles' => $application,
-		'colonne' => $colonne
-	];
+    $desc = lister_tables_objets_sql(table_objet_sql($objet));
+
+    // pas de liste de roles, on sort
+    if (!isset($desc['roles_titres']) || !($titres = $desc['roles_titres'])) {
+        return false;
+    }
+
+    // on vérifie que la table de liaison existe
+    include_spip('action/editer_liens');
+    if (!$lien = objet_associable($objet)) {
+        return false;
+    }
+
+    // on cherche ensuite si la colonne existe bien dans la table de liaison (par défaut 'role')
+    $colonne = $desc['roles_colonne'] ?? 'role';
+    $trouver_table = charger_fonction('trouver_table', 'base');
+    [, $table_lien] = $lien;
+    $desc_lien = $trouver_table($table_lien);
+    if (!isset($desc_lien['field'][$colonne])) {
+        return false;
+    }
+
+    // sur quoi peuvent s'appliquer nos rôles
+    if (!$application = $desc['roles_objets']) {
+        return false;
+    }
+
+    // destination presente, on restreint si possible
+    if ($objet_destination) {
+        $objet_destination = table_objet($objet_destination);
+
+        // pour l'objet
+        if (isset($application[$objet_destination])) {
+            $application = $application[$objet_destination];
+            // sinon pour tous les objets
+        } elseif (isset($application['*'])) {
+            $application = $application['*'];
+        } // sinon tant pis
+        else {
+            return false;
+        }
+    }
+
+    // tout est ok
+    return [
+        'titres' => $titres,
+        'roles' => $application,
+        'colonne' => $colonne
+    ];
 }
 
 /**
@@ -109,11 +109,11 @@ 
  *     Nom de la colonne, sinon vide
  **/
 function roles_colonne($objet, $objet_destination) {
-	if ($roles = roles_presents($objet, $objet_destination)) {
-		return $roles['colonne'];
-	}
+    if ($roles = roles_presents($objet, $objet_destination)) {
+        return $roles['colonne'];
+    }
 
-	return '';
+    return '';
 }
 
 
@@ -135,21 +135,21 @@ 
  *     Liste ('', '', array()) sinon.
  **/
 function roles_trouver_dans_qualif($objet, $objet_destination, $qualif = []) {
-	// si des rôles sont possibles, on les utilise
-	$role = $colonne_role = ''; # role défini
-	// condition du where par defaut
-	$cond = [];
-	if ($roles = roles_presents($objet, $objet_destination)) {
-		$colonne_role = $roles['colonne'];
-		// qu'il n'est pas défini
-		if (!isset($qualif[$colonne_role]) || !($role = $qualif[$colonne_role])) {
-			$role = $roles['roles']['defaut'];
-		}
-		// where
-		$cond = ["$colonne_role=" . sql_quote($role)];
-	}
-
-	return [$role, $colonne_role, $cond];
+    // si des rôles sont possibles, on les utilise
+    $role = $colonne_role = ''; # role défini
+    // condition du where par defaut
+    $cond = [];
+    if ($roles = roles_presents($objet, $objet_destination)) {
+        $colonne_role = $roles['colonne'];
+        // qu'il n'est pas défini
+        if (!isset($qualif[$colonne_role]) || !($role = $qualif[$colonne_role])) {
+            $role = $roles['roles']['defaut'];
+        }
+        // where
+        $cond = ["$colonne_role=" . sql_quote($role)];
+    }
+
+    return [$role, $colonne_role, $cond];
 }
 
 /**
@@ -171,18 +171,18 @@ 
  *     Liste (Tableau de conditions where complété du role, Colonne du role, role utilisé)
  **/
 function roles_creer_condition_role($objet_source, $objet, $cond, $tous_si_absent = false) {
-	// role par défaut, colonne
-	[$role_defaut, $colonne_role] = roles_trouver_dans_qualif($objet_source, $objet);
-
-	// chercher d'eventuels rôles transmis
-	$role = ($cond['role'] ?? ($tous_si_absent ? '*' : $role_defaut));
-	unset($cond['role']); // cette condition est particuliere...
-	// on ajoute la condition du role aux autres conditions.
-	if ($colonne_role && $role != '*') {
-		$cond[] = "$colonne_role=" . sql_quote($role);
-	}
-
-	return [$cond, $colonne_role, $role];
+    // role par défaut, colonne
+    [$role_defaut, $colonne_role] = roles_trouver_dans_qualif($objet_source, $objet);
+
+    // chercher d'eventuels rôles transmis
+    $role = ($cond['role'] ?? ($tous_si_absent ? '*' : $role_defaut));
+    unset($cond['role']); // cette condition est particuliere...
+    // on ajoute la condition du role aux autres conditions.
+    if ($colonne_role && $role != '*') {
+        $cond[] = "$colonne_role=" . sql_quote($role);
+    }
+
+    return [$cond, $colonne_role, $role];
 }
 
 /**
@@ -204,28 +204,28 @@ 
  */
 function roles_complets($objet_source, $objet, $id_objet, $objet_lien) {
 
-	$presents = roles_presents_liaisons($objet_source, $objet, $id_objet, $objet_lien);
-	// pas de roles sur ces objets => la liste par defaut, comme sans role
-	if ($presents === false) {
-		return lister_objets_lies($objet_source, $objet, $id_objet, $objet_lien);
-	}
-
-	// types de roles possibles
-	$roles_possibles = $presents['roles']['roles']['choix'];
-	// couples id / roles
-	$ids = $presents['ids'];
-
-	// pour chaque groupe, on fait le diff entre tous les roles possibles
-	// et les roles attribués à l'élément : s'il en reste, c'est que l'élément
-	// n'est pas complet
-	$complets = [];
-	foreach ($ids as $id => $roles_presents) {
-		if (!array_diff($roles_possibles, $roles_presents)) {
-			$complets[] = $id;
-		}
-	}
-
-	return $complets;
+    $presents = roles_presents_liaisons($objet_source, $objet, $id_objet, $objet_lien);
+    // pas de roles sur ces objets => la liste par defaut, comme sans role
+    if ($presents === false) {
+        return lister_objets_lies($objet_source, $objet, $id_objet, $objet_lien);
+    }
+
+    // types de roles possibles
+    $roles_possibles = $presents['roles']['roles']['choix'];
+    // couples id / roles
+    $ids = $presents['ids'];
+
+    // pour chaque groupe, on fait le diff entre tous les roles possibles
+    // et les roles attribués à l'élément : s'il en reste, c'est que l'élément
+    // n'est pas complet
+    $complets = [];
+    foreach ($ids as $id => $roles_presents) {
+        if (!array_diff($roles_possibles, $roles_presents)) {
+            $complets[] = $id;
+        }
+    }
+
+    return $complets;
 }
 
 
@@ -242,17 +242,17 @@ 
  */
 function roles_presents_sur_id($id_objet_source, $objet_source, $objet, $id_objet, $objet_lien) {
 
-	$presents = roles_presents_liaisons($objet_source, $objet, $id_objet, $objet_lien);
-	// pas de roles sur ces objets => la liste par defaut, comme sans role
-	if ($presents === false) {
-		return [];
-	}
+    $presents = roles_presents_liaisons($objet_source, $objet, $id_objet, $objet_lien);
+    // pas de roles sur ces objets => la liste par defaut, comme sans role
+    if ($presents === false) {
+        return [];
+    }
 
-	if (!isset($presents['ids'][$id_objet_source])) {
-		return [];
-	}
+    if (!isset($presents['ids'][$id_objet_source])) {
+        return [];
+    }
 
-	return $presents['ids'][$id_objet_source];
+    return $presents['ids'][$id_objet_source];
 }
 
 
@@ -280,47 +280,47 @@ 
  *     - False si pas de role déclarés
  */
 function roles_presents_liaisons($objet_source, $objet, $id_objet, $objet_lien) {
-	static $done = [];
-
-	// stocker le résultat
-	$hash = "$objet_source-$objet-$id_objet-$objet_lien";
-	if (isset($done[$hash])) {
-		return $done[$hash];
-	}
-
-	// pas de roles sur ces objets, on sort
-	$roles = roles_presents($objet_lien, ($objet_lien == $objet) ? $objet_source : $objet);
-	if (!$roles) {
-		return $done[$hash] = false;
-	}
-
-	// inspiré de lister_objets_lies()
-	if ($objet_lien == $objet) {
-		$res = objet_trouver_liens([$objet => $id_objet], [$objet_source => '*']);
-	} else {
-		$res = objet_trouver_liens([$objet_source => '*'], [$objet => $id_objet]);
-	}
-
-	// types de roles possibles
-	$roles_possibles = $roles['roles']['choix'];
-	// colonne du role
-	$colonne = $roles['colonne'];
-
-	// on recupere par id, et role existant
-	$ids = [];
-	while ($row = array_shift($res)) {
-		$id = $row[$objet_source];
-		if (!isset($ids[$id])) {
-			$ids[$id] = [];
-		}
-		// tableau des roles présents
-		$ids[$id][] = $row[$colonne];
-	}
-
-	return $done[$hash] = [
-		'roles' => $roles,
-		'ids' => $ids
-	];
+    static $done = [];
+
+    // stocker le résultat
+    $hash = "$objet_source-$objet-$id_objet-$objet_lien";
+    if (isset($done[$hash])) {
+        return $done[$hash];
+    }
+
+    // pas de roles sur ces objets, on sort
+    $roles = roles_presents($objet_lien, ($objet_lien == $objet) ? $objet_source : $objet);
+    if (!$roles) {
+        return $done[$hash] = false;
+    }
+
+    // inspiré de lister_objets_lies()
+    if ($objet_lien == $objet) {
+        $res = objet_trouver_liens([$objet => $id_objet], [$objet_source => '*']);
+    } else {
+        $res = objet_trouver_liens([$objet_source => '*'], [$objet => $id_objet]);
+    }
+
+    // types de roles possibles
+    $roles_possibles = $roles['roles']['choix'];
+    // colonne du role
+    $colonne = $roles['colonne'];
+
+    // on recupere par id, et role existant
+    $ids = [];
+    while ($row = array_shift($res)) {
+        $id = $row[$objet_source];
+        if (!isset($ids[$id])) {
+            $ids[$id] = [];
+        }
+        // tableau des roles présents
+        $ids[$id][] = $row[$colonne];
+    }
+
+    return $done[$hash] = [
+        'roles' => $roles,
+        'ids' => $ids
+    ];
 }
 
 
@@ -338,33 +338,33 @@ 
  *     - false si pas de role déclarés
  */
 function roles_connus_en_base($objet_source, $objet, $objet_lien) {
-	static $done = [];
-
-	// stocker le résultat
-	$hash = "$objet_source-$objet-$objet_lien";
-	if (isset($done[$hash])) {
-		return $done[$hash];
-	}
-
-	if (!$lien = objet_associable($objet_lien)) {
-		return $done[$hash] = false;
-	}
-
-	// pas de roles sur ces objets, on sort
-	$roles = roles_presents($objet_lien, ($objet_lien == $objet) ? $objet_source : $objet);
-	if (!$roles) {
-		return $done[$hash] = false;
-	}
-
-	[$primary, $l] = $lien;
-	$colone_role = $roles['colonne'];
-
-	$all = sql_allfetsel(
-		"DISTINCT $colone_role",
-		$l,
-		'objet=' . sql_quote(($objet_source == $objet_lien) ? $objet : $objet_source)
-	);
-	$done[$hash] = array_map('reset', $all);
-
-	return $done[$hash];
+    static $done = [];
+
+    // stocker le résultat
+    $hash = "$objet_source-$objet-$objet_lien";
+    if (isset($done[$hash])) {
+        return $done[$hash];
+    }
+
+    if (!$lien = objet_associable($objet_lien)) {
+        return $done[$hash] = false;
+    }
+
+    // pas de roles sur ces objets, on sort
+    $roles = roles_presents($objet_lien, ($objet_lien == $objet) ? $objet_source : $objet);
+    if (!$roles) {
+        return $done[$hash] = false;
+    }
+
+    [$primary, $l] = $lien;
+    $colone_role = $roles['colonne'];
+
+    $all = sql_allfetsel(
+        "DISTINCT $colone_role",
+        $l,
+        'objet=' . sql_quote(($objet_source == $objet_lien) ? $objet : $objet_source)
+    );
+    $done[$hash] = array_map('reset', $all);
+
+    return $done[$hash];
 }

Spacing +3 added lines, -3 removed lines

@@ -146,7 +146,7 @@ 
 			$role = $roles['roles']['defaut'];
 		}
 		// where
-		$cond = ["$colonne_role=" . sql_quote($role)];
+		$cond = ["$colonne_role=".sql_quote($role)];
 	}
 
 	return [$role, $colonne_role, $cond];
@@ -179,7 +179,7 @@ 
 	unset($cond['role']); // cette condition est particuliere...
 	// on ajoute la condition du role aux autres conditions.
 	if ($colonne_role && $role != '*') {
-		$cond[] = "$colonne_role=" . sql_quote($role);
+		$cond[] = "$colonne_role=".sql_quote($role);
 	}
 
 	return [$cond, $colonne_role, $role];
@@ -362,7 +362,7 @@ 
 	$all = sql_allfetsel(
 		"DISTINCT $colone_role",
 		$l,
-		'objet=' . sql_quote(($objet_source == $objet_lien) ? $objet : $objet_source)
+		'objet='.sql_quote(($objet_source == $objet_lien) ? $objet : $objet_source)
 	);
 	$done[$hash] = array_map('reset', $all);
 

ecrire/inc/filtres_mime.php

Spacing +8 added lines, -8 removed lines

@@ -119,7 +119,7 @@ 
 	static $t1 = ['&', '<', '>'];
 	static $t2 = ['&amp;', '&lt;', '&gt;'];
 
-	return '<pre>' . str_replace($t1, $t2, (string) $t) . '</pre>';
+	return '<pre>'.str_replace($t1, $t2, (string) $t).'</pre>';
 }
 
 /**
@@ -139,12 +139,12 @@ 
 	foreach ($lignes as &$l) {
 		$l = implode('|', $l);
 	}
-	$corps = implode("\n", $lignes) . "\n";
-	$corps = $caption .
-		"\n|{{" .
-		implode('}}|{{', $entete) .
-		'}}|' .
-		"\n|" .
+	$corps = implode("\n", $lignes)."\n";
+	$corps = $caption.
+		"\n|{{".
+		implode('}}|{{', $entete).
+		'}}|'.
+		"\n|".
 		str_replace("\n", "|\n|", $corps);
 	include_spip('inc/texte');
 
@@ -188,7 +188,7 @@ 
 	// Pourquoi SafeHtml transforme-t-il en texte les scripts dans Body ?
 	$t = safehtml(preg_replace(',<script.*?</script>,is', '', $t));
 
-	return ($style ? "\n<style>" . $style . '</style>' : '') . $t;
+	return ($style ? "\n<style>".$style.'</style>' : '').$t;
 }
 
 /**

Indentation +53 added lines, -53 removed lines

@@ -21,7 +21,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 include_spip('inc/filtres');
 
@@ -38,7 +38,7 @@ 
  * @return string Rien.
  **/
 function filtre_image_dist($t) {
-	return '';
+    return '';
 }
 
 /**
@@ -51,7 +51,7 @@ 
  * @return string Rien.
  **/
 function filtre_audio_dist($t) {
-	return '';
+    return '';
 }
 
 /**
@@ -64,7 +64,7 @@ 
  * @return string Rien.
  **/
 function filtre_video_dist($t) {
-	return '';
+    return '';
 }
 
 /**
@@ -77,7 +77,7 @@ 
  * @return string Rien.
  **/
 function filtre_application_dist($t) {
-	return '';
+    return '';
 }
 
 /**
@@ -90,7 +90,7 @@ 
  * @return string Rien.
  **/
 function filtre_message_dist($t) {
-	return '';
+    return '';
 }
 
 /**
@@ -103,7 +103,7 @@ 
  * @return string Rien.
  **/
 function filtre_multipart_dist($t) {
-	return '';
+    return '';
 }
 
 /**
@@ -116,10 +116,10 @@ 
  * @return string Contenu échappé.
  **/
 function filtre_text_dist($t) {
-	static $t1 = ['&', '<', '>'];
-	static $t2 = ['&amp;', '&lt;', '&gt;'];
+    static $t1 = ['&', '<', '>'];
+    static $t2 = ['&amp;', '&lt;', '&gt;'];
 
-	return '<pre>' . str_replace($t1, $t2, (string) $t) . '</pre>';
+    return '<pre>' . str_replace($t1, $t2, (string) $t) . '</pre>';
 }
 
 /**
@@ -134,21 +134,21 @@ 
  *     Tableau (formaté en SPIP)
  **/
 function filtre_text_csv_dist($t) {
-	include_spip('inc/csv');
-	[$entete, $lignes, $caption] = analyse_csv($t);
-	foreach ($lignes as &$l) {
-		$l = implode('|', $l);
-	}
-	$corps = implode("\n", $lignes) . "\n";
-	$corps = $caption .
-		"\n|{{" .
-		implode('}}|{{', $entete) .
-		'}}|' .
-		"\n|" .
-		str_replace("\n", "|\n|", $corps);
-	include_spip('inc/texte');
-
-	return propre($corps);
+    include_spip('inc/csv');
+    [$entete, $lignes, $caption] = analyse_csv($t);
+    foreach ($lignes as &$l) {
+        $l = implode('|', $l);
+    }
+    $corps = implode("\n", $lignes) . "\n";
+    $corps = $caption .
+        "\n|{{" .
+        implode('}}|{{', $entete) .
+        '}}|' .
+        "\n|" .
+        str_replace("\n", "|\n|", $corps);
+    include_spip('inc/texte');
+
+    return propre($corps);
 }
 
 /**
@@ -162,33 +162,33 @@ 
  * @return string Code html sécurisé ou texte échappé
  **/
 function filtre_text_html_dist($t) {
-	if (!preg_match(',^(.*?)<body[^>]*>(.*)</body>,is', $t, $r)) {
-		return appliquer_filtre($t, 'text/plain');
-	}
-
-	[, $h, $t] = $r;
-
-	$style = '';
-	// recuperer les styles internes
-	if (preg_match_all(',<style>(.*?)</style>,is', $h, $r, PREG_PATTERN_ORDER)) {
-		$style = implode("\n", $r[1]);
-	}
-	// ... et externes
-
-	include_spip('inc/distant');
-	if (preg_match_all(',<link[^>]+type=.text/css[^>]*>,is', $h, $r, PREG_PATTERN_ORDER)) {
-		foreach ($r[0] as $l) {
-			preg_match("/href='([^']*)'/", str_replace('"', "'", $l), $m);
-			$page = recuperer_url($m[1]);
-			$page = $page['page'] ?? '';
-			$style .= "\n/* $l */\n"
-				. str_replace('<', '', (string) $page);
-		}
-	}
-	// Pourquoi SafeHtml transforme-t-il en texte les scripts dans Body ?
-	$t = safehtml(preg_replace(',<script.*?</script>,is', '', $t));
-
-	return ($style ? "\n<style>" . $style . '</style>' : '') . $t;
+    if (!preg_match(',^(.*?)<body[^>]*>(.*)</body>,is', $t, $r)) {
+        return appliquer_filtre($t, 'text/plain');
+    }
+
+    [, $h, $t] = $r;
+
+    $style = '';
+    // recuperer les styles internes
+    if (preg_match_all(',<style>(.*?)</style>,is', $h, $r, PREG_PATTERN_ORDER)) {
+        $style = implode("\n", $r[1]);
+    }
+    // ... et externes
+
+    include_spip('inc/distant');
+    if (preg_match_all(',<link[^>]+type=.text/css[^>]*>,is', $h, $r, PREG_PATTERN_ORDER)) {
+        foreach ($r[0] as $l) {
+            preg_match("/href='([^']*)'/", str_replace('"', "'", $l), $m);
+            $page = recuperer_url($m[1]);
+            $page = $page['page'] ?? '';
+            $style .= "\n/* $l */\n"
+                . str_replace('<', '', (string) $page);
+        }
+    }
+    // Pourquoi SafeHtml transforme-t-il en texte les scripts dans Body ?
+    $t = safehtml(preg_replace(',<script.*?</script>,is', '', $t));
+
+    return ($style ? "\n<style>" . $style . '</style>' : '') . $t;
 }
 
 /**
@@ -201,7 +201,7 @@ 
  * @return string Code HTML des balises `<param>`
  **/
 function filtre_audio_x_pn_realaudio($id) {
-	return "
+    return "
 	<param name='controls' value='PositionSlider' />
 	<param name='controls' value='ImageWindow' />
 	<param name='controls' value='PlayButton' />

ecrire/inc/importer_csv.php

Indentation +71 added lines, -71 removed lines

@@ -10,7 +10,7 @@ 
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/charsets');
@@ -41,14 +41,14 @@ 
  * @return array
  */
 function importer_csv_importcharset($texte, $definir_charset_source = false) {
-	// le plus frequent, en particulier avec les trucs de ms@@@
-	static $charset_source = 'iso-8859-1';
-	if ($definir_charset_source) {
-		$charset_source = $definir_charset_source === true ? 'iso-8859-1' : $definir_charset_source;
-	}
-	// mais open-office sait faire mieux, donc mefiance !
-	$charset = is_utf8($texte) ? 'utf-8' : $charset_source;
-	return importer_charset($texte, $charset);
+    // le plus frequent, en particulier avec les trucs de ms@@@
+    static $charset_source = 'iso-8859-1';
+    if ($definir_charset_source) {
+        $charset_source = $definir_charset_source === true ? 'iso-8859-1' : $definir_charset_source;
+    }
+    // mais open-office sait faire mieux, donc mefiance !
+    $charset = is_utf8($texte) ? 'utf-8' : $charset_source;
+    return importer_charset($texte, $charset);
 }
 
 /**
@@ -59,7 +59,7 @@ 
  * @return string
  */
 function importer_csv_nettoie_key($key) {
-	return translitteration($key);
+    return translitteration($key);
 }
 
 /**
@@ -78,70 +78,70 @@ 
  */
 function inc_importer_csv_dist($file, $options = []) {
 
-	// support ancienne syntaxe
-	// inc_importer_csv_dist($file, $head = false, $delim = ',', $enclos = '"', $len = 10000, $charset_source = '')
-	if (!is_array($options)) {
-		$args = func_get_args();
-		$options = [];
-		foreach ([1 => 'head', 2 => 'delim', 3 => 'enclos', 4 => 'len', 5 => 'charset_source'] as $k => $option) {
-			if (!empty($args[$k])) {
-				$options[$option] = $args[$k];
-			}
-		}
-	}
+    // support ancienne syntaxe
+    // inc_importer_csv_dist($file, $head = false, $delim = ',', $enclos = '"', $len = 10000, $charset_source = '')
+    if (!is_array($options)) {
+        $args = func_get_args();
+        $options = [];
+        foreach ([1 => 'head', 2 => 'delim', 3 => 'enclos', 4 => 'len', 5 => 'charset_source'] as $k => $option) {
+            if (!empty($args[$k])) {
+                $options[$option] = $args[$k];
+            }
+        }
+    }
 
-	$default_options = [
-		'head' => false,
-		'delim' => ',',
-		'enclos' => '"',
-		'len' => 10000,
-		'charset_source' => '',
-	];
-	$options = array_merge($default_options, $options);
+    $default_options = [
+        'head' => false,
+        'delim' => ',',
+        'enclos' => '"',
+        'len' => 10000,
+        'charset_source' => '',
+    ];
+    $options = array_merge($default_options, $options);
 
 
-	$return = [];
-	if (@file_exists($file) && ($handle = fopen($file, 'r'))) {
-		if ($options['charset_source']) {
-			importer_csv_importcharset('', $options['charset_source']);
-		}
-		if ($options['head']) {
-			$header = fgetcsv($handle, $options['len'], $options['delim'], $options['enclos']);
-			if ($header) {
-				$header = array_map('importer_csv_importcharset', $header);
-				$header = array_map('importer_csv_nettoie_key', $header);
-				$header_type = [];
-				foreach ($header as $heading) {
-					$header_type[$heading] = isset($header_type[$heading]) ? 'array' : 'scalar';
-				}
-			}
-		}
+    $return = [];
+    if (@file_exists($file) && ($handle = fopen($file, 'r'))) {
+        if ($options['charset_source']) {
+            importer_csv_importcharset('', $options['charset_source']);
+        }
+        if ($options['head']) {
+            $header = fgetcsv($handle, $options['len'], $options['delim'], $options['enclos']);
+            if ($header) {
+                $header = array_map('importer_csv_importcharset', $header);
+                $header = array_map('importer_csv_nettoie_key', $header);
+                $header_type = [];
+                foreach ($header as $heading) {
+                    $header_type[$heading] = isset($header_type[$heading]) ? 'array' : 'scalar';
+                }
+            }
+        }
 
-		while (($data = fgetcsv($handle, $options['len'], $options['delim'], $options['enclos'])) !== false) {
-			$data = array_map('importer_csv_importcharset', $data);
-			if ($options['head'] && isset($header)) {
-				$row = [];
-				foreach ($header as $key => $heading) {
-					if ($header_type[$heading] == 'array') {
-						if (!isset($row[$heading])) {
-							$row[$heading] = [];
-						}
-						if (isset($data[$key]) && strlen($data[$key])) {
-							$row[$heading][] = $data[$key];
-						}
-					} else {
-						$row[$heading] = $data[$key] ?? '';
-					}
-				}
-				$return[] = $row;
-			} else {
-				$return[] = $data;
-			}
-		}
-		if ($options['charset_source']) {
-			importer_csv_importcharset('', true);
-		}
-	}
+        while (($data = fgetcsv($handle, $options['len'], $options['delim'], $options['enclos'])) !== false) {
+            $data = array_map('importer_csv_importcharset', $data);
+            if ($options['head'] && isset($header)) {
+                $row = [];
+                foreach ($header as $key => $heading) {
+                    if ($header_type[$heading] == 'array') {
+                        if (!isset($row[$heading])) {
+                            $row[$heading] = [];
+                        }
+                        if (isset($data[$key]) && strlen($data[$key])) {
+                            $row[$heading][] = $data[$key];
+                        }
+                    } else {
+                        $row[$heading] = $data[$key] ?? '';
+                    }
+                }
+                $return[] = $row;
+            } else {
+                $return[] = $data;
+            }
+        }
+        if ($options['charset_source']) {
+            importer_csv_importcharset('', true);
+        }
+    }
 
-	return $return ?: false;
+    return $return ?: false;
 }

ecrire/inc/recherche_to_array.php

Spacing +7 added lines, -7 removed lines

@@ -66,7 +66,7 @@ 
 		$_id_table = reset($_id_table);
 	}
 
-	$requete['SELECT'][] = 't.' . $_id_table;
+	$requete['SELECT'][] = 't.'.$_id_table;
 	$a = [];
 	// Recherche fulltext
 	foreach ($champs as $champ => $poids) {
@@ -77,13 +77,13 @@ 
 				$champ = "t.$champ";
 			}
 			$requete['SELECT'][] = $champ;
-			$a[] = $champ . ' ' . $methode . ' ' . $q;
+			$a[] = $champ.' '.$methode.' '.$q;
 		}
 	}
 	if ($a) {
 		$requete['WHERE'][] = implode(' OR ', $a);
 	}
-	$requete['FROM'][] = table_objet_sql($table) . ' AS t';
+	$requete['FROM'][] = table_objet_sql($table).' AS t';
 
 	$results = [];
 
@@ -244,8 +244,8 @@ 
 					);
 				} // cas table de liaison generique spip_xxx_yyy
 				elseif (
-					($t = $trouver_table($table_arrivee . '_' . $table_depart, $serveur))
-					|| ($t = $trouver_table($table_depart . '_' . $table_arrivee, $serveur))
+					($t = $trouver_table($table_arrivee.'_'.$table_depart, $serveur))
+					|| ($t = $trouver_table($table_depart.'_'.$table_arrivee, $serveur))
 				) {
 					$s = sql_select(
 						"$cle_depart,$cle_arrivee",
@@ -284,12 +284,12 @@ 
 				}
 				if (isset($joint['champs']) && $joint['champs']) {
 					foreach ($joint['champs'] as $c => $val) {
-						$results[$id]['champs'][$table_liee . '.' . $c] = $val;
+						$results[$id]['champs'][$table_liee.'.'.$c] = $val;
 					}
 				}
 				if (isset($joint['matches']) && $joint['matches']) {
 					foreach ($joint['matches'] as $c => $val) {
-						$results[$id]['matches'][$table_liee . '.' . $c] = $val;
+						$results[$id]['matches'][$table_liee.'.'.$c] = $val;
 					}
 				}
 			}

Indentation +260 added lines, -260 removed lines

@@ -10,291 +10,291 @@
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 
 // methodes sql
 function inc_recherche_to_array_dist($recherche, $options = []) {
 
-	// options par defaut
-	$options = array_merge(
-		[
-			'score' => true,
-			'champs' => false,
-			'toutvoir' => false,
-			'matches' => false,
-			'jointures' => false
-		],
-		$options
-	);
+    // options par defaut
+    $options = array_merge(
+        [
+            'score' => true,
+            'champs' => false,
+            'toutvoir' => false,
+            'matches' => false,
+            'jointures' => false
+        ],
+        $options
+    );
 
-	include_spip('inc/rechercher');
-	include_spip('inc/autoriser');
+    include_spip('inc/rechercher');
+    include_spip('inc/autoriser');
 
-	$requete = [
-		'SELECT' => [],
-		'FROM' => [],
-		'WHERE' => [],
-		'GROUPBY' => [],
-		'ORDERBY' => [],
-		'LIMIT' => '',
-		'HAVING' => []
-	];
+    $requete = [
+        'SELECT' => [],
+        'FROM' => [],
+        'WHERE' => [],
+        'GROUPBY' => [],
+        'ORDERBY' => [],
+        'LIMIT' => '',
+        'HAVING' => []
+    ];
 
-	$table = sinon($options['table'], 'article');
-	if ($options['champs']) {
-		$champs = $options['champs'];
-	} else {
-		$l = liste_des_champs();
-		$champs = $l['article'];
-	}
-	$serveur = $options['serveur'];
+    $table = sinon($options['table'], 'article');
+    if ($options['champs']) {
+        $champs = $options['champs'];
+    } else {
+        $l = liste_des_champs();
+        $champs = $l['article'];
+    }
+    $serveur = $options['serveur'];
 
-	[$methode, $q, $preg] = expression_recherche($recherche, $options);
+    [$methode, $q, $preg] = expression_recherche($recherche, $options);
 
-	$jointures = $options['jointures']
-		? liste_des_jointures()
-		: [];
+    $jointures = $options['jointures']
+        ? liste_des_jointures()
+        : [];
 
-	$_id_table = id_table_objet($table);
+    $_id_table = id_table_objet($table);
 
-	// c'est un pis-aller : ca a peu de chance de marcher, mais mieux quand meme que en conservant la ','
-	// (aka ca marche au moins dans certains cas comme avec spip_formulaires_reponses_champs)
-	if (str_contains((string) $_id_table, ',')) {
-		$_id_table = explode(',', (string) $_id_table);
-		$_id_table = reset($_id_table);
-	}
+    // c'est un pis-aller : ca a peu de chance de marcher, mais mieux quand meme que en conservant la ','
+    // (aka ca marche au moins dans certains cas comme avec spip_formulaires_reponses_champs)
+    if (str_contains((string) $_id_table, ',')) {
+        $_id_table = explode(',', (string) $_id_table);
+        $_id_table = reset($_id_table);
+    }
 
-	$requete['SELECT'][] = 't.' . $_id_table;
-	$a = [];
-	// Recherche fulltext
-	foreach ($champs as $champ => $poids) {
-		if (is_array($champ)) {
-			spip_log('requetes imbriquees interdites');
-		} else {
-			if (!str_contains((string) $champ, '.')) {
-				$champ = "t.$champ";
-			}
-			$requete['SELECT'][] = $champ;
-			$a[] = $champ . ' ' . $methode . ' ' . $q;
-		}
-	}
-	if ($a) {
-		$requete['WHERE'][] = implode(' OR ', $a);
-	}
-	$requete['FROM'][] = table_objet_sql($table) . ' AS t';
+    $requete['SELECT'][] = 't.' . $_id_table;
+    $a = [];
+    // Recherche fulltext
+    foreach ($champs as $champ => $poids) {
+        if (is_array($champ)) {
+            spip_log('requetes imbriquees interdites');
+        } else {
+            if (!str_contains((string) $champ, '.')) {
+                $champ = "t.$champ";
+            }
+            $requete['SELECT'][] = $champ;
+            $a[] = $champ . ' ' . $methode . ' ' . $q;
+        }
+    }
+    if ($a) {
+        $requete['WHERE'][] = implode(' OR ', $a);
+    }
+    $requete['FROM'][] = table_objet_sql($table) . ' AS t';
 
-	$results = [];
+    $results = [];
 
-	$s = sql_select(
-		$requete['SELECT'],
-		$requete['FROM'],
-		$requete['WHERE'],
-		implode(' ', $requete['GROUPBY']),
-		$requete['ORDERBY'],
-		$requete['LIMIT'],
-		$requete['HAVING'],
-		$serveur
-	);
+    $s = sql_select(
+        $requete['SELECT'],
+        $requete['FROM'],
+        $requete['WHERE'],
+        implode(' ', $requete['GROUPBY']),
+        $requete['ORDERBY'],
+        $requete['LIMIT'],
+        $requete['HAVING'],
+        $serveur
+    );
 
-	while (
-		($t = sql_fetch($s, $serveur))
-		&& (!isset($t['score']) || $t['score'] > 0)
-	) {
-		$id = (int) $t[$_id_table];
+    while (
+        ($t = sql_fetch($s, $serveur))
+        && (!isset($t['score']) || $t['score'] > 0)
+    ) {
+        $id = (int) $t[$_id_table];
 
-		if ($options['toutvoir'] || autoriser('voir', $table, $id)) {
-			// indiquer les champs concernes
-			$champs_vus = [];
-			$score = 0;
-			$matches = [];
+        if ($options['toutvoir'] || autoriser('voir', $table, $id)) {
+            // indiquer les champs concernes
+            $champs_vus = [];
+            $score = 0;
+            $matches = [];
 
-			$vu = false;
-			foreach ($champs as $champ => $poids) {
-				$champ = explode('.', (string) $champ);
-				$champ = end($champ);
-				// translitteration_rapide uniquement si on est deja en utf-8
-				$value = ($GLOBALS['meta']['charset'] == 'utf-8' ? translitteration_rapide($t[$champ]) : translitteration($t[$champ]));
-				if (
-					$n =
-					($options['score'] || $options['matches'])
-						? preg_match_all($preg, (string) $value, $regs, PREG_SET_ORDER)
-						: preg_match($preg, (string) $value)
-				) {
-					$vu = true;
+            $vu = false;
+            foreach ($champs as $champ => $poids) {
+                $champ = explode('.', (string) $champ);
+                $champ = end($champ);
+                // translitteration_rapide uniquement si on est deja en utf-8
+                $value = ($GLOBALS['meta']['charset'] == 'utf-8' ? translitteration_rapide($t[$champ]) : translitteration($t[$champ]));
+                if (
+                    $n =
+                    ($options['score'] || $options['matches'])
+                        ? preg_match_all($preg, (string) $value, $regs, PREG_SET_ORDER)
+                        : preg_match($preg, (string) $value)
+                ) {
+                    $vu = true;
 
-					if ($options['champs']) {
-						$champs_vus[$champ] = $t[$champ];
-					}
-					if ($options['score']) {
-						// compter les points avec un peu de discernement : on pondere par la longueur du match compte en chars
-						$score += $poids * strlen(implode('', array_column($regs, 0)));
-					}
+                    if ($options['champs']) {
+                        $champs_vus[$champ] = $t[$champ];
+                    }
+                    if ($options['score']) {
+                        // compter les points avec un peu de discernement : on pondere par la longueur du match compte en chars
+                        $score += $poids * strlen(implode('', array_column($regs, 0)));
+                    }
 
-					if ($options['matches']) {
-						$matches[$champ] = $regs;
-					}
+                    if ($options['matches']) {
+                        $matches[$champ] = $regs;
+                    }
 
-					if (!$options['champs'] && !$options['score'] && !$options['matches']) {
-						break;
-					}
-				}
-			}
+                    if (!$options['champs'] && !$options['score'] && !$options['matches']) {
+                        break;
+                    }
+                }
+            }
 
-			if ($vu) {
-				if (!isset($results)) {
-					$results = [];
-				}
-				$results[$id] = [];
-				if ($champs_vus) {
-					$results[$id]['champs'] = $champs_vus;
-				}
-				if ($score) {
-					$results[$id]['score'] = $score;
-				}
-				if ($matches) {
-					$results[$id]['matches'] = $matches;
-				}
-			}
-		}
-	}
+            if ($vu) {
+                if (!isset($results)) {
+                    $results = [];
+                }
+                $results[$id] = [];
+                if ($champs_vus) {
+                    $results[$id]['champs'] = $champs_vus;
+                }
+                if ($score) {
+                    $results[$id]['score'] = $score;
+                }
+                if ($matches) {
+                    $results[$id]['matches'] = $matches;
+                }
+            }
+        }
+    }
 
 
-	// Gerer les donnees associees
-	// ici on est un peu naze : pas capables de reconstruire une jointure complexe
-	// on ne sait passer que par table de laison en 1 coup
-	if (
-		isset($jointures[$table])
-		&& ($joints = recherche_en_base(
-			$recherche,
-			$jointures[$table],
-			array_merge($options, ['jointures' => false])
-		))
-	) {
-		include_spip('action/editer_liens');
-		$trouver_table = charger_fonction('trouver_table', 'base');
-		$cle_depart = id_table_objet($table);
-		$table_depart = table_objet($table, $serveur);
-		$desc_depart = $trouver_table($table_depart, $serveur);
-		$depart_associable = objet_associable($table);
-		foreach ($joints as $table_liee => $ids_trouves) {
-			// on peut definir une fonction de recherche jointe pour regler les cas particuliers
-			if (
-				!(
-				($rechercher_joints = charger_fonction("rechercher_joints_{$table}_{$table_liee}", 'inc', true))
-				|| ($rechercher_joints = charger_fonction("rechercher_joints_objet_{$table_liee}", 'inc', true))
-				|| ($rechercher_joints = charger_fonction("rechercher_joints_{$table}_objet_lie", 'inc', true))
-				)
-			) {
-				$cle_arrivee = id_table_objet($table_liee);
-				$table_arrivee = table_objet($table_liee, $serveur);
-				$desc_arrivee = $trouver_table($table_arrivee, $serveur);
-				// cas simple : $cle_depart dans la table_liee
-				if (isset($desc_arrivee['field'][$cle_depart])) {
-					$s = sql_select(
-						"$cle_depart, $cle_arrivee",
-						$desc_arrivee['table_sql'],
-						sql_in($cle_arrivee, array_keys($ids_trouves)),
-						'',
-						'',
-						'',
-						'',
-						$serveur
-					);
-				} // cas simple : $cle_arrivee dans la table
-				elseif (isset($desc_depart['field'][$cle_arrivee])) {
-					$s = sql_select(
-						"$cle_depart, $cle_arrivee",
-						$desc_depart['table_sql'],
-						sql_in($cle_arrivee, array_keys($ids_trouves)),
-						'',
-						'',
-						'',
-						'',
-						$serveur
-					);
-				}
-				// sinon cherchons une table de liaison
-				// cas recherche principale article, objet lie document : passer par spip_documents_liens
-				elseif ($l = objet_associable($table_liee)) {
-					[$primary, $table_liens] = $l;
-					$s = sql_select(
-						"id_objet as $cle_depart, $primary as $cle_arrivee",
-						$table_liens,
-						["objet='$table'", sql_in($primary, array_keys($ids_trouves))],
-						'',
-						'',
-						'',
-						'',
-						$serveur
-					);
-				} // cas recherche principale auteur, objet lie article: passer par spip_auteurs_liens
-				elseif ($l = $depart_associable) {
-					[$primary, $table_liens] = $l;
-					$s = sql_select(
-						"$primary as $cle_depart, id_objet as $cle_arrivee",
-						$table_liens,
-						["objet='$table_liee'", sql_in('id_objet', array_keys($ids_trouves))],
-						'',
-						'',
-						'',
-						'',
-						$serveur
-					);
-				} // cas table de liaison generique spip_xxx_yyy
-				elseif (
-					($t = $trouver_table($table_arrivee . '_' . $table_depart, $serveur))
-					|| ($t = $trouver_table($table_depart . '_' . $table_arrivee, $serveur))
-				) {
-					$s = sql_select(
-						"$cle_depart,$cle_arrivee",
-						$t['table_sql'],
-						sql_in($cle_arrivee, array_keys($ids_trouves)),
-						'',
-						'',
-						'',
-						'',
-						$serveur
-					);
-				}
-			} else {
-				[$cle_depart, $cle_arrivee, $s] = $rechercher_joints(
-					$table,
-					$table_liee,
-					array_keys($ids_trouves),
-					$serveur
-				);
-			}
+    // Gerer les donnees associees
+    // ici on est un peu naze : pas capables de reconstruire une jointure complexe
+    // on ne sait passer que par table de laison en 1 coup
+    if (
+        isset($jointures[$table])
+        && ($joints = recherche_en_base(
+            $recherche,
+            $jointures[$table],
+            array_merge($options, ['jointures' => false])
+        ))
+    ) {
+        include_spip('action/editer_liens');
+        $trouver_table = charger_fonction('trouver_table', 'base');
+        $cle_depart = id_table_objet($table);
+        $table_depart = table_objet($table, $serveur);
+        $desc_depart = $trouver_table($table_depart, $serveur);
+        $depart_associable = objet_associable($table);
+        foreach ($joints as $table_liee => $ids_trouves) {
+            // on peut definir une fonction de recherche jointe pour regler les cas particuliers
+            if (
+                !(
+                ($rechercher_joints = charger_fonction("rechercher_joints_{$table}_{$table_liee}", 'inc', true))
+                || ($rechercher_joints = charger_fonction("rechercher_joints_objet_{$table_liee}", 'inc', true))
+                || ($rechercher_joints = charger_fonction("rechercher_joints_{$table}_objet_lie", 'inc', true))
+                )
+            ) {
+                $cle_arrivee = id_table_objet($table_liee);
+                $table_arrivee = table_objet($table_liee, $serveur);
+                $desc_arrivee = $trouver_table($table_arrivee, $serveur);
+                // cas simple : $cle_depart dans la table_liee
+                if (isset($desc_arrivee['field'][$cle_depart])) {
+                    $s = sql_select(
+                        "$cle_depart, $cle_arrivee",
+                        $desc_arrivee['table_sql'],
+                        sql_in($cle_arrivee, array_keys($ids_trouves)),
+                        '',
+                        '',
+                        '',
+                        '',
+                        $serveur
+                    );
+                } // cas simple : $cle_arrivee dans la table
+                elseif (isset($desc_depart['field'][$cle_arrivee])) {
+                    $s = sql_select(
+                        "$cle_depart, $cle_arrivee",
+                        $desc_depart['table_sql'],
+                        sql_in($cle_arrivee, array_keys($ids_trouves)),
+                        '',
+                        '',
+                        '',
+                        '',
+                        $serveur
+                    );
+                }
+                // sinon cherchons une table de liaison
+                // cas recherche principale article, objet lie document : passer par spip_documents_liens
+                elseif ($l = objet_associable($table_liee)) {
+                    [$primary, $table_liens] = $l;
+                    $s = sql_select(
+                        "id_objet as $cle_depart, $primary as $cle_arrivee",
+                        $table_liens,
+                        ["objet='$table'", sql_in($primary, array_keys($ids_trouves))],
+                        '',
+                        '',
+                        '',
+                        '',
+                        $serveur
+                    );
+                } // cas recherche principale auteur, objet lie article: passer par spip_auteurs_liens
+                elseif ($l = $depart_associable) {
+                    [$primary, $table_liens] = $l;
+                    $s = sql_select(
+                        "$primary as $cle_depart, id_objet as $cle_arrivee",
+                        $table_liens,
+                        ["objet='$table_liee'", sql_in('id_objet', array_keys($ids_trouves))],
+                        '',
+                        '',
+                        '',
+                        '',
+                        $serveur
+                    );
+                } // cas table de liaison generique spip_xxx_yyy
+                elseif (
+                    ($t = $trouver_table($table_arrivee . '_' . $table_depart, $serveur))
+                    || ($t = $trouver_table($table_depart . '_' . $table_arrivee, $serveur))
+                ) {
+                    $s = sql_select(
+                        "$cle_depart,$cle_arrivee",
+                        $t['table_sql'],
+                        sql_in($cle_arrivee, array_keys($ids_trouves)),
+                        '',
+                        '',
+                        '',
+                        '',
+                        $serveur
+                    );
+                }
+            } else {
+                [$cle_depart, $cle_arrivee, $s] = $rechercher_joints(
+                    $table,
+                    $table_liee,
+                    array_keys($ids_trouves),
+                    $serveur
+                );
+            }
 
-			while ($t = is_array($s) ? array_shift($s) : sql_fetch($s)) {
-				$id = $t[$cle_depart];
-				$joint = $ids_trouves[$t[$cle_arrivee]];
-				if (!isset($results)) {
-					$results = [];
-				}
-				if (!isset($results[$id])) {
-					$results[$id] = [];
-				}
-				if (isset($joint['score']) && $joint['score']) {
-					if (!isset($results[$id]['score'])) {
-						$results[$id]['score'] = 0;
-					}
-					$results[$id]['score'] += $joint['score'];
-				}
-				if (isset($joint['champs']) && $joint['champs']) {
-					foreach ($joint['champs'] as $c => $val) {
-						$results[$id]['champs'][$table_liee . '.' . $c] = $val;
-					}
-				}
-				if (isset($joint['matches']) && $joint['matches']) {
-					foreach ($joint['matches'] as $c => $val) {
-						$results[$id]['matches'][$table_liee . '.' . $c] = $val;
-					}
-				}
-			}
-		}
-	}
+            while ($t = is_array($s) ? array_shift($s) : sql_fetch($s)) {
+                $id = $t[$cle_depart];
+                $joint = $ids_trouves[$t[$cle_arrivee]];
+                if (!isset($results)) {
+                    $results = [];
+                }
+                if (!isset($results[$id])) {
+                    $results[$id] = [];
+                }
+                if (isset($joint['score']) && $joint['score']) {
+                    if (!isset($results[$id]['score'])) {
+                        $results[$id]['score'] = 0;
+                    }
+                    $results[$id]['score'] += $joint['score'];
+                }
+                if (isset($joint['champs']) && $joint['champs']) {
+                    foreach ($joint['champs'] as $c => $val) {
+                        $results[$id]['champs'][$table_liee . '.' . $c] = $val;
+                    }
+                }
+                if (isset($joint['matches']) && $joint['matches']) {
+                    foreach ($joint['matches'] as $c => $val) {
+                        $results[$id]['matches'][$table_liee . '.' . $c] = $val;
+                    }
+                }
+            }
+        }
+    }
 
-	return $results;
+    return $results;
 }

ecrire/inc/presenter_enfants.php

Indentation +171 added lines, -171 removed lines

@@ -10,7 +10,7 @@ 
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/autoriser');
@@ -29,74 +29,74 @@ 
  *  Un tableau des sous rubriques
  */
 function enfant_rub($collection, $debut = 0, $limite = 500) {
-	$voir_logo = (isset($GLOBALS['meta']['image_process']) && $GLOBALS['meta']['image_process'] != 'non');
-	$logo = '';
-
-	if ($voir_logo) {
-		$chercher_logo = charger_fonction('chercher_logo', 'inc');
-		include_spip('inc/filtres_images_mini');
-	}
-
-	$res = [];
-
-	$result = sql_select(
-		'id_rubrique, id_parent, titre, descriptif, lang',
-		'spip_rubriques',
-		'id_parent=' . (int) $collection,
-		'',
-		'0+titre,titre',
-		$debut == -1 ? '' : "$debut,$limite"
-	);
-	while ($row = sql_fetch($result)) {
-		$id_rubrique = $row['id_rubrique'];
-		$id_parent = $row['id_parent'];
-		// pour etre sur de passer par tous les traitements
-		$titre = generer_objet_info($id_rubrique, 'rubrique', 'titre');
-		if ('' !== ($rang = recuperer_numero($row['titre']))) {
-			$rang = "<span class='rang'>$rang.</span> ";
-		}
-
-		if (autoriser('voir', 'rubrique', $id_rubrique)) {
-			$les_sous_enfants = sous_enfant_rub($id_rubrique);
-
-			changer_typo($row['lang']);
-			$lang_dir = lang_dir($row['lang']);
-			$descriptif = propre($row['descriptif']);
-
-			if ($voir_logo && ($logo = $chercher_logo($id_rubrique, 'id_rubrique', 'on'))) {
-				[$fid, $dir, $nom, $format] = $logo;
-				$logo = image_recadre_avec_fallback("<img src='$fid' alt='' />", 70, 70);
-				if ($logo) {
-					$logo = wrap(inserer_attribut($logo, 'class', 'logo'), '<span class="logo-carre">');
-				}
-			}
-
-			$lib_bouton = (acces_restreint_rubrique($id_rubrique)
-				? http_img_pack('auteur-0minirezo-16.png', '', " width='16' height='16'", _T('image_administrer_rubrique'))
-				: '') .
-				" <a class='titremlien' dir='$lang_dir'" .
-				($row['lang'] !== $GLOBALS['spip_lang'] ? " hreflang='" . $row['lang'] . "'" : '') .
-				" href='" .
-				generer_objet_url($id_rubrique, 'rubrique') .
-				"'><span class='titre'>" .
-				$rang . $titre
-				. '</span>'
-				. (is_string($logo) ? $logo : '')
-				. '</a>';
-
-			$titre = bouton_block_depliable($lib_bouton, $les_sous_enfants ? false : -1, "enfants$id_rubrique")
-				. ($descriptif ? "\n<div class='descriptif'>$descriptif</div>" : '')
-				;
-
-			$res[] =
-				debut_cadre_sous_rub(($id_parent ? 'rubrique-24.png' : 'secteur-24.png'), true, '', $titre) .
-				$les_sous_enfants .
-				fin_cadre_sous_rub();
-		}
-	}
-
-	changer_typo($GLOBALS['spip_lang']); # remettre la typo de l'interface pour la suite
-	return $res;
+    $voir_logo = (isset($GLOBALS['meta']['image_process']) && $GLOBALS['meta']['image_process'] != 'non');
+    $logo = '';
+
+    if ($voir_logo) {
+        $chercher_logo = charger_fonction('chercher_logo', 'inc');
+        include_spip('inc/filtres_images_mini');
+    }
+
+    $res = [];
+
+    $result = sql_select(
+        'id_rubrique, id_parent, titre, descriptif, lang',
+        'spip_rubriques',
+        'id_parent=' . (int) $collection,
+        '',
+        '0+titre,titre',
+        $debut == -1 ? '' : "$debut,$limite"
+    );
+    while ($row = sql_fetch($result)) {
+        $id_rubrique = $row['id_rubrique'];
+        $id_parent = $row['id_parent'];
+        // pour etre sur de passer par tous les traitements
+        $titre = generer_objet_info($id_rubrique, 'rubrique', 'titre');
+        if ('' !== ($rang = recuperer_numero($row['titre']))) {
+            $rang = "<span class='rang'>$rang.</span> ";
+        }
+
+        if (autoriser('voir', 'rubrique', $id_rubrique)) {
+            $les_sous_enfants = sous_enfant_rub($id_rubrique);
+
+            changer_typo($row['lang']);
+            $lang_dir = lang_dir($row['lang']);
+            $descriptif = propre($row['descriptif']);
+
+            if ($voir_logo && ($logo = $chercher_logo($id_rubrique, 'id_rubrique', 'on'))) {
+                [$fid, $dir, $nom, $format] = $logo;
+                $logo = image_recadre_avec_fallback("<img src='$fid' alt='' />", 70, 70);
+                if ($logo) {
+                    $logo = wrap(inserer_attribut($logo, 'class', 'logo'), '<span class="logo-carre">');
+                }
+            }
+
+            $lib_bouton = (acces_restreint_rubrique($id_rubrique)
+                ? http_img_pack('auteur-0minirezo-16.png', '', " width='16' height='16'", _T('image_administrer_rubrique'))
+                : '') .
+                " <a class='titremlien' dir='$lang_dir'" .
+                ($row['lang'] !== $GLOBALS['spip_lang'] ? " hreflang='" . $row['lang'] . "'" : '') .
+                " href='" .
+                generer_objet_url($id_rubrique, 'rubrique') .
+                "'><span class='titre'>" .
+                $rang . $titre
+                . '</span>'
+                . (is_string($logo) ? $logo : '')
+                . '</a>';
+
+            $titre = bouton_block_depliable($lib_bouton, $les_sous_enfants ? false : -1, "enfants$id_rubrique")
+                . ($descriptif ? "\n<div class='descriptif'>$descriptif</div>" : '')
+                ;
+
+            $res[] =
+                debut_cadre_sous_rub(($id_parent ? 'rubrique-24.png' : 'secteur-24.png'), true, '', $titre) .
+                $les_sous_enfants .
+                fin_cadre_sous_rub();
+        }
+    }
+
+    changer_typo($GLOBALS['spip_lang']); # remettre la typo de l'interface pour la suite
+    return $res;
 }
 
 /**
@@ -109,71 +109,71 @@ 
  *  Le contenu du bloc dépliable
  */
 function sous_enfant_rub($collection2) {
-	$nb = sql_countsel('spip_rubriques', 'id_parent=' . (int) $collection2);
-
-	$retour = '';
-	$pagination = '';
-	$debut = 0;
-	$limite = 500;
-
-	/**
-	 * On ne va afficher que 500 résultats max
-	 * Si > 500 on affiche une pagination
-	 */
-	if ($nb > $limite) {
-		$debut = _request('debut_rubrique' . $collection2) ?: $debut;
-		$pagination = chercher_filtre('pagination');
-		$pagination = '<nav class="pagination">' . $pagination(
-			$nb,
-			'_rubrique' . $collection2,
-			$debut,
-			$limite,
-			true,
-			'prive'
-		) . '</nav>';
-		$limite = $debut + $limite;
-	}
-
-	$result = sql_select(
-		'id_rubrique, id_parent, titre, lang',
-		'spip_rubriques',
-		'id_parent=' . (int) $collection2,
-		'',
-		'0+titre,titre',
-		$debut == -1 ? '' : "$debut,$limite"
-	);
-
-	while ($row = sql_fetch($result)) {
-		$id_rubrique2 = $row['id_rubrique'];
-		$titre2 = generer_objet_info(
-			$id_rubrique2,
-			'rubrique',
-			'titre'
-		); // pour etre sur de passer par tous les traitements
-		if ('' !== ($rang2 = recuperer_numero($row['titre']))) {
-			$rang2 = "<span class='rang'>$rang2.</span> ";
-		}
-
-		changer_typo($row['lang']);
-		$lang_dir = lang_dir($row['lang']);
-		if (autoriser('voir', 'rubrique', $id_rubrique2)) {
-			$retour .= "\n<li class='item' dir='$lang_dir'><a href='" . generer_objet_url(
-				$id_rubrique2,
-				'rubrique'
-			) . "'>" . $rang2 . $titre2 . "</a></li>\n";
-		}
-	}
-
-	$retour = $pagination . $retour . $pagination;
-
-	if (!$retour) {
-		return '';
-	}
-
-	return debut_block_depliable($debut > 0, "enfants$collection2")
-	. "\n<ul class='liste-items sous-sous-rub'>\n"
-	. $retour
-	. "</ul>\n" . fin_block() . "\n\n";
+    $nb = sql_countsel('spip_rubriques', 'id_parent=' . (int) $collection2);
+
+    $retour = '';
+    $pagination = '';
+    $debut = 0;
+    $limite = 500;
+
+    /**
+     * On ne va afficher que 500 résultats max
+     * Si > 500 on affiche une pagination
+     */
+    if ($nb > $limite) {
+        $debut = _request('debut_rubrique' . $collection2) ?: $debut;
+        $pagination = chercher_filtre('pagination');
+        $pagination = '<nav class="pagination">' . $pagination(
+            $nb,
+            '_rubrique' . $collection2,
+            $debut,
+            $limite,
+            true,
+            'prive'
+        ) . '</nav>';
+        $limite = $debut + $limite;
+    }
+
+    $result = sql_select(
+        'id_rubrique, id_parent, titre, lang',
+        'spip_rubriques',
+        'id_parent=' . (int) $collection2,
+        '',
+        '0+titre,titre',
+        $debut == -1 ? '' : "$debut,$limite"
+    );
+
+    while ($row = sql_fetch($result)) {
+        $id_rubrique2 = $row['id_rubrique'];
+        $titre2 = generer_objet_info(
+            $id_rubrique2,
+            'rubrique',
+            'titre'
+        ); // pour etre sur de passer par tous les traitements
+        if ('' !== ($rang2 = recuperer_numero($row['titre']))) {
+            $rang2 = "<span class='rang'>$rang2.</span> ";
+        }
+
+        changer_typo($row['lang']);
+        $lang_dir = lang_dir($row['lang']);
+        if (autoriser('voir', 'rubrique', $id_rubrique2)) {
+            $retour .= "\n<li class='item' dir='$lang_dir'><a href='" . generer_objet_url(
+                $id_rubrique2,
+                'rubrique'
+            ) . "'>" . $rang2 . $titre2 . "</a></li>\n";
+        }
+    }
+
+    $retour = $pagination . $retour . $pagination;
+
+    if (!$retour) {
+        return '';
+    }
+
+    return debut_block_depliable($debut > 0, "enfants$collection2")
+    . "\n<ul class='liste-items sous-sous-rub'>\n"
+    . $retour
+    . "</ul>\n" . fin_block() . "\n\n";
 }
 
 /**
@@ -188,41 +188,41 @@ 
  *  Le contenu textuel affiché, la liste des sous rubriques
  */
 function afficher_enfant_rub($id_rubrique = 0) {
-	$pagination = '';
-	$debut = 0;
-	$limite = 500;
-
-	$nb = sql_countsel('spip_rubriques', 'id_parent=' . (int) $id_rubrique);
-
-	if ($nb > $limite) {
-		$debut = _request('debut_rubrique' . $id_rubrique) ?: $debut;
-		$pagination = chercher_filtre('pagination');
-		$pagination = '<br class="nettoyeur"><nav class="pagination">' .
-			$pagination($nb, '_rubrique' . $id_rubrique, $debut, $limite, true, 'prive') .
-		'</nav>';
-	}
-
-	$les_enfants = enfant_rub($id_rubrique, $debut, $limite);
-
-	if (!$n = count($les_enfants)) {
-		return '';
-	}
-
-	if ($n == 1) {
-		$les_enfants = reset($les_enfants);
-		$les_enfants2 = '';
-	} else {
-		$n = ceil($n / 2);
-		$les_enfants2 = implode('', array_slice($les_enfants, $n));
-		$les_enfants = implode('', array_slice($les_enfants, 0, $n));
-	}
-
-	return $pagination
-		. "<div class='gauche'>"
-		. $les_enfants
-		. '</div>'
-		. "<div class='droite'>"
-		. $les_enfants2
-		. '</div>'
-		. $pagination;
+    $pagination = '';
+    $debut = 0;
+    $limite = 500;
+
+    $nb = sql_countsel('spip_rubriques', 'id_parent=' . (int) $id_rubrique);
+
+    if ($nb > $limite) {
+        $debut = _request('debut_rubrique' . $id_rubrique) ?: $debut;
+        $pagination = chercher_filtre('pagination');
+        $pagination = '<br class="nettoyeur"><nav class="pagination">' .
+            $pagination($nb, '_rubrique' . $id_rubrique, $debut, $limite, true, 'prive') .
+        '</nav>';
+    }
+
+    $les_enfants = enfant_rub($id_rubrique, $debut, $limite);
+
+    if (!$n = count($les_enfants)) {
+        return '';
+    }
+
+    if ($n == 1) {
+        $les_enfants = reset($les_enfants);
+        $les_enfants2 = '';
+    } else {
+        $n = ceil($n / 2);
+        $les_enfants2 = implode('', array_slice($les_enfants, $n));
+        $les_enfants = implode('', array_slice($les_enfants, 0, $n));
+    }
+
+    return $pagination
+        . "<div class='gauche'>"
+        . $les_enfants
+        . '</div>'
+        . "<div class='droite'>"
+        . $les_enfants2
+        . '</div>'
+        . $pagination;
 }

Spacing +24 added lines, -24 removed lines

@@ -42,7 +42,7 @@ 
 	$result = sql_select(
 		'id_rubrique, id_parent, titre, descriptif, lang',
 		'spip_rubriques',
-		'id_parent=' . (int) $collection,
+		'id_parent='.(int) $collection,
 		'',
 		'0+titre,titre',
 		$debut == -1 ? '' : "$debut,$limite"
@@ -73,13 +73,13 @@ 
 
 			$lib_bouton = (acces_restreint_rubrique($id_rubrique)
 				? http_img_pack('auteur-0minirezo-16.png', '', " width='16' height='16'", _T('image_administrer_rubrique'))
-				: '') .
-				" <a class='titremlien' dir='$lang_dir'" .
-				($row['lang'] !== $GLOBALS['spip_lang'] ? " hreflang='" . $row['lang'] . "'" : '') .
-				" href='" .
-				generer_objet_url($id_rubrique, 'rubrique') .
-				"'><span class='titre'>" .
-				$rang . $titre
+				: '').
+				" <a class='titremlien' dir='$lang_dir'".
+				($row['lang'] !== $GLOBALS['spip_lang'] ? " hreflang='".$row['lang']."'" : '').
+				" href='".
+				generer_objet_url($id_rubrique, 'rubrique').
+				"'><span class='titre'>".
+				$rang.$titre
 				. '</span>'
 				. (is_string($logo) ? $logo : '')
 				. '</a>';
@@ -89,8 +89,8 @@ 
 				;
 
 			$res[] =
-				debut_cadre_sous_rub(($id_parent ? 'rubrique-24.png' : 'secteur-24.png'), true, '', $titre) .
-				$les_sous_enfants .
+				debut_cadre_sous_rub(($id_parent ? 'rubrique-24.png' : 'secteur-24.png'), true, '', $titre).
+				$les_sous_enfants.
 				fin_cadre_sous_rub();
 		}
 	}
@@ -109,7 +109,7 @@ 
  *  Le contenu du bloc dépliable
  */
 function sous_enfant_rub($collection2) {
-	$nb = sql_countsel('spip_rubriques', 'id_parent=' . (int) $collection2);
+	$nb = sql_countsel('spip_rubriques', 'id_parent='.(int) $collection2);
 
 	$retour = '';
 	$pagination = '';
@@ -121,23 +121,23 @@ 
 	 * Si > 500 on affiche une pagination
 	 */
 	if ($nb > $limite) {
-		$debut = _request('debut_rubrique' . $collection2) ?: $debut;
+		$debut = _request('debut_rubrique'.$collection2) ?: $debut;
 		$pagination = chercher_filtre('pagination');
-		$pagination = '<nav class="pagination">' . $pagination(
+		$pagination = '<nav class="pagination">'.$pagination(
 			$nb,
-			'_rubrique' . $collection2,
+			'_rubrique'.$collection2,
 			$debut,
 			$limite,
 			true,
 			'prive'
-		) . '</nav>';
+		).'</nav>';
 		$limite = $debut + $limite;
 	}
 
 	$result = sql_select(
 		'id_rubrique, id_parent, titre, lang',
 		'spip_rubriques',
-		'id_parent=' . (int) $collection2,
+		'id_parent='.(int) $collection2,
 		'',
 		'0+titre,titre',
 		$debut == -1 ? '' : "$debut,$limite"
@@ -157,14 +157,14 @@ 
 		changer_typo($row['lang']);
 		$lang_dir = lang_dir($row['lang']);
 		if (autoriser('voir', 'rubrique', $id_rubrique2)) {
-			$retour .= "\n<li class='item' dir='$lang_dir'><a href='" . generer_objet_url(
+			$retour .= "\n<li class='item' dir='$lang_dir'><a href='".generer_objet_url(
 				$id_rubrique2,
 				'rubrique'
-			) . "'>" . $rang2 . $titre2 . "</a></li>\n";
+			)."'>".$rang2.$titre2."</a></li>\n";
 		}
 	}
 
-	$retour = $pagination . $retour . $pagination;
+	$retour = $pagination.$retour.$pagination;
 
 	if (!$retour) {
 		return '';
@@ -173,7 +173,7 @@ 
 	return debut_block_depliable($debut > 0, "enfants$collection2")
 	. "\n<ul class='liste-items sous-sous-rub'>\n"
 	. $retour
-	. "</ul>\n" . fin_block() . "\n\n";
+	. "</ul>\n".fin_block()."\n\n";
 }
 
 /**
@@ -192,13 +192,13 @@ 
 	$debut = 0;
 	$limite = 500;
 
-	$nb = sql_countsel('spip_rubriques', 'id_parent=' . (int) $id_rubrique);
+	$nb = sql_countsel('spip_rubriques', 'id_parent='.(int) $id_rubrique);
 
 	if ($nb > $limite) {
-		$debut = _request('debut_rubrique' . $id_rubrique) ?: $debut;
+		$debut = _request('debut_rubrique'.$id_rubrique) ?: $debut;
 		$pagination = chercher_filtre('pagination');
-		$pagination = '<br class="nettoyeur"><nav class="pagination">' .
-			$pagination($nb, '_rubrique' . $id_rubrique, $debut, $limite, true, 'prive') .
+		$pagination = '<br class="nettoyeur"><nav class="pagination">'.
+			$pagination($nb, '_rubrique'.$id_rubrique, $debut, $limite, true, 'prive').
 		'</nav>';
 	}
 

ecrire/inc/preselectionner_parent_nouvel_objet.php

Indentation +34 added lines, -34 removed lines

@@ -10,10 +10,10 @@ 
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 if (!defined('_AUTO_SELECTION_RUBRIQUE')) {
-	define('_AUTO_SELECTION_RUBRIQUE', false);
+    define('_AUTO_SELECTION_RUBRIQUE', false);
 }
 
 
@@ -27,39 +27,39 @@ 
  * @return string
  */
 function inc_preselectionner_parent_nouvel_objet_dist($objet, $row) {
-	if (!_AUTO_SELECTION_RUBRIQUE) {
-		return '';
-	}
+    if (!_AUTO_SELECTION_RUBRIQUE) {
+        return '';
+    }
 
-	if (!isset($row['id_rubrique'])) {
-		return '';
-	}
+    if (!isset($row['id_rubrique'])) {
+        return '';
+    }
 
-	$id_rubrique = '';
-	if ($GLOBALS['connect_id_rubrique']) {
-		// si admin restreint : sa rubrique
-		$id_rubrique = $GLOBALS['connect_id_rubrique'][0];
-	} elseif (
-		is_int(_AUTO_SELECTION_RUBRIQUE)
-		&& sql_fetsel('id_rubrique', 'spip_rubriques', 'id_rubrique=' . (int) _AUTO_SELECTION_RUBRIQUE)
-	) {
-		$id_rubrique = _AUTO_SELECTION_RUBRIQUE;
-	} else {
-		// sinon la derniere rubrique cree
-		$row_rub = sql_fetsel('id_rubrique', 'spip_rubriques', '', '', 'id_rubrique DESC', '0,1');
-		$id_rubrique = $row_rub['id_rubrique'];
-	}
-	// si le choix ne convient pas, on cherche dans un secteur
-	if (!autoriser('creer' . $objet . 'dans', 'rubrique', $id_rubrique)) {
-		$id_rubrique = '';
-		// manque de chance, la rubrique n'est pas autorisee, on cherche un des secteurs autorises
-		$res = sql_select('id_rubrique', 'spip_rubriques', 'id_parent=0');
-		while (!$id_rubrique && ($row_rub = sql_fetch($res))) {
-			if (autoriser('creer' . $objet . 'dans', 'rubrique', $row_rub['id_rubrique'])) {
-				$id_rubrique = $row_rub['id_rubrique'];
-			}
-		}
-	}
+    $id_rubrique = '';
+    if ($GLOBALS['connect_id_rubrique']) {
+        // si admin restreint : sa rubrique
+        $id_rubrique = $GLOBALS['connect_id_rubrique'][0];
+    } elseif (
+        is_int(_AUTO_SELECTION_RUBRIQUE)
+        && sql_fetsel('id_rubrique', 'spip_rubriques', 'id_rubrique=' . (int) _AUTO_SELECTION_RUBRIQUE)
+    ) {
+        $id_rubrique = _AUTO_SELECTION_RUBRIQUE;
+    } else {
+        // sinon la derniere rubrique cree
+        $row_rub = sql_fetsel('id_rubrique', 'spip_rubriques', '', '', 'id_rubrique DESC', '0,1');
+        $id_rubrique = $row_rub['id_rubrique'];
+    }
+    // si le choix ne convient pas, on cherche dans un secteur
+    if (!autoriser('creer' . $objet . 'dans', 'rubrique', $id_rubrique)) {
+        $id_rubrique = '';
+        // manque de chance, la rubrique n'est pas autorisee, on cherche un des secteurs autorises
+        $res = sql_select('id_rubrique', 'spip_rubriques', 'id_parent=0');
+        while (!$id_rubrique && ($row_rub = sql_fetch($res))) {
+            if (autoriser('creer' . $objet . 'dans', 'rubrique', $row_rub['id_rubrique'])) {
+                $id_rubrique = $row_rub['id_rubrique'];
+            }
+        }
+    }
 
-	return $id_rubrique;
+    return $id_rubrique;
 }

Spacing +3 added lines, -3 removed lines

@@ -41,7 +41,7 @@ 
 		$id_rubrique = $GLOBALS['connect_id_rubrique'][0];
 	} elseif (
 		is_int(_AUTO_SELECTION_RUBRIQUE)
-		&& sql_fetsel('id_rubrique', 'spip_rubriques', 'id_rubrique=' . (int) _AUTO_SELECTION_RUBRIQUE)
+		&& sql_fetsel('id_rubrique', 'spip_rubriques', 'id_rubrique='.(int) _AUTO_SELECTION_RUBRIQUE)
 	) {
 		$id_rubrique = _AUTO_SELECTION_RUBRIQUE;
 	} else {
@@ -50,12 +50,12 @@ 
 		$id_rubrique = $row_rub['id_rubrique'];
 	}
 	// si le choix ne convient pas, on cherche dans un secteur
-	if (!autoriser('creer' . $objet . 'dans', 'rubrique', $id_rubrique)) {
+	if (!autoriser('creer'.$objet.'dans', 'rubrique', $id_rubrique)) {
 		$id_rubrique = '';
 		// manque de chance, la rubrique n'est pas autorisee, on cherche un des secteurs autorises
 		$res = sql_select('id_rubrique', 'spip_rubriques', 'id_parent=0');
 		while (!$id_rubrique && ($row_rub = sql_fetch($res))) {
-			if (autoriser('creer' . $objet . 'dans', 'rubrique', $row_rub['id_rubrique'])) {
+			if (autoriser('creer'.$objet.'dans', 'rubrique', $row_rub['id_rubrique'])) {
 				$id_rubrique = $row_rub['id_rubrique'];
 			}
 		}

ecrire/inc/nfslock.php

Indentation +88 added lines, -88 removed lines

@@ -7,7 +7,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/acces');
@@ -101,93 +101,93 @@ 
  * @return int|bool Timestamp du verrou, false si erreur
  */
 function spip_nfslock($fichier, $max_age = 0) {
-	$tries = 0;
+    $tries = 0;
 
-	if (!$max_age) {
-		$max_age = _DEFAULT_LOCKTIME;
-	}
-	$lock_file = _DIR_TMP . _NAME_LOCK . '-' . substr(md5($fichier), 0, 8);
+    if (!$max_age) {
+        $max_age = _DEFAULT_LOCKTIME;
+    }
+    $lock_file = _DIR_TMP . _NAME_LOCK . '-' . substr(md5($fichier), 0, 8);
 
 
-	/*
+    /*
 	 * 1. create a tmp file with a psuedo random file name. we also make
 	 *    tpath which is a buffer to store the full pathname of the tmp file.
 	 */
 
-	$id = creer_uniqid();
-	$tpath = _DIR_TMP . "slock.$id";
-	$tmpfd = @fopen($tpath, 'w'); // hum, le 'x' necessite php4,3,2 ...
-	if (!$tmpfd) {  /* open failed */
-		@fclose($tmpfd);
-		spip_unlink($tpath);
+    $id = creer_uniqid();
+    $tpath = _DIR_TMP . "slock.$id";
+    $tmpfd = @fopen($tpath, 'w'); // hum, le 'x' necessite php4,3,2 ...
+    if (!$tmpfd) {  /* open failed */
+        @fclose($tmpfd);
+        spip_unlink($tpath);
 
-		return false; //NFSL_SYSF
-	}
+        return false; //NFSL_SYSF
+    }
 
-	/*
+    /*
 	 * 2. make fullpath, a buffer for the full pathname of the lock file.
 	 *    then start looping trying to lock it
 	 */
 
-	while ($tries < 10) {
-		/*
+    while ($tries < 10) {
+        /*
 		 * 3. link tmp file to lock file.  if it goes, we win and we clean
 		 *    up and return the st_ctime of the lock file.
 		 */
 
-		if (link($tpath, $lock_file) == 1) {
-			spip_unlink($tpath); /* got it! */
-			@fclose($tmpfd);
-			if (($our_tmp = lstat($lock_file)) == false) {  /* stat failed... shouldn't happen */
-				spip_unlink($lock_file);
+        if (link($tpath, $lock_file) == 1) {
+            spip_unlink($tpath); /* got it! */
+            @fclose($tmpfd);
+            if (($our_tmp = lstat($lock_file)) == false) {  /* stat failed... shouldn't happen */
+                spip_unlink($lock_file);
 
-				return false; // (NFSL_SYSF);
-			}
+                return false; // (NFSL_SYSF);
+            }
 
-			return ($our_tmp['ctime']);
-		}
+            return ($our_tmp['ctime']);
+        }
 
-		/*
+        /*
 		 * 4. the lock failed.  check for a stale lock file, being mindful
 		 *    of NFS and the fact the time is set from the NFS server.  we
 		 *    do a write on the tmp file to update its time to the server's
 		 *    idea of "now."
 		 */
 
-		$old_stat = lstat($lock_file);
-		if (@fwrite($tmpfd, 'zz', 2) != 2 || !$our_tmp = fstat($tmpfd)) {
-			break;
-		} /* something bogus is going on */
+        $old_stat = lstat($lock_file);
+        if (@fwrite($tmpfd, 'zz', 2) != 2 || !$our_tmp = fstat($tmpfd)) {
+            break;
+        } /* something bogus is going on */
 
 
-		if ($old_stat != false && (($old_stat['ctime'] + $max_age) < $our_tmp['ctime'])) {
-			spip_unlink($lock_file); /* break the stale lock */
-			$tries++;
-			/* It is CRITICAL that we sleep after breaking
+        if ($old_stat != false && (($old_stat['ctime'] + $max_age) < $our_tmp['ctime'])) {
+            spip_unlink($lock_file); /* break the stale lock */
+            $tries++;
+            /* It is CRITICAL that we sleep after breaking
 			 * the lock. Otherwise, we could race with
 			 * another process and unlink it's newly-
 			 * created file.
 			 */
-			sleep(1 + random_int(0, 4));
-			continue;
-		}
+            sleep(1 + random_int(0, 4));
+            continue;
+        }
 
-		/*
+        /*
 		 * 5. try again
 		 */
 
-		$tries++;
-		sleep(1 + random_int(0, 4));
-	}
+        $tries++;
+        sleep(1 + random_int(0, 4));
+    }
 
-	/*
+    /*
 	 * 6. give up, failure.
 	 */
 
-	spip_unlink($tpath);
-	@fclose($tmpfd);
+    spip_unlink($tpath);
+    @fclose($tmpfd);
 
-	return false; //(NFSL_LOCKED);
+    return false; //(NFSL_LOCKED);
 }
 
 /**
@@ -231,71 +231,71 @@ 
  * return bool true si déverrouillé, false sinon
  */
 function spip_nfsunlock($fichier, $birth, $max_age = 0, $test = false) {
-	$id = creer_uniqid();
-	if (!$max_age) {
-		$max_age = _DEFAULT_LOCKTIME;
-	}
+    $id = creer_uniqid();
+    if (!$max_age) {
+        $max_age = _DEFAULT_LOCKTIME;
+    }
 
-	/*
+    /*
 	 * 1. Build a temp file and stat that to get an idea of what the server
 	 *    thinks the current time is (our_tmp.st_ctime)..
 	 */
 
-	$tpath = _DIR_TMP . "stime.$id";
-	$tmpfd = @fopen($tpath, 'w');
-	if (!$tmpfd || @fwrite($tmpfd, 'zz', 2) != 2 || !($our_tmp = fstat($tmpfd))) {
-		/* The open failed, or we can't write the file, or we can't stat it */
-		@fclose($tmpfd);
-		spip_unlink($tpath);
+    $tpath = _DIR_TMP . "stime.$id";
+    $tmpfd = @fopen($tpath, 'w');
+    if (!$tmpfd || @fwrite($tmpfd, 'zz', 2) != 2 || !($our_tmp = fstat($tmpfd))) {
+        /* The open failed, or we can't write the file, or we can't stat it */
+        @fclose($tmpfd);
+        spip_unlink($tpath);
 
-		return false; //(NFSL_SYSF);
-	}
+        return false; //(NFSL_SYSF);
+    }
 
-	@fclose($tmpfd);    /* We don't need this once we have our_tmp.st_ctime. */
-	spip_unlink($tpath);
+    @fclose($tmpfd);    /* We don't need this once we have our_tmp.st_ctime. */
+    spip_unlink($tpath);
 
-	/*
+    /*
 	 * 2. make fullpath, a buffer for the full pathname of the lock file
 	 */
 
-	$lock_file = _DIR_TMP . _NAME_LOCK . '-' . substr(md5($fichier), 0, 8);
+    $lock_file = _DIR_TMP . _NAME_LOCK . '-' . substr(md5($fichier), 0, 8);
 
-	/*
+    /*
 	 * 3. If the ctime hasn't been modified, unlink the file and return. If the
 	 *    lock has expired, sleep the usual random interval before returning.
 	 *    If we didn't sleep, there could be a race if the caller immediately
 	 *    tries to relock the file.
 	 */
 
-	if (
-		($old_stat = @lstat($lock_file))  /* stat succeeds so file is there */
-		&& ($old_stat['ctime'] == $birth)
-	) {  /* hasn't been modified since birth */
-		if (!$test) {
-			spip_unlink($lock_file);
-		}      /* so the lock is ours to remove */
-		if ($our_tmp['ctime'] >= $birth + $max_age) {  /* the lock has expired */
-			if (!$test) {
-				return false;
-			} //(NFSL_LOST);
-			sleep(1 + (random_int(0, 4)));    /* so sleep a bit */
-		}
-
-		return true;//(NFSL_OK);			/* success */
-	}
-
-	/*
+    if (
+        ($old_stat = @lstat($lock_file))  /* stat succeeds so file is there */
+        && ($old_stat['ctime'] == $birth)
+    ) {  /* hasn't been modified since birth */
+        if (!$test) {
+            spip_unlink($lock_file);
+        }      /* so the lock is ours to remove */
+        if ($our_tmp['ctime'] >= $birth + $max_age) {  /* the lock has expired */
+            if (!$test) {
+                return false;
+            } //(NFSL_LOST);
+            sleep(1 + (random_int(0, 4)));    /* so sleep a bit */
+        }
+
+        return true;//(NFSL_OK);			/* success */
+    }
+
+    /*
 	 * 4. Either ctime has been modified, or the entire lock file is missing.
 	 *    If the lock should still be ours, based on the ctime of the temp
 	 *    file, return with NFSL_STOLEN. If not, then our lock is expired and
 	 *    someone else has grabbed the file, so return NFSL_LOST.
 	 */
 
-	if ($our_tmp['ctime'] < $birth + $max_age) { /* lock was stolen */
-		return false;
-	} //(NFSL_STOLEN);
+    if ($our_tmp['ctime'] < $birth + $max_age) { /* lock was stolen */
+        return false;
+    } //(NFSL_STOLEN);
 
-	return false; //(NFSL_LOST);	/* The lock must have expired first. */
+    return false; //(NFSL_LOST);	/* The lock must have expired first. */
 }
 
 
@@ -319,5 +319,5 @@ 
  * return bool true si déverrouillé, false sinon
  */
 function spip_nfslock_test($fichier, $birth, $max_age = 0) {
-	return spip_nfsunlock($fichier, $birth, $max_age, true);
+    return spip_nfsunlock($fichier, $birth, $max_age, true);
 }

Spacing +7 added lines, -7 removed lines

@@ -106,7 +106,7 @@ 
 	if (!$max_age) {
 		$max_age = _DEFAULT_LOCKTIME;
 	}
-	$lock_file = _DIR_TMP . _NAME_LOCK . '-' . substr(md5($fichier), 0, 8);
+	$lock_file = _DIR_TMP._NAME_LOCK.'-'.substr(md5($fichier), 0, 8);
 
 
 	/*
@@ -115,7 +115,7 @@ 
 	 */
 
 	$id = creer_uniqid();
-	$tpath = _DIR_TMP . "slock.$id";
+	$tpath = _DIR_TMP."slock.$id";
 	$tmpfd = @fopen($tpath, 'w'); // hum, le 'x' necessite php4,3,2 ...
 	if (!$tmpfd) {  /* open failed */
 		@fclose($tmpfd);
@@ -241,7 +241,7 @@ 
 	 *    thinks the current time is (our_tmp.st_ctime)..
 	 */
 
-	$tpath = _DIR_TMP . "stime.$id";
+	$tpath = _DIR_TMP."stime.$id";
 	$tmpfd = @fopen($tpath, 'w');
 	if (!$tmpfd || @fwrite($tmpfd, 'zz', 2) != 2 || !($our_tmp = fstat($tmpfd))) {
 		/* The open failed, or we can't write the file, or we can't stat it */
@@ -251,14 +251,14 @@ 
 		return false; //(NFSL_SYSF);
 	}
 
-	@fclose($tmpfd);    /* We don't need this once we have our_tmp.st_ctime. */
+	@fclose($tmpfd); /* We don't need this once we have our_tmp.st_ctime. */
 	spip_unlink($tpath);
 
 	/*
 	 * 2. make fullpath, a buffer for the full pathname of the lock file
 	 */
 
-	$lock_file = _DIR_TMP . _NAME_LOCK . '-' . substr(md5($fichier), 0, 8);
+	$lock_file = _DIR_TMP._NAME_LOCK.'-'.substr(md5($fichier), 0, 8);
 
 	/*
 	 * 3. If the ctime hasn't been modified, unlink the file and return. If the
@@ -278,10 +278,10 @@ 
 			if (!$test) {
 				return false;
 			} //(NFSL_LOST);
-			sleep(1 + (random_int(0, 4)));    /* so sleep a bit */
+			sleep(1 + (random_int(0, 4))); /* so sleep a bit */
 		}
 
-		return true;//(NFSL_OK);			/* success */
+		return true; //(NFSL_OK);			/* success */
 	}
 
 	/*