Recherche avancée

Sur d’autres sites (9688)

• Evolution #4102 : Ordre des inclures dans cache/charger_plugins_options.php

  24 février 2018, par RastaPopoulos ♥

  Bah oui je parle d’UN fichier parce que je parle spécifiquement des fichiers options, les fichiers fonctions n’ayant strictement rien à voir.

  La différence entre les deux n’est pas dans ce qu’on décide d’y mettre, la différence c’est que les fonctions.php sont inclus lors de la compilation des squelettes, alors que les fichiers options.php sont inclus à chaque hit en permanence. Ça n’a aucun rapport avec le fait d’être prévu pour mettre des define(), ce n’est absolument pas spécifiquement prévu pour ça.

  Le principe des surcharges c’est qu’on vient toujours après, que ce soit pour une fonction, ou pour une globale, t’as beau faire des isset(), ça change rien qu’on veut arriver après et recevoir le contenu de ce qui a précédé. Donc quand on surcharge une globale, c’est pas forcément pour la remplacer entièrement, on veut absolument avoir le contenu de ce qui a été mis dedans avant. Enfin bref comme pour les pipelines quoi, on arrive après et donc dans le flux on reçoit ce qui a été mis dedans par le noyau et les plugins qui précèdent. Et dans les fichiers d’options, on peut y surcharger des fonctions dist() aussi, que ce soit du noyau ou d’autres plugins, peu importe où se trouve ces fonctions au départ.

  Pour un nouveau pipeline "pre_options" ou "initialisation", je ne vois pas en quoi les autres plugins que le tiens aurait à l’utiliser. C’est TOI uniquement qui a besoin de l’utiliser, justement, les autres plugins continueraient de définir certains de leurs define() dans leur fichier options.php. Mais toi avec ce plugin tu passerais avant les options, c’est tout l’utilité. Donc une fois qu’il est créé, ya rien à attendre.

• Evolution #4102 : Ordre des inclures dans cache/charger_plugins_options.php

  24 février 2018, par placido .

  il faut absolument qu’il continue d’y avoir un fichier avec le même ordre puisqu’on y met des pipelines, des variables globales (GLOBALS[’z_blocs’] par ex mais il y en a d’autres) qui DOIVENT être dans l’ordre actuel avec ceux qui nécessitent en dernier pour pouvoir passer après.

  Ok, alors faisons un point sur le rôle des fichiers xxx_options.php vs xxx_fonctions.php.
  Il me semblait justement que le fichier xxx_options.php est dévolu principalement aux define(), et de manière dérogatoire à quelques fonctions dont on souhaite qu’elles prennent la main très rapidement à chaque hit (de ce double emploi nait la limitation en fait).

  Oui, l’ordre des fichiers fonctions.php est important pour les surcharges, en effet. (Note qu’il n’était pas question de modifier cet ordre, seulement celui des fichiers options.php ; tu parles d’*un* fichier mais il y a en 2 en l’occurence).

  Donc la question, c’est : Est-ce un probleme que l’ordre d’appel des fichiers options.php diffère de celui des fichiers fonctions.php ?
  Plus j’y réfléchis, plus je me dis que ce n’est pas un problème, car les fichiers options.php ne contiennent pas - à ma modeste connaissance - d’éléments surchargeables :
  - les define() sont des constantes
  - les GLOBALS[’trucs’] livrés par défaut sont (devraient) être conditionnés par un isset()
  - les pipelines - outre qu’ils ont leur place dans le fichier dédié, ne sont pas surchargeables (function already exists !)
  - les fonctions qui y sont définies ne sont pas surchargeables non plus ( c-à-d suffixées par _dist), du moins je n’ai pas d’exemple en tête, infirmant ceci.

  Si tu as un exemple d’une fonction dûment déclarée dans pluginX_options.php et qui se veut surchargeable, je prends.

  J’ai modifié plugin.php en persevérant dans cette voie. Les premiers tests que j’ai pu faire en local avec une quinzaine de plugins actifs n’ont, pour l’instant, pas montré d’erreurs.
  À suivre...

  Et donc il faudrait plutôt un pipeline au tout début, dédié à ce cas des define()

  J’approuve. C’est sans conteste la solution la plus propre et la plus souple. Mais de là à ce qu’une majorité signicative de plugins adoptent cet hypothétique (quoique salvateur) pipeline... #SPIP4.0 #année2024

• Evolution #4102 : Ordre des inclures dans cache/charger_plugins_options.php

  23 février 2018, par RastaPopoulos ♥

  Je l’ai déjà dit juste avant, il faut absolument qu’il continue d’y avoir un fichier avec le même ordre puisqu’on y met des pipelines, des variables globales (GLOBALS[’z_blocs’] par ex mais il y en a d’autres) qui DOIVENT être dans l’ordre actuel avec ceux qui nécessitent en dernier pour pouvoir passer après.

  Le cas des define() qui doivent être en tout premier est le seul cas particulier où ça doit être avant, alors on ne va pas prendre ça comme cas principal sur lequel se baser. C’est celui là le cas rare et surtout qui ne concerne absolument que les define().

  Dans tous les autres cas, ceux qui veulent surcharger doivent se trouver après, pas avant.

  Et donc il faudrait plutôt un pipeline au tout début, dédié à ce cas des define(), ou pas que, mais où on sait qu’on est au tout début de SPIP, avant même les inclusions des fichiers. Un pipeline "initialisation" en fait, c’est même plus générique que "pre_options" comme je disais avant.