XoopsModules\Extcal\Form\FormFileCheckBox

Complexity

Total Complexity

5

Size/Duplication

Total Lines	35
Duplicated Lines	11.43 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	render()	4	15	4

<?php namespace XoopsModules\Extcal\Form;
/*
 * You may not change or alter any portion of this comment or credits
 * of supporting developers from this source code or any supporting source code
 * which is considered copyrighted (c) material of the original comment or credit authors.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
 */

/**
 * @copyright    {@link https://xoops.org/ XOOPS Project}
 * @license      {@link http://www.gnu.org/licenses/gpl-2.0.html GNU GPL 2 or later}
 * @package      extcal
 * @since
 * @author       XOOPS Development Team,
 */

/**
 * Class FormFileCheckBox.
 */
class FormFileCheckBox extends \XoopsFormCheckBox
{
    /**
     * @param      $caption
     * @param      $name
     * @param null $value
     *
     * @return FormFileCheckBox

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct($caption, $name, $value = null)
    {
        parent::__construct($caption, $name, $value);
    }

    /**
     * prepare HTML for output.
     *
     * @return string
     */
    public function render()
    {
        $ret = '';
        if (count($this->getOptions()) > 1 && '[]' !== substr($this->getName(), -2, 2)) {

Consider using $this->name. There is an issue with getName() and APC-enabled PHP versions.

This code seems to be duplicated across your project.

            $newname = $this->getName() . '[]';

Consider using $this->name. There is an issue with getName() and APC-enabled PHP versions.

            $this->setName($newname);
        }
        foreach ($this->getOptions() as $value => $name) {
            $ret .= "<input type='checkbox' name='" . $this->getName() . "' value='" . $value . "'";

Consider using $this->name. There is an issue with getName() and APC-enabled PHP versions.

            $ret .= ' checked';
            $ret .= $this->getExtra() . '>' . $name . "<br>\n";
        }

        return $ret;
    }
}