Waca\DataObjects\Credential

Complexity

Total Complexity

23

Size/Duplication

Total Lines	217
Duplicated Lines	28.11 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

17 Methods

Rating	Name	Duplication	Size	Complexity
A	getUserId()	0	4	1
A	setUserId()	0	4	1
A	getFactor()	0	4	1
A	setFactor()	0	4	1
A	getType()	0	4	1
A	setType()	0	4	1
A	getData()	0	4	1
A	setData()	0	4	1
A	getVersion()	0	4	1
A	setVersion()	0	4	1
A	getTimeout()	0	8	2
A	setTimeout()	0	9	2
A	getDisabled()	0	4	1
A	setDisabled()	0	4	1
A	getPriority()	0	4	1
A	setPriority()	0	4	1
B	save()	61	61	5

<?php
/******************************************************************************
 * Wikipedia Account Creation Assistance tool                                 *
 *                                                                            *
 * All code in this file is released into the public domain by the ACC        *
 * Development Team. Please see team.json for a list of contributors.         *
 ******************************************************************************/

namespace Waca\DataObjects;

use DateTimeImmutable;
use Exception;
use Waca\DataObject;
use Waca\Exceptions\OptimisticLockFailedException;

class Credential extends DataObject
{
    /** @var int */
    private $user;
    /** @var int */
    private $factor;
    /** @var string */
    private $type;
    /** @var string */
    private $data;
    /** @var int */
    private $version;
    private $timeout;
    /** @var int */
    private $disabled = 0;
    /** @var int */
    private $priority;

    /**
     * @return int
     */
    public function getUserId()
    {
        return $this->user;
    }

    /**
     * @param int $user
     */
    public function setUserId($user)
    {
        $this->user = $user;
    }

    /**
     * @return int
     */
    public function getFactor()
    {
        return $this->factor;
    }

    /**
     * @param int $factor
     */
    public function setFactor($factor)
    {
        $this->factor = $factor;
    }

    /**
     * @return string
     */
    public function getType()
    {
        return $this->type;
    }

    /**
     * @param string $type
     */
    public function setType($type)
    {
        $this->type = $type;
    }

    /**
     * @return string
     */
    public function getData()
    {
        return $this->data;
    }

    /**
     * @param string $data
     */
    public function setData($data)
    {
        $this->data = $data;
    }

    /**
     * @return int
     */
    public function getVersion()
    {
        return $this->version;
    }

    /**
     * @param int $version
     */
    public function setVersion($version)
    {
        $this->version = $version;
    }

    /**
     * @return mixed

Consider making the return type a bit more specific; maybe use null|DateTimeImmutable.

     */
    public function getTimeout()
    {
        if ($this->timeout === null) {
            return null;
        }

        return new DateTimeImmutable($this->timeout);
    }

    /**
     * @param mixed $timeout

Consider making the type for parameter $timeout a bit more specific; maybe use null|DateTimeImmutable.

     */
    public function setTimeout(DateTimeImmutable $timeout = null)
    {
        if ($timeout === null) {
            $this->timeout = null;
        }
        else {
            $this->timeout = $timeout->format('Y-m-d H:i:s');
        }
    }

    /**
     * @return int
     */
    public function getDisabled()
    {
        return $this->disabled;
    }

    /**
     * @param int $disabled
     */
    public function setDisabled($disabled)
    {
        $this->disabled = $disabled;
    }

    /**
     * @return int
     */
    public function getPriority()
    {
        return $this->priority;
    }

    /**
     * @param int $priority
     */
    public function setPriority($priority)
    {
        $this->priority = $priority;
    }

    public function save()

This method seems to be duplicated in your project.

    {
        if ($this->isNew()) {
            // insert
            $statement = $this->dbObject->prepare(<<<SQL
INSERT INTO credential ( updateversion, user, factor, type, data, version, timeout, disabled, priority )
VALUES ( 0, :user, :factor, :type, :data, :version, :timeout, :disabled, :priority );
SQL
            );
            $statement->bindValue(":user", $this->user);
            $statement->bindValue(":factor", $this->factor);
            $statement->bindValue(":type", $this->type);
            $statement->bindValue(":data", $this->data);
            $statement->bindValue(":version", $this->version);
            $statement->bindValue(":timeout", $this->timeout);
            $statement->bindValue(":disabled", $this->disabled);
            $statement->bindValue(":priority", $this->priority);

            if ($statement->execute()) {
                $this->id = (int)$this->dbObject->lastInsertId();
            }
            else {
                throw new Exception($statement->errorInfo());
            }
        }
        else {
            // update
            $statement = $this->dbObject->prepare(<<<SQL
                UPDATE credential
                SET   factor = :factor
                    , data = :data
                    , version = :version
                    , timeout = :timeout
                    , disabled = :disabled
                    , priority = :priority
                    , updateversion = updateversion + 1
                WHERE id = :id AND updateversion = :updateversion;
SQL
            );

            $statement->bindValue(':id', $this->id);
            $statement->bindValue(':updateversion', $this->updateversion);

            $statement->bindValue(":factor", $this->factor);
            $statement->bindValue(":data", $this->data);
            $statement->bindValue(":version", $this->version);
            $statement->bindValue(":timeout", $this->timeout);
            $statement->bindValue(":disabled", $this->disabled);
            $statement->bindValue(":priority", $this->priority);

            if (!$statement->execute()) {
                throw new Exception($statement->errorInfo());
            }

            if ($statement->rowCount() !== 1) {
                throw new OptimisticLockFailedException();
            }

            $this->updateversion++;
        }
    }
}