Fi\PannelloAmministrazioneBundle\Command\CheckgitversionCommand

Complexity

Total Complexity

17

Size/Duplication

Total Lines	101
Duplicated Lines	0 %

Test Coverage

Coverage

8.96%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	isWindows()	0	6	2
A	configure()	0	6	1
D	getGitVersion()	0	46	10
B	execute()	0	34	4

<?php

namespace Fi\PannelloAmministrazioneBundle\Command;

use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Finder\Finder;
use Symfony\Component\Process\Process;
use Symfony\Component\Console\Formatter\OutputFormatterStyle;

class CheckgitversionCommand extends ContainerAwareCommand
{
    protected function configure()
    {
        $this
            ->setName('pannelloamministrazione:checkgitversion')
            ->setDescription('Controllo versioni bundles')
            ->setHelp('Controlla le versioni git dei bundles');
    }

    protected function execute(InputInterface $input, OutputInterface $output)
    {
        if (self::isWindows()) {
            $output->writeln('<info>Non previsto in ambiente windows</info>');

            return 0;
        }

        $projectDir = substr($this->getContainer()->get('kernel')->getRootDir(), 0, -4);

        $composerbundles = array();
        $composerbundlespath = $projectDir.DIRECTORY_SEPARATOR.'vendor'.DIRECTORY_SEPARATOR.'fi';
        $findercomposerbundle = new Finder();
        $findercomposerbundle->in($composerbundlespath)->sortByName()->directories()->depth('== 0');

        foreach ($findercomposerbundle as $file) {
            $fullcomposerbundlepath = $composerbundlespath.DIRECTORY_SEPARATOR.$file->getBasename();
            $local = $this->getGitVersion($fullcomposerbundlepath, false);
            $remote = $this->getGitVersion($fullcomposerbundlepath, true);
            $style = new OutputFormatterStyle('blue', 'white', array('bold', 'blink'));
            $output->getFormatter()->setStyle('warning', $style);
            if ($local !== $remote) {
                $remote = '<warning> * '.$remote.' * </warning>';
            }
            $output->writeln('<info>'.$file->getBasename().'</info> '.$local.' -> '.$remote);

            $composerbundles[] = array(
                'name' => $file->getBasename(),
                'path' => $fullcomposerbundlepath,
                'version' => $this->getGitVersion($fullcomposerbundlepath),
            );
        }

        return 0;
    }

    private function getGitVersion($path, $remote = false)
    {
        if (self::isWindows()) {
            return '';
        }

        if ($remote) {
            //Remote
            $cmd = 'cd '.$path;
            $remotetag = $cmd.";git ls-remote -t | awk '{print $2}' | cut -d '/' -f 3 | cut -d '^' -f 1 | sort --version-sort | tail -1";
            $process = new Process($remotetag);
            $process->setTimeout(60 * 100);
            $process->run();
            if ($process->isSuccessful()) {
                $version = trim($process->getOutput());
                if (preg_match('/\d+(?:\.\d+)+/', $version, $matches)) {
                    return $matches[0]; //returning the first match
                }
            }

            return '?';
        } else {
            //Local
            $cmd = 'cd '.$path;
            $process = new Process($cmd.';git branch | '."grep ' * '");
            $process->setTimeout(60 * 100);
            $process->run();
            if ($process->isSuccessful()) {
                $out = explode(chr(10), $process->getOutput());
                foreach ($out as $line) {
                    if (strpos($line, '* ') !== false) {
                        $version = trim(strtolower(str_replace('* ', '', $line)));
                        if ($version == 'master') {
                            return $version;
                        } else {
                            if (preg_match('/\d+(?:\.\d+)+/', $version, $matches)) {
                                return $matches[0]; //returning the first match
                            }
                        }
                    }
                }
            } else {

This else statement is empty and can be removed.

                //echo $process->getErrorOutput();

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

            }

            return '?';
        }
    }

    public static function isWindows()
    {
        if (PHP_OS == 'WINNT') {
            return true;
        } else {
            return false;
        }
    }
}