Version20140408113030_AddItemGenres::setContainer() - Code Metrics - anime-db/catalog-bundle - Measure and Improve Code Quality continuously with Scrutinizer

setContainer() A
last analyzed 2017-06-23 22:09 UTC

↳ Parent: Version20140408113030_AddItemGenres

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	4
rs	10
cc	1
eloc	2
nc	1
nop	1

<?php
/**
 * AnimeDb package.
 *
 * @author    Peter Gribanov <[email protected]>
 * @copyright Copyright (c) 2011, Peter Gribanov
 * @license   http://opensource.org/licenses/GPL-3.0 GPL v3
 */

namespace AnimeDb\Bundle\CatalogBundle\DoctrineMigrations;

use Doctrine\DBAL\Migrations\AbstractMigration;
use Doctrine\ORM\EntityManagerInterface;
use Symfony\Component\DependencyInjection\ContainerAwareInterface;
use Symfony\Component\DependencyInjection\ContainerInterface;
use Doctrine\DBAL\Schema\Schema;
use AnimeDb\Bundle\CatalogBundle\Entity\Genre;

/**
 * Auto-generated Migration: Please modify to your needs!
 */
class Version20140408113030_AddItemGenres extends AbstractMigration implements ContainerAwareInterface
{
    /**
     * Entity manager.
     *
     * @var EntityManagerInterface
     */
    protected $em;

    /**
     * Add genres.
     *
     * @var array
     */
    protected $genres = [
        'Cars' => 'Машины',
        'Demons' => 'Демоны',
        'Game' => 'Игры',
        'Magic' => 'Магия',
        'Space' => 'Космос',
        'Super Power' => 'Супер сила',
        'Harem' => 'Гарем',
        'Supernatural' => 'Сверхъестественное',
        'Gender Bender' => 'Смена пола',
    ];

    /**
     * Set container.
     *
     * @param ContainerInterface $container
     */
    public function setContainer(ContainerInterface $container = null)
    {
        $this->em = $container->get('doctrine.orm.entity_manager');
function someFunction(A $objectMaybe = null)
{
    if ($objectMaybe instanceof A) {
        $objectMaybe->doSomething();
    }
}
    }

    /**
     * @param Schema $schema
     */
    public function up(Schema $schema)
    {
        $rep = $this->em->getRepository('Gedmo\\Translatable\\Entity\\Translation');
        foreach ($this->genres as $en => $ru) {
            $genre = new Genre();

            $genre->setName($en)->setTranslatableLocale('en');
            $this->em->persist($genre);
            $this->em->flush($genre);


            $rep->translate($genre, 'name', 'ru', $ru);
        }
        $this->em->flush();
    }

    /**
     * @param Schema $schema
     */
    public function down(Schema $schema)
    {
        $rep = $this->em->getRepository('AnimeDbCatalogBundle:Genre');
        foreach (array_keys($this->genres) as $en) {
            $this->em->remove($rep->findOneBy(['name' => $en]));
        }
    }
}


1			<?php
2			/**
3			* AnimeDb package.
4			*
5			* @author Peter Gribanov <[email protected]>
6			* @copyright Copyright (c) 2011, Peter Gribanov
7			* @license http://opensource.org/licenses/GPL-3.0 GPL v3
8			*/
9
10			namespace AnimeDb\Bundle\CatalogBundle\DoctrineMigrations;
11
12			use Doctrine\DBAL\Migrations\AbstractMigration;
13			use Doctrine\ORM\EntityManagerInterface;
14			use Symfony\Component\DependencyInjection\ContainerAwareInterface;
15			use Symfony\Component\DependencyInjection\ContainerInterface;
16			use Doctrine\DBAL\Schema\Schema;
17			use AnimeDb\Bundle\CatalogBundle\Entity\Genre;
18
19			/**
20			* Auto-generated Migration: Please modify to your needs!
21			*/
22			class Version20140408113030_AddItemGenres extends AbstractMigration implements ContainerAwareInterface
23			{
24			/**
25			* Entity manager.
26			*
27			* @var EntityManagerInterface
28			*/
29			protected $em;
30
31			/**
32			* Add genres.
33			*
34			* @var array
35			*/
36			protected $genres = [
37			'Cars' => 'Машины',
38			'Demons' => 'Демоны',
39			'Game' => 'Игры',
40			'Magic' => 'Магия',
41			'Space' => 'Космос',
42			'Super Power' => 'Супер сила',
43			'Harem' => 'Гарем',
44			'Supernatural' => 'Сверхъестественное',
45			'Gender Bender' => 'Смена пола',
46			];
47
48			/**
49			* Set container.
50			*
51			* @param ContainerInterface $container
52			*/
53			public function setContainer(ContainerInterface $container = null)
54			{
55			$this->em = $container->get('doctrine.orm.entity_manager');
			0 ignored issues – show Bug introduced 2016-01-02 16:13 UTC by Report Bug Copy Issue Report It seems like `$container` is not always an object, but can also be of type `null`. Maybe add an additional type check? If a variable is not always an object, we recommend to add an additional type check to ensure your method call is safe: function someFunction(A $objectMaybe = null) { if ($objectMaybe instanceof A) { $objectMaybe->doSomething(); } } Loading history...
56			}
57
58			/**
59			* @param Schema $schema
60			*/
61			public function up(Schema $schema)
62			{
63			$rep = $this->em->getRepository('Gedmo\\Translatable\\Entity\\Translation');
64			foreach ($this->genres as $en => $ru) {
65			$genre = new Genre();
66
67			$genre->setName($en)->setTranslatableLocale('en');
68			$this->em->persist($genre);
69			$this->em->flush($genre);
			0 ignored issues – show Unused Code introduced 2016-03-22 13:23 UTC by Report Bug Copy Issue Report The call to `EntityManagerInterface::flush()` has too many arguments starting with `$genre`. This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue. If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. In this case you can add the `@ignore` PhpDoc annotation to the duplicate definition and it will be ignored. Loading history...
70
71			$rep->translate($genre, 'name', 'ru', $ru);
72			}
73			$this->em->flush();
74			}
75
76			/**
77			* @param Schema $schema
78			*/
79			public function down(Schema $schema)
80			{
81			$rep = $this->em->getRepository('AnimeDbCatalogBundle:Genre');
82			foreach (array_keys($this->genres) as $en) {
83			$this->em->remove($rep->findOneBy(['name' => $en]));
84			}
85			}
86			}
87

anime-db / catalog-bundle

setContainer() A last analyzed 2017-06-23 22:09 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

setContainer() A
last analyzed 2017-06-23 22:09 UTC