Issues in GalleryBlockService.php - All Issues - Inspection of "Merge remote-tracking branch 'origin/3.x'" - sonata-project/SonataMediaBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( bb6905...021686 )

by Grégoire

created 2019-11-29 07:44 UTC

src/Block/GalleryBlockService.php (1 issue)

Showing only issues like (Show All)

Check for nulls being passed to functions/methods that do not accept null.

Bug Minor

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.21.0
 *
 * @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());
/** @return stdClass|null */
function mayReturnNull() { }

function doesNotAcceptNull(stdClass $x) { }

// With potential error.
function withoutCheck() {
    $x = mayReturnNull();
    doesNotAcceptNull($x); // Potential error here.
}

// Safe - Alternative 1
function withCheck1() {
    $x = mayReturnNull();
    if ( ! $x instanceof stdClass) {
        throw new \LogicException('$x must be defined.');
    }
    doesNotAcceptNull($x);
}

// Safe - Alternative 2
function withCheck2() {
    $x = mayReturnNull();
    if ($x instanceof stdClass) {
        doesNotAcceptNull($x);
    }
}

            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.21.0
40			*
41			* @author Thomas Rabaix <[email protected]>
42			*/
43			class GalleryBlockService extends AbstractAdminBlockService
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());
			0 ignored issues – show Bug introduced 2018-07-12 18:37 UTC by Report Bug Copy Issue Report Show Similar Issues like this It seems like `$galleryItem->getMedia()` can be `null`; however, `getMediaType()` does not accept `null`, maybe add an additional type check? Unless you are absolutely sure that the expression can never be null because of other conditions, we strongly recommend to add an additional type check to your code: /** @return stdClass\|null */ function mayReturnNull() { } function doesNotAcceptNull(stdClass $x) { } // With potential error. function withoutCheck() { $x = mayReturnNull(); doesNotAcceptNull($x); // Potential error here. } // Safe - Alternative 1 function withCheck1() { $x = mayReturnNull(); if ( ! $x instanceof stdClass) { throw new \LogicException('$x must be defined.'); } doesNotAcceptNull($x); } // Safe - Alternative 2 function withCheck2() { $x = mayReturnNull(); if ($x instanceof stdClass) { doesNotAcceptNull($x); } } Loading history...
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 ( bb6905...021686 )

src/Block/GalleryBlockService.php (1 issue)

Showing only issues like (Show All)

Introduced By

Upgrade to new PHP Analysis Engine

Duplication Side-by-Side

Filter issues like