Kafkiansky\SmsRu\Message\To

Complexity

Total Complexity

5

Size/Duplication

Total Lines	59
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	useIconv()	0	3	1
A	getMessage()	0	3	1
A	asMulti()	0	3	1
A	__construct()	0	5	1
A	getTo()	0	3	1

<?php

declare(strict_types=1);

namespace Kafkiansky\SmsRu\Message;

final class To extends Recipient
{
    /**
     * @var string
     */
    private $message;

    /**
     * @var array
     */
    private $tos;

    /**
     * @var bool
     */
    private $useIconv;

    /**
     * @param string|array $tos
     * @param string       $message
     * @param bool         $useIconv
     */
    public function __construct($tos, string $message, bool $useIconv = false)
    {
        $this->message = $message;
        $this->tos = $tos;

It seems like $tos can also be of type string. However, the property $tos is declared as type array. Maybe add an additional type check?

        $this->useIconv = $useIconv;
    }

    /**
     * @return array|string
     */
    public function getTo()
    {
        return $this->tos;
    }

    /**
     * @return string
     */
    public function getMessage(): string
    {
        return $this->message;
    }

    /**
     * {@inheritdoc}
     */
    public function useIconv(): bool
    {
        return $this->useIconv;
    }

    /**
     * @return bool
     */
    public function asMulti(): bool
    {
        return false;
    }
}