WsdlToPhp\WsdlHandler\Tag\AbstractTagImport

Complexity

Total Complexity

6

Size/Duplication

Total Lines	23
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	getLocationAttributeValue()	0	17	6

<?php

declare(strict_types=1);

namespace WsdlToPhp\WsdlHandler\Tag;

abstract class AbstractTagImport extends Tag
{
    public const ATTRIBUTE_LOCATION = 'location';
    public const ATTRIBUTE_SCHEMA_LOCATION = 'schemaLocation';
    public const ATTRIBUTE_SCHEMA_LOCATION_ = 'schemalocation';

    public function getLocationAttributeValue(): string
    {
        $location = '';

        if ($this->hasAttribute(self::ATTRIBUTE_LOCATION)) {
            $location = $this->getAttributeValue(self::ATTRIBUTE_LOCATION, true);
        } elseif ($this->hasAttribute(self::ATTRIBUTE_SCHEMA_LOCATION)) {
            $location = $this->getAttributeValue(self::ATTRIBUTE_SCHEMA_LOCATION, true);
        } elseif ($this->hasAttribute(self::ATTRIBUTE_SCHEMA_LOCATION_)) {
            $location = $this->getAttributeValue(self::ATTRIBUTE_SCHEMA_LOCATION_, true);
        }

        if (!empty($location) && './' === mb_substr($location, 0, 2)) {

It seems like $location can also be of type boolean; however, parameter $string of mb_substr() does only seem to accept string, maybe add an additional type check?

            $location = mb_substr($location, 2);
        }

        return $location;

The expression return $location could return the type boolean|null which is incompatible with the type-hinted return string. Consider adding an additional type-check to rule them out.

    }
}