Eccube\Command\GenerateProxyCommand::execute()

Complexity

Conditions	14
Paths	738

Size

Total Lines	108
Code Lines	57

Duplication

Lines	7
Ratio	6.48 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        /** @var \Eccube\Application $app */
        $app = $this->getSilexApplication();

//        // プロキシのクリア

57% of this comment could be valid code. Did you maybe forget this after debugging?

//        $files = Finder::create()
//            ->in($app['config']['root_dir'].'/app/cache/doctrine/entity-proxies')
//            ->name('*.php')
//            ->files();
//        $fs = new Filesystem();
//        foreach ($files as $file) {
//            $output->writeln('remove -> '.$file->getRealPath());
//            $fs->remove($file->getRealPath());
//        }

        // Acmeからファイルを抽出
        $files = Finder::create()
            ->in(
                [
                    $app['config']['root_dir'].'/app/Acme/Entity',
                ]
            )
            ->name('*.php')
            ->files();

        // traitの一覧を取得
        $traits = [];
        $includedFiles = [];
        foreach ($files as $file) {
            require_once $file->getRealPath();
            $includedFiles[] = $file->getRealPath();
        }

        $declared = get_declared_traits();

        foreach ($declared as $className) {

This code seems to be duplicated across your project.

            $rc = new \ReflectionClass($className);
            $sourceFile = $rc->getFileName();
            if (in_array($sourceFile, $includedFiles)) {
                $traits[] = $className;
            }
        }

        // traitから@EntityExtensionを抽出
        $reader = new AnnotationReader();
        $proxies = [];
        foreach ($traits as $trait) {
            $anno = $reader->getClassAnnotation(new \ReflectionClass($trait), EntityExtension::class);

Are you sure the assignment to $anno is correct as $reader->getClassAnnotation(new \ReflectionClass($trait), \Eccube\Annotation\EntityExtension::class) (which targets Doctrine\Common\Annotations\AnnotationReader::getClassAnnotation()) seems to always return null.

            if ($anno) {
                $proxies[$anno->value][] = $trait;
            }
        }
        // プロキシファイルの生成
        foreach ($proxies as $targetEntity => $traits) {
            $rc = new \Zend\Code\Reflection\ClassReflection($targetEntity);
            $generator
                = \Zend\Code\Generator\ClassGenerator::fromReflection($rc);

            $uses = \Zend\Code\Generator\FileGenerator::fromReflectedFileName($rc->getFileName())
                ->getUses();

            foreach ($uses as $use) {
                $generator->addUse($use[0], $use[1]);
            }

            foreach ($traits as $trait) {
                $rt = new \Zend\Code\Reflection\ClassReflection($trait);
                foreach ($rt->getProperties() as $prop) {
                    // すでにProxyがある場合, $generatorにuse XxxTrait;が存在せず,

40% of this comment could be valid code. Did you maybe forget this after debugging?

                    // traitに定義されているフィールド,メソッドがクラス側に追加されてしまう
                    if ($generator->hasProperty($prop->getName())) {
                        // $generator->removeProperty()はzend-code 2.6.3 では未実装なのでリフレクションで削除.

40% of this comment could be valid code. Did you maybe forget this after debugging?

                        $generatorRefObj = new \ReflectionObject($generator);
                        $generatorRefProp = $generatorRefObj->getProperty('properties');
                        $generatorRefProp->setAccessible(true);
                        $properies = $generatorRefProp->getValue($generator);
                        unset($properies[$prop->getName()]);
                        $generatorRefProp->setValue($generator, $properies);
                    }
                }
                foreach ($rt->getMethods() as $method) {
                    if ($generator->hasMethod($method->getName())) {

Consider using $method->name. There is an issue with getName() and APC-enabled PHP versions.

                        $generator->removeMethod($method->getName());

Consider using $method->name. There is an issue with getName() and APC-enabled PHP versions.

                    }
                }
                $generator->addTrait('\\'.$trait);
            }

            // extendしたクラスが相対パスになるので
            $extendClass = $generator->getExtendedClass();
            $generator->setExtendedClass('\\'.$extendClass);

            // interfaceが相対パスになるので
            $interfaces = $generator->getImplementedInterfaces();
            foreach ($interfaces as &$interface) {
                $interface = '\\'.$interface;
            }
            $generator->setImplementedInterfaces($interfaces);

            $dir = $app['config']['root_dir'].'/app/proxy/entity';
            $file = basename($rc->getFileName());

            $code = $generator->generate();
            file_put_contents($dir.'/'.$file, '<?php '.PHP_EOL.$code);
            $output->writeln('gen -> '.$dir.'/'.$file);
        }
    }
}