Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Problem/Motivation
\Drupal\Component\Plugin\Factory\FactoryInterface does not document andy @throws for createInstance(), yet \Drupal\Component\Plugin\Factory\DefaultFactory will throw a \Drupal\Component\Plugin\Exception\PluginException for an invalid plugin ID or definition.
Proposed resolution
Add @throws to the interface documentation
Comment | File | Size | Author |
---|---|---|---|
#3 | 2261391-2.patch | 1.2 KB | pwolanin |
#1 | 2261391-1.patch | 1.07 KB | pwolanin |
Comments
Comment #1
pwolanin CreditAttribution: pwolanin commentedSimple doc patch. Also fixes the @file text.
Comment #2
Crell CreditAttribution: Crell commented"are invalid", I think?
Comment #3
pwolanin CreditAttribution: pwolanin commentedreworded and made capitalization consistent.
Comment #4
Crell CreditAttribution: Crell commentedI can't see the bot objecting...
Comment #6
jhodgdonThanks! Committed to 8.x.