SherifElfadaly / Laravel-Api-Skeleton

src/Modules/Users/Providers/ModuleServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
         $this->loadViewsFrom(__DIR__.'/../Resources/Views', 'users');
 
         $this->loadMigrationsFrom(module_path('users', 'Database/Migrations', 'app'));
-        if (!$this->app->configurationIsCached()) {
+        if ( ! $this->app->configurationIsCached()) {
             $this->loadConfigsFrom(module_path('users', 'Config', 'app'));
         }
     }

src/Modules/Users/Http/Resources/AclUser.php

Spacing +2 added lines, -2 removed lines

@@ -18,7 +18,7 @@ 
      */
     public function toArray($request)
     {
-        if (! $this->resource) {
+        if ( ! $this->resource) {
             return [];
         }
 
@@ -31,7 +31,7 @@ 
             'readNotifications' => NotificationResource::collection($this->whenLoaded('readNotifications')),
             'unreadNotifications' => NotificationResource::collection($this->whenLoaded('unreadNotifications')),
             'roles' => RoleResource::collection($this->whenLoaded('roles')),
-            'permissions' => $this->when($this->permissions, function () {
+            'permissions' => $this->when($this->permissions, function() {
                 return PermissionResource::collection($this->permissions);
             }),
             'oauthClients' => OauthClientResource::collection($this->whenLoaded('oauthClients')),

src/Modules/Users/Services/UserService.php

Spacing +12 added lines, -12 removed lines

@@ -67,7 +67,7 @@ 
         $permissions = [];
         $user        = $this->repo->find(\Auth::id(), $relations);
         foreach ($user->roles as $role) {
-            $role->permissions->each(function ($permission) use (&$permissions) {
+            $role->permissions->each(function($permission) use (&$permissions) {
                 $permissions[] = $permission;
             });
         }
@@ -130,7 +130,7 @@ 
     public function assignRoles($userId, $roleIds)
     {
         $user = false;
-        \DB::transaction(function () use ($userId, $roleIds, &$user) {
+        \DB::transaction(function() use ($userId, $roleIds, &$user) {
             $user = $this->repo->find($userId);
             $this->repo->detachRoles($user);
             $this->repo->attachRoles($user, $roleIds);
@@ -148,11 +148,11 @@ 
      */
     public function login($email, $password)
     {
-        if (! $user = $this->repo->first(['email' => $email])) {
+        if ( ! $user = $this->repo->first(['email' => $email])) {
             \Errors::loginFailed();
         } elseif ($user->blocked) {
             \Errors::userIsBlocked();
-        } elseif (! config('user.disable_confirm_email') && ! $user->confirmed) {
+        } elseif ( ! config('user.disable_confirm_email') && ! $user->confirmed) {
             \Errors::emailNotConfirmed();
         }
 
@@ -171,11 +171,11 @@ 
         $accessToken = $authCode ? Arr::get(\Socialite::driver($type)->getAccessTokenResponse($authCode), 'access_token') : $accessToken;
         $user        = \Socialite::driver($type)->userFromToken($accessToken)->user;
 
-        if (! Arr::has($user, 'email')) {
+        if ( ! Arr::has($user, 'email')) {
             \Errors::noSocialEmail();
         }
 
-        if (! $this->repo->first(['email' => $user['email']]) && ! $this->repo->deleted(['email' => $user['email']])->count()) {
+        if ( ! $this->repo->first(['email' => $user['email']]) && ! $this->repo->deleted(['email' => $user['email']])->count()) {
             $this->register(Arr::get($user, 'name'), $user['email'], '', true);
         }
 
@@ -193,13 +193,13 @@ 
     {
         $data['confirmed'] = $skipConfirmEmail;
 
-        if($roleId){
+        if ($roleId) {
             $data ['roles'] = [['id' => $roleId]];
         }
 
         $user = $this->repo->save($data);
 
-        if (! $skipConfirmEmail && ! config('user.disable_confirm_email')) {
+        if ( ! $skipConfirmEmail && ! config('user.disable_confirm_email')) {
             $this->sendConfirmationEmail($user->email);
         }
 
@@ -240,7 +240,7 @@ 
      */
     public function sendReset($email)
     {
-        if (! $user = $this->repo->first(['email' => $email])) {
+        if ( ! $user = $this->repo->first(['email' => $email])) {
             \Errors::notFound('email');
         }
 
@@ -264,7 +264,7 @@ 
             'password'              => $password,
             'password_confirmation' => $passwordConfirmation,
             'token'                 => $token
-        ], function ($user, $password) {
+        ], function($user, $password) {
             $this->repo->save(['id' => $user->id, 'password' => $password]);
         });
 
@@ -297,7 +297,7 @@ 
     public function changePassword($password, $oldPassword)
     {
         $user = \Auth::user();
-        if (! \Hash::check($oldPassword, $user->password)) {
+        if ( ! \Hash::check($oldPassword, $user->password)) {
             \Errors::invalidOldPassword();
         }
 
@@ -312,7 +312,7 @@ 
      */
     public function confirmEmail($confirmationCode)
     {
-        if (! $user = $this->repo->first(['confirmation_code' => $confirmationCode])) {
+        if ( ! $user = $this->repo->first(['confirmation_code' => $confirmationCode])) {
             \Errors::invalidConfirmationCode();
         }
 

src/Modules/Users/Routes/web.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 |
 */
 
-Route::group(['prefix' => 'users'], function () {
+Route::group(['prefix' => 'users'], function() {
 
     Route::get('/confirm/email/{token}', 'UserController@confirmEmailPage')->name('confirm_email_page');
     Route::get('/reset/password/{token}', 'UserController@resetPasswordPage')->name('reset_password_page');

src/Modules/Permissions/Providers/ModuleServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
         $this->loadViewsFrom(__DIR__.'/../Resources/Views', 'permissions');
 
         $this->loadMigrationsFrom(module_path('permissions', 'Database/Migrations', 'app'));
-        if (!$this->app->configurationIsCached()) {
+        if ( ! $this->app->configurationIsCached()) {
             $this->loadConfigsFrom(module_path('permissions', 'Config', 'app'));
         }
     }

src/Modules/Core/Providers/ModuleServiceProvider.php

Spacing +5 added lines, -5 removed lines

@@ -17,7 +17,7 @@ 
         $this->loadViewsFrom(__DIR__.'/../Resources/Views', 'core');
 
         $this->loadMigrationsFrom(module_path('core', 'Database/Migrations', 'app'));
-        if (!$this->app->configurationIsCached()) {
+        if ( ! $this->app->configurationIsCached()) {
             $this->loadConfigsFrom(module_path('core', 'Config', 'app'));
         }
     }
@@ -30,22 +30,22 @@ 
     public function register()
     {
         //Bind Core Facade to the Service Container
-        $this->app->singleton('Core', function () {
+        $this->app->singleton('Core', function() {
             return new \App\Modules\Core\Core;
         });
 
         //Bind Errors Facade to the Service Container
-        $this->app->singleton('Errors', function () {
+        $this->app->singleton('Errors', function() {
             return new \App\Modules\Core\Errors\Errors;
         });
 
         //Bind Media Facade to the Service Container
-        $this->app->singleton('Media', function () {
+        $this->app->singleton('Media', function() {
             return new \App\Modules\Core\Utl\Media;
         });
 
         //Bind ApiConsumer Facade to the Service Container
-        $this->app->singleton('ApiConsumer', function () {
+        $this->app->singleton('ApiConsumer', function() {
             $app = app();
             return new \App\Modules\Core\Utl\ApiConsumer($app, $app['request'], $app['router']);
         });

src/Modules/Core/Http/Middleware/CheckPermissions.php

Spacing +3 added lines, -3 removed lines

@@ -56,8 +56,8 @@ 
         $permission          = $routeActions[1];
 
         $this->auth->shouldUse('api');
-        if (! in_array($permission, $skipLoginCheck)) {
-            $this->authMiddleware->handle($request, function ($request) use ($modelName, $skipPermissionCheck, $permission) {
+        if ( ! in_array($permission, $skipLoginCheck)) {
+            $this->authMiddleware->handle($request, function($request) use ($modelName, $skipPermissionCheck, $permission) {
                 $user             = $this->auth->user();
                 $isPasswordClient = $user->token() ? $user->token()->client->password_client : false;
     
@@ -66,7 +66,7 @@ 
                 }
     
                 if ($isPasswordClient && (in_array($permission, $skipPermissionCheck) || $this->userService->can($permission, $modelName))) {
-                } elseif (! $isPasswordClient && $user->tokenCan($modelName.'-'.$permission)) {
+                } elseif ( ! $isPasswordClient && $user->tokenCan($modelName.'-'.$permission)) {
                 } else {
                     \Errors::noPermissions();
                 }

src/Modules/Core/BaseClasses/BaseEnum.php

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
     {
         $stringArr = [];
         collect(self::all())->each(function($item, $key) use (&$stringArr) {
-            $stringArr[] = $key . ': ' . $item;
+            $stringArr[] = $key.': '.$item;
         });
 
         return implode(',', $stringArr);

src/Modules/Core/Decorators/CachingDecorator.php

Spacing +2 added lines, -2 removed lines

@@ -58,8 +58,8 @@
 
         if ($this->cacheConfig && $this->cacheConfig == 'cache') {
             $page     = \Request::get('page') !== null ? \Request::get('page') : '1';
-            $cacheKey = $name . $page . \Session::get('locale') . serialize($arguments);
-            return $this->cache->tags([$this->cacheTag])->rememberForever($cacheKey, function () use ($arguments, $name) {
+            $cacheKey = $name.$page.\Session::get('locale').serialize($arguments);
+            return $this->cache->tags([$this->cacheTag])->rememberForever($cacheKey, function() use ($arguments, $name) {
                 return call_user_func_array([$this->service, $name], $arguments);
             });
         } elseif ($this->cacheConfig) {