Nip\Form\Traits\HasClassTrait::addClass()

Complexity

Conditions	2
Paths	2

Size

Total Lines	13
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	10
CRAP Score	2

Importance

Changes	3
Bugs	0	Features	3

<?php
    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;
    }