Apix\Log\Emitter\Async

Complexity

Total Complexity

6

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	2
A	send()	0	28	4

<?php

namespace Apix\Log\Emitter;

use Psr\Log\InvalidArgumentException;

class Async extends AbstractEmitter
{
    /**
     * Holds the emitter/transporter's command string.
     * The placeholders in order are:
     *      - #1.    (string)    payload
     *      - #2.    (string)    url.
     */
    protected $transporter_cmd = 'curl -X POST -d %1$s \'%2$s\'';

    /**
     * Constructor.
     *
     * @param string $transporter_cmd The tranpsort command string
     */
    public function __construct($transporter_cmd = null)
    {
        $this->transporter_cmd = $transporter_cmd ?: $this->transporter_cmd;
    }

    /**
     * {@inheritdoc}
     */
    public function send($payload)
    {
        if (!is_string($payload)) {
            throw new InvalidArgumentException(sprintf(
                'Expects a string, got: (%s) %s.',
                gettype($payload), json_encode($payload)
            ));
        }

        $cmd = sprintf(
            $this->transporter_cmd,
            escapeshellarg($payload),
            $this->url // urlencode($this->url)
        );

        if (!$this->debug) {
            $cmd .= ' > /dev/null 2>&1 &';
            // $cmd .= ' >&- 2>&- &';
        }

        exec($cmd, $output, $return_var);

        if ($return_var != 0) {

It seems like you are loosely comparing $return_var of type integer|null to 0; this is ambiguous as not only 0 == 0 is true, but null == 0 is true, too. Consider using a strict comparison ===.

            $this->handleError($return_var, $output);

$output is of type null|array, but the function expects a string.

        }

        return $return_var == 0;

It seems like you are loosely comparing $return_var of type integer|null to 0; this is ambiguous as not only 0 == 0 is true, but null == 0 is true, too. Consider using a strict comparison ===.

    }
}