Manipulator::__construct() - Code Metrics - webino/WebinoDraw - Measure and Improve Code Quality continuously with Scrutinizer

Manipulator::__construct() A
last analyzed 2019-12-25 16:24 UTC

↳ Parent: Manipulator

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	5
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	1

<?php
/**
 * Webino (http://webino.sk)
 *
 * @link        https://github.com/webino/WebinoDraw for the canonical source repository
 * @copyright   Copyright (c) 2012-2017 Webino, s. r. o. (http://webino.sk)
 * @author      Peter Bačinský <[email protected]>
 * @license     BSD-3-Clause
 */

namespace WebinoDraw\Manipulator;

use WebinoDraw\Dom\NodeInterface;
use WebinoDraw\Manipulator\Plugin\PluginArgument;
use WebinoDraw\Manipulator\Plugin\PluginInterface;
use WebinoDraw\VarTranslator\VarTranslator;
use Zend\Stdlib\PriorityQueue;

/**
 * Class Manipulator
 */
class Manipulator
{
    /**
     * @var VarTranslator
     */
    protected $varTranslator;

    /**
     * @var PriorityQueue
     */
    protected $plugins;

    /**
     * @param VarTranslator $varTranslator
     */
    public function __construct(VarTranslator $varTranslator)
    {
        $this->varTranslator = $varTranslator;
        $this->plugins       = new PriorityQueue;
    }

    /**
     * @param PluginInterface $plugin
     * @param int $priority
     * @return self
     */
    public function setPlugin(PluginInterface $plugin, $priority = 1)
    {
        $this->plugins->insert($plugin, $priority);
        return $this;
    }

    /**
     * @param array $options
     * @return self
     */
    public function manipulate(array $options)
    {
        $arg = $this->createPluginArgument($options);

        $this->eachPlugin(
            Plugin\PreLoopPluginInterface::class,
            function ($plugin) use ($arg) {
                $plugin->preLoop($arg);
            }
        );

        if ($arg->isManipulationStopped()) {
            return $this;
        }

        foreach ($arg->getNodes()->toArray() as $node) {
            if (!($node instanceof NodeInterface) || empty($node->parentNode)) {
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
                continue;
            }

            $arg->setNode($node);

            $this->eachPlugin(
                Plugin\InLoopPluginInterface::class,
                function ($plugin) use ($arg) {
                    $plugin->inLoop($arg);
                }
            );

            if ($arg->isManipulationStopped()) {
                return $this;
            }
        }

        $this->eachPlugin(
            Plugin\PostLoopPluginInterface::class,
            function ($plugin) use ($arg) {
                $plugin->postLoop($arg);
            }
        );

        return $this;
    }

    /**
     * @param array $options
     * @return PluginArgument
     */
    protected function createPluginArgument(array $options)
    {
        return new PluginArgument($options);
    }

    /**
     * @param string $className
     * @param callable $callback
     * @return self
     */
    protected function eachPlugin($className, callable $callback)
    {
        foreach ($this->plugins as $plugin) {
            is_subclass_of($plugin, $className)

                and $callback($plugin);
        }

        return $this;
    }
}


1			<?php
2			/**
3			* Webino (http://webino.sk)
4			*
5			* @link https://github.com/webino/WebinoDraw for the canonical source repository
6			* @copyright Copyright (c) 2012-2017 Webino, s. r. o. (http://webino.sk)
7			* @author Peter Bačinský <[email protected]>
8			* @license BSD-3-Clause
9			*/
10
11			namespace WebinoDraw\Manipulator;
12
13			use WebinoDraw\Dom\NodeInterface;
14			use WebinoDraw\Manipulator\Plugin\PluginArgument;
15			use WebinoDraw\Manipulator\Plugin\PluginInterface;
16			use WebinoDraw\VarTranslator\VarTranslator;
17			use Zend\Stdlib\PriorityQueue;
18
19			/**
20			* Class Manipulator
21			*/
22			class Manipulator
23			{
24			/**
25			* @var VarTranslator
26			*/
27			protected $varTranslator;
28
29			/**
30			* @var PriorityQueue
31			*/
32			protected $plugins;
33
34			/**
35			* @param VarTranslator $varTranslator
36			*/
37			public function __construct(VarTranslator $varTranslator)
38			{
39			$this->varTranslator = $varTranslator;
40			$this->plugins = new PriorityQueue;
41			}
42
43			/**
44			* @param PluginInterface $plugin
45			* @param int $priority
46			* @return self
47			*/
48			public function setPlugin(PluginInterface $plugin, $priority = 1)
49			{
50			$this->plugins->insert($plugin, $priority);
51			return $this;
52			}
53
54			/**
55			* @param array $options
56			* @return self
57			*/
58			public function manipulate(array $options)
59			{
60			$arg = $this->createPluginArgument($options);
61
62			$this->eachPlugin(
63			Plugin\PreLoopPluginInterface::class,
64			function ($plugin) use ($arg) {
65			$plugin->preLoop($arg);
66			}
67			);
68
69			if ($arg->isManipulationStopped()) {
70			return $this;
71			}
72
73			foreach ($arg->getNodes()->toArray() as $node) {
74			if (!($node instanceof NodeInterface) \|\| empty($node->parentNode)) {
			0 ignored issues – show Bug introduced 2015-12-15 01:22 UTC by Report Bug Copy Issue Report Accessing `parentNode` on the interface `WebinoDraw\Dom\NodeInterface` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
75			continue;
76			}
77
78			$arg->setNode($node);
79
80			$this->eachPlugin(
81			Plugin\InLoopPluginInterface::class,
82			function ($plugin) use ($arg) {
83			$plugin->inLoop($arg);
84			}
85			);
86
87			if ($arg->isManipulationStopped()) {
88			return $this;
89			}
90			}
91
92			$this->eachPlugin(
93			Plugin\PostLoopPluginInterface::class,
94			function ($plugin) use ($arg) {
95			$plugin->postLoop($arg);
96			}
97			);
98
99			return $this;
100			}
101
102			/**
103			* @param array $options
104			* @return PluginArgument
105			*/
106			protected function createPluginArgument(array $options)
107			{
108			return new PluginArgument($options);
109			}
110
111			/**
112			* @param string $className
113			* @param callable $callback
114			* @return self
115			*/
116			protected function eachPlugin($className, callable $callback)
117			{
118			foreach ($this->plugins as $plugin) {
119			is_subclass_of($plugin, $className)
			0 ignored issues – show Bug introduced 2015-12-15 01:22 UTC by Report Bug Copy Issue Report Due to PHP Bug #53727, `is_subclass_of` might return inconsistent results on some PHP versions if `$className` can be an interface. If so, you could instead use `ReflectionClass::implementsInterface`. Loading history...
120			and $callback($plugin);
121			}
122
123			return $this;
124			}
125			}
126

webino / WebinoDraw

Manipulator::__construct() A last analyzed 2019-12-25 16:24 UTC

Complexity

Size

Duplication

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like

Manipulator::__construct() A
last analyzed 2019-12-25 16:24 UTC