Completed
Pull Request — 5.3 (#2650)
by Jeroen
15:22
created
src/Kunstmaan/NodeBundle/Event/SlugSecurityEvent.php 1 patch
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@  discard block
 block discarded – undo
26 26
     }
27 27
 
28 28
     /**
29
-     * @param mixed $node
29
+     * @param \Kunstmaan\NodeBundle\Entity\Node $node
30 30
      */
31 31
     public function setNode($node)
32 32
     {
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
     }
45 45
 
46 46
     /**
47
-     * @param mixed $nodeTranslation
47
+     * @param \Kunstmaan\NodeBundle\Entity\NodeTranslation $nodeTranslation
48 48
      */
49 49
     public function setNodeTranslation($nodeTranslation)
50 50
     {
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
     }
81 81
 
82 82
     /**
83
-     * @param mixed $request
83
+     * @param \Symfony\Component\HttpFoundation\Request $request
84 84
      */
85 85
     public function setRequest($request)
86 86
     {
Please login to merge, or discard this patch.
src/Kunstmaan/AdminBundle/Form/MediaTokenTransformer.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -11,7 +11,7 @@  discard block
 block discarded – undo
11 11
 class MediaTokenTransformer implements DataTransformerInterface
12 12
 {
13 13
     /**
14
-     * @param mixed $content
14
+     * @param string $content
15 15
      *
16 16
      * @return string
17 17
      */
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
     }
45 45
 
46 46
     /**
47
-     * @param mixed $content
47
+     * @param string $content
48 48
      *
49 49
      * @return string
50 50
      */
Please login to merge, or discard this patch.
src/Kunstmaan/NodeBundle/Form/EventListener/URLChooserFormSubscriber.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -10,7 +10,6 @@
 block discarded – undo
10 10
 use Symfony\Component\Form\FormEvent;
11 11
 use Symfony\Component\Form\FormEvents;
12 12
 use Symfony\Component\Validator\Constraints\Email;
13
-use Symfony\Component\Validator\Constraints\Url;
14 13
 
15 14
 class URLChooserFormSubscriber implements EventSubscriberInterface
16 15
 {
Please login to merge, or discard this patch.
Kunstmaan/NodeBundle/Form/EventListener/URLChooserLinkTypeSubscriber.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -9,7 +9,6 @@
 block discarded – undo
9 9
 use Symfony\Component\Form\FormEvent;
10 10
 use Symfony\Component\Form\FormEvents;
11 11
 use Symfony\Component\Validator\Constraints\Email;
12
-use Symfony\Component\Validator\Constraints\Url;
13 12
 
14 13
 /**
15 14
  * Class URLChooserLinkTypeSubscriber
Please login to merge, or discard this patch.
SeoBundle/Tests/unit/DependencyInjection/KunstmaanSeoExtensionTest.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@
 block discarded – undo
24 24
     }
25 25
 
26 26
     /**
27
-     * @return ExtensionInterface[]
27
+     * @return KunstmaanSeoExtension[]
28 28
      */
29 29
     protected function getContainerExtensions()
30 30
     {
Please login to merge, or discard this patch.
MultiDomainBundle/Tests/unit/EventListener/HostOverrideListenerTest.php 1 patch
Doc Comments   +6 added lines patch added patch discarded remove patch
@@ -138,6 +138,9 @@  discard block
 block discarded – undo
138 138
         $object->onKernelResponse($event);
139 139
     }
140 140
 
141
+    /**
142
+     * @param \PHPUnit\Framework\MockObject\MockObject $flashBag
143
+     */
141 144
     private function getHostOverrideListener($flashBag)
142 145
     {
143 146
         $session = $this->getMockBuilder('Symfony\Component\HttpFoundation\Session\Session')
@@ -172,6 +175,9 @@  discard block
 block discarded – undo
172 175
         return $listener;
173 176
     }
174 177
 
178
+    /**
179
+     * @param \PHPUnit\Framework\MockObject\MockObject $response
180
+     */
175 181
     private function getFilterResponseEvent($request, $response, $requestType = HttpKernelInterface::MASTER_REQUEST)
176 182
     {
177 183
         $event = $this->getMockBuilder('Symfony\Component\HttpKernel\Event\FilterResponseEvent')
Please login to merge, or discard this patch.
src/Kunstmaan/AdminListBundle/Controller/AdminListController.php 1 patch
Doc Comments   +8 added lines, -5 removed lines patch added patch discarded remove patch
@@ -33,7 +33,7 @@  discard block
 block discarded – undo
33 33
     /**
34 34
      * You can override this method to return the correct entity manager when using multiple databases ...
35 35
      *
36
-     * @return \Doctrine\Common\Persistence\ObjectManager|object
36
+     * @return \Doctrine\Persistence\ObjectManager
37 37
      */
38 38
     protected function getEntityManager()
39 39
     {
@@ -44,7 +44,7 @@  discard block
 block discarded – undo
44 44
      * Shows the list of entities
45 45
      *
46 46
      * @param AbstractAdminListConfigurator $configurator
47
-     * @param null|Request                  $request
47
+     * @param Request                  $request
48 48
      *
49 49
      * @return Response
50 50
      */
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
      *
94 94
      * @param AbstractAdminListConfigurator $configurator The adminlist configurator
95 95
      * @param string                        $type         The type to add
96
-     * @param null|Request                  $request
96
+     * @param Request                  $request
97 97
      *
98 98
      * @throws AccessDeniedHttpException
99 99
      *
@@ -188,7 +188,7 @@  discard block
 block discarded – undo
188 188
      *
189 189
      * @param AbstractAdminListConfigurator $configurator The adminlist configurator
190 190
      * @param string                        $entityId     The id of the entity that will be edited
191
-     * @param null|Request                  $request
191
+     * @param Request                  $request
192 192
      *
193 193
      * @throws NotFoundHttpException
194 194
      * @throws AccessDeniedHttpException
@@ -342,7 +342,7 @@  discard block
 block discarded – undo
342 342
      *
343 343
      * @param AbstractAdminListConfigurator $configurator The adminlist configurator
344 344
      * @param int                           $entityId     The id to delete
345
-     * @param null|Request                  $request
345
+     * @param Request                  $request
346 346
      *
347 347
      * @throws NotFoundHttpException
348 348
      * @throws AccessDeniedHttpException
@@ -469,6 +469,9 @@  discard block
 block discarded – undo
469 469
         );
470 470
     }
471 471
 
472
+    /**
473
+     * @param \Doctrine\Persistence\ObjectRepository $repo
474
+     */
472 475
     private function getMaxSortableField($repo, $sort)
473 476
     {
474 477
         $maxWeight = $repo->createQueryBuilder('i')
Please login to merge, or discard this patch.
src/Kunstmaan/AdminListBundle/Controller/EntityLockCheckController.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -17,7 +17,7 @@
 block discarded – undo
17 17
     /**
18 18
      * You can override this method to return the correct entity manager when using multiple databases ...
19 19
      *
20
-     * @return \Doctrine\Common\Persistence\ObjectManager|object
20
+     * @return \Doctrine\Persistence\ObjectManager
21 21
      */
22 22
     protected function getEntityManager()
23 23
     {
Please login to merge, or discard this patch.
src/Kunstmaan/NodeBundle/Tests/unit/Router/SlugRouterTest.php 1 patch
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -65,6 +65,10 @@
 block discarded – undo
65 65
         $object->match('/en/some-uri');
66 66
     }
67 67
 
68
+    /**
69
+     * @param \self|null $request
70
+     * @param NodeTranslation $nodeTranslation
71
+     */
68 72
     private function getContainer($request, $multiLanguage = false, $nodeTranslation = null)
69 73
     {
70 74
         $container = $this->createMock('Symfony\Component\DependencyInjection\ContainerInterface');
Please login to merge, or discard this patch.