drupol\htmltag\HtmlTag

Complexity

Total Complexity

9

Size/Duplication

Total Lines	106
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	tag()	0	32	4
A	attributes()	0	23	3
A	attribute()	0	16	2

<?php

namespace drupol\htmltag;

use drupol\htmltag\Attribute\AttributeFactory;
use drupol\htmltag\Attributes\AttributesFactory;
use drupol\htmltag\Tag\CommentFactory;
use drupol\htmltag\Tag\TagFactory;

/**
 * Class HtmlTag
 */
class HtmlTag implements HtmlTagInterface
{
    /**
     * The attribute factory classname.
     *
     * @var string
     */
    protected $attribute_factory_classname = AttributeFactory::class;

    /**
     * The attributes factory classname.
     *
     * @var string
     */
    protected $attributes_factory_classname = AttributesFactory::class;

    /**
     * The tag factory classname.
     *
     * @var string
     */
    protected $tag_factory_classname = TagFactory::class;

    /**
     * {@inheritdoc}
     */
    public static function tag(
        $name,
        array $attributes = [],
        $content = null,
        $attribute_factory_classname = null,
        $attributes_factory_classname = null,
        $tag_factory_classname = null
    ) {
        $static = new static();

        $attributes_factory_classname = null == $attributes_factory_classname ?

It seems like you are loosely comparing $attributes_factory_classname of type string|null against null; this is ambiguous if the string can be empty. Consider using a strict comparison === instead.

            $static->attributes_factory_classname :
            $attributes_factory_classname;

        $attribute_factory_classname = null == $attribute_factory_classname ?

It seems like you are loosely comparing $attribute_factory_classname of type string|null against null; this is ambiguous if the string can be empty. Consider using a strict comparison === instead.

            $static->attribute_factory_classname :
            $attribute_factory_classname;

        $tag_factory_classname = null == $tag_factory_classname ?

It seems like you are loosely comparing $tag_factory_classname of type string|null against null; this is ambiguous if the string can be empty. Consider using a strict comparison === instead.

            $static->tag_factory_classname :
            $tag_factory_classname;
        /** @var \drupol\htmltag\Tag\TagFactoryInterface $tag_factory_classname */
        $tag_factory_classname = (new \ReflectionClass($tag_factory_classname))->newInstance();

        return $tag_factory_classname::build(
            $name,
            $attributes,
            $content,
            $attribute_factory_classname,
            $attributes_factory_classname
        );
    }

    /**
     * {@inheritdoc}
     */
    public static function attributes(
        array $attributes = [],
        $attribute_factory_classname = null,
        $attributes_factory_classname = null
    ) {
        $static = new static();

        $attribute_factory_classname = null == $attribute_factory_classname ?

It seems like you are loosely comparing $attribute_factory_classname of type string|null against null; this is ambiguous if the string can be empty. Consider using a strict comparison === instead.

            $static->attribute_factory_classname :
            $attribute_factory_classname;

        $attributes_factory_classname = null == $attributes_factory_classname ?

It seems like you are loosely comparing $attributes_factory_classname of type string|null against null; this is ambiguous if the string can be empty. Consider using a strict comparison === instead.

            $static->attributes_factory_classname :
            $attributes_factory_classname;

        /** @var \drupol\htmltag\Attributes\AttributesFactoryInterface $attributes_factory_classname */
        $attributes_factory_classname = (new \ReflectionClass($attributes_factory_classname))->newInstance();

        return $attributes_factory_classname::build(
            $attributes,
            $attribute_factory_classname
        );
    }

    /**
     * {@inheritdoc}
     */
    public static function attribute(
        $name,
        $value,
        $attribute_factory_classname = null
    ) {
        $static = new static();

        $attribute_factory_classname = null == $attribute_factory_classname ?

It seems like you are loosely comparing $attribute_factory_classname of type string|null against null; this is ambiguous if the string can be empty. Consider using a strict comparison === instead.

            $static->attribute_factory_classname :
            $attribute_factory_classname;

        /** @var \drupol\htmltag\Attribute\AttributeFactoryInterface $attribute_factory_classname */
        $attribute_factory_classname = (new \ReflectionClass($attribute_factory_classname))->newInstance();

        return $attribute_factory_classname::build($name, $value);
    }
}