Completed
Pull Request — master (#130)
by Deven
02:49
created
src/Controller/Component/GithubApiComponent.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -33,7 +33,7 @@
 block discarded – undo
33 33
  * or not to return a status.
34 34
  *
35 35
  * @param String  $path the api path to preform the request to
36
- * @param Array   $data the data to send in the request. This works with both GET
36
+ * @param string   $data the data to send in the request. This works with both GET
37 37
  *                      and Post requests
38 38
  * @param String  $method the method type of the request
39 39
  * @param Boolean $returnStatus whether to return the status code with the
Please login to merge, or discard this patch.
src/Controller/GithubController.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -190,7 +190,7 @@
 block discarded – undo
190 190
      * @param Object $response the response returned by Github api
191 191
      * @param Integer $status status returned by Github API
192 192
      *
193
-     * @return error string
193
+     * @return string string
194 194
      */
195 195
 	protected function _getErrors($response, $status) {
196 196
 		$errorString = "There were some problems with the issue submission."
Please login to merge, or discard this 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 App\Controller\AppController;
6 6
 use Cake\Core\Configure;
7
-use Cake\Log\Log;
8 7
 use Cake\Routing\Router;
9 8
 use Cake\Event\Event;
10 9
 use Cake\ORM\TableRegistry;
Please login to merge, or discard this patch.
src/Controller/IncidentsController.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 App\Controller\AppController;
6 6
 use Cake\Network\Exception\NotFoundException;
7
-use App\Utility\Sanitize;
8 7
 /**
9 8
  * Incidents controller handling incident creation and rendering
10 9
  *
Please login to merge, or discard this patch.
src/Controller/NotificationsController.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -3,7 +3,6 @@
 block discarded – undo
3 3
 namespace App\Controller;
4 4
 
5 5
 use App\Controller\AppController;
6
-use Cake\Log\Log;
7 6
 use Cake\Routing\Router;
8 7
 use Cake\Event\Event;
9 8
 use Cake\ORM\TableRegistry;
Please login to merge, or discard this patch.
src/Controller/ReportsController.php 2 patches
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -6,7 +6,6 @@
 block discarded – undo
6 6
 use App\Utility\Sanitize;
7 7
 use Cake\Core\Configure;
8 8
 use Cake\Log\Log;
9
-use Cake\ORM\Table;
10 9
 use Cake\ORM\TableRegistry;
11 10
 use Cake\Network\Exception\NotFoundException;
12 11
 /**
Please login to merge, or discard this patch.
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -289,6 +289,9 @@
 block discarded – undo
289 289
 		$this->set("related_entries", $relatedEntries);
290 290
 	}
291 291
 
292
+    /**
293
+     * @param string $key
294
+     */
292 295
     protected function _findArrayList($results, $key) {
293 296
         $output = array();
294 297
 		foreach ($results as $row) {
Please login to merge, or discard this patch.
src/Model/Table/IncidentsTable.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -2,7 +2,6 @@
 block discarded – undo
2 2
 /* vim: set noexpandtab sw=2 ts=2 sts=2: */
3 3
 namespace App\Model\Table;
4 4
 
5
-use App\Model\AppModel;
6 5
 use Cake\ORM\Table;
7 6
 use Cake\ORM\TableRegistry;
8 7
 use Cake\Log\Log;
Please login to merge, or discard this patch.
src/Model/Table/ReportsTable.php 2 patches
Doc Comments   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
 /**
94 94
  * Retrieves the incident records that are related to the current report
95 95
  *
96
- * @return Array the list of incidents ordered by creation date desc
96
+ * @return \Cake\ORM\Query the list of incidents ordered by creation date desc
97 97
  */
98 98
 	public function getIncidents() {
99 99
 		$incidents = TableRegistry::get('Incidents')->find('all', array(
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
 /**
108 108
  * Retrieves the report records that are related to the current report
109 109
  *
110
- * @return Array the list of related reports
110
+ * @return \Cake\ORM\Query the list of related reports
111 111
  */
112 112
 	public function getRelatedReports() {
113 113
 		return $this->find("all", array(
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
  * Retrieves the incident records that are related to the current report that
120 120
  * also have a description
121 121
  *
122
- * @return Array the list of incidents ordered by description lenght desc
122
+ * @return \Cake\ORM\Query the list of incidents ordered by description lenght desc
123 123
  */
124 124
 	public function getIncidentsWithDescription() {
125 125
 		return TableRegistry::get('Incidents')->find('all', array(
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
  * Retrieves the incident records that are related to the current report that
138 138
  * that have a different stacktrace
139 139
  *
140
- * @return Array the list of incidents
140
+ * @return \Cake\ORM\Query the list of incidents
141 141
  */
142 142
 	public function getIncidentsWithDifferentStacktrace() {
143 143
 		return TableRegistry::get('Incidents')->find('all', array(
Please login to merge, or discard this patch.
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -3,7 +3,6 @@
 block discarded – undo
3 3
 namespace App\Model\Table;
4 4
 
5 5
 use Cake\ORM\Table;
6
-use App\Model\AppModel;
7 6
 use Cake\Model\Model;
8 7
 use Cake\Routing\Router;
9 8
 use Cake\ORM\TableRegistry;
Please login to merge, or discard this patch.
src/View/Helper/ReportsHelper.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 App\Utility\Sanitize;
6 6
 use App\View\Helper\AppHelper;
7
-use App\View\Helper\IncidentsHelper;
8 7
 use Cake\Utility\Inflector;
9 8
 use Cake\View\View;
10 9
 
Please login to merge, or discard this patch.
tests/TestCase/Model/Table/ReportsTableTest.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -2,7 +2,6 @@
 block discarded – undo
2 2
 namespace App\Test\TestCase\Model\Table;
3 3
 
4 4
 use Cake\ORM\TableRegistry;
5
-
6 5
 use Cake\TestSuite\TestCase;
7 6
 
8 7
 class ReportsTableTest extends TestCase {
Please login to merge, or discard this patch.