Ngtfkx\Laradeck\FormBuilder\Traits\Common

Complexity

Total Complexity

20

Size/Duplication

Total Lines	112
Duplicated Lines	17.86 %

Coupling/Cohesion

Components	3
Dependencies	0

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	id()	0	6	1
A	name()	0	6	1
A	class()	10	10	3
A	classes()	10	10	3
B	style()	0	17	5
A	styles()	0	8	2
A	attr()	0	6	1
A	attrs()	0	14	4

<?php

namespace Ngtfkx\Laradeck\FormBuilder\Traits;


use Illuminate\Support\Collection;

trait Common
{
    /**
     * @var string
     */
    protected $id;

    /**
     * @var string
     */
    protected $name;

    /**
     * @var Collection
     */
    protected $classes;

    /**
     * @var Collection
     */
    protected $attributes;

    /**
     * @var Collection
     */
    protected $styles;

    public function id($value): self
    {
        $this->id = $value;

        return $this;
    }

    public function name($value): self
    {
        $this->name = $value;

        return $this;
    }

    public function class(...$classes): self

This method seems to be duplicated in your project.

Possible parse error: non-abstract method defined as abstract

It is generally advisable to only define one property per statement.

    {
        foreach($classes as $class) {
            if(!$this->classes->contains($class)) {
                $this->classes->push($class);
            }
        }

        return $this;

The visibility should be declared for property $this.

    }

    public function classes(iterable $classes): self

This method seems to be duplicated in your project.

    {
        foreach($classes as $class) {
            if(!$this->classes->contains($class)) {
                $this->classes->push($class);
            }
        }

        return $this;
    }

    public function style(string $key, string $value = null): self
    {
        if (empty($value)) {
            $pos = strpos($key, ':', 2);
            if ($pos !== false) {
                $parts = explode(':', strrev($key));
                $key = strrev($parts[1]);
                $value = strrev($parts[0]);
            }
        }

        if ($key && $value) {

The expression $value of type null|string is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

            $this->styles->put($key, $value);
        }

        return $this;
    }

    public function styles(iterable $styles): self
    {
        foreach($styles as $key => $value) {
            $this->styles->put($key, $value);
        }

        return $this;
    }

    public function attr(string $key, string $value = null): self
    {
        $this->attributes->put($key, $value);

        return $this;
    }

    public function attrs(iterable $attributes): self
    {
        foreach($attributes as $key => $value) {
            if ($key === 'class') {
                $this->class($value);
            } else if($key === 'style') {
                $this->style($value);
            } else {
                $this->attributes->put($key, $value);
            }
        }

        return $this;
    }
}