Completed
Pull Request — master (#171)
by Deven
03:17
created
src/Controller/ReportsController.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -320,6 +320,9 @@
 block discarded – undo
320 320
         $this->set('related_entries', $relatedEntries);
321 321
     }
322 322
 
323
+    /**
324
+     * @param string $key
325
+     */
323 326
     protected function _findArrayList($results, $key)
324 327
     {
325 328
         $output = array();
Please login to merge, or discard this patch.
src/Model/Table/IncidentsTable.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -228,7 +228,7 @@  discard block
 block discarded – undo
228 228
      *
229 229
      * @param array $bugReport the bug report being checked
230 230
      *                         Integer $index: for php exception type
231
-     * @param mixed $index
231
+     * @param integer $index
232 232
      *
233 233
      * @return array the first similar report or null
234 234
      */
@@ -254,7 +254,7 @@  discard block
 block discarded – undo
254 254
      *
255 255
      * @param array $bugReport the bug report the report record is being created for
256 256
      *                         Integer $index: for php exception type
257
-     * @param mixed $index
257
+     * @param integer $index
258 258
      *
259 259
      * @return array an array with the report fields can be used with Report->save
260 260
      */
Please login to merge, or discard this patch.
src/Model/Table/ReportsTable.php 1 patch
Doc Comments   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
     /**
96 96
      * Retrieves the incident records that are related to the current report.
97 97
      *
98
-     * @return array the list of incidents ordered by creation date desc
98
+     * @return \Cake\ORM\Query the list of incidents ordered by creation date desc
99 99
      */
100 100
     public function getIncidents()
101 101
     {
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
111 111
     /**
112 112
      * Retrieves the report records that are related to the current report.
113 113
      *
114
-     * @return array the list of related reports
114
+     * @return \Cake\ORM\Query the list of related reports
115 115
      */
116 116
     public function getRelatedReports()
117 117
     {
@@ -124,7 +124,7 @@  discard block
 block discarded – undo
124 124
      * Retrieves the incident records that are related to the current report that
125 125
      * also have a description.
126 126
      *
127
-     * @return array the list of incidents ordered by description lenght desc
127
+     * @return \Cake\ORM\Query the list of incidents ordered by description lenght desc
128 128
      */
129 129
     public function getIncidentsWithDescription()
130 130
     {
@@ -143,7 +143,7 @@  discard block
 block discarded – undo
143 143
      * Retrieves the incident records that are related to the current report that
144 144
      * that have a different stacktrace.
145 145
      *
146
-     * @return array the list of incidents
146
+     * @return \Cake\ORM\Query the list of incidents
147 147
      */
148 148
     public function getIncidentsWithDifferentStacktrace()
149 149
     {
Please login to merge, or discard this patch.
config/Migrations/20170515135017_fix_long_pma_versions.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -29,6 +29,9 @@
 block discarded – undo
29 29
         // but the migration itself is idempotent
30 30
     }
31 31
 
32
+    /**
33
+     * @param string $table
34
+     */
32 35
     private function _sanitizeVersionsInTable($table)
33 36
     {
34 37
         $sql = 'SELECT `id`, `pma_version` FROM `' . $table . '`';
Please login to merge, or discard this patch.
src/Controller/Component/MailerComponent.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -20,7 +20,6 @@
 block discarded – undo
20 20
 namespace App\Controller\Component;
21 21
 
22 22
 use Cake\Controller\Component;
23
-use Cake\ORM\TableRegistry;
24 23
 use Cake\Mailer\Email;
25 24
 use Cake\Core\Configure;
26 25
 
Please login to merge, or discard this patch.
src/Controller/EventsController.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -105,7 +105,7 @@
 block discarded – undo
105 105
     /**
106 106
      * Validate HTTP Request recieved
107 107
      *
108
-     * @param Request $request Request object
108
+     * @param \Cake\Http\ServerRequest $request Request object
109 109
      *
110 110
      * @return int status code based on if this is a valid request
111 111
      */
Please login to merge, or discard this patch.
src/Controller/GithubController.php 1 patch
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -228,7 +228,7 @@  discard block
 block discarded – undo
228 228
      * @param object $response the response returned by Github api
229 229
      * @param int    $status   status returned by Github API
230 230
      *
231
-     * @return error string
231
+     * @return string string
232 232
      */
233 233
     protected function _getErrors($response, $status)
234 234
     {
@@ -367,7 +367,7 @@  discard block
 block discarded – undo
367 367
      * Get Incident counts for a report and
368 368
      * all its related reports
369 369
      *
370
-     * @param $reportId Report ID
370
+     * @param integer $reportId Report ID
371 371
      *
372 372
      * @return $total_incident_count Total Incident count for a report
373 373
      */
@@ -403,7 +403,7 @@  discard block
 block discarded – undo
403 403
      *
404 404
      * @param $issueState Linked Github issue's state
405 405
      *
406
-     * @return Corresponding status to which the linked report should be updated to
406
+     * @return string status to which the linked report should be updated to
407 407
      */
408 408
     protected function _getReportStatusFromIssueState($issueState)
409 409
     {
Please login to merge, or discard this patch.
src/Controller/DevelopersController.php 1 patch
Unused Use Statements   -2 removed lines patch added patch discarded remove patch
@@ -21,8 +21,6 @@
 block discarded – undo
21 21
 
22 22
 use Cake\Core\Configure;
23 23
 use Cake\Event\Event;
24
-use Cake\Network\Exception\NotFoundException;
25
-use Cake\ORM\TableRegistry;
26 24
 
27 25
 /**
28 26
  * Developer controller handling developer login/logout/register.
Please login to merge, or discard this patch.
tests/TestCase/Controller/ReportsControllerTest.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -4,7 +4,6 @@
 block discarded – undo
4 4
 
5 5
 use Cake\ORM\TableRegistry;
6 6
 use Cake\TestSuite\IntegrationTestCase;
7
-use Cake\Network\Exception\NotFoundException;
8 7
 
9 8
 class ReportsControllerTest extends IntegrationTestCase
10 9
 {
Please login to merge, or discard this patch.