spip / SPIP

ecrire/exec/info_plugin.php

Indentation +20 added lines, -20 removed lines

@@ -7,7 +7,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/actions');
@@ -21,23 +21,23 @@ 
  * @uses ajax_retour()
  */
 function exec_info_plugin_dist() {
-	if (!autoriser('configurer', '_plugins')) {
-		include_spip('inc/minipres');
-		echo minipres();
-	} else {
-		$plug = _DIR_RACINE . htmlspecialchars((string) _request('plugin'));
-		$get_infos = charger_fonction('get_infos', 'plugins');
-		$dir = '';
-		if (str_starts_with($plug, (string) _DIR_PLUGINS)) {
-			$dir = _DIR_PLUGINS;
-		} elseif (str_starts_with($plug, (string) _DIR_PLUGINS_DIST)) {
-			$dir = _DIR_PLUGINS_DIST;
-		}
-		if ($dir) {
-			$plug = substr($plug, strlen((string) $dir));
-		}
-		$info = $get_infos($plug, false, $dir);
-		$afficher_plugin = charger_fonction('afficher_plugin', 'plugins');
-		ajax_retour(affiche_bloc_plugin($plug, $info, $dir));
-	}
+    if (!autoriser('configurer', '_plugins')) {
+        include_spip('inc/minipres');
+        echo minipres();
+    } else {
+        $plug = _DIR_RACINE . htmlspecialchars((string) _request('plugin'));
+        $get_infos = charger_fonction('get_infos', 'plugins');
+        $dir = '';
+        if (str_starts_with($plug, (string) _DIR_PLUGINS)) {
+            $dir = _DIR_PLUGINS;
+        } elseif (str_starts_with($plug, (string) _DIR_PLUGINS_DIST)) {
+            $dir = _DIR_PLUGINS_DIST;
+        }
+        if ($dir) {
+            $plug = substr($plug, strlen((string) $dir));
+        }
+        $info = $get_infos($plug, false, $dir);
+        $afficher_plugin = charger_fonction('afficher_plugin', 'plugins');
+        ajax_retour(affiche_bloc_plugin($plug, $info, $dir));
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
 		include_spip('inc/minipres');
 		echo minipres();
 	} else {
-		$plug = _DIR_RACINE . htmlspecialchars((string) _request('plugin'));
+		$plug = _DIR_RACINE.htmlspecialchars((string) _request('plugin'));
 		$get_infos = charger_fonction('get_infos', 'plugins');
 		$dir = '';
 		if (str_starts_with($plug, (string) _DIR_PLUGINS)) {

ecrire/exec/rechercher.php

Indentation +96 added lines, -96 removed lines

@@ -18,7 +18,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/actions');
@@ -31,13 +31,13 @@ 
  * @uses ajax_retour()
  **/
 function exec_rechercher_dist() {
-	$id = (int) _request('id');
-	$exclus = (int) _request('exclus');
-	$rac = spip_htmlentities(_request('rac'));
-	$type = _request('type');
-	$do = _request('do');
-	$r = preg_match('/^\w*$/', (string) $do) ? exec_rechercher_args($id, $type, $exclus, $rac, $do) : '';
-	ajax_retour($r);
+    $id = (int) _request('id');
+    $exclus = (int) _request('exclus');
+    $rac = spip_htmlentities(_request('rac'));
+    $type = _request('type');
+    $do = _request('do');
+    $r = preg_match('/^\w*$/', (string) $do) ? exec_rechercher_args($id, $type, $exclus, $rac, $do) : '';
+    ajax_retour($r);
 }
 
 /**
@@ -54,61 +54,61 @@ 
  * @return string
  **/
 function exec_rechercher_args($id, $type, $exclus, $rac, $do) {
-	if (!$do) {
-		$do = 'aff';
-	}
-
-	$points = $rub = [];
-
-	$where = preg_split(',\s+,', $type);
-	if ($where) {
-		foreach ($where as $k => $v) {
-			$where[$k] = "'%" . substr(str_replace('%', '\%', sql_quote($v, '', 'string')), 1, -1) . "%'";
-		}
-		$where_titre = ('(titre LIKE ' . implode(' AND titre LIKE ', $where) . ')');
-		$where_desc = ('(descriptif LIKE ' . implode(' AND descriptif LIKE ', $where) . ')');
-		$where_id = ('(id_rubrique = ' . (int) $type . ')');
-
-		if ($exclus) {
-			include_spip('inc/rubriques');
-			$where_exclus = ' AND ' . sql_in('id_rubrique', calcul_branche_in($exclus), 'NOT');
-		} else {
-			$where_exclus = '';
-		}
-
-		foreach (
-			[
-				3 => $where_titre,
-				2 => $where_desc,
-				1 => $where_id,
-			] as $point => $recherche
-		) {
-			$res = sql_select('id_rubrique, id_parent, titre', 'spip_rubriques', "$recherche$where_exclus");
-			while ($row = sql_fetch($res)) {
-				$id_rubrique = $row['id_rubrique'];
-				if (!isset($rub[$id_rubrique])) {
-					$rub[$id_rubrique] = [];
-				}
-				$rub[$id_rubrique]['titre'] = typo($row['titre']);
-				$rub[$id_rubrique]['id_parent'] = $row['id_parent'];
-				if (!isset($points[$id_rubrique])) {
-					$points[$id_rubrique] = 0;
-				}
-				$points[$id_rubrique] += $point;
-			}
-		}
-	}
-
-	if ($points) {
-		arsort($points);
-		$style = " style='background-image: url(" . chemin_image('secteur-12.png') . ")'";
-		foreach ($rub as $k => $v) {
-			$rub[$k]['atts'] = ($v['id_parent'] ? $style : '')
-				. " class='petite-rubrique'";
-		}
-	}
-
-	return (proposer_item($points, $rub, $rac, $type, $do));
+    if (!$do) {
+        $do = 'aff';
+    }
+
+    $points = $rub = [];
+
+    $where = preg_split(',\s+,', $type);
+    if ($where) {
+        foreach ($where as $k => $v) {
+            $where[$k] = "'%" . substr(str_replace('%', '\%', sql_quote($v, '', 'string')), 1, -1) . "%'";
+        }
+        $where_titre = ('(titre LIKE ' . implode(' AND titre LIKE ', $where) . ')');
+        $where_desc = ('(descriptif LIKE ' . implode(' AND descriptif LIKE ', $where) . ')');
+        $where_id = ('(id_rubrique = ' . (int) $type . ')');
+
+        if ($exclus) {
+            include_spip('inc/rubriques');
+            $where_exclus = ' AND ' . sql_in('id_rubrique', calcul_branche_in($exclus), 'NOT');
+        } else {
+            $where_exclus = '';
+        }
+
+        foreach (
+            [
+                3 => $where_titre,
+                2 => $where_desc,
+                1 => $where_id,
+            ] as $point => $recherche
+        ) {
+            $res = sql_select('id_rubrique, id_parent, titre', 'spip_rubriques', "$recherche$where_exclus");
+            while ($row = sql_fetch($res)) {
+                $id_rubrique = $row['id_rubrique'];
+                if (!isset($rub[$id_rubrique])) {
+                    $rub[$id_rubrique] = [];
+                }
+                $rub[$id_rubrique]['titre'] = typo($row['titre']);
+                $rub[$id_rubrique]['id_parent'] = $row['id_parent'];
+                if (!isset($points[$id_rubrique])) {
+                    $points[$id_rubrique] = 0;
+                }
+                $points[$id_rubrique] += $point;
+            }
+        }
+    }
+
+    if ($points) {
+        arsort($points);
+        $style = " style='background-image: url(" . chemin_image('secteur-12.png') . ")'";
+        foreach ($rub as $k => $v) {
+            $rub[$k]['atts'] = ($v['id_parent'] ? $style : '')
+                . " class='petite-rubrique'";
+        }
+    }
+
+    return (proposer_item($points, $rub, $rac, $type, $do));
 }
 
 
@@ -131,37 +131,37 @@ 
  **/
 function proposer_item($ids, $titles, $rac, $type, $do) {
 
-	if (!$ids) {
-		return "<br /><br /><div style='padding: 5px; color: red;'><b>"
-		. spip_htmlentities($type)
-		. '</b> :  ' . _T('avis_aucun_resultat') . '</div>';
-	}
-
-	$ret = '';
-	$info = generer_url_ecrire('informer', "type=rubrique&rac=$rac&id=");
-
-	$onClick = "aff_selection(this.firstChild.title,'$rac" . "_selection','$info', event)";
-
-	$ondbClick = "$do(this.firstChild.firstChild.nodeValue,this.firstChild.title,'selection_rubrique', 'id_parent');";
-
-	foreach (array_keys($ids) as $id) {
-		$titre = strtr(
-			str_replace("'", '&#8217;', str_replace('"', '&#34;', (string) textebrut($titles[$id]['titre']))),
-			"\n\r",
-			'  '
-		);
-
-		$ret .= "<div class='highlight off'\nonclick=\"changerhighlight(this); "
-			. $onClick
-			. "\"\nondblclick=\""
-			. $ondbClick
-			. $onClick
-			. ' "><div'
-			. $titles[$id]['atts']
-			. " title='$id'>&nbsp; "
-			. $titre
-			. '</div></div>';
-	}
-
-	return $ret;
+    if (!$ids) {
+        return "<br /><br /><div style='padding: 5px; color: red;'><b>"
+        . spip_htmlentities($type)
+        . '</b> :  ' . _T('avis_aucun_resultat') . '</div>';
+    }
+
+    $ret = '';
+    $info = generer_url_ecrire('informer', "type=rubrique&rac=$rac&id=");
+
+    $onClick = "aff_selection(this.firstChild.title,'$rac" . "_selection','$info', event)";
+
+    $ondbClick = "$do(this.firstChild.firstChild.nodeValue,this.firstChild.title,'selection_rubrique', 'id_parent');";
+
+    foreach (array_keys($ids) as $id) {
+        $titre = strtr(
+            str_replace("'", '&#8217;', str_replace('"', '&#34;', (string) textebrut($titles[$id]['titre']))),
+            "\n\r",
+            '  '
+        );
+
+        $ret .= "<div class='highlight off'\nonclick=\"changerhighlight(this); "
+            . $onClick
+            . "\"\nondblclick=\""
+            . $ondbClick
+            . $onClick
+            . ' "><div'
+            . $titles[$id]['atts']
+            . " title='$id'>&nbsp; "
+            . $titre
+            . '</div></div>';
+    }
+
+    return $ret;
 }

Spacing +8 added lines, -8 removed lines

@@ -63,15 +63,15 @@ 
 	$where = preg_split(',\s+,', $type);
 	if ($where) {
 		foreach ($where as $k => $v) {
-			$where[$k] = "'%" . substr(str_replace('%', '\%', sql_quote($v, '', 'string')), 1, -1) . "%'";
+			$where[$k] = "'%".substr(str_replace('%', '\%', sql_quote($v, '', 'string')), 1, -1)."%'";
 		}
-		$where_titre = ('(titre LIKE ' . implode(' AND titre LIKE ', $where) . ')');
-		$where_desc = ('(descriptif LIKE ' . implode(' AND descriptif LIKE ', $where) . ')');
-		$where_id = ('(id_rubrique = ' . (int) $type . ')');
+		$where_titre = ('(titre LIKE '.implode(' AND titre LIKE ', $where).')');
+		$where_desc = ('(descriptif LIKE '.implode(' AND descriptif LIKE ', $where).')');
+		$where_id = ('(id_rubrique = '.(int) $type.')');
 
 		if ($exclus) {
 			include_spip('inc/rubriques');
-			$where_exclus = ' AND ' . sql_in('id_rubrique', calcul_branche_in($exclus), 'NOT');
+			$where_exclus = ' AND '.sql_in('id_rubrique', calcul_branche_in($exclus), 'NOT');
 		} else {
 			$where_exclus = '';
 		}
@@ -101,7 +101,7 @@ 
 
 	if ($points) {
 		arsort($points);
-		$style = " style='background-image: url(" . chemin_image('secteur-12.png') . ")'";
+		$style = " style='background-image: url(".chemin_image('secteur-12.png').")'";
 		foreach ($rub as $k => $v) {
 			$rub[$k]['atts'] = ($v['id_parent'] ? $style : '')
 				. " class='petite-rubrique'";
@@ -134,13 +134,13 @@ 
 	if (!$ids) {
 		return "<br /><br /><div style='padding: 5px; color: red;'><b>"
 		. spip_htmlentities($type)
-		. '</b> :  ' . _T('avis_aucun_resultat') . '</div>';
+		. '</b> :  '._T('avis_aucun_resultat').'</div>';
 	}
 
 	$ret = '';
 	$info = generer_url_ecrire('informer', "type=rubrique&rac=$rac&id=");
 
-	$onClick = "aff_selection(this.firstChild.title,'$rac" . "_selection','$info', event)";
+	$onClick = "aff_selection(this.firstChild.title,'$rac"."_selection','$info', event)";
 
 	$ondbClick = "$do(this.firstChild.firstChild.nodeValue,this.firstChild.title,'selection_rubrique', 'id_parent');";
 

ecrire/exec/selectionner.php

Indentation +16 added lines, -16 removed lines

@@ -15,7 +15,7 @@ 
  * @package SPIP\Core\Rubriques
  **/
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/actions');
@@ -27,21 +27,21 @@ 
  * @uses ajax_retour()
  **/
 function exec_selectionner_dist() {
-	$id = (int) _request('id');
-	$exclus = (int) _request('exclus');
-	$type = _request('type');
-	$rac = _request('racine');
-	$do = _request('do');
-	if (preg_match('/^\w*$/', (string) $do)) {
-		if (!$do) {
-			$do = 'aff';
-		}
+    $id = (int) _request('id');
+    $exclus = (int) _request('exclus');
+    $type = _request('type');
+    $rac = _request('racine');
+    $do = _request('do');
+    if (preg_match('/^\w*$/', (string) $do)) {
+        if (!$do) {
+            $do = 'aff';
+        }
 
-		$selectionner = charger_fonction('selectionner', 'inc');
+        $selectionner = charger_fonction('selectionner', 'inc');
 
-		$r = $selectionner($id, 'choix_parent', $exclus, $rac, $type != 'breve', $do);
-	} else {
-		$r = '';
-	}
-	ajax_retour($r);
+        $r = $selectionner($id, 'choix_parent', $exclus, $rac, $type != 'breve', $do);
+    } else {
+        $r = '';
+    }
+    ajax_retour($r);
 }

ecrire/exec/admin_plugin.php

Indentation +202 added lines, -202 removed lines

@@ -16,7 +16,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 include_spip('inc/config');
@@ -36,26 +36,26 @@ 
  */
 function exec_admin_plugin_dist($retour = '') {
 
-	if (!autoriser('configurer', '_plugins')) {
-		include_spip('inc/minipres');
-		echo minipres();
-	} else {
-		// on fait la verif du path avant tout,
-		// et l'installation des qu'on est dans la colonne principale
-		// si jamais la liste des plugins actifs change, il faut faire un refresh du hit
-		// pour etre sur que les bons fichiers seront charges lors de l'install
-		$new = actualise_plugins_actifs();
-		if ($new && _request('actualise') < 2) {
-			include_spip('inc/headers');
-			if (isset($GLOBALS['fichier_php_compile_recent'])) {
-				// attendre eventuellement l'invalidation du cache opcode
-				spip_attend_invalidation_opcode_cache($GLOBALS['fichier_php_compile_recent']);
-			}
-			redirige_par_entete(parametre_url(self(), 'actualise', _request('actualise') + 1, '&'));
-		} else {
-			admin_plug_args(_request('voir'), _request('erreur'), _request('format'));
-		}
-	}
+    if (!autoriser('configurer', '_plugins')) {
+        include_spip('inc/minipres');
+        echo minipres();
+    } else {
+        // on fait la verif du path avant tout,
+        // et l'installation des qu'on est dans la colonne principale
+        // si jamais la liste des plugins actifs change, il faut faire un refresh du hit
+        // pour etre sur que les bons fichiers seront charges lors de l'install
+        $new = actualise_plugins_actifs();
+        if ($new && _request('actualise') < 2) {
+            include_spip('inc/headers');
+            if (isset($GLOBALS['fichier_php_compile_recent'])) {
+                // attendre eventuellement l'invalidation du cache opcode
+                spip_attend_invalidation_opcode_cache($GLOBALS['fichier_php_compile_recent']);
+            }
+            redirige_par_entete(parametre_url(self(), 'actualise', _request('actualise') + 1, '&'));
+        } else {
+            admin_plug_args(_request('voir'), _request('erreur'), _request('format'));
+        }
+    }
 }
 
 /**
@@ -80,132 +80,132 @@ 
  *     Format d'affichage (liste ou arborescence)
  **/
 function admin_plug_args($quoi, $erreur, $format) {
-	$lpf = null;
-	$lcpas = null;
-	$lcpaffichesup = null;
-	if (!$quoi) {
-		$quoi = 'actifs';
-	}
-	// empecher l'affichage des erreurs dans le bandeau, on le donne ensuite
-	// format brut par plugin
-	$GLOBALS['erreurs_activation_raw'] = plugin_donne_erreurs(true, false);
-	// format resume mis en forme
-	$erreur_activation = plugin_donne_erreurs();
-	$commencer_page = charger_fonction('commencer_page', 'inc');
-	echo $commencer_page(_T('icone_admin_plugin'), 'configuration', 'plugin');
-
-	echo debut_gauche();
-	echo recuperer_fond('prive/squelettes/navigation/configurer', ['exec' => 'admin_plugin']);
-
-	echo pipeline(
-		'affiche_gauche',
-		[
-			'args' => ['exec' => 'admin_plugin'],
-			'data' => afficher_librairies()
-		]
-	);
-
-	echo debut_droite();
-	echo gros_titre(_T('icone_admin_plugin'), '');
-
-	// Barre d'onglets de premier niveau
-	echo barre_onglets('plugins', 'plugins_actifs');
-	// Barre d'onglets de second niveau
-	$onglet2 = $quoi == 'actifs' ? 'plugins_actifs' : 'admin_plugin';
-	echo debut_onglet('onglets_simple second');
-	echo onglet(_T('plugins_tous_liste'), generer_url_ecrire('admin_plugin', 'voir=tous'), 'admin_plugin', $onglet2);
-	echo onglet(_T('plugins_actifs_liste'), generer_url_ecrire('admin_plugin'), 'plugins_actifs', $onglet2);
-	echo fin_onglet();
-
-	// message d'erreur au retour d'une operation
-	if ($erreur) {
-		echo "<div class='error'>" . spip_htmlspecialchars($erreur) . '</div>';
-	}
-	if ($erreur_activation) {
-		echo "<div class='error'>$erreur_activation</div>";
-	}
-
-	// la mise a jour de cette meta a ete faite par ecrire_plugin_actifs
-	$actifs = (array) unserialize($GLOBALS['meta']['plugin']);
-	$lcpa = $actifs + (array) unserialize($GLOBALS['meta']['plugin_attente']);
-
-	// Les affichages se basent sur le repertoire, pas sur le nom
-	$actifs = liste_chemin_plugin($actifs, '');
-	if (defined('_DIR_PLUGINS_SUPPL')) {
-		$lcpas = liste_chemin_plugin($lcpa, _DIR_PLUGINS_SUPPL);
-	}
-	$lcpa = liste_chemin_plugin($lcpa);
-
-	// on installe les plugins maintenant,
-	// cela permet aux scripts d'install de faire des affichages (moches...)
-	plugin_installes_meta();
-
-	echo "<div class='liste-plugins formulaire_spip'>";
-	echo debut_cadre_trait_couleur('plugin-24.png', true, '', _T('plugins_liste'), 'plugins');
-
-	if ($quoi !== 'actifs') {
-		$lpf = liste_plugin_files();
-		if ($lpf) {
-			echo '<p>' . _T('texte_presente_plugin') . '</p>';
-		} else {
-			if (!@is_dir(_DIR_PLUGINS)) {
-				echo '<p>' . _T('plugin_info_automatique_ftp', ['rep' => joli_repertoire(_DIR_PLUGINS)])
-					. ' &mdash; ' . _T('plugin_info_automatique_creer') . '</p>';
-			}
-		}
-		$lcpaffiche = $lpf;
-		if (defined('_DIR_PLUGINS_SUPPL')) {
-			$lcpaffichesup = liste_plugin_files(_DIR_PLUGINS_SUPPL);
-		}
-	} else {
-		// la liste
-		// $quoi=='actifs'
-		$lcpaffiche = $lcpa;
-		if (defined('_DIR_PLUGINS_SUPPL')) {
-			$lcpaffichesup = $lcpas;
-		}
-	}
-
-	if ($quoi == 'actifs' || $lpf) {
-		$nb = is_countable($lcpa) ? count($lcpa) : 0;
-		if (defined('_DIR_PLUGINS_SUPPL')) {
-			$nb += is_countable($lcpas) ? count($lcpas) : 0;
-		}
-		echo '<h3>' . sinon(
-			singulier_ou_pluriel($nb, 'plugins_actif_un', 'plugins_actifs', 'count'),
-			_T('plugins_actif_aucun')
-		) . '</h3>';
-	}
-
-	if (empty($format)) {
-		$format = 'liste';
-	} elseif (!in_array($format, ['liste', 'repertoires'])) {
-		$format = 'repertoires';
-	}
-
-	$afficher = charger_fonction("afficher_$format", 'plugins');
-	$corps = $afficher(self(), $lcpaffiche, $lcpa, $actifs);
-	if (defined('_DIR_PLUGINS_SUPPL')) {
-		$corps .= $afficher(self(), $lcpaffichesup, $lcpas, $actifs, _DIR_PLUGINS_SUPPL);
-	}
-
-	if ($corps) {
-		$corps .= "\n<div class='boutons' style='display:none;'>"
-			. "<input type='submit' class='submit save' value='" . _T('bouton_enregistrer')
-			. "' />"
-			. '</div>';
-	}
-
-	echo redirige_action_post('activer_plugins', 'activer', 'admin_plugin', '', $corps);
-
-	echo fin_cadre_trait_couleur();
-
-	if ($quoi == 'actifs') {
-		echo affiche_les_plugins_verrouilles($actifs);
-	}
-	echo '</div>';
-
-	echo http_script("
+    $lpf = null;
+    $lcpas = null;
+    $lcpaffichesup = null;
+    if (!$quoi) {
+        $quoi = 'actifs';
+    }
+    // empecher l'affichage des erreurs dans le bandeau, on le donne ensuite
+    // format brut par plugin
+    $GLOBALS['erreurs_activation_raw'] = plugin_donne_erreurs(true, false);
+    // format resume mis en forme
+    $erreur_activation = plugin_donne_erreurs();
+    $commencer_page = charger_fonction('commencer_page', 'inc');
+    echo $commencer_page(_T('icone_admin_plugin'), 'configuration', 'plugin');
+
+    echo debut_gauche();
+    echo recuperer_fond('prive/squelettes/navigation/configurer', ['exec' => 'admin_plugin']);
+
+    echo pipeline(
+        'affiche_gauche',
+        [
+            'args' => ['exec' => 'admin_plugin'],
+            'data' => afficher_librairies()
+        ]
+    );
+
+    echo debut_droite();
+    echo gros_titre(_T('icone_admin_plugin'), '');
+
+    // Barre d'onglets de premier niveau
+    echo barre_onglets('plugins', 'plugins_actifs');
+    // Barre d'onglets de second niveau
+    $onglet2 = $quoi == 'actifs' ? 'plugins_actifs' : 'admin_plugin';
+    echo debut_onglet('onglets_simple second');
+    echo onglet(_T('plugins_tous_liste'), generer_url_ecrire('admin_plugin', 'voir=tous'), 'admin_plugin', $onglet2);
+    echo onglet(_T('plugins_actifs_liste'), generer_url_ecrire('admin_plugin'), 'plugins_actifs', $onglet2);
+    echo fin_onglet();
+
+    // message d'erreur au retour d'une operation
+    if ($erreur) {
+        echo "<div class='error'>" . spip_htmlspecialchars($erreur) . '</div>';
+    }
+    if ($erreur_activation) {
+        echo "<div class='error'>$erreur_activation</div>";
+    }
+
+    // la mise a jour de cette meta a ete faite par ecrire_plugin_actifs
+    $actifs = (array) unserialize($GLOBALS['meta']['plugin']);
+    $lcpa = $actifs + (array) unserialize($GLOBALS['meta']['plugin_attente']);
+
+    // Les affichages se basent sur le repertoire, pas sur le nom
+    $actifs = liste_chemin_plugin($actifs, '');
+    if (defined('_DIR_PLUGINS_SUPPL')) {
+        $lcpas = liste_chemin_plugin($lcpa, _DIR_PLUGINS_SUPPL);
+    }
+    $lcpa = liste_chemin_plugin($lcpa);
+
+    // on installe les plugins maintenant,
+    // cela permet aux scripts d'install de faire des affichages (moches...)
+    plugin_installes_meta();
+
+    echo "<div class='liste-plugins formulaire_spip'>";
+    echo debut_cadre_trait_couleur('plugin-24.png', true, '', _T('plugins_liste'), 'plugins');
+
+    if ($quoi !== 'actifs') {
+        $lpf = liste_plugin_files();
+        if ($lpf) {
+            echo '<p>' . _T('texte_presente_plugin') . '</p>';
+        } else {
+            if (!@is_dir(_DIR_PLUGINS)) {
+                echo '<p>' . _T('plugin_info_automatique_ftp', ['rep' => joli_repertoire(_DIR_PLUGINS)])
+                    . ' &mdash; ' . _T('plugin_info_automatique_creer') . '</p>';
+            }
+        }
+        $lcpaffiche = $lpf;
+        if (defined('_DIR_PLUGINS_SUPPL')) {
+            $lcpaffichesup = liste_plugin_files(_DIR_PLUGINS_SUPPL);
+        }
+    } else {
+        // la liste
+        // $quoi=='actifs'
+        $lcpaffiche = $lcpa;
+        if (defined('_DIR_PLUGINS_SUPPL')) {
+            $lcpaffichesup = $lcpas;
+        }
+    }
+
+    if ($quoi == 'actifs' || $lpf) {
+        $nb = is_countable($lcpa) ? count($lcpa) : 0;
+        if (defined('_DIR_PLUGINS_SUPPL')) {
+            $nb += is_countable($lcpas) ? count($lcpas) : 0;
+        }
+        echo '<h3>' . sinon(
+            singulier_ou_pluriel($nb, 'plugins_actif_un', 'plugins_actifs', 'count'),
+            _T('plugins_actif_aucun')
+        ) . '</h3>';
+    }
+
+    if (empty($format)) {
+        $format = 'liste';
+    } elseif (!in_array($format, ['liste', 'repertoires'])) {
+        $format = 'repertoires';
+    }
+
+    $afficher = charger_fonction("afficher_$format", 'plugins');
+    $corps = $afficher(self(), $lcpaffiche, $lcpa, $actifs);
+    if (defined('_DIR_PLUGINS_SUPPL')) {
+        $corps .= $afficher(self(), $lcpaffichesup, $lcpas, $actifs, _DIR_PLUGINS_SUPPL);
+    }
+
+    if ($corps) {
+        $corps .= "\n<div class='boutons' style='display:none;'>"
+            . "<input type='submit' class='submit save' value='" . _T('bouton_enregistrer')
+            . "' />"
+            . '</div>';
+    }
+
+    echo redirige_action_post('activer_plugins', 'activer', 'admin_plugin', '', $corps);
+
+    echo fin_cadre_trait_couleur();
+
+    if ($quoi == 'actifs') {
+        echo affiche_les_plugins_verrouilles($actifs);
+    }
+    echo '</div>';
+
+    echo http_script("
 	jQuery(function(){
 		jQuery('.plugins li.item a[rel=info]').click(function(){
 			var li = jQuery(this).parents('li').eq(0);
@@ -231,15 +231,15 @@ 
 	});
 	");
 
-	echo pipeline(
-		'affiche_milieu',
-		[
-			'args' => ['exec' => 'admin_plugin'],
-			'data' => ''
-		]
-	);
+    echo pipeline(
+        'affiche_milieu',
+        [
+            'args' => ['exec' => 'admin_plugin'],
+            'data' => ''
+        ]
+    );
 
-	echo fin_gauche(), fin_page();
+    echo fin_gauche(), fin_page();
 }
 
 /**
@@ -253,23 +253,23 @@ 
  *     Code HTML
  **/
 function affiche_les_plugins_verrouilles($actifs) {
-	if ((!$liste = liste_plugin_files(_DIR_PLUGINS_DIST))) {
-		return '';
-	}
-
-	$afficher = charger_fonction('afficher_liste', 'plugins');
-	$liste = $afficher(self(), $liste, [], $actifs, _DIR_PLUGINS_DIST);
-
-	return
-		"<div id='plugins_dist'>"
-		. debut_cadre_trait_couleur('', true, '', _T('plugins_liste_dist'), 'liste_plugins_dist')
-		. '<p>'
-		. _T('plugin_info_plugins_dist_1', ['plugins_dist' => joli_repertoire(_DIR_PLUGINS_DIST)])
-		. '<br />' . _T('plugin_info_plugins_dist_2')
-		. '</p>'
-		. $liste
-		. fin_cadre_trait_couleur()
-		. "</div>\n";
+    if ((!$liste = liste_plugin_files(_DIR_PLUGINS_DIST))) {
+        return '';
+    }
+
+    $afficher = charger_fonction('afficher_liste', 'plugins');
+    $liste = $afficher(self(), $liste, [], $actifs, _DIR_PLUGINS_DIST);
+
+    return
+        "<div id='plugins_dist'>"
+        . debut_cadre_trait_couleur('', true, '', _T('plugins_liste_dist'), 'liste_plugins_dist')
+        . '<p>'
+        . _T('plugin_info_plugins_dist_1', ['plugins_dist' => joli_repertoire(_DIR_PLUGINS_DIST)])
+        . '<br />' . _T('plugin_info_plugins_dist_2')
+        . '</p>'
+        . $liste
+        . fin_cadre_trait_couleur()
+        . "</div>\n";
 }
 
 /**
@@ -281,18 +281,18 @@ 
  */
 function afficher_librairies() {
 
-	if (!$libs = liste_librairies()) {
-		return '';
-	}
-	ksort($libs);
-	$res = debut_cadre_enfonce('', true, '', _T('plugin_librairies_installees'));
-	$res .= '<dl>';
-	foreach ($libs as $lib => $rep) {
-		$res .= "<dt>$lib</dt><dd>" . joli_repertoire($rep) . "</dd>\n";
-	}
-	$res .= '</dl>';
-
-	return $res . fin_cadre_enfonce();
+    if (!$libs = liste_librairies()) {
+        return '';
+    }
+    ksort($libs);
+    $res = debut_cadre_enfonce('', true, '', _T('plugin_librairies_installees'));
+    $res .= '<dl>';
+    foreach ($libs as $lib => $rep) {
+        $res .= "<dt>$lib</dt><dd>" . joli_repertoire($rep) . "</dd>\n";
+    }
+    $res .= '</dl>';
+
+    return $res . fin_cadre_enfonce();
 }
 
 
@@ -303,22 +303,22 @@ 
  *     Tableau (nom de la lib => repertoire , ...)
  */
 function liste_librairies() {
-	$libs = [];
-	foreach (array_reverse(creer_chemin()) as $d) {
-		if (
-			is_dir($dir = $d . 'lib/')
-			&& ($t = opendir($dir))
-		) {
-			while (($f = readdir($t)) !== false) {
-				if (
-					$f[0] != '.'
-					&& is_dir("$dir/$f")
-				) {
-					$libs[$f] = $dir;
-				}
-			}
-		}
-	}
-
-	return $libs;
+    $libs = [];
+    foreach (array_reverse(creer_chemin()) as $d) {
+        if (
+            is_dir($dir = $d . 'lib/')
+            && ($t = opendir($dir))
+        ) {
+            while (($f = readdir($t)) !== false) {
+                if (
+                    $f[0] != '.'
+                    && is_dir("$dir/$f")
+                ) {
+                    $libs[$f] = $dir;
+                }
+            }
+        }
+    }
+
+    return $libs;
 }

Spacing +11 added lines, -11 removed lines

@@ -119,7 +119,7 @@ 
 
 	// message d'erreur au retour d'une operation
 	if ($erreur) {
-		echo "<div class='error'>" . spip_htmlspecialchars($erreur) . '</div>';
+		echo "<div class='error'>".spip_htmlspecialchars($erreur).'</div>';
 	}
 	if ($erreur_activation) {
 		echo "<div class='error'>$erreur_activation</div>";
@@ -146,11 +146,11 @@ 
 	if ($quoi !== 'actifs') {
 		$lpf = liste_plugin_files();
 		if ($lpf) {
-			echo '<p>' . _T('texte_presente_plugin') . '</p>';
+			echo '<p>'._T('texte_presente_plugin').'</p>';
 		} else {
 			if (!@is_dir(_DIR_PLUGINS)) {
-				echo '<p>' . _T('plugin_info_automatique_ftp', ['rep' => joli_repertoire(_DIR_PLUGINS)])
-					. ' &mdash; ' . _T('plugin_info_automatique_creer') . '</p>';
+				echo '<p>'._T('plugin_info_automatique_ftp', ['rep' => joli_repertoire(_DIR_PLUGINS)])
+					. ' &mdash; '._T('plugin_info_automatique_creer').'</p>';
 			}
 		}
 		$lcpaffiche = $lpf;
@@ -171,10 +171,10 @@ 
 		if (defined('_DIR_PLUGINS_SUPPL')) {
 			$nb += is_countable($lcpas) ? count($lcpas) : 0;
 		}
-		echo '<h3>' . sinon(
+		echo '<h3>'.sinon(
 			singulier_ou_pluriel($nb, 'plugins_actif_un', 'plugins_actifs', 'count'),
 			_T('plugins_actif_aucun')
-		) . '</h3>';
+		).'</h3>';
 	}
 
 	if (empty($format)) {
@@ -191,7 +191,7 @@ 
 
 	if ($corps) {
 		$corps .= "\n<div class='boutons' style='display:none;'>"
-			. "<input type='submit' class='submit save' value='" . _T('bouton_enregistrer')
+			. "<input type='submit' class='submit save' value='"._T('bouton_enregistrer')
 			. "' />"
 			. '</div>';
 	}
@@ -265,7 +265,7 @@ 
 		. debut_cadre_trait_couleur('', true, '', _T('plugins_liste_dist'), 'liste_plugins_dist')
 		. '<p>'
 		. _T('plugin_info_plugins_dist_1', ['plugins_dist' => joli_repertoire(_DIR_PLUGINS_DIST)])
-		. '<br />' . _T('plugin_info_plugins_dist_2')
+		. '<br />'._T('plugin_info_plugins_dist_2')
 		. '</p>'
 		. $liste
 		. fin_cadre_trait_couleur()
@@ -288,11 +288,11 @@ 
 	$res = debut_cadre_enfonce('', true, '', _T('plugin_librairies_installees'));
 	$res .= '<dl>';
 	foreach ($libs as $lib => $rep) {
-		$res .= "<dt>$lib</dt><dd>" . joli_repertoire($rep) . "</dd>\n";
+		$res .= "<dt>$lib</dt><dd>".joli_repertoire($rep)."</dd>\n";
 	}
 	$res .= '</dl>';
 
-	return $res . fin_cadre_enfonce();
+	return $res.fin_cadre_enfonce();
 }
 
 
@@ -306,7 +306,7 @@ 
 	$libs = [];
 	foreach (array_reverse(creer_chemin()) as $d) {
 		if (
-			is_dir($dir = $d . 'lib/')
+			is_dir($dir = $d.'lib/')
 			&& ($t = opendir($dir))
 		) {
 			while (($f = readdir($t)) !== false) {

ecrire/action/instituer_langue_objet.php

Indentation +37 added lines, -37 removed lines

@@ -16,7 +16,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -30,43 +30,43 @@ 
  * @return string
  */
 function action_instituer_langue_objet_dist($objet, $id, $id_rubrique, $changer_lang, $serveur = '') {
-	if ($changer_lang) {
-		$table_objet_sql = table_objet_sql($objet);
-		$id_table_objet = id_table_objet($objet);
-		$trouver_table = charger_fonction('trouver_table', 'base');
-		$desc = $trouver_table($table_objet_sql, $serveur);
+    if ($changer_lang) {
+        $table_objet_sql = table_objet_sql($objet);
+        $id_table_objet = id_table_objet($objet);
+        $trouver_table = charger_fonction('trouver_table', 'base');
+        $desc = $trouver_table($table_objet_sql, $serveur);
 
-		$set = [];
-		if (isset($desc['field']['langue_choisie'])) {
-			$set['langue_choisie'] = 'oui';
-		}
+        $set = [];
+        if (isset($desc['field']['langue_choisie'])) {
+            $set['langue_choisie'] = 'oui';
+        }
 
-		if ($changer_lang != 'herit') {
-			$set['lang'] = $changer_lang;
-			sql_updateq($table_objet_sql, $set, "$id_table_objet=" . (int) $id, [], $serveur);
-			include_spip('inc/rubriques'); // pour calculer_langues_rubriques et calculer_langues_utilisees
-			if ($table_objet_sql == 'spip_rubriques') {
-				calculer_langues_rubriques();
-			}
-			$langues = calculer_langues_utilisees($serveur);
-			ecrire_meta('langues_utilisees', $langues);
-		} else {
-			$langue_parent = sql_getfetsel('lang', 'spip_rubriques', 'id_rubrique=' . (int) $id_rubrique);
-			if (!$langue_parent) {
-				$langue_parent = $GLOBALS['meta']['langue_site'];
-			}
-			$changer_lang = $langue_parent;
-			$set['lang'] = $changer_lang;
-			if (isset($set['langue_choisie'])) {
-				$set['langue_choisie'] = 'non';
-			}
-			sql_updateq($table_objet_sql, $set, "$id_table_objet=" . (int) $id, [], $serveur);
-			if ($table_objet_sql == 'spip_rubriques') {
-				include_spip('inc/rubriques');
-				calculer_langues_rubriques();
-			}
-		}
-	}
+        if ($changer_lang != 'herit') {
+            $set['lang'] = $changer_lang;
+            sql_updateq($table_objet_sql, $set, "$id_table_objet=" . (int) $id, [], $serveur);
+            include_spip('inc/rubriques'); // pour calculer_langues_rubriques et calculer_langues_utilisees
+            if ($table_objet_sql == 'spip_rubriques') {
+                calculer_langues_rubriques();
+            }
+            $langues = calculer_langues_utilisees($serveur);
+            ecrire_meta('langues_utilisees', $langues);
+        } else {
+            $langue_parent = sql_getfetsel('lang', 'spip_rubriques', 'id_rubrique=' . (int) $id_rubrique);
+            if (!$langue_parent) {
+                $langue_parent = $GLOBALS['meta']['langue_site'];
+            }
+            $changer_lang = $langue_parent;
+            $set['lang'] = $changer_lang;
+            if (isset($set['langue_choisie'])) {
+                $set['langue_choisie'] = 'non';
+            }
+            sql_updateq($table_objet_sql, $set, "$id_table_objet=" . (int) $id, [], $serveur);
+            if ($table_objet_sql == 'spip_rubriques') {
+                include_spip('inc/rubriques');
+                calculer_langues_rubriques();
+            }
+        }
+    }
 
-	return $changer_lang;
+    return $changer_lang;
 }

Spacing +3 added lines, -3 removed lines

@@ -43,7 +43,7 @@ 
 
 		if ($changer_lang != 'herit') {
 			$set['lang'] = $changer_lang;
-			sql_updateq($table_objet_sql, $set, "$id_table_objet=" . (int) $id, [], $serveur);
+			sql_updateq($table_objet_sql, $set, "$id_table_objet=".(int) $id, [], $serveur);
 			include_spip('inc/rubriques'); // pour calculer_langues_rubriques et calculer_langues_utilisees
 			if ($table_objet_sql == 'spip_rubriques') {
 				calculer_langues_rubriques();
@@ -51,7 +51,7 @@ 
 			$langues = calculer_langues_utilisees($serveur);
 			ecrire_meta('langues_utilisees', $langues);
 		} else {
-			$langue_parent = sql_getfetsel('lang', 'spip_rubriques', 'id_rubrique=' . (int) $id_rubrique);
+			$langue_parent = sql_getfetsel('lang', 'spip_rubriques', 'id_rubrique='.(int) $id_rubrique);
 			if (!$langue_parent) {
 				$langue_parent = $GLOBALS['meta']['langue_site'];
 			}
@@ -60,7 +60,7 @@ 
 			if (isset($set['langue_choisie'])) {
 				$set['langue_choisie'] = 'non';
 			}
-			sql_updateq($table_objet_sql, $set, "$id_table_objet=" . (int) $id, [], $serveur);
+			sql_updateq($table_objet_sql, $set, "$id_table_objet=".(int) $id, [], $serveur);
 			if ($table_objet_sql == 'spip_rubriques') {
 				include_spip('inc/rubriques');
 				calculer_langues_rubriques();

ecrire/action/tester.php

Indentation +86 added lines, -86 removed lines

@@ -16,7 +16,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -29,100 +29,100 @@ 
  * redirige sur l'image ainsi créée (sinon sur une image d'echec).
  **/
 function action_tester_dist() {
-	$arg = _request('arg');
+    $arg = _request('arg');
 
-	// verifier les formats acceptes par GD
-	if ($arg === 'gd2') {
-		$gd_formats = [];
+    // verifier les formats acceptes par GD
+    if ($arg === 'gd2') {
+        $gd_formats = [];
 
-		if (function_exists('imagetypes')) {
-			if (imagetypes() & IMG_GIF) {
-				$gd_formats[] = 'gif';
-			}
-			if (imagetypes() & IMG_JPG) {
-				$gd_formats[] = 'jpg';
-			}
-			if (imagetypes() & IMG_PNG) {
-				$gd_formats[] = 'png';
-			}
-			if (imagetypes() & IMG_WEBP) {
-				$gd_formats[] = 'webp';
-			}
-		}
+        if (function_exists('imagetypes')) {
+            if (imagetypes() & IMG_GIF) {
+                $gd_formats[] = 'gif';
+            }
+            if (imagetypes() & IMG_JPG) {
+                $gd_formats[] = 'jpg';
+            }
+            if (imagetypes() & IMG_PNG) {
+                $gd_formats[] = 'png';
+            }
+            if (imagetypes() & IMG_WEBP) {
+                $gd_formats[] = 'webp';
+            }
+        }
 
-		$gd_formats = implode(',', $gd_formats);
-		ecrire_meta('gd_formats_read', $gd_formats);
-		ecrire_meta('gd_formats', $gd_formats);
-	} elseif ($arg == 'netpbm') {
-		// verifier les formats netpbm
-		if (!defined('_PNMSCALE_COMMAND')) {
-			define('_PNMSCALE_COMMAND', 'pnmscale');
-		} // Securite : mes_options.php peut preciser le chemin absolu
-		if (_PNMSCALE_COMMAND == '') {
-			return;
-		}
-		$netpbm_formats = [];
+        $gd_formats = implode(',', $gd_formats);
+        ecrire_meta('gd_formats_read', $gd_formats);
+        ecrire_meta('gd_formats', $gd_formats);
+    } elseif ($arg == 'netpbm') {
+        // verifier les formats netpbm
+        if (!defined('_PNMSCALE_COMMAND')) {
+            define('_PNMSCALE_COMMAND', 'pnmscale');
+        } // Securite : mes_options.php peut preciser le chemin absolu
+        if (_PNMSCALE_COMMAND == '') {
+            return;
+        }
+        $netpbm_formats = [];
 
-		$jpegtopnm_command = str_replace(
-			'pnmscale',
-			'jpegtopnm',
-			(string) _PNMSCALE_COMMAND
-		);
-		$pnmtojpeg_command = str_replace(
-			'pnmscale',
-			'pnmtojpeg',
-			(string) _PNMSCALE_COMMAND
-		);
+        $jpegtopnm_command = str_replace(
+            'pnmscale',
+            'jpegtopnm',
+            (string) _PNMSCALE_COMMAND
+        );
+        $pnmtojpeg_command = str_replace(
+            'pnmscale',
+            'pnmtojpeg',
+            (string) _PNMSCALE_COMMAND
+        );
 
-		$vignette = _ROOT_IMG_PACK . 'test.jpg';
-		$dest = _DIR_VAR . 'test-jpg.jpg';
-		$commande = "$jpegtopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
-		spip_log($commande);
-		exec($commande);
-		if (($taille = @getimagesize($dest)) && $taille[1] == 10) {
-			$netpbm_formats[] = 'jpg';
-		}
-		$giftopnm_command = str_replace('pnmscale', 'giftopnm', (string) _PNMSCALE_COMMAND);
-		$pnmtojpeg_command = str_replace('pnmscale', 'pnmtojpeg', (string) _PNMSCALE_COMMAND);
-		$vignette = _ROOT_IMG_PACK . 'test.gif';
-		$dest = _DIR_VAR . 'test-gif.jpg';
-		$commande = "$giftopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
-		spip_log($commande);
-		exec($commande);
-		if (($taille = @getimagesize($dest)) && $taille[1] == 10) {
-			$netpbm_formats[] = 'gif';
-		}
+        $vignette = _ROOT_IMG_PACK . 'test.jpg';
+        $dest = _DIR_VAR . 'test-jpg.jpg';
+        $commande = "$jpegtopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
+        spip_log($commande);
+        exec($commande);
+        if (($taille = @getimagesize($dest)) && $taille[1] == 10) {
+            $netpbm_formats[] = 'jpg';
+        }
+        $giftopnm_command = str_replace('pnmscale', 'giftopnm', (string) _PNMSCALE_COMMAND);
+        $pnmtojpeg_command = str_replace('pnmscale', 'pnmtojpeg', (string) _PNMSCALE_COMMAND);
+        $vignette = _ROOT_IMG_PACK . 'test.gif';
+        $dest = _DIR_VAR . 'test-gif.jpg';
+        $commande = "$giftopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
+        spip_log($commande);
+        exec($commande);
+        if (($taille = @getimagesize($dest)) && $taille[1] == 10) {
+            $netpbm_formats[] = 'gif';
+        }
 
-		$pngtopnm_command = str_replace('pnmscale', 'pngtopnm', (string) _PNMSCALE_COMMAND);
-		$vignette = _ROOT_IMG_PACK . 'test.png';
-		$dest = _DIR_VAR . 'test-gif.jpg';
-		$commande = "$pngtopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
-		spip_log($commande);
-		exec($commande);
-		if (($taille = @getimagesize($dest)) && $taille[1] == 10) {
-			$netpbm_formats[] = 'png';
-		}
+        $pngtopnm_command = str_replace('pnmscale', 'pngtopnm', (string) _PNMSCALE_COMMAND);
+        $vignette = _ROOT_IMG_PACK . 'test.png';
+        $dest = _DIR_VAR . 'test-gif.jpg';
+        $commande = "$pngtopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
+        spip_log($commande);
+        exec($commande);
+        if (($taille = @getimagesize($dest)) && $taille[1] == 10) {
+            $netpbm_formats[] = 'png';
+        }
 
-		ecrire_meta('netpbm_formats', implode(',', $netpbm_formats ?: []));
-	}
+        ecrire_meta('netpbm_formats', implode(',', $netpbm_formats ?: []));
+    }
 
-	// et maintenant envoyer la vignette de tests
-	if (in_array($arg, ['gd2', 'imagick', 'convert', 'netpbm'])) {
-		include_spip('inc/filtres');
-		include_spip('inc/filtres_images_mini');
-		$taille_preview = 150;
-		$image = _image_valeurs_trans(_DIR_IMG_PACK . 'test_image.jpg', "reduire-$taille_preview-$taille_preview", 'jpg');
+    // et maintenant envoyer la vignette de tests
+    if (in_array($arg, ['gd2', 'imagick', 'convert', 'netpbm'])) {
+        include_spip('inc/filtres');
+        include_spip('inc/filtres_images_mini');
+        $taille_preview = 150;
+        $image = _image_valeurs_trans(_DIR_IMG_PACK . 'test_image.jpg', "reduire-$taille_preview-$taille_preview", 'jpg');
 
-		$image['fichier_dest'] = _DIR_VAR . "test_$arg";
+        $image['fichier_dest'] = _DIR_VAR . "test_$arg";
 
-		if (
-			($preview = _image_creer_vignette($image, $taille_preview, $taille_preview, $arg, true))
-			&& $preview['width'] * $preview['height'] > 0
-		) {
-			redirige_par_entete($preview['fichier']);
-		}
-	}
+        if (
+            ($preview = _image_creer_vignette($image, $taille_preview, $taille_preview, $arg, true))
+            && $preview['width'] * $preview['height'] > 0
+        ) {
+            redirige_par_entete($preview['fichier']);
+        }
+    }
 
-	# image echec
-	redirige_par_entete(chemin_image('echec-reducteur-xx.svg'));
+    # image echec
+    redirige_par_entete(chemin_image('echec-reducteur-xx.svg'));
 }

Spacing +11 added lines, -11 removed lines

@@ -74,9 +74,9 @@ 
 			(string) _PNMSCALE_COMMAND
 		);
 
-		$vignette = _ROOT_IMG_PACK . 'test.jpg';
-		$dest = _DIR_VAR . 'test-jpg.jpg';
-		$commande = "$jpegtopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
+		$vignette = _ROOT_IMG_PACK.'test.jpg';
+		$dest = _DIR_VAR.'test-jpg.jpg';
+		$commande = "$jpegtopnm_command $vignette | "._PNMSCALE_COMMAND." -width 10 | $pnmtojpeg_command > $dest";
 		spip_log($commande);
 		exec($commande);
 		if (($taille = @getimagesize($dest)) && $taille[1] == 10) {
@@ -84,9 +84,9 @@ 
 		}
 		$giftopnm_command = str_replace('pnmscale', 'giftopnm', (string) _PNMSCALE_COMMAND);
 		$pnmtojpeg_command = str_replace('pnmscale', 'pnmtojpeg', (string) _PNMSCALE_COMMAND);
-		$vignette = _ROOT_IMG_PACK . 'test.gif';
-		$dest = _DIR_VAR . 'test-gif.jpg';
-		$commande = "$giftopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
+		$vignette = _ROOT_IMG_PACK.'test.gif';
+		$dest = _DIR_VAR.'test-gif.jpg';
+		$commande = "$giftopnm_command $vignette | "._PNMSCALE_COMMAND." -width 10 | $pnmtojpeg_command > $dest";
 		spip_log($commande);
 		exec($commande);
 		if (($taille = @getimagesize($dest)) && $taille[1] == 10) {
@@ -94,9 +94,9 @@ 
 		}
 
 		$pngtopnm_command = str_replace('pnmscale', 'pngtopnm', (string) _PNMSCALE_COMMAND);
-		$vignette = _ROOT_IMG_PACK . 'test.png';
-		$dest = _DIR_VAR . 'test-gif.jpg';
-		$commande = "$pngtopnm_command $vignette | " . _PNMSCALE_COMMAND . " -width 10 | $pnmtojpeg_command > $dest";
+		$vignette = _ROOT_IMG_PACK.'test.png';
+		$dest = _DIR_VAR.'test-gif.jpg';
+		$commande = "$pngtopnm_command $vignette | "._PNMSCALE_COMMAND." -width 10 | $pnmtojpeg_command > $dest";
 		spip_log($commande);
 		exec($commande);
 		if (($taille = @getimagesize($dest)) && $taille[1] == 10) {
@@ -111,9 +111,9 @@ 
 		include_spip('inc/filtres');
 		include_spip('inc/filtres_images_mini');
 		$taille_preview = 150;
-		$image = _image_valeurs_trans(_DIR_IMG_PACK . 'test_image.jpg', "reduire-$taille_preview-$taille_preview", 'jpg');
+		$image = _image_valeurs_trans(_DIR_IMG_PACK.'test_image.jpg', "reduire-$taille_preview-$taille_preview", 'jpg');
 
-		$image['fichier_dest'] = _DIR_VAR . "test_$arg";
+		$image['fichier_dest'] = _DIR_VAR."test_$arg";
 
 		if (
 			($preview = _image_creer_vignette($image, $taille_preview, $taille_preview, $arg, true))

ecrire/action/tester_taille.php

Indentation +79 added lines, -79 removed lines

@@ -17,7 +17,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 include_spip('inc/headers');
 
@@ -34,12 +34,12 @@ 
  *     Sortie du buffer
  **/
 function action_tester_taille_error_handler($output) {
-	// on est ici, donc echec lors de la creation de l'image
-	if (!empty($GLOBALS['redirect'])) {
-		return redirige_formulaire($GLOBALS['redirect']);
-	}
+    // on est ici, donc echec lors de la creation de l'image
+    if (!empty($GLOBALS['redirect'])) {
+        return redirige_formulaire($GLOBALS['redirect']);
+    }
 
-	return $output;
+    return $output;
 }
 
 
@@ -57,77 +57,77 @@ 
  **/
 function action_tester_taille_dist() {
 
-	if (!autoriser('configurer')) {
-		return;
-	}
-
-	$taille = _request('arg');
-	$taille = explode('-', (string) $taille);
-
-	$GLOBALS['taille_max'] = end($taille);
-	$GLOBALS['taille_min'] = 0;
-	if (count($taille) > 1) {
-		$GLOBALS['taille_min'] = reset($taille);
-	}
-
-	// si l'intervalle est assez petit, on garde la valeur min
-	if ($GLOBALS['taille_max'] * $GLOBALS['taille_max'] - $GLOBALS['taille_min'] * $GLOBALS['taille_min'] < 50000) {
-		$t = ($GLOBALS['taille_min'] * $GLOBALS['taille_min']);
-		if ($GLOBALS['taille_min'] !== $GLOBALS['taille_max']) {
-			$t *= 0.9; // marge de securite
-			echo round($t / 1_000_000, 3) . ' Mpx';
-		} else {
-			// c'est un cas "on a reussi la borne max initiale, donc on a pas de limite connue"
-			$t = 0;
-			echo '&infin;';
-		}
-		ecrire_meta('max_taille_vignettes', $t, 'non');
-		die();
-	}
-
-	$taille = $GLOBALS['taille_test'] = round(($GLOBALS['taille_max'] + $GLOBALS['taille_min']) / 2);
-
-	include_spip('inc/filtres');
-	// des inclusions representatives d'un hit prive et/ou public pour la conso memoire
-	include_spip('public/assembler');
-	include_spip('public/balises');
-	include_spip('public/boucles');
-	include_spip('public/cacher');
-	include_spip('public/compiler');
-	include_spip('public/composer');
-	include_spip('public/criteres');
-	include_spip('public/interfaces');
-	include_spip('public/parametrer');
-	include_spip('public/phraser_html');
-	include_spip('public/references');
-
-	include_spip('inc/presentation');
-	include_spip('inc/charsets');
-	include_spip('inc/documents');
-	include_spip('inc/header');
-	propre('<doc1>'); // charger propre avec le trairement d'un modele
-
-	$i = _request('i') + 1;
-	$image_source = chemin_image('test.png');
-	$GLOBALS['redirect'] = generer_url_action(
-		'tester_taille',
-		"i=$i&arg=" . $GLOBALS['taille_min'] . '-' . $GLOBALS['taille_test']
-	);
-
-	ob_start('action_tester_taille_error_handler');
-	filtrer('image_recadre', $image_source, $taille, $taille);
-	$GLOBALS['redirect'] = generer_url_action('tester_taille', "i=$i&arg=$taille-" . $GLOBALS['taille_max']);
-
-	// si la valeur intermediaire a reussi, on teste la valeur maxi qui est peut etre sous estimee
-	// si $GLOBALS['taille_min']==0 (car on est au premier coup)
-	if ($GLOBALS['taille_min'] == 0) {
-		$taille = $GLOBALS['taille_max'];
-		filtrer('image_recadre', $image_source, $taille, $taille);
-		$GLOBALS['redirect'] = generer_url_action('tester_taille', "i=$i&arg=$taille-" . $GLOBALS['taille_max']);
-	}
-	ob_end_clean();
-
-
-	// on est ici, donc pas de plantage
-	echo redirige_formulaire($GLOBALS['redirect']);
+    if (!autoriser('configurer')) {
+        return;
+    }
+
+    $taille = _request('arg');
+    $taille = explode('-', (string) $taille);
+
+    $GLOBALS['taille_max'] = end($taille);
+    $GLOBALS['taille_min'] = 0;
+    if (count($taille) > 1) {
+        $GLOBALS['taille_min'] = reset($taille);
+    }
+
+    // si l'intervalle est assez petit, on garde la valeur min
+    if ($GLOBALS['taille_max'] * $GLOBALS['taille_max'] - $GLOBALS['taille_min'] * $GLOBALS['taille_min'] < 50000) {
+        $t = ($GLOBALS['taille_min'] * $GLOBALS['taille_min']);
+        if ($GLOBALS['taille_min'] !== $GLOBALS['taille_max']) {
+            $t *= 0.9; // marge de securite
+            echo round($t / 1_000_000, 3) . ' Mpx';
+        } else {
+            // c'est un cas "on a reussi la borne max initiale, donc on a pas de limite connue"
+            $t = 0;
+            echo '&infin;';
+        }
+        ecrire_meta('max_taille_vignettes', $t, 'non');
+        die();
+    }
+
+    $taille = $GLOBALS['taille_test'] = round(($GLOBALS['taille_max'] + $GLOBALS['taille_min']) / 2);
+
+    include_spip('inc/filtres');
+    // des inclusions representatives d'un hit prive et/ou public pour la conso memoire
+    include_spip('public/assembler');
+    include_spip('public/balises');
+    include_spip('public/boucles');
+    include_spip('public/cacher');
+    include_spip('public/compiler');
+    include_spip('public/composer');
+    include_spip('public/criteres');
+    include_spip('public/interfaces');
+    include_spip('public/parametrer');
+    include_spip('public/phraser_html');
+    include_spip('public/references');
+
+    include_spip('inc/presentation');
+    include_spip('inc/charsets');
+    include_spip('inc/documents');
+    include_spip('inc/header');
+    propre('<doc1>'); // charger propre avec le trairement d'un modele
+
+    $i = _request('i') + 1;
+    $image_source = chemin_image('test.png');
+    $GLOBALS['redirect'] = generer_url_action(
+        'tester_taille',
+        "i=$i&arg=" . $GLOBALS['taille_min'] . '-' . $GLOBALS['taille_test']
+    );
+
+    ob_start('action_tester_taille_error_handler');
+    filtrer('image_recadre', $image_source, $taille, $taille);
+    $GLOBALS['redirect'] = generer_url_action('tester_taille', "i=$i&arg=$taille-" . $GLOBALS['taille_max']);
+
+    // si la valeur intermediaire a reussi, on teste la valeur maxi qui est peut etre sous estimee
+    // si $GLOBALS['taille_min']==0 (car on est au premier coup)
+    if ($GLOBALS['taille_min'] == 0) {
+        $taille = $GLOBALS['taille_max'];
+        filtrer('image_recadre', $image_source, $taille, $taille);
+        $GLOBALS['redirect'] = generer_url_action('tester_taille', "i=$i&arg=$taille-" . $GLOBALS['taille_max']);
+    }
+    ob_end_clean();
+
+
+    // on est ici, donc pas de plantage
+    echo redirige_formulaire($GLOBALS['redirect']);
 }

ecrire/action/editer_liens.php

Indentation +558 added lines, -558 removed lines

@@ -28,7 +28,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 // charger la gestion les rôles sur les objets
@@ -46,22 +46,22 @@ 
  *     - array(clé primaire, nom de la table de lien) si associable
  */
 function objet_associable($objet) {
-	$trouver_table = charger_fonction('trouver_table', 'base');
-	$table_sql = table_objet_sql($objet);
-
-	$l = '';
-	if (
-		($primary = id_table_objet($objet))
-		&& $trouver_table($l = $table_sql . '_liens', '', true, ['log_missing' => false])
-		&& !preg_match(',[^\w],', $primary)
-		&& !preg_match(',[^\w],', $l)
-	) {
-		return [$primary, $l];
-	}
-
-	spip_log("Objet $objet non associable : ne dispose pas d'une cle primaire $primary OU d'une table liens $l");
-
-	return false;
+    $trouver_table = charger_fonction('trouver_table', 'base');
+    $table_sql = table_objet_sql($objet);
+
+    $l = '';
+    if (
+        ($primary = id_table_objet($objet))
+        && $trouver_table($l = $table_sql . '_liens', '', true, ['log_missing' => false])
+        && !preg_match(',[^\w],', $primary)
+        && !preg_match(',[^\w],', $l)
+    ) {
+        return [$primary, $l];
+    }
+
+    spip_log("Objet $objet non associable : ne dispose pas d'une cle primaire $primary OU d'une table liens $l");
+
+    return false;
 }
 
 /**
@@ -87,13 +87,13 @@ 
  * @return bool|int
  */
 function objet_associer($objets_source, $objets_lies, $qualif = null) {
-	$modifs = objet_traiter_liaisons('lien_insert', $objets_source, $objets_lies, $qualif);
+    $modifs = objet_traiter_liaisons('lien_insert', $objets_source, $objets_lies, $qualif);
 
-	if ($qualif) {
-		objet_qualifier_liens($objets_source, $objets_lies, $qualif);
-	}
+    if ($qualif) {
+        objet_qualifier_liens($objets_source, $objets_lies, $qualif);
+    }
 
-	return $modifs; // pas d'erreur
+    return $modifs; // pas d'erreur
 }
 
 
@@ -127,7 +127,7 @@ 
  * @return bool|int
  */
 function objet_dissocier($objets_source, $objets_lies, $cond = null) {
-	return objet_traiter_liaisons('lien_delete', $objets_source, $objets_lies, $cond);
+    return objet_traiter_liaisons('lien_delete', $objets_source, $objets_lies, $cond);
 }
 
 
@@ -152,7 +152,7 @@ 
  * @return bool|int
  */
 function objet_qualifier_liens($objets_source, $objets_lies, $qualif) {
-	return objet_traiter_liaisons('lien_set', $objets_source, $objets_lies, $qualif);
+    return objet_traiter_liaisons('lien_set', $objets_source, $objets_lies, $qualif);
 }
 
 
@@ -187,7 +187,7 @@ 
  *     Liste des trouvailles
  */
 function objet_trouver_liens($objets_source, $objets_lies, $cond = null) {
-	return objet_traiter_liaisons('lien_find', $objets_source, $objets_lies, $cond);
+    return objet_traiter_liaisons('lien_find', $objets_source, $objets_lies, $cond);
 }
 
 
@@ -211,8 +211,8 @@ 
  * @return int
  */
 function objet_optimiser_liens($objets_source, $objets_lies) {
-	spip_log('objet_optimiser_liens : ' . json_encode($objets_source, JSON_THROW_ON_ERROR) . ', ' . json_encode($objets_lies, JSON_THROW_ON_ERROR), 'genie' . _LOG_DEBUG);
-	return objet_traiter_liaisons('lien_optimise', $objets_source, $objets_lies);
+    spip_log('objet_optimiser_liens : ' . json_encode($objets_source, JSON_THROW_ON_ERROR) . ', ' . json_encode($objets_lies, JSON_THROW_ON_ERROR), 'genie' . _LOG_DEBUG);
+    return objet_traiter_liaisons('lien_optimise', $objets_source, $objets_lies);
 }
 
 
@@ -232,30 +232,30 @@ 
  *     Nombre de liens copiés
  */
 function objet_dupliquer_liens($objet, $id_source, $id_cible, $types = null, $exclure_types = null) {
-	include_spip('base/objets');
-	$tables = lister_tables_objets_sql();
-	$n = 0;
-	foreach ($tables as $table_sql => $infos) {
-		if ((is_null($types) || in_array($infos['type'], $types)) && (is_null($exclure_types) || !in_array($infos['type'], $exclure_types)) && objet_associable($infos['type'])) {
-			$liens = (($infos['type'] == $objet)
-				? objet_trouver_liens([$objet => $id_source], '*')
-				: objet_trouver_liens([$infos['type'] => '*'], [$objet => $id_source]));
-			foreach ($liens as $lien) {
-				$n++;
-				if ($infos['type'] == $objet) {
-					if (
-						(is_null($types) || in_array($lien['objet'], $types)) && (is_null($exclure_types) || !in_array($lien['objet'], $exclure_types))
-					) {
-						objet_associer([$objet => $id_cible], [$lien['objet'] => $lien[$lien['objet']]], $lien);
-					}
-				} else {
-					objet_associer([$infos['type'] => $lien[$infos['type']]], [$objet => $id_cible], $lien);
-				}
-			}
-		}
-	}
-
-	return $n;
+    include_spip('base/objets');
+    $tables = lister_tables_objets_sql();
+    $n = 0;
+    foreach ($tables as $table_sql => $infos) {
+        if ((is_null($types) || in_array($infos['type'], $types)) && (is_null($exclure_types) || !in_array($infos['type'], $exclure_types)) && objet_associable($infos['type'])) {
+            $liens = (($infos['type'] == $objet)
+                ? objet_trouver_liens([$objet => $id_source], '*')
+                : objet_trouver_liens([$infos['type'] => '*'], [$objet => $id_source]));
+            foreach ($liens as $lien) {
+                $n++;
+                if ($infos['type'] == $objet) {
+                    if (
+                        (is_null($types) || in_array($lien['objet'], $types)) && (is_null($exclure_types) || !in_array($lien['objet'], $exclure_types))
+                    ) {
+                        objet_associer([$objet => $id_cible], [$lien['objet'] => $lien[$lien['objet']]], $lien);
+                    }
+                } else {
+                    objet_associer([$infos['type'] => $lien[$infos['type']]], [$objet => $id_cible], $lien);
+                }
+            }
+        }
+    }
+
+    return $n;
 }
 
 /**
@@ -297,38 +297,38 @@ 
  * @return bool|int|array
  */
 function objet_traiter_liaisons($operation, $objets_source, $objets_lies, $set = null) {
-	// accepter une syntaxe minimale pour supprimer tous les liens
-	if ($objets_lies == '*') {
-		$objets_lies = ['*' => '*'];
-	}
-	$modifs = 0; // compter le nombre de modifications
-	$echec = null;
-	foreach ($objets_source as $objet => $ids) {
-		if ($a = objet_associable($objet)) {
-			[$primary, $l] = $a;
-			if (!is_array($ids)) {
-				$ids = [$ids];
-			} elseif (reset($ids) == 'NOT') {
-				// si on demande un array('NOT',...) => recuperer la liste d'ids correspondants
-				$where = lien_where($primary, $ids, '*', '*');
-				$ids = sql_allfetsel($primary, $l, $where);
-				$ids = array_map('reset', $ids);
-			}
-			foreach ($ids as $id) {
-				$res = $operation($objet, $primary, $l, $id, $objets_lies, $set);
-				if ($res === false) {
-					spip_log("objet_traiter_liaisons [Echec] : $operation sur $objet/$primary/$l/$id", _LOG_ERREUR);
-					$echec = true;
-				} else {
-					$modifs = ($modifs ? (is_array($res) ? array_merge($modifs, $res) : $modifs + $res) : $res);
-				}
-			}
-		} else {
-			$echec = true;
-		}
-	}
-
-	return ($echec ? false : $modifs); // pas d'erreur
+    // accepter une syntaxe minimale pour supprimer tous les liens
+    if ($objets_lies == '*') {
+        $objets_lies = ['*' => '*'];
+    }
+    $modifs = 0; // compter le nombre de modifications
+    $echec = null;
+    foreach ($objets_source as $objet => $ids) {
+        if ($a = objet_associable($objet)) {
+            [$primary, $l] = $a;
+            if (!is_array($ids)) {
+                $ids = [$ids];
+            } elseif (reset($ids) == 'NOT') {
+                // si on demande un array('NOT',...) => recuperer la liste d'ids correspondants
+                $where = lien_where($primary, $ids, '*', '*');
+                $ids = sql_allfetsel($primary, $l, $where);
+                $ids = array_map('reset', $ids);
+            }
+            foreach ($ids as $id) {
+                $res = $operation($objet, $primary, $l, $id, $objets_lies, $set);
+                if ($res === false) {
+                    spip_log("objet_traiter_liaisons [Echec] : $operation sur $objet/$primary/$l/$id", _LOG_ERREUR);
+                    $echec = true;
+                } else {
+                    $modifs = ($modifs ? (is_array($res) ? array_merge($modifs, $res) : $modifs + $res) : $res);
+                }
+            }
+        } else {
+            $echec = true;
+        }
+    }
+
+    return ($echec ? false : $modifs); // pas d'erreur
 }
 
 
@@ -357,116 +357,116 @@ 
  *     Nombre d'insertions faites, false si échec.
  */
 function lien_insert($objet_source, $primary, $table_lien, $id, $objets, $qualif) {
-	$ins = 0;
-	$echec = null;
-	if (is_null($qualif)) {
-		$qualif = [];
-	}
-
-	foreach ($objets as $objet => $id_objets) {
-		if (!is_array($id_objets)) {
-			$id_objets = [$id_objets];
-		}
-
-		// role, colonne, where par défaut
-		[$role, $colonne_role, $cond] =
-			roles_trouver_dans_qualif($objet_source, $objet, $qualif);
-
-		foreach ($id_objets as $id_objet) {
-			$objet = (($objet == '*') ? $objet : objet_type($objet)); # securite
-
-			$insertions = [
-				'id_objet' => $id_objet,
-				'objet' => $objet,
-				$primary => $id
-			];
-			// rôle en plus s'il est défini
-			if ($role) {
-				$insertions += [
-					$colonne_role => $role
-				];
-			}
-
-			if (lien_triables($table_lien)) {
-				if (isset($qualif['rang_lien'])) {
-					$rang = $qualif['rang_lien'];
-				}
-				else {
-					$where = lien_where($primary, $id, $objet, $id_objet);
-					// si il y a deja un lien pour ce couple (avec un autre role?) on reprend le meme rang si non nul
-					if (!$rang = (int) sql_getfetsel('rang_lien', $table_lien, $where)) {
-						$where = lien_rang_where($table_lien, $primary, $id, $objet, $id_objet);
-						$rang = (int) sql_getfetsel('max(rang_lien)', $table_lien, $where);
-						// si aucun lien n'a de rang, on en introduit pas, on garde zero
-						if ($rang > 0) {
-							$rang = (int) $rang + 1;
-						}
-					}
-				}
-				$insertions['rang_lien'] = $rang;
-			}
-
-			$args = [
-				'table_lien' => $table_lien,
-				'objet_source' => $objet_source,
-				'id_objet_source' => $id,
-				'objet' => $objet,
-				'id_objet' => $id_objet,
-				'role' => $role,
-				'colonne_role' => $colonne_role,
-				'action' => 'insert',
-			];
-
-			// Envoyer aux plugins
-			$insertions = pipeline(
-				'pre_edition_lien',
-				[
-					'args' => $args,
-					'data' => $insertions
-				]
-			);
-			$args['id_objet'] = $insertions['id_objet'];
-
-			$where = lien_where($primary, $id, $objet, $id_objet, $cond);
-
-			if (
-				(($id_objet = (int) $insertions['id_objet']) || in_array($objet, ['site', 'rubrique'])) && !sql_getfetsel($primary, $table_lien, $where)
-			) {
-				if (lien_triables($table_lien) && isset($insertions['rang_lien']) && (int) $insertions['rang_lien']) {
-					$where_meme_lien = lien_where($primary, $id, $objet, $id_objet);
-					$where_meme_lien = implode(' AND ', $where_meme_lien);
-					// on decale les liens de rang_lien>=la valeur inseree pour faire la place
-					// sauf sur le meme lien avec un role eventuellement different
-					$w = lien_rang_where($table_lien, $primary, $id, $objet, $id_objet, ['rang_lien>=' . (int) $insertions['rang_lien'], "NOT($where_meme_lien)"]);
-					sql_update($table_lien, ['rang_lien' => 'rang_lien+1'], $w);
-				}
-
-				$e = sql_insertq($table_lien, $insertions);
-				if ($e !== false) {
-					$ins++;
-					lien_propage_date_modif($objet, $id_objet);
-					lien_propage_date_modif($objet_source, $id);
-					// Envoyer aux plugins
-					pipeline(
-						'post_edition_lien',
-						[
-							'args' => $args,
-							'data' => $insertions
-						]
-					);
-				} else {
-					$echec = true;
-				}
-			}
-		}
-	}
-	// si on a fait des insertions, on reordonne les liens concernes
-	// pas la peine si $qualif['rang_lien'] etait fournie, on va passer dans lien_set a suivre et donc finir le recomptage
-	if ($ins > 0 && empty($qualif['rang_lien'])) {
-		lien_ordonner($objet_source, $primary, $table_lien, $id, $objets);
-	}
-
-	return ($echec ? false : $ins);
+    $ins = 0;
+    $echec = null;
+    if (is_null($qualif)) {
+        $qualif = [];
+    }
+
+    foreach ($objets as $objet => $id_objets) {
+        if (!is_array($id_objets)) {
+            $id_objets = [$id_objets];
+        }
+
+        // role, colonne, where par défaut
+        [$role, $colonne_role, $cond] =
+            roles_trouver_dans_qualif($objet_source, $objet, $qualif);
+
+        foreach ($id_objets as $id_objet) {
+            $objet = (($objet == '*') ? $objet : objet_type($objet)); # securite
+
+            $insertions = [
+                'id_objet' => $id_objet,
+                'objet' => $objet,
+                $primary => $id
+            ];
+            // rôle en plus s'il est défini
+            if ($role) {
+                $insertions += [
+                    $colonne_role => $role
+                ];
+            }
+
+            if (lien_triables($table_lien)) {
+                if (isset($qualif['rang_lien'])) {
+                    $rang = $qualif['rang_lien'];
+                }
+                else {
+                    $where = lien_where($primary, $id, $objet, $id_objet);
+                    // si il y a deja un lien pour ce couple (avec un autre role?) on reprend le meme rang si non nul
+                    if (!$rang = (int) sql_getfetsel('rang_lien', $table_lien, $where)) {
+                        $where = lien_rang_where($table_lien, $primary, $id, $objet, $id_objet);
+                        $rang = (int) sql_getfetsel('max(rang_lien)', $table_lien, $where);
+                        // si aucun lien n'a de rang, on en introduit pas, on garde zero
+                        if ($rang > 0) {
+                            $rang = (int) $rang + 1;
+                        }
+                    }
+                }
+                $insertions['rang_lien'] = $rang;
+            }
+
+            $args = [
+                'table_lien' => $table_lien,
+                'objet_source' => $objet_source,
+                'id_objet_source' => $id,
+                'objet' => $objet,
+                'id_objet' => $id_objet,
+                'role' => $role,
+                'colonne_role' => $colonne_role,
+                'action' => 'insert',
+            ];
+
+            // Envoyer aux plugins
+            $insertions = pipeline(
+                'pre_edition_lien',
+                [
+                    'args' => $args,
+                    'data' => $insertions
+                ]
+            );
+            $args['id_objet'] = $insertions['id_objet'];
+
+            $where = lien_where($primary, $id, $objet, $id_objet, $cond);
+
+            if (
+                (($id_objet = (int) $insertions['id_objet']) || in_array($objet, ['site', 'rubrique'])) && !sql_getfetsel($primary, $table_lien, $where)
+            ) {
+                if (lien_triables($table_lien) && isset($insertions['rang_lien']) && (int) $insertions['rang_lien']) {
+                    $where_meme_lien = lien_where($primary, $id, $objet, $id_objet);
+                    $where_meme_lien = implode(' AND ', $where_meme_lien);
+                    // on decale les liens de rang_lien>=la valeur inseree pour faire la place
+                    // sauf sur le meme lien avec un role eventuellement different
+                    $w = lien_rang_where($table_lien, $primary, $id, $objet, $id_objet, ['rang_lien>=' . (int) $insertions['rang_lien'], "NOT($where_meme_lien)"]);
+                    sql_update($table_lien, ['rang_lien' => 'rang_lien+1'], $w);
+                }
+
+                $e = sql_insertq($table_lien, $insertions);
+                if ($e !== false) {
+                    $ins++;
+                    lien_propage_date_modif($objet, $id_objet);
+                    lien_propage_date_modif($objet_source, $id);
+                    // Envoyer aux plugins
+                    pipeline(
+                        'post_edition_lien',
+                        [
+                            'args' => $args,
+                            'data' => $insertions
+                        ]
+                    );
+                } else {
+                    $echec = true;
+                }
+            }
+        }
+    }
+    // si on a fait des insertions, on reordonne les liens concernes
+    // pas la peine si $qualif['rang_lien'] etait fournie, on va passer dans lien_set a suivre et donc finir le recomptage
+    if ($ins > 0 && empty($qualif['rang_lien'])) {
+        lien_ordonner($objet_source, $primary, $table_lien, $id, $objets);
+    }
+
+    return ($echec ? false : $ins);
 }
 
 
@@ -479,47 +479,47 @@ 
  * @param array|string $objets
  */
 function lien_ordonner($objet_source, $primary, $table_lien, $id, $objets) {
-	if (!lien_triables($table_lien)) {
-		return;
-	}
-
-	$deja_reordonne = [];
-
-	foreach ($objets as $objet => $id_objets) {
-		if (!is_array($id_objets)) {
-			$id_objets = [$id_objets];
-		}
-
-		foreach ($id_objets as $id_objet) {
-			if (empty($deja_reordonne[$id][$objet][$id_objet])) {
-				$objet = (($objet == '*') ? $objet : objet_type($objet)); # securite
-
-				$where = lien_rang_where($table_lien, $primary, $id, $objet, $id_objet);
-				$liens = sql_allfetsel("$primary, id_objet, objet, rang_lien", $table_lien, $where, '', 'rang_lien');
-
-				$rangs = array_column($liens, 'rang_lien');
-				if (count($rangs) && (max($rangs) > 0 || min($rangs) < 0)) {
-					$rang = 1;
-					foreach ($liens as $lien) {
-						if (empty($deja_reordonne[$lien[$primary]][$lien['objet']][$lien['id_objet']])) {
-							$where = lien_where($primary, $lien[$primary], $lien['objet'], $lien['id_objet'], ['rang_lien!=' . (int) $rang]);
-							sql_updateq($table_lien, ['rang_lien' => $rang], $where);
-
-							if (empty($deja_reordonne[$lien[$primary]])) {
-								$deja_reordonne[$lien[$primary]] = [];
-							}
-							if (empty($deja_reordonne[$lien[$primary]][$lien['objet']])) {
-								$deja_reordonne[$lien[$primary]][$lien['objet']] = [];
-							}
-							$deja_reordonne[$lien[$primary]][$lien['objet']][$lien['id_objet']] = $rang;
-
-							$rang++;
-						}
-					}
-				}
-			}
-		}
-	}
+    if (!lien_triables($table_lien)) {
+        return;
+    }
+
+    $deja_reordonne = [];
+
+    foreach ($objets as $objet => $id_objets) {
+        if (!is_array($id_objets)) {
+            $id_objets = [$id_objets];
+        }
+
+        foreach ($id_objets as $id_objet) {
+            if (empty($deja_reordonne[$id][$objet][$id_objet])) {
+                $objet = (($objet == '*') ? $objet : objet_type($objet)); # securite
+
+                $where = lien_rang_where($table_lien, $primary, $id, $objet, $id_objet);
+                $liens = sql_allfetsel("$primary, id_objet, objet, rang_lien", $table_lien, $where, '', 'rang_lien');
+
+                $rangs = array_column($liens, 'rang_lien');
+                if (count($rangs) && (max($rangs) > 0 || min($rangs) < 0)) {
+                    $rang = 1;
+                    foreach ($liens as $lien) {
+                        if (empty($deja_reordonne[$lien[$primary]][$lien['objet']][$lien['id_objet']])) {
+                            $where = lien_where($primary, $lien[$primary], $lien['objet'], $lien['id_objet'], ['rang_lien!=' . (int) $rang]);
+                            sql_updateq($table_lien, ['rang_lien' => $rang], $where);
+
+                            if (empty($deja_reordonne[$lien[$primary]])) {
+                                $deja_reordonne[$lien[$primary]] = [];
+                            }
+                            if (empty($deja_reordonne[$lien[$primary]][$lien['objet']])) {
+                                $deja_reordonne[$lien[$primary]][$lien['objet']] = [];
+                            }
+                            $deja_reordonne[$lien[$primary]][$lien['objet']][$lien['id_objet']] = $rang;
+
+                            $rang++;
+                        }
+                    }
+                }
+            }
+        }
+    }
 }
 
 
@@ -530,13 +530,13 @@ 
  * @return mixed
  */
 function lien_triables($table_lien) {
-	static $triables = [];
-	if (!isset($triables[$table_lien])) {
-		$trouver_table = charger_fonction('trouver_table', 'base');
-		$desc = $trouver_table($table_lien);
-		$triables[$table_lien] = $desc && isset($desc['field']['rang_lien']);
-	}
-	return $triables[$table_lien];
+    static $triables = [];
+    if (!isset($triables[$table_lien])) {
+        $trouver_table = charger_fonction('trouver_table', 'base');
+        $desc = $trouver_table($table_lien);
+        $triables[$table_lien] = $desc && isset($desc['field']['rang_lien']);
+    }
+    return $triables[$table_lien];
 }
 
 
@@ -552,50 +552,50 @@ 
  * @return array                        Liste des conditions
  */
 function lien_where($primary, $id_source, $objet, $id_objet, $cond = []) {
-	if (
-		!is_array($id_source) && !strlen($id_source) || !strlen($objet) || !is_array($id_objet) && !strlen($id_objet)
-	) {
-		return ['0=1'];
-	} // securite
-
-	$not = '';
-	if (is_array($id_source) && reset($id_source) == 'NOT') {
-		$not = array_shift($id_source);
-		$id_source = reset($id_source);
-	}
-
-	$where = $cond;
-
-	if ($id_source !== '*') {
-		$where[] = (is_array($id_source) ? sql_in(
-			addslashes($primary),
-			array_map('intval', $id_source),
-			$not
-		) : addslashes($primary) . ($not ? '<>' : '=') . (int) $id_source);
-	} elseif ($not) {
-		$where[] = '0=1';
-	} // idiot mais quand meme
-
-	$not = '';
-	if (is_array($id_objet) && reset($id_objet) == 'NOT') {
-		$not = array_shift($id_objet);
-		$id_objet = reset($id_objet);
-	}
-
-	if ($objet !== '*') {
-		$where[] = 'objet=' . sql_quote($objet);
-	}
-	if ($id_objet !== '*') {
-		$where[] = (is_array($id_objet) ? sql_in(
-			'id_objet',
-			array_map('intval', $id_objet),
-			$not
-		) : 'id_objet' . ($not ? '<>' : '=') . (int) $id_objet);
-	} elseif ($not) {
-		$where[] = '0=1';
-	} // idiot mais quand meme
-
-	return $where;
+    if (
+        !is_array($id_source) && !strlen($id_source) || !strlen($objet) || !is_array($id_objet) && !strlen($id_objet)
+    ) {
+        return ['0=1'];
+    } // securite
+
+    $not = '';
+    if (is_array($id_source) && reset($id_source) == 'NOT') {
+        $not = array_shift($id_source);
+        $id_source = reset($id_source);
+    }
+
+    $where = $cond;
+
+    if ($id_source !== '*') {
+        $where[] = (is_array($id_source) ? sql_in(
+            addslashes($primary),
+            array_map('intval', $id_source),
+            $not
+        ) : addslashes($primary) . ($not ? '<>' : '=') . (int) $id_source);
+    } elseif ($not) {
+        $where[] = '0=1';
+    } // idiot mais quand meme
+
+    $not = '';
+    if (is_array($id_objet) && reset($id_objet) == 'NOT') {
+        $not = array_shift($id_objet);
+        $id_objet = reset($id_objet);
+    }
+
+    if ($objet !== '*') {
+        $where[] = 'objet=' . sql_quote($objet);
+    }
+    if ($id_objet !== '*') {
+        $where[] = (is_array($id_objet) ? sql_in(
+            'id_objet',
+            array_map('intval', $id_objet),
+            $not
+        ) : 'id_objet' . ($not ? '<>' : '=') . (int) $id_objet);
+    } elseif ($not) {
+        $where[] = '0=1';
+    } // idiot mais quand meme
+
+    return $where;
 }
 
 /**
@@ -610,13 +610,13 @@ 
  */
 function lien_rang_where($table_lien, $primary, $id_source, $objet, $id_objet, $cond = []) {
 
-	// si on veut compter les rangs autrement que le core ne le fait par defaut, fournir le where adhoc
-	if (function_exists($f = 'lien_rang_where_' . $table_lien)) {
-		return $f($primary, $id_source, $objet, $id_objet, $cond);
-	}
+    // si on veut compter les rangs autrement que le core ne le fait par defaut, fournir le where adhoc
+    if (function_exists($f = 'lien_rang_where_' . $table_lien)) {
+        return $f($primary, $id_source, $objet, $id_objet, $cond);
+    }
 
-	// par defaut c'est un rang compté pour tous les id_source d'un couple objet-id_objet
-	return lien_where($primary, '*', $objet, $id_objet, $cond);
+    // par defaut c'est un rang compté pour tous les id_source d'un couple objet-id_objet
+    return lien_where($primary, '*', $objet, $id_objet, $cond);
 }
 
 /**
@@ -647,90 +647,90 @@ 
  */
 function lien_delete($objet_source, $primary, $table_lien, $id, $objets, $cond = null) {
 
-	$retire = [];
-	$dels = 0;
-	$echec = false;
-	if (is_null($cond)) {
-		$cond = [];
-	}
-
-	foreach ($objets as $objet => $id_objets) {
-		$objet = ($objet == '*') ? $objet : objet_type($objet); # securite
-		if (!is_array($id_objets) || reset($id_objets) == 'NOT') {
-			$id_objets = [$id_objets];
-		}
-		foreach ($id_objets as $id_objet) {
-			[$cond, $colonne_role, $role] = roles_creer_condition_role($objet_source, $objet, $cond);
-			// id_objet peut valoir '*'
-			$where = lien_where($primary, $id, $objet, $id_objet, $cond);
-
-			// lire les liens existants pour propager la date de modif
-			$select = "$primary,id_objet,objet";
-			if ($colonne_role) {
-				$select .= ",$colonne_role";
-			}
-			$liens = sql_allfetsel($select, $table_lien, $where);
-
-			// iterer sur les liens pour permettre aux plugins de gerer
-			foreach ($liens as $l) {
-				$args = [
-					'table_lien' => $table_lien,
-					'objet_source' => $objet_source,
-					'id_objet_source' => $l[$primary],
-					'objet' => $l['objet'],
-					'id_objet' => $l['id_objet'],
-					'colonne_role' => $colonne_role,
-					'role' => ($colonne_role ? $l[$colonne_role] : ''),
-					'action' => 'delete',
-				];
-
-				// Envoyer aux plugins
-				$l = pipeline(
-					'pre_edition_lien',
-					[
-						'args' => $args,
-						'data' => $l
-					]
-				);
-				$args['id_objet'] = $id_o = $l['id_objet'];
-
-				if (($id_o = (int) $l['id_objet']) || in_array($l['objet'], ['site', 'rubrique'])) {
-					$where = lien_where($primary, $l[$primary], $l['objet'], $id_o, $cond);
-					$e = sql_delete($table_lien, $where);
-					if ($e !== false) {
-						$dels += $e;
-						lien_propage_date_modif($l['objet'], $id_o);
-						lien_propage_date_modif($objet_source, $l[$primary]);
-					} else {
-						$echec = true;
-					}
-					$retire[] = [
-						'source' => [$objet_source => $l[$primary]],
-						'lien' => [$l['objet'] => $id_o],
-						'type' => $l['objet'],
-						'role' => ($colonne_role ? $l[$colonne_role] : ''),
-						'id' => $id_o
-					];
-					// Envoyer aux plugins
-					pipeline(
-						'post_edition_lien',
-						[
-							'args' => $args,
-							'data' => $l
-						]
-					);
-				}
-			}
-		}
-	}
-	// si on a supprime des liens, on reordonne les liens concernes
-	if ($dels) {
-		lien_ordonner($objet_source, $primary, $table_lien, $id, $objets);
-	}
-
-	pipeline('trig_supprimer_objets_lies', $retire);
-
-	return ($echec ? false : $dels);
+    $retire = [];
+    $dels = 0;
+    $echec = false;
+    if (is_null($cond)) {
+        $cond = [];
+    }
+
+    foreach ($objets as $objet => $id_objets) {
+        $objet = ($objet == '*') ? $objet : objet_type($objet); # securite
+        if (!is_array($id_objets) || reset($id_objets) == 'NOT') {
+            $id_objets = [$id_objets];
+        }
+        foreach ($id_objets as $id_objet) {
+            [$cond, $colonne_role, $role] = roles_creer_condition_role($objet_source, $objet, $cond);
+            // id_objet peut valoir '*'
+            $where = lien_where($primary, $id, $objet, $id_objet, $cond);
+
+            // lire les liens existants pour propager la date de modif
+            $select = "$primary,id_objet,objet";
+            if ($colonne_role) {
+                $select .= ",$colonne_role";
+            }
+            $liens = sql_allfetsel($select, $table_lien, $where);
+
+            // iterer sur les liens pour permettre aux plugins de gerer
+            foreach ($liens as $l) {
+                $args = [
+                    'table_lien' => $table_lien,
+                    'objet_source' => $objet_source,
+                    'id_objet_source' => $l[$primary],
+                    'objet' => $l['objet'],
+                    'id_objet' => $l['id_objet'],
+                    'colonne_role' => $colonne_role,
+                    'role' => ($colonne_role ? $l[$colonne_role] : ''),
+                    'action' => 'delete',
+                ];
+
+                // Envoyer aux plugins
+                $l = pipeline(
+                    'pre_edition_lien',
+                    [
+                        'args' => $args,
+                        'data' => $l
+                    ]
+                );
+                $args['id_objet'] = $id_o = $l['id_objet'];
+
+                if (($id_o = (int) $l['id_objet']) || in_array($l['objet'], ['site', 'rubrique'])) {
+                    $where = lien_where($primary, $l[$primary], $l['objet'], $id_o, $cond);
+                    $e = sql_delete($table_lien, $where);
+                    if ($e !== false) {
+                        $dels += $e;
+                        lien_propage_date_modif($l['objet'], $id_o);
+                        lien_propage_date_modif($objet_source, $l[$primary]);
+                    } else {
+                        $echec = true;
+                    }
+                    $retire[] = [
+                        'source' => [$objet_source => $l[$primary]],
+                        'lien' => [$l['objet'] => $id_o],
+                        'type' => $l['objet'],
+                        'role' => ($colonne_role ? $l[$colonne_role] : ''),
+                        'id' => $id_o
+                    ];
+                    // Envoyer aux plugins
+                    pipeline(
+                        'post_edition_lien',
+                        [
+                            'args' => $args,
+                            'data' => $l
+                        ]
+                    );
+                }
+            }
+        }
+    }
+    // si on a supprime des liens, on reordonne les liens concernes
+    if ($dels) {
+        lien_ordonner($objet_source, $primary, $table_lien, $id, $objets);
+    }
+
+    pipeline('trig_supprimer_objets_lies', $retire);
+
+    return ($echec ? false : $dels);
 }
 
 
@@ -753,67 +753,67 @@ 
  * @return bool|int
  */
 function lien_optimise($objet_source, $primary, $table_lien, $id, $objets) {
-	include_spip('genie/optimiser');
-	$echec = false;
-	$dels = 0;
-	foreach ($objets as $objet => $id_objets) {
-		$objet = ($objet == '*') ? $objet : objet_type($objet); # securite
-		if (!is_array($id_objets) || reset($id_objets) == 'NOT') {
-			$id_objets = [$id_objets];
-		}
-		foreach ($id_objets as $id_objet) {
-			$where = lien_where($primary, $id, $objet, $id_objet);
-			# les liens vers un objet inexistant
-			$r = sql_select('DISTINCT objet', $table_lien, $where);
-			while ($t = sql_fetch($r)) {
-				$type = $t['objet'];
-				$spip_table_objet = table_objet_sql($type);
-				$id_table_objet = id_table_objet($type);
-				$res = sql_select(
-					"L.$primary AS id,L.id_objet",
-					// la condition de jointure inclue L.objet='xxx' pour ne joindre que les bonnes lignes
-					// du coups toutes les lignes avec un autre objet ont un id_xxx=NULL puisque LEFT JOIN
-					// il faut les eliminier en repetant la condition dans le where L.objet='xxx'
-					"$table_lien AS L
+    include_spip('genie/optimiser');
+    $echec = false;
+    $dels = 0;
+    foreach ($objets as $objet => $id_objets) {
+        $objet = ($objet == '*') ? $objet : objet_type($objet); # securite
+        if (!is_array($id_objets) || reset($id_objets) == 'NOT') {
+            $id_objets = [$id_objets];
+        }
+        foreach ($id_objets as $id_objet) {
+            $where = lien_where($primary, $id, $objet, $id_objet);
+            # les liens vers un objet inexistant
+            $r = sql_select('DISTINCT objet', $table_lien, $where);
+            while ($t = sql_fetch($r)) {
+                $type = $t['objet'];
+                $spip_table_objet = table_objet_sql($type);
+                $id_table_objet = id_table_objet($type);
+                $res = sql_select(
+                    "L.$primary AS id,L.id_objet",
+                    // la condition de jointure inclue L.objet='xxx' pour ne joindre que les bonnes lignes
+                    // du coups toutes les lignes avec un autre objet ont un id_xxx=NULL puisque LEFT JOIN
+                    // il faut les eliminier en repetant la condition dans le where L.objet='xxx'
+                    "$table_lien AS L
 									LEFT JOIN $spip_table_objet AS O
 										ON (O.$id_table_objet=L.id_objet AND L.objet=" . sql_quote($type) . ')',
-					'L.objet=' . sql_quote($type) . " AND O.$id_table_objet IS NULL"
-				);
-				// sur une cle primaire composee, pas d'autres solutions que de virer un a un
-				while ($row = sql_fetch($res)) {
-					if ($primary === 'id_document' && in_array($type, ['site', 'rubrique']) && !(int) $row['id_objet']) {
-						continue; // gaffe, c'est le logo du site ou des rubriques!
-					}
-					$e = sql_delete(
-						$table_lien,
-						["$primary=" . $row['id'], 'id_objet=' . $row['id_objet'], 'objet=' . sql_quote($type)]
-					);
-					if ($e != false) {
-						$dels += $e;
-						spip_log(
-							'lien_optimise: Entree ' . $row['id'] . '/' . $row['id_objet'] . "/$type supprimee dans la table $table_lien",
-							'genie' . _LOG_INFO_IMPORTANTE
-						);
-					}
-				}
-			}
-
-			# les liens depuis un objet inexistant
-			$table_source = table_objet_sql($objet_source);
-			// filtrer selon $id, $objet, $id_objet eventuellement fournis
-			// (en general '*' pour chaque)
-			$where = lien_where("L.$primary", $id, $objet, $id_objet);
-			$where[] = "O.$primary IS NULL";
-			$res = sql_select(
-				"L.$primary AS id",
-				"$table_lien AS L LEFT JOIN $table_source AS O ON L.$primary=O.$primary",
-				$where
-			);
-			$dels += optimiser_sansref($table_lien, $primary, $res);
-		}
-	}
-
-	return ($echec ? false : $dels);
+                    'L.objet=' . sql_quote($type) . " AND O.$id_table_objet IS NULL"
+                );
+                // sur une cle primaire composee, pas d'autres solutions que de virer un a un
+                while ($row = sql_fetch($res)) {
+                    if ($primary === 'id_document' && in_array($type, ['site', 'rubrique']) && !(int) $row['id_objet']) {
+                        continue; // gaffe, c'est le logo du site ou des rubriques!
+                    }
+                    $e = sql_delete(
+                        $table_lien,
+                        ["$primary=" . $row['id'], 'id_objet=' . $row['id_objet'], 'objet=' . sql_quote($type)]
+                    );
+                    if ($e != false) {
+                        $dels += $e;
+                        spip_log(
+                            'lien_optimise: Entree ' . $row['id'] . '/' . $row['id_objet'] . "/$type supprimee dans la table $table_lien",
+                            'genie' . _LOG_INFO_IMPORTANTE
+                        );
+                    }
+                }
+            }
+
+            # les liens depuis un objet inexistant
+            $table_source = table_objet_sql($objet_source);
+            // filtrer selon $id, $objet, $id_objet eventuellement fournis
+            // (en general '*' pour chaque)
+            $where = lien_where("L.$primary", $id, $objet, $id_objet);
+            $where[] = "O.$primary IS NULL";
+            $res = sql_select(
+                "L.$primary AS id",
+                "$table_lien AS L LEFT JOIN $table_source AS O ON L.$primary=O.$primary",
+                $where
+            );
+            $dels += optimiser_sansref($table_lien, $primary, $res);
+        }
+    }
+
+    return ($echec ? false : $dels);
 }
 
 
@@ -845,92 +845,92 @@ 
  *     Nombre de modifications faites, false si échec.
  */
 function lien_set($objet_source, $primary, $table_lien, $id, $objets, $qualif) {
-	$echec = null;
-	$ok = 0;
-	$reordonner = false;
-	if (!$qualif) {
-		return false;
-	}
-	// nettoyer qualif qui peut venir directement d'un objet_trouver_lien :
-	unset($qualif[$primary]);
-	unset($qualif[$objet_source]);
-	if (isset($qualif['objet'])) {
-		unset($qualif[$qualif['objet']]);
-	}
-	unset($qualif['objet']);
-	unset($qualif['id_objet']);
-	foreach ($objets as $objet => $id_objets) {
-		// role, colonne, where par défaut
-		[$role, $colonne_role, $cond] =
-			roles_trouver_dans_qualif($objet_source, $objet, $qualif);
-
-		$objet = ($objet == '*') ? $objet : objet_type($objet); # securite
-		if (!is_array($id_objets) || reset($id_objets) == 'NOT') {
-			$id_objets = [$id_objets];
-		}
-		foreach ($id_objets as $id_objet) {
-			$args = [
-				'table_lien' => $table_lien,
-				'objet_source' => $objet_source,
-				'id_objet_source' => $id,
-				'objet' => $objet,
-				'id_objet' => $id_objet,
-				'role' => $role,
-				'colonne_role' => $colonne_role,
-				'action' => 'modifier',
-			];
-
-			// Envoyer aux plugins
-			$qualif = pipeline(
-				'pre_edition_lien',
-				[
-					'args' => $args,
-					'data' => $qualif,
-				]
-			);
-			$args['id_objet'] = $id_objet;
-
-			if (lien_triables($table_lien) && isset($qualif['rang_lien'])) {
-				if ((int) $qualif['rang_lien']) {
-					// on decale les liens de rang_lien>=la valeur inseree pour faire la place
-					// sauf sur le meme lien avec un role eventuellement different
-					$where_meme_lien = lien_where($primary, $id, $objet, $id_objet);
-					$where_meme_lien = implode(' AND ', $where_meme_lien);
-					$w = lien_rang_where($table_lien, $primary, $id, $objet, $id_objet, ['rang_lien>=' . (int) $qualif['rang_lien'], "NOT($where_meme_lien)"]);
-					sql_update($table_lien, ['rang_lien' => 'rang_lien+1'], $w);
-				}
-				// tous les liens de même rôle recoivent le rang indiqué aussi
-				if (roles_colonne($objet_source, $objet)) {
-					$w = lien_where($primary, $id, $objet, $id_objet);
-					sql_updateq($table_lien, ['rang_lien' => (int) $qualif['rang_lien']], $w);
-				}
-				$reordonner = true;
-			}
-
-			$where = lien_where($primary, $id, $objet, $id_objet, $cond);
-			$e = sql_updateq($table_lien, $qualif, $where);
-
-			if ($e === false) {
-				$echec = true;
-			} else {
-				// Envoyer aux plugins
-				pipeline(
-					'post_edition_lien',
-					[
-						'args' => $args,
-						'data' => $qualif
-					]
-				);
-				$ok++;
-			}
-		}
-	}
-	// si on a fait des modif de rang, on reordonne les liens concernes
-	if ($reordonner) {
-		lien_ordonner($objet_source, $primary, $table_lien, $id, $objets);
-	}
-
-	return ($echec ? false : $ok);
+    $echec = null;
+    $ok = 0;
+    $reordonner = false;
+    if (!$qualif) {
+        return false;
+    }
+    // nettoyer qualif qui peut venir directement d'un objet_trouver_lien :
+    unset($qualif[$primary]);
+    unset($qualif[$objet_source]);
+    if (isset($qualif['objet'])) {
+        unset($qualif[$qualif['objet']]);
+    }
+    unset($qualif['objet']);
+    unset($qualif['id_objet']);
+    foreach ($objets as $objet => $id_objets) {
+        // role, colonne, where par défaut
+        [$role, $colonne_role, $cond] =
+            roles_trouver_dans_qualif($objet_source, $objet, $qualif);
+
+        $objet = ($objet == '*') ? $objet : objet_type($objet); # securite
+        if (!is_array($id_objets) || reset($id_objets) == 'NOT') {
+            $id_objets = [$id_objets];
+        }
+        foreach ($id_objets as $id_objet) {
+            $args = [
+                'table_lien' => $table_lien,
+                'objet_source' => $objet_source,
+                'id_objet_source' => $id,
+                'objet' => $objet,
+                'id_objet' => $id_objet,
+                'role' => $role,
+                'colonne_role' => $colonne_role,
+                'action' => 'modifier',
+            ];
+
+            // Envoyer aux plugins
+            $qualif = pipeline(
+                'pre_edition_lien',
+                [
+                    'args' => $args,
+                    'data' => $qualif,
+                ]
+            );
+            $args['id_objet'] = $id_objet;
+
+            if (lien_triables($table_lien) && isset($qualif['rang_lien'])) {
+                if ((int) $qualif['rang_lien']) {
+                    // on decale les liens de rang_lien>=la valeur inseree pour faire la place
+                    // sauf sur le meme lien avec un role eventuellement different
+                    $where_meme_lien = lien_where($primary, $id, $objet, $id_objet);
+                    $where_meme_lien = implode(' AND ', $where_meme_lien);
+                    $w = lien_rang_where($table_lien, $primary, $id, $objet, $id_objet, ['rang_lien>=' . (int) $qualif['rang_lien'], "NOT($where_meme_lien)"]);
+                    sql_update($table_lien, ['rang_lien' => 'rang_lien+1'], $w);
+                }
+                // tous les liens de même rôle recoivent le rang indiqué aussi
+                if (roles_colonne($objet_source, $objet)) {
+                    $w = lien_where($primary, $id, $objet, $id_objet);
+                    sql_updateq($table_lien, ['rang_lien' => (int) $qualif['rang_lien']], $w);
+                }
+                $reordonner = true;
+            }
+
+            $where = lien_where($primary, $id, $objet, $id_objet, $cond);
+            $e = sql_updateq($table_lien, $qualif, $where);
+
+            if ($e === false) {
+                $echec = true;
+            } else {
+                // Envoyer aux plugins
+                pipeline(
+                    'post_edition_lien',
+                    [
+                        'args' => $args,
+                        'data' => $qualif
+                    ]
+                );
+                $ok++;
+            }
+        }
+    }
+    // si on a fait des modif de rang, on reordonne les liens concernes
+    if ($reordonner) {
+        lien_ordonner($objet_source, $primary, $table_lien, $id, $objets);
+    }
+
+    return ($echec ? false : $ok);
 }
 
 /**
@@ -960,23 +960,23 @@ 
  * @return array
  */
 function lien_find($objet_source, $primary, $table_lien, $id, $objets, $cond = null) {
-	$trouve = [];
-	foreach ($objets as $objet => $id_objets) {
-		$objet = ($objet == '*') ? $objet : objet_type($objet); # securite
-		// gerer les roles s'il y en a dans $cond
-		[$cond] = roles_creer_condition_role($objet_source, $objet, $cond, true);
-		// lien_where prend en charge les $id_objets sous forme int ou array
-		$where = lien_where($primary, $id, $objet, $id_objets, $cond);
-		$liens = sql_allfetsel('*', $table_lien, $where);
-		// ajouter les entrees objet_source et objet cible par convenance
-		foreach ($liens as $l) {
-			$l[$objet_source] = $l[$primary];
-			$l[$l['objet']] = $l['id_objet'];
-			$trouve[] = $l;
-		}
-	}
-
-	return $trouve;
+    $trouve = [];
+    foreach ($objets as $objet => $id_objets) {
+        $objet = ($objet == '*') ? $objet : objet_type($objet); # securite
+        // gerer les roles s'il y en a dans $cond
+        [$cond] = roles_creer_condition_role($objet_source, $objet, $cond, true);
+        // lien_where prend en charge les $id_objets sous forme int ou array
+        $where = lien_where($primary, $id, $objet, $id_objets, $cond);
+        $liens = sql_allfetsel('*', $table_lien, $where);
+        // ajouter les entrees objet_source et objet cible par convenance
+        foreach ($liens as $l) {
+            $l[$objet_source] = $l[$primary];
+            $l[$l['objet']] = $l['id_objet'];
+            $trouve[] = $l;
+        }
+    }
+
+    return $trouve;
 }
 
 /**
@@ -987,25 +987,25 @@ 
  * @param array|int $ids
  */
 function lien_propage_date_modif($objet, $ids) {
-	static $done = [];
-	$hash = md5($objet . serialize($ids));
-
-	// sql_updateq, peut être un rien lent.
-	// On évite de l'appeler 2 fois sur les mêmes choses
-	if (isset($done[$hash])) {
-		return;
-	}
-
-	$trouver_table = charger_fonction('trouver_table', 'base');
-
-	$table = table_objet_sql($objet);
-	if (
-		($desc = $trouver_table($table)) && isset($desc['field']['date_modif'])
-	) {
-		$primary = id_table_objet($objet);
-		$where = (is_array($ids) ? sql_in($primary, array_map('intval', $ids)) : "$primary=" . (int) $ids);
-		sql_updateq($table, ['date_modif' => date('Y-m-d H:i:s')], $where);
-	}
-
-	$done[$hash] = true;
+    static $done = [];
+    $hash = md5($objet . serialize($ids));
+
+    // sql_updateq, peut être un rien lent.
+    // On évite de l'appeler 2 fois sur les mêmes choses
+    if (isset($done[$hash])) {
+        return;
+    }
+
+    $trouver_table = charger_fonction('trouver_table', 'base');
+
+    $table = table_objet_sql($objet);
+    if (
+        ($desc = $trouver_table($table)) && isset($desc['field']['date_modif'])
+    ) {
+        $primary = id_table_objet($objet);
+        $where = (is_array($ids) ? sql_in($primary, array_map('intval', $ids)) : "$primary=" . (int) $ids);
+        sql_updateq($table, ['date_modif' => date('Y-m-d H:i:s')], $where);
+    }
+
+    $done[$hash] = true;
 }

Spacing +16 added lines, -16 removed lines

@@ -52,7 +52,7 @@ 
 	$l = '';
 	if (
 		($primary = id_table_objet($objet))
-		&& $trouver_table($l = $table_sql . '_liens', '', true, ['log_missing' => false])
+		&& $trouver_table($l = $table_sql.'_liens', '', true, ['log_missing' => false])
 		&& !preg_match(',[^\w],', $primary)
 		&& !preg_match(',[^\w],', $l)
 	) {
@@ -211,7 +211,7 @@ 
  * @return int
  */
 function objet_optimiser_liens($objets_source, $objets_lies) {
-	spip_log('objet_optimiser_liens : ' . json_encode($objets_source, JSON_THROW_ON_ERROR) . ', ' . json_encode($objets_lies, JSON_THROW_ON_ERROR), 'genie' . _LOG_DEBUG);
+	spip_log('objet_optimiser_liens : '.json_encode($objets_source, JSON_THROW_ON_ERROR).', '.json_encode($objets_lies, JSON_THROW_ON_ERROR), 'genie'._LOG_DEBUG);
 	return objet_traiter_liaisons('lien_optimise', $objets_source, $objets_lies);
 }
 
@@ -437,7 +437,7 @@ 
 					$where_meme_lien = implode(' AND ', $where_meme_lien);
 					// on decale les liens de rang_lien>=la valeur inseree pour faire la place
 					// sauf sur le meme lien avec un role eventuellement different
-					$w = lien_rang_where($table_lien, $primary, $id, $objet, $id_objet, ['rang_lien>=' . (int) $insertions['rang_lien'], "NOT($where_meme_lien)"]);
+					$w = lien_rang_where($table_lien, $primary, $id, $objet, $id_objet, ['rang_lien>='.(int) $insertions['rang_lien'], "NOT($where_meme_lien)"]);
 					sql_update($table_lien, ['rang_lien' => 'rang_lien+1'], $w);
 				}
 
@@ -502,7 +502,7 @@ 
 					$rang = 1;
 					foreach ($liens as $lien) {
 						if (empty($deja_reordonne[$lien[$primary]][$lien['objet']][$lien['id_objet']])) {
-							$where = lien_where($primary, $lien[$primary], $lien['objet'], $lien['id_objet'], ['rang_lien!=' . (int) $rang]);
+							$where = lien_where($primary, $lien[$primary], $lien['objet'], $lien['id_objet'], ['rang_lien!='.(int) $rang]);
 							sql_updateq($table_lien, ['rang_lien' => $rang], $where);
 
 							if (empty($deja_reordonne[$lien[$primary]])) {
@@ -571,7 +571,7 @@ 
 			addslashes($primary),
 			array_map('intval', $id_source),
 			$not
-		) : addslashes($primary) . ($not ? '<>' : '=') . (int) $id_source);
+		) : addslashes($primary).($not ? '<>' : '=').(int) $id_source);
 	} elseif ($not) {
 		$where[] = '0=1';
 	} // idiot mais quand meme
@@ -583,14 +583,14 @@ 
 	}
 
 	if ($objet !== '*') {
-		$where[] = 'objet=' . sql_quote($objet);
+		$where[] = 'objet='.sql_quote($objet);
 	}
 	if ($id_objet !== '*') {
 		$where[] = (is_array($id_objet) ? sql_in(
 			'id_objet',
 			array_map('intval', $id_objet),
 			$not
-		) : 'id_objet' . ($not ? '<>' : '=') . (int) $id_objet);
+		) : 'id_objet'.($not ? '<>' : '=').(int) $id_objet);
 	} elseif ($not) {
 		$where[] = '0=1';
 	} // idiot mais quand meme
@@ -611,7 +611,7 @@ 
 function lien_rang_where($table_lien, $primary, $id_source, $objet, $id_objet, $cond = []) {
 
 	// si on veut compter les rangs autrement que le core ne le fait par defaut, fournir le where adhoc
-	if (function_exists($f = 'lien_rang_where_' . $table_lien)) {
+	if (function_exists($f = 'lien_rang_where_'.$table_lien)) {
 		return $f($primary, $id_source, $objet, $id_objet, $cond);
 	}
 
@@ -776,8 +776,8 @@ 
 					// il faut les eliminier en repetant la condition dans le where L.objet='xxx'
 					"$table_lien AS L
 									LEFT JOIN $spip_table_objet AS O
-										ON (O.$id_table_objet=L.id_objet AND L.objet=" . sql_quote($type) . ')',
-					'L.objet=' . sql_quote($type) . " AND O.$id_table_objet IS NULL"
+										ON (O.$id_table_objet=L.id_objet AND L.objet=".sql_quote($type).')',
+					'L.objet='.sql_quote($type)." AND O.$id_table_objet IS NULL"
 				);
 				// sur une cle primaire composee, pas d'autres solutions que de virer un a un
 				while ($row = sql_fetch($res)) {
@@ -786,13 +786,13 @@ 
 					}
 					$e = sql_delete(
 						$table_lien,
-						["$primary=" . $row['id'], 'id_objet=' . $row['id_objet'], 'objet=' . sql_quote($type)]
+						["$primary=".$row['id'], 'id_objet='.$row['id_objet'], 'objet='.sql_quote($type)]
 					);
 					if ($e != false) {
 						$dels += $e;
 						spip_log(
-							'lien_optimise: Entree ' . $row['id'] . '/' . $row['id_objet'] . "/$type supprimee dans la table $table_lien",
-							'genie' . _LOG_INFO_IMPORTANTE
+							'lien_optimise: Entree '.$row['id'].'/'.$row['id_objet']."/$type supprimee dans la table $table_lien",
+							'genie'._LOG_INFO_IMPORTANTE
 						);
 					}
 				}
@@ -896,7 +896,7 @@ 
 					// sauf sur le meme lien avec un role eventuellement different
 					$where_meme_lien = lien_where($primary, $id, $objet, $id_objet);
 					$where_meme_lien = implode(' AND ', $where_meme_lien);
-					$w = lien_rang_where($table_lien, $primary, $id, $objet, $id_objet, ['rang_lien>=' . (int) $qualif['rang_lien'], "NOT($where_meme_lien)"]);
+					$w = lien_rang_where($table_lien, $primary, $id, $objet, $id_objet, ['rang_lien>='.(int) $qualif['rang_lien'], "NOT($where_meme_lien)"]);
 					sql_update($table_lien, ['rang_lien' => 'rang_lien+1'], $w);
 				}
 				// tous les liens de même rôle recoivent le rang indiqué aussi
@@ -988,7 +988,7 @@ 
  */
 function lien_propage_date_modif($objet, $ids) {
 	static $done = [];
-	$hash = md5($objet . serialize($ids));
+	$hash = md5($objet.serialize($ids));
 
 	// sql_updateq, peut être un rien lent.
 	// On évite de l'appeler 2 fois sur les mêmes choses
@@ -1003,7 +1003,7 @@ 
 		($desc = $trouver_table($table)) && isset($desc['field']['date_modif'])
 	) {
 		$primary = id_table_objet($objet);
-		$where = (is_array($ids) ? sql_in($primary, array_map('intval', $ids)) : "$primary=" . (int) $ids);
+		$where = (is_array($ids) ? sql_in($primary, array_map('intval', $ids)) : "$primary=".(int) $ids);
 		sql_updateq($table, ['date_modif' => date('Y-m-d H:i:s')], $where);
 	}
 

Braces +1 added lines, -2 removed lines

@@ -390,8 +390,7 @@
 			if (lien_triables($table_lien)) {
 				if (isset($qualif['rang_lien'])) {
 					$rang = $qualif['rang_lien'];
-				}
-				else {
+				} else {
 					$where = lien_where($primary, $id, $objet, $id_objet);
 					// si il y a deja un lien pour ce couple (avec un autre role?) on reprend le meme rang si non nul
 					if (!$rang = (int) sql_getfetsel('rang_lien', $table_lien, $where)) {

ecrire/action/instituer_langue_rubrique.php

Indentation +35 added lines, -35 removed lines

@@ -16,7 +16,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -24,41 +24,41 @@ 
  **/
 function action_instituer_langue_rubrique_dist() {
 
-	$securiser_action = charger_fonction('securiser_action', 'inc');
-	$arg = $securiser_action();
-	$changer_lang = _request('changer_lang');
+    $securiser_action = charger_fonction('securiser_action', 'inc');
+    $arg = $securiser_action();
+    $changer_lang = _request('changer_lang');
 
-	[$id_rubrique, $id_parent] = preg_split('/\W/', (string) $arg);
+    [$id_rubrique, $id_parent] = preg_split('/\W/', (string) $arg);
 
-	if (
-		$changer_lang
-		&& $id_rubrique > 0
-		&& $GLOBALS['meta']['multi_rubriques'] == 'oui'
-		&& ($GLOBALS['meta']['multi_secteurs'] == 'non' || $id_parent == 0)
-	) {
-		if ($changer_lang != 'herit') {
-			sql_updateq(
-				'spip_rubriques',
-				['lang' => $changer_lang, 'langue_choisie' => 'oui'],
-				"id_rubrique=$id_rubrique"
-			);
-		} else {
-			if ($id_parent == 0) {
-				$langue_parent = $GLOBALS['meta']['langue_site'];
-			} else {
-				$langue_parent = sql_getfetsel('lang', 'spip_rubriques', "id_rubrique=$id_parent");
-			}
-			sql_updateq(
-				'spip_rubriques',
-				['lang' => $langue_parent, 'langue_choisie' => 'non'],
-				"id_rubrique=$id_rubrique"
-			);
-		}
-		include_spip('inc/rubriques');
-		calculer_langues_rubriques();
+    if (
+        $changer_lang
+        && $id_rubrique > 0
+        && $GLOBALS['meta']['multi_rubriques'] == 'oui'
+        && ($GLOBALS['meta']['multi_secteurs'] == 'non' || $id_parent == 0)
+    ) {
+        if ($changer_lang != 'herit') {
+            sql_updateq(
+                'spip_rubriques',
+                ['lang' => $changer_lang, 'langue_choisie' => 'oui'],
+                "id_rubrique=$id_rubrique"
+            );
+        } else {
+            if ($id_parent == 0) {
+                $langue_parent = $GLOBALS['meta']['langue_site'];
+            } else {
+                $langue_parent = sql_getfetsel('lang', 'spip_rubriques', "id_rubrique=$id_parent");
+            }
+            sql_updateq(
+                'spip_rubriques',
+                ['lang' => $langue_parent, 'langue_choisie' => 'non'],
+                "id_rubrique=$id_rubrique"
+            );
+        }
+        include_spip('inc/rubriques');
+        calculer_langues_rubriques();
 
-		// invalider les caches marques de cette rubrique
-		include_spip('inc/invalideur');
-		suivre_invalideur("id='rubrique/$id_rubrique'");
-	}
+        // invalider les caches marques de cette rubrique
+        include_spip('inc/invalideur');
+        suivre_invalideur("id='rubrique/$id_rubrique'");
+    }
 }