Re: strrpos au lieu de stops pour affichage_final ?
Yohann Prigent <[email protected]>
| Newsgroups | gmane.comp.web.spip.devel |
|---|---|
| Message-ID | <[email protected]> |
C’est sûr que supprimer le commentaire a été la solution de facilité. Mais avoir des commentaires après le </body, contenant </body semble être un cas encore plus rare que d’avoir des commentaires contenant </body avant le </body, non :) ? M’enfin, c’est pas grand chose, mais un strrpos au lieu d’un strpos me semble quand même plus adapté. -- Yohann Prigent From: Fil <[email protected]> Reply: Fil <[email protected]> Date: December 13, 2015 at 2:47:04 PM To: Yohann Prigent <[email protected]> CC: [email protected] <[email protected]> Subject: Re: [spip-dev] strrpos au lieu de stops pour affichage_final ? Note que si ton commentaire était après le </body> il faudrait un autre correctif. Ce qu'il faudrait c'est éviter de faire la modif dans un commentaire. Mais dans affichage_final il faut éviter les calculs coûteux, et donc on va pas le faire et assumer que ça foire dans certains cas. Supprime le commentaire et le tour est joué ? -- Fil 2015-12-13 13:40 GMT+01:00 Yohann Prigent <[email protected]>: Hello les écureuils, Je viens de remarquer un petit soucis sur un SPIP 3.0.21. J’avais un commentaire fermant, avant le body, du type : <!-- This tag must be placed between the <body> and </body> tags, as close as possible to the opening tag. Creation Date: 12/07/2015 --> Et SPIP a inséré les scripts introduits par affichage_final comme ceci : his tag must be placed between the <body> and <script type=“texte/javascript”></script></body> tags, as close as possible to the opening tag. Creation Date: 12/07/2015 --> D’où ma question : ne vaut-il mieux pas modifier la ligne 230 de ecrire/inc/pipelines.php pour utiliser strrpos au lieu de strpos ? Bon week-end :) -- Yohann Prigent _______________________________________________ liste: http://listes.rezo.net/mailman/listinfo/spip-dev doc: http://www.spip.net/ dev: http://trac.rezo.net/trac/spip/ irc://irc.freenode.net/spip