Indigo\Ini\Parser

Complexity

Total Complexity

21

Size/Duplication

Total Lines	91
Duplicated Lines	15.38 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

88.89%

Importance

Changes	1
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
B	parse()	0	26	5
C	normalize()	14	46	16

<?php

namespace Indigo\Ini;

use Indigo\Ini\Exception\ParserException;

/**
 * Parses an INI string.
 *
 * @author Márk Sági-Kazár <mark.sagikazar@gmail.com>
 */
class Parser
{
    /**
     * Parses an INI string.
     *
     * @param string $ini
     *
     * @return array
     */
    public function parse($ini)
    {
        if (!is_string($ini)) {
            throw new ParserException('Cannot parse non-string INI data');
        }

        $scannerMode = defined('INI_SCANNER_TYPED') ? INI_SCANNER_TYPED : INI_SCANNER_NORMAL;

        $parsedIni = @parse_ini_string($ini, true, $scannerMode);

        if (false === $parsedIni) {
            $e = error_get_last();
            throw new ParserException('Error during parsing INI: '.$e['message']);
        }

        // Prior to 5.6.1 we have to do some internal parsing as well
        if (false === defined('INI_SCANNER_TYPED')) {
            // We cannot use INI_SCANNER_RAW by default because it is buggy under PHP 5.3.14 and 5.4.4
            // http://3v4l.org/m24cT
            $rawIni = @parse_ini_string($ini, true, INI_SCANNER_RAW);

            $parsedIni = $this->normalize($parsedIni, $rawIni);
        }

        return $parsedIni;
    }

    /**
     * Normalizes INI and array values.
     *
     * @param $value
     * @param $rawValue
     *
     * @return bool|int|null|string|array
     */
    protected function normalize($value, $rawValue)
    {
        // Normalize array values
        if (is_array($value)) {
            foreach ($value as $i => &$subValue) {
                $subValue = $this->normalize($subValue, $rawValue[$i]);
            }

            return $value;
        }

        // Don't normalize non-string value
        if (!is_string($value)) {
            return $value;
        }

        // Normalize true boolean value
        if ($value === '1'

This code seems to be duplicated across your project.

            && (strcasecmp($rawValue, 'true') === 0
                || strcasecmp($rawValue, 'yes') === 0
                || strcasecmp($rawValue, 'on') === 0)
        ) {
            return true;
        }

        // Normalize false boolean value
        if ($value === ''

This code seems to be duplicated across your project.

            && (strcasecmp($rawValue, 'false') === 0
                || strcasecmp($rawValue, 'no') === 0
                || strcasecmp($rawValue, 'off') === 0)
        ) {
            return false;
        }

        // Normalize null value
        if ($value === '' && strcasecmp($rawValue, 'null') === 0) {
            return null;
        }

        // Normalize numeric value
        if (is_numeric($value) && ((string) ($value + 0) === $value)) {
            return $value + 0;

The expression $value + 0; of type integer|double adds the type double to the return on line 97 which is incompatible with the return type documented by Indigo\Ini\Parser::normalize of type boolean|integer|null|string|array.

        }

        return $value;

The expression return $value; of type integer|double|string is incompatible with the return type documented by Indigo\Ini\Parser::normalize of type boolean|integer|null|string|array as it can also be of type double which is not included in this return type.

    }
}