mambax7 / extcal

class/form/formfilecheckbox.php

<?php
/*
 * 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 ExtcalFormFileCheckBox.
 */
class ExtcalFormFileCheckBox extends XoopsFormCheckBox
{
    /**
     * @param      $caption
     * @param      $name
     * @param null $value
     *
     * @return ExtcalFormFileCheckBox
     */
    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() . '[]';
            $this->setName($newname);
        }
        foreach ($this->getOptions() as $value => $name) {
            $ret .= "<input type='checkbox' name='" . $this->getName() . "' value='" . $value . "'";
            $ret .= ' checked';
            $ret .= $this->getExtra() . '>' . $name . "<br>\n";
        }

        return $ret;
    }
}