MediaAdminController::listAction() - Code Metrics - sonata-project/SonataMediaBundle - Measure and Improve Code Quality continuously with Scrutinizer

MediaAdminController::listAction() B
last analyzed 2020-08-28 02:17 UTC

↳ Parent: MediaAdminController

Complexity

Conditions	10
Paths	48

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	54
rs	7.1369
c	0
b	0
f	0
cc	10
nc	48
nop	1

How to fix Long Method Complexity

<?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\Controller;

use Sonata\AdminBundle\Controller\CRUDController as Controller;
use Symfony\Component\Form\FormRenderer;
use Symfony\Component\Form\FormView;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;

/**
 * @final since sonata-project/media-bundle 3.21.0
 */
class MediaAdminController extends Controller
{
    public function createAction(?Request $request = null)
    {
        $this->admin->checkAccess('create');

        if (!$request->get('provider') && $request->isMethod('get')) {
            $pool = $this->get('sonata.media.pool');

            return $this->render('@SonataMedia/MediaAdmin/select_provider.html.twig', [
                'providers' => $pool->getProvidersByContext(
                    $request->get('context', $pool->getDefaultContext())
                ),
                'action' => 'create',
            ]);
        }

        return parent::createAction();
    }

    public function render($view, array $parameters = [], ?Response $response = null)
    {
        $parameters['media_pool'] = $this->get('sonata.media.pool');
        $parameters['persistent_parameters'] = $this->admin->getPersistentParameters();

        return parent::renderWithExtraParams($view, $parameters, $response);
class Daddy
{
    protected function getFirstName()
    {
        return "Eidur";
    }

    protected function getSurName()
    {
        return "Gudjohnsen";
    }
}

class Son
{
    public function getFirstName()
    {
        return parent::getSurname();
    }
}
    }

    public function listAction(?Request $request = null)
    {
        $this->admin->checkAccess('list');

        if ($listMode = $request->get('_list_mode', 'mosaic')) {
            $this->admin->setListMode($listMode);
        }

        $datagrid = $this->admin->getDatagrid();

        $filters = $request->get('filter');

        // set the default context
        if (!$filters || !\array_key_exists('context', $filters)) {
            $context = $this->admin->getPersistentParameter('context', $this->get('sonata.media.pool')->getDefaultContext());

        } else {
            $context = $filters['context']['value'];
        }

        $datagrid->setValue('context', null, $context);

        $rootCategory = null;
        if ($this->has('sonata.media.manager.category')) {
            $rootCategory = $this->get('sonata.media.manager.category')->getRootCategory($context);
        }

        if (null !== $rootCategory && !$filters) {
            $datagrid->setValue('category', null, $rootCategory->getId());
        }
        if ($this->has('sonata.media.manager.category') && $request->get('category')) {
            $category = $this->get('sonata.media.manager.category')->findOneBy([
                'id' => (int) $request->get('category'),
                'context' => $context,
            ]);

            if (!empty($category)) {
                $datagrid->setValue('category', null, $category->getId());
            } else {
                $datagrid->setValue('category', null, $rootCategory->getId());
            }
        }

        $formView = $datagrid->getForm()->createView();

        $this->setFormTheme($formView, $this->admin->getFilterTheme());

        return $this->render($this->admin->getTemplate('list'), [

            'action' => 'list',
            'form' => $formView,
            'datagrid' => $datagrid,
            'root_category' => $rootCategory,
            'csrf_token' => $this->getCsrfToken('sonata.batch'),
        ]);
    }

    /**
     * Sets the admin form theme to form view. Used for compatibility between Symfony versions.
     */
    private function setFormTheme(FormView $formView, array $theme)

    {
        $twig = $this->get('twig');

        $twig->getRuntime(FormRenderer::class)->setTheme($formView, $theme);
    }
}


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\Controller;
15
16			use Sonata\AdminBundle\Controller\CRUDController as Controller;
17			use Symfony\Component\Form\FormRenderer;
18			use Symfony\Component\Form\FormView;
19			use Symfony\Component\HttpFoundation\Request;
20			use Symfony\Component\HttpFoundation\Response;
21
22			/**
23			* @final since sonata-project/media-bundle 3.21.0
24			*/
25			class MediaAdminController extends Controller
26			{
27			public function createAction(?Request $request = null)
28			{
29			$this->admin->checkAccess('create');
30
31			if (!$request->get('provider') && $request->isMethod('get')) {
32			$pool = $this->get('sonata.media.pool');
33
34			return $this->render('@SonataMedia/MediaAdmin/select_provider.html.twig', [
35			'providers' => $pool->getProvidersByContext(
36			$request->get('context', $pool->getDefaultContext())
37			),
38			'action' => 'create',
39			]);
40			}
41
42			return parent::createAction();
43			}
44
45			public function render($view, array $parameters = [], ?Response $response = null)
46			{
47			$parameters['media_pool'] = $this->get('sonata.media.pool');
48			$parameters['persistent_parameters'] = $this->admin->getPersistentParameters();
49
50			return parent::renderWithExtraParams($view, $parameters, $response);
			0 ignored issues – show Comprehensibility Bug introduced 2018-05-26 09:38 UTC by Report Bug Copy Issue Report It seems like you call parent on a different method (`renderWithExtraParams()` instead of `render()`). Are you sure this is correct? If so, you might want to change this to `$this->renderWithExtraParams()`. This check looks for a call to a parent method whose name is different than the method from which it is called. Consider the following code: class Daddy { protected function getFirstName() { return "Eidur"; } protected function getSurName() { return "Gudjohnsen"; } } class Son { public function getFirstName() { return parent::getSurname(); } } The `getFirstName()` method in the `Son` calls the wrong method in the parent class. Loading history...
51			}
52
53			public function listAction(?Request $request = null)
54			{
55			$this->admin->checkAccess('list');
56
57			if ($listMode = $request->get('_list_mode', 'mosaic')) {
58			$this->admin->setListMode($listMode);
59			}
60
61			$datagrid = $this->admin->getDatagrid();
62
63			$filters = $request->get('filter');
64
65			// set the default context
66			if (!$filters \|\| !\array_key_exists('context', $filters)) {
67			$context = $this->admin->getPersistentParameter('context', $this->get('sonata.media.pool')->getDefaultContext());
			0 ignored issues – show Bug introduced 2015-11-21 05:46 UTC by Report Bug Copy Issue Report The method `getPersistentParameter()` does not exist on `Sonata\AdminBundle\Admin\AdminInterface`. Did you maybe mean `getPersistentParameters()`? This check marks calls to methods that do not seem to exist on an object. This is most likely the result of a method being renamed without all references to it being renamed likewise. Loading history...
68			} else {
69			$context = $filters['context']['value'];
70			}
71
72			$datagrid->setValue('context', null, $context);
73
74			$rootCategory = null;
75			if ($this->has('sonata.media.manager.category')) {
76			$rootCategory = $this->get('sonata.media.manager.category')->getRootCategory($context);
77			}
78
79			if (null !== $rootCategory && !$filters) {
80			$datagrid->setValue('category', null, $rootCategory->getId());
81			}
82			if ($this->has('sonata.media.manager.category') && $request->get('category')) {
83			$category = $this->get('sonata.media.manager.category')->findOneBy([
84			'id' => (int) $request->get('category'),
85			'context' => $context,
86			]);
87
88			if (!empty($category)) {
89			$datagrid->setValue('category', null, $category->getId());
90			} else {
91			$datagrid->setValue('category', null, $rootCategory->getId());
92			}
93			}
94
95			$formView = $datagrid->getForm()->createView();
96
97			$this->setFormTheme($formView, $this->admin->getFilterTheme());
98
99			return $this->render($this->admin->getTemplate('list'), [
			0 ignored issues – show Deprecated Code introduced 2020-02-11 02:23 UTC by Report Bug Copy Issue Report The method `Sonata\AdminBundle\Admin...nterface::getTemplate()` has been deprecated with message: since sonata-project/admin-bundle 3.35. To be removed in 4.0. Use TemplateRegistry services instead This method has been deprecated. The supplier of the class has supplied an explanatory message. The explanatory message should give you some clue as to whether and when the method will be removed from the class and what other method or class to use instead. Loading history...
100			'action' => 'list',
101			'form' => $formView,
102			'datagrid' => $datagrid,
103			'root_category' => $rootCategory,
104			'csrf_token' => $this->getCsrfToken('sonata.batch'),
105			]);
106			}
107
108			/**
109			* Sets the admin form theme to form view. Used for compatibility between Symfony versions.
110			*/
111			private function setFormTheme(FormView $formView, array $theme)
			0 ignored issues – show Bug introduced 2019-02-23 15:54 UTC by Report Bug Copy Issue Report Consider using a different method name as you override a private method of the parent class. Overwriting private methods is generally fine as long as you also use private visibility. It might still be preferable for understandability to use a different method name. Loading history...
112			{
113			$twig = $this->get('twig');
114
115			$twig->getRuntime(FormRenderer::class)->setTheme($formView, $theme);
116			}
117			}
118

sonata-project / SonataMediaBundle

MediaAdminController::listAction() B last analyzed 2020-08-28 02:17 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like

MediaAdminController::listAction() B
last analyzed 2020-08-28 02:17 UTC