- Issue created by @joachim
- First commit to issue fork.
- 🇬🇧United Kingdom nexusnovaz
I've added the statement and opened the MR 12755. However, i know this is dependent on 🐛 DefaultLazyPluginCollection::initializePlugin() throws the wrong sort of exception Active , however, im hoping that we can merge that in without any issues once upstream. I won't mark as needs review just yet, but will keep an eye on that parent ticket.
- 🇬🇧United Kingdom joachim
The new docs need to go on LazyPluginCollection, not the child class.
And I think there are several cases where an exception is thrown, and child classes may add more, so we should probably be vague and just say 'if the plugin cannot be instantiated'.
And yes, you're right to wait for the other issue to get in.
- 🇬🇧United Kingdom nexusnovaz
I've made a change in the location of the comment and the comment itself. But once again, i know this needs to wait to be merged in! Will leave as Active for now.