Waca\DataObjects\OAuthToken

Complexity

Total Complexity

15

Size/Duplication

Total Lines	152
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

11 Methods

Rating	Name	Duplication	Size	Complexity
B	save()	54	54	5
A	getUserId()	4	4	1
A	setUserId()	4	4	1
A	getToken()	4	4	1
A	setToken()	4	4	1
A	getSecret()	4	4	1
A	setSecret()	4	4	1
A	getType()	4	4	1
A	setType()	4	4	1
A	getExpiry()	4	4	1
A	setExpiry()	4	4	1

<?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 Exception;
use Waca\DataObject;
use Waca\Exceptions\OptimisticLockFailedException;

class OAuthToken extends DataObject

This class seems to be duplicated in your project.

{
    /** @var int */
    private $user;
    /** @var string */
    private $token;
    /** @var string */
    private $secret;
    /** @var string */
    private $type;
    /** @var string */
    private $expiry;

    public function save()
    {
        if ($this->isNew()) {
            // insert
            $statement = $this->dbObject->prepare(<<<SQL
                INSERT INTO oauthtoken ( user, token, secret, type, expiry )
                VALUES ( :user, :token, :secret, :type, :expiry );
SQL
            );
            $statement->bindValue(":user", $this->user);
            $statement->bindValue(":token", $this->token);
            $statement->bindValue(":secret", $this->secret);
            $statement->bindValue(":type", $this->type);
            $statement->bindValue(":expiry", $this->expiry);

            if ($statement->execute()) {
                $this->id = (int)$this->dbObject->lastInsertId();
            }
            else {
                throw new Exception($statement->errorInfo());
            }
        }
        else {
            // update
            $statement = $this->dbObject->prepare(<<<SQL
                UPDATE oauthtoken
                SET   token = :token
                    , secret = :secret
                    , type = :type
                    , expiry = :expiry
                    , updateversion = updateversion + 1
                WHERE id = :id AND updateversion = :updateversion;
SQL
            );

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

            $statement->bindValue(":token", $this->token);
            $statement->bindValue(":secret", $this->secret);
            $statement->bindValue(":type", $this->type);
            $statement->bindValue(":expiry", $this->expiry);

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

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

            $this->updateversion++;
        }
    }

    #region properties

    /**
     * @return mixed

Consider making the return type a bit more specific; maybe use integer.

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

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

    /**
     * @return mixed

Consider making the return type a bit more specific; maybe use string.

     */
    public function getToken()
    {
        return $this->token;
    }

    /**
     * @param mixed $token
     */
    public function setToken($token)
    {
        $this->token = $token;
    }

    /**
     * @return mixed

Consider making the return type a bit more specific; maybe use string.

     */
    public function getSecret()
    {
        return $this->secret;
    }

    /**
     * @param mixed $secret
     */
    public function setSecret($secret)
    {
        $this->secret = $secret;
    }

    /**
     * @return mixed

Consider making the return type a bit more specific; maybe use string.

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

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

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

    /**
     * @param string $expiry
     */
    public function setExpiry($expiry)
    {
        $this->expiry = $expiry;
    }
    #endregion

}