TranslatableAdminExtension - Code Metrics - Inspection of "Merge pull request #70 from jorrit/stof-1.1.0-comp..." - sonata-project/SonataTranslationBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( fd0af8...9bd62a )

by Nicolas

created 2016-01-11 11:50 UTC

TranslatableAdminExtension A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	4
Bugs	1	Features	3

Metric	Value
wmc	4
c	4
b	1
f	3
lcom	1
cbo	3
dl	0
loc	37
rs	10

3 Methods

Rating	Name	Size	Complexity
A	__construct()	5	1
A	getTranslatableListener()	4	1
A	alterObject()	10	2

<?php

/*
 * This file is part of the Sonata 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\TranslationBundle\Admin\Extension\Gedmo;

use Gedmo\Translatable\TranslatableListener;
use Sonata\AdminBundle\Admin\AdminInterface;
use Sonata\TranslationBundle\Admin\Extension\AbstractTranslatableAdminExtension;
use Sonata\TranslationBundle\Checker\TranslatableChecker;

/**
 * @author Nicolas Bastien <[email protected]>
 */
class TranslatableAdminExtension extends AbstractTranslatableAdminExtension
{
    /**
     * @var TranslatableListener
     */
    protected $translatableListener;

    public function __construct(TranslatableChecker $translatableChecker, TranslatableListener $translatableListener)
    {
        parent::__construct($translatableChecker);
        $this->translatableListener = $translatableListener;
    }

    /**
     * @param AdminInterface $admin
/**
 * @param array $germany
 * @param array $island
 * @param array $italy
 */
function finale($germany, $island) {
    return "2:1";
}
     *
     * @return TranslatableListener
     */
    protected function getTranslatableListener()
    {
        return $this->translatableListener;
    }

    /**
     * {@inheritdoc}
     */
    public function alterObject(AdminInterface $admin, $object)
    {
        if ($this->getTranslatableChecker()->isTranslatable($object)) {
            $this->getTranslatableListener()->setTranslatableLocale($this->getTranslatableLocale($admin));
            $this->getTranslatableListener()->setTranslationFallback('');
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

            $this->getContainer($admin)->get('doctrine')->getManager()->refresh($object);
            $object->setLocale($this->getTranslatableLocale($admin));
        }
    }
}


1			<?php
2
3			/*
4			* This file is part of the Sonata package.
5			*
6			* (c) Thomas Rabaix <[email protected]>
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*/
11
12			namespace Sonata\TranslationBundle\Admin\Extension\Gedmo;
13
14			use Gedmo\Translatable\TranslatableListener;
15			use Sonata\AdminBundle\Admin\AdminInterface;
16			use Sonata\TranslationBundle\Admin\Extension\AbstractTranslatableAdminExtension;
17			use Sonata\TranslationBundle\Checker\TranslatableChecker;
18
19			/**
20			* @author Nicolas Bastien <[email protected]>
21			*/
22			class TranslatableAdminExtension extends AbstractTranslatableAdminExtension
23			{
24			/**
25			* @var TranslatableListener
26			*/
27			protected $translatableListener;
28
29			public function __construct(TranslatableChecker $translatableChecker, TranslatableListener $translatableListener)
30			{
31			parent::__construct($translatableChecker);
32			$this->translatableListener = $translatableListener;
33			}
34
35			/**
36			* @param AdminInterface $admin
			0 ignored issues – show Bug introduced 2016-01-11 07:02 UTC by Report Bug Copy Issue Report There is no parameter named `$admin`. Was it maybe removed? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. Consider the following example. The parameter `$italy` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $island * @param array $italy */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was removed, but the annotation was not. Loading history...
37			*
38			* @return TranslatableListener
39			*/
40			protected function getTranslatableListener()
41			{
42			return $this->translatableListener;
43			}
44
45			/**
46			* {@inheritdoc}
47			*/
48			public function alterObject(AdminInterface $admin, $object)
49			{
50			if ($this->getTranslatableChecker()->isTranslatable($object)) {
51			$this->getTranslatableListener()->setTranslatableLocale($this->getTranslatableLocale($admin));
52			$this->getTranslatableListener()->setTranslationFallback('');
			0 ignored issues – show Documentation introduced 2016-01-04 16:10 UTC by Report Bug Copy Issue Report `''` is of type `string`, but the function expects a `boolean`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
53
54			$this->getContainer($admin)->get('doctrine')->getManager()->refresh($object);
55			$object->setLocale($this->getTranslatableLocale($admin));
56			}
57			}
58			}
59

sonata-project / SonataTranslationBundle

Push — master ( fd0af8...9bd62a )

TranslatableAdminExtension A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

3 Methods

Duplication Side-by-Side

Filter issues like