Inspection of "Fixing bugs" - SherifElfadaly/Laravel-Api-Skeleton - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 8b745b...e40879 )

by Sherif

created 2017-10-16 12:40 UTC

Status

Doc Comments +3 added lines, -2 removed lines patch added patch discarded remove patch

@@ -91,7 +91,7 @@  discard block
 block discarded – undo
      * Generate headers for the given route.
      * 
      * @param  array  &$route
-     * @param  object $reflectionClass
+     * @param  \ReflectionClass $reflectionClass
      * @param  string $method
      * @param  array  $skipLoginCheck
      * @return void
@@ -118,6 +118,7 @@  discard block
 block discarded – undo
      * 
      * @param  array  &$route
      * @param  object $reflectionMethod]
+     * @param \ReflectionMethod $reflectionMethod
      * @return void
      */
     protected function processDocBlock(&$route, $reflectionMethod)
@@ -140,7 +141,7 @@  discard block
 block discarded – undo
      * Generate post body for the given route.
      * 
      * @param  array  &$route
-     * @param  object $reflectionMethod
+     * @param  \ReflectionMethod $reflectionMethod
      * @param  array  $validationRules
      * @return void
      */

Please login to merge, or discard this patch.

src/Modules/V1/Core/Http/Controllers/BaseApiController.php 1 patch

Doc Comments +17 added lines, -17 removed lines patch added patch discarded remove patch

@@ -45,8 +45,8 @@  discard block
 block discarded – undo
      * Fetch all records with relations from storage.
      * 
      * @param  string  $sortBy The name of the column to sort by.
-     * @param  boolean $desc   Sort ascending or descinding (1: desc, 0: asc).
-     * @return \Illuminate\Http\Response
+     * @param  integer $desc   Sort ascending or descinding (1: desc, 0: asc).
+     * @return \Illuminate\Http\JsonResponse|null
      */
     public function index($sortBy = 'created_at', $desc = 1) 
     {
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
      * Fetch the single object with relations from storage.
      * 
      * @param  integer $id Id of the requested model.
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\JsonResponse|null
      */
     public function find($id) 
     {
@@ -77,8 +77,8 @@  discard block
 block discarded – undo
      * @param  string  $query   The search text.
      * @param  integer $perPage Number of rows per page default 15.
      * @param  string  $sortBy  The name of the column to sort by.
-     * @param  boolean $desc    Sort ascending or descinding (1: desc, 0: asc).
-     * @return \Illuminate\Http\Response
+     * @param  integer $desc    Sort ascending or descinding (1: desc, 0: asc).
+     * @return \Illuminate\Http\JsonResponse|null
      */
     public function search($query = '', $perPage = 15, $sortBy = 'created_at', $desc = 1) 
     {
@@ -94,8 +94,8 @@  discard block
 block discarded – undo
      * 
      * @param  \Illuminate\Http\Request  $request
      * @param  string  $sortBy The name of the column to sort by.
-     * @param  boolean $desc   Sort ascending or descinding (1: desc, 0: asc).
-     * @return \Illuminate\Http\Response
+     * @param  integer $desc   Sort ascending or descinding (1: desc, 0: asc).
+     * @return \Illuminate\Http\JsonResponse|null
      */
     public function findby(Request $request, $sortBy = 'created_at', $desc = 1) 
     {
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
      * condition.
      * 
      * @param  \Illuminate\Http\Request  $request
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\JsonResponse|null
      */
     public function first(Request $request) 
     {
@@ -125,8 +125,8 @@  discard block
 block discarded – undo
      * 
      * @param  integer $perPage Number of rows per page default 15.
      * @param  string  $sortBy  The name of the column to sort by.
-     * @param  boolean $desc    Sort ascending or descinding (1: desc, 0: asc).
-     * @return \Illuminate\Http\Response
+     * @param  integer $desc    Sort ascending or descinding (1: desc, 0: asc).
+     * @return \Illuminate\Http\JsonResponse|null
      */
     public function paginate($perPage = 15, $sortBy = 'created_at', $desc = 1) 
     {
@@ -143,8 +143,8 @@  discard block
 block discarded – undo
      * @param  \Illuminate\Http\Request  $request
      * @param  integer $perPage Number of rows per page default 15.
      * @param  string  $sortBy  The name of the column to sort by.
-     * @param  boolean $desc    Sort ascending or descinding (1: desc, 0: asc).
-     * @return \Illuminate\Http\Response
+     * @param  integer $desc    Sort ascending or descinding (1: desc, 0: asc).
+     * @return \Illuminate\Http\JsonResponse|null
      */
     public function paginateby(Request $request, $perPage = 15, $sortBy = 'created_at', $desc = 1) 
     {
@@ -158,7 +158,7 @@  discard block
 block discarded – undo
      * Save the given model to storage.
      * 
      * @param  \Illuminate\Http\Request  $request
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\JsonResponse|null
      */
     public function save(Request $request) 
     {
@@ -191,7 +191,7 @@  discard block
 block discarded – undo
      * Delete by the given id from storage.
      * 
      * @param  integer $id Id of the deleted model.
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\JsonResponse|null
      */
     public function delete($id) 
     {
@@ -207,8 +207,8 @@  discard block
 block discarded – undo
      * @param  \Illuminate\Http\Request  $request
      * @param  integer $perPage Number of rows per page default 15.
      * @param  string  $sortBy  The name of the column to sort by.
-     * @param  boolean $desc    Sort ascending or descinding (1: desc, 0: asc).
-     * @return \Illuminate\Http\Response
+     * @param  integer $desc    Sort ascending or descinding (1: desc, 0: asc).
+     * @return \Illuminate\Http\JsonResponse
      */
     public function deleted(Request $request, $perPage = 15, $sortBy = 'created_at', $desc = 1) 
     {
@@ -219,7 +219,7 @@  discard block
 block discarded – undo
      * Restore the deleted model.
      * 
      * @param  integer $id Id of the restored model.
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\JsonResponse|null
      */
     public function restore($id) 
     {

Please login to merge, or discard this patch.

src/Modules/V1/Core/AbstractRepositories/AbstractRepository.php 1 patch

Doc Comments +7 added lines, -7 removed lines patch added patch discarded remove patch

@@ -24,7 +24,7 @@  discard block
 block discarded – undo
      *
      * @param  array   $relations
      * @param  string  $sortBy
-     * @param  boolean $desc
+     * @param  integer $desc
      * @param  array   $columns
      * @return collection
      */
@@ -42,7 +42,7 @@  discard block
 block discarded – undo
      * @param  integer $perPage
      * @param  array   $relations
      * @param  string  $sortBy
-     * @param  boolean $desc
+     * @param  integer $desc
      * @param  array   $columns
      * @return collection
      */
@@ -131,7 +131,7 @@  discard block
 block discarded – undo
      * @param  integer $perPage
      * @param  array   $relations
      * @param  string  $sortBy
-     * @param  boolean $desc
+     * @param  integer $desc
      * @param  array   $columns
      * @return collection
      */
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
      * @param  integer $perPage
      * @param  array   $relations
      * @param  string  $sortBy
-     * @param  boolean $desc
+     * @param  integer $desc
      * @param  array   $columns
      * @return collection
      */
@@ -412,7 +412,7 @@  discard block
 block discarded – undo
      * Update record in the storage based on the given
      * condition.
      * 
-     * @param  var $value condition value
+     * @param  boolean $value condition value
      * @param  array $data
      * @param  string $attribute condition column name
      * @return void
@@ -489,7 +489,7 @@  discard block
 block discarded – undo
      * @param  array   $conditions array of conditions
      * @param  array   $relations
      * @param  string  $sortBy
-     * @param  boolean $desc
+     * @param  integer $desc
      * @param  array   $columns
      * @return collection
      */
@@ -521,7 +521,7 @@  discard block
 block discarded – undo
      * @param  array   $conditions array of conditions
      * @param  integer $perPage
      * @param  string  $sortBy
-     * @param  boolean $desc
+     * @param  integer $desc
      * @param  array   $columns
      * @return collection
      */

Please login to merge, or discard this patch.

		@@ -91,7 +91,7 @@ discard block
		block discarded – undo
91	91	* Generate headers for the given route.
92	92	*
93	93	* @param array &$route
94		- * @param object $reflectionClass
	94	+ * @param \ReflectionClass $reflectionClass
95	95	* @param string $method
96	96	* @param array $skipLoginCheck
97	97	* @return void
		@@ -118,6 +118,7 @@ discard block
		block discarded – undo
118	118	*
119	119	* @param array &$route
120	120	* @param object $reflectionMethod]
	121	+ * @param \ReflectionMethod $reflectionMethod
121	122	* @return void
122	123	*/
123	124	protected function processDocBlock(&$route, $reflectionMethod)
		@@ -140,7 +141,7 @@ discard block
		block discarded – undo
140	141	* Generate post body for the given route.
141	142	*
142	143	* @param array &$route
143		- * @param object $reflectionMethod
	144	+ * @param \ReflectionMethod $reflectionMethod
144	145	* @param array $validationRules
145	146	* @return void
146	147	*/

		@@ -45,8 +45,8 @@ discard block
		block discarded – undo
45	45	* Fetch all records with relations from storage.
46	46	*
47	47	* @param string $sortBy The name of the column to sort by.
48		- * @param boolean $desc Sort ascending or descinding (1: desc, 0: asc).
49		- * @return \Illuminate\Http\Response
	48	+ * @param integer $desc Sort ascending or descinding (1: desc, 0: asc).
	49	+ * @return \Illuminate\Http\JsonResponse\|null
50	50	*/
51	51	public function index($sortBy = 'created_at', $desc = 1)
52	52	{
		@@ -60,7 +60,7 @@ discard block
		block discarded – undo
60	60	* Fetch the single object with relations from storage.
61	61	*
62	62	* @param integer $id Id of the requested model.
63		- * @return \Illuminate\Http\Response
	63	+ * @return \Illuminate\Http\JsonResponse\|null
64	64	*/
65	65	public function find($id)
66	66	{
		@@ -77,8 +77,8 @@ discard block
		block discarded – undo
77	77	* @param string $query The search text.
78	78	* @param integer $perPage Number of rows per page default 15.
79	79	* @param string $sortBy The name of the column to sort by.
80		- * @param boolean $desc Sort ascending or descinding (1: desc, 0: asc).
81		- * @return \Illuminate\Http\Response
	80	+ * @param integer $desc Sort ascending or descinding (1: desc, 0: asc).
	81	+ * @return \Illuminate\Http\JsonResponse\|null
82	82	*/
83	83	public function search($query = '', $perPage = 15, $sortBy = 'created_at', $desc = 1)
84	84	{
		@@ -94,8 +94,8 @@ discard block
		block discarded – undo
94	94	*
95	95	* @param \Illuminate\Http\Request $request
96	96	* @param string $sortBy The name of the column to sort by.
97		- * @param boolean $desc Sort ascending or descinding (1: desc, 0: asc).
98		- * @return \Illuminate\Http\Response
	97	+ * @param integer $desc Sort ascending or descinding (1: desc, 0: asc).
	98	+ * @return \Illuminate\Http\JsonResponse\|null
99	99	*/
100	100	public function findby(Request $request, $sortBy = 'created_at', $desc = 1)
101	101	{
		@@ -110,7 +110,7 @@ discard block
		block discarded – undo
110	110	* condition.
111	111	*
112	112	* @param \Illuminate\Http\Request $request
113		- * @return \Illuminate\Http\Response
	113	+ * @return \Illuminate\Http\JsonResponse\|null
114	114	*/
115	115	public function first(Request $request)
116	116	{
		@@ -125,8 +125,8 @@ discard block
		block discarded – undo
125	125	*
126	126	* @param integer $perPage Number of rows per page default 15.
127	127	* @param string $sortBy The name of the column to sort by.
128		- * @param boolean $desc Sort ascending or descinding (1: desc, 0: asc).
129		- * @return \Illuminate\Http\Response
	128	+ * @param integer $desc Sort ascending or descinding (1: desc, 0: asc).
	129	+ * @return \Illuminate\Http\JsonResponse\|null
130	130	*/
131	131	public function paginate($perPage = 15, $sortBy = 'created_at', $desc = 1)
132	132	{
		@@ -143,8 +143,8 @@ discard block
		block discarded – undo
143	143	* @param \Illuminate\Http\Request $request
144	144	* @param integer $perPage Number of rows per page default 15.
145	145	* @param string $sortBy The name of the column to sort by.
146		- * @param boolean $desc Sort ascending or descinding (1: desc, 0: asc).
147		- * @return \Illuminate\Http\Response
	146	+ * @param integer $desc Sort ascending or descinding (1: desc, 0: asc).
	147	+ * @return \Illuminate\Http\JsonResponse\|null
148	148	*/
149	149	public function paginateby(Request $request, $perPage = 15, $sortBy = 'created_at', $desc = 1)
150	150	{
		@@ -158,7 +158,7 @@ discard block
		block discarded – undo
158	158	* Save the given model to storage.
159	159	*
160	160	* @param \Illuminate\Http\Request $request
161		- * @return \Illuminate\Http\Response
	161	+ * @return \Illuminate\Http\JsonResponse\|null
162	162	*/
163	163	public function save(Request $request)
164	164	{
		@@ -191,7 +191,7 @@ discard block
		block discarded – undo
191	191	* Delete by the given id from storage.
192	192	*
193	193	* @param integer $id Id of the deleted model.
194		- * @return \Illuminate\Http\Response
	194	+ * @return \Illuminate\Http\JsonResponse\|null
195	195	*/
196	196	public function delete($id)
197	197	{
		@@ -207,8 +207,8 @@ discard block
		block discarded – undo
207	207	* @param \Illuminate\Http\Request $request
208	208	* @param integer $perPage Number of rows per page default 15.
209	209	* @param string $sortBy The name of the column to sort by.
210		- * @param boolean $desc Sort ascending or descinding (1: desc, 0: asc).
211		- * @return \Illuminate\Http\Response
	210	+ * @param integer $desc Sort ascending or descinding (1: desc, 0: asc).
	211	+ * @return \Illuminate\Http\JsonResponse
212	212	*/
213	213	public function deleted(Request $request, $perPage = 15, $sortBy = 'created_at', $desc = 1)
214	214	{
		@@ -219,7 +219,7 @@ discard block
		block discarded – undo
219	219	* Restore the deleted model.
220	220	*
221	221	* @param integer $id Id of the restored model.
222		- * @return \Illuminate\Http\Response
	222	+ * @return \Illuminate\Http\JsonResponse\|null
223	223	*/
224	224	public function restore($id)
225	225	{

		@@ -24,7 +24,7 @@ discard block
		block discarded – undo
24	24	*
25	25	* @param array $relations
26	26	* @param string $sortBy
27		- * @param boolean $desc
	27	+ * @param integer $desc
28	28	* @param array $columns
29	29	* @return collection
30	30	*/
		@@ -42,7 +42,7 @@ discard block
		block discarded – undo
42	42	* @param integer $perPage
43	43	* @param array $relations
44	44	* @param string $sortBy
45		- * @param boolean $desc
	45	+ * @param integer $desc
46	46	* @param array $columns
47	47	* @return collection
48	48	*/
		@@ -131,7 +131,7 @@ discard block
		block discarded – undo
131	131	* @param integer $perPage
132	132	* @param array $relations
133	133	* @param string $sortBy
134		- * @param boolean $desc
	134	+ * @param integer $desc
135	135	* @param array $columns
136	136	* @return collection
137	137	*/
		@@ -149,7 +149,7 @@ discard block
		block discarded – undo
149	149	* @param integer $perPage
150	150	* @param array $relations
151	151	* @param string $sortBy
152		- * @param boolean $desc
	152	+ * @param integer $desc
153	153	* @param array $columns
154	154	* @return collection
155	155	*/
		@@ -412,7 +412,7 @@ discard block
		block discarded – undo
412	412	* Update record in the storage based on the given
413	413	* condition.
414	414	*
415		- * @param var $value condition value
	415	+ * @param boolean $value condition value
416	416	* @param array $data
417	417	* @param string $attribute condition column name
418	418	* @return void
		@@ -489,7 +489,7 @@ discard block
		block discarded – undo
489	489	* @param array $conditions array of conditions
490	490	* @param array $relations
491	491	* @param string $sortBy
492		- * @param boolean $desc
	492	+ * @param integer $desc
493	493	* @param array $columns
494	494	* @return collection
495	495	*/
		@@ -521,7 +521,7 @@ discard block
		block discarded – undo
521	521	* @param array $conditions array of conditions
522	522	* @param integer $perPage
523	523	* @param string $sortBy
524		- * @param boolean $desc
	524	+ * @param integer $desc
525	525	* @param array $columns
526	526	* @return collection
527	527	*/

SherifElfadaly / Laravel-Api-Skeleton

Push — master ( 8b745b...e40879 )

Status

Category

Doc Comments +3 added lines, -2 removed lines patch added patch discarded remove patch

Doc Comments +17 added lines, -17 removed lines patch added patch discarded remove patch

Doc Comments +7 added lines, -7 removed lines patch added patch discarded remove patch