ArrayWorkflowFactory - Code Metrics - old-town/old-town-workflow - Measure and Improve Code Quality continuously with Scrutinizer

ArrayWorkflowFactory A
last analyzed 2016-03-11 07:02 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
wmc	4
c	1
b	0
f	0
lcom	1
cbo	2
dl	0
loc	29
ccs	0
cts	12
cp	0
rs	10

1 Method

Rating	Name	Duplication	Size	Complexity
A	initDone()	0	13	4

<?php
/**
 * @link    https://github.com/old-town/old-town-workflow
 * @author  Malofeykin Andrey  <[email protected]>
 */
namespace OldTown\Workflow\Loader;

use OldTown\Workflow\Exception\FactoryException;
use OldTown\Workflow\Exception\InvalidParsingWorkflowException;
use OldTown\Workflow\Loader\XMLWorkflowFactory\WorkflowConfig;


/**
 * Class ArrayWorkflowFactory
 *
 * @package OldTown\Workflow\Loader
 */
class  ArrayWorkflowFactory extends XmlWorkflowFactory
{
    /**
     *
     * @var string
     */
    const WORKFLOWS_PROPERTY = 'workflows';

    /**
     *
     * @return void
     * @throws FactoryException
     * @throws InvalidParsingWorkflowException
     * @throws \OldTown\Workflow\Exception\RemoteException
     */
    public function initDone()
    {
        $this->reload = true === $this->getProperties()->getProperty(static::RELOAD_PROPERTY, false);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $workflows = $this->getProperties()->getProperty(static::WORKFLOWS_PROPERTY, false);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

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

        $basedir = null;
        foreach ($workflows as $name => $workflowItem) {
            $type = array_key_exists('type', $workflowItem) ?  $workflowItem['type'] : WorkflowConfig::FILE_TYPE;
            $location = array_key_exists('location', $workflowItem) ?  $workflowItem['location'] : '';
            $config = $this->buildWorkflowConfig($basedir, $type, $location);
            $this->workflows[$name] = $config;
        }
    }
}


1			<?php
2			/**
3			* @link https://github.com/old-town/old-town-workflow
4			* @author Malofeykin Andrey <[email protected]>
5			*/
6			namespace OldTown\Workflow\Loader;
7
8			use OldTown\Workflow\Exception\FactoryException;
9			use OldTown\Workflow\Exception\InvalidParsingWorkflowException;
10			use OldTown\Workflow\Loader\XMLWorkflowFactory\WorkflowConfig;
11
12
13			/**
14			* Class ArrayWorkflowFactory
15			*
16			* @package OldTown\Workflow\Loader
17			*/
18			class ArrayWorkflowFactory extends XmlWorkflowFactory
19			{
20			/**
21			*
22			* @var string
23			*/
24			const WORKFLOWS_PROPERTY = 'workflows';
25
26			/**
27			*
28			* @return void
29			* @throws FactoryException
30			* @throws InvalidParsingWorkflowException
31			* @throws \OldTown\Workflow\Exception\RemoteException
32			*/
33			public function initDone()
34			{
35			$this->reload = true === $this->getProperties()->getProperty(static::RELOAD_PROPERTY, false);
			0 ignored issues – show Documentation introduced 2016-02-12 06:27 UTC by Report Bug Copy Issue Report `false` is of type `boolean`, but the function expects a `null\|string`. 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...
36			$workflows = $this->getProperties()->getProperty(static::WORKFLOWS_PROPERTY, false);
			0 ignored issues – show Documentation introduced 2016-02-12 06:27 UTC by Report Bug Copy Issue Report `false` is of type `boolean`, but the function expects a `null\|string`. 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...
37
38			$basedir = null;
39			foreach ($workflows as $name => $workflowItem) {
40			$type = array_key_exists('type', $workflowItem) ? $workflowItem['type'] : WorkflowConfig::FILE_TYPE;
41			$location = array_key_exists('location', $workflowItem) ? $workflowItem['location'] : '';
42			$config = $this->buildWorkflowConfig($basedir, $type, $location);
43			$this->workflows[$name] = $config;
44			}
45			}
46			}
47

old-town / old-town-workflow

GitHub Access Token became invalid

ArrayWorkflowFactory A last analyzed 2016-03-11 07:02 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

1 Method

Duplication Side-by-Side

Filter issues like

ArrayWorkflowFactory A
last analyzed 2016-03-11 07:02 UTC