hiqdev\php\confirmator\ServiceTrait

Complexity

Total Complexity

10

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

85%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	issueToken()	0	7	1
A	checkToken()	0	9	2
A	findToken()	0	10	3
A	removeToken()	0	4	1
A	readToken()	0	4	2
A	writeToken()	0	4	1

<?php

/*
 * Library for confirmation tokens
 *
 * @link      https://github.com/hiqdev/php-confirmator
 * @package   php-confirmator
 * @license   BSD-3-Clause
 * @copyright Copyright (c) 2016, HiQDev (http://hiqdev.com/)
 */

namespace hiqdev\php\confirmator;

trait ServiceTrait
{
    public function issueToken(array $data)
    {
        $token = new Token($this, $data);
        $this->writeToken($token);

        return $token;
    }

    public function checkToken($token, array $data)
    {
        $token = $this->findToken($token);
        if (!$token) {
            return false;
        }

        return $token->check($data);
    }

    public function findToken($token)
    {
        if ($token instanceof Token) {
            return $token;
        }

        $data = $this->readToken($token);

        return empty($data) ? null : new Token($this, $data, $token);
    }

    public function removeToken($token)
    {
        return $this->getStorage()->remove((string) $token);

It seems like getStorage() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    protected function readToken($string)
    {
        return $this->getStorage()->has($string) ? json_decode($this->getStorage()->get($string), true) : null;

It seems like getStorage() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    protected function writeToken(Token $token)
    {
        return $this->getStorage()->set((string) $token, json_encode($token->mget()));

It seems like getStorage() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }
}