fsi-open /
admin-bundle
These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more
| 1 | <?php |
||
| 2 | |||
| 3 | declare(strict_types=1); |
||
| 4 | |||
| 5 | /** |
||
| 6 | * (c) FSi sp. z o.o. <[email protected]> |
||
| 7 | * |
||
| 8 | * For the full copyright and license information, please view the LICENSE |
||
| 9 | * file that was distributed with this source code. |
||
| 10 | */ |
||
| 11 | |||
| 12 | namespace FSi\Bundle\AdminBundle\Admin\CRUD\Context\Request; |
||
| 13 | |||
| 14 | use FSi\Bundle\AdminBundle\Admin\Context\Request\AbstractHandler; |
||
| 15 | use FSi\Bundle\AdminBundle\Event\AdminEvent; |
||
| 16 | use FSi\Bundle\AdminBundle\Event\ListEvent; |
||
| 17 | use FSi\Bundle\AdminBundle\Event\ListEvents; |
||
| 18 | use FSi\Bundle\AdminBundle\Exception\RequestHandlerException; |
||
| 19 | use Symfony\Component\HttpFoundation\Request; |
||
| 20 | use Symfony\Component\HttpFoundation\Response; |
||
| 21 | |||
| 22 | class DataSourceBindParametersHandler extends AbstractHandler |
||
| 23 | { |
||
| 24 | public function handleRequest(AdminEvent $event, Request $request): ?Response |
||
| 25 | { |
||
| 26 | $event = $this->validateEvent($event); |
||
| 27 | |||
| 28 | if ($event->hasResponse()) { |
||
| 29 | return $event->getResponse(); |
||
| 30 | } |
||
| 31 | |||
| 32 | $this->eventDispatcher->dispatch(ListEvents::LIST_DATASOURCE_REQUEST_PRE_BIND, $event); |
||
| 33 | if ($event->hasResponse()) { |
||
| 34 | return $event->getResponse(); |
||
| 35 | } |
||
| 36 | |||
| 37 | $event->getDataSource()->bindParameters($request); |
||
|
0 ignored issues
–
show
|
|||
| 38 | |||
| 39 | $this->eventDispatcher->dispatch(ListEvents::LIST_DATASOURCE_REQUEST_POST_BIND, $event); |
||
| 40 | if ($event->hasResponse()) { |
||
| 41 | return $event->getResponse(); |
||
| 42 | } |
||
| 43 | |||
| 44 | return null; |
||
| 45 | } |
||
| 46 | |||
| 47 | private function validateEvent(AdminEvent $event): ListEvent |
||
| 48 | { |
||
| 49 | if (!$event instanceof ListEvent) { |
||
| 50 | throw new RequestHandlerException(sprintf('%s require ListEvent', get_class($this))); |
||
| 51 | } |
||
| 52 | |||
| 53 | return $event; |
||
| 54 | } |
||
| 55 | } |
||
| 56 |
It seems like the type of the argument is not accepted by the function/method which you are calling.
In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.
We suggest to add an explicit type cast like in the following example: