Czogori\DamiBundle\Helper\PreparationMigrationDirectory

Complexity

Total Complexity

6

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	7
Bugs	2	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
B	prepare()	0	19	5

<?php

namespace Czogori\DamiBundle\Helper;

use Symfony\Component\Finder\Finder;
use Symfony\Component\Filesystem\Filesystem;

class PreparationMigrationDirectory
{
    private $baseDirectory;
    private $bundles;

    /**
     * Constructor
     *
     * @param string                                                $baseDirectory Base directory for migrations.
     * @param Symfony\Component\HttpKernel\Bundle\BundleInterface[] $bundles       Array of BunleInterface instances.
     */
    public function __construct($baseDirectory, $bundles)
    {
        $this->baseDirectory = $baseDirectory;
        $this->bundles = $bundles;
        $this->fileSystem = new Filesystem();

The property fileSystem does not exist. Did you maybe forget to declare it?

    }

    /**
     * Prepare migration directory to use.
     *
     * @return void
     */
    public function prepare()
    {
        $migrationDirectory = $this->baseDirectory;
        if ($this->fileSystem->exists($migrationDirectory)) {
            $this->fileSystem->remove(array($migrationDirectory));
        }
        $this->fileSystem->mkdir($migrationDirectory);

        foreach ($this->bundles as $bundle) {
            $bundleMigrationDirectory = $bundle->getPath() . '/Resources/migrations';
            if (file_exists($bundleMigrationDirectory)) {
                $finder = new Finder();
                $finder->files('*.php')->in($bundleMigrationDirectory);

The call to Finder::files() has too many arguments starting with '*.php'.

                foreach ($finder as $file) {
                   $this->fileSystem->copy($file->getRealpath(), $migrationDirectory . '/' . $file->getFilename(), true);
                }
            }
        }
    }
}