Filter out blacklisted addons before calling included hook #7857
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The
ember-engines
caches the host addons (theapp.project.addons
list) on the first addon hook callback, which isincluded
. This is generally OK, except in the case whenEmberApp
blacklists some of the addons.Because the list of the addons is actually filtered after the hook is called,
ember-engines
deduplication algorithm is using the incorrect list of parent addons, because it has cached the list on the first hook call -included
). This leads toember-engines
deduplicating dependencies, which are not actually available in the app.