spip / SPIP

ecrire/plugins/get_infos.php

Indentation +101 added lines, -101 removed lines

@@ -17,7 +17,7 @@ 
  **/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -33,110 +33,110 @@ 
  * @return array
  */
 function plugins_get_infos_dist($plug = false, $reload = false, $dir = _DIR_PLUGINS, $clean_old = false) {
-	$contenu = null;
-	$res = null;
-	static $cache = '';
-	static $filecache = '';
-
-	if ($cache === '') {
-		$filecache = _DIR_TMP . 'plugin_xml_cache.gz';
-		if (is_file($filecache)) {
-			lire_fichier($filecache, $contenu);
-			$cache = unserialize($contenu);
-		}
-		if (!is_array($cache)) {
-			$cache = [];
-		}
-	}
-
-	if (defined('_VAR_MODE') and _VAR_MODE == 'recalcul') {
-		$reload = true;
-	}
-
-	if ($plug === false) {
-		ecrire_fichier($filecache, serialize($cache));
-
-		return $cache;
-	} elseif (is_string($plug)) {
-		$res = plugins_get_infos_un($plug, $reload, $dir, $cache);
-	} elseif (is_array($plug)) {
-		$res = false;
-		if (!$reload) {
-			$reload = -1;
-		}
-		foreach ($plug as $nom) {
-			$res |= plugins_get_infos_un($nom, $reload, $dir, $cache);
-		}
-
-		// Nettoyer le cache des vieux plugins qui ne sont plus la
-		if ($clean_old and isset($cache[$dir]) and is_countable($cache[$dir]) ? count($cache[$dir]) : 0) {
-			foreach (array_keys($cache[$dir]) as $p) {
-				if (!in_array($p, $plug)) {
-					unset($cache[$dir][$p]);
-				}
-			}
-		}
-	}
-	if ($res) {
-		ecrire_fichier($filecache, serialize($cache));
-	}
-	if (!isset($cache[$dir])) {
-		return [];
-	}
-	if (is_string($plug)) {
-		return $cache[$dir][$plug] ?? [];
-	} else {
-		return $cache[$dir];
-	}
+    $contenu = null;
+    $res = null;
+    static $cache = '';
+    static $filecache = '';
+
+    if ($cache === '') {
+        $filecache = _DIR_TMP . 'plugin_xml_cache.gz';
+        if (is_file($filecache)) {
+            lire_fichier($filecache, $contenu);
+            $cache = unserialize($contenu);
+        }
+        if (!is_array($cache)) {
+            $cache = [];
+        }
+    }
+
+    if (defined('_VAR_MODE') and _VAR_MODE == 'recalcul') {
+        $reload = true;
+    }
+
+    if ($plug === false) {
+        ecrire_fichier($filecache, serialize($cache));
+
+        return $cache;
+    } elseif (is_string($plug)) {
+        $res = plugins_get_infos_un($plug, $reload, $dir, $cache);
+    } elseif (is_array($plug)) {
+        $res = false;
+        if (!$reload) {
+            $reload = -1;
+        }
+        foreach ($plug as $nom) {
+            $res |= plugins_get_infos_un($nom, $reload, $dir, $cache);
+        }
+
+        // Nettoyer le cache des vieux plugins qui ne sont plus la
+        if ($clean_old and isset($cache[$dir]) and is_countable($cache[$dir]) ? count($cache[$dir]) : 0) {
+            foreach (array_keys($cache[$dir]) as $p) {
+                if (!in_array($p, $plug)) {
+                    unset($cache[$dir][$p]);
+                }
+            }
+        }
+    }
+    if ($res) {
+        ecrire_fichier($filecache, serialize($cache));
+    }
+    if (!isset($cache[$dir])) {
+        return [];
+    }
+    if (is_string($plug)) {
+        return $cache[$dir][$plug] ?? [];
+    } else {
+        return $cache[$dir];
+    }
 }
 
 
 function plugins_get_infos_un($plug, $reload, $dir, &$cache) {
-	if (!is_readable($file = "$dir$plug/paquet.xml")) {
-		return false;
-	}
-	$time = intval(@filemtime($file));
-	if ($time < 0) {
-		return false;
-	}
-	$md5 = md5_file($file);
-
-	$pcache = $cache[$dir][$plug] ?? ['filemtime' => 0, 'md5_file' => ''];
-
-	// si le cache est valide
-	if (
-		(intval($reload) <= 0)
-		and ($time > 0)
-		and ($time <= $pcache['filemtime'])
-		and $md5 == $pcache['md5_file']
-	) {
-		return false;
-	}
-
-	// si on arrive pas a lire le fichier, se contenter du cache
-	if (!($texte = spip_file_get_contents($file))) {
-		return false;
-	}
-
-	$f = charger_fonction('infos_paquet', 'plugins');
-	$ret = $f($texte, $plug, $dir);
-	$ret['filemtime'] = $time;
-	$ret['md5_file'] = $md5;
-	// Si on lit le paquet.xml de SPIP, on rajoute un procure php afin que les plugins puissent
-	// utiliser un necessite php. SPIP procure donc la version php courante du serveur.
-	// chaque librairie php est aussi procurée, par exemple 'php:curl'.
-	if (isset($ret['prefix']) and $ret['prefix'] == 'spip') {
-		$ret['procure']['php'] = ['nom' => 'php', 'version' => phpversion()];
-		foreach (get_loaded_extensions() as $ext) {
-			$ret['procure']['php:' . $ext] = ['nom' => 'php:' . $ext, 'version' => phpversion($ext)];
-		}
-	}
-	$diff = ($ret != $pcache);
-
-	if ($diff) {
-		$cache[$dir][$plug] = $ret;
+    if (!is_readable($file = "$dir$plug/paquet.xml")) {
+        return false;
+    }
+    $time = intval(@filemtime($file));
+    if ($time < 0) {
+        return false;
+    }
+    $md5 = md5_file($file);
+
+    $pcache = $cache[$dir][$plug] ?? ['filemtime' => 0, 'md5_file' => ''];
+
+    // si le cache est valide
+    if (
+        (intval($reload) <= 0)
+        and ($time > 0)
+        and ($time <= $pcache['filemtime'])
+        and $md5 == $pcache['md5_file']
+    ) {
+        return false;
+    }
+
+    // si on arrive pas a lire le fichier, se contenter du cache
+    if (!($texte = spip_file_get_contents($file))) {
+        return false;
+    }
+
+    $f = charger_fonction('infos_paquet', 'plugins');
+    $ret = $f($texte, $plug, $dir);
+    $ret['filemtime'] = $time;
+    $ret['md5_file'] = $md5;
+    // Si on lit le paquet.xml de SPIP, on rajoute un procure php afin que les plugins puissent
+    // utiliser un necessite php. SPIP procure donc la version php courante du serveur.
+    // chaque librairie php est aussi procurée, par exemple 'php:curl'.
+    if (isset($ret['prefix']) and $ret['prefix'] == 'spip') {
+        $ret['procure']['php'] = ['nom' => 'php', 'version' => phpversion()];
+        foreach (get_loaded_extensions() as $ext) {
+            $ret['procure']['php:' . $ext] = ['nom' => 'php:' . $ext, 'version' => phpversion($ext)];
+        }
+    }
+    $diff = ($ret != $pcache);
+
+    if ($diff) {
+        $cache[$dir][$plug] = $ret;
 #       echo count($cache[$dir]), $dir,$plug, " $reloadc<br>";
-	}
+    }
 
-	return $diff;
+    return $diff;
 }

ecrire/plugins/extraire_pipelines.php

Indentation +24 added lines, -24 removed lines

@@ -11,7 +11,7 @@ 
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 
@@ -21,28 +21,28 @@ 
  * @param array $arbre
  */
 function plugins_extraire_pipelines_dist(&$arbre) {
-	$pipes = null;
-	$tag = null;
-	$pipeline = [];
-	if (spip_xml_match_nodes(',^pipeline,', $arbre, $pipes)) {
-		foreach ($pipes as $tag => $p) {
-			if (!is_array($p[0])) {
-				[$tag, $att] = spip_xml_decompose_tag($tag);
-				$pipeline[] = $att;
-			} else {
-				foreach ($p as $pipe) {
-					$att = [];
-					if (is_array($pipe)) {
-						foreach ($pipe as $k => $t) {
-							$att[$k] = trim(end($t));
-						}
-					}
-					$pipeline[] = $att;
-				}
-			}
-		}
-		unset($arbre[$tag]);
-	}
+    $pipes = null;
+    $tag = null;
+    $pipeline = [];
+    if (spip_xml_match_nodes(',^pipeline,', $arbre, $pipes)) {
+        foreach ($pipes as $tag => $p) {
+            if (!is_array($p[0])) {
+                [$tag, $att] = spip_xml_decompose_tag($tag);
+                $pipeline[] = $att;
+            } else {
+                foreach ($p as $pipe) {
+                    $att = [];
+                    if (is_array($pipe)) {
+                        foreach ($pipe as $k => $t) {
+                            $att[$k] = trim(end($t));
+                        }
+                    }
+                    $pipeline[] = $att;
+                }
+            }
+        }
+        unset($arbre[$tag]);
+    }
 
-	return $pipeline;
+    return $pipeline;
 }

ecrire/plugins/infos_paquet.php

Indentation +207 added lines, -207 removed lines

@@ -11,7 +11,7 @@ 
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -25,64 +25,64 @@ 
  * @return array
  */
 function plugins_infos_paquet($desc, $plug = '', $dir_plugins = _DIR_PLUGINS) {
-	static $process = [ // tableau constant
-		'debut' => 'paquet_debutElement',
-		'fin' => 'paquet_finElement',
-		'text' => 'paquet_textElement'
-	];
-
-	$valider_xml = charger_fonction('valider', 'xml');
-	$vxml = $valider_xml($desc, false, $process, 'paquet.dtd', 'utf-8');
-	if (!$vxml->err) {
-		// On veut toutes les variantes selon la version de SPIP
-		if (!$plug) {
-			return $vxml->versions;
-		}
-
-		// compatibilite avec l'existant:
-		$tree = $vxml->versions['0'];
-
-		// l'arbre renvoie parfois un tag vide... etrange. Pas la peine de garder ca.
-		if (isset($tree['']) and !strlen($tree[''])) {
-			unset($tree['']);
-		}
-
-		$tree['slogan'] = $tree['prefix'] . '_slogan';
-		$tree['description'] = $tree['prefix'] . '_description';
-		paquet_readable_files($tree, "$dir_plugins$plug/");
-		if (!$tree['chemin']) {
-			$tree['chemin'] = [];
-		}
-
-		// On verifie qu'il existe des balises spip qu'il faudrait rajouter dans
-		// la structure d'infos du paquet en fonction de la version spip courante
-		if ((is_countable($vxml->versions) ? count($vxml->versions) : 0) > 1) {
-			$vspip = $GLOBALS['spip_version_branche'];
-			foreach ($vxml->versions as $_compatibilite => $_version) {
-				if (
-					($_version['balise'] == 'spip')
-					and (plugin_version_compatible($_compatibilite, $vspip, 'spip'))
-				) {
-					// on merge les sous-balises de la balise spip compatible avec celles de la
-					// balise paquet
-					foreach ($_version as $_index => $_balise) {
-						if ($_index and $_index != 'balise') {
-							$tree[$_index] = array_merge($tree[$_index], $_balise);
-						}
-					}
-				}
-			}
-		}
-
-		return $tree;
-	}
-
-	// Prendre les messages d'erreur sans les numeros de lignes
-	$msg = array_column($vxml->err, 0);
-	$t = _T('plugins_erreur', ['plugins' => $plug]);
-	array_unshift($msg, $t . " <ul class='erreur_xml'><li>" . reset($msg) . '</li></ul>');
-
-	return ['erreur' => $msg];
+    static $process = [ // tableau constant
+        'debut' => 'paquet_debutElement',
+        'fin' => 'paquet_finElement',
+        'text' => 'paquet_textElement'
+    ];
+
+    $valider_xml = charger_fonction('valider', 'xml');
+    $vxml = $valider_xml($desc, false, $process, 'paquet.dtd', 'utf-8');
+    if (!$vxml->err) {
+        // On veut toutes les variantes selon la version de SPIP
+        if (!$plug) {
+            return $vxml->versions;
+        }
+
+        // compatibilite avec l'existant:
+        $tree = $vxml->versions['0'];
+
+        // l'arbre renvoie parfois un tag vide... etrange. Pas la peine de garder ca.
+        if (isset($tree['']) and !strlen($tree[''])) {
+            unset($tree['']);
+        }
+
+        $tree['slogan'] = $tree['prefix'] . '_slogan';
+        $tree['description'] = $tree['prefix'] . '_description';
+        paquet_readable_files($tree, "$dir_plugins$plug/");
+        if (!$tree['chemin']) {
+            $tree['chemin'] = [];
+        }
+
+        // On verifie qu'il existe des balises spip qu'il faudrait rajouter dans
+        // la structure d'infos du paquet en fonction de la version spip courante
+        if ((is_countable($vxml->versions) ? count($vxml->versions) : 0) > 1) {
+            $vspip = $GLOBALS['spip_version_branche'];
+            foreach ($vxml->versions as $_compatibilite => $_version) {
+                if (
+                    ($_version['balise'] == 'spip')
+                    and (plugin_version_compatible($_compatibilite, $vspip, 'spip'))
+                ) {
+                    // on merge les sous-balises de la balise spip compatible avec celles de la
+                    // balise paquet
+                    foreach ($_version as $_index => $_balise) {
+                        if ($_index and $_index != 'balise') {
+                            $tree[$_index] = array_merge($tree[$_index], $_balise);
+                        }
+                    }
+                }
+            }
+        }
+
+        return $tree;
+    }
+
+    // Prendre les messages d'erreur sans les numeros de lignes
+    $msg = array_column($vxml->err, 0);
+    $t = _T('plugins_erreur', ['plugins' => $plug]);
+    array_unshift($msg, $t . " <ul class='erreur_xml'><li>" . reset($msg) . '</li></ul>');
+
+    return ['erreur' => $msg];
 }
 
 /**
@@ -94,11 +94,11 @@ 
  * @return void
  */
 function paquet_readable_files(&$tree, $dir) {
-	$prefix = strtolower($tree['prefix']);
+    $prefix = strtolower($tree['prefix']);
 
-	$tree['options'] = (is_readable($dir . $f = ($prefix . '_options.php'))) ? [$f] : [];
-	$tree['fonctions'] = (is_readable($dir . $f = ($prefix . '_fonctions.php'))) ? [$f] : [];
-	$tree['install'] = (is_readable($dir . $f = ($prefix . '_administrations.php'))) ? [$f] : [];
+    $tree['options'] = (is_readable($dir . $f = ($prefix . '_options.php'))) ? [$f] : [];
+    $tree['fonctions'] = (is_readable($dir . $f = ($prefix . '_fonctions.php'))) ? [$f] : [];
+    $tree['install'] = (is_readable($dir . $f = ($prefix . '_administrations.php'))) ? [$f] : [];
 }
 
 /**
@@ -115,35 +115,35 @@ 
  * @param array $attrs
  */
 function paquet_debutElement($phraseur, $name, $attrs) {
-	xml_debutElement($phraseur, $name, $attrs);
-	if ($phraseur->err) {
-		return;
-	}
-	if (($name == 'paquet') or ($name == 'spip')) {
-		if ($name == 'spip') {
-			$n = $attrs['compatibilite'];
-			$attrs = [];
-		} else {
-			$n = '0';
-			$phraseur->contenu['paquet'] = $attrs;
-			$attrs['menu'] = [];
-			$attrs['chemin'] = [];
-			$attrs['necessite'] = [];
-			$attrs['lib'] = [];
-			$attrs['onglet'] = [];
-			$attrs['procure'] = [];
-			$attrs['pipeline'] = [];
-			$attrs['utilise'] = [];
-			$attrs['style'] = [];
-			$attrs['script'] = [];
-			$attrs['genie'] = [];
-		}
-		$phraseur->contenu['compatible'] = $n;
-		$phraseur->versions[$phraseur->contenu['compatible']] = $attrs;
-	} else {
-		$phraseur->versions[$phraseur->contenu['compatible']][$name][0] = $attrs;
-	}
-	$phraseur->versions[$phraseur->contenu['compatible']][''] = '';
+    xml_debutElement($phraseur, $name, $attrs);
+    if ($phraseur->err) {
+        return;
+    }
+    if (($name == 'paquet') or ($name == 'spip')) {
+        if ($name == 'spip') {
+            $n = $attrs['compatibilite'];
+            $attrs = [];
+        } else {
+            $n = '0';
+            $phraseur->contenu['paquet'] = $attrs;
+            $attrs['menu'] = [];
+            $attrs['chemin'] = [];
+            $attrs['necessite'] = [];
+            $attrs['lib'] = [];
+            $attrs['onglet'] = [];
+            $attrs['procure'] = [];
+            $attrs['pipeline'] = [];
+            $attrs['utilise'] = [];
+            $attrs['style'] = [];
+            $attrs['script'] = [];
+            $attrs['genie'] = [];
+        }
+        $phraseur->contenu['compatible'] = $n;
+        $phraseur->versions[$phraseur->contenu['compatible']] = $attrs;
+    } else {
+        $phraseur->versions[$phraseur->contenu['compatible']][$name][0] = $attrs;
+    }
+    $phraseur->versions[$phraseur->contenu['compatible']][''] = '';
 }
 
 /**
@@ -155,11 +155,11 @@ 
  * @param string $data
  */
 function paquet_textElement($phraseur, $data) {
-	xml_textElement($phraseur, $data);
-	if ($phraseur->err or !(trim($data))) {
-		return;
-	}
-	$phraseur->versions[$phraseur->contenu['compatible']][''] .= $data;
+    xml_textElement($phraseur, $data);
+    if ($phraseur->err or !(trim($data))) {
+        return;
+    }
+    $phraseur->versions[$phraseur->contenu['compatible']][''] .= $data;
 }
 
 /**
@@ -171,32 +171,32 @@ 
  * @param string $name
  */
 function paquet_finElement($phraseur, $name) {
-	if ($phraseur->err) {
-		return;
-	}
-	$n = $phraseur->contenu['compatible'];
-
-	if (isset($phraseur->versions[$n][$name][0]) and is_array($phraseur->versions[$n][$name][0])) {
-		$attrs = $phraseur->versions[$n][$name][0];
-		unset($phraseur->versions[$n][$name][0]);
-	} else {
-		$attrs = [];
-	}
-
-	$texte = trim($phraseur->versions[$n]['']);
-	$phraseur->versions[$n][''] = '';
-
-	$f = 'info_paquet_' . $name;
-	if (function_exists($f)) {
-		$f($phraseur, $attrs, $texte);
-	} elseif (!$attrs) {
-		$phraseur->versions[$n][$name] = $texte;
-	} else {
-		// Traitement generique. Si $attrs['nom'] n'existe pas, ce n'est pas normal ici
-		$phraseur->versions[$n][$name][$attrs['nom']] = $attrs;
-		#	  echo("<br>pour $name $n " . $attrs['nom']); var_dump($phraseur->versions[$n]);
-	}
-	xml_finElement($phraseur, $name, $attrs);
+    if ($phraseur->err) {
+        return;
+    }
+    $n = $phraseur->contenu['compatible'];
+
+    if (isset($phraseur->versions[$n][$name][0]) and is_array($phraseur->versions[$n][$name][0])) {
+        $attrs = $phraseur->versions[$n][$name][0];
+        unset($phraseur->versions[$n][$name][0]);
+    } else {
+        $attrs = [];
+    }
+
+    $texte = trim($phraseur->versions[$n]['']);
+    $phraseur->versions[$n][''] = '';
+
+    $f = 'info_paquet_' . $name;
+    if (function_exists($f)) {
+        $f($phraseur, $attrs, $texte);
+    } elseif (!$attrs) {
+        $phraseur->versions[$n][$name] = $texte;
+    } else {
+        // Traitement generique. Si $attrs['nom'] n'existe pas, ce n'est pas normal ici
+        $phraseur->versions[$n][$name][$attrs['nom']] = $attrs;
+        #	  echo("<br>pour $name $n " . $attrs['nom']); var_dump($phraseur->versions[$n]);
+    }
+    xml_finElement($phraseur, $name, $attrs);
 }
 
 /**
@@ -208,13 +208,13 @@ 
  * @param string $texte
  */
 function info_paquet_licence($phraseur, $attrs, $texte) {
-	if (isset($attrs['lien'])) {
-		$lien = $attrs['lien'];
-	} else {
-		$lien = '';
-	}
-	$n = $phraseur->contenu['compatible'];
-	$phraseur->versions[$n]['licence'][] = ['nom' => $texte, 'url' => $lien];
+    if (isset($attrs['lien'])) {
+        $lien = $attrs['lien'];
+    } else {
+        $lien = '';
+    }
+    $n = $phraseur->contenu['compatible'];
+    $phraseur->versions[$n]['licence'][] = ['nom' => $texte, 'url' => $lien];
 }
 
 /**
@@ -226,14 +226,14 @@ 
  * @param string $texte
  */
 function info_paquet_chemin($phraseur, $attrs, $texte) {
-	$n = $phraseur->contenu['compatible'];
-	if (isset($attrs['path'])) {
-		if (isset($attrs['type'])) {
-			$phraseur->versions[$n]['chemin'][] = ['path' => $attrs['path'], 'type' => $attrs['type']];
-		} else {
-			$phraseur->versions[$n]['chemin'][] = ['path' => $attrs['path']];
-		}
-	}
+    $n = $phraseur->contenu['compatible'];
+    if (isset($attrs['path'])) {
+        if (isset($attrs['type'])) {
+            $phraseur->versions[$n]['chemin'][] = ['path' => $attrs['path'], 'type' => $attrs['type']];
+        } else {
+            $phraseur->versions[$n]['chemin'][] = ['path' => $attrs['path']];
+        }
+    }
 }
 
 
@@ -247,24 +247,24 @@ 
  * @param string $texte
  */
 function info_paquet_auteur($phraseur, $attrs, $texte) {
-	#  echo 'auteur ', $texte;  var_dump($attrs);
-	if (isset($attrs['mail'])) {
-		if (strpos($attrs['mail'], '@')) {
-			$attrs['mail'] = str_replace('@', ' AT ', $attrs['mail']);
-		}
-		$mail = $attrs['mail'];
-	} else {
-		$mail = '';
-	}
-
-	if (isset($attrs['lien'])) {
-		$lien = $attrs['lien'];
-	} else {
-		$lien = '';
-	}
-
-	$n = $phraseur->contenu['compatible'];
-	$phraseur->versions[$n]['auteur'][] = ['nom' => $texte, 'url' => $lien, 'mail' => $mail];
+    #  echo 'auteur ', $texte;  var_dump($attrs);
+    if (isset($attrs['mail'])) {
+        if (strpos($attrs['mail'], '@')) {
+            $attrs['mail'] = str_replace('@', ' AT ', $attrs['mail']);
+        }
+        $mail = $attrs['mail'];
+    } else {
+        $mail = '';
+    }
+
+    if (isset($attrs['lien'])) {
+        $lien = $attrs['lien'];
+    } else {
+        $lien = '';
+    }
+
+    $n = $phraseur->contenu['compatible'];
+    $phraseur->versions[$n]['auteur'][] = ['nom' => $texte, 'url' => $lien, 'mail' => $mail];
 }
 
 /**
@@ -277,14 +277,14 @@ 
  */
 function info_paquet_credit($phraseur, $attrs, $texte) {
 
-	if (isset($attrs['lien'])) {
-		$lien = $attrs['lien'];
-	} else {
-		$lien = '';
-	}
+    if (isset($attrs['lien'])) {
+        $lien = $attrs['lien'];
+    } else {
+        $lien = '';
+    }
 
-	$n = $phraseur->contenu['compatible'];
-	$phraseur->versions[$n]['credit'][] = ['nom' => $texte, 'url' => $lien];
+    $n = $phraseur->contenu['compatible'];
+    $phraseur->versions[$n]['credit'][] = ['nom' => $texte, 'url' => $lien];
 }
 
 /**
@@ -296,8 +296,8 @@ 
  * @param string $texte
  */
 function info_paquet_copyright($phraseur, $attrs, $texte) {
-	$n = $phraseur->contenu['compatible'];
-	$phraseur->versions[$n]['copyright'][] = $texte;
+    $n = $phraseur->contenu['compatible'];
+    $phraseur->versions[$n]['copyright'][] = $texte;
 }
 
 /**
@@ -309,9 +309,9 @@ 
  * @param string $texte
  */
 function info_paquet_paquet($phraseur, $attrs, $texte) {
-	$n = 0;
-	$phraseur->versions[$n]['dtd'] = 'paquet';
-	$phraseur->versions[$n]['balise'] = 'paquet';
+    $n = 0;
+    $phraseur->versions[$n]['dtd'] = 'paquet';
+    $phraseur->versions[$n]['balise'] = 'paquet';
 }
 
 /**
@@ -323,8 +323,8 @@ 
  * @param string $texte
  **/
 function info_paquet_traduire($phraseur, $attrs, $texte) {
-	$n = $phraseur->contenu['compatible'];
-	$phraseur->versions[$n]['traduire'][] = $attrs;
+    $n = $phraseur->contenu['compatible'];
+    $phraseur->versions[$n]['traduire'][] = $attrs;
 }
 
 /**
@@ -336,8 +336,8 @@ 
  * @param string $texte
  */
 function info_paquet_spip($phraseur, $attrs, $texte) {
-	$n = $phraseur->contenu['compatible'];
-	$phraseur->versions[$n]['balise'] = 'spip';
+    $n = $phraseur->contenu['compatible'];
+    $phraseur->versions[$n]['balise'] = 'spip';
 }
 
 
@@ -349,8 +349,8 @@ 
  * @param string $texte
  */
 function info_paquet_pipeline($phraseur, $attrs, $texte) {
-	$n = $phraseur->contenu['compatible'];
-	$phraseur->versions[$n]['pipeline'][] = $attrs;
+    $n = $phraseur->contenu['compatible'];
+    $phraseur->versions[$n]['pipeline'][] = $attrs;
 }
 
 
@@ -363,23 +363,23 @@ 
  * @param string $texte
  */
 function info_paquet_style($phraseur, $attrs, $texte) {
-	$lien = $chemin = $type = $media = '';
-
-	include_spip('inc/utils');
-	if (tester_url_absolue($attrs['source'])) {
-		$lien = $attrs['source'];
-	} else {
-		$chemin = $attrs['source'];
-	}
-	if (isset($attrs['type'])) {
-		$type = $attrs['type'];
-	}
-	if (isset($attrs['media'])) {
-		$media = $attrs['media'];
-	}
-
-	$n = $phraseur->contenu['compatible'];
-	$phraseur->versions[$n]['style'][] = ['url' => $lien, 'path' => $chemin, 'type' => $type, 'media' => $media];
+    $lien = $chemin = $type = $media = '';
+
+    include_spip('inc/utils');
+    if (tester_url_absolue($attrs['source'])) {
+        $lien = $attrs['source'];
+    } else {
+        $chemin = $attrs['source'];
+    }
+    if (isset($attrs['type'])) {
+        $type = $attrs['type'];
+    }
+    if (isset($attrs['media'])) {
+        $media = $attrs['media'];
+    }
+
+    $n = $phraseur->contenu['compatible'];
+    $phraseur->versions[$n]['style'][] = ['url' => $lien, 'path' => $chemin, 'type' => $type, 'media' => $media];
 }
 
 
@@ -392,20 +392,20 @@ 
  * @param string $texte
  */
 function info_paquet_script($phraseur, $attrs, $texte) {
-	$lien = $chemin = $type = $media = '';
-
-	include_spip('inc/utils');
-	if (tester_url_absolue($attrs['source'])) {
-		$lien = $attrs['source'];
-	} else {
-		$chemin = $attrs['source'];
-	}
-	if (isset($attrs['type'])) {
-		$type = $attrs['type'];
-	}
-
-	$n = $phraseur->contenu['compatible'];
-	$phraseur->versions[$n]['script'][] = ['url' => $lien, 'path' => $chemin, 'type' => $type];
+    $lien = $chemin = $type = $media = '';
+
+    include_spip('inc/utils');
+    if (tester_url_absolue($attrs['source'])) {
+        $lien = $attrs['source'];
+    } else {
+        $chemin = $attrs['source'];
+    }
+    if (isset($attrs['type'])) {
+        $type = $attrs['type'];
+    }
+
+    $n = $phraseur->contenu['compatible'];
+    $phraseur->versions[$n]['script'][] = ['url' => $lien, 'path' => $chemin, 'type' => $type];
 }
 
 /**
@@ -416,6 +416,6 @@ 
  * @param string $texte
  */
 function info_paquet_genie($phraseur, $attrs, $texte) {
-	$n = $phraseur->contenu['compatible'];
-	$phraseur->versions[$n]['genie'][] = $attrs;
+    $n = $phraseur->contenu['compatible'];
+    $phraseur->versions[$n]['genie'][] = $attrs;
 }

ecrire/plugins/extraire_boutons.php

Indentation +27 added lines, -27 removed lines

@@ -11,7 +11,7 @@ 
 \***************************************************************************/
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -21,31 +21,31 @@ 
  * @return <type>
  */
 function plugins_extraire_boutons_dist($arbre) {
-	$les_boutons = null;
-	$ret = ['bouton' => [], 'onglet' => []];
-	// recuperer les boutons et onglets si necessaire
-	spip_xml_match_nodes(',^(bouton|onglet)\s,', $arbre, $les_boutons);
-	if (is_array($les_boutons) && count($les_boutons)) {
-		$ret['bouton'] = [];
-		$ret['onglet'] = [];
-		foreach ($les_boutons as $bouton => $val) {
-			$bouton = spip_xml_decompose_tag($bouton);
-			$type = reset($bouton);
-			$bouton = end($bouton);
-			if (isset($bouton['id'])) {
-				$id = $bouton['id'];
-				$val = reset($val);
-				if (is_array($val)) {
-					$ret[$type][$id]['parent'] = $bouton['parent'] ?? '';
-					$ret[$type][$id]['position'] = $bouton['position'] ?? '';
-					$ret[$type][$id]['titre'] = isset($val['titre']) ? trim(spip_xml_aplatit($val['titre'])) : '';
-					$ret[$type][$id]['icone'] = isset($val['icone']) ? trim(end($val['icone'])) : '';
-					$ret[$type][$id]['action'] = isset($val['url']) ? trim(end($val['url'])) : '';
-					$ret[$type][$id]['parametres'] = isset($val['args']) ? trim(end($val['args'])) : '';
-				}
-			}
-		}
-	}
+    $les_boutons = null;
+    $ret = ['bouton' => [], 'onglet' => []];
+    // recuperer les boutons et onglets si necessaire
+    spip_xml_match_nodes(',^(bouton|onglet)\s,', $arbre, $les_boutons);
+    if (is_array($les_boutons) && count($les_boutons)) {
+        $ret['bouton'] = [];
+        $ret['onglet'] = [];
+        foreach ($les_boutons as $bouton => $val) {
+            $bouton = spip_xml_decompose_tag($bouton);
+            $type = reset($bouton);
+            $bouton = end($bouton);
+            if (isset($bouton['id'])) {
+                $id = $bouton['id'];
+                $val = reset($val);
+                if (is_array($val)) {
+                    $ret[$type][$id]['parent'] = $bouton['parent'] ?? '';
+                    $ret[$type][$id]['position'] = $bouton['position'] ?? '';
+                    $ret[$type][$id]['titre'] = isset($val['titre']) ? trim(spip_xml_aplatit($val['titre'])) : '';
+                    $ret[$type][$id]['icone'] = isset($val['icone']) ? trim(end($val['icone'])) : '';
+                    $ret[$type][$id]['action'] = isset($val['url']) ? trim(end($val['url'])) : '';
+                    $ret[$type][$id]['parametres'] = isset($val['args']) ? trim(end($val['args'])) : '';
+                }
+            }
+        }
+    }
 
-	return $ret;
+    return $ret;
 }

ecrire/plugins/installer.php

Indentation +135 added lines, -135 removed lines

@@ -18,7 +18,7 @@ 
 
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -54,96 +54,96 @@ 
  */
 function plugins_installer_dist($plug, $action, $dir_type = '_DIR_PLUGINS') {
 
-	// Charger les informations du XML du plugin et vérification de l'existence d'une installation
-	$get_infos = charger_fonction('get_infos', 'plugins');
-	$infos = $get_infos($plug, false, constant($dir_type));
-	if (!isset($infos['install']) or !$infos['install']) {
-		return false;
-	}
-
-	// Passer en chemin absolu si possible, c'est plus efficace
-	$dir = str_replace('_DIR_', '_ROOT_', $dir_type);
-	if (!defined($dir)) {
-		$dir = $dir_type;
-	}
-	$dir = constant($dir);
-	foreach ($infos['install'] as $file) {
-		$file = $dir . $plug . '/' . trim($file);
-		if (file_exists($file)) {
-			include_once($file);
-		}
-	}
-
-	// Détermination de la table meta et du nom de la meta plugin
-	$table = 'meta';
-	if (isset($infos['meta']) and ($infos['meta'] !== 'meta')) {
-		$table = $infos['meta'];
-		// S'assurer que les metas de la table spécifique sont bien accessibles dans la globale
-		lire_metas($table);
-	}
-	$nom_meta = $infos['prefix'] . '_base_version';
-
-	// Détermination de la fonction à appeler et de ses arguments
-	$f = $infos['prefix'] . '_install';
-	if (!function_exists($f)) {
-		$f = isset($infos['schema']) ? 'spip_plugin_install' : '';
-		$arg = $infos;
-		// On passe la table et la meta pour éviter de les recalculer dans la fonction appelée
-		$arg['meta'] = $table;
-		$arg['nom_meta'] = $nom_meta;
-	} else {
-		// Ancienne méthode d'installation - TODO à supprimer à terme
-		// stupide: info deja dans le nom
-		$arg = $infos['prefix'];
-	}
-	$version = $infos['schema'] ?? '';
-
-	if (!$f) {
-		// installation sans operation particuliere
-		$infos['install_test'] = [true, ''];
-		return $infos;
-	}
-
-	// Tester si l'action demandée est nécessaire ou pas.
-	$test = $f('test', $arg, $version);
-	if ($action == 'uninstall') {
-		$test = !$test;
-	}
-	// Si deja fait, on ne fait rien et on ne dit rien
-	if ($test) {
-		return true;
-	}
-
-	// Si install et que l'on a la meta d'installation, c'est un upgrade. On le consigne dans $infos
-	// pour renvoyer le bon message en retour de la fonction.
-	if ($action == 'install' && !empty($GLOBALS[$table][$nom_meta])) {
-		$infos['upgrade'] = true;
-	}
-
-	// executer l'installation ou l'inverse
-	// et renvoyer la trace (mais il faudrait passer en AJAX plutot)
-	ob_start();
-	$f($action, $arg, $version);
-	$aff = ob_get_contents();
-	ob_end_clean();
-
-	// vider le cache des descriptions de tables a chaque (de)installation
-	$trouver_table = charger_fonction('trouver_table', 'base');
-	$trouver_table('');
-	$infos['install_test'] = [$f('test', $arg, $version), $aff];
-
-	// Si la table meta n'est pas spip_meta et qu'on est dans la première installation du plugin
-	// on force la création du fichier cache à la date du moment.
-	// On relit les metas de la table pour être sur que la globale soit à jour pour touch_meta.
-	if (
-		($table !== 'meta')
-		and ($action == 'install')
-		and empty($infos['upgrade'])
-	) {
-		touch_meta(false, $table);
-	}
-
-	return $infos;
+    // Charger les informations du XML du plugin et vérification de l'existence d'une installation
+    $get_infos = charger_fonction('get_infos', 'plugins');
+    $infos = $get_infos($plug, false, constant($dir_type));
+    if (!isset($infos['install']) or !$infos['install']) {
+        return false;
+    }
+
+    // Passer en chemin absolu si possible, c'est plus efficace
+    $dir = str_replace('_DIR_', '_ROOT_', $dir_type);
+    if (!defined($dir)) {
+        $dir = $dir_type;
+    }
+    $dir = constant($dir);
+    foreach ($infos['install'] as $file) {
+        $file = $dir . $plug . '/' . trim($file);
+        if (file_exists($file)) {
+            include_once($file);
+        }
+    }
+
+    // Détermination de la table meta et du nom de la meta plugin
+    $table = 'meta';
+    if (isset($infos['meta']) and ($infos['meta'] !== 'meta')) {
+        $table = $infos['meta'];
+        // S'assurer que les metas de la table spécifique sont bien accessibles dans la globale
+        lire_metas($table);
+    }
+    $nom_meta = $infos['prefix'] . '_base_version';
+
+    // Détermination de la fonction à appeler et de ses arguments
+    $f = $infos['prefix'] . '_install';
+    if (!function_exists($f)) {
+        $f = isset($infos['schema']) ? 'spip_plugin_install' : '';
+        $arg = $infos;
+        // On passe la table et la meta pour éviter de les recalculer dans la fonction appelée
+        $arg['meta'] = $table;
+        $arg['nom_meta'] = $nom_meta;
+    } else {
+        // Ancienne méthode d'installation - TODO à supprimer à terme
+        // stupide: info deja dans le nom
+        $arg = $infos['prefix'];
+    }
+    $version = $infos['schema'] ?? '';
+
+    if (!$f) {
+        // installation sans operation particuliere
+        $infos['install_test'] = [true, ''];
+        return $infos;
+    }
+
+    // Tester si l'action demandée est nécessaire ou pas.
+    $test = $f('test', $arg, $version);
+    if ($action == 'uninstall') {
+        $test = !$test;
+    }
+    // Si deja fait, on ne fait rien et on ne dit rien
+    if ($test) {
+        return true;
+    }
+
+    // Si install et que l'on a la meta d'installation, c'est un upgrade. On le consigne dans $infos
+    // pour renvoyer le bon message en retour de la fonction.
+    if ($action == 'install' && !empty($GLOBALS[$table][$nom_meta])) {
+        $infos['upgrade'] = true;
+    }
+
+    // executer l'installation ou l'inverse
+    // et renvoyer la trace (mais il faudrait passer en AJAX plutot)
+    ob_start();
+    $f($action, $arg, $version);
+    $aff = ob_get_contents();
+    ob_end_clean();
+
+    // vider le cache des descriptions de tables a chaque (de)installation
+    $trouver_table = charger_fonction('trouver_table', 'base');
+    $trouver_table('');
+    $infos['install_test'] = [$f('test', $arg, $version), $aff];
+
+    // Si la table meta n'est pas spip_meta et qu'on est dans la première installation du plugin
+    // on force la création du fichier cache à la date du moment.
+    // On relit les metas de la table pour être sur que la globale soit à jour pour touch_meta.
+    if (
+        ($table !== 'meta')
+        and ($action == 'install')
+        and empty($infos['upgrade'])
+    ) {
+        touch_meta(false, $table);
+    }
+
+    return $infos;
 }
 
 /**
@@ -159,25 +159,25 @@ 
  * @return bool|void
  */
 function spip_plugin_install($action, $infos, $version_cible) {
-	$nom_meta = $infos['nom_meta'];
-	$table = $infos['meta'];
-	switch ($action) {
-		case 'test':
-			return (isset($GLOBALS[$table])
-				and isset($GLOBALS[$table][$nom_meta])
-				and spip_version_compare($GLOBALS[$table][$nom_meta], $version_cible, '>='));
-			break;
-		case 'install':
-			if (function_exists($upgrade = $infos['prefix'] . '_upgrade')) {
-				$upgrade($nom_meta, $version_cible, $table);
-			}
-			break;
-		case 'uninstall':
-			if (function_exists($vider_tables = $infos['prefix'] . '_vider_tables')) {
-				$vider_tables($nom_meta, $table);
-			}
-			break;
-	}
+    $nom_meta = $infos['nom_meta'];
+    $table = $infos['meta'];
+    switch ($action) {
+        case 'test':
+            return (isset($GLOBALS[$table])
+                and isset($GLOBALS[$table][$nom_meta])
+                and spip_version_compare($GLOBALS[$table][$nom_meta], $version_cible, '>='));
+            break;
+        case 'install':
+            if (function_exists($upgrade = $infos['prefix'] . '_upgrade')) {
+                $upgrade($nom_meta, $version_cible, $table);
+            }
+            break;
+        case 'uninstall':
+            if (function_exists($vider_tables = $infos['prefix'] . '_vider_tables')) {
+                $vider_tables($nom_meta, $table);
+            }
+            break;
+    }
 }
 
 
@@ -196,29 +196,29 @@ 
  * @return array Tableau des plugins actifs
  **/
 function liste_plugin_actifs() {
-	$liste = $GLOBALS['meta']['plugin'] ?? '';
-	if (!$liste) {
-		return [];
-	}
-	if (!is_array($liste = unserialize($liste))) {
-		// compatibilite pre 1.9.2, mettre a jour la meta
-		spip_log("MAJ meta plugin vieille version : $liste", 'plugin');
-		$new = true;
-		[, $liste] = liste_plugin_valides(explode(',', $liste));
-	} else {
-		$new = false;
-		// compat au moment d'une migration depuis version anterieure
-		// si pas de dir_type, alors c'est _DIR_PLUGINS
-		foreach ($liste as $prefix => $infos) {
-			if (!isset($infos['dir_type'])) {
-				$liste[$prefix]['dir_type'] = '_DIR_PLUGINS';
-				$new = true;
-			}
-		}
-	}
-	if ($new) {
-		ecrire_meta('plugin', serialize($liste));
-	}
-
-	return $liste;
+    $liste = $GLOBALS['meta']['plugin'] ?? '';
+    if (!$liste) {
+        return [];
+    }
+    if (!is_array($liste = unserialize($liste))) {
+        // compatibilite pre 1.9.2, mettre a jour la meta
+        spip_log("MAJ meta plugin vieille version : $liste", 'plugin');
+        $new = true;
+        [, $liste] = liste_plugin_valides(explode(',', $liste));
+    } else {
+        $new = false;
+        // compat au moment d'une migration depuis version anterieure
+        // si pas de dir_type, alors c'est _DIR_PLUGINS
+        foreach ($liste as $prefix => $infos) {
+            if (!isset($infos['dir_type'])) {
+                $liste[$prefix]['dir_type'] = '_DIR_PLUGINS';
+                $new = true;
+            }
+        }
+    }
+    if ($new) {
+        ecrire_meta('plugin', serialize($liste));
+    }
+
+    return $liste;
 }

ecrire/maj/2021.php

Indentation +3 added lines, -3 removed lines

@@ -19,10 +19,10 @@
  * @package SPIP\Core\SQL\Upgrade
  **/
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 $GLOBALS['maj'][2021_02_18_00] = [
-	['sql_alter', "TABLE spip_auteurs CHANGE imessage imessage VARCHAR(3) DEFAULT '' NOT NULL" ],
-	['sql_updateq', 'spip_auteurs', ['imessage' => 'oui'], "imessage != 'non' OR imessage IS NULL" ],
+    ['sql_alter', "TABLE spip_auteurs CHANGE imessage imessage VARCHAR(3) DEFAULT '' NOT NULL" ],
+    ['sql_updateq', 'spip_auteurs', ['imessage' => 'oui'], "imessage != 'non' OR imessage IS NULL" ],
 ];

ecrire/maj/legacy/v30.php

Indentation +230 added lines, -230 removed lines

@@ -18,18 +18,18 @@ 
  * @package SPIP\Core\SQL\Upgrade
  **/
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 
 $GLOBALS['maj'][16428] = [
-	['maj_liens', 'auteur'], // creer la table liens
-	['maj_liens', 'auteur', 'article'],
-	['sql_drop_table', 'spip_auteurs_articles'],
-	['maj_liens', 'auteur', 'rubrique'],
-	['sql_drop_table', 'spip_auteurs_rubriques'],
-	['maj_liens', 'auteur', 'message'],
-	['sql_drop_table', 'spip_auteurs_messages'],
+    ['maj_liens', 'auteur'], // creer la table liens
+    ['maj_liens', 'auteur', 'article'],
+    ['sql_drop_table', 'spip_auteurs_articles'],
+    ['maj_liens', 'auteur', 'rubrique'],
+    ['sql_drop_table', 'spip_auteurs_rubriques'],
+    ['maj_liens', 'auteur', 'message'],
+    ['sql_drop_table', 'spip_auteurs_messages'],
 ];
 
 /**
@@ -55,89 +55,89 @@ 
  */
 function maj_liens($pivot, $l = '') {
 
-	@define('_LOG_FILTRE_GRAVITE', 8);
-
-	$exceptions_pluriel = ['forum' => 'forum', 'syndic' => 'syndic'];
-
-	$pivot = preg_replace(',[^\w],', '', $pivot); // securite
-	$pivots = ($exceptions_pluriel[$pivot] ?? $pivot . 's');
-	$liens = 'spip_' . $pivots . '_liens';
-	$id_pivot = 'id_' . $pivot;
-	// Creer spip_auteurs_liens
-	global $tables_auxiliaires;
-	if (!$l) {
-		include_spip('base/auxiliaires');
-		include_spip('base/create');
-		creer_ou_upgrader_table($liens, $tables_auxiliaires[$liens], false);
-	} else {
-		// Preparer
-		$l = preg_replace(',[^\w],', '', $l); // securite
-		$primary = "id_$l";
-		$objet = ($l == 'syndic' ? 'site' : $l);
-		$ls = ($exceptions_pluriel[$l] ?? $l . 's');
-		$ancienne_table = 'spip_' . $pivots . '_' . $ls;
-		$pool = 400;
-
-		$trouver_table = charger_fonction('trouver_table', 'base');
-		if (!$desc = $trouver_table($ancienne_table)) {
-			return;
-		}
-
-		// securite pour ne pas perdre de donnees
-		if (!$trouver_table($liens)) {
-			return;
-		}
-
-		$champs = $desc['field'];
-		if (isset($champs['maj'])) {
-			unset($champs['maj']);
-		}
-		if (isset($champs[$primary])) {
-			unset($champs[$primary]);
-		}
-
-		$champs = array_keys($champs);
-		// ne garder que les champs qui existent sur la table destination
-		if ($desc_cible = $trouver_table($liens)) {
-			$champs = array_intersect($champs, array_keys($desc_cible['field']));
-		}
-
-		$champs[] = "$primary as id_objet";
-		$champs[] = "'$objet' as objet";
-		$champs = implode(', ', $champs);
-
-		// Recopier les donnees
-		$sub_pool = 100;
-		while ($ids = array_map('reset', sql_allfetsel("$primary", $ancienne_table, '', '', '', "0,$sub_pool"))) {
-			$insert = [];
-			foreach ($ids as $id) {
-				$n = sql_countsel($liens, "objet='$objet' AND id_objet=" . intval($id));
-				while ($t = sql_allfetsel($champs, $ancienne_table, "$primary=" . intval($id), '', $id_pivot, "$n,$pool")) {
-					$n += is_countable($t) ? count($t) : 0;
-					// empiler en s'assurant a minima de l'unicite
-					while ($r = array_shift($t)) {
-						$insert[$r[$id_pivot] . ':' . $r['id_objet']] = $r;
-					}
-					if (count($insert) >= $sub_pool) {
-						maj_liens_insertq_multi_check($liens, $insert, $tables_auxiliaires[$liens]);
-						$insert = [];
-					}
-					// si timeout, sortir, la relance nous ramenera dans cette fonction
-					// et on verifiera/repartira de la
-					if (time() >= _TIME_OUT) {
-						return;
-					}
-				}
-				if (time() >= _TIME_OUT) {
-					return;
-				}
-			}
-			if (count($insert)) {
-				maj_liens_insertq_multi_check($liens, $insert, $tables_auxiliaires[$liens]);
-			}
-			sql_delete($ancienne_table, sql_in($primary, $ids));
-		}
-	}
+    @define('_LOG_FILTRE_GRAVITE', 8);
+
+    $exceptions_pluriel = ['forum' => 'forum', 'syndic' => 'syndic'];
+
+    $pivot = preg_replace(',[^\w],', '', $pivot); // securite
+    $pivots = ($exceptions_pluriel[$pivot] ?? $pivot . 's');
+    $liens = 'spip_' . $pivots . '_liens';
+    $id_pivot = 'id_' . $pivot;
+    // Creer spip_auteurs_liens
+    global $tables_auxiliaires;
+    if (!$l) {
+        include_spip('base/auxiliaires');
+        include_spip('base/create');
+        creer_ou_upgrader_table($liens, $tables_auxiliaires[$liens], false);
+    } else {
+        // Preparer
+        $l = preg_replace(',[^\w],', '', $l); // securite
+        $primary = "id_$l";
+        $objet = ($l == 'syndic' ? 'site' : $l);
+        $ls = ($exceptions_pluriel[$l] ?? $l . 's');
+        $ancienne_table = 'spip_' . $pivots . '_' . $ls;
+        $pool = 400;
+
+        $trouver_table = charger_fonction('trouver_table', 'base');
+        if (!$desc = $trouver_table($ancienne_table)) {
+            return;
+        }
+
+        // securite pour ne pas perdre de donnees
+        if (!$trouver_table($liens)) {
+            return;
+        }
+
+        $champs = $desc['field'];
+        if (isset($champs['maj'])) {
+            unset($champs['maj']);
+        }
+        if (isset($champs[$primary])) {
+            unset($champs[$primary]);
+        }
+
+        $champs = array_keys($champs);
+        // ne garder que les champs qui existent sur la table destination
+        if ($desc_cible = $trouver_table($liens)) {
+            $champs = array_intersect($champs, array_keys($desc_cible['field']));
+        }
+
+        $champs[] = "$primary as id_objet";
+        $champs[] = "'$objet' as objet";
+        $champs = implode(', ', $champs);
+
+        // Recopier les donnees
+        $sub_pool = 100;
+        while ($ids = array_map('reset', sql_allfetsel("$primary", $ancienne_table, '', '', '', "0,$sub_pool"))) {
+            $insert = [];
+            foreach ($ids as $id) {
+                $n = sql_countsel($liens, "objet='$objet' AND id_objet=" . intval($id));
+                while ($t = sql_allfetsel($champs, $ancienne_table, "$primary=" . intval($id), '', $id_pivot, "$n,$pool")) {
+                    $n += is_countable($t) ? count($t) : 0;
+                    // empiler en s'assurant a minima de l'unicite
+                    while ($r = array_shift($t)) {
+                        $insert[$r[$id_pivot] . ':' . $r['id_objet']] = $r;
+                    }
+                    if (count($insert) >= $sub_pool) {
+                        maj_liens_insertq_multi_check($liens, $insert, $tables_auxiliaires[$liens]);
+                        $insert = [];
+                    }
+                    // si timeout, sortir, la relance nous ramenera dans cette fonction
+                    // et on verifiera/repartira de la
+                    if (time() >= _TIME_OUT) {
+                        return;
+                    }
+                }
+                if (time() >= _TIME_OUT) {
+                    return;
+                }
+            }
+            if (count($insert)) {
+                maj_liens_insertq_multi_check($liens, $insert, $tables_auxiliaires[$liens]);
+            }
+            sql_delete($ancienne_table, sql_in($primary, $ids));
+        }
+    }
 }
 
 /**
@@ -151,86 +151,86 @@ 
  * @return void
  **/
 function maj_liens_insertq_multi_check($table, $couples, $desc = []) {
-	$n_before = sql_countsel($table);
-	sql_insertq_multi($table, $couples, $desc);
-	$n_after = sql_countsel($table);
-	if (($n_after - $n_before) == count($couples)) {
-		return;
-	}
-	// si ecart, on recommence l'insertion ligne par ligne...
-	// moins rapide mais secure : seul le couple en doublon echouera, et non toute la serie
-	foreach ($couples as $c) {
-		sql_insertq($table, $c, $desc);
-	}
+    $n_before = sql_countsel($table);
+    sql_insertq_multi($table, $couples, $desc);
+    $n_after = sql_countsel($table);
+    if (($n_after - $n_before) == count($couples)) {
+        return;
+    }
+    // si ecart, on recommence l'insertion ligne par ligne...
+    // moins rapide mais secure : seul le couple en doublon echouera, et non toute la serie
+    foreach ($couples as $c) {
+        sql_insertq($table, $c, $desc);
+    }
 }
 
 $GLOBALS['maj'][17311] = [
-	[
-		'ecrire_meta',
-		'multi_objets',
-		implode(
-			',',
-			array_diff(
-				[
-					(isset($GLOBALS['meta']['multi_rubriques']) and $GLOBALS['meta']['multi_rubriques'] == 'oui')
-						? 'spip_rubriques' : '',
-					(isset($GLOBALS['meta']['multi_articles']) and $GLOBALS['meta']['multi_articles'] == 'oui')
-						? 'spip_articles' : ''
-				],
-				['']
-			)
-		)
-	],
-	[
-		'ecrire_meta',
-		'gerer_trad_objets',
-		implode(
-			',',
-			array_diff(
-				[
-					(isset($GLOBALS['meta']['gerer_trad']) and $GLOBALS['meta']['gerer_trad'] == 'oui')
-						? 'spip_articles' : ''
-				],
-				['']
-			)
-		)
-	],
+    [
+        'ecrire_meta',
+        'multi_objets',
+        implode(
+            ',',
+            array_diff(
+                [
+                    (isset($GLOBALS['meta']['multi_rubriques']) and $GLOBALS['meta']['multi_rubriques'] == 'oui')
+                        ? 'spip_rubriques' : '',
+                    (isset($GLOBALS['meta']['multi_articles']) and $GLOBALS['meta']['multi_articles'] == 'oui')
+                        ? 'spip_articles' : ''
+                ],
+                ['']
+            )
+        )
+    ],
+    [
+        'ecrire_meta',
+        'gerer_trad_objets',
+        implode(
+            ',',
+            array_diff(
+                [
+                    (isset($GLOBALS['meta']['gerer_trad']) and $GLOBALS['meta']['gerer_trad'] == 'oui')
+                        ? 'spip_articles' : ''
+                ],
+                ['']
+            )
+        )
+    ],
 ];
 $GLOBALS['maj'][17555] = [
-	['sql_alter', "TABLE spip_resultats ADD table_objet varchar(30) DEFAULT '' NOT NULL"],
-	['sql_alter', "TABLE spip_resultats ADD serveur char(16) DEFAULT '' NOT NULL"],
+    ['sql_alter', "TABLE spip_resultats ADD table_objet varchar(30) DEFAULT '' NOT NULL"],
+    ['sql_alter', "TABLE spip_resultats ADD serveur char(16) DEFAULT '' NOT NULL"],
 ];
 
 $GLOBALS['maj'][17563] = [
-	['sql_alter', "TABLE spip_articles ADD virtuel VARCHAR(255) DEFAULT '' NOT NULL"],
-	['sql_update', 'spip_articles', ['virtuel' => 'SUBSTRING(chapo,2)', 'chapo' => "''"], "chapo LIKE '=_%'"],
+    ['sql_alter', "TABLE spip_articles ADD virtuel VARCHAR(255) DEFAULT '' NOT NULL"],
+    ['sql_update', 'spip_articles', ['virtuel' => 'SUBSTRING(chapo,2)', 'chapo' => "''"], "chapo LIKE '=_%'"],
 ];
 
 $GLOBALS['maj'][17577] = [
-	['maj_tables', ['spip_jobs', 'spip_jobs_liens']],
+    ['maj_tables', ['spip_jobs', 'spip_jobs_liens']],
 ];
 
 $GLOBALS['maj'][17743] = [
-	['sql_update', 'spip_auteurs', ['prefs' => 'bio', 'bio' => "''"], "statut='nouveau' AND bio<>''"],
+    ['sql_update', 'spip_auteurs', ['prefs' => 'bio', 'bio' => "''"], "statut='nouveau' AND bio<>''"],
 ];
 
 $GLOBALS['maj'][18219] = [
-	['sql_alter', 'TABLE spip_rubriques DROP id_import'],
-	['sql_alter', 'TABLE spip_rubriques DROP export'],
+    ['sql_alter', 'TABLE spip_rubriques DROP id_import'],
+    ['sql_alter', 'TABLE spip_rubriques DROP export'],
 ];
 
 $GLOBALS['maj'][18310] = [
-	['sql_alter', "TABLE spip_auteurs_liens CHANGE vu vu VARCHAR(6) DEFAULT 'non' NOT NULL"],
+    ['sql_alter', "TABLE spip_auteurs_liens CHANGE vu vu VARCHAR(6) DEFAULT 'non' NOT NULL"],
 ];
 
 $GLOBALS['maj'][18597] = [
-	['sql_alter', "TABLE spip_rubriques ADD profondeur smallint(5) DEFAULT '0' NOT NULL"],
-	['maj_propager_les_secteurs'],
+    ['sql_alter', "TABLE spip_rubriques ADD profondeur smallint(5) DEFAULT '0' NOT NULL"],
+    ['maj_propager_les_secteurs'],
 ];
 
 $GLOBALS['maj'][18955] = [
-	['sql_alter', 'TABLE spip_auteurs_liens ADD INDEX id_objet (id_objet)'],
-	['sql_alter', 'TABLE spip_auteurs_liens ADD INDEX objet (objet)'],
+    ['sql_alter', 'TABLE spip_auteurs_liens ADD INDEX id_objet (id_objet)'],
+    ['sql_alter', 'TABLE spip_auteurs_liens ADD INDEX objet (objet)'],
 ];
 
 /**
@@ -239,8 +239,8 @@ 
  * @uses propager_les_secteurs()
  **/
 function maj_propager_les_secteurs() {
-	include_spip('inc/rubriques');
-	propager_les_secteurs();
+    include_spip('inc/rubriques');
+    propager_les_secteurs();
 }
 
 /**
@@ -253,87 +253,87 @@ 
 function maj_collation_sqlite() {
 
 
-	include_spip('base/dump');
-	$tables = base_lister_toutes_tables();
-
-	// rien a faire si base non sqlite
-	if (strncmp($GLOBALS['connexions'][0]['type'], 'sqlite', 6) !== 0) {
-		return;
-	}
-
-	$trouver_table = charger_fonction('trouver_table', 'base');
-	// forcer le vidage de cache
-	$trouver_table('');
-
-	// cas particulier spip_auteurs : retablir le collate binary sur le login
-	$desc = $trouver_table('spip_auteurs');
-	spip_log('spip_auteurs : ' . var_export($desc['field'], true), 'maj.' . _LOG_INFO_IMPORTANTE);
-	if (stripos($desc['field']['login'], 'BINARY') === false) {
-		spip_log('Retablir champ login BINARY sur table spip_auteurs', 'maj');
-		sql_alter('table spip_auteurs change login login VARCHAR(255) BINARY');
-		$trouver_table('');
-		$new_desc = $trouver_table('spip_auteurs');
-		spip_log('Apres conversion spip_auteurs : ' . var_export($new_desc['field'], true), 'maj.' . _LOG_INFO_IMPORTANTE);
-	}
-
-	foreach ($tables as $table) {
-		if (time() >= _TIME_OUT) {
-			return;
-		}
-		if ($desc = $trouver_table($table)) {
-			$desc_collate = _sqlite_remplacements_definitions_table($desc['field']);
-			if ($d = array_diff($desc['field'], $desc_collate)) {
-				spip_log("Table $table COLLATE incorrects", 'maj');
-
-				// cas particulier spip_urls :
-				// supprimer les doublons avant conversion sinon echec (on garde les urls les plus recentes)
-				if ($table == 'spip_urls') {
-					// par date DESC pour conserver les urls les plus recentes
-					$data = sql_allfetsel('*', 'spip_urls', '', '', 'date DESC');
-					$urls = [];
-					foreach ($data as $d) {
-						$key = $d['id_parent'] . '::' . strtolower($d['url']);
-						if (!isset($urls[$key])) {
-							$urls[$key] = true;
-						} else {
-							spip_log(
-								'Suppression doublon dans spip_urls avant conversion : ' . serialize($d),
-								'maj.' . _LOG_INFO_IMPORTANTE
-							);
-							sql_delete('spip_urls', 'id_parent=' . sql_quote($d['id_parent']) . ' AND url=' . sql_quote($d['url']));
-						}
-					}
-				}
-				foreach ($desc['field'] as $field => $type) {
-					if ($desc['field'][$field] !== $desc_collate[$field]) {
-						spip_log("Conversion COLLATE table $table", 'maj.' . _LOG_INFO_IMPORTANTE);
-						sql_alter("table $table change $field $field " . $desc_collate[$field]);
-						$trouver_table('');
-						$new_desc = $trouver_table($table);
-						spip_log(
-							"Apres conversion $table : " . var_export($new_desc['field'], true),
-							'maj.' . _LOG_INFO_IMPORTANTE
-						);
-						continue 2; // inutile de continuer pour cette table : un seul alter remet tout a jour en sqlite
-					}
-				}
-			}
-		}
-	}
-
-	// forcer le vidage de cache
-	$trouver_table('');
+    include_spip('base/dump');
+    $tables = base_lister_toutes_tables();
+
+    // rien a faire si base non sqlite
+    if (strncmp($GLOBALS['connexions'][0]['type'], 'sqlite', 6) !== 0) {
+        return;
+    }
+
+    $trouver_table = charger_fonction('trouver_table', 'base');
+    // forcer le vidage de cache
+    $trouver_table('');
+
+    // cas particulier spip_auteurs : retablir le collate binary sur le login
+    $desc = $trouver_table('spip_auteurs');
+    spip_log('spip_auteurs : ' . var_export($desc['field'], true), 'maj.' . _LOG_INFO_IMPORTANTE);
+    if (stripos($desc['field']['login'], 'BINARY') === false) {
+        spip_log('Retablir champ login BINARY sur table spip_auteurs', 'maj');
+        sql_alter('table spip_auteurs change login login VARCHAR(255) BINARY');
+        $trouver_table('');
+        $new_desc = $trouver_table('spip_auteurs');
+        spip_log('Apres conversion spip_auteurs : ' . var_export($new_desc['field'], true), 'maj.' . _LOG_INFO_IMPORTANTE);
+    }
+
+    foreach ($tables as $table) {
+        if (time() >= _TIME_OUT) {
+            return;
+        }
+        if ($desc = $trouver_table($table)) {
+            $desc_collate = _sqlite_remplacements_definitions_table($desc['field']);
+            if ($d = array_diff($desc['field'], $desc_collate)) {
+                spip_log("Table $table COLLATE incorrects", 'maj');
+
+                // cas particulier spip_urls :
+                // supprimer les doublons avant conversion sinon echec (on garde les urls les plus recentes)
+                if ($table == 'spip_urls') {
+                    // par date DESC pour conserver les urls les plus recentes
+                    $data = sql_allfetsel('*', 'spip_urls', '', '', 'date DESC');
+                    $urls = [];
+                    foreach ($data as $d) {
+                        $key = $d['id_parent'] . '::' . strtolower($d['url']);
+                        if (!isset($urls[$key])) {
+                            $urls[$key] = true;
+                        } else {
+                            spip_log(
+                                'Suppression doublon dans spip_urls avant conversion : ' . serialize($d),
+                                'maj.' . _LOG_INFO_IMPORTANTE
+                            );
+                            sql_delete('spip_urls', 'id_parent=' . sql_quote($d['id_parent']) . ' AND url=' . sql_quote($d['url']));
+                        }
+                    }
+                }
+                foreach ($desc['field'] as $field => $type) {
+                    if ($desc['field'][$field] !== $desc_collate[$field]) {
+                        spip_log("Conversion COLLATE table $table", 'maj.' . _LOG_INFO_IMPORTANTE);
+                        sql_alter("table $table change $field $field " . $desc_collate[$field]);
+                        $trouver_table('');
+                        $new_desc = $trouver_table($table);
+                        spip_log(
+                            "Apres conversion $table : " . var_export($new_desc['field'], true),
+                            'maj.' . _LOG_INFO_IMPORTANTE
+                        );
+                        continue 2; // inutile de continuer pour cette table : un seul alter remet tout a jour en sqlite
+                    }
+                }
+            }
+        }
+    }
+
+    // forcer le vidage de cache
+    $trouver_table('');
 }
 
 
 $GLOBALS['maj'][19236] = [
-	['sql_updateq', 'spip_meta', ['impt' => 'oui'], "nom='version_installee'"], // version base principale
-	['sql_updateq', 'spip_meta', ['impt' => 'oui'], "nom LIKE '%_base_version'"],  // version base plugins
-	['maj_collation_sqlite'],
+    ['sql_updateq', 'spip_meta', ['impt' => 'oui'], "nom='version_installee'"], // version base principale
+    ['sql_updateq', 'spip_meta', ['impt' => 'oui'], "nom LIKE '%_base_version'"],  // version base plugins
+    ['maj_collation_sqlite'],
 ];
 
 $GLOBALS['maj'][19268] = [
-	['supprimer_toutes_sessions'],
+    ['supprimer_toutes_sessions'],
 ];
 
 /**
@@ -342,13 +342,13 @@ 
  * Obligera tous les auteurs à se reconnecter !
  **/
 function supprimer_toutes_sessions() {
-	spip_log('supprimer sessions auteur');
-	if ($dir = opendir(_DIR_SESSIONS)) {
-		while (($f = readdir($dir)) !== false) {
-			spip_unlink(_DIR_SESSIONS . $f);
-			if (time() >= _TIME_OUT) {
-				return;
-			}
-		}
-	}
+    spip_log('supprimer sessions auteur');
+    if ($dir = opendir(_DIR_SESSIONS)) {
+        while (($f = readdir($dir)) !== false) {
+            spip_unlink(_DIR_SESSIONS . $f);
+            if (time() >= _TIME_OUT) {
+                return;
+            }
+        }
+    }
 }

ecrire/action/editer_auteur.php

Indentation +279 added lines, -279 removed lines

@@ -17,7 +17,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -37,41 +37,41 @@ 
  */
 function action_editer_auteur_dist($arg = null) {
 
-	if (is_null($arg)) {
-		$securiser_action = charger_fonction('securiser_action', 'inc');
-		$arg = $securiser_action();
-	}
-
-
-	// si id_auteur n'est pas un nombre, c'est une creation
-	if (!$id_auteur = intval($arg)) {
-		if (($id_auteur = auteur_inserer()) > 0) {
-			# cf. GROS HACK
-			# recuperer l'eventuel logo charge avant la creation
-			# ils ont un id = 0-id_auteur de la session
-			$id_hack = 0 - $GLOBALS['visiteur_session']['id_auteur'];
-			$chercher_logo = charger_fonction('chercher_logo', 'inc');
-			foreach (['on', 'off'] as $type) {
-				if ($logo = $chercher_logo($id_hack, 'id_auteur', $type)) {
-					if ($logo = reset($logo)) {
-						rename($logo, str_replace($id_hack, $id_auteur, $logo));
-					}
-				}
-			}
-		}
-	}
-
-	// Enregistre l'envoi dans la BD
-	$err = '';
-	if ($id_auteur > 0) {
-		$err = auteur_modifier($id_auteur);
-	}
-
-	if ($err) {
-		spip_log("echec editeur auteur: $err", _LOG_ERREUR);
-	}
-
-	return [$id_auteur, $err];
+    if (is_null($arg)) {
+        $securiser_action = charger_fonction('securiser_action', 'inc');
+        $arg = $securiser_action();
+    }
+
+
+    // si id_auteur n'est pas un nombre, c'est une creation
+    if (!$id_auteur = intval($arg)) {
+        if (($id_auteur = auteur_inserer()) > 0) {
+            # cf. GROS HACK
+            # recuperer l'eventuel logo charge avant la creation
+            # ils ont un id = 0-id_auteur de la session
+            $id_hack = 0 - $GLOBALS['visiteur_session']['id_auteur'];
+            $chercher_logo = charger_fonction('chercher_logo', 'inc');
+            foreach (['on', 'off'] as $type) {
+                if ($logo = $chercher_logo($id_hack, 'id_auteur', $type)) {
+                    if ($logo = reset($logo)) {
+                        rename($logo, str_replace($id_hack, $id_auteur, $logo));
+                    }
+                }
+            }
+        }
+    }
+
+    // Enregistre l'envoi dans la BD
+    $err = '';
+    if ($id_auteur > 0) {
+        $err = auteur_modifier($id_auteur);
+    }
+
+    if ($err) {
+        spip_log("echec editeur auteur: $err", _LOG_ERREUR);
+    }
+
+    return [$id_auteur, $err];
 }
 
 /**
@@ -88,44 +88,44 @@ 
  */
 function auteur_inserer($source = null, $set = null) {
 
-	// Ce qu'on va demander comme modifications
-	$champs = [];
-	$champs['source'] = $source ?: 'spip';
-
-	$champs['login'] = '';
-	$champs['statut'] = '5poubelle';  // inutilisable tant qu'il n'a pas ete renseigne et institue
-	$champs['webmestre'] = 'non';
-	if (empty($champs['imessage'])) {
-		$champs['imessage'] = 'oui';
-	}
-
-	if ($set) {
-		$champs = array_merge($champs, $set);
-	}
-
-	// Envoyer aux plugins
-	$champs = pipeline(
-		'pre_insertion',
-		[
-			'args' => [
-				'table' => 'spip_auteurs',
-			],
-			'data' => $champs
-		]
-	);
-	$id_auteur = sql_insertq('spip_auteurs', $champs);
-	pipeline(
-		'post_insertion',
-		[
-			'args' => [
-				'table' => 'spip_auteurs',
-				'id_objet' => $id_auteur
-			],
-			'data' => $champs
-		]
-	);
-
-	return $id_auteur;
+    // Ce qu'on va demander comme modifications
+    $champs = [];
+    $champs['source'] = $source ?: 'spip';
+
+    $champs['login'] = '';
+    $champs['statut'] = '5poubelle';  // inutilisable tant qu'il n'a pas ete renseigne et institue
+    $champs['webmestre'] = 'non';
+    if (empty($champs['imessage'])) {
+        $champs['imessage'] = 'oui';
+    }
+
+    if ($set) {
+        $champs = array_merge($champs, $set);
+    }
+
+    // Envoyer aux plugins
+    $champs = pipeline(
+        'pre_insertion',
+        [
+            'args' => [
+                'table' => 'spip_auteurs',
+            ],
+            'data' => $champs
+        ]
+    );
+    $id_auteur = sql_insertq('spip_auteurs', $champs);
+    pipeline(
+        'post_insertion',
+        [
+            'args' => [
+                'table' => 'spip_auteurs',
+                'id_objet' => $id_auteur
+            ],
+            'data' => $champs
+        ]
+    );
+
+    return $id_auteur;
 }
 
 
@@ -150,70 +150,70 @@ 
  */
 function auteur_modifier($id_auteur, $set = null, $force_update = false) {
 
-	include_spip('inc/modifier');
-	include_spip('inc/filtres');
-	$c = collecter_requests(
-	// white list
-		objet_info('auteur', 'champs_editables'),
-		// black list
-		$force_update ? [] : ['webmestre', 'pass', 'login'],
-		// donnees eventuellement fournies
-		$set
-	);
-
-	if (
-		$err = objet_modifier_champs(
-			'auteur',
-			$id_auteur,
-			[
-			'data' => $set,
-			'nonvide' => ['nom' => _T('ecrire:item_nouvel_auteur')]
-			],
-			$c
-		)
-	) {
-		return $err;
-	}
-	$session = $c;
-
-	$err = '';
-	if (!$force_update) {
-		// Modification de statut, changement de rubrique ?
-		$c = collecter_requests(
-		// white list
-			[
-				'statut',
-				'new_login',
-				'new_pass',
-				'login',
-				'pass',
-				'webmestre',
-				'restreintes',
-				'id_parent'
-			],
-			// black list
-			[],
-			// donnees eventuellement fournies
-			$set
-		);
-		if (isset($c['new_login']) and !isset($c['login'])) {
-			$c['login'] = $c['new_login'];
-		}
-		if (isset($c['new_pass']) and !isset($c['pass'])) {
-			$c['pass'] = $c['new_pass'];
-		}
-		$err = auteur_instituer($id_auteur, $c);
-		$session = array_merge($session, $c);
-	}
-
-	// .. mettre a jour les sessions de cet auteur
-	include_spip('inc/session');
-	$session['id_auteur'] = $id_auteur;
-	unset($session['new_login']);
-	unset($session['new_pass']);
-	actualiser_sessions($session);
-
-	return $err;
+    include_spip('inc/modifier');
+    include_spip('inc/filtres');
+    $c = collecter_requests(
+    // white list
+        objet_info('auteur', 'champs_editables'),
+        // black list
+        $force_update ? [] : ['webmestre', 'pass', 'login'],
+        // donnees eventuellement fournies
+        $set
+    );
+
+    if (
+        $err = objet_modifier_champs(
+            'auteur',
+            $id_auteur,
+            [
+            'data' => $set,
+            'nonvide' => ['nom' => _T('ecrire:item_nouvel_auteur')]
+            ],
+            $c
+        )
+    ) {
+        return $err;
+    }
+    $session = $c;
+
+    $err = '';
+    if (!$force_update) {
+        // Modification de statut, changement de rubrique ?
+        $c = collecter_requests(
+        // white list
+            [
+                'statut',
+                'new_login',
+                'new_pass',
+                'login',
+                'pass',
+                'webmestre',
+                'restreintes',
+                'id_parent'
+            ],
+            // black list
+            [],
+            // donnees eventuellement fournies
+            $set
+        );
+        if (isset($c['new_login']) and !isset($c['login'])) {
+            $c['login'] = $c['new_login'];
+        }
+        if (isset($c['new_pass']) and !isset($c['pass'])) {
+            $c['pass'] = $c['new_pass'];
+        }
+        $err = auteur_instituer($id_auteur, $c);
+        $session = array_merge($session, $c);
+    }
+
+    // .. mettre a jour les sessions de cet auteur
+    include_spip('inc/session');
+    $session['id_auteur'] = $id_auteur;
+    unset($session['new_login']);
+    unset($session['new_pass']);
+    actualiser_sessions($session);
+
+    return $err;
 }
 
 /**
@@ -234,9 +234,9 @@ 
  * @return string
  */
 function auteur_associer($id_auteur, $objets, $qualif = null) {
-	include_spip('action/editer_liens');
+    include_spip('action/editer_liens');
 
-	return objet_associer(['auteur' => $id_auteur], $objets, $qualif);
+    return objet_associer(['auteur' => $id_auteur], $objets, $qualif);
 }
 
 /**
@@ -255,9 +255,9 @@ 
  * @return string
  */
 function auteur_dissocier($id_auteur, $objets) {
-	include_spip('action/editer_liens');
+    include_spip('action/editer_liens');
 
-	return objet_dissocier(['auteur' => $id_auteur], $objets);
+    return objet_dissocier(['auteur' => $id_auteur], $objets);
 }
 
 /**
@@ -278,9 +278,9 @@ 
  * @return bool|int
  */
 function auteur_qualifier($id_auteur, $objets, $qualif) {
-	include_spip('action/editer_liens');
+    include_spip('action/editer_liens');
 
-	return objet_qualifier_liens(['auteur' => $id_auteur], $objets, $qualif);
+    return objet_qualifier_liens(['auteur' => $id_auteur], $objets, $qualif);
 }
 
 
@@ -299,139 +299,139 @@ 
  * @return bool|string
  */
 function auteur_instituer($id_auteur, $c, $force_webmestre = false) {
-	if (!$id_auteur = intval($id_auteur)) {
-		return false;
-	}
-	$erreurs = []; // contiendra les differentes erreurs a traduire par _T()
-	$champs = [];
-
-	// les memoriser pour les faire passer dans le pipeline pre_edition
-	if (isset($c['login']) and strlen($c['login'])) {
-		$champs['login'] = $c['login'];
-	}
-	if (isset($c['pass']) and strlen($c['pass'])) {
-		$champs['pass'] = $c['pass'];
-	}
-
-	$statut = $statut_ancien = sql_getfetsel('statut', 'spip_auteurs', 'id_auteur=' . intval($id_auteur));
-
-	if (
-		isset($c['statut'])
-		and (autoriser('modifier', 'auteur', $id_auteur, null, ['statut' => $c['statut']]))
-	) {
-		$statut = $champs['statut'] = $c['statut'];
-	}
-
-	// Restreindre avant de declarer l'auteur
-	// (section critique sur les droits)
-	if (isset($c['id_parent']) and $c['id_parent']) {
-		if (is_array($c['restreintes'])) {
-			$c['restreintes'][] = $c['id_parent'];
-		} else {
-			$c['restreintes'] = [$c['id_parent']];
-		}
-	}
-
-	if (
-		isset($c['webmestre'])
-		and ($force_webmestre or autoriser('modifier', 'auteur', $id_auteur, null, ['webmestre' => '?']))
-	) {
-		$champs['webmestre'] = $c['webmestre'] == 'oui' ? 'oui' : 'non';
-	}
-
-	// si statut change et n'est pas 0minirezo, on force webmestre a non
-	if (isset($c['statut']) and $c['statut'] !== '0minirezo') {
-		$champs['webmestre'] = $c['webmestre'] = 'non';
-	}
-
-	// Envoyer aux plugins
-	$champs = pipeline(
-		'pre_edition',
-		[
-			'args' => [
-				'table' => 'spip_auteurs',
-				'id_objet' => $id_auteur,
-				'action' => 'instituer',
-				'statut_ancien' => $statut_ancien,
-			],
-			'data' => $champs
-		]
-	);
-
-	if (
-		isset($c['restreintes']) and is_array($c['restreintes'])
-		and autoriser('modifier', 'auteur', $id_auteur, null, ['restreint' => $c['restreintes']])
-	) {
-		$rubriques = array_map('intval', $c['restreintes']);
-		$rubriques = array_unique($rubriques);
-		$rubriques = array_diff($rubriques, [0]);
-		auteur_dissocier($id_auteur, ['rubrique' => '*']);
-		auteur_associer($id_auteur, ['rubrique' => $rubriques]);
-	}
-
-	$flag_ecrire_acces = false;
-	// commencer par traiter les cas particuliers des logins et pass
-	// avant les autres ecritures en base
-	if (isset($champs['login']) or isset($champs['pass'])) {
-		$auth_methode = sql_getfetsel('source', 'spip_auteurs', 'id_auteur=' . intval($id_auteur));
-		include_spip('inc/auth');
-		if (isset($champs['login']) and strlen($champs['login'])) {
-			if (!auth_modifier_login($auth_methode, $champs['login'], $id_auteur)) {
-				$erreurs[] = 'ecrire:impossible_modifier_login_auteur';
-			}
-		}
-		if (isset($champs['pass']) and strlen($champs['pass'])) {
-			$champs['login'] = sql_getfetsel('login', 'spip_auteurs', 'id_auteur=' . intval($id_auteur));
-			if (!auth_modifier_pass($auth_methode, $champs['login'], $champs['pass'], $id_auteur)) {
-				$erreurs[] = 'ecrire:impossible_modifier_pass_auteur';
-			}
-		}
-		unset($champs['login']);
-		unset($champs['pass']);
-		$flag_ecrire_acces = true;
-	}
-
-	if (!(is_countable($champs) ? count($champs) : 0)) {
-		return implode(' ', array_map('_T', $erreurs));
-	}
-	sql_updateq('spip_auteurs', $champs, 'id_auteur=' . $id_auteur);
-
-	// .. mettre a jour les fichiers .htpasswd et .htpasswd-admin
-	if (
-		$flag_ecrire_acces
-		or isset($champs['statut'])
-	) {
-		include_spip('inc/acces');
-		ecrire_acces();
-	}
-
-	// Invalider les caches
-	include_spip('inc/invalideur');
-	suivre_invalideur("id='auteur/$id_auteur'");
-
-	// Pipeline
-	pipeline(
-		'post_edition',
-		[
-			'args' => [
-				'table' => 'spip_auteurs',
-				'id_objet' => $id_auteur,
-				'action' => 'instituer',
-				'statut_ancien' => $statut_ancien,
-			],
-			'data' => $champs
-		]
-	);
-
-
-	// Notifications
-	if ($notifications = charger_fonction('notifications', 'inc')) {
-		$notifications(
-			'instituerauteur',
-			$id_auteur,
-			['statut' => $statut, 'statut_ancien' => $statut_ancien]
-		);
-	}
-
-	return implode(' ', array_map('_T', $erreurs));
+    if (!$id_auteur = intval($id_auteur)) {
+        return false;
+    }
+    $erreurs = []; // contiendra les differentes erreurs a traduire par _T()
+    $champs = [];
+
+    // les memoriser pour les faire passer dans le pipeline pre_edition
+    if (isset($c['login']) and strlen($c['login'])) {
+        $champs['login'] = $c['login'];
+    }
+    if (isset($c['pass']) and strlen($c['pass'])) {
+        $champs['pass'] = $c['pass'];
+    }
+
+    $statut = $statut_ancien = sql_getfetsel('statut', 'spip_auteurs', 'id_auteur=' . intval($id_auteur));
+
+    if (
+        isset($c['statut'])
+        and (autoriser('modifier', 'auteur', $id_auteur, null, ['statut' => $c['statut']]))
+    ) {
+        $statut = $champs['statut'] = $c['statut'];
+    }
+
+    // Restreindre avant de declarer l'auteur
+    // (section critique sur les droits)
+    if (isset($c['id_parent']) and $c['id_parent']) {
+        if (is_array($c['restreintes'])) {
+            $c['restreintes'][] = $c['id_parent'];
+        } else {
+            $c['restreintes'] = [$c['id_parent']];
+        }
+    }
+
+    if (
+        isset($c['webmestre'])
+        and ($force_webmestre or autoriser('modifier', 'auteur', $id_auteur, null, ['webmestre' => '?']))
+    ) {
+        $champs['webmestre'] = $c['webmestre'] == 'oui' ? 'oui' : 'non';
+    }
+
+    // si statut change et n'est pas 0minirezo, on force webmestre a non
+    if (isset($c['statut']) and $c['statut'] !== '0minirezo') {
+        $champs['webmestre'] = $c['webmestre'] = 'non';
+    }
+
+    // Envoyer aux plugins
+    $champs = pipeline(
+        'pre_edition',
+        [
+            'args' => [
+                'table' => 'spip_auteurs',
+                'id_objet' => $id_auteur,
+                'action' => 'instituer',
+                'statut_ancien' => $statut_ancien,
+            ],
+            'data' => $champs
+        ]
+    );
+
+    if (
+        isset($c['restreintes']) and is_array($c['restreintes'])
+        and autoriser('modifier', 'auteur', $id_auteur, null, ['restreint' => $c['restreintes']])
+    ) {
+        $rubriques = array_map('intval', $c['restreintes']);
+        $rubriques = array_unique($rubriques);
+        $rubriques = array_diff($rubriques, [0]);
+        auteur_dissocier($id_auteur, ['rubrique' => '*']);
+        auteur_associer($id_auteur, ['rubrique' => $rubriques]);
+    }
+
+    $flag_ecrire_acces = false;
+    // commencer par traiter les cas particuliers des logins et pass
+    // avant les autres ecritures en base
+    if (isset($champs['login']) or isset($champs['pass'])) {
+        $auth_methode = sql_getfetsel('source', 'spip_auteurs', 'id_auteur=' . intval($id_auteur));
+        include_spip('inc/auth');
+        if (isset($champs['login']) and strlen($champs['login'])) {
+            if (!auth_modifier_login($auth_methode, $champs['login'], $id_auteur)) {
+                $erreurs[] = 'ecrire:impossible_modifier_login_auteur';
+            }
+        }
+        if (isset($champs['pass']) and strlen($champs['pass'])) {
+            $champs['login'] = sql_getfetsel('login', 'spip_auteurs', 'id_auteur=' . intval($id_auteur));
+            if (!auth_modifier_pass($auth_methode, $champs['login'], $champs['pass'], $id_auteur)) {
+                $erreurs[] = 'ecrire:impossible_modifier_pass_auteur';
+            }
+        }
+        unset($champs['login']);
+        unset($champs['pass']);
+        $flag_ecrire_acces = true;
+    }
+
+    if (!(is_countable($champs) ? count($champs) : 0)) {
+        return implode(' ', array_map('_T', $erreurs));
+    }
+    sql_updateq('spip_auteurs', $champs, 'id_auteur=' . $id_auteur);
+
+    // .. mettre a jour les fichiers .htpasswd et .htpasswd-admin
+    if (
+        $flag_ecrire_acces
+        or isset($champs['statut'])
+    ) {
+        include_spip('inc/acces');
+        ecrire_acces();
+    }
+
+    // Invalider les caches
+    include_spip('inc/invalideur');
+    suivre_invalideur("id='auteur/$id_auteur'");
+
+    // Pipeline
+    pipeline(
+        'post_edition',
+        [
+            'args' => [
+                'table' => 'spip_auteurs',
+                'id_objet' => $id_auteur,
+                'action' => 'instituer',
+                'statut_ancien' => $statut_ancien,
+            ],
+            'data' => $champs
+        ]
+    );
+
+
+    // Notifications
+    if ($notifications = charger_fonction('notifications', 'inc')) {
+        $notifications(
+            'instituerauteur',
+            $id_auteur,
+            ['statut' => $statut, 'statut_ancien' => $statut_ancien]
+        );
+    }
+
+    return implode(' ', array_map('_T', $erreurs));
 }

ecrire/action/instituer_objet.php

Indentation +19 added lines, -19 removed lines

@@ -17,7 +17,7 @@ 
  */
 
 if (!defined('_ECRIRE_INC_VERSION')) {
-	return;
+    return;
 }
 
 /**
@@ -29,24 +29,24 @@ 
  */
 function action_instituer_objet_dist($arg = null) {
 
-	if (is_null($arg)) {
-		$securiser_action = charger_fonction('securiser_action', 'inc');
-		$arg = $securiser_action();
-	}
+    if (is_null($arg)) {
+        $securiser_action = charger_fonction('securiser_action', 'inc');
+        $arg = $securiser_action();
+    }
 
-	[$objet, $id_objet, $statut] = preg_split('/\W/', $arg);
-	if (!$statut) {
-		$statut = _request('statut_nouv');
-	} // cas POST
-	if (!$statut) {
-		return;
-	} // impossible mais sait-on jamais
+    [$objet, $id_objet, $statut] = preg_split('/\W/', $arg);
+    if (!$statut) {
+        $statut = _request('statut_nouv');
+    } // cas POST
+    if (!$statut) {
+        return;
+    } // impossible mais sait-on jamais
 
-	if (
-		$id_objet = intval($id_objet)
-		and autoriser('instituer', $objet, $id_objet, '', ['statut' => $statut])
-	) {
-		include_spip('action/editer_objet');
-		objet_modifier($objet, $id_objet, ['statut' => $statut]);
-	}
+    if (
+        $id_objet = intval($id_objet)
+        and autoriser('instituer', $objet, $id_objet, '', ['statut' => $statut])
+    ) {
+        include_spip('action/editer_objet');
+        objet_modifier($objet, $id_objet, ['statut' => $statut]);
+    }
 }