loophp\phptree\Exporter\Image

Complexity

Total Complexity

9

Size/Duplication

Total Lines	96
Duplicated Lines	0 %

Test Coverage

Coverage

92%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	getConvertCommand()	0	7	1
A	setExecutable()	0	5	1
A	__construct()	0	4	2
A	getFormat()	0	3	1
A	export()	0	9	2
A	setFormat()	0	5	1
A	getExecutable()	0	3	1

<?php

declare(strict_types=1);

namespace loophp\phptree\Exporter;

use Exception;
use loophp\phptree\Node\NodeInterface;

use const PHP_OS;

/**
 * Class Image.
 */
final class Image extends AbstractExporter
{
    /**
     * @var string
     */
    private $executable = 'dot';

    /**
     * @var string
     */
    private $format = 'svg';

    /**
     * Image constructor.
     */
    public function __construct()
    {
        if (0 === mb_stripos(PHP_OS, 'WIN')) {
            $this->executable = 'dot.exe';
        }
    }

    /**
     * @param \loophp\phptree\Node\NodeInterface $node
     *
     * @throws Exception
     *
     * @return string
     */
    public function export(NodeInterface $node): string
    {
        if (false === $tmp = tempnam(sys_get_temp_dir(), 'phptree-export-')) {
            return '';
        }

        file_put_contents($tmp, (new Gv())->export($node));

        return shell_exec($this->getConvertCommand($tmp));

The expression return shell_exec($this->getConvertCommand($tmp)) could return the type null which is incompatible with the type-hinted return string. Consider adding an additional type-check to rule them out.

    }

    /**
     * Get the executable to use.
     *
     * @return string
     */
    public function getExecutable(): string
    {
        return $this->executable;
    }

    /**
     * @return string
     */
    public function getFormat(): string
    {
        return $this->format;
    }

    /**
     * Change the executable to use.
     *
     * @param string $executable
     *
     * @return \loophp\phptree\Exporter\Image
     */
    public function setExecutable(string $executable): self
    {
        $this->executable = $executable;

        return $this;
    }

    /**
     * @param string $format
     *
     * @return \loophp\phptree\Exporter\Image
     */
    public function setFormat(string $format): self
    {
        $this->format = $format;

        return $this;
    }

    /**
     * @param string $path
     *
     * @return string
     */
    private function getConvertCommand(string $path): string
    {
        return sprintf(
            '%s -T%s %s',
            $this->getExecutable(),
            $this->getFormat(),
            $path
        );
    }
}