Re: r24046 - spip/ecrire/inc

Le 05/09/2018 à 12:15, RastaPopoulos a écrit :
> Le 05/09/2018 à 11:53, Matthieu Marcillaud a écrit :
>> Mais que de toutes façons je ne pense pas que ça soit pertinent : cette
>> constante était fausse pour tout plugin 'procure' et n’avait aucun sens
>> (ça n’indique pas de répertoire réellement).
> 
> Ah bé c'est bien de le savoir et de documenter la manière propre de le
> faire, parce que yen a un sacré paquet des tests de ce genre pour savoir
> si un plugin est bien là et faire des choses en plus en conséquence, me
> semble-t-il.

Heu, je ne lançais pas de troll hein…

Et je pense que le point d’entrée correct est peut être 
test_plugin_actif() comme signale Real3t pour ces tests. L’intérêt est 
de passer par une fonction commune qu’on peut modifier ensuite plutôt 
que chacun tester une implémentation. Mais bref.

Comprends bien que les constantes en questions déclaraient des

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.

MM.
_______________________________________________
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