SherifElfadaly / Laravel-Api-Skeleton

src/Modules/Core/Console/Commands/GenerateDoc.php

Doc Comments +2 added lines, -2 removed lines

@@ -124,7 +124,7 @@ 
 	 * based on the docblock.
 	 * 
 	 * @param  array  &$route
-	 * @param  object $reflectionMethod
+	 * @param  \ReflectionMethod $reflectionMethod
 	 * @return void
 	 */
 	protected function processDocBlock(&$route, $reflectionMethod)
@@ -148,7 +148,7 @@ 
 	 * Generate post body for the given route.
 	 * 
 	 * @param  array  &$route
-	 * @param  object $reflectionMethod
+	 * @param  \ReflectionMethod $reflectionMethod
 	 * @param  array  $validationRules
 	 * @return void
 	 */

Spacing +9 added lines, -9 removed lines

@@ -43,7 +43,7 @@ 
 			if ($route) 
 			{
 				$actoinArray = explode('@', $route['action']);
-				if(array_get($actoinArray, 1, false))
+				if (array_get($actoinArray, 1, false))
 				{
 					$controller       = $actoinArray[0];
 					$method           = $actoinArray[1];
@@ -62,7 +62,7 @@ 
 					$route['response'] = $this->getResponseObject($classProperties['model'], $route['name'], $route['returnDocBlock']);
 
 					preg_match('/api\/([^#]+)\//iU', $route['uri'], $module);
-					$docData['modules'][$module[1]][substr($route['prefix'], strlen('/api/' . $module[1] . '/') - 1)][] = $route;
+					$docData['modules'][$module[1]][substr($route['prefix'], strlen('/api/'.$module[1].'/') - 1)][] = $route;
 
 					$this->getModels($classProperties['model'], $docData);   
 				}
@@ -81,7 +81,7 @@ 
 	 */
 	protected function getRoutes()
 	{
-		return collect(\Route::getRoutes())->map(function ($route) {
+		return collect(\Route::getRoutes())->map(function($route) {
 			if (strpos($route->uri(), 'api') !== false) 
 			{
 				return [
@@ -113,7 +113,7 @@ 
 		];
 
 
-		if (! $skipLoginCheck || ! in_array($method, $skipLoginCheck)) 
+		if ( ! $skipLoginCheck || ! in_array($method, $skipLoginCheck)) 
 		{
 			$route['headers']['Authorization'] = 'Bearer {token}';
 		}
@@ -167,16 +167,16 @@ 
 				}
 				else
 				{
-					$route['body'] = eval('return ' . str_replace(',\'.$request->get(\'id\')', ',{id}\'', $match[1]) . ';');
+					$route['body'] = eval('return '.str_replace(',\'.$request->get(\'id\')', ',{id}\'', $match[1]).';');
 				}
 
 				foreach ($route['body'] as &$rule) 
 				{
-					if(strpos($rule, 'unique'))
+					if (strpos($rule, 'unique'))
 					{
 						$rule = substr($rule, 0, strpos($rule, 'unique') + 6);
 					}
-					elseif(strpos($rule, 'exists'))
+					elseif (strpos($rule, 'exists'))
 					{
 						$rule = substr($rule, 0, strpos($rule, 'exists') - 1);
 					}
@@ -249,10 +249,10 @@ 
 			$model      = factory($modelClass)->make();
 			$modelArr   = $model->toArray();
 
-			if ( $model->trans && ! $model->trans->count()) 
+			if ($model->trans && ! $model->trans->count()) 
 			{
 				$modelArr['trans'] = [
-					'en' => factory($modelClass . 'Translation')->make()->toArray()
+					'en' => factory($modelClass.'Translation')->make()->toArray()
 				];
 			}
 

Braces +3 added lines, -6 removed lines

@@ -164,8 +164,7 @@ 
 				if ($match[1] == '$this->validationRules')
 				{
 					$route['body'] = $validationRules;
-				}
-				else
+				} else
 				{
 					$route['body'] = eval('return ' . str_replace(',\'.$request->get(\'id\')', ',{id}\'', $match[1]) . ';');
 				}
@@ -175,14 +174,12 @@ 
 					if(strpos($rule, 'unique'))
 					{
 						$rule = substr($rule, 0, strpos($rule, 'unique') + 6);
-					}
-					elseif(strpos($rule, 'exists'))
+					} elseif(strpos($rule, 'exists'))
 					{
 						$rule = substr($rule, 0, strpos($rule, 'exists') - 1);
 					}
 				}
-			}
-			else
+			} else
 			{
 				$route['body'] = 'conditions';
 			}

files/Handler.php

Braces +6 added lines, -12 removed lines

@@ -57,29 +57,23 @@
 			if ($exception instanceof \Illuminate\Database\QueryException) 
 			{
 				\ErrorHandler::dbQueryError();
-			}
-			else if ($exception instanceof \predis\connection\connectionexception) 
+			} else if ($exception instanceof \predis\connection\connectionexception) 
 			{
 				\ErrorHandler::redisNotRunning();
-			}
-			else if ($exception instanceof \GuzzleHttp\Exception\ClientException) 
+			} else if ($exception instanceof \GuzzleHttp\Exception\ClientException) 
 			{
 				\ErrorHandler::connectionError();
-			}
-			else if ($exception instanceof \Symfony\Component\HttpKernel\Exception\HttpException) 
+			} else if ($exception instanceof \Symfony\Component\HttpKernel\Exception\HttpException) 
 			{
 				return \Response::json($exception->getMessage(), $exception->getStatusCode());   
-			}
-			else if ($exception instanceof \Illuminate\Validation\ValidationException) 
+			} else if ($exception instanceof \Illuminate\Validation\ValidationException) 
 			{
 				return \Response::json($exception->errors(), 422);   
-			}
-			else if ( ! $exception instanceof \Symfony\Component\Debug\Exception\FatalErrorException)
+			} else if ( ! $exception instanceof \Symfony\Component\Debug\Exception\FatalErrorException)
 			{
 				return parent::render($request, $exception);
 			}
-		}
-		else
+		} else
 		{
 			return parent::render($request, $exception);
 		}

files/AuthServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
 	{
 		$this->registerPolicies();
 
-		Passport::routes(function ($router) {
+		Passport::routes(function($router) {
 			$router->forAuthorization();
 			$router->forAccessTokens();
 			$router->forPersonalAccessTokens();

src/Modules/Reporting/Providers/RouteServiceProvider.php

Spacing +2 added lines, -2 removed lines

@@ -54,7 +54,7 @@ 
 		Route::group([
 			'middleware' => 'web',
 			'namespace'  => $this->namespace,
-		], function ($router) {
+		], function($router) {
 			require module_path('reporting', 'Routes/web.php');
 		});
 	}
@@ -72,7 +72,7 @@ 
 			'middleware' => 'api',
 			'namespace'  => $this->namespace,
 			'prefix'     => 'api',
-		], function ($router) {
+		], function($router) {
 			require module_path('reporting', 'Routes/api.php');
 		});
 	}

src/Modules/Reporting/Report.php

Spacing +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Database\Eloquent\SoftDeletes;
 
-class Report extends Model{
+class Report extends Model {
 
 	use SoftDeletes;
 	protected $table    = 'reports';

src/Modules/Reporting/Repositories/ReportRepository.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
 		{
 			\ErrorHandler::notFound('report');
 		}
-		else if (! $skipPermission && ! \Core::users()->can($report->view_name, 'reports'))
+		else if ( ! $skipPermission && ! \Core::users()->can($report->view_name, 'reports'))
 		{
 			\ErrorHandler::noPermissions();
 		}

Braces +2 added lines, -4 removed lines

@@ -39,8 +39,7 @@ 
 		if ( ! $report) 
 		{
 			\ErrorHandler::notFound('report');
-		}
-		else if (! $skipPermission && ! \Core::users()->can($report->view_name, 'reports'))
+		} else if (! $skipPermission && ! \Core::users()->can($report->view_name, 'reports'))
 		{
 			\ErrorHandler::noPermissions();
 		}
@@ -61,8 +60,7 @@ 
 		if ($perPage) 
 		{
 			return $report->paginate($perPage);
-		}
-		else
+		} else
 		{
 			return $report->get();  
 		}

src/Modules/Reporting/Http/Controllers/ReportsController.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
 	 * to preform actions like (add, edit ... etc).
 	 * @var string
 	 */
-	protected $model               = 'reports';
+	protected $model = 'reports';
 
 	/**
 	 * List of all route actions that the base api controller

src/Modules/Reporting/Database/Factories/ReportFactory.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-$factory->define(App\Modules\Reporting\Report::class, function (Faker\Generator $faker) {
+$factory->define(App\Modules\Reporting\Report::class, function(Faker\Generator $faker) {
 	return [
 		'report_name' => $faker->randomElement(['Users Count', 'Low Stock Products', 'Active Users']),
 		'view_name'   => $faker->word(),

src/Modules/Reporting/Database/Seeds/AssignRelationsSeeder.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
 		/**
 		 * Assign the permissions to the admin group.
 		 */
-		\DB::table('permissions')->orderBy('created_at', 'asc')->whereIn('model', ['reports'])->each(function ($permission) use ($adminGroupId) {
+		\DB::table('permissions')->orderBy('created_at', 'asc')->whereIn('model', ['reports'])->each(function($permission) use ($adminGroupId) {
 			\DB::table('groups_permissions')->insert(
 				[
 				'permission_id' => $permission->id,