StatementRepository - Code Metrics - php-xapi/repository-doctrine-orm - Measure and Improve Code Quality continuously with Scrutinizer

StatementRepository A
last analyzed 2019-06-21 10:39 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

Metric	Value
wmc	4
lcom	1
cbo	2
dl	0
loc	30
rs	10
c	0
b	0
f	0

3 Methods

Rating	Name	Size	Complexity
A	findStatement()	4	1
A	findStatements()	4	1
A	storeStatement()	8	2

<?php

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

namespace XApi\Repository\ORM;

use Doctrine\ORM\EntityRepository;
use XApi\Repository\Doctrine\Mapping\Statement;
use XApi\Repository\Doctrine\Repository\Mapping\StatementRepository as BaseStatementRepository;

/**
 * @author Christian Flothmann <[email protected]>
 */
final class StatementRepository extends EntityRepository implements BaseStatementRepository
{
    /**
     * {@inheritdoc}
     */
    public function findStatement(array $criteria)
    {
        return parent::findOneBy($criteria);
class Daddy
{
    protected function getFirstName()
    {
        return "Eidur";
    }

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

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

    /**
     * {@inheritdoc}
     */
    public function findStatements(array $criteria)
    {
        return parent::findBy($criteria);
class Daddy
{
    protected function getFirstName()
    {
        return "Eidur";
    }

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

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

    /**
     * {@inheritdoc}
     */
    public function storeStatement(Statement $mappedStatement, $flush = true)
    {
        $this->_em->persist($mappedStatement);

        if ($flush) {
            $this->_em->flush();
        }
    }
}


1			<?php
2
3			/*
4			* This file is part of the xAPI package.
5			*
6			* (c) Christian Flothmann <[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 XApi\Repository\ORM;
13
14			use Doctrine\ORM\EntityRepository;
15			use XApi\Repository\Doctrine\Mapping\Statement;
16			use XApi\Repository\Doctrine\Repository\Mapping\StatementRepository as BaseStatementRepository;
17
18			/**
19			* @author Christian Flothmann <[email protected]>
20			*/
21			final class StatementRepository extends EntityRepository implements BaseStatementRepository
22			{
23			/**
24			* {@inheritdoc}
25			*/
26			public function findStatement(array $criteria)
27			{
28			return parent::findOneBy($criteria);
			0 ignored issues – show Comprehensibility Bug introduced 2016-10-05 17:07 UTC by Report Bug Copy Issue Report It seems like you call parent on a different method (`findOneBy()` instead of `findStatement()`). Are you sure this is correct? If so, you might want to change this to `$this->findOneBy()`. 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...
29			}
30
31			/**
32			* {@inheritdoc}
33			*/
34			public function findStatements(array $criteria)
35			{
36			return parent::findBy($criteria);
			0 ignored issues – show Comprehensibility Bug introduced 2016-10-05 17:07 UTC by Report Bug Copy Issue Report It seems like you call parent on a different method (`findBy()` instead of `findStatements()`). Are you sure this is correct? If so, you might want to change this to `$this->findBy()`. 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...
37			}
38
39			/**
40			* {@inheritdoc}
41			*/
42			public function storeStatement(Statement $mappedStatement, $flush = true)
43			{
44			$this->_em->persist($mappedStatement);
45
46			if ($flush) {
47			$this->_em->flush();
48			}
49			}
50			}
51

php-xapi / repository-doctrine-orm

StatementRepository A last analyzed 2019-06-21 10:39 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

3 Methods

Duplication Side-by-Side

Filter issues like

StatementRepository A
last analyzed 2019-06-21 10:39 UTC