ManagerFactory - Code Metrics - Inspection of "Merge pull request #274: User account removal" - cross-solution/YAWIK - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( 031ed9...9561de )

unknown

created 2016-09-02 15:01 UTC

ManagerFactory A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	11
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes	1
Bugs	0	Features	0

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

1 Method

Rating	Name	Duplication	Size	Complexity
A	createService()	0	7	1

<?php
/**
 * @filesource
 * @copyright (c) 2013 - 2016 Cross Solution (http://cross-solution.de)
 * @license MIT
 * @author Miroslav Fedeleš <[email protected]>
 * @since 0.27
 */
namespace Auth\Factory\Dependency;

use Zend\ServiceManager\FactoryInterface;
use Zend\ServiceManager\ServiceLocatorInterface;

class ManagerFactory implements FactoryInterface
{

    public function createService(ServiceLocatorInterface $serviceLocator)
    {
        $manager = new \Auth\Dependency\Manager($serviceLocator->get('Core/DocumentManager'));
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $manager->setEventManager($serviceLocator->get('Auth/Dependency/Manager/Events'));
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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


1			<?php
2			/**
3			* @filesource
4			* @copyright (c) 2013 - 2016 Cross Solution (http://cross-solution.de)
5			* @license MIT
6			* @author Miroslav Fedeleš <[email protected]>
7			* @since 0.27
8			*/
9			namespace Auth\Factory\Dependency;
10
11			use Zend\ServiceManager\FactoryInterface;
12			use Zend\ServiceManager\ServiceLocatorInterface;
13
14			class ManagerFactory implements FactoryInterface
15			{
16
17			public function createService(ServiceLocatorInterface $serviceLocator)
18			{
19			$manager = new \Auth\Dependency\Manager($serviceLocator->get('Core/DocumentManager'));
			0 ignored issues – show Documentation introduced 2016-09-01 13:21 UTC by Report Bug Copy Issue Report `$serviceLocator->get('Core/DocumentManager')` is of type `object\|array`, but the function expects a `object<Doctrine\ODM\MongoDB\DocumentManager>`. 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...
20			$manager->setEventManager($serviceLocator->get('Auth/Dependency/Manager/Events'));
			0 ignored issues – show Documentation introduced 2016-08-23 09:18 UTC by Report Bug Copy Issue Report `$serviceLocator->get('Au...ndency/Manager/Events')` is of type `object\|array`, but the function expects a `object<Zend\EventManager\EventManagerInterface>`. 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...
21
22			return $manager;
23			}
24			}
25

cross-solution / YAWIK

Push — develop ( 031ed9...9561de )

ManagerFactory A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

1 Method

Duplication Side-by-Side

Filter issues like