PlayerComponent

Complexity

Total Complexity

10

Size/Duplication

Total Lines	57
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	getModel()	0	4	1
A	getUid()	0	4	1
A	getClubChallenge()	0	4	1
A	getNewContest()	0	5	1
A	init()	0	7	1
A	rest()	0	4	1
A	checkClubChallenge()	0	11	3
A	checkNewContest()	0	4	1

<?php
/**
 * @property Player $model
 * @property integer $uid
 * @property boolean $clubChallenge
 * @property boolean $newContest
 */
class PlayerComponent extends CApplicationComponent
{
    private $model;
    private $clubChallenge = false;
    private $newContest = false;

Equals sign not aligned with surrounding assignments; expected 4 spaces but found 1 space

    public function getModel()
    {
        return $this->model;
    }

    public function getUid()
    {
        return $this->model->uid;

The property uid cannot be accessed from this context as it is declared private in class Player.

    }

    public function getClubChallenge()
    {
        return $this->clubChallenge;
    }

    public function getNewContest()
    {
        $lastSeen = (int)Yii::app()->redis->getClient()->get('contest:lastcheck:'.$this->model->uid);

The property uid cannot be accessed from this context as it is declared private in class Player.

        return $this->newContest > $lastSeen;
    }

    public function init()
    {
        $this->model = new Player();
        $this->model->setAllAttributes();
        $this->checkClubChallenge();
        $this->checkNewContest();
    }

    public function rest()
    {
        $this->model->rest();
    }

    protected function checkClubChallenge()
    {
        if (!$this->model->in_club) {

The property in_club cannot be accessed from this context as it is declared private in class Player.

            return false;
        }

        $lastChallenge = Yii::app()->redis->getClient()->get('reminder:challenge:'.$this->model->in_club);

The property in_club cannot be accessed from this context as it is declared private in class Player.

        if ($lastChallenge >= time()) {
            $this->clubChallenge = true;
        }
    }

    protected function checkNewContest()
    {
        $this->newContest = (bool)Yii::app()->redis->getClient()->get('contest:active');
    }
}

As per coding style, files should not end with a newline character.