Cakasim\Payone\Sdk\Api\Message\Payment\Parameter\SequenceNumber

Complexity

Total Complexity

3

Size/Duplication

Total Lines	23
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	setSequenceNumber()	0	4	1
A	getSequenceNumber()	0	6	2

<?php

declare(strict_types=1);

namespace Cakasim\Payone\Sdk\Api\Message\Payment\Parameter;

/**
 * Implements getter and setter for the sequence number API request parameter.
 *
 * @author Fabian Böttcher <me@cakasim.de>
 * @since 0.1.0
 */
trait SequenceNumber
{
    /**
     * @inheritDoc
     */
    public function getSequenceNumber(): ?int
    {
        $sequenceNumber = $this->getParameter('sequencenumber');

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

        return $sequenceNumber !== null
            ? (int) $sequenceNumber
            : null;
    }

    /**
     * Sets the sequence number.
     *
     * @param int $sequenceNumber The sequence number.
     * @return $this
     */
    public function setSequenceNumber(int $sequenceNumber): self
    {
        $this->setParameter('sequencenumber', (string) $sequenceNumber);

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

        return $this;
    }
}