Issues in GalleryBlockService.php - New Issues - Inspection of "Use Twig namespaced classes" - sonata-project/SonataMediaBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 460827...4b7807 )

by Christian

created 2019-09-28 14:41 UTC

src/Block/GalleryBlockService.php (1 issue)

Labels

Deprecated Code 1

Severity

Minor 1

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

<?php

declare(strict_types=1);

/*
 * This file is part of the Sonata Project package.
 *
 * (c) Thomas Rabaix <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Sonata\MediaBundle\Block;

use Doctrine\ORM\Mapping\ClassMetadataInfo;
use Sonata\AdminBundle\Admin\AdminInterface;
use Sonata\AdminBundle\Form\FormMapper;
use Sonata\AdminBundle\Form\Type\ModelListType;
use Sonata\BlockBundle\Block\BlockContextInterface;
use Sonata\BlockBundle\Block\Service\AbstractAdminBlockService;
use Sonata\BlockBundle\Meta\Metadata;
use Sonata\BlockBundle\Model\BlockInterface;
use Sonata\Doctrine\Model\ManagerInterface;
use Sonata\Form\Type\ImmutableArrayType;
use Sonata\MediaBundle\Model\GalleryInterface;
use Sonata\MediaBundle\Model\MediaInterface;
use Sonata\MediaBundle\Provider\Pool;
use Symfony\Component\DependencyInjection\ContainerInterface;
use Symfony\Component\Form\Extension\Core\Type\CheckboxType;
use Symfony\Component\Form\Extension\Core\Type\ChoiceType;
use Symfony\Component\Form\Extension\Core\Type\NumberType;
use Symfony\Component\Form\Extension\Core\Type\TextType;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\OptionsResolver\OptionsResolver;
use Symfony\Component\Templating\EngineInterface;

/**
 * @final since sonata-project/media-bundle 3.x
 *
 * @author Thomas Rabaix <[email protected]>
 */
class GalleryBlockService extends AbstractAdminBlockService

{
    /**
     * @var ManagerInterface
     */
    protected $galleryAdmin;

    /**
     * @var ManagerInterface
     */
    protected $galleryManager;

    /**
     * @param string $name
     */
    public function __construct($name, EngineInterface $templating, ContainerInterface $container, ManagerInterface $galleryManager)
    {
        parent::__construct($name, $templating);

        $this->galleryManager = $galleryManager;
        $this->container = $container;
    }

    /**
     * @return Pool
     */
    public function getMediaPool()
    {
        return $this->container->get('sonata.media.pool');
    }

    /**
     * @return AdminInterface
     */
    public function getGalleryAdmin()
    {
        if (!$this->galleryAdmin) {
            $this->galleryAdmin = $this->container->get('sonata.media.admin.gallery');
        }

        return $this->galleryAdmin;
    }

    /**
     * {@inheritdoc}
     */
    public function configureSettings(OptionsResolver $resolver): void
    {
        $resolver->setDefaults([
            'gallery' => false,
            'title' => null,
            'translation_domain' => null,
            'icon' => null,
            'class' => null,
            'context' => false,
            'format' => false,
            'pauseTime' => 3000,
            'startPaused' => false,
            'template' => '@SonataMedia/Block/block_gallery.html.twig',
            'galleryId' => null,
        ]);
    }

    /**
     * {@inheritdoc}
     */
    public function buildEditForm(FormMapper $formMapper, BlockInterface $block): void
    {
        $contextChoices = [];

        foreach ($this->getMediaPool()->getContexts() as $name => $context) {
            $contextChoices[$name] = $name;
        }

        $gallery = $block->getSetting('galleryId');

        $formatChoices = [];

        if ($gallery instanceof GalleryInterface) {
            $formats = $this->getMediaPool()->getFormatNamesByContext($gallery->getContext());

            foreach ($formats as $code => $format) {
                $formatChoices[$code] = $code;
            }
        }

        // simulate an association ...
        $fieldDescription = $this->getGalleryAdmin()->getModelManager()->getNewFieldDescriptionInstance($this->getGalleryAdmin()->getClass(), 'media', [
            'translation_domain' => 'SonataMediaBundle',
        ]);
        $fieldDescription->setAssociationAdmin($this->getGalleryAdmin());
        $fieldDescription->setAdmin($formMapper->getAdmin());
        $fieldDescription->setOption('edit', 'list');
        $fieldDescription->setAssociationMapping(['fieldName' => 'gallery', 'type' => ClassMetadataInfo::MANY_TO_ONE]);

        $builder = $formMapper->create('galleryId', ModelListType::class, [
            'sonata_field_description' => $fieldDescription,
            'class' => $this->getGalleryAdmin()->getClass(),
            'model_manager' => $this->getGalleryAdmin()->getModelManager(),
            'label' => 'form.label_gallery',
        ]);

        $formMapper->add('settings', ImmutableArrayType::class, [
            'keys' => [
                ['title', TextType::class, [
                    'label' => 'form.label_title',
                    'required' => false,
                ]],
                ['translation_domain', TextType::class, [
                    'label' => 'form.label_translation_domain',
                    'required' => false,
                ]],
                ['icon', TextType::class, [
                    'label' => 'form.label_icon',
                    'required' => false,
                ]],
                ['class', TextType::class, [
                    'label' => 'form.label_class',
                    'required' => false,
                ]],
                ['context', ChoiceType::class, [
                    'required' => true,
                    'choices' => $contextChoices,
                    'label' => 'form.label_context',
                ]],
                ['format', ChoiceType::class, [
                    'required' => \count($formatChoices) > 0,
                    'choices' => $formatChoices,
                    'label' => 'form.label_format',
                ]],
                [$builder, null, []],
                ['pauseTime', NumberType::class, [
                    'label' => 'form.label_pause_time',
                ]],
                ['startPaused', CheckboxType::class, [
                    'required' => false,
                    'label' => 'form.label_start_paused',
                ]],
            ],
            'translation_domain' => 'SonataMediaBundle',
        ]);
    }

    /**
     * {@inheritdoc}
     */
    public function execute(BlockContextInterface $blockContext, Response $response = null)
    {
        $gallery = $blockContext->getBlock()->getSetting('galleryId');

        return $this->renderResponse($blockContext->getTemplate(), [
            'gallery' => $gallery,
            'block' => $blockContext->getBlock(),
            'settings' => $blockContext->getSettings(),
            'elements' => $gallery ? $this->buildElements($gallery) : [],
        ], $response);
    }

    /**
     * {@inheritdoc}
     */
    public function load(BlockInterface $block): void
    {
        $gallery = $block->getSetting('galleryId');

        if ($gallery) {
            $gallery = $this->galleryManager->findOneBy(['id' => $gallery]);
        }

        $block->setSetting('galleryId', $gallery);
    }

    /**
     * {@inheritdoc}
     */
    public function prePersist(BlockInterface $block): void
    {
        $block->setSetting('galleryId', \is_object($block->getSetting('galleryId')) ? $block->getSetting('galleryId')->getId() : null);
    }

    /**
     * {@inheritdoc}
     */
    public function preUpdate(BlockInterface $block): void
    {
        $block->setSetting('galleryId', \is_object($block->getSetting('galleryId')) ? $block->getSetting('galleryId')->getId() : null);
    }

    /**
     * {@inheritdoc}
     */
    public function getBlockMetadata($code = null)
    {
        return new Metadata($this->getName(), (null !== $code ? $code : $this->getName()), false, 'SonataMediaBundle', [
            'class' => 'fa fa-picture-o',
        ]);
    }

    private function buildElements(GalleryInterface $gallery): array
    {
        $elements = [];
        foreach ($gallery->getGalleryItems() as $galleryItem) {
            if (!$galleryItem->getEnabled()) {
                continue;
            }

            $type = $this->getMediaType($galleryItem->getMedia());

            if (null === $type) {
                continue;
            }

            $elements[] = [
                'title' => $galleryItem->getMedia()->getName(),
                'caption' => $galleryItem->getMedia()->getDescription(),
                'type' => $type,
                'media' => $galleryItem->getMedia(),
            ];
        }

        return $elements;
    }

    private function getMediaType(MediaInterface $media): ?string
    {
        if ('video/x-flv' === $media->getContentType()) {
            return 'video';
        }
        if ('image' === substr($media->getContentType(), 0, 5)) {
            return 'image';
        }

        return null;
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			/*
6			* This file is part of the Sonata Project package.
7			*
8			* (c) Thomas Rabaix <[email protected]>
9			*
10			* For the full copyright and license information, please view the LICENSE
11			* file that was distributed with this source code.
12			*/
13
14			namespace Sonata\MediaBundle\Block;
15
16			use Doctrine\ORM\Mapping\ClassMetadataInfo;
17			use Sonata\AdminBundle\Admin\AdminInterface;
18			use Sonata\AdminBundle\Form\FormMapper;
19			use Sonata\AdminBundle\Form\Type\ModelListType;
20			use Sonata\BlockBundle\Block\BlockContextInterface;
21			use Sonata\BlockBundle\Block\Service\AbstractAdminBlockService;
22			use Sonata\BlockBundle\Meta\Metadata;
23			use Sonata\BlockBundle\Model\BlockInterface;
24			use Sonata\Doctrine\Model\ManagerInterface;
25			use Sonata\Form\Type\ImmutableArrayType;
26			use Sonata\MediaBundle\Model\GalleryInterface;
27			use Sonata\MediaBundle\Model\MediaInterface;
28			use Sonata\MediaBundle\Provider\Pool;
29			use Symfony\Component\DependencyInjection\ContainerInterface;
30			use Symfony\Component\Form\Extension\Core\Type\CheckboxType;
31			use Symfony\Component\Form\Extension\Core\Type\ChoiceType;
32			use Symfony\Component\Form\Extension\Core\Type\NumberType;
33			use Symfony\Component\Form\Extension\Core\Type\TextType;
34			use Symfony\Component\HttpFoundation\Response;
35			use Symfony\Component\OptionsResolver\OptionsResolver;
36			use Symfony\Component\Templating\EngineInterface;
37
38			/**
39			* @final since sonata-project/media-bundle 3.x
40			*
41			* @author Thomas Rabaix <[email protected]>
42			*/
43			class GalleryBlockService extends AbstractAdminBlockService
			0 ignored issues – show Deprecated Code introduced 2019-09-28 14:53 UTC by Report Bug Copy Issue Report Show Similar Issues like this The class `Sonata\BlockBundle\Block...stractAdminBlockService` has been deprecated with message: since sonata-project/block-bundle 3.16 without any replacement This class, trait or interface has been deprecated. The supplier of the file has supplied an explanatory message. The explanatory message should give you some clue as to whether and when the type will be removed from the class and what other constant to use instead. Loading history...
44			{
45			/**
46			* @var ManagerInterface
47			*/
48			protected $galleryAdmin;
49
50			/**
51			* @var ManagerInterface
52			*/
53			protected $galleryManager;
54
55			/**
56			* @param string $name
57			*/
58			public function __construct($name, EngineInterface $templating, ContainerInterface $container, ManagerInterface $galleryManager)
59			{
60			parent::__construct($name, $templating);
61
62			$this->galleryManager = $galleryManager;
63			$this->container = $container;
64			}
65
66			/**
67			* @return Pool
68			*/
69			public function getMediaPool()
70			{
71			return $this->container->get('sonata.media.pool');
72			}
73
74			/**
75			* @return AdminInterface
76			*/
77			public function getGalleryAdmin()
78			{
79			if (!$this->galleryAdmin) {
80			$this->galleryAdmin = $this->container->get('sonata.media.admin.gallery');
81			}
82
83			return $this->galleryAdmin;
84			}
85
86			/**
87			* {@inheritdoc}
88			*/
89			public function configureSettings(OptionsResolver $resolver): void
90			{
91			$resolver->setDefaults([
92			'gallery' => false,
93			'title' => null,
94			'translation_domain' => null,
95			'icon' => null,
96			'class' => null,
97			'context' => false,
98			'format' => false,
99			'pauseTime' => 3000,
100			'startPaused' => false,
101			'template' => '@SonataMedia/Block/block_gallery.html.twig',
102			'galleryId' => null,
103			]);
104			}
105
106			/**
107			* {@inheritdoc}
108			*/
109			public function buildEditForm(FormMapper $formMapper, BlockInterface $block): void
110			{
111			$contextChoices = [];
112
113			foreach ($this->getMediaPool()->getContexts() as $name => $context) {
114			$contextChoices[$name] = $name;
115			}
116
117			$gallery = $block->getSetting('galleryId');
118
119			$formatChoices = [];
120
121			if ($gallery instanceof GalleryInterface) {
122			$formats = $this->getMediaPool()->getFormatNamesByContext($gallery->getContext());
123
124			foreach ($formats as $code => $format) {
125			$formatChoices[$code] = $code;
126			}
127			}
128
129			// simulate an association ...
130			$fieldDescription = $this->getGalleryAdmin()->getModelManager()->getNewFieldDescriptionInstance($this->getGalleryAdmin()->getClass(), 'media', [
131			'translation_domain' => 'SonataMediaBundle',
132			]);
133			$fieldDescription->setAssociationAdmin($this->getGalleryAdmin());
134			$fieldDescription->setAdmin($formMapper->getAdmin());
135			$fieldDescription->setOption('edit', 'list');
136			$fieldDescription->setAssociationMapping(['fieldName' => 'gallery', 'type' => ClassMetadataInfo::MANY_TO_ONE]);
137
138			$builder = $formMapper->create('galleryId', ModelListType::class, [
139			'sonata_field_description' => $fieldDescription,
140			'class' => $this->getGalleryAdmin()->getClass(),
141			'model_manager' => $this->getGalleryAdmin()->getModelManager(),
142			'label' => 'form.label_gallery',
143			]);
144
145			$formMapper->add('settings', ImmutableArrayType::class, [
146			'keys' => [
147			['title', TextType::class, [
148			'label' => 'form.label_title',
149			'required' => false,
150			]],
151			['translation_domain', TextType::class, [
152			'label' => 'form.label_translation_domain',
153			'required' => false,
154			]],
155			['icon', TextType::class, [
156			'label' => 'form.label_icon',
157			'required' => false,
158			]],
159			['class', TextType::class, [
160			'label' => 'form.label_class',
161			'required' => false,
162			]],
163			['context', ChoiceType::class, [
164			'required' => true,
165			'choices' => $contextChoices,
166			'label' => 'form.label_context',
167			]],
168			['format', ChoiceType::class, [
169			'required' => \count($formatChoices) > 0,
170			'choices' => $formatChoices,
171			'label' => 'form.label_format',
172			]],
173			[$builder, null, []],
174			['pauseTime', NumberType::class, [
175			'label' => 'form.label_pause_time',
176			]],
177			['startPaused', CheckboxType::class, [
178			'required' => false,
179			'label' => 'form.label_start_paused',
180			]],
181			],
182			'translation_domain' => 'SonataMediaBundle',
183			]);
184			}
185
186			/**
187			* {@inheritdoc}
188			*/
189			public function execute(BlockContextInterface $blockContext, Response $response = null)
190			{
191			$gallery = $blockContext->getBlock()->getSetting('galleryId');
192
193			return $this->renderResponse($blockContext->getTemplate(), [
194			'gallery' => $gallery,
195			'block' => $blockContext->getBlock(),
196			'settings' => $blockContext->getSettings(),
197			'elements' => $gallery ? $this->buildElements($gallery) : [],
198			], $response);
199			}
200
201			/**
202			* {@inheritdoc}
203			*/
204			public function load(BlockInterface $block): void
205			{
206			$gallery = $block->getSetting('galleryId');
207
208			if ($gallery) {
209			$gallery = $this->galleryManager->findOneBy(['id' => $gallery]);
210			}
211
212			$block->setSetting('galleryId', $gallery);
213			}
214
215			/**
216			* {@inheritdoc}
217			*/
218			public function prePersist(BlockInterface $block): void
219			{
220			$block->setSetting('galleryId', \is_object($block->getSetting('galleryId')) ? $block->getSetting('galleryId')->getId() : null);
221			}
222
223			/**
224			* {@inheritdoc}
225			*/
226			public function preUpdate(BlockInterface $block): void
227			{
228			$block->setSetting('galleryId', \is_object($block->getSetting('galleryId')) ? $block->getSetting('galleryId')->getId() : null);
229			}
230
231			/**
232			* {@inheritdoc}
233			*/
234			public function getBlockMetadata($code = null)
235			{
236			return new Metadata($this->getName(), (null !== $code ? $code : $this->getName()), false, 'SonataMediaBundle', [
237			'class' => 'fa fa-picture-o',
238			]);
239			}
240
241			private function buildElements(GalleryInterface $gallery): array
242			{
243			$elements = [];
244			foreach ($gallery->getGalleryItems() as $galleryItem) {
245			if (!$galleryItem->getEnabled()) {
246			continue;
247			}
248
249			$type = $this->getMediaType($galleryItem->getMedia());
250
251			if (null === $type) {
252			continue;
253			}
254
255			$elements[] = [
256			'title' => $galleryItem->getMedia()->getName(),
257			'caption' => $galleryItem->getMedia()->getDescription(),
258			'type' => $type,
259			'media' => $galleryItem->getMedia(),
260			];
261			}
262
263			return $elements;
264			}
265
266			private function getMediaType(MediaInterface $media): ?string
267			{
268			if ('video/x-flv' === $media->getContentType()) {
269			return 'video';
270			}
271			if ('image' === substr($media->getContentType(), 0, 5)) {
272			return 'image';
273			}
274
275			return null;
276			}
277			}
278

sonata-project / SonataMediaBundle

Push — master ( 460827...4b7807 )

src/Block/GalleryBlockService.php (1 issue)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

Duplication Side-by-Side

Filter issues like