Cherrypulp\LaravelPackageGenerator\Commands\Traits\ManipulatesPackageFolder

Complexity

Total Complexity

6

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	removePackageFolder()	0	12	3
A	createPackageFolder()	0	15	3

<?php

namespace Cherrypulp\LaravelPackageGenerator\Commands\Traits;

use Illuminate\Support\Facades\File;
use Cherrypulp\LaravelPackageGenerator\Exceptions\RuntimeException;

trait ManipulatesPackageFolder
{
    /**
     * Create package folder.
     *
     * @param string $packagePath
     *
     * @throws RuntimeException
     */
    protected function createPackageFolder($packagePath)
    {
        $this->info('Create package folder.');

It seems like info() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        if (File::exists($packagePath)) {
            $this->info('Package folder already exists. Skipping.');

            return;
        }

        if (! File::makeDirectory($packagePath, 0755, true)) {
            throw new RuntimeException('Cannot create package folder');
        }

        $this->info('Package folder was successfully created.');
    }

    /**
     * Remove package folder.
     *
     * @param $packagePath
     *
     * @throws RuntimeException
     */
    protected function removePackageFolder($packagePath)
    {
        $this->info('Remove package folder.');

        if (File::exists($packagePath)) {
            if (! File::deleteDirectory($packagePath)) {
                throw new RuntimeException('Cannot remove package folder');
            }

            $this->info('Package folder was successfully removed.');
        } else {
            $this->info('Package folder does not exists. Skipping.');
        }
    }
}