ProjectHandler::init() - Code Metrics - Inspection of "Adjusted filter regex" - phingofficial/phing - Measure and Improve Code Quality continuously with Scrutinizer

Passed
Push — master ( e1f86a...4e1a3a )

by Siad
created 2021-01-24 21:08 UTC
ProjectHandler::init() F

↳ Parent: ProjectHandler
Complexity

Conditions	20
Paths	2081
Size

Total Lines	94
Code Lines	62
Duplication

Lines	0
Ratio	0 %
Code Coverage

Tests	44
CRAP Score	26.5703
Importance

Changes
Metric	Value
cc	20
eloc	62
nc	2081
nop	2
dl	0
loc	94
ccs	44
cts	59
cp	0.7458
crap	26.5703
rs	0
c	0
b	0
f	0
How to fix Long Method Complexity

<?php
/**
 * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
 * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
 * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
 * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
 * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
 * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
 * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
 * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
 * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 *
 * This software consists of voluntary contributions made by many individuals
 * and is licensed under the LGPL. For more information please see
 * <http://phing.info>.
 */

namespace Phing\Parser;

use Phing\Parser\ElementHandler;
use Phing\Parser\ExpatParseException;
use Phing\Parser\ExpatParser;
use Phing\Parser\AbstractHandler;
use Phing\Parser\TargetHandler;
use Phing\Util\StringHelper;
use Phing\Io\File;
use Phing\Parser\XmlContext;
use Phing\Parser\ProjectConfigurator;

/**
 * Handler class for the <project> XML element This class handles all elements
 * under the <project> element.
 *
 * @author    Andreas Aderhold <[email protected]>
 * @copyright (c) 2001,2002 THYRELL. All rights reserved
 * @package   phing.parser
 */
class ProjectHandler extends AbstractHandler
{

    /**
     * The phing project configurator object.
     *
     * @var ProjectConfigurator
     */
    private $configurator;

    /**
     * @var XmlContext
     */
    private $context;

    /**
     * Constructs a new ProjectHandler
     *
     * @param ExpatParser $parser the ExpatParser object
     * @param AbstractHandler $parentHandler the parent handler that invoked this handler
     * @param ProjectConfigurator $configurator the ProjectConfigurator object
     * @param XmlContext $context
     */
    public function __construct(
        ExpatParser $parser,
        AbstractHandler $parentHandler,
        ProjectConfigurator $configurator,
        XmlContext $context
    ) {
        parent::__construct($parser, $parentHandler);

        $this->configurator = $configurator;
        $this->context = $context;
    }

    /**
     * Executes initialization actions required to setup the project. Usually
     * this method handles the attributes of a tag.
     *
     * @param string $tag the tag that comes in
     * @param array $attrs attributes the tag carries
     * @throws ExpatParseException if attributes are incomplete or invalid
     */
    public function init($tag, $attrs)
    public function init(/** @scrutinizer ignore-unused */ $tag, $attrs)
    {
        $def = null;
        $name = null;
        $id = null;
        $desc = null;
        $baseDir = null;
        $ver = null;
        $strict = null;

        // some shorthands
        $project = $this->configurator->project;
        $buildFileParent = $this->configurator->buildFileParent;

        foreach ($attrs as $key => $value) {
            if ($key === "default") {
                $def = $value;
            } elseif ($key === "name") {
                $name = $value;
            } elseif ($key === "id") {
                $id = $value;
            } elseif ($key === "basedir") {
                $baseDir = $value;
            } elseif ($key === "description") {
                $desc = $value;
            } elseif ($key === "phingVersion") {
                $ver = $value;
            } elseif ($key === 'strict') {
                $strict = $value;
            } else {
                throw new ExpatParseException("Unexpected attribute '$key'");
            }
        }
        // these things get done no matter what
        if (null == $name) {
            $name = basename($this->configurator->getBuildFile());
        }

        $canonicalName = self::canonicalName($name);
        $this->configurator->setCurrentProjectName($canonicalName);
        $path = (string) $this->configurator->getBuildFile();
        $project->setUserProperty("phing.file.{$canonicalName}", $path);
        $project->setUserProperty("phing.dir.{$canonicalName}", dirname($path));

        if ($this->configurator->isIgnoringProjectTag()) {
            return;
        }

        if ($def === null) {
            throw new ExpatParseException(
                "The default attribute of project is required"
            );
        }

        $project->setDefaultTarget($def);

        if ($name !== null) {
            $project->setName($name);
            $project->addReference($name, $project);
        }

        if ($id !== null) {
            $project->addReference($id, $project);
        }

        if ($desc !== null) {
            $project->setDescription($desc);
        }

        if ($ver !== null) {
            $project->setPhingVersion($ver);
        }

        if ($strict !== null) {
            $project->setStrictMode(StringHelper::booleanValue($strict));
        }

        if ($project->getProperty("project.basedir") !== null) {

            $project->setBasedir($project->getProperty("project.basedir"));
        } else {
            if ($baseDir === null) {
                $project->setBasedir($buildFileParent->getAbsolutePath());
            } else {
                // check whether the user has specified an absolute path
                $f = new File($baseDir);
                if ($f->isAbsolute()) {
                    $project->setBasedir($baseDir);
                } else {
                    $project->setBaseDir($project->resolveFile($baseDir, $buildFileParent));
                }
            }
        }

        $project->addTarget("", $this->context->getImplicitTarget());
    }

    /**
     * Handles start elements within the <project> tag by creating and
     * calling the required handlers for the detected element.
     *
     * @param string $name the tag that comes in
     * @param array $attrs attributes the tag carries
     * @throws ExpatParseException if a unxepected element occurs
     */
    public function startElement($name, $attrs)
    {
        $project = $this->configurator->project;
        $types = $project->getDataTypeDefinitions();


        if ($name === "target" || $name === "extension-point") {
            $tf = new TargetHandler($this->parser, $this, $this->configurator, $this->context);
            $tf->init($name, $attrs);
        } else {
            $tf = new ElementHandler(
                $this->parser,
                $this,
                $this->configurator,
                null,
                null,
                $this->context->getImplicitTarget()
            );
            $tf->init($name, $attrs);
        }
    }

    /**
     * @param $name
     * @return mixed
     */
    public static function canonicalName($name)
    {
        return preg_replace('/\W/', '_', strtolower($name));
    }
}

phingofficial / phing

Push — master ( e1f86a...4e1a3a )

ProjectHandler::init() F

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like
