Conditions | 1 |
Paths | 1 |
Total Lines | 8 |
Lines | 0 |
Ratio | 0 % |
Tests | 0 |
CRAP Score | 2 |
Changes | 0 |
1 | <?php |
||
53 | public function handleRemainingContents(ContextInterface $context, Cursor $cursor) |
||
54 | { |
||
55 | // TODO: Is this even really needed? Can/should this be abstract instead? |
||
56 | // create paragraph container for line |
||
57 | $context->addBlock(new Paragraph()); |
||
58 | $cursor->advanceToNextNonSpaceOrTab(); |
||
59 | $context->getTip()->addLine($cursor->getRemainder()); |
||
|
|||
60 | } |
||
61 | |||
67 |
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: