Distilleries\Expendable\Models\LockableTrait

Complexity

Total Complexity

4

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Test Coverage

Coverage

40%

Importance

Changes	2
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	unlock()	0	9	1
A	lock()	0	9	1
A	isLocked()	0	4	1
A	incrementLock()	0	8	1

<?php namespace Distilleries\Expendable\Models;


trait LockableTrait
{
    protected $config_key_security_lock = 'expendable.auth.nb_of_try';
    protected $column_nb_of_try_name = 'nb_of_try';

    public function isLocked()
    {

        return ($this->{$this->column_nb_of_try_name} >= config($this->config_key_security_lock));

    }

    public function incrementLock()
    {

        $this->{$this->column_nb_of_try_name} += 1;
        \DB::table($this->getTable())

It seems like getTable() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            ->where($this->getKeyName(), $this->getKey())

It seems like getKeyName() must be provided by classes using this trait. How about adding it as abstract method to this trait?

It seems like getKey() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            ->increment($this->column_nb_of_try_name);
        return $this;

    }

    public function unlock()
    {

        \DB::table($this->getTable())
            ->where($this->getKeyName(), $this->getKey())
            ->decrement($this->column_nb_of_try_name, $this->{$this->column_nb_of_try_name});

        $this->{$this->column_nb_of_try_name} = 0;
        return $this;

    }

    public function lock()
    {
        $this->{$this->column_nb_of_try_name} = config($this->config_key_security_lock) + 1;

        \DB::table($this->getTable())
            ->where($this->getKeyName(), $this->getKey())
            ->increment($this->column_nb_of_try_name, $this->{$this->column_nb_of_try_name} );

        return $this;
    }

}