Fabrica\Tools\Plugin\Irc

Complexity

Total Complexity

14

Size/Duplication

Total Lines	107
Duplicated Lines	7.48 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	pluginName()	0	4	1
A	__construct()	8	16	2
A	execute()	0	27	5
A	executeIrcCommands()	0	21	5
A	executeIrcCommand()	0	4	1

<?php

namespace Fabrica\Tools\Plugin;

use Fabrica\Tools\Builder;
use Fabrica\Models\Infra\Ci\Build;
use Fabrica\Tools\Plugin;

/**
 * IRC Plugin - Sends a notification to an IRC channel
 *
 * @author Ricardo Sierra <ricardo@sierratecnologia.com>
 */
class Irc extends Plugin
{
    protected $message;
    protected $server;
    protected $port;
    protected $room;
    protected $nick;

    /**
     * @return string
     */
    public static function pluginName()
    {
        return 'irc';
    }
    
    /**
     * {@inheritdoc}
     */
    public function __construct(Builder $builder, Build $build, array $options = [])
    {
        parent::__construct($builder, $build, $options);

        $this->message = $options['message'];
        $buildSettings = $this->builder->getConfig('build_settings');

        if (isset($buildSettings['irc'])) {

This code seems to be duplicated across your project.

            $irc = $buildSettings['irc'];

            $this->server = $irc['server'];
            $this->port   = $irc['port'];
            $this->room   = $irc['room'];
            $this->nick   = $irc['nick'];
        }
    }

    /**
     * Run IRC plugin.
     *
     * @return bool
     */
    public function execute()
    {
        $msg = $this->builder->interpolate($this->message);

        if (empty($this->server) || empty($this->room) || empty($this->nick)) {
            $this->builder->logFailure('You must configure a server, room and nick.');
        }

        if (empty($this->port)) {
            $this->port = 6667;
        }

        $sock = fsockopen($this->server, $this->port);
        stream_set_timeout($sock, 1);

        $connectCommands = [
            'USER ' . $this->nick . ' 0 * :' . $this->nick,
            'NICK ' . $this->nick,
        ];
        $this->executeIrcCommands($sock, $connectCommands);
        $this->executeIrcCommand($sock, 'JOIN ' . $this->room);
        $this->executeIrcCommand($sock, 'PRIVMSG ' . $this->room . ' :' . $msg);

        fclose($sock);

        return true;
    }

    /**
     * @param  resource $socket
     * @param  array    $commands
     * @return bool
     */
    private function executeIrcCommands($socket, array $commands)
    {
        foreach ($commands as $command) {
            fputs($socket, $command . "\n");
        }

        $pingBack = false;

        // almost all servers expect pingback!
        while ($response = fgets($socket)) {
            $matches = [];
            if (preg_match('/^PING \\:([A-Z0-9]+)/', $response, $matches)) {
                $pingBack = $matches[1];
            }
        }

        if ($pingBack) {

The expression $pingBack of type string|false is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== false instead.

            $command = 'PONG :' . $pingBack . "\n";
            fputs($socket, $command);
        }
    }

    /**
     *
     * @param  resource $socket
     * @param  string   $command
     * @return bool
     */
    private function executeIrcCommand($socket, $command)
    {
        return $this->executeIrcCommands($socket, [$command]);
    }
}