nyacide / amma

app/Services/ImageProcessor.php

Doc Comments +9 added lines, -9 removed lines

@@ -32,8 +32,8 @@ 
      * @param $image
      * @param $imageable
      * @param null $data
-     * @param null $location
-     * @return static
+     * @param string $location
+     * @return Image|null
      * @throws Exception
      */
     public function uploadAndCreate($image, $imageable, $data = null, $location = null)
@@ -55,11 +55,11 @@ 
     /**
      * Download image to server and get as UploadedFile object.
      *
-     * @param $url
+     * @param string $url
      * @param $imageable
      * @param array|null $data
      * @param string $location
-     * @return static
+     * @return Image
      */
     public function downloadAndUpload($url, $imageable, array $data = null, $location = 'upload/images/')
     {
@@ -144,7 +144,7 @@ 
     /**
      * Set location.
      *
-     * @param $location
+     * @param string $location
      * @return $this
      */
     public function setLocation($location)
@@ -155,7 +155,7 @@ 
     }
 
     /**
-     * @param $location
+     * @param string $location
      * @throws Exception
      */
     protected function validateLocation($location)
@@ -212,9 +212,9 @@ 
     /**
      * Standart pattern.
      *
-     * @param $location
-     * @param $filename
-     * @param $extension
+     * @param string $location
+     * @param string $filename
+     * @param string|null $extension
      * @return string
      */
     private function pattern($filename, $extension, $location = null)

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use App\Image;
 use Exception;
-use Illuminate\Database\Eloquent\Model;
 use Illuminate\Http\UploadedFile;
 
 class ImageProcessor

Spacing +16 added lines, -16 removed lines

@@ -38,12 +38,12 @@ 
      */
     public function uploadAndCreate($image, $imageable, $data = null, $location = null)
     {
-        if($this->validateImage($image)) {
+        if ($this->validateImage($image)) {
             $original = $image->getClientOriginalName();
             if (isset($location))
                 $this->setLocation($location);
 
-            $data['original'] = $original;
+            $data[ 'original' ] = $original;
 
             if ($imageInfo = $this->upload($image))
             {
@@ -64,17 +64,17 @@ 
     public function downloadAndUpload($url, $imageable, array $data = null, $location = 'upload/images/')
     {
         $file = file_get_contents($url); // download the image from url.
-        $extension = $this->parseExtension(getimagesize($url)['mime']);
+        $extension = $this->parseExtension(getimagesize($url)[ 'mime' ]);
 
         if (isset($location))
             $this->setLocation($location);
-        $filename = time() .'_'. str_random(15);
+        $filename = time().'_'.str_random(15);
 
         $path_to_image = $this->pattern($filename, $extension, $location);
         file_put_contents($path_to_image, $file);
 
         $imageInfo = new UploadedFile($path_to_image, str_random(10));
-        $data = array_merge(['original' => $filename], $data);
+        $data = array_merge([ 'original' => $filename ], $data);
 
         return $this->create($imageInfo->getRealPath(), $imageable, $data);
     }
@@ -117,7 +117,7 @@ 
      */
     public function destroy($image)
     {
-        if($image) {
+        if ($image) {
             $location = ltrim($image->image, '/');
             @unlink(base_path("public/{$location}"));
 
@@ -133,7 +133,7 @@ 
      */
     public function destroyImageOnly($image)
     {
-        if($image) {
+        if ($image) {
             $location = ltrim($image->image, '/');
             return @unlink(base_path("public/{$location}"));
         }
@@ -162,20 +162,20 @@ 
     {
         $fileExists = file_exists($location);
 
-        if ((! $fileExists && ! mkdir($location, 0777, true)) || ! is_writeable($location)) {
+        if ((!$fileExists && !mkdir($location, 0777, true)) || !is_writeable($location)) {
             throw new Exception("Location [$location] not exists or not writable");
         }
     }
 
     protected function upload($image)
     {
-        $location = base_path("public/" . $this->location);
+        $location = base_path("public/".$this->location);
 
         $this->validateLocation($location);
 
         if ($image) {
             if ($image->isValid()) {
-                $filename = rtrim($location, '/') . '/' . ltrim($this->getNewImageName($image), '/');
+                $filename = rtrim($location, '/').'/'.ltrim($this->getNewImageName($image), '/');
 
                 $this->filename = $filename;
 
@@ -198,13 +198,13 @@ 
         if ($old_avatar = \Auth::user()->images()->avatar()->first())
             $this->destroy($old_avatar);
 
-        if(is_string($avatar)) { // check if url.
-            $this->downloadAndUpload($avatar, \Auth::user(), ['type' => 'avatar'], $location);
+        if (is_string($avatar)) { // check if url.
+            $this->downloadAndUpload($avatar, \Auth::user(), [ 'type' => 'avatar' ], $location);
 
             return $this;
         }
 
-        $this->uploadAndCreate($avatar, \Auth::user(), ['type' => 'avatar'], $location);
+        $this->uploadAndCreate($avatar, \Auth::user(), [ 'type' => 'avatar' ], $location);
 
         return $this;
     }
@@ -219,7 +219,7 @@ 
      */
     private function pattern($filename, $extension, $location = null)
     {
-        if(! $location)
+        if (!$location)
             return sprintf('%s.%s', $filename, $extension);
 
         return sprintf('%s%s.%s', $location, $filename, $extension);
@@ -258,8 +258,8 @@ 
         return $this->getModel()->create([
             'imageable_id' => $imageable->id,
             'imageable_type' => get_class($imageable),
-            'type' => isset($data['type']) ? $data['type'] : 'cover',
-            'original' => isset($data['original']) ? $data['original'] : str_random(13),
+            'type' => isset($data[ 'type' ]) ? $data[ 'type' ] : 'cover',
+            'original' => isset($data[ 'original' ]) ? $data[ 'original' ] : str_random(13),
             'image' => str_replace(base_path('public'), '', $path),
         ]);
     }

Braces +12 added lines, -8 removed lines

@@ -40,8 +40,9 @@ 
     {
         if($this->validateImage($image)) {
             $original = $image->getClientOriginalName();
-            if (isset($location))
-                $this->setLocation($location);
+            if (isset($location)) {
+                            $this->setLocation($location);
+            }
 
             $data['original'] = $original;
 
@@ -66,8 +67,9 @@ 
         $file = file_get_contents($url); // download the image from url.
         $extension = $this->parseExtension(getimagesize($url)['mime']);
 
-        if (isset($location))
-            $this->setLocation($location);
+        if (isset($location)) {
+                    $this->setLocation($location);
+        }
         $filename = time() .'_'. str_random(15);
 
         $path_to_image = $this->pattern($filename, $extension, $location);
@@ -195,8 +197,9 @@ 
      */
     public function changeAvatar($avatar, $location = 'upload/images/user_avatars/')
     {
-        if ($old_avatar = \Auth::user()->images()->avatar()->first())
-            $this->destroy($old_avatar);
+        if ($old_avatar = \Auth::user()->images()->avatar()->first()) {
+                    $this->destroy($old_avatar);
+        }
 
         if(is_string($avatar)) { // check if url.
             $this->downloadAndUpload($avatar, \Auth::user(), ['type' => 'avatar'], $location);
@@ -219,8 +222,9 @@ 
      */
     private function pattern($filename, $extension, $location = null)
     {
-        if(! $location)
-            return sprintf('%s.%s', $filename, $extension);
+        if(! $location) {
+                    return sprintf('%s.%s', $filename, $extension);
+        }
 
         return sprintf('%s%s.%s', $location, $filename, $extension);
     }

app/Services/SocialiteUser.php

Doc Comments +2 added lines, -2 removed lines

@@ -70,7 +70,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     public function getProvider()
     {
@@ -229,7 +229,7 @@ 
     /**
      * Login the user.
      *
-     * @param $user
+     * @param \App\User $user
      */
     public function login($user)
     {

Spacing +9 added lines, -9 removed lines

@@ -55,7 +55,7 @@ 
                 $this->getProvider(), $this->user()->getId()
             );
 
-            if($user = $s_user->user)
+            if ($user = $s_user->user)
                 return $user;
 
             return $this;
@@ -63,7 +63,7 @@ 
 
         $social = $this->getOrCreateEmptySocialUser();
 
-        if($this->checkEmail())
+        if ($this->checkEmail())
             return $this->createSocialUserWithEmail($social);
 
         return $this;
@@ -126,7 +126,7 @@ 
      */
     protected function getOrCreateEmptySocialUser()
     {
-        if($this->socialRepository->checkProviderUser($this->getProvider(), $this->user()->getId()))
+        if ($this->socialRepository->checkProviderUser($this->getProvider(), $this->user()->getId()))
             return $this->socialRepository->getUserByProvider($this->getProvider(), $this->user()->getId());
 
         return $this->socialRepository->createEmpty(
@@ -159,7 +159,7 @@ 
      */
     private function associateSocialiteUser($account, $email = null)
     {
-        if(! $email)
+        if (!$email)
             $email = $this->user()->getEmail();
 
         $user = $this->getUserRepository()->getByEmail($email);
@@ -185,10 +185,10 @@ 
      */
     public function tryToAssociateUser($social, $email = null)
     {
-        if(! $email)
+        if (!$email)
             $email = $this->user()->getEmail();
 
-        if($this->getUserRepository()->checkIfUserExists($email))
+        if ($this->getUserRepository()->checkIfUserExists($email))
             return $this->associateSocialiteUser($social, $email);
 
         return false;
@@ -209,8 +209,8 @@ 
             'email' => $this->user()->getEmail(),
             'name' => $this->user()->getName(),
             'password' => $this->users->hashPassword(str_random(45)),
-            'firstname' => @$names[0],
-            'lastname' => @$names[1]
+            'firstname' => @$names[ 0 ],
+            'lastname' => @$names[ 1 ]
         ]);
     }
 
@@ -220,7 +220,7 @@ 
      */
     public function getFirstAndLustNames($name = null)
     {
-        if(! $name)
+        if (!$name)
             $name = $this->user()->getName();
 
         return explode(' ', $name);

Braces +21 added lines, -14 removed lines

@@ -55,16 +55,18 @@ 
                 $this->getProvider(), $this->user()->getId()
             );
 
-            if($user = $s_user->user)
-                return $user;
+            if($user = $s_user->user) {
+                            return $user;
+            }
 
             return $this;
         }
 
         $social = $this->getOrCreateEmptySocialUser();
 
-        if($this->checkEmail())
-            return $this->createSocialUserWithEmail($social);
+        if($this->checkEmail()) {
+                    return $this->createSocialUserWithEmail($social);
+        }
 
         return $this;
     }
@@ -126,8 +128,9 @@ 
      */
     protected function getOrCreateEmptySocialUser()
     {
-        if($this->socialRepository->checkProviderUser($this->getProvider(), $this->user()->getId()))
-            return $this->socialRepository->getUserByProvider($this->getProvider(), $this->user()->getId());
+        if($this->socialRepository->checkProviderUser($this->getProvider(), $this->user()->getId())) {
+                    return $this->socialRepository->getUserByProvider($this->getProvider(), $this->user()->getId());
+        }
 
         return $this->socialRepository->createEmpty(
             $this->getProvider(), $this->user()
@@ -159,8 +162,9 @@ 
      */
     private function associateSocialiteUser($account, $email = null)
     {
-        if(! $email)
-            $email = $this->user()->getEmail();
+        if(! $email) {
+                    $email = $this->user()->getEmail();
+        }
 
         $user = $this->getUserRepository()->getByEmail($email);
         $account->user()->associate($user);
@@ -185,11 +189,13 @@ 
      */
     public function tryToAssociateUser($social, $email = null)
     {
-        if(! $email)
-            $email = $this->user()->getEmail();
+        if(! $email) {
+                    $email = $this->user()->getEmail();
+        }
 
-        if($this->getUserRepository()->checkIfUserExists($email))
-            return $this->associateSocialiteUser($social, $email);
+        if($this->getUserRepository()->checkIfUserExists($email)) {
+                    return $this->associateSocialiteUser($social, $email);
+        }
 
         return false;
     }
@@ -220,8 +226,9 @@ 
      */
     public function getFirstAndLustNames($name = null)
     {
-        if(! $name)
-            $name = $this->user()->getName();
+        if(! $name) {
+                    $name = $this->user()->getName();
+        }
 
         return explode(' ', $name);
     }

app/User.php

Doc Comments +1 added lines, -1 removed lines

@@ -111,7 +111,7 @@
     /**
      * Check if user has wallet.
      *
-     * @return bool
+     * @return boolean|null
      */
     public function haveWallet()
     {

Spacing +2 added lines, -2 removed lines

@@ -88,7 +88,7 @@ 
      */
     public function isSocialite()
     {
-        return (bool)$this->socialite;
+        return (bool) $this->socialite;
     }
 
     /**
@@ -98,7 +98,7 @@ 
      */
     public function checkAvatar()
     {
-        return (bool)$this->images()->avatar()->first();
+        return (bool) $this->images()->avatar()->first();
     }
 
     public function isAdmin()

app/Http/Requests/UpdateUserPassword.php

Spacing +2 added lines, -2 removed lines

@@ -19,12 +19,12 @@
      */
     public function rules()
     {
-        \Validator::extend('old_password', function ($attribute, $value, $parameters, $validator) {
+        \Validator::extend('old_password', function($attribute, $value, $parameters, $validator) {
             return \Hash::check($value, current($parameters));
         });
 
         return [
-            'old_password' => 'required|old_password:' . Auth::user()->password,
+            'old_password' => 'required|old_password:'.Auth::user()->password,
             'password' => 'required|min:6|confirmed',
         ];
     }

app/Http/Requests/VendorUpdateFormRequest.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
     public function rules()
     {
         return [
-            'name' => 'required|min:3|max:250|unique:vendors,name,' . $this->route('vendor')->id,
+            'name' => 'required|min:3|max:250|unique:vendors,name,'.$this->route('vendor')->id,
             'email' => 'email',
             'phone' => 'required|digits:8',
             'description' => 'min:10|max:250'

app/Http/Requests/ResendConfirmationRequest.php

Spacing +2 added lines, -2 removed lines

@@ -11,7 +11,7 @@ 
      */
     public function authorize()
     {
-        if(! Auth::user()->confirmed)
+        if (!Auth::user()->confirmed)
             return true;
         
         return false;
@@ -22,6 +22,6 @@ 
      */
     public function rules()
     {
-        return [];
+        return [ ];
     }
 }
\ No newline at end of file

Braces +3 added lines, -2 removed lines

@@ -11,8 +11,9 @@
      */
     public function authorize()
     {
-        if(! Auth::user()->confirmed)
-            return true;
+        if(! Auth::user()->confirmed) {
+                    return true;
+        }
         
         return false;
     }

app/Http/Requests/GetEmailFormRequest.php

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
      */
     public function authorize()
     {
-        return (bool)!$this->route()->getParameter('social')->user;
+        return (bool) !$this->route()->getParameter('social')->user;
     }
 
     /**

app/Http/Requests/UpdateUserSettings.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
             'fname' => 'required|min:3|max:20',
             'lname' => 'required|min:3|max:20',
             'phone' => 'required|digits:8',
-            'email' => 'required|unique:users,email,' . \Auth::id() . ',id',
+            'email' => 'required|unique:users,email,'.\Auth::id().',id',
         ];
     }
 }
\ No newline at end of file

app/Http/Requests/ExitProductRequest.php

Braces +3 added lines, -2 removed lines

@@ -15,9 +15,10 @@
 
         if ($involve
             && $this->getInvolvedRepository()->checkIfAuthInvolved($involve->product)
-        )
-            if ($involve->active !== 0) {
+        ) {
+                    if ($involve->active !== 0) {
                 return true;
+        }
             }
 
         return false;