Skip to content

Conversation

aeslaughter
Copy link

It doesn't seem necessary to require the extension to register itself when it can be done automatically. Calling the registration method manually makes it easy to forget (e.g., the meta extension) thus not allowing the list to be inspected by other extensions that may require an extension.

It doesn't seem necessary to require the extension to register itself when it can be done automatically. Calling the registration method manually makes it easy to forget (e.g., the meta extension) thus not allowing the list to be inspected by other extensions that may require an extension.
@waylan
Copy link
Member

waylan commented Apr 10, 2017

The existing behavior was an intensional design decision. Not all extensions need to be registered and we don't force them to be. In fact for many extensions, there is absolutely no benefit to registering them. It is up to each extension developer to decide whether that extension needs to be registered.

@aeslaughter
Copy link
Author

Just because one developer doesn't need it to be registered doesn't mean another doesn't. For example, I would like to check for dependencies and it certainly doesn't hurt to register an extension. Personally, I think the inconsistency is problematic, but I will find another way.

@waylan waylan added this to the Version 3.0 milestone Apr 10, 2017
@waylan
Copy link
Member

waylan commented Apr 10, 2017

@aeslaughter when the extension registration feature was added, its sole envisioned purpose was to provide for the "reset" feature. As it turns out, it is useful for other purposes and may be due for reassessment. However, as the 2.x branch is in a feature freeze while we work on 3.0, this would be something to address in 3.0.

@waylan
Copy link
Member

waylan commented Jul 24, 2018

We will not be changing this. An extension is "registered" if it needs to utilize the "reset" feature. and that would only ever be a decision made by the developer of the extension. If you are trying to determine if a certain extension is in use, then that should be done in another way as it is outside the scope of the "register" feature.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants