ExpatParser::__construct() - Code Metrics - Inspection of "Fix static analysis issue" - phingofficial/phing - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — main ( 3cec7d...c34b86 )

by Michiel

created 2025-02-18 15:32 UTC

ExpatParser::__construct() A

↳ Parent: ExpatParser

Complexity

Conditions	2
Paths	2

Size

Total Lines	10
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	2.0078

Importance

Changes

Metric	Value
eloc	7
dl	0
loc	10
ccs	7
cts	8
cp	0.875
rs	10
c	0
b	0
f	0
cc	2
nc	2
nop	2
crap	2.0078

<?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 Exception;
use Phing\Io\IOException;
use Phing\Io\Reader;
use SplFileObject;

/**
 * This class is a wrapper for the PHP's internal expat parser.
 *
 * It takes an XML file represented by a abstract path name, and starts
 * parsing the file and calling the different "trap" methods inherited from
 * the AbstractParser class.
 *
 * Those methods then invoke the represenatative methods in the registered
 * handler classes.
 *
 * @author    Andreas Aderhold <[email protected]>
 * @copyright 2001,2002 THYRELL. All rights reserved
 */
class ExpatParser extends AbstractSAXParser
{
    /**
     * @var resource
     */
    private $parser;

    /**
     * @var Reader
     */
    private $reader;

    /**
     * @var SplFileObject
     */
    private $file;

    private $buffer = 4096;


    /**
     * @var Location current cursor pos in XML file
     */
    private $location;

    /**
     * Constructs a new ExpatParser object.
     *
     * The constructor accepts a PhingFile object that represents the filename
     * for the file to be parsed. It sets up php's internal expat parser
     * and options.
     *
     * @param Reader $reader   the Reader Object that is to be read from
     * @param string $filename filename to read
     *
     * @throws Exception if the given argument is not a PhingFile object
     */
    public function __construct(Reader $reader, $filename = null)
    {
        $this->reader = $reader;
        if (null !== $filename) {
            $this->file = new SplFileObject($filename);
        }
        $this->parser = xml_parser_create();
class Id
{
    public $id;

    public function __construct($id)
    {
        $this->id = $id;
    }

}

class Account
{
    /** @var  Id $id */
    public $id;
}

$account_id = false;

if (starsAreRight()) {
    $account_id = new Id(42);
}

$account = new Account();
if ($account instanceof Id)
{
    $account->id = $account_id;
}
        $this->location = new Location();
        xml_set_element_handler($this->parser, [$this, 'startElement'], [$this, 'endElement']);
        xml_set_character_data_handler($this->parser, [$this, 'characters']);
    }

    /**
     * Override PHP's parser default settings, created in the constructor.
     *
     * @param $opt
     * @param $val
     *
     * @return bool true if the option could be set, otherwise false
     *
     * @internal param the $string option to set
     */
    public function parserSetOption($opt, $val)
    {
        return xml_parser_set_option($this->parser, $opt, $val);
    }

    /**
     * Returns the location object of the current parsed element. It describes
     * the location of the element within the XML file (line, char).
     *
     * @return Location the location of the current parser
     */
    public function getLocation()
    {
        if (null !== $this->file) {
            $path = false !== $this->file->getRealPath() ? $this->file->getRealPath() : null;
        } else {
            $path = $this->reader->getResource();
        }
        $this->location = new Location(
            $path,
            xml_get_current_line_number($this->parser),
            xml_get_current_column_number(
                $this->parser
            )
        );

        return $this->location;
    }

    /**
     * Starts the parsing process.
     *
     * @throws ExpatParseException if something gone wrong during parsing
     * @throws IOException         if XML file can not be accessed
     *
     * @return int 1 if the parsing succeeded
     */
    public function parse()
    {
        while (($data = $this->reader->read()) !== -1) {
            if (!xml_parse($this->parser, $data, $this->reader->eof())) {
            if (!xml_parse($this->parser, $data, $this->reader->/** @scrutinizer ignore-call */ eof())) {
                $error = xml_error_string(xml_get_error_code($this->parser));
                $e = new ExpatParseException($error, $this->getLocation());
                xml_parser_free($this->parser);

                throw $e;
            }
        }
        xml_parser_free($this->parser);

        return 1;
    }
}


phingofficial / phing

Push — main ( 3cec7d...c34b86 )

ExpatParser::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like