SherifElfadaly / Laravel-Api-Skeleton

src/Modules/Acl/AclUser.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 use App\User;
 use Illuminate\Database\Eloquent\SoftDeletes;
-use Illuminate\Notifications\Notifiable;
 use Laravel\Passport\HasApiTokens;
 
 class AclUser extends User {

Braces +1 added lines, -2 removed lines

@@ -67,8 +67,7 @@
                 }
 
                 $tokens[] = $device->device_token;
-            } 
-            catch (\Exception $e) 
+            } catch (\Exception $e) 
             {
                 $device->forceDelete();
             }

Indentation +28 added lines, -28 removed lines

@@ -30,10 +30,10 @@ 
 		return \Carbon\Carbon::parse($value)->tz(\Session::get('time-zone'))->toDateTimeString();
 	}
 
-    /**
-     * Get the profile picture url.
-     * @return string
-     */
+	/**
+	 * Get the profile picture url.
+	 * @return string
+	 */
 	public function getProfilePictureAttribute($value)
 	{
 		return url(\Storage::url($value));
@@ -89,30 +89,30 @@ 
 	 * 
 	 * @return array
 	 */
-    public function routeNotificationForFCM()
-    {
-        $devices = \Core::pushNotificationDevices()->findBy(['user_id' => $this->id]);
-        $tokens  = [];
-
-        foreach ($devices as $device) 
-        {
-            try
-            {
-                if (\Core::users()->accessTokenExpiredOrRevoked($device->access_token)) 
-                {
-                    continue;
-                }
-
-                $tokens[] = $device->device_token;
-            } 
-            catch (\Exception $e) 
-            {
-                $device->forceDelete();
-            }
-        }
-
-        return $tokens;
-    }
+	public function routeNotificationForFCM()
+	{
+		$devices = \Core::pushNotificationDevices()->findBy(['user_id' => $this->id]);
+		$tokens  = [];
+
+		foreach ($devices as $device) 
+		{
+			try
+			{
+				if (\Core::users()->accessTokenExpiredOrRevoked($device->access_token)) 
+				{
+					continue;
+				}
+
+				$tokens[] = $device->device_token;
+			} 
+			catch (\Exception $e) 
+			{
+				$device->forceDelete();
+			}
+		}
+
+		return $tokens;
+	}
 
 	/**
 	 * The channels the user receives notification broadcasts on.

src/Modules/Acl/Repositories/UserRepository.php

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php namespace App\Modules\Acl\Repositories;
 
 use App\Modules\Core\AbstractRepositories\AbstractRepository;
-use Lcobucci\JWT\ValidationData;
 use Illuminate\Support\Arr;
 
 class UserRepository extends AbstractRepository

Indentation +23 added lines, -23 removed lines

@@ -293,14 +293,14 @@ 
 	 */
 	public function confirmEmail($confirmationCode)
 	{
-        if ( ! $user = $this->first(['confirmation_code' => $confirmationCode])) 
-        {
-            \ErrorHandler::invalidConfirmationCode();
-        }
-
-        $user->confirmed         = 1;
-        $user->confirmation_code = null;
-        $user->save();
+		if ( ! $user = $this->first(['confirmation_code' => $confirmationCode])) 
+		{
+			\ErrorHandler::invalidConfirmationCode();
+		}
+
+		$user->confirmed         = 1;
+		$user->confirmation_code = null;
+		$user->save();
 	}
 
 	/**
@@ -375,23 +375,23 @@ 
 		$this->save($data);
 	}
 
-    /**
-     * Ensure access token hasn't expired or revoked.
-     * 
-     * @param  string $accessToken
-     * @return boolean
-     */
-    public function accessTokenExpiredOrRevoked($accessToken)
-    {
-        $accessToken = json_decode($accessToken, true);
+	/**
+	 * Ensure access token hasn't expired or revoked.
+	 * 
+	 * @param  string $accessToken
+	 * @return boolean
+	 */
+	public function accessTokenExpiredOrRevoked($accessToken)
+	{
+		$accessToken = json_decode($accessToken, true);
         
-        if (\Carbon\Carbon::parse($accessToken['expires_at'])->isPast() || $accessToken['revoked']) 
-        {
-            return true;
-        }
+		if (\Carbon\Carbon::parse($accessToken['expires_at'])->isPast() || $accessToken['revoked']) 
+		{
+			return true;
+		}
 
-        return false;
-    }
+		return false;
+	}
 
 	/**
 	 * Revoke the given access token and all 

src/Modules/Acl/Database/Migrations/2015_12_20_124153_users.php

Indentation +16 added lines, -16 removed lines

@@ -13,26 +13,26 @@ 
 	public function up()
 	{
 		Schema::create('users', function(Blueprint $table) {
-            $table->increments('id');
-            $table->string('profile_picture', 150)->nullable();
-            $table->string('name', 100)->nullable();
-            $table->string('email')->unique();
-            $table->string('password', 60)->nullable();
-            $table->boolean('blocked')->default(0);
-            $table->boolean('confirmed')->default(0);
-            $table->string('confirmation_code')->nullable();
-            $table->string('locale', 2)->default('en');
-            $table->string('timezone', 50)->default('Africa/Cairo');
-            $table->softDeletes();
-            $table->rememberToken();
-            $table->timestamps();
-        });
+			$table->increments('id');
+			$table->string('profile_picture', 150)->nullable();
+			$table->string('name', 100)->nullable();
+			$table->string('email')->unique();
+			$table->string('password', 60)->nullable();
+			$table->boolean('blocked')->default(0);
+			$table->boolean('confirmed')->default(0);
+			$table->string('confirmation_code')->nullable();
+			$table->string('locale', 2)->default('en');
+			$table->string('timezone', 50)->default('Africa/Cairo');
+			$table->softDeletes();
+			$table->rememberToken();
+			$table->timestamps();
+		});
 
 		/**
 		 * Create Default users.
 		 */
 		\DB::table('users')->insertGetId(
-            [
+			[
 			'name'       => 'Admin',
 			'email'      => 'admin@user.com',
 			'password'   => bcrypt('123456'),
@@ -40,7 +40,7 @@ 
 			'created_at' => \DB::raw('NOW()'),
 			'updated_at' => \DB::raw('NOW()')
 			]
-        );
+		);
 	}
 
 	/**

src/Modules/Acl/ModelObservers/AclUserObserver.php

Indentation +4 added lines, -4 removed lines

@@ -7,10 +7,10 @@
 
 	public function saving($model)
 	{
-        if ($model->isDirty('profile_picture')) 
-        {
-            \Media::deleteImage($model->getOriginal('profile_picture'));
-        }
+		if ($model->isDirty('profile_picture')) 
+		{
+			\Media::deleteImage($model->getOriginal('profile_picture'));
+		}
 	}
 
 	public function saved($model)

src/Modules/Core/Http/Controllers/BaseApiController.php

Indentation +32 added lines, -32 removed lines

@@ -43,9 +43,9 @@ 
 			return $next($request);
 		});
 
-        $this->setSessions();
-        $this->checkPermission($route);
-        $this->setRelations($route);
+		$this->setSessions();
+		$this->checkPermission($route);
+		$this->setRelations($route);
 	}
 
 	/**
@@ -221,7 +221,7 @@ 
 			$user             = \Auth::user();
 			$permission       = $permission !== 'index' ? $permission : 'list';
 			$isPasswordClient = $user->token()->client->password_client;
-            $this->updateLocaleAndTimezone($user);
+			$this->updateLocaleAndTimezone($user);
 
 			if ($user->blocked)
 			{
@@ -287,32 +287,32 @@ 
 		$this->relations = $relations && isset($relations[$route]) ? $relations[$route] : [];
 	}
 
-    /**
-     * Update the logged in user locale and time zone.
-     * 
-     * @param  object $user
-     * @return void
-     */
-    private function updateLocaleAndTimezone($user)
-    {   
-        $update   = false;
-        $locale   = \Session::get('locale');
-        $timezone = \Session::get('time-zone');
-        if ($locale && $locale !== 'all' && $locale !== $user->locale)
-        {
-            $user->locale = $locale;
-            $update       = true;
-        }
-
-        if ($timezone && $timezone !== $user->timezone)
-        {
-            $user->timezone = $timezone;
-            $update       = true;
-        }
-
-        if ($update) 
-        {
-            $user->save();
-        }
-    }
+	/**
+	 * Update the logged in user locale and time zone.
+	 * 
+	 * @param  object $user
+	 * @return void
+	 */
+	private function updateLocaleAndTimezone($user)
+	{   
+		$update   = false;
+		$locale   = \Session::get('locale');
+		$timezone = \Session::get('time-zone');
+		if ($locale && $locale !== 'all' && $locale !== $user->locale)
+		{
+			$user->locale = $locale;
+			$update       = true;
+		}
+
+		if ($timezone && $timezone !== $user->timezone)
+		{
+			$user->timezone = $timezone;
+			$update       = true;
+		}
+
+		if ($update) 
+		{
+			$user->save();
+		}
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -307,7 +307,7 @@
         if ($timezone && $timezone !== $user->timezone)
         {
             $user->timezone = $timezone;
-            $update       = true;
+            $update = true;
         }
 
         if ($update) 

src/Modules/Core/Utl/Media.php

Indentation +53 added lines, -53 removed lines

@@ -2,63 +2,63 @@
 
 class Media
 {
-    /**
-     * Upload the given image.
-     * 
-     * @param  object  $image
-     * @param  string  $dir
-     * @return string
-     */
-    public function uploadImage($image, $dir) 
-    {
-        $image = \Image::make($image);
-        return $this->saveImage($image, $dir);
-    }
+	/**
+	 * Upload the given image.
+	 * 
+	 * @param  object  $image
+	 * @param  string  $dir
+	 * @return string
+	 */
+	public function uploadImage($image, $dir) 
+	{
+		$image = \Image::make($image);
+		return $this->saveImage($image, $dir);
+	}
 
-    /**
-     * Upload the given image.
-     * 
-     * @param  object  $image
-     * @param  string  $dir
-     * @return string
-     */
-    public function uploadImageBas64($image, $dir) 
-    {
-        if ( ! strlen($image)) 
-        {
-            return null;
-        }
+	/**
+	 * Upload the given image.
+	 * 
+	 * @param  object  $image
+	 * @param  string  $dir
+	 * @return string
+	 */
+	public function uploadImageBas64($image, $dir) 
+	{
+		if ( ! strlen($image)) 
+		{
+			return null;
+		}
 
-        $base  = base64_decode(preg_replace('#^data:image/\w+;base64,#i', '', $image));
-        $image = \Image::make($base);
+		$base  = base64_decode(preg_replace('#^data:image/\w+;base64,#i', '', $image));
+		$image = \Image::make($base);
 
-        return $this->saveImage($image, $dir);
-    }
+		return $this->saveImage($image, $dir);
+	}
 
-    /**
-     * Delete the given image.
-     * 
-     * @param  object $path
-     * @return void
-     */
-    public function deleteImage($path) 
-    {   
-        \Storage::delete($path);
-    }
+	/**
+	 * Delete the given image.
+	 * 
+	 * @param  object $path
+	 * @return void
+	 */
+	public function deleteImage($path) 
+	{   
+		\Storage::delete($path);
+	}
 
-    /**
-     * Save the given image.
-     * 
-     * @param  object  $image
-     * @param  string  $dir
-     * @return string
-     */
-    protected function saveImage($image, $dir) 
-    {
-        $imageName = 'image'.uniqid().time().'.jpg';
-        $path      = 'public'.DIRECTORY_SEPARATOR.'uploads'.DIRECTORY_SEPARATOR.$dir.DIRECTORY_SEPARATOR.$imageName;
-        \Storage::put($path, $image->stream());
+	/**
+	 * Save the given image.
+	 * 
+	 * @param  object  $image
+	 * @param  string  $dir
+	 * @return string
+	 */
+	protected function saveImage($image, $dir) 
+	{
+		$imageName = 'image'.uniqid().time().'.jpg';
+		$path      = 'public'.DIRECTORY_SEPARATOR.'uploads'.DIRECTORY_SEPARATOR.$dir.DIRECTORY_SEPARATOR.$imageName;
+		\Storage::put($path, $image->stream());
 
-        return $path;
-    }
+		return $path;
+	}
 }

src/Modules/Core/Utl/ErrorHandler.php

Indentation +5 added lines, -5 removed lines

@@ -104,11 +104,11 @@
 		abort($error['status'], $error['message']);
 	}
 
-    public function invalidConfirmationCode()
-    {
-        $error = ['status' => 400, 'message' => trans('core::errors.invalidConfirmationCode')];
-        abort($error['status'], $error['message']);
-    }
+	public function invalidConfirmationCode()
+	{
+		$error = ['status' => 400, 'message' => trans('core::errors.invalidConfirmationCode')];
+		abort($error['status'], $error['message']);
+	}
 
 	public function notFound($text)
 	{

files/channels.php

Indentation +1 added lines, -1 removed lines

@@ -12,5 +12,5 @@
 */
 
 Broadcast::channel('users.{id}', function ($user, $id) {
-    return (int) $user->id === (int) $id;
+	return (int) $user->id === (int) $id;
 });

Spacing +1 added lines, -1 removed lines

@@ -11,6 +11,6 @@
 |
 */
 
-Broadcast::channel('users.{id}', function ($user, $id) {
+Broadcast::channel('users.{id}', function($user, $id) {
     return (int) $user->id === (int) $id;
 });