nyacide / amma

app/Repositories/LotDeliveryPaymentRepository.php

Doc Comments +2 added lines, -2 removed lines

@@ -7,7 +7,7 @@ 
 class LotDeliveryPaymentRepository extends Repository
 {
     /**
-     * @return Banner
+     * @return LotDeliveryPayment
      */
     public function getModel()
     {
@@ -18,7 +18,7 @@ 
      * Get ad-blocks for extended banners block.
      *
      * @param $count
-     * @return mixed
+     * @return boolean
      */
     public function save($lot, array $data)
     {

Spacing +4 added lines, -4 removed lines

@@ -24,13 +24,13 @@
     {
         $model = self::getModel();
         $model->where('lot_id', $lot->id)->delete();
-        $insert = []; $i = 0;
+        $insert = [ ]; $i = 0;
         if (!empty($data)) {
             foreach ($data as $key1 => $method) {
                 foreach ($method as $key2 => $item) {
-                    $insert[$i]['lot_id']      = $lot->id;
-                    $insert[$i]['method_id']   = (int)$item;
-                    $insert[$i]['method_type'] = $key1;
+                    $insert[ $i ][ 'lot_id' ]      = $lot->id;
+                    $insert[ $i ][ 'method_id' ]   = (int) $item;
+                    $insert[ $i ][ 'method_type' ] = $key1;
                     $i++;
                 }
 

app/Repositories/LotRepository.php

Doc Comments +10 added lines, -7 removed lines

@@ -20,7 +20,7 @@ 
      * Create plain lot for attached items to him.
      *
      * @param Vendor $vendor
-     * @return mixed
+     * @return Lot
      */
     public function createDraft(Vendor $vendor)
     {
@@ -49,8 +49,8 @@ 
     /**
      * Add empty lot or modify just existed (drafted)..
      *
-     * @param $vendor
-     * @return Lot|mixed
+     * @param Vendor $vendor
+     * @return Lot
      */
     public function addLot($vendor)
     {
@@ -81,7 +81,7 @@ 
     /**
      * Get user's lots.
      *
-     * @param $user
+     * @param \Illuminate\Contracts\Auth\Authenticatable|null $user
      * @param $perPage
      * @return \Illuminate\Support\Collection|null
      */
@@ -130,7 +130,7 @@ 
     /**
      * Delete lot.
      *
-     * @param $lot
+     * @param Lot $lot
      */
     public function delete($lot)
     {
@@ -147,7 +147,7 @@ 
      * Convert string date to \Carbon/Carbon timestamp.
      *
      * @param $date
-     * @return static
+     * @return Carbon
      */
     public function dateToTimestamp($date)
     {
@@ -181,6 +181,9 @@ 
         return $new_date;
     }
 
+    /**
+     * @param Lot $lot
+     */
     public function save($lot, array $data)
     {
         $lot->fill([
@@ -202,7 +205,7 @@ 
     /**
      * Change category.
      *
-     * @param $lot
+     * @param Lot $lot
      * @param $category_id
      *
      * @return void

Spacing +31 added lines, -31 removed lines

@@ -42,7 +42,7 @@ 
             ->first();
     }
 
-    public function statusChange(){
+    public function statusChange() {
 
     }
 
@@ -54,7 +54,7 @@ 
      */
     public function addLot($vendor)
     {
-        if($lot = $this->getDraftedLot($vendor))
+        if ($lot = $this->getDraftedLot($vendor))
             return $lot;
 
         return $this->createDraft($vendor);
@@ -70,7 +70,7 @@ 
     {
         $query = $this->getModel();
 
-        if($vendor)
+        if ($vendor)
             $query = $query->where('vendor_id', $vendor->id);
 
         $lot = $query->drafted()->first();
@@ -89,11 +89,11 @@ 
     {
         $model = self::getModel();
 
-        $vendors = [];
+        $vendors = [ ];
 
         $user->vendors()->active()->get()
             ->each(function($vendor) use (&$vendors){
-                $vendors[] = $vendor->id;
+                $vendors[ ] = $vendor->id;
             });
 
         $lots = $this->getModel()
@@ -106,25 +106,25 @@ 
         return ($lots->count()) ? $lots : null;
     }
 
-    public function userLotsPendingComision($user,$lotId=null)
+    public function userLotsPendingComision($user, $lotId = null)
     {
 
         $model = self::getModel();
 
-        $vendors = [];
+        $vendors = [ ];
         $user->vendors()->active()->get()
             ->each(function($vendor) use (&$vendors){
-                $vendors[] = $vendor->id;
+                $vendors[ ] = $vendor->id;
             });
         $query = $this->getModel()->where('verify_status', $model::STATUS_VERIFY_PENDING);
         if (!empty($vendors)) {
             $query->whereIn('vendor_id', $vendors);
         }
         if ($lotId != null) {
-            $query->where('id','!=', $lotId);
+            $query->where('id', '!=', $lotId);
         }
         $sum = $query->sum('comision');
-        return  $sum ? (int)$sum : 0;
+        return  $sum ? (int) $sum : 0;
     }
 
     /**
@@ -135,7 +135,7 @@ 
     public function delete($lot)
     {
         $model = self::getModel();
-        if($lot->status !== $model::STATUS_DELETED)
+        if ($lot->status !== $model::STATUS_DELETED)
         {
             $lot->status = $model::STATUS_DELETED;
 
@@ -153,7 +153,7 @@ 
     {
         $dates = $this->reformatDateString($date);
 
-        return Carbon::createFromDate($dates['y'], $dates['m'], $dates['d']);
+        return Carbon::createFromDate($dates[ 'y' ], $dates[ 'm' ], $dates[ 'd' ]);
     }
 
     /**
@@ -167,15 +167,15 @@ 
     {
         $datas = explode($delimiter, $date);
 
-        if(count($datas) == 3) {
-            $new_date['d'] = $datas[0];
-            $new_date['m'] = $datas[1];
-            $new_date['y'] = $datas[2];
+        if (count($datas) == 3) {
+            $new_date[ 'd' ] = $datas[ 0 ];
+            $new_date[ 'm' ] = $datas[ 1 ];
+            $new_date[ 'y' ] = $datas[ 2 ];
         } else {
             $now = Carbon::now();
-            $new_date['d'] = $now->day;
-            $new_date['m'] = $now->month;
-            $new_date['y'] = $now->year;
+            $new_date[ 'd' ] = $now->day;
+            $new_date[ 'm' ] = $now->month;
+            $new_date[ 'y' ] = $now->year;
         }
 
         return $new_date;
@@ -184,16 +184,16 @@ 
     public function save($lot, array $data)
     {
         $lot->fill([
-            'name'                 => isset($data['name']) ? $data['name'] : $lot->present()->renderDraftedName(),
-            'category_id'          => isset($data['category']) ? $data['category'] : null,
-            'currency_id'          => isset($data['currency']) ? (int)$data['currency'] : null,
-            'description'          => isset($data['description']) ? $data['description'] : null,
-            'yield_amount'         => isset($data['yield_amount']) ? $data['yield_amount'] : null,
-            'public_date'          => isset($data['public_date']) ? $this->dateToTimestamp($data['public_date']) : Carbon::now()->addDays(1),
-            'expire_date'          => isset($data['expirate_date']) ? $this->dateToTimestamp($data['expirate_date']) :Carbon::now()->addDays(5),
-            'comision'             => isset($data['comision']) ? $data['comision'] : 0,
-            'description_delivery' => isset($data['description_delivery']) ? $data['description_delivery'] : null,
-            'description_payment'  => isset($data['description_payment']) ? $data['description_payment'] : null,
+            'name'                 => isset($data[ 'name' ]) ? $data[ 'name' ] : $lot->present()->renderDraftedName(),
+            'category_id'          => isset($data[ 'category' ]) ? $data[ 'category' ] : null,
+            'currency_id'          => isset($data[ 'currency' ]) ? (int) $data[ 'currency' ] : null,
+            'description'          => isset($data[ 'description' ]) ? $data[ 'description' ] : null,
+            'yield_amount'         => isset($data[ 'yield_amount' ]) ? $data[ 'yield_amount' ] : null,
+            'public_date'          => isset($data[ 'public_date' ]) ? $this->dateToTimestamp($data[ 'public_date' ]) : Carbon::now()->addDays(1),
+            'expire_date'          => isset($data[ 'expirate_date' ]) ? $this->dateToTimestamp($data[ 'expirate_date' ]) : Carbon::now()->addDays(5),
+            'comision'             => isset($data[ 'comision' ]) ? $data[ 'comision' ] : 0,
+            'description_delivery' => isset($data[ 'description_delivery' ]) ? $data[ 'description_delivery' ] : null,
+            'description_payment'  => isset($data[ 'description_payment' ]) ? $data[ 'description_payment' ] : null,
         ])->save();
         return $lot;
     }
@@ -222,7 +222,7 @@ 
      */
     public function checkIfPossibleToChangeCategory(Lot $lot)
     {
-        if(! count($lot->products))
+        if (!count($lot->products))
             return true;
 
         return false;
@@ -231,7 +231,7 @@ 
     public function getLatestLot($limit = 10)
     {
         return self::getModel()
-            ->orderBy('id','DESC')
+            ->orderBy('id', 'DESC')
             ->active()
             ->limit($limit)
             ->get();

Braces +12 added lines, -8 removed lines

@@ -32,10 +32,11 @@ 
 
     public function find($slug)
     {
-        if (is_numeric($slug))
-            return $this->getModel()
+        if (is_numeric($slug)) {
+                    return $this->getModel()
                 ->whereId((int) $slug)
                 ->first();
+        }
 
         return $this->getModel()
             ->whereSlug($slug)
@@ -54,8 +55,9 @@ 
      */
     public function addLot($vendor)
     {
-        if($lot = $this->getDraftedLot($vendor))
-            return $lot;
+        if($lot = $this->getDraftedLot($vendor)) {
+                    return $lot;
+        }
 
         return $this->createDraft($vendor);
     }
@@ -70,8 +72,9 @@ 
     {
         $query = $this->getModel();
 
-        if($vendor)
-            $query = $query->where('vendor_id', $vendor->id);
+        if($vendor) {
+                    $query = $query->where('vendor_id', $vendor->id);
+        }
 
         $lot = $query->drafted()->first();
 
@@ -222,8 +225,9 @@ 
      */
     public function checkIfPossibleToChangeCategory(Lot $lot)
     {
-        if(! count($lot->products))
-            return true;
+        if(! count($lot->products)) {
+                    return true;
+        }
 
         return false;
     }

app/Repositories/MethodDeliveryPaymentRepository.php

Doc Comments +1 added lines

@@ -27,6 +27,7 @@
      * Get public posts.
      *
      * @param $perPage
+     * @param string $type
      * @return \Illuminate\Database\Eloquent\Collection
      */
     public function getPublic($type)

app/Repositories/ModelColorsRepository.php

Doc Comments +3 added lines

@@ -38,6 +38,9 @@
         return $color;
     }
 
+    /**
+     * @param integer $slug
+     */
     public function find($slug)
     {
         if (is_numeric($slug))

Spacing +9 added lines, -9 removed lines

@@ -15,25 +15,25 @@ 
         return new Color();
     }
 
-    public function create(array $data,$specSize)
+    public function create(array $data, $specSize)
     {
         return self::getModel()
             ->create([
                 'size_id'    => $specSize->id,
                 'product_id' => $specSize->product_id,
-                'color_hash' => (isset($data['color_hash']) ? $data['color_hash'] : ''),
-                'amount'     => (isset($data['amount']) ? $data['amount'] : '')
+                'color_hash' => (isset($data[ 'color_hash' ]) ? $data[ 'color_hash' ] : ''),
+                'amount'     => (isset($data[ 'amount' ]) ? $data[ 'amount' ] : '')
             ]);
     }
     public function save(array $data, $specSize)
     {
-        $key = ((isset($data['key']) &&  $data['key'] != null) ? $data['key'] : null);
+        $key = ((isset($data[ 'key' ]) && $data[ 'key' ] != null) ? $data[ 'key' ] : null);
         $color             = self::getModel()->firstOrNew(array('key'=>$key));
         $color->size_id    = $specSize->id;
         $color->product_id = $specSize->product_id;
-        $color->color_hash = (isset($data['color_hash']) ? $data['color_hash'] : '');
-        $color->amount     = (isset($data['amount']) ? $data['amount'] : '');
-        $color->key         = (isset($data['key']) ? $data['key'] : '');
+        $color->color_hash = (isset($data[ 'color_hash' ]) ? $data[ 'color_hash' ] : '');
+        $color->amount     = (isset($data[ 'amount' ]) ? $data[ 'amount' ] : '');
+        $color->key = (isset($data[ 'key' ]) ? $data[ 'key' ] : '');
         $color->save();
         return $color;
     }
@@ -65,7 +65,7 @@ 
     public function delete($color)
     {
         if (is_numeric($color))
-            $this->find((int)$color)->delete();
+            $this->find((int) $color)->delete();
 
         if ($color instanceof Model)
             $color->delete();
@@ -81,7 +81,7 @@ 
 
     public function getById($id) {
         return self::getModel()
-            ->where('size_id',$id)
+            ->where('size_id', $id)
             ->get();
     }
 

Braces +12 added lines, -8 removed lines

@@ -40,10 +40,11 @@ 
 
     public function find($slug)
     {
-        if (is_numeric($slug))
-            return $this->getModel()
+        if (is_numeric($slug)) {
+                    return $this->getModel()
                 ->whereId((int) $slug)
                 ->first();
+        }
 
         return $this->getModel()
             ->whereSlug($slug)
@@ -64,11 +65,13 @@ 
      */
     public function delete($color)
     {
-        if (is_numeric($color))
-            $this->find((int)$color)->delete();
+        if (is_numeric($color)) {
+                    $this->find((int)$color)->delete();
+        }
 
-        if ($color instanceof Model)
-            $color->delete();
+        if ($color instanceof Model) {
+                    $color->delete();
+        }
     }
 
     public function getByProductAndColor($product, $color)
@@ -94,8 +97,9 @@ 
      */
     public function hasColor($product, $color)
     {
-        if ($this->getByProductAndColor($product, $color))
-            return true;
+        if ($this->getByProductAndColor($product, $color)) {
+                    return true;
+        }
 
         return false;
     }

app/Repositories/ProductsRepository.php

Doc Comments +6 added lines, -3 removed lines

@@ -80,7 +80,7 @@ 
      *
      * @param $product
      * @param $data
-     * @return mixed
+     * @return Model
      */
     public function update($product, $data)
     {
@@ -127,7 +127,7 @@ 
      * Convert string date to \Carbon/Carbon timestamp.
      *
      * @param $date
-     * @return static
+     * @return Carbon
      */
     public function dateToTimestamp($date)
     {
@@ -361,7 +361,7 @@ 
      * Create plain product for \App\Lot $lot
      *
      * @param Lot $lot
-     * @return static
+     * @return Product
      */
     public function createPlain(Lot $lot)
     {
@@ -382,6 +382,9 @@ 
         return $product;
     }
 
+    /**
+     * @param Product $product
+     */
     public function saveProduct($product, array $data)
     {
         $product->fill([

Spacing +38 added lines, -38 removed lines

@@ -62,16 +62,16 @@ 
     {
         return self::getModel()
             ->create([
-                'vendor_id' => $data['vendor_id'],
-                'name' => (isset($data['name']) ? $data['name'] : ''),
-                'price' => (isset($data['price']) ? $data['price'] : ''),
-                'sale' => (isset($data['sale'])) ? $data['sale'] : 0,
-                'count' => (isset($data['count'])) ? $data['count'] : 1,
-                'type' => (isset($data['type'])) ? $data['type'] : 'new',
-                'status' => (isset($data['status'])) ? $data['status'] : 'drafted',
-                'published_date' => (isset($data['published_date']) ? $data['published_date'] : Carbon::now()),
-                'expiration_date' => (isset($data['expiration_date']) ? $data['expiration_date'] : Carbon::now()),
-                'active' => (isset($data['active']) ? $data['active'] : 0)
+                'vendor_id' => $data[ 'vendor_id' ],
+                'name' => (isset($data[ 'name' ]) ? $data[ 'name' ] : ''),
+                'price' => (isset($data[ 'price' ]) ? $data[ 'price' ] : ''),
+                'sale' => (isset($data[ 'sale' ])) ? $data[ 'sale' ] : 0,
+                'count' => (isset($data[ 'count' ])) ? $data[ 'count' ] : 1,
+                'type' => (isset($data[ 'type' ])) ? $data[ 'type' ] : 'new',
+                'status' => (isset($data[ 'status' ])) ? $data[ 'status' ] : 'drafted',
+                'published_date' => (isset($data[ 'published_date' ]) ? $data[ 'published_date' ] : Carbon::now()),
+                'expiration_date' => (isset($data[ 'expiration_date' ]) ? $data[ 'expiration_date' ] : Carbon::now()),
+                'active' => (isset($data[ 'active' ]) ? $data[ 'active' ] : 0)
             ]);
     }
 
@@ -84,19 +84,19 @@ 
      */
     public function update($product, $data)
     {
-        if(! $product instanceof Model)
+        if (!$product instanceof Model)
             throw new Exception('First argument MUST be an instance of '.Model::class);
 
         $product->fill([
-            'name'            => (isset($data['name']) ? $data['name'] : $product->name),
-            'price'           => (isset($data['price']) ? $data['price'] : $product->price),
-            'sale'            => (isset($data['sale'])) ? $this->formatSale($data['sale']) : $product->sale,
-            'count'           => (isset($data['count'])) ? $data['count'] : $product->count,
-            'description'     => (isset($data['description'])) ? $data['description'] : $product->description,
-            'type'            => (isset($data['type'])) ? $data['type'] : 'new',
+            'name'            => (isset($data[ 'name' ]) ? $data[ 'name' ] : $product->name),
+            'price'           => (isset($data[ 'price' ]) ? $data[ 'price' ] : $product->price),
+            'sale'            => (isset($data[ 'sale' ])) ? $this->formatSale($data[ 'sale' ]) : $product->sale,
+            'count'           => (isset($data[ 'count' ])) ? $data[ 'count' ] : $product->count,
+            'description'     => (isset($data[ 'description' ])) ? $data[ 'description' ] : $product->description,
+            'type'            => (isset($data[ 'type' ])) ? $data[ 'type' ] : 'new',
             'status'          => ($product->status == 'drafted') ? 'notverified' : $product->status,
-            'published_date'  => (isset($data['published_date']) ? $this->dateToTimestamp($data['published_date']) : $product->published_date),
-            'expiration_date' => (isset($data['expiration_date']) ? $this->dateToTimestamp($data['expiration_date']) : $product->published_date),
+            'published_date'  => (isset($data[ 'published_date' ]) ? $this->dateToTimestamp($data[ 'published_date' ]) : $product->published_date),
+            'expiration_date' => (isset($data[ 'expiration_date' ]) ? $this->dateToTimestamp($data[ 'expiration_date' ]) : $product->published_date),
             'active'          => 1
         ]);
 
@@ -116,9 +116,9 @@ 
     {
         $datas = explode($delimiter, $date);
 
-        $new_date['d'] = $datas[0];
-        $new_date['m'] = $datas[1];
-        $new_date['y'] = $datas[2];
+        $new_date[ 'd' ] = $datas[ 0 ];
+        $new_date[ 'm' ] = $datas[ 1 ];
+        $new_date[ 'y' ] = $datas[ 2 ];
 
         return $new_date;
     }
@@ -133,7 +133,7 @@ 
     {
         $dates = $this->reformatDateString($date);
 
-        return Carbon::createFromDate($dates['y'], $dates['m'], $dates['d']);
+        return Carbon::createFromDate($dates[ 'y' ], $dates[ 'm' ], $dates[ 'd' ]);
     }
 
     /**
@@ -174,9 +174,9 @@ 
         }
 
         if ($random_element_3 == 1) {
-            $query->whereBetween('sale', [1, 50]);
+            $query->whereBetween('sale', [ 1, 50 ]);
         } else {
-            $query->whereBetween('sale', [51, 100]);
+            $query->whereBetween('sale', [ 51, 100 ]);
         }
 
         return $query->get();
@@ -207,22 +207,22 @@ 
     public function countInLotProduct($id)
     {
         return self::getModel()
-            ->where('lot_id',$id)
+            ->where('lot_id', $id)
             ->count();
     }
 
     public function search($filters)
     {
 
-        $product = $filters['search'];
+        $product = $filters[ 'search' ];
 
-        if(isset($filters['category']))
-            $category = $filters['category'];
+        if (isset($filters[ 'category' ]))
+            $category = $filters[ 'category' ];
 
-        if(empty($product) && (!isset($category)))
+        if (empty($product) && (!isset($category)))
             return null;
 
-        if(isset($category))
+        if (isset($category))
         {
             $query = $this->getModel()
                 ->select('products.*', 'categoryable.category_id')
@@ -260,11 +260,11 @@ 
      *
      * @param mixed
      */
-    public function getSameProduct($id,$limit=10)
+    public function getSameProduct($id, $limit = 10)
     {
         $query = $this->getModel()
             ->select('products.*')
-            ->where('sub_category_id',$id)
+            ->where('sub_category_id', $id)
             ->where('products.active', 1)
             /*->where('lots.expire_date', '>', Carbon::now())*/
             ->limit($limit);
@@ -340,13 +340,13 @@ 
             ->where('lots.expire_date', '>', Carbon::now())
             ->orderBy('lots.expire_date', self::ASC);
 
-        if(request()->get('name'))
+        if (request()->get('name'))
             $query->orderBy('products.name', request()->get('name') == self::ASC ? self::ASC : self::DESC);
 
-        if(request()->get('created_at'))
+        if (request()->get('created_at'))
             $query->orderBy('products.created_at', request()->get('created_at') == self::ASC ? self::ASC : self::DESC);
 
-        if(request()->get('price'))
+        if (request()->get('price'))
             $query->orderBy('products.price', request()->get('price') == self::ASC ? self::ASC : self::DESC);
 
         $query->join('lots', 'lots.id', '=', 'products.lot_id')
@@ -368,7 +368,7 @@ 
         $product = self::getModel()
             ->create([
                 'lot_id' => $lot->id,
-                'uniqid' => substr(str_replace('.','',uniqid('00'.rand(),true)),0,10)
+                'uniqid' => substr(str_replace('.', '', uniqid('00'.rand(), true)), 0, 10)
 
             ]);
         /*$spec_price =  SpecPrice::create([
@@ -389,7 +389,7 @@ 
             'old_price'       => isset($data['old_price']) ? $data['old_price'] : null,
             'price'           => isset($data['price']) ? $data['price'] : null,
             'sale'            => isset($data['sale']) ? $data['sale'] : null,*/
-            'sub_category_id' => isset($data['sub_category']) ? $data['sub_category'] : null
+            'sub_category_id' => isset($data[ 'sub_category' ]) ? $data[ 'sub_category' ] : null
         ]);
 
         $product->save();

Braces +21 added lines, -14 removed lines

@@ -40,11 +40,12 @@ 
      */
     public function find($slug)
     {
-        if (is_numeric($slug))
-            return $this->getModel()
+        if (is_numeric($slug)) {
+                    return $this->getModel()
                 ->whereId((int) $slug)
 //                ->whereIn('status', ['published', 'drafted', 'notverified', 'completed'])
                 ->first();
+        }
 
         return $this->getModel()
             ->whereSlug($slug)
@@ -84,8 +85,9 @@ 
      */
     public function update($product, $data)
     {
-        if(! $product instanceof Model)
-            throw new Exception('First argument MUST be an instance of '.Model::class);
+        if(! $product instanceof Model) {
+                    throw new Exception('First argument MUST be an instance of '.Model::class);
+        }
 
         $product->fill([
             'name'            => (isset($data['name']) ? $data['name'] : $product->name),
@@ -216,11 +218,13 @@ 
 
         $product = $filters['search'];
 
-        if(isset($filters['category']))
-            $category = $filters['category'];
+        if(isset($filters['category'])) {
+                    $category = $filters['category'];
+        }
 
-        if(empty($product) && (!isset($category)))
-            return null;
+        if(empty($product) && (!isset($category))) {
+                    return null;
+        }
 
         if(isset($category))
         {
@@ -340,14 +344,17 @@ 
             ->where('lots.expire_date', '>', Carbon::now())
             ->orderBy('lots.expire_date', self::ASC);
 
-        if(request()->get('name'))
-            $query->orderBy('products.name', request()->get('name') == self::ASC ? self::ASC : self::DESC);
+        if(request()->get('name')) {
+                    $query->orderBy('products.name', request()->get('name') == self::ASC ? self::ASC : self::DESC);
+        }
 
-        if(request()->get('created_at'))
-            $query->orderBy('products.created_at', request()->get('created_at') == self::ASC ? self::ASC : self::DESC);
+        if(request()->get('created_at')) {
+                    $query->orderBy('products.created_at', request()->get('created_at') == self::ASC ? self::ASC : self::DESC);
+        }
 
-        if(request()->get('price'))
-            $query->orderBy('products.price', request()->get('price') == self::ASC ? self::ASC : self::DESC);
+        if(request()->get('price')) {
+                    $query->orderBy('products.price', request()->get('price') == self::ASC ? self::ASC : self::DESC);
+        }
 
         $query->join('lots', 'lots.id', '=', 'products.lot_id')
             ->where('lots.status', Lot::STATUS_COMPLETE)

app/Repositories/RecoverpasswordRepository.php

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace App\Repositories;
 
 use App\RecoverPassword;
-use Auth;
 
 class RecoverPasswordRepository extends Repository
 {

app/Repositories/SocialiteRepository.php

Doc Comments +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
     /**
      * Callback to json.
      *
-     * @param $callback
+     * @param ProviderUser $callback
      * @return string
      */
     private function cleanCallback($callback)

Spacing +2 added lines, -2 removed lines

@@ -101,8 +101,8 @@
     private function cleanCallback($callback)
     {
         $array_callback = (array) $callback;
-        $user           = $array_callback['user'];
-        unset($array_callback['user']);
+        $user           = $array_callback[ 'user' ];
+        unset($array_callback[ 'user' ]);
         return json_encode(array_merge($array_callback, $user));
     }
 }
\ No newline at end of file

app/Repositories/SpecPriceRepository.php

Doc Comments +5 added lines, -2 removed lines

@@ -16,7 +16,7 @@ 
 
     /**
      * @param $product_id
-     * @return static
+     * @return SpecPrice
      */
     public function createPlain($product_id)
     {
@@ -65,7 +65,7 @@ 
      * 
      * @param $spec
      * @param array $data
-     * @return mixed
+     * @return SpecPrice
      */
 
     public function create(array $data, $product)
@@ -81,6 +81,9 @@ 
             ]);
     }
 
+    /**
+     * @param Product $product
+     */
     public function save(array $data, $product)
     {
         $key = ((isset($data['key']) &&  $data['key'] != null) ? $data['key'] : null);

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace App\Repositories;
 
 use App\SpecPrice;
-use App\Product;
 class SpecPriceRepository extends Repository
 {
     /**

Spacing +13 added lines, -13 removed lines

@@ -74,24 +74,24 @@ 
             ->create([
                 'lot_id'     => $product->lot_id,
                 'product_id' => $product->id,
-                'new_price'  => (isset($data['new_price']) ? $data['new_price'] : ''),
-                'old_price'  => (isset($data['old_price']) ? $data['old_price'] : ''),
-                'sale'       => (isset($data['sale'])) ? $data['sale'] : 0,
-                'name'       => (isset($data['name'])) ? $data['name'] : null
+                'new_price'  => (isset($data[ 'new_price' ]) ? $data[ 'new_price' ] : ''),
+                'old_price'  => (isset($data[ 'old_price' ]) ? $data[ 'old_price' ] : ''),
+                'sale'       => (isset($data[ 'sale' ])) ? $data[ 'sale' ] : 0,
+                'name'       => (isset($data[ 'name' ])) ? $data[ 'name' ] : null
             ]);
     }
 
     public function save(array $data, $product)
     {
-        $key = ((isset($data['key']) &&  $data['key'] != null) ? $data['key'] : null);
+        $key = ((isset($data[ 'key' ]) && $data[ 'key' ] != null) ? $data[ 'key' ] : null);
         $price = self::getModel()->firstOrNew(array('key'=>$key));
         $price->product_id = $product->id;
         $price->lot_id     = $product->lot_id;
-        $price->new_price  = (isset($data['new_price']) ? $data['new_price'] : '');
-        $price->old_price  = (isset($data['old_price']) ? $data['old_price'] : '');
-        $price->sale       = (isset($data['sale'])) ? $data['sale'] : 0;
-        $price->name       = (isset($data['name']) ? $data['name'] : ''); 
-        $price->key        = (isset($data['key']) ? $data['key'] : '');
+        $price->new_price  = (isset($data[ 'new_price' ]) ? $data[ 'new_price' ] : '');
+        $price->old_price  = (isset($data[ 'old_price' ]) ? $data[ 'old_price' ] : '');
+        $price->sale       = (isset($data[ 'sale' ])) ? $data[ 'sale' ] : 0;
+        $price->name       = (isset($data[ 'name' ]) ? $data[ 'name' ] : ''); 
+        $price->key        = (isset($data[ 'key' ]) ? $data[ 'key' ] : '');
         $price->save();
         return $price;
     }
@@ -99,9 +99,9 @@ 
     public function update($spec, array $data)
     {
         $spec->fill([
-            'size'       => isset($data['size']) ? $data['size'] : null,
-            'color_hash' => isset($data['color']) ? $data['color'] : null,
-            'amount'     => isset($data['sold']) ? $data['sold'] : null
+            'size'       => isset($data[ 'size' ]) ? $data[ 'size' ] : null,
+            'color_hash' => isset($data[ 'color' ]) ? $data[ 'color' ] : null,
+            'amount'     => isset($data[ 'sold' ]) ? $data[ 'sold' ] : null
         ]);
         
         $spec->save();

app/Repositories/SubscribeRepository.php

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace App\Repositories;
 
 use App\Subscribe;
-use Auth;
 
 class SubscribeRepository extends Repository
 {

Indentation +3 added lines, -3 removed lines

@@ -23,8 +23,8 @@ 
     {
         return self::getModel()
             ->create([
-	        'email'  => $data['email'],
-	        'token'  => str_random(30),
+            'email'  => $data['email'],
+            'token'  => str_random(30),
         ]);
     }
 
@@ -32,7 +32,7 @@ 
     {
         $model = $this->getByEmail($email);
 
-	       return (bool) $model;
+            return (bool) $model;
     }
 
     /**

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
     {
         return self::getModel()
             ->create([
-	        'email'  => $data['email'],
+	        'email'  => $data[ 'email' ],
 	        'token'  => str_random(30),
         ]);
     }