App\Purchase

Complexity

Total Complexity

17

Size/Duplication

Total Lines	208
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

38.46%

Importance

Changes	11
Bugs	1	Features	6

15 Methods

Rating	Name	Duplication	Size	Complexity
A	scopeDelivered()	0	4	1
A	scopeStored()	0	4	1
A	scopePendingStorage()	0	4	1
A	getStatusAttribute()	0	11	3
A	isPendingStorage()	0	4	1
A	guitars()	0	4	1
A	getStoredAttribute()	0	4	1
A	getPendingStorageAttribute()	0	4	1
A	getSoldAttribute()	0	4	1
A	hasArrived()	0	4	1
A	isStatus()	0	4	1
A	supplier()	0	4	1
A	make()	0	4	1
A	notifications()	0	4	1
A	makeNotification()	0	12	1

<?php

namespace App;

use App\Events\NotificationWasCreated;
use Carbon\Carbon;
use Illuminate\Database\Eloquent;
use Illuminate\Support\Facades\DB;

class Purchase extends Eloquent\Model
{
    const UPCOMING = 'upcoming';
    const DELIVERED = 'delivered';
    const COMPLETED = 'completed';

    use Eloquent\SoftDeletes;

    /**
     * We do not want any timestamps.
     *
     * @var bool
     */
    public $timestamps = false;

    /**
     * The attributes that are not mass assignable.
     *
     * @var array
     */
    protected $guarded = [];

    /**
     * Date mutators.
     *
     * @var array
     */
    protected $dates = ['date_purchased', 'delivery_date', 'deleted_at'];

    /**
     * Default date format.
     *
     * @var string
     */
    protected $dateFormat = 'Y-m-d';

    /**
     * Delivered Scope.
     *
     * @param $query
     *
     * @return Eloquent\Builder
     */
    public function scopeDelivered($query)
    {
        return $query->where('delivery_date', '<=', Carbon::today());
    }

    /**
     * Stored Scope.
     *
     * @param $query
     *
     * @return Eloquent\Builder
     */
    public function scopeStored($query)
    {
        return $query->has('guitars', '=', DB::raw('quantity'));
    }

    /**
     * Pending Storage scope.
     *
     * @param $query
     *
     * @return Eloquent\Builder
     */
    public function scopePendingStorage($query)
    {
        return $query->has('guitars', '<', DB::raw('quantity'));
    }

    /**
     * Get status attribute.
     *
     * @return string
     */
    public function getStatusAttribute()
    {
        if ($this->delivery_date->isFuture()) {

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

            return self::UPCOMING;
        }
        if ( ! $this->isPendingStorage()) {
            return self::COMPLETED;
        }

        return self::DELIVERED;
    }

    /**
     * Has Guitars Pending Storage?
     *
     * @return bool
     */
    public function isPendingStorage()
    {
        return $this->guitars()->count() < $this->quantity;

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

    }

    /**
     * Guitars bought.
     *
     * @return Eloquent\Relations\HasMany
     */
    public function guitars()
    {
        return $this->hasMany(Guitar::class);
    }

    /**
     * Storage attribute.
     *
     * @return int
     */
    public function getStoredAttribute()
    {
        return $this->guitars()->count();
    }

    /**
     * Pending Storage attribute.
     *
     * @return int
     */
    public function getPendingStorageAttribute()
    {
        return $this->quantity - $this->stored;

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

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

    }

    /**
     * Number of Guitars Sold.
     *
     * @return int
     */
    public function getSoldAttribute()
    {
        return $this->guitars()->sold()->count();
    }

    /**
     * Whether the purchase has arrived.
     *
     * @return bool
     */
    public function hasArrived()
    {
        return $this->isStatus(self::DELIVERED);
    }

    /**
     * Verify the status.
     *
     * @param $status
     *
     * @return bool
     */
    public function isStatus($status)
    {
        return $this->status == $status;

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

    }

    /**
     * Supplier from which guitars were bought.
     *
     * @return Eloquent\Relations\BelongsTo
     */
    public function supplier()
    {
        return $this->belongsTo(Supplier::class);
    }

    /**
     * Make of the guitars.
     *
     * @return Eloquent\Relations\BelongsTo
     */
    public function make()
    {
        return $this->belongsTo(Make::class);
    }

    /**
     * Notifications about the purchase.
     *
     * @return Eloquent\Relations\MorphMany
     */
    public function notifications()
    {
        return $this->morphMany(Notification::class, 'notifiable');
    }

    /**
     * Create a new notification.
     * @return Notification
     */
    public function makeNotification()
    {
        $notification = $this->notifications()->create([
            'icon'    => 'flight_land',
            'link'    => '/purchases/' . $this->id,

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

            'message' => "Purchase of {$this->quantity} {$this->make->name} has arrived from {$this->supplier->name}, {$this->supplier->location}.",

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

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

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

        ]);

        event(new NotificationWasCreated($notification));

$notification of type object<Illuminate\Database\Eloquent\Model> is not a sub-type of object<App\Notification>. It seems like you assume a child class of the class Illuminate\Database\Eloquent\Model to be always present.

        return $notification;
    }
}