ApiKey

Complexity

Total Complexity

11

Size/Duplication

Total Lines	91
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	assignResult()	0	7	1
A	createKey()	0	11	1
A	getKeyByOwner()	0	10	2
A	getKey()	0	4	1
A	getName()	0	4	1
A	getOwner()	0	4	1
A	getStatus()	0	4	1
A	getKeys()	0	10	2
A	generateKey()	0	10	1

<?php

class ApiKey extends Model
{
    protected $name;

    protected $owner;

    protected $key;

    protected $status;

    /**
     * The name of the database table used for queries
     */
    const TABLE = "api_keys";

    /**
     * {@inheritdoc}
     */
    protected function assignResult($key)
    {
        $this->name   = $key['name'];
        $this->owner  = Player::get($key['owner']);
        $this->key    = $key['key'];
        $this->status = $key['status'];
    }

    public static function createKey($name, $owner)
    {
        $key = self::create(array(
            'name'   => $name,
            'owner'  => $owner,
            'key'    => self::generateKey(),
            'status' => "active"
        ), 'siss');

        return $key;
    }

    public static function getKeyByOwner($owner)
    {
        $key = parent::fetchIdFrom($owner, "owner", 'i');

It seems like you call parent on a different method (fetchIdFrom() instead of getKeyByOwner()). Are you sure this is correct? If so, you might want to change this to $this->fetchIdFrom().

        if ($key == null) {
            return self::createKey("Automatically generated key", $owner);
        }

        return self::get($key);
    }

    public function getKey()
    {
        return $this->key;
    }

    public function getName()
    {
        return $this->name;
    }

    public function getOwner()
    {
        return $this->owner;
    }

    public function getStatus()
    {
        return $this->status;
    }

    public static function getKeys($owner = -1)
    {
        if ($owner > 0) {
            $ids = self::fetchIdsFrom("owner", array($owner), "i", false, "WHERE status = 'active'");
            return self::arrayIdToModel($ids);
        }

        $ids = self::fetchIdsFrom("status", array("active"), "s");
        return self::arrayIdToModel($ids);
    }

    protected static function generateKey()
    {
        list($usec, $sec) = explode(' ', microtime());
        $seed = (float) $sec + ((float) $usec * 100000);
        srand($seed);

        $key = rand();

        return substr(sha1($key), 24);
    }
}