Shopware\Psh\ScriptRuntime\ScriptLoader\BashScriptParser

Complexity

Total Complexity

10

Size/Duplication

Total Lines	66
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	parseContent()	0	13	2
A	testScriptFileFitsRequirements()	0	10	3
A	testContentContainsMarker()	0	6	2
A	shouldWarnAboutBestPractice()	0	11	3

<?php declare(strict_types=1);

namespace Shopware\Psh\ScriptRuntime\ScriptLoader;

use Shopware\Psh\Listing\Script;
use Shopware\Psh\ScriptRuntime\BashCommand;

class BashScriptParser implements ScriptParser
{
    const TYPE_DIRECT_EXECUTE = '<PSH_EXECUTE_THROUGH_CMD>';

    /**
     * @todo add validation and notice
     */
    const SHOULD_BE_PRESENT = "set -euo pipefail";

    /**
     * {@inheritdoc}
     */
    public function parseContent(string $content, Script $script): array
    {
        $this->testContentContainsMarker($content);
        $this->testScriptFileFitsRequirements($script);

        $shouldWarn = $this->shouldWarnAboutBestPractice($content);

        if ($shouldWarn) {
            $warning = self::SHOULD_BE_PRESENT;

$warning is not used, you could remove the assignment.

        }

        return [new BashCommand($script)];
    }

    /**
     * @param Script $script
     */
    private function testScriptFileFitsRequirements(Script $script)
    {
        if (!is_executable($script->getPath())) {
            throw new \RuntimeException('Bash scripts can only be executed if they are executable please execute <bold>chmod +x ' . $script->getPath() . '</bold>');
        }

        if (!is_writable($script->getDirectory())) {
            throw new \RuntimeException('Bash scripts can only be executed if they are in a writable directory please execute <bold>chmod +w ' . $script->getDirectory() . '</bold>');
        }
    }

    /**
     * @param string $content
     */
    private function testContentContainsMarker(string $content)
    {
        if (strpos($content, self::TYPE_DIRECT_EXECUTE) === false) {
            throw new ScriptNotSupportedByParser('Marker for execution missing');
        }
    }

    /**
     * @param string $content
     * @return bool
     */
    private function shouldWarnAboutBestPractice(string $content): bool
    {
        $firstLines = explode("\n", $content, 5);
        foreach ($firstLines as $line) {
            if ($line === self::SHOULD_BE_PRESENT) {
                return false;
            }
        }

        return true;
    }
}