ProductsRepository::update() - Code Metrics - Inspection of "28.12.2016 m2" - nyacide/amma - Measure and Improve Code Quality continuously with Scrutinizer

Completed
Push — master ( 28284e...eaa9bf )

unknown
created 2016-12-28 08:02 UTC
ProductsRepository::update() C

↳ Parent: ProductsRepository
Complexity

Conditions	11
Paths	2
Size

Total Lines	22
Code Lines	16
Duplication

Lines	0
Ratio	0 %
Importance

Changes
Metric	Value
dl	0
loc	22
rs	5.9012
c	0
b	0
f	0
cc	11
eloc	16
nc	2
nop	2
How to fix Complexity

<?php

namespace App\Repositories;

use App\ImprovedSpec;
use App\SpecPrice;
use App\Lot;
use App\Product;
use Carbon\Carbon;
use Illuminate\Database\Eloquent\Model;
use Mockery\CountValidator\Exception;

class ProductsRepository extends Repository
{
    /**
     * @return Product
     */
    public function getModel()
    {
        return new Product();
    }

    /**
     * Get all published products
     *
     * @return mixed
     */
    public function getPublic()
    {
        return self::getModel()

            ->published()
            ->get();
    }

    /**
     * Find product by id/slug.
     *
     * @param $slug
     * @return Product
     */
    public function find($slug)

    {
        if (is_numeric($slug))
            return $this->getModel()
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
                ->whereId((int) $slug)
//                ->whereIn('status', ['published', 'drafted', 'notverified', 'completed'])

                ->first();

        return $this->getModel()
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
            ->whereSlug($slug)
//            ->whereIn('status', ['published', 'drafted', 'notverified', 'completed'])

            ->first();
    }

    /**
     * Create product.
     * 
     * @param array $data
     * @return Product
     */
    public function create(array $data)
    {
        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)
            ]);
    }

    /**
     * Update data.
     *
     * @param $product
     * @param $data
     * @return mixed
     */
    public function update($product, $data)
    {
        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',
            '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),
            'active'          => 1
        ]);

        $product->save();

        return $product;
    }

    /**
     * Reformat date.
     *
     * @param $date
     * @param string $delimiter
     * @return mixed
     */
    public function reformatDateString($date, $delimiter = '.')
    {
        $datas = explode($delimiter, $date);

        $new_date['d'] = $datas[0];
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
        $new_date['m'] = $datas[1];
        $new_date['y'] = $datas[2];

        return $new_date;
    }

    /**
     * Convert string date to \Carbon/Carbon timestamp.
     *
     * @param $date
     * @return static
     */
    public function dateToTimestamp($date)
    {
        $dates = $this->reformatDateString($date);

        return Carbon::createFromDate($dates['y'], $dates['m'], $dates['d']);
    }

    /**
     * Remove product row from table.
     *
     * @param $id
     * @return bool|null
     * @throws \Exception
     */
    public function delete($id)
    {
        return $this->find($id)->delete();
    }

    /**
     * Get random products.
     *
     * @return \Illuminate\Database\Eloquent\Collection
     */
    public function getSomeRandomProducts()
    {
        $random_element_1 = rand(1, 2);
        $random_element_2 = rand(1, 2);
        $random_element_3 = rand(1, 2);

        $query = self::getModel()->select('*');
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }

        if ($random_element_1 == 1) {
            $query->published();
        } else {
            $query->drafted();
        }

        if ($random_element_2 == 1) {
            $query->whereType('old');
        } else {
            $query->whereType('new');
        }

        if ($random_element_3 == 1) {
            $query->whereBetween('sale', [1, 50]);
        } else {
            $query->whereBetween('sale', [51, 100]);
        }

        return $query->get();
    }

    /**
     * Get drafted product by id.
     * 
     * @param $id
     * @return mixed
     */
    public function findDrafted($id)
    {
        return self::getModel()
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
            ->whereId($id)
            ->drafted()
            ->first();
    }

    public function getCount($id)
    {
        return self::getModel()
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
            ->whereId($id)
            ->pluck('count')
            ->first();
    }

    public function countInLotProduct($id)
    {
        return self::getModel()
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
            ->where('lot_id',$id)
            ->count();
    }

    public function search($filters)
    {

        $product = $filters['search'];

        if(isset($filters['category']))
            $category = $filters['category'];

        if(empty($product) && (!isset($category)))
            return null;

        if(isset($category))
        {
            $query = $this->getModel()
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
                ->select('products.*', 'categoryable.category_id')
                ->where('products.name', 'like', '%'.$product.'%')
                ->join('categoryable', 'products.id', '=', 'categoryable.categoryable_id')
                ->where('categoryable.categoryable_type', get_class(self::getModel()))
                ->where('categoryable.category_id', $category);
        } else
        {
            $query = $this->getModel()
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
                ->where('name', 'like', '%'.$product.'%');
        }

        $query->where('products.active', 1);
            /*->whereIn('active', ['published', 'completed']);*/


        return $query->get();
    }

    /**
     * Remove percent from sale if it exists.
     *
     * @param $sale
     * @return int
     */
    private function formatSale($sale)
    {
        list($sale, $percent) = explode('%', $sale);
<?php

function returnThreeValues() {
    return array('a', 'b', 'c');
}

list($a, $b, $c) = returnThreeValues();

print $a . " - " . $c;

        return (int) $sale;
    }

    /**
     * Get same products.
     *
     * @param mixed
     */
    public function getSameProduct($id,$limit=10)
    {
        $query = $this->getModel()
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
            ->select('products.*')
            ->where('sub_category_id',$id)
            ->where('products.active', 1)
            /*->where('lots.expire_date', '>', Carbon::now())*/

            ->limit($limit);

        $query->join('lots', 'lots.id', '=', 'products.lot_id')
            ->where('lots.verify_status', 'verified');

        return $query->get();

    }

    /**
     * Get public latest created products.
     *
     * @param int $count
     * @return mixed
     */
    public function getPublicLatest($count = 8)
    {
        return $this->getModel()

            ->active()
//            ->published() // todo: on production back it.

            ->orderBy('id', self::DESC)
            ->take($count)
            ->get();
    }

    /**
     * Get popular products.
     *
     * @return mixed
     */
    public function getFeaturedPublic($count = 8)
    {
        return $this->getModel()

//            ->published()
            ->featured()
            ->active()
            ->orderBy('id', self::DESC)
            ->take($count)
            ->get();
    }

    /**
     * Get expire soon products.
     *
     * @param int $count
     * @return mixed
     */
    public function getPublicExpireSoon($count = 8)
    {
        return $this->getModel()

//            ->published()
            ->active()
//            ->where('expiration_date', '>', Carbon::now())

//            ->orderBy('expiration_date', self::ASC)
            ->orderBy('id', self::ASC)
            ->take($count)
            ->get();
    }

    /**
     * Expire soon products.
     *
     * @param int $paginate
     * @return mixed
     */
    public function getExpireSoon($paginate = 10)
    {
        $query = $this->getModel()
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
            ->select('products.*')
            ->where('products.active', 1)
            ->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('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);

        $query->join('lots', 'lots.id', '=', 'products.lot_id')
            ->where('lots.status', Lot::STATUS_COMPLETE)
            ->where('lots.verify_status', Lot::STATUS_VERIFY_ACCEPTED);

//        return $query->orderBy('id', self::ASC)

            return $query->paginate($paginate);
    }

    /**
     * Create plain product for \App\Lot $lot
     *
     * @param Lot $lot
     * @return static
     */
    public function createPlain(Lot $lot)
    {
        $product = self::getModel()
            ->create([
                'lot_id' => $lot->id,
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
                'uniqid' => substr(str_replace('.','',uniqid('00'.rand(),true)),0,10)

            ]);
        /*$spec_price =  SpecPrice::create([

            'product_id' => $product->id
        ]);
        ImprovedSpec::create([
            'product_id' => $product->id,
            'price_spec_id' => $spec_price->id
        ]);*/

        return $product;
    }

    public function saveProduct($product, array $data)
    {
        $product->fill([
            /*'name'            => isset($data['name']) ? $data['name'] : null,

            '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
        ]);

        $product->save();

        return $product;
    }
}
1			<?php
2
3			namespace App\Repositories;
4
5			use App\ImprovedSpec;
6			use App\SpecPrice;
7			use App\Lot;
8			use App\Product;
9			use Carbon\Carbon;
10			use Illuminate\Database\Eloquent\Model;
11			use Mockery\CountValidator\Exception;
12
13			class ProductsRepository extends Repository
14			{
15			/**
16			* @return Product
17			*/
18			public function getModel()
19			{
20			return new Product();
21			}
22
23			/**
24			* Get all published products
25			*
26			* @return mixed
27			*/
28			public function getPublic()
29			{
30			return self::getModel()
			0 ignored issues – show Bug introduced 2016-12-28 08:14 UTC by Report Bug Copy Issue Report The method `published()` does not exist on `App\Product`. Did you maybe mean `scopePublished()`? This check marks calls to methods that do not seem to exist on an object. This is most likely the result of a method being renamed without all references to it being renamed likewise. Loading history...
31			->published()
32			->get();
33			}
34
35			/**
36			* Find product by id/slug.
37			*
38			* @param $slug
39			* @return Product
nyacide / amma

Push — master ( 28284e...eaa9bf )

ProductsRepository::update() C

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like
