Liip\ImagineBundle\DependencyInjection\Compiler\NonFunctionalFilterExceptionPass

Complexity

Total Complexity

8

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	process()	0	37	8

<?php

namespace Liip\ImagineBundle\DependencyInjection\Compiler;

use Liip\ImagineBundle\Imagine\Filter\Loader\NonFunctionalPasteFilterLoader;
use Liip\ImagineBundle\Imagine\Filter\Loader\NonFunctionalWatermarkFilterLoader;
use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
use Symfony\Component\DependencyInjection\ContainerBuilder;

/**
 * For transitioning from Symfony 4 to Symfony 5 with the removal
 * of the kernel.root_dir parameter.
 */
class NonFunctionalFilterExceptionPass implements CompilerPassInterface
{
    public function process(ContainerBuilder $container)
    {
        $canFiltersStillFunction = $container->hasParameter('kernel.root_dir');
        $throwWarning = function(string $filterName) use ($canFiltersStillFunction) {
            $message = sprintf(
                'The "%s" filter %s in Symfony 5.0. Please use "%s_image" and adapt the "image" option to be relative to the "%%kernel.project_dir%%" instead of "%%kernel.root_dir%%".',
                $filterName,
                $canFiltersStillFunction ? 'is deprecated and will not work' : 'no longer works',
                $filterName
            );

            if ($canFiltersStillFunction) {
                @trigger_error($message, E_USER_DEPRECATED);

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

            } else {
                throw new \InvalidArgumentException($message);
            }
        };

        $filterSets = $container->getParameter('liip_imagine.filter_sets');
        foreach ($filterSets as $filterSet) {
            foreach ($filterSet['filters'] as $filterName => $filter) {
                if ($filterName === 'paste') {
                    $throwWarning('paste');
                }

                if ($filterName === 'watermark') {
                    $throwWarning('watermark');
                }
            }
        }

        // remove the definitions entirely if kernel.root_dir does not exist
        if (!$canFiltersStillFunction) {
            $container->removeDefinition('liip_imagine.filter.loader.watermark');
            $container->removeDefinition('liip_imagine.filter.loader.paste');
        }
    }
}