dokuwiki\Form\FieldsetOpenElement

Complexity

Total Complexity

3

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	toHTML()	0	7	2

<?php

namespace dokuwiki\Form;

/**
 * Class FieldsetOpenElement
 *
 * Opens a Fieldset with an optional legend
 *
 * @package dokuwiki\Form
 */
class FieldsetOpenElement extends TagOpenElement
{

    /**
     * @param string $legend
     * @param array $attributes
     */
    public function __construct($legend='', $attributes = array())
    {
        // this is a bit messy and we just do it for the nicer class hierarchy
        // the parent would expect the tag in $value but we're storing the
        // legend there, so we have to set the type manually
        parent::__construct($legend, $attributes);
        $this->type = 'fieldsetopen';
    }

    /**
     * The HTML representation of this element
     *
     * @return string
     */
    public function toHTML()
    {
        $html = '<fieldset '.buildAttributes($this->attrs()).'>';

It seems like $this->attrs() targeting dokuwiki\Form\Element::attrs() can also be of type this<dokuwiki\Form\FieldsetOpenElement>; however, buildAttributes() does only seem to accept array, maybe add an additional type check?

        $legend = $this->val();
        if ($legend) $html .= DOKU_LF.'<legend>'.hsc($legend).'</legend>';

It seems like $legend defined by $this->val() on line 36 can also be of type this<dokuwiki\Form\FieldsetOpenElement>; however, hsc() does only seem to accept string, maybe add an additional type check?

        return $html;
    }
}