App\Repositories\VendorRepository

Complexity

Total Complexity

6

Size/Duplication

Total Lines	75
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	4

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getModel()	0	4	1
A	create()	0	22	2
A	update()	0	20	2
A	getPublic()	0	7	1

<?php

namespace App\Repositories;

use App\Orders\CreateWalletOrder;
use App\Services\ImageProcessor;
use App\Vendor;
use Auth;

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

    /**
     * Create method.
     *
     * @param array $data
     * @return Vendor
     */
    public function create(array $data)
    {
        $vendor = self::getModel()
            ->create([
                'user_id' => Auth::id(),
                'name' => $data['name'],
                'email' => $data['email'],
                'phone' => $data['phone'],
                'description' => $data['description']
            ]);

        (new CreateWalletOrder(\Auth::user()));

        if (isset($data['image'])) {
            $file = $data['image'];
            $location = 'upload/vendors/' . $vendor->id;

The property id does not exist on object<App\Vendor>. Since you implemented __get, maybe consider adding a @property annotation.

            $processor = new ImageProcessor();
            $processor->uploadAndCreate($file, $vendor, $data, $location);
        }

        return $vendor;
    }

    /**
     * Update method.
     *
     * @param Vendor $vendor
     * @param $data
     * @throws \Exception
     */
    public function update(Vendor $vendor, $data)
    {
        $vendor->fill([
            'name' => $data['name'],
            'email' => $data['email'],
            'description' => $data['description'],
            'phone' => $data['phone']
        ]);

        if (isset($data['image'])) {
            $file = $data['image'];
            $location = 'upload/vendors/' . $vendor->id;

The property id does not exist on object<App\Vendor>. Since you implemented __get, maybe consider adding a @property annotation.

            $processor = new ImageProcessor();
            $processor->destroy($vendor->images()->first()); // delete old logo.

            $processor->uploadAndCreate($file, $vendor, $data, $location);
        }

        $vendor->save();
    }

    public function getPublic($paginate = 9)
    {
        return $this->getModel()

The method active() does not exist on App\Vendor. Did you maybe mean scopeActive()?

            ->active() //todo: get popular first...
            ->orderBy('id', 'DESC')
            ->paginate($paginate);
    }
}