drupol\htmltag\HtmlTag

Complexity

Total Complexity

11

Size/Duplication

Total Lines	134
Duplicated Lines	24.63 %

Coupling/Cohesion

Components	2
Dependencies	4

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	tag()	0	32	4
A	comment()	17	17	2
A	attributes()	0	23	3
A	attribute()	16	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;

    /**
     * The comment factory classname.
     *
     * @var string
     */
    protected $comment_factory_classname = CommentFactory::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 comment(

This method seems to be duplicated in your project.

        $content = null,
        $comment_factory_classname = null
    ) {
        $static = new static();

        $comment_factory_classname = null == $comment_factory_classname ?
            $static->comment_factory_classname :
            $comment_factory_classname;

        /** @var \drupol\htmltag\Tag\CommentFactory $comment_factory_classname */
        $comment_factory_classname = (new \ReflectionClass($comment_factory_classname))->newInstance();

        return $comment_factory_classname::build(
            $content
        );
    }

    /**
     * {@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(

This method seems to be duplicated in your project.

        $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);
    }
}