"
@@ -154,48 +160,46 @@ function valider_resultats($res, $mode)
. "
"
. _T('zbug_profile', array('time' =>''))
. "
"
- . _T('message')
- . "
Page
args"
+ . _T('public:message')
+ . "
"
+ . _T('ecrire:info_url')
. "
"
. $table
- . "
");
+ . "
");
}
// http://doc.spip.org/@valider_script
-function valider_script($transformer_xml, $script, $dir, $ext)
+function valider_script($transformer_xml, $script, $dir, $ext, $args=true)
{
$script = basename($script, '.php');
$dir = basename($dir);
$f = charger_fonction($script, $dir, true);
// ne pas se controler soi-meme ni l'index du repertoire ni un fichier annexe
+
if ($script == _request('exec') OR $script=='index' OR !$f)
- return array('/', 0, '', $script,'');
+ return array(0, array(), $script,'');
- list($texte, $err) = $transformer_xml($f, true);
+ list($texte, $err) = $transformer_xml($f, $args);
$appel = '';
// s'il y a l'attribut minipres, le test est non significatif
- // le script necessite peut-etre des arguments, on lui en donne,
- // en appelant la fonction _args associee si elle existe
- // Si ca ne marche toujours pas, les arguments n'étaient pas bons
- // ou c'est une authentification pour action d'administration;
- // tant pis, on signale le cas par un resultat negatif
-
- if (strpos($texte, "id='minipres'")) {
- if (!$g = charger_fonction($script . '_args', $dir, true)) {
- $res = 0 - strlen($res);
- } else {
- $args = array(1, 'id_article', 1);
- list($texte, $err) = $transformer_xml($g, $args);
- $appel = 'id_article=1&type=id_article&id=1';
- if (strpos($texte, "id='minipres'")) {
- $res = 0 - strlen($texte);
- } else $res = strlen($texte);
- }
- } else $res = strlen($texte);
+ // le script necessite peut-etre des arguments.
+ // On regarde alors s'il existe une fonction de meme nom
+ // mais avec "_args" au bout:
+ // elle est censee recevoir les valeurs de $_REQUEST et ne pas faire
+ // les controles d'autorisation (fait par la fonction principale)
+ // Si ou on l'appelle avec des arguments arbitraires;
+ // si nouvel echec on abandonne:
+ // que faire contre l'absence de reflexivite et de typage de ce fichu PHP
+ if (strpos($texte, "id='minipres'")
+ AND ($g = charger_fonction($script . '_args', $dir, true))) {
+ $args = array(1, 'id_article', 1, 0);
+ list($texte, $err) = $transformer_xml($g, $args);
+ $appel = 'id_article=1&type=id_article&id=1';
+ }
$appel = valider_pseudo_url($dir, $script, $appel);
- return array(count($err), $res, $err, $script, $appel);
+ return array($texte, $err, $script, $appel);
}
// http://doc.spip.org/@valider_pseudo_url
@@ -213,9 +217,9 @@ function valider_skel($transformer_xml, $file, $dir, $ext)
{
if (!lire_fichier ($file, $text)) return array('/', '/', $file,'');
if (!strpos($text, 'DOCTYPE')) {
- preg_match(",Content[-]Type: *\w+/(\S)+,", $text, $r);
+ preg_match(",Content[-]Type:\s*[^/]+/([^ ;]+),", $text, $r);
if ($r[1] === 'css' OR $r[1] === 'plain')
- return array('/', 'DOCTYPE?', $file,'');
+ return array(0, array(), $file,'');
}
if ($ext != 'html') {
@@ -225,21 +229,21 @@ function valider_skel($transformer_xml, $file, $dir, $ext)
$script = $file;
} else {
$script = basename($file,'.html');
- // pas de validation solitaire pour les squelettes internes, a revoir.
+ // les squelettes en sous-repertoire sont problematiques,
+ // traitons au moins le cas prive/exec
if (substr_count($dir, '/') <= 1) {
$url = generer_url_public($script, $contexte);
- } else $url = '';
+ } else $url = valider_pseudo_url(basename($dir), basename($file, '.html'), $contexte);
$composer = charger_fonction('composer', 'public');
list($skel_nom, $skel_code) = $composer($text, 'html', 'html', $file);
spip_log("compilation de $file en " . strlen($skel_code) . " octets de nom $skel_nom");
- if (!$skel_nom) return array('/', '/', $file,'');
+ if (!$skel_nom) return array('/', 0, $file,'');
$contexte = valider_contexte($skel_code, $file);
$page = $skel_nom(array('cache'=>''), array($contexte));
}
list($texte, $err) = $transformer_xml($page['texte']);
- $res = strlen($texte);
- return array(count($err), $res, $err, $script, $url);
+ return array($texte, $err, $script, $url);
}
// Analyser le code pour construire un contexte plausible complet
@@ -290,11 +294,19 @@ function valider_dir($files, $ext, $dir)
$res = array();
$transformer_xml = charger_fonction('valider', 'xml');
$valideur = $ext=='php' ? 'valider_script' : 'valider_skel' ;
+ include_spip('public/assembler');
foreach($files as $f) {
spip_timer($f);
$val = $valideur($transformer_xml, $f, $dir, $ext);
+ // Ne pas saturer la memoire, donner juste la taille de la page
+ // avec un nombre negatif quand c'est un message d'erreur
+ if (is_string($val[0])) {
+ $n = strlen($val[0]);
+ $val[0] = strpos($val[0], "id='minipres'") ? (0-$n):$n;
+ }
$n = spip_timer($f);
$val[]= $n;
+ array_unshift($val, count($val[1]));
spip_log("validation de $f en $n secondes");
$res[]= $val;
}