Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Right now it is protected and you need to completely override by PluginManagerBase::createInstance() and extend DefaultFactory::createInstance() to use it while expanding your plugin's constructor.
Comment | File | Size | Author |
---|---|---|---|
#1 | plugin-1889826-1.patch | 4 KB | tim.plunkett |
Comments
Comment #1
tim.plunkettThis blocks #1871696: Convert block instances to configuration entities to resolve architectural issues
Comment #2
tim.plunkettThe other option would be to make it a non-static public method on DiscoveryInterface. That has the benefit of not being static, but I think it makes more sense on the factory.
Comment #3
msonnabaum CreditAttribution: msonnabaum commentedI think this is fine.
Comment #4
msonnabaum CreditAttribution: msonnabaum commentedSince this is blocking #1871696: Convert block instances to configuration entities to resolve architectural issues.
Comment #5
tim.plunkettThis blocks a critical.
Comment #6
Dries CreditAttribution: Dries commentedCommitted to 8.x. I don't mind it being static; static method calls are usually faster than virtual method calls.