Starkerxp\StructureBundle\Generator\AbstractGenerator

Complexity

Total Complexity

24

Size/Duplication

Total Lines	117
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	setKernel()	0	6	1
A	traiterLesFichiers()	0	16	3
	getFichiers()	0	1	?
F	traiterLeFichier()	0	47	18
A	genererParameters()	0	21	1

<?php

namespace Starkerxp\StructureBundle\Generator;

use Sensio\Bundle\GeneratorBundle\Generator\Generator;
use Symfony\Component\HttpKernel\KernelInterface;
use Symfony\Component\Yaml\Yaml;

abstract class AbstractGenerator extends Generator
{
    /**
     * @var KernelInterface
     */
    protected $kernel;

    /**
     * AbstractGenerator constructor.
     * @param KernelInterface $kernel
     */
    public function __construct(KernelInterface $kernel)
    {
        $this->kernel = $kernel;
    }

    /**
     * @param KernelInterface $kernel
     * @return AbstractGenerator
     */
    public function setKernel(KernelInterface $kernel)
    {
        $this->kernel = $kernel;

        return $this;
    }

    protected function traiterLesFichiers($bundle, $parameters, $search, $replace)
    {
        foreach ($this->getFichiers() as $template) {
            try {
                $this->kernel->locateResource("@StarkerxpStructureBundle/Resources/views/Gabarit/".$template.".twig");
            } catch (\InvalidArgumentException $e) {
                throw new \InvalidArgumentException("Il manque un fichier de template ".$template);
            }
            $fichierACreerModifier = $bundle->getPath().str_replace(
                    $search,
                    $replace,
                    $template
                );
            $this->traiterLeFichier($template, $fichierACreerModifier, $parameters);
        }
    }

    public abstract function getFichiers();

The abstract declaration must precede the visibility declaration

    public function traiterLeFichier($fichier, $target, $parameters)
    {
        if (file_exists($target) && explode('.', basename($target))[1] == 'yml') {
            $currentServices = Yaml::parse(file_get_contents($target));
            $newServices = Yaml::parse($this->render($fichier.'.twig', $parameters));
            // gestion de fichiers sans services ni parameters => routing
            $listeDesParametres = !empty($currentServices['parameters']) ? array_keys($currentServices['parameters']) : [];
            $listeDesServices = !empty($currentServices['services']) ? array_keys($currentServices['services']) : [];
            $listeDesDeclarations = !empty($currentServices) ? array_keys($currentServices) : [];
            $listeDesNouveauxParametres = !empty($newServices['parameters']) ? array_keys($newServices['parameters']) : [];
            $listeDesNouveauxServices = !empty($newServices['services']) ? array_keys($newServices['services']) : [];
            $listeDesNouvellesDeclarations = !empty($newServices) ? array_keys($newServices) : [];

            $diffServices = array_diff($listeDesNouveauxServices, $listeDesServices);
            $diffParametres = array_diff($listeDesNouveauxParametres, $listeDesParametres);
            $diffDeclarations = array_diff($listeDesNouvellesDeclarations, $listeDesDeclarations);
            if (empty($diffServices) && empty($diffParametres) && empty($diffDeclarations)) {
                return false;
            }
            foreach ($diffServices as $libelle) {
                $currentServices['services'][$libelle] = $newServices['services'][$libelle];
            }
            foreach ($diffParametres as $libelle) {
                $currentServices['parameters'][$libelle] = $newServices['parameters'][$libelle];
            }
            foreach ($diffDeclarations as $libelle) {
                $currentServices[$libelle] = $newServices[$libelle];
            }
            $content = Yaml::dump($currentServices, 3, 4);
            $flink = fopen($target, 'w');
            if ($flink) {
                $write = fwrite($flink, $content);
                if ($write) {
                    fclose($flink);
                }
            }

            return true;
        }
        // On ne modifie pas un fichier existant.
        if (file_exists($target)) {
            return false;
        }
        $this->renderFile($fichier.'.twig', $target, $parameters);

        return true;
    }

    protected function genererParameters($clef, $controller)
    {
        $controller = explode(':', $controller);

Consider using a different name than the parameter $controller. This often makes code more readable.

        $bundle = $this->kernel->getBundle($controller[0]);
        $libelle = ucfirst($controller[1]);

        $clef = ucfirst(strtolower($clef));

Consider using a different name than the parameter $clef. This often makes code more readable.

        $parameters = [
            'nom'.$clef => $libelle,
            'nom'.$clef.'Camelize' => preg_replace('#\B([A-Z])#', '_\1', $libelle),
            'namespace'.$clef => $bundle->getNamespace(),
            'namespace'.$clef.'Bundle' => '@'.$bundle->getName(),
            'namespace'.$clef.'FQC' => str_replace('\\', '\\\\', $bundle->getNamespace()),
        ];
        $parameters['nomService'.$clef] = strtolower(
            str_replace(['_Bundle', '@'], '', preg_replace('#\B([A-Z])#', '_\1', $parameters['namespace'.$clef.'Bundle']))
        );

        return $parameters;

    }

}

As per coding style, files should not end with a newline character.