Bonjour,
Qu'est ce que c'est que cette vague de remplacement ubuesque de include_spip() par include(chemin en dur) ?
C'est un non sens complet qui va a l'encontre du fonctionnement meme de spip.
Que tout fichier inclus passe par include_spip au lieu de include(chemin en dur) est essentiel à l'homogénéité de fonctionnement du core et des plugins.
Cela permet la surcharge potentielle de tout fichier de tout plugin. Il m'arrive régulièrement de faire ce type d'opération, qui n'a pas a être réservé au core
Si tu vas par là il n'y a plus qu'à remettre des include dans tout le core, ca ira plus vite.
Au nom de quoi peux tu décider que cout_fonctions ne peut pas être surchargé par qui en a besoin ?
C'est vraiment n'importe quoi, là, tout ça pour chassez des pétouilles de pouilleme de ms que tu n'as meme pas mesuré.
On sait très bien que find_in_path/include_spip est appelé à de nombreuses reprises. Mais c'est l'essence meme du fonctionnement de spip. Revenir sur ça c'est tuer le principe général.
Maintenant, si l'on parle mesure, chaque appel à find_in_path coute en moyenne 1ms (car attention, il ne faut pas mesurer le temps passé à l'inclusion meme, uniquement celui passé à rechercher le fichier)
Tu peux travailler à optimiser cela si tu veux, mais commence par te munir des outils et de la méthodologie nécessaire car on n'attrape pas de microbes avec une loupe.
Et je suis totalement en désaccord avec ce commit qui va a l'encontre du principe meme de Spip
Cédric
Le 21 juin 08 à 00:52, real3t@gmail.com a écrit :
Author: real3t@gmail.com
Date: Sat Jun 21 00:52:52 2008
New Revision: 21046Log:
oups (merci PatV sur IRC)
SPIP n'aime pas la symétrieModified:
_plugins_/_stable_/couteau_suisse/exec/cs_boite_rss.php
_plugins_/_stable_/couteau_suisse/outils/boites_privees.phpModified: _plugins_/_stable_/couteau_suisse/exec/cs_boite_rss.php
--- _plugins_/_stable_/couteau_suisse/exec/cs_boite_rss.php (original)
+++ _plugins_/_stable_/couteau_suisse/exec/cs_boite_rss.php Sat Jun 21 00:52:52 2008
@@ -60,7 +60,7 @@
$p .= "<li style='padding-top:0.6em;'>$t</li>";
}
} else {
- include(_DIR_PLUGIN_COUTEAU_SUISSE_.'cout_fonctions');
+ include(_DIR_PLUGIN_COUTEAU_SUISSE.'cout_fonctions');
$p = '<span style="color: red;">'._T('couteauprive:erreur:probleme', array('pb'=>cs_lien(_CS_RSS_SOURCE,_T('couteauprive:erreur:distant')))).'</span>';
}
include_spip('inc/filtres');Modified: _plugins_/_stable_/couteau_suisse/outils/boites_privees.php
--- _plugins_/_stable_/couteau_suisse/outils/boites_privees.php (original)
+++ _plugins_/_stable_/couteau_suisse/outils/boites_privees.php Sat Jun 21 00:52:52 2008
@@ -23,7 +23,7 @@
// compatibilite SPIP 1.92
$fetch = function_exists('sql_fetch')?'sql_fetch':'spip_fetch_array';
// pour cs_lien()
- include(_DIR_PLUGIN_COUTEAU_SUISSE_.'cout_fonctions');
+ include(_DIR_PLUGIN_COUTEAU_SUISSE.'cout_fonctions');
$flux['data'] .= debut_cadre_relief(find_in_path('img/couteau-24.gif'),true,'',_T('icone_statistiques_visites'))
. "<p><b>"._T('couteau:derniers_connectes')."</b></p>"
. cs_derniers_connectes($fetch)_______________________________________________
Spip-zone-commit@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone-commit
