SherifElfadaly / Laravel-Api-Skeleton

src/Modules/PushNotificationDevices/Repositories/PushNotificationDeviceRepository.php

Unused Use Statements -3 removed lines

@@ -1,9 +1,6 @@
 <?php namespace App\Modules\PushNotificationDevices\Repositories;
 
 use App\Modules\Core\BaseClasses\BaseRepository;
-use LaravelFCM\Message\OptionsBuilder;
-use LaravelFCM\Message\PayloadDataBuilder;
-use LaravelFCM\Message\PayloadNotificationBuilder;
 use App\Modules\PushNotificationDevices\PushNotificationDevice;
 
 class PushNotificationDeviceRepository extends BaseRepository

src/Modules/Users/Http/Controllers/UserController.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use Illuminate\Http\Request;
 use App\Modules\Core\BaseClasses\BaseApiController;
 use App\Modules\Users\Services\UserService;
-use App\Modules\Users\Proxy\LoginProxy;
 use App\Modules\Core\Http\Resources\General as GeneralResource;
 use Illuminate\Support\Facades\App;
 use App\Modules\Users\Http\Requests\AssignRoles;

Spacing +1 added lines, -1 removed lines

@@ -199,7 +199,7 @@
      */
     public function changePassword(ChangePassword $request)
     {
-        return new GeneralResource($this->service->changePassword($request->get('password') , $request->get('old_password')));
+        return new GeneralResource($this->service->changePassword($request->get('password'), $request->get('old_password')));
     }
 
     /**

src/Modules/Users/Reppsitories/UserRepository.php

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php namespace App\Modules\Users\Repositories;
 
 use App\Modules\Core\BaseClasses\BaseRepository;
-use Illuminate\Support\Arr;
 use App\Modules\Users\AclUser;
 
 class UserRepository extends BaseRepository

src/Modules/Users/ModelObservers/AclUserObserver.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
     public function updated($model)
     {
         if ($model->isDirty('blocked') && $model->blocked) {
-            $model->tokens()->each(function ($token) {
+            $model->tokens()->each(function($token) {
 
                 \Core::oauthClients()->revokeAccessToken($token);
             });

src/Modules/Users/Routes/api.php

Spacing +2 added lines, -2 removed lines

@@ -13,7 +13,7 @@ 
 |
 */
 
-Route::group(['prefix' => 'users'], function () {
+Route::group(['prefix' => 'users'], function() {
 
     Route::get('/', 'UserController@index');
     Route::get('/{id}', 'UserController@find');
@@ -26,7 +26,7 @@ 
     Route::get('unblock/{id}', 'UserController@unblock');
     Route::post('assign/roles', 'UserController@assignRoles');
 
-    Route::group(['prefix' => 'account'], function () {
+    Route::group(['prefix' => 'account'], function() {
 
         Route::get('my', 'UserController@account');
         Route::get('logout', 'UserController@logout');

src/Modules/Users/Database/Seeds/UsersTableSeeder.php

Spacing +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
          */
         $role = Role::updateOrInsert([
             'name' => 'Admin',
-        ],[
+        ], [
             'created_at' => \DB::raw('NOW()'),
             'updated_at' => \DB::raw('NOW()')
         ]);
@@ -30,7 +30,7 @@ 
          */
         AclUser::updateOrInsert([
             'email' => 'admin@user.com',
-        ],[
+        ], [
             'name'       => 'Admin',
             'password'   => \Hash::make('123456'),
             'confirmed'  => 1,

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

Spacing +1 added lines, -1 removed lines

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

src/Modules/Users/Services/UserService.php

Spacing +12 added lines, -12 removed lines

@@ -74,7 +74,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->repo][$permission->id] = $permission->name;
             });
         }
@@ -137,7 +137,7 @@ 
     public function assignRoles($userId, $roleIds)
     {
         $user = false;
-        \DB::transaction(function () use ($userId, $permissionIds, &$user) {
+        \DB::transaction(function() use ($userId, $permissionIds, &$user) {
             $user = $this->repo->find($userId);
             $this->repo->detachPermissions($userId);
             $this->repo->attachPermissions($userId, $roleIds);
@@ -156,15 +156,15 @@ 
      */
     public function login($email, $password, $adminLogin = false)
     {
-        if (! $user = $this->repo->first(['email' => $email])) {
+        if ( ! $user = $this->repo->first(['email' => $email])) {
             \ErrorHandler::loginFailed();
         } elseif ($adminLogin && ! $this->hasRoles(['Admin'], $user)) {
             \ErrorHandler::loginFailed();
-        } elseif (! $adminLogin && $this->hasRoles(['Admin'], $user)) {
+        } elseif ( ! $adminLogin && $this->hasRoles(['Admin'], $user)) {
             \ErrorHandler::loginFailed();
         } elseif ($user->blocked) {
             \ErrorHandler::userIsBlocked();
-        } elseif (! config('skeleton.disable_confirm_email') && ! $user->confirmed) {
+        } elseif ( ! config('skeleton.disable_confirm_email') && ! $user->confirmed) {
             \ErrorHandler::emailNotConfirmed();
         }
 
@@ -184,11 +184,11 @@ 
         $access_token = $authCode ? Arr::get(\Socialite::driver($type)->getAccessTokenResponse($authCode), 'access_token') : $accessToken;
         $user         = \Socialite::driver($type)->userFromToken($access_token);
 
-        if (! $user->email) {
+        if ( ! $user->email) {
             \ErrorHandler::noSocialEmail();
         }
 
-        if (! $this->repo->first(['email' => $user->email])) {
+        if ( ! $this->repo->first(['email' => $user->email])) {
             $this->register(['email' => $user->email, 'password' => ''], true);
         }
 
@@ -213,7 +213,7 @@ 
             'confirmed' => $skipConfirmEmail
         ]);
 
-        if (! $skipConfirmEmail && ! config('skeleton.disable_confirm_email')) {
+        if ( ! $skipConfirmEmail && ! config('skeleton.disable_confirm_email')) {
             $this->sendConfirmationEmail($user->email);
         }
 
@@ -254,7 +254,7 @@ 
      */
     public function sendReset($email)
     {
-        if (! $user = $this->repo->first(['email' => $email])) {
+        if ( ! $user = $this->repo->first(['email' => $email])) {
             \ErrorHandler::notFound('email');
         }
 
@@ -278,7 +278,7 @@ 
             'password'              => $password, 
             'password_confirmation' => $passwordConfirmation, 
             'token'                 => $token
-        ], function ($user, $password) {
+        ], function($user, $password) {
             $this->repo->save(['id' => $user->id, 'password' => $password]);
         });
 
@@ -315,7 +315,7 @@ 
     public function changePassword($password, $oldPassword)
     {
         $user = \Auth::user();
-        if (! \Hash::check($oldPassword, $user->password)) {
+        if ( ! \Hash::check($oldPassword, $user->password)) {
             \ErrorHandler::invalidOldPassword();
         }
 
@@ -330,7 +330,7 @@ 
      */
     public function confirmEmail($confirmationCode)
     {
-        if (! $user = $this->repo->first(['confirmation_code' => $confirmationCode])) {
+        if ( ! $user = $this->repo->first(['confirmation_code' => $confirmationCode])) {
             \ErrorHandler::invalidConfirmationCode();
         }
 

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

Spacing +1 added lines, -1 removed lines

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