These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more
1 | <?php |
||
2 | |||
3 | namespace Kunstmaan\AdminBundle\Helper\Menu; |
||
4 | |||
5 | use Symfony\Component\HttpFoundation\Request; |
||
6 | |||
7 | /** |
||
8 | * MenuAdaptor to add the Modules MenuItem to the top menu |
||
9 | */ |
||
10 | class ModulesMenuAdaptor implements MenuAdaptorInterface |
||
11 | { |
||
12 | /** |
||
13 | * In this method you can add children for a specific parent, but also remove and change the already created children |
||
14 | * |
||
15 | * @param MenuBuilder $menu The MenuBuilder |
||
16 | * @param MenuItem[] &$children The current children |
||
17 | * @param MenuItem $parent The parent Menu item |
||
0 ignored issues
–
show
|
|||
18 | * @param Request $request The Request |
||
0 ignored issues
–
show
Should the type for parameter
$request not be null|Request ?
This check looks for It makes a suggestion as to what type it considers more descriptive. Most often this is a case of a parameter that can be null in addition to its declared types.
Loading history...
|
|||
19 | */ |
||
20 | 1 | public function adaptChildren(MenuBuilder $menu, array &$children, MenuItem $parent = null, Request $request = null) |
|
21 | { |
||
22 | 1 | if (\is_null($parent)) { |
|
23 | 1 | $menuItem = new TopMenuItem($menu); |
|
24 | $menuItem |
||
25 | 1 | ->setRoute('KunstmaanAdminBundle_modules') |
|
26 | 1 | ->setLabel('modules.title') |
|
27 | 1 | ->setUniqueId('modules') |
|
28 | 1 | ->setParent($parent); |
|
29 | 1 | if (stripos($request->attributes->get('_route'), $menuItem->getRoute()) === 0) { |
|
30 | 1 | $menuItem->setActive(true); |
|
31 | } |
||
32 | 1 | $children[] = $menuItem; |
|
33 | } |
||
34 | 1 | } |
|
35 | } |
||
36 |
This check looks for
@param
annotations where the type inferred by our type inference engine differs from the declared type.It makes a suggestion as to what type it considers more descriptive.
Most often this is a case of a parameter that can be null in addition to its declared types.