Jerodev\PhpIrcClient\Options\ClientOptions

Complexity

Total Complexity

3

Size/Duplication

Total Lines	65
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

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

<?php

namespace Jerodev\PhpIrcClient\Options;

class ClientOptions
{
    /**
     *  Automaticly connect to the irc server when creating the client.
     *
     *  @var bool
     */
    public $autoConnect;

    /**
     *  Automaticly rejoin a channel when kicked.
     *
     *  @var bool
     */
    public $autoRejoin;

    /**
     *  The channel names to join when connecting.
     *
     *  @var string[]
     */
    public $channels;

    /**
     *  The amount of time in milliseconds to wait between sending messages to the irc server.
     *
     *  @var int
     */
    public $floodProtectionDelay;

    /**
     *  The nickname to use on the server.
     *
     *  @var null|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?

        $this->autoRejoin = false;
        $this->autoConnect = false;
    }

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

        return $options;
    }
}