From 020f2c65d5be72100da7091412e50a0049599a4c Mon Sep 17 00:00:00 2001
From: "Committo,Ergo:sum" <esj@rezo.net>
Date: Thu, 3 Sep 2009 14:55:18 +0000
Subject: [PATCH] =?UTF-8?q?La=20mention=20que=20la=20variable=20{{{$contex?=
=?UTF-8?q?te=5Finclus}}}=20=C3=A9tait=20provisoirement=20globale=20pour?=
=?UTF-8?q?=20le=20d=C3=A9busqueur=20n'a=20que=20trop=20dur=C3=A9.=20Cette?=
=?UTF-8?q?=20variable=20globale=20n'a=20aucun=20int=C3=A9r=C3=AAt=20dans?=
=?UTF-8?q?=20le=20noyau,=20on=20supprimer=20donc=20cette=20d=C3=A9clarati?=
=?UTF-8?q?on=20mais=20aussi=20la=20production=20de=20cette=20variable=20i?=
=?UTF-8?q?nterm=C3=A9diaire=20qui=20alourdit=20le=20code=20sans=20raison.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Par ailleurs, à supposer qu'il soit toujours utile de contrôler l'index {{{fond}}} d'un contexte d'inclusion, ce n'est pas la peine de le faire dans {{{inclure_balise_dynamique}}} puisque {{{inclure_page}}} s'en charge.
---
ecrire/public/assembler.php | 8 +-------
ecrire/public/compiler.php | 10 ++--------
2 files changed, 3 insertions(+), 15 deletions(-)
diff --git a/ecrire/public/assembler.php b/ecrire/public/assembler.php
index 678e34687c..fe0cc6ca23 100644
--- a/ecrire/public/assembler.php
+++ b/ecrire/public/assembler.php
@@ -217,8 +217,8 @@ function inclure_page($fond, $contexte, $connect='') {
// enlever le fond de contexte inclus car sinon il prend la main
// dans les sous inclusions -> boucle infinie d'inclusion identique
+ // (cette precaution n'est probablement plus utile)
unset($contexte['fond']);
- // mais le donner pour le calcul du cache
$page = $fond;
$cacher = charger_fonction('cacher', 'public');
// Les quatre derniers parametres sont modifies par la fonction:
@@ -248,7 +248,6 @@ function inclure_page($fond, $contexte, $connect='') {
// http://doc.spip.org/@inclure_balise_dynamique
function inclure_balise_dynamique($texte, $echo=true, $ligne=0) {
- global $contexte_inclus; # provisoire : c'est pour le debuggueur
if (is_array($texte)) {
@@ -258,15 +257,10 @@ function inclure_balise_dynamique($texte, $echo=true, $ligne=0) {
$d = isset($GLOBALS['delais']) ? $GLOBALS['delais'] : NULL;
$GLOBALS['delais'] = $delainc;
- // les balises dynamiques passent toujours leur $fond
- // si un 'fond' est present dans le contexte il vient d'autre part (de la bdd par exemple:p)
- // et c'est le crash assure
- $contexte_inclus['fond'] = $fond;
$page = recuperer_fond($fond,$contexte_inclus,array('trim'=>false, 'raw' => true));
$texte = $page['texte'];
- // attention $contexte_inclus a pu changer pendant l'eval ci dessus
$GLOBALS['delais'] = $d;
// Faire remonter les entetes
if (is_array($page['entetes'])) {
diff --git a/ecrire/public/compiler.php b/ecrire/public/compiler.php
index 619958c191..f70e800716 100644
--- a/ecrire/public/compiler.php
+++ b/ecrire/public/compiler.php
@@ -183,16 +183,10 @@ function calculer_inclure($p, &$boucles, $id_boucle) {
if ($ajax)
$_options[] = "\"ajax\"=>true";
$code = " ' . argumenter_squelette($code) . '";
- $code = "echo " . sprintf(CODE_RECUPERER_FOND, $code, '$contexte_inclus', implode(',',$_options), "_request(\"connect\")") . ';';
+ $code = "echo " . sprintf(CODE_RECUPERER_FOND, $code, $contexte, implode(',',$_options), "_request(\"connect\")") . ';';
}
-
- $code = "\n'<".
- "?php\n".'$contexte_inclus = '.$contexte.";\n"
- . $code
- . "\n?'." . "'>'";
-
- return $code;
+ return "\n'<". "?php ". $code . "\n?'." . "'>'";
}
//
--
GitLab