Crummy\Phlack\Common\Formatter\Sequencer

Complexity

Total Complexity

5

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes	3
Bugs	2	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	format()	0	4	1
A	alert()	0	4	1
A	sequence()	0	6	2
A	command()	0	7	1

<?php

namespace Crummy\Phlack\Common\Formatter;

use Crummy\Phlack\WebHook\CommandInterface;

class Sequencer implements FormatterInterface
{
    const SEQUENCE = '<%s>';

    /**
     * @param string $text  The text to be sequenced
     * @param string $label An optional label
     *
     * @return string
     */
    public function format($text, $label = null)
    {
        return $this->sequence($text, $label);

It seems like $label defined by parameter $label on line 17 can also be of type string; however, Crummy\Phlack\Common\Formatter\Sequencer::sequence() does only seem to accept null, maybe add an additional type check?

    }

    /**
     * @param string $text
     * @param null   $label
     *
     * @return string
     */
    public static function sequence($text, $label = null)
    {
        $text = $label ? $text.'|'.$label : $text;

        return sprintf(static::SEQUENCE, $text);
    }

    /**
     * @param CommandInterface $command
     *
     * @return array
     */
    public static function command(CommandInterface $command)
    {
        return [
            'channel'   => self::sequence('#'.$command['channel_id'], $command['channel_name']),
            'user'      => self::sequence('@'.$command['user_id'], $command['user_name']),
        ];
    }

    /**
     * @param string $channel
     *
     * @return string
     */
    public static function alert($channel)
    {
        return self::sequence('!'.$channel);
    }
}