bytic / form

src/Traits/HasClassTrait.php

<?php

namespace Nip\Form\Traits;

/**
 * Trait HasClassTrait
 * @package Nip\Form\Traits
 */
trait HasClassTrait
{
    /**
     * @return $this
     */
    public function addClass()
    {
        $classes = func_get_args();
        $classes = array_map('trim', $classes);
        if (is_array($classes)) {

The condition is_array($classes) is always true.

            $classes = implode(' ', $classes) . ' ' . $this->getAttrib('class');

It seems like getAttrib() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            $newClasses = explode(' ', $classes);
            $newClasses = array_filter($newClasses);
            $newClasses = array_unique($newClasses);
            $this->setAttrib('class', trim(implode(' ', $newClasses)));

It seems like setAttrib() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }

        return $this;
    }

    /**
     * @return $this
     */
    public function removeClass()
    {
        $removeClasses = func_get_args();
        $removeClasses = array_map('trim', $removeClasses);
        if (is_array($removeClasses)) {

The condition is_array($removeClasses) is always true.

            $classes = explode(' ', $this->getAttrib('class'));
            foreach ($removeClasses as $class) {
                $key = array_search($class, $classes);
                if ($key !== false) {
                    unset($classes[$key]);
                }
            }
            $this->setAttrib('class', implode(' ', $classes));
        }

        return $this;
    }

    /**
     * @param string $class
     * @return bool
     */
    public function hasClass($class)
    {
        return in_array($class, explode(' ', $this->getAttrib('class')));
    }
}