IlyasKazi\Instamojo\Models\InstamojoPayment

Complexity

Total Complexity

2

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	boot()	0	10	1
A	user()	0	3	1

<?php

namespace IlyasKazi\Instamojo\Models;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\SoftDeletes;

class InstamojoPayment extends Model
{
    use SoftDeletes;

    /**
     * The table associated with the model.
     *
     * @var string
     */
    protected $table = 'instamojo_payment';

    /**
     * The attributes that are mass assignable.
     *
     * @var array
     */
    protected $fillable = ['user_id','buyer_email','buyer_name','buyer_phone','currency','amount','unit_price', 'fees','longurl','payment_id', 'payment_request_id','purpose','shorturl','request_status','payment_status', 'affiliate_commission', 'instrument_type', 'billing_instrument', 'created_by'];
    
    
    /**
     * The "booting" method of the model.
     *
     * @return void
     */
    public static function boot()
    {
        parent::boot();

        static::creating(function($model) {
            $model->created_by = \Auth::user()->id;

Accessing id on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

        });

        static::updating(function($model) {
            $model->updated_by = \Auth::user()->id;

Accessing id on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

        });
    }
    
    /**
     * [user description]
     * @return [type] [description]

The doc comment [type] at position 0 could not be parsed: Unknown type name '[' at position 0 in [type].

     */
    public function user()
    {
        return $this->hasOne('App\User', 'id', 'user_id');
    }
}