Conditions | 1 |
Paths | 1 |
Total Lines | 7 |
Lines | 0 |
Ratio | 0 % |
Tests | 6 |
CRAP Score | 1 |
Changes | 0 |
1 | <?php |
||
28 | 1752 | public function __construct(AbstractBlock $container, ReferenceMap $referenceMap) |
|
29 | { |
||
30 | 1752 | $this->referenceMap = $referenceMap; |
|
31 | 1752 | $this->container = $container; |
|
32 | 1752 | $this->cursor = new Cursor(\trim($container->getStringContent())); |
|
|
|||
33 | 1752 | $this->delimiterStack = new DelimiterStack(); |
|
34 | 1752 | } |
|
35 | |||
68 |
Let’s take a look at an example:
In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different sub-classes of User which does not have a getDisplayName() method, the code will break.
Available Fixes
Change the type-hint for the parameter:
Add an additional type-check:
Add the method to the parent class: