Spatie\Html\Html

Complexity

Total Complexity

41

Size/Duplication

Total Lines	375
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	14

Importance

Changes

0

31 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	a()	0	6	1
A	button()	0	6	1
A	checkbox()	0	5	1
A	div()	0	4	1
A	email()	0	4	1
A	element()	0	4	1
A	input()	0	8	2
A	fieldset()	0	6	2
A	form()	0	20	4
A	hidden()	0	4	1
A	label()	0	6	1
A	legend()	0	4	1
A	mailto()	0	4	1
A	option()	0	7	1
A	password()	0	4	1
A	radio()	0	5	1
A	select()	0	8	2
A	span()	0	4	1
A	submit()	0	4	1
A	tel()	0	4	1
A	text()	0	4	1
A	textarea()	0	7	1
A	token()	0	4	1
A	model()	0	6	1
A	modelForm()	0	6	1
A	endModel()	0	6	1
A	closeModelForm()	0	6	1
A	old()	0	14	4
A	fieldName()	0	4	1
A	ensureModelIsAvailable()	0	6	2

<?php

namespace Spatie\Html;

use Spatie\Html\Elements\A;
use Illuminate\Http\Request;
use Spatie\Html\Elements\Div;
use Spatie\Html\Elements\Form;
use Spatie\Html\Elements\Span;
use Spatie\Html\Elements\Input;
use Spatie\Html\Elements\Label;
use Spatie\Html\Elements\Button;
use Spatie\Html\Elements\Legend;
use Spatie\Html\Elements\Option;
use Spatie\Html\Elements\Select;
use Spatie\Html\Elements\Element;
use Spatie\Html\Elements\Fieldset;
use Spatie\Html\Elements\Textarea;
use Illuminate\Contracts\Support\Htmlable;

class Html
{
    /** @var \Illuminate\Http\Request */
    protected $request;

    /** @var \ArrayAccess|array */
    protected $model;

    public function __construct(Request $request)

You have injected the Request via parameter $request. This is generally not recommended as there might be multiple instances during a request cycle (f.e. when using sub-requests). Instead, it is recommended to inject the RequestStack and retrieve the current request each time you need it via getCurrentRequest().

    {
        $this->request = $request;
    }

    /**
     * @param string $href
     * @param string $text
     *
     * @return \Spatie\Html\Elements\A
     */
    public function a(?string $href = '', ?string $text = '')
    {
        return A::create()
            ->attributeIf($href, 'href', $href)

$href is of type string, but the function expects a boolean.

            ->text($text);
    }

    /**
     * @param string $type
     * @param string $text
     *
     * @return \Spatie\Html\Elements\Button
     */
    public function button(?string $text = '', ?string $type = 'button')
    {
        return Button::create()
            ->type($type)
            ->text($text);
    }

    /**
     * @param string $value
     *
     * @return \Spatie\Html\Elements\Input
     */
    public function checkbox(string $name = '', ?bool $checked = false, ?string $value = '1')
    {
        return $this->input('checkbox', $name, $value)
            ->attributeIf((bool) $this->old($name, $checked), 'checked');

$checked is of type boolean, but the function expects a string.

    }

    /**
     * @param \Spatie\Html\HtmlElement|string $contents
     *
     * @return \Spatie\Html\Elements\Div
     */
    public function div($contents = null)
    {
        return Div::create()->children($contents);
    }

    /**
     * @param string $value
     *
     * @return \Spatie\Html\Elements\Input
     */
    public function email(string $name = '', ?string $value = '')
    {
        return $this->input('email', $name, $value);
    }

    /**
     * @param string $tag
     *
     * @return \Spatie\Html\Elements\Element
     */
    public function element(string $tag)
    {
        return Element::withTag($tag);
    }

    /**
     * @param string $type
     * @param string $name
     * @param string $value
     *
     * @return \Spatie\Html\Elements\Input
     */
    public function input(?string $type = '', string $name = '', ?string $value = '')
    {
        return Input::create()
            ->attributeIf($type, 'type', $type)

$type is of type string, but the function expects a boolean.

            ->attributeIf($name, 'name', $this->fieldName($name))

$name is of type string, but the function expects a boolean.

            ->attributeIf($name, 'id', $this->fieldName($name))

$name is of type string, but the function expects a boolean.

            ->attributeIf($name && $this->old($name, $value), 'value', $this->old($name, $value));

It seems like $this->old($name, $value) targeting Spatie\Html\Html::old() can also be of type array or null; however, Spatie\Html\BaseElement::attributeIf() does only seem to accept string, maybe add an additional type check?

    }

    /**
     * @param \Spatie\Html\HtmlElement|string $legend
     *
     * @return \Spatie\Html\Elements\Fieldset
     */
    public function fieldset($legend = null)
    {
        return $legend ?
            Fieldset::create()->legend($legend) :
            Fieldset::create();
    }

    /**
     * @param string $method
     * @param string $action
     * @param array $parameters

There is no parameter named $parameters. Was it maybe removed?

     *
     * @return \Spatie\Html\Elements\Form
     */
    public function form(string $method = 'POST', string $action = '')
    {
        $method = strtoupper($method);
        $form = Form::create();

        // If Laravel needs to spoof the form's method, we'll append a hidden
        // field containing the actual method
        if (in_array($method, ['DELETE', 'PATCH', 'PUT'])) {
            $form = $form->addChild($this->hidden('_method')->value($method));
        }

        // On any other method that get, the form needs a CSRF token
        if ($method !== 'GET') {
            $form = $form->addChild($this->token());
        }

        return $form
            ->method($method === 'GET' ? 'GET' : 'POST')
            ->attributeIf($action, 'action', $action);

$action is of type string, but the function expects a boolean.

    }

    /**
     * @param string $name
     * @param string $value
     *
     * @return \Spatie\Html\Elements\Input
     */
    public function hidden(string $name = '', ?string $value = '')
    {
        return $this->input('hidden', $name, $value);
    }

    /**
     * @param \Spatie\Html\HtmlElement|iterable|string|null $contents
     * @param string $for
     *
     * @return \Spatie\Html\Elements\Label
     */
    public function label($contents = null, string $for = '')
    {
        return Label::create()
            ->attributeIf($for, 'for', $this->fieldName($for))

$for is of type string, but the function expects a boolean.

            ->children($contents);
    }

    /**
     * @param \Spatie\Html\HtmlElement|string $contents
     *
     * @return \Spatie\Html\Elements\Legend
     */
    public function legend($contents = null)
    {
        return Legend::create()->html($contents);

It seems like $contents defined by parameter $contents on line 186 can also be of type null or object<Spatie\Html\HtmlElement>; however, Spatie\Html\BaseElement::html() does only seem to accept string, maybe add an additional type check?

    }

    /**
     * @param string $email
     * @param string $text
     *
     * @return \Spatie\Html\Elements\A
     */
    public function mailto(string $email, string $text = '')
    {
        return $this->a('mailto:'.$email, $text);
    }

    /**
     * @param string $text
     * @param string $value
     * @param bool $selected
     *
     * @return \Spatie\Html\Elements\Option
     */
    public function option(?string $text = '', ?string $value = '', $selected = false)
    {
        return Option::create()
            ->text($text)
            ->value($value)
            ->selectedIf($selected);
    }

    /**
     * @param string $value

There is no parameter named $value. Was it maybe removed?

     *
     * @return \Spatie\Html\Elements\Input
     */
    public function password(string $name = '')
    {
        return $this->input('password', $name)->value('');
    }

    /**
     * @param string $value
     *
     * @return \Spatie\Html\Elements\Input
     */
    public function radio(string $name = '', ?bool $checked = false, ?string $value = '')
    {
        return $this->input('radio', $name, $value)
            ->attributeIf((bool) $this->old($name, $checked), 'checked');

$checked is of type boolean, but the function expects a string.

    }

    /**
     * @param string $name
     * @param iterable $options
     * @param string $value
     *
     * @return \Spatie\Html\Elements\Select
     */
    public function select(string $name = '', iterable $options = [], ?string $value = '')
    {
        return Select::create()
            ->attributeIf($name, 'name', $this->fieldName($name))

$name is of type string, but the function expects a boolean.

            ->attributeIf($name, 'id', $this->fieldName($name))

$name is of type string, but the function expects a boolean.

            ->options($options)

It seems like $options defined by parameter $options on line 245 can also be of type array; however, Spatie\Html\Elements\Select::options() does only seem to accept object<Spatie\Html\Elements\iterable>, maybe add an additional type check?

            ->value($name ? $this->old($name, $value) : '');

It seems like $name ? $this->old($name, $value) : '' can also be of type array; however, Spatie\Html\Elements\Select::value() does only seem to accept string, maybe add an additional type check?

    }

    /**
     * @param \Spatie\Html\HtmlElement|string $contents
     *
     * @return \Spatie\Html\Elements\Span
     */
    public function span($contents = null)
    {
        return Span::create()->children($contents);
    }

    /**
     * @param string $value

There is no parameter named $value. Was it maybe removed?

     *
     * @return \Spatie\Html\Elements\Button
     */
    public function submit(?string $text = '')
    {
        return $this->button($text, 'submit');
    }

    /**
     * @param string $number
     * @param string $text
     *
     * @return \Spatie\Html\Elements\A
     */
    public function tel(?string $number, ?string $text = '')
    {
        return $this->a('tel:'.$number, $text);
    }

    /**
     * @param string $name
     * @param string $value
     *
     * @return \Spatie\Html\Elements\Input
     */
    public function text(string $name = '', ?string $value = '')
    {
        return $this->input('text', $name, $value);
    }

    /**
     * @param string $name
     * @param string $value
     *
     * @return \Spatie\Html\Elements\Textarea
     */
    public function textarea(string $name = '', ?string $value = '')
    {
        return Textarea::create()
            ->attributeIf($name, 'name', $this->fieldName($name))

$name is of type string, but the function expects a boolean.

            ->attributeIf($name, 'id', $this->fieldName($name))

$name is of type string, but the function expects a boolean.

            ->value($this->old($name, $value));

It seems like $this->old($name, $value) targeting Spatie\Html\Html::old() can also be of type array or null; however, Spatie\Html\Elements\Textarea::value() does only seem to accept string, maybe add an additional type check?

    }

    /**
     * @return \Spatie\Html\Elements\Input
     */
    public function token()
    {
        return $this->hidden('_token')->value($this->request->session()->token());

The method token() does not seem to exist on object<Symfony\Component\HttpFoundation\Session\SessionInterface>.

    }

    /**
     * @param \ArrayAccess|array $model
     *
     * @return $this
     */
    public function model($model)
    {
        $this->model = $model;

        return $this;
    }

    /**
     * @param \ArrayAccess|array $model
     * @param string $method
     * @param string $action
     *
     * @return \Spatie\Html\Elements\Form
     */
    public function modelForm($model, string $method = 'POST', string $action = ''): Form
    {
        $this->model($model);

        return $this->form($method, $action);
    }

    /**
     * @return $this
     */
    public function endModel()
    {
        $this->model = null;

        return $this;
    }

    /**
     * @return \Illuminate\Contracts\Support\Htmlable
     */
    public function closeModelForm(): Htmlable
    {
        $this->endModel();

        return $this->form()->close();
    }

    /**
     * @param string $name
     *
     * @return mixed
     */
    protected function old(string $name, ?string $value = '')
    {
        if (empty($name)) {
            return;
        }

        // If there's no default value provided, and the html builder currently
        // has a model assigned, try to retrieve a value from the model.
        if (empty($value) && $this->model) {
            $value = $this->model[$name] ?? '';
        }

        return $this->request->old($name, $value);
    }

    protected function fieldName(string $name): string
    {
        return $name;
    }

    protected function ensureModelIsAvailable()
    {
        if (empty($this->model)) {
            throw new Exception('Method requires a model to be set on the html builder');
        }
    }
}