NotificationChannels\Line\LineMessage

Complexity

Total Complexity

3

Size/Duplication

Total Lines	24
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	16	3

<?php

namespace NotificationChannels\Line;

use NotificationChannels\Line\Exceptions\CouldNotSendNotification;
use LINE\LINEBot\MessageBuilder\TextMessageBuilder;

class LineMessage implements TextMessageBuilder
{
    /** @var string[] */
    private $texts;
    /** @var array */
    private $message = [];

The property $message is not used and could be removed.

    public function __construct($text, $extraTexts = null)
    {
        $extra = [];
        if (!is_null($extraTexts)) {
            $args = func_get_args();
            $extra = array_slice($args, 1);
        }

        if (count($extra) >= 4) {
            throw CouldNotSendNotification::exceededOfMessages(count($extra) + 1);
        }

        $this->texts = array_merge([$text], $extra);

It seems like array_merge(array($text), $extra) of type array is incompatible with the declared type array<integer,string> of property $texts.

        $this->contents = $contents;

The property contents does not exist. Did you maybe forget to declare it?

The variable $contents does not exist. Did you forget to declare it?

    }
}