Jerodev\PhpIrcClient\Options\ClientOptions

Complexity

Total Complexity

3

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	2
A	connectionOptions()	0	6	1

<?php

namespace Jerodev\PhpIrcClient\Options;

class ClientOptions
{
    /** @var string[] */
    public $channels;

    /** @var int */
    public $floodProtectionDelay;

    /** @var string */
    public $nickname;

    /**
     *  @param string $nickname The nickname used on the irc server.
     *  @param string|string[] $channels The channels to join on connection.
     */
    public function __construct(string $nickname = null, $channels = [])
    {
        $this->nickname = $nickname;

        if (!is_array($channels)) {
            $channels = [$channels];
        }
        $this->channels = $channels;

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

    }

    /**
     *  Get the options for the IrcConnection from this collection.
     *
     *  @return ConnectionOptions
     */
    public function connectionOptions(): ConnectionOptions
    {
        $options = new ConnectionOptions();
        $options->floodProtectionDelay = $this->floodProtectionDelay;

        return $options;
    }
}