Completed
Pull Request — master (#11)
by Lucas
37:59 queued 26:54
created
src/LoginCidadao/APIBundle/Controller/BaseController.php 2 patches
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -14,6 +14,9 @@
 block discarded – undo
14 14
 class BaseController extends FOSRestController
15 15
 {
16 16
 
17
+    /**
18
+     * @param SerializationContext $context
19
+     */
17 20
     protected function renderWithContext($content, $context = null)
18 21
     {
19 22
         $person = $this->getUser();
Please login to merge, or discard this patch.
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -33,11 +33,11 @@
 block discarded – undo
33 33
         $person = $this->getUser();
34 34
         $serializer = $this->get('jms_serializer');
35 35
         return $serializer->serialize($person, 'json',
36
-                                      SerializationContext::create()->setGroups($scope));
36
+                                        SerializationContext::create()->setGroups($scope));
37 37
     }
38 38
 
39 39
     protected function getClientScope(PersonInterface $user,
40
-                                      ClientInterface $client = null)
40
+                                        ClientInterface $client = null)
41 41
     {
42 42
         if ($client === null) {
43 43
             $client = $this->getClient();
Please login to merge, or discard this patch.
src/LoginCidadao/APIBundle/Controller/NotificationCallbackController.php 1 patch
Doc Comments   +1 added lines, -3 removed lines patch added patch discarded remove patch
@@ -26,13 +26,11 @@
 block discarded – undo
26 26
      *   templateVar="failedCallbacks"
27 27
      * )
28 28
      *
29
-     * @param Request                $request       the request object
30
-     * @param ParamFetcherInterface  $paramFetcher  param fetcher service
31 29
      *
32 30
      * @REST\Get("/notifications/callbacks/failed", name="api_1_get_failed_notification_callbacks")
33 31
      * @Audit\Loggable(type="SELECT")
34 32
      *
35
-     * @return array
33
+     * @return \Symfony\Component\HttpFoundation\Response
36 34
      */
37 35
     public function getFailedByClientAction()
38 36
     {
Please login to merge, or discard this patch.
src/LoginCidadao/APIBundle/Controller/NotificationController.php 2 patches
Doc Comments   +3 added lines, -4 removed lines patch added patch discarded remove patch
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
      * @param ParamFetcherInterface  $paramFetcher  param fetcher service
50 50
      * @Audit\Loggable(type="SELECT")
51 51
      *
52
-     * @return array
52
+     * @return \Symfony\Component\HttpFoundation\Response
53 53
      */
54 54
     public function getNotificationsAction(Request $request,
55 55
                                            ParamFetcherInterface $paramFetcher)
@@ -93,9 +93,8 @@  discard block
 block discarded – undo
93 93
      *   }
94 94
      * )
95 95
      * @REST\View(templateVar="notification")
96
-     * @param Request $request the request object
97 96
      * @param int     $id
98
-     * @return \LoginCidadao\NotificationBundle\Entity\Notification
97
+     * @return \Symfony\Component\HttpFoundation\Response
99 98
      * @throws NotFoundHttpException when notification not exist
100 99
      * @throws AccessDeniedHttpException when trying to access another user's Notification
101 100
      * @REST\Get("/notifications/{id}", name="api_1_get_notification")
@@ -172,7 +171,7 @@  discard block
 block discarded – undo
172 171
     /**
173 172
      * Fetch a Notification or throw an 404 Exception.
174 173
      *
175
-     * @param mixed $id
174
+     * @param integer $id
176 175
      * @return NotificationInterface
177 176
      * @throws NotFoundHttpException
178 177
      */
Please login to merge, or discard this patch.
Indentation   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
      * @return array
53 53
      */
54 54
     public function getNotificationsAction(Request $request,
55
-                                           ParamFetcherInterface $paramFetcher)
55
+                                            ParamFetcherInterface $paramFetcher)
56 56
     {
57 57
         $offset = $paramFetcher->get('offset');
58 58
         $offset = null == $offset ? 0 : $offset;
@@ -64,13 +64,13 @@  discard block
 block discarded – undo
64 64
 
65 65
         if (array_search('get_all_notifications', $scopes) === false) {
66 66
             $notifications = $this->getNotificationHandler()->getAllFromPersonByClient($person,
67
-                                                                                       $client,
68
-                                                                                       $limit,
69
-                                                                                       $offset);
67
+                                                                                        $client,
68
+                                                                                        $limit,
69
+                                                                                        $offset);
70 70
         } else {
71 71
             $notifications = $this->getNotificationHandler()->getAllFromPerson($person,
72
-                                                                               $limit,
73
-                                                                               $offset);
72
+                                                                                $limit,
73
+                                                                                $offset);
74 74
         }
75 75
 
76 76
         return $this->renderWithContext($notifications);
@@ -192,7 +192,7 @@  discard block
 block discarded – undo
192 192
         $user = $token->getUser();
193 193
         if ($user instanceof ClientUser) {
194 194
             return $this->validateNotificationAsClient($user->getClient(),
195
-                                                       $request);
195
+                                                        $request);
196 196
         } elseif ($user instanceof PersonInterface) {
197 197
             return $this->validateNotificationAsPerson($user, $request);
198 198
         } else {
Please login to merge, or discard this patch.
src/LoginCidadao/APIBundle/Controller/PersonController.php 2 patches
Doc Comments   +6 added lines patch added patch discarded remove patch
@@ -104,6 +104,9 @@  discard block
 block discarded – undo
104 104
         return $this->handleView($view);
105 105
     }
106 106
 
107
+    /**
108
+     * @param \Closure $callback
109
+     */
107 110
     private function runTimeLimited($callback, $waitTime = 1)
108 111
     {
109 112
         $maxExecutionTime = ini_get('max_execution_time');
@@ -127,6 +130,9 @@  discard block
 block discarded – undo
127 130
         throw new RequestTimeoutException("Request Timeout");
128 131
     }
129 132
 
133
+    /**
134
+     * @param \DateTime $updatedAt
135
+     */
130 136
     private function getCheckUpdateCallback($id, $updatedAt, $lastUpdatedAt)
131 137
     {
132 138
         $em     = $this->getDoctrine()->getEntityManager();
Please login to merge, or discard this patch.
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -173,10 +173,10 @@
 block discarded – undo
173 173
             ->createQueryBuilder('a')
174 174
             ->select('cnc, p')
175 175
             ->join('LoginCidadaoCoreBundle:Person', 'p', 'WITH',
176
-                   'a.person = p')
176
+                    'a.person = p')
177 177
             ->join('LoginCidadaoOAuthBundle:Client', 'c', 'WITH', 'a.client = c')
178 178
             ->join('LoginCidadaoCoreBundle:ConfigNotCli', 'cnc', 'WITH',
179
-                   'cnc.client = c')
179
+                    'cnc.client = c')
180 180
             ->where('c.id = '.$client->getId().' and p.id = :person_id and cnc.id = :config_id')
181 181
             ->getQuery();
182 182
         $rowR      = array();
Please login to merge, or discard this patch.
src/LoginCidadao/APIBundle/Entity/ActionLog.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -410,7 +410,7 @@  discard block
 block discarded – undo
410 410
     /**
411 411
      * Get response code
412 412
      *
413
-     * @return integer
413
+     * @return string
414 414
      */
415 415
     public function getResponseCode()
416 416
     {
@@ -428,7 +428,7 @@  discard block
 block discarded – undo
428 428
     }
429 429
 
430 430
     /**
431
-     * @return Client
431
+     * @return ClientInterface
432 432
      */
433 433
     public function getClient()
434 434
     {
Please login to merge, or discard this patch.
src/LoginCidadao/APIBundle/Entity/LogoutKey.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -79,6 +79,9 @@
 block discarded – undo
79 79
         return $this;
80 80
     }
81 81
 
82
+    /**
83
+     * @param string $logoutKey
84
+     */
82 85
     public function setKey($logoutKey)
83 86
     {
84 87
         $this->logoutKey = $logoutKey;
Please login to merge, or discard this patch.
src/LoginCidadao/APIBundle/Security/Audit/ActionLogger.php 2 patches
Doc Comments   +6 added lines, -1 removed lines patch added patch discarded remove patch
@@ -57,6 +57,9 @@  discard block
 block discarded – undo
57 57
         $annotation->setActionLogId($log->getId());
58 58
     }
59 59
 
60
+    /**
61
+     * @param integer $responseCode
62
+     */
60 63
     public function updateResponseCode($actionLogId, $responseCode)
61 64
     {
62 65
 
@@ -83,7 +86,6 @@  discard block
 block discarded – undo
83 86
     }
84 87
 
85 88
     /**
86
-     * @param Loggable $annotation
87 89
      * @param Request $request
88 90
      * @param array $controllerAction
89 91
      * @param string $auditUsername
@@ -137,6 +139,9 @@  discard block
 block discarded – undo
137 139
         $this->em->flush($log);
138 140
     }
139 141
 
142
+    /**
143
+     * @param boolean $isImpersonating
144
+     */
140 145
     public function registerImpersonate(Request $request,
141 146
                                         PersonInterface $person,
142 147
                                         PersonInterface $impersonator,
Please login to merge, or discard this patch.
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
      * @return ActionLog
91 91
      */
92 92
     private function initLog(Request $request, $actionType, $controllerAction,
93
-                             $auditUsername)
93
+                                $auditUsername)
94 94
     {
95 95
         $controller = get_class($controllerAction[0]);
96 96
         $action     = $controllerAction[1];
@@ -124,7 +124,7 @@  discard block
 block discarded – undo
124 124
     }
125 125
 
126 126
     public function registerLogin(Request $request, PersonInterface $person,
127
-                                  array $controllerAction)
127
+                                    array $controllerAction)
128 128
     {
129 129
         $auditUsername = $this->auditConfig->getCurrentUsername();
130 130
         $actionType    = ActionLog::TYPE_LOGIN;
Please login to merge, or discard this patch.
src/LoginCidadao/CoreBundle/Command/PopulateDatabaseCommand.php 1 patch
Doc Comments   +10 added lines patch added patch discarded remove patch
@@ -31,6 +31,9 @@  discard block
 block discarded – undo
31 31
         $this->createCategories($output);
32 32
     }
33 33
 
34
+    /**
35
+     * @param string $dir
36
+     */
34 37
     private function loadDumpFiles($dir, OutputInterface $output)
35 38
     {
36 39
         $em = $this->getManager();
@@ -94,6 +97,10 @@  discard block
 block discarded – undo
94 97
         return compact('id', 'name', 'state_id', 'stat', 'reviewed');
95 98
     }
96 99
 
100
+    /**
101
+     * @param string $fileName
102
+     * @param \Doctrine\DBAL\Statement $query
103
+     */
97 104
     private function loopInsert($dir, $fileName, $query, $prepareFunction, $debug = false)
98 105
     {
99 106
         $entries = 0;
@@ -112,6 +119,9 @@  discard block
 block discarded – undo
112 119
         return $entries;
113 120
     }
114 121
 
122
+    /**
123
+     * @param string $dir
124
+     */
115 125
     protected function createDefaultOAuthClient($dir, OutputInterface $output)
116 126
     {
117 127
         if (!($this->getDefaultOAuthClient() instanceof Client)) {
Please login to merge, or discard this patch.
src/LoginCidadao/CoreBundle/Command/UpgradeCommand.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -82,6 +82,9 @@
 block discarded – undo
82 82
         $io->progressFinish();
83 83
     }
84 84
 
85
+    /**
86
+     * @param string $env
87
+     */
85 88
     private function clearCache(SymfonyStyle $io, $env)
86 89
     {
87 90
         $io->section("Clearing cache ($env)...");
Please login to merge, or discard this patch.