Skip to content

README.md

Latest commit

 

History

History
29 lines (18 loc) · 2.21 KB

README.md

File metadata and controls

29 lines (18 loc) · 2.21 KB
Raw

Module Manager :FOR[]:NEEDS[] (failed) Proof of Concept

This is a FAILED Proof of Concept for using :FOR[]:NEEDS[] while patching third-parties add'ons for features introduced by one Add'On (and that not rarely depends on a third one).

Unfortunately, it failed due an excessive trust on the MM Documentation that not only prevented me to identify the code on MM where the (in my view) misbehaviour would be happening, but also leaded me to cook a Test Case where the problem didn't triggered, fooling me into believing I had a valid theory.

The subjacent idea, however, is still valid - but it can't be implemented using stock Module Manager as I was intending. That really sucks.

The aftermath is an update on the Forum's Module Manager documentation, in the hopes no one falls into this booby trap again.

Current Work in Progress Proposal

To split the :FOR behaviour into two new MM patches (and not only one as previously):

  • :THIS, a patch for creating "tags" (modnames in the documentation) and nothing more.
    • It will be always "executed" to preserve that part of the :FOR semantics.
  • :AT, a new patch that will be used for ordering the patching.
    • Implementing this part of the :FOR semantics.
  • :FOR would be left as it is forever, and it would be probably be a nice idea to issue a warning (only once) about it, as IMHO this thing should be considered deprecated once :THIS and AT: is implemented - if it will ever happen, of course...

References