Eccube\Command\GenProxyCommand

Complexity

Total Complexity

15

Size/Duplication

Total Lines	118
Duplicated Lines	5.93 %

Coupling/Cohesion

Components	0
Dependencies	9

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	6	1
F	execute()	7	108	14

<?php
/*
 * This file is part of EC-CUBE
 *
 * Copyright(c) 2000-2015 LOCKON CO.,LTD. All Rights Reserved.
 *
 * http://www.lockon.co.jp/
 *
 * This program is free software; you can redistribute it and/or
 * modify it under the terms of the GNU General Public License
 * as published by the Free Software Foundation; either version 2
 * of the License, or (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with this program; if not, write to the Free Software
 * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.
 */

namespace Eccube\Command;

use Doctrine\Common\Annotations\AnnotationReader;
use Eccube\Annotation\EntityExt;
use Knp\Command\Command;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Filesystem\Filesystem;
use Symfony\CS\Finder;


class GenProxyCommand extends Command

Missing class doc comment

{
    protected function configure()
    {
        $this
            ->setName('debug:entity-proxy')
            ->setDescription('generate entity proxies');
    }

    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から@EntityExtを抽出
        $reader = new AnnotationReader();
        $proxies = [];
        foreach ($traits as $trait) {
            $anno = $reader->getClassAnnotation(new \ReflectionClass($trait), EntityExt::class);

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

            if ($anno) {
                $proxies[$anno->target][] = $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/cache/doctrine/entity-proxies';
            $file = basename($rc->getFileName());

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