Re: r24046 - spip/ecrire/inc

Yo,


Le mer. 5 sept. 2018 à 14:16, Matthieu Marcillaud <[email protected]> a
écrit :

>
> define('_DIR_PLUGIN_ITERATEURS',_DIR_RESTREINT.'procure:iterateurs/');
>
> ou encore
>
> define('_DIR_PLUGIN_PHP_CURL',_DIR_RESTREINT.'procure:php:curl/');
>
> Ce qui est totalement aberrant et trompeur.
>
> On peut améliorer le test_plugin_actif() pour tester les procure au
> besoin, mais se baser sur ce genre de truc me semble incohérent.
>
>
Absolument.
D'ailleurs dans les squelettes on utilise plutôt la balise #PLUGIN qui fait
appel au filtre info_plugin().
Par contre, je crois qu'il y avait aussi une histoire de cache qui pouvait
empêcher parfois de détecter le plugin ou de récupérer une information mise
à jour alors que le test du répertoire lui donne toujours la bonne
information.
J'avais d'ailleurs rajouté une option reload au filtre pour forcer le
recalcul de l'information recherchée sur le plugin (j'ai du m'en servir sur
Boussole je crois).

Donc je suis d'accord avec Matthieu qu'il serait préférable d'appeler une
fonction plutôt que de tester des constantes.
Le mieux d'ailleurs serait d'utiliser le filtre existant.

++
Eric