Conditions | 1 |
Paths | 1 |
Total Lines | 11 |
Code Lines | 7 |
Lines | 0 |
Ratio | 0 % |
Changes | 0 |
1 | <?php |
||
42 | public function __construct(AbstractResourceFixture $productReviewFixture) |
||
43 | { |
||
44 | $this->productReviewFixture = $productReviewFixture; |
||
45 | |||
46 | $this->faker = \Faker\Factory::create(); |
||
47 | $this->optionsResolver = |
||
48 | (new OptionsResolver()) |
||
49 | ->setRequired('amount') |
||
50 | ->setAllowedTypes('amount', 'int') |
||
51 | ; |
||
52 | } |
||
53 | |||
82 |
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: