App\Support\Console\DockerCompose\YamlBuilder

Complexity

Total Complexity

2

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Test Coverage

Coverage

88.24%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	destroy()	0	3	1
A	build()	0	15	1

<?php

namespace App\Support\Console\DockerCompose;

use App\Models\PhpVersion;
use App\Support\Contracts\ImageRepository;

class YamlBuilder
{
    /**
     * Build the docker-compose.yaml file
     *
     * @param $imageSet
     * @throws \Throwable
     */
    public function build(ImageRepository $imageSet)
    {
        file_put_contents(
            config('porter.docker-compose-file'),
            view("docker_compose.{$imageSet->getName()}.base")->with([
                'home' => setting('home'),
                'host_machine_name' => setting('host_machine_name'),
                'activePhpVersions' => PhpVersion::active()->get(),
                'useMysql' => setting('use_mysql') == 'on',
                'useRedis' => setting('use_redis') == 'on',
                'useBrowser' => setting('use_browser') == 'on',
                'imageSet' => $imageSet->getName(),
                'imageSetPath' => $imageSet->getPath(),
                'libraryPath' => config('porter.library_path'),
            ])->render()
        );
    }

    /**
     * Destroy the docker-compose.yaml file
     */
    public function destroy()
    {
        @unlink(config('porter.docker-compose-file'));

It seems like you do not handle an error condition for unlink(). This can introduce security issues, and is generally not recommended.

    }
}