Irfa\Lockout\Func\Lockout

Complexity

Total Complexity

16

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Importance

Changes	3
Bugs	0	Features	2

9 Methods

Rating	Name	Duplication	Size	Complexity
A	unlock()	0	9	3
A	_lock()	0	2	1
A	message()	0	2	1
A	check()	0	14	4
A	_check()	0	2	1
A	lock()	0	9	3
A	_unlock()	0	2	1
A	clearLocked()	0	2	1
A	isLocked()	0	2	1

<?php 
namespace Irfa\Lockout\Func;

use Irfa\Lockout\Func\Core;
use View;

class Lockout extends Core {
	
    public function unlock($username){
        if(is_array($username)){
            foreach($username as $key){
                $this->_unlock($key);
            }
        } else{
            $this->_unlock($username);
        }
        return true;
    }
  	public function isLocked($username){
  		return $this->is_locked($username);
  	}
    public function message(){
        return $this->showMessage();
    }
    public function lock($username){
        if(is_array($username)){
            foreach($username as $key){
                    $this->_lock($key);
            }
        } else{
                $this->_lock($username);
        }
        return true;
    }
    public function check($username) {	 
        $ret = null;
        if(is_array($username)){
            foreach($username as $key){
                $get = $this->_check($key);
                if(!empty($get)){
                    $ret[] = $get;
                }
            }
        } else{
                $ret = $this->_check($username);
        }
        $ret = json_encode($ret);
        return json_decode($ret);
    }
    public function clearLocked(){
        return $this->clear_all();
    }
    
    private function _unlock($username){
        $this->unlock_account($username);
    }
    private function _lock($username){
        $this->lock_account($username);
    }
    private function _check($username){
        return json_decode($this->check_account($username),true);

It seems like $this->check_account($username) can also be of type false; however, parameter $json of json_decode() does only seem to accept string, maybe add an additional type check?

    }
}