Nip\Form\Renderer\Traits\HasAttribsTrait::renderAttributes()

Complexity

Conditions	4
Paths	6

Size

Total Lines	24
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	15
CRAP Score	4.0039

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function renderAttributes($overrides = [])
    {
        $attribs = $this->getElement()->getAttribs();

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

        if (!isset($attribs['title'])) {
            $attribs['title'] = $this->getElement()->getLabel();
        }
        $allowedAttribs = $this->getAllowedAttributes();
        $attribs = array_filter(
            $attribs,
            function ($key) use ($allowedAttribs) {
                return $this->canRenderAttribute($key, $allowedAttribs);
            },
            ARRAY_FILTER_USE_KEY
        );
        $return = '';
        foreach ($attribs as $name => $value) {
            if (in_array($name, array_keys($overrides))) {
                $value = $overrides[$name];
            }

            $return .= ' ' . $name . '="' . $value . '"';
        }

        return $return;
    }