Zicht\Bundle\FrameworkExtraBundle\Command\ListUserImagesCommand

Complexity

Total Complexity

14

Size/Duplication

Total Lines	93
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
C	execute()	0	57	12
A	configure()	0	8	1
A	humanFilesize()	0	6	1

<?php
/**
 * @author Rik van der Kemp <rik@zicht.nl>
 * @copyright Zicht Online <http://www.zicht.nl>
 */

namespace Zicht\Bundle\FrameworkExtraBundle\Command;

use Doctrine\ORM\EntityManager;
use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;

/**
 * Generates an overview of all images used in the defined Entity and the defined fields.
 *
 * Results are sorted by size descending.
 */
class ListUserImagesCommand extends ContainerAwareCommand
{
    /**
     * @{inheritDoc}
     */
    protected function configure()
    {
        $this
            ->setName('zicht:content:list-images')
            ->addArgument('entity', InputArgument::REQUIRED, 'The entity to query. Example: ZichtWebsiteBundle:Page:ContentPage ')
            ->addArgument('fields', InputArgument::REQUIRED, 'The fields to check for images, comma seperated. Example: body,teaser')
            ->addOption('concat', 'c', InputOption::VALUE_OPTIONAL, 'Optional concatenation string. Example: -c "CONCAT(\'http://www.krollermuller.nl/\', p.language, \'/\', p.id)" ')
            ->setDescription('List images used in specific fields of an Entity and show their file size.');
    }

    /**
     * @{inheritDoc}
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        /** @var EntityManager $em */
        $em = $this->getContainer()->get('doctrine')->getManager();

        $userFields = explode(',', $input->getArgument('fields'));

It seems like $input->getArgument('fields') can also be of type string[]; however, parameter $string of explode() does only seem to accept string, maybe add an additional type check?

        $fields = [];

        foreach ($userFields as $fieldName) {
            $fields[$fieldName] = 'p.' . trim($fieldName);
        }

        if ('' !== $input->getOption('concat')) {
            $fields['custom'] = $input->getOption('concat') . ' as custom';
        }

        $dql = sprintf('SELECT p.id, %s FROM %s p', implode(', ', $fields), $input->getArgument('entity'));

It seems like $input->getArgument('entity') can also be of type string[]; however, parameter $args of sprintf() does only seem to accept string, maybe add an additional type check?

        $qb = $em->createQuery($dql);

        if (array_key_exists('custom', $fields)) {
            unset($fields['custom']);
        }

        $list = [];

        foreach ($qb->getResult() as $record) {
            foreach (array_keys($fields) as $field) {
                if (preg_match_all('/\<img.*src=\"\/media(.*?)\".*\>/', $record[$field], $matches)) {
                    foreach ($matches[1] as $image) {
                        $imagePath = './web/media' . $image;
                        if (file_exists($imagePath)) {
                            $fileSize = filesize($imagePath);
                            $arr = [
                                'id' => $record['id'],
                                'image' => $image,
                                'size' => $this->humanFilesize($fileSize),
                            ];

                            if ('' !== $input->getOption('concat')) {
                                $arr['custom'] = $record['custom'];
                            }

                            $list[$fileSize][] = $arr;
                        } else {
                            $output->writeln('File not found ' . $image);
                        }
                    }
                }
            }
        }

        krsort($list);

        foreach ($list as $fileSize => $sizes) {
            foreach ($sizes as $size => $info) {
                $output->writeln(implode(', ', $info));
            }
        }
    }

    /**
     * Kindly ripped from PHP.net :P.
     *
     * @param string $bytes
     * @param int $decimals
     *
     * @return string
     */
    private function humanFilesize($bytes, $decimals = 2)
    {
        $sz = 'BKMGTP';
        $factor = (int)floor((strlen($bytes) - 1) / 3);

        return sprintf("%.{$decimals}f", $bytes / pow(1024, $factor)) . @$sz[$factor];
    }
}