[spip-dev] r24046 - spip/ecrire/inc

marcimat@rezo.net a écrit le 04/09/2018 à 12:55 :

Author: marcimat@rezo.net
Date: 2018-09-04 12:55:48 +0200 (mar, 04 sep 2018)
New Revision: 24046

Log:
?\195?\137viter de cr?\195?\169er de fausses constantes de chemins pour les "procures". C?\226?\128?\153est inutile.

Modified:
    spip/ecrire/inc/plugin.php

Details: http://core.spip.org/projects/spip/repository/revisions/24046

Je crains que ça casse des codes qui font :
if(defined('_DIR_PLUGIN_PREFIX')) {
pour vérifier la présence d'un plugin.

En particulier :
https://zone.spip.net/trac/spip-zone/browser/spip-zone/_plugins_/ieconfig/trunk/formulaires/ieconfig_import.php#L255

Me trompe-je ?

Je crains que ça casse des codes qui font :
if(defined('_DIR_PLUGIN_PREFIX')) {
pour vérifier la présence d'un plugin.

Je dirais que… déjà cette méthode serait à revoir via (filtre_info_plugin_dist).

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).

MM.

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.

effectivement, je vois pas penser plusieurs fois.
Alors même qu'on a
https://code.spip.net/autodoc/tree/ecrire/inc/utils.php.html#function_test_plugin_actif

Maïeul a écrit le 05/09/2018 à 12:21 :

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.

Yo,