TDMCreateFormRadio

Complexity

Total Complexity

12

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

1 Method

Rating	Name	Duplication	Size	Complexity
D	render()	0	43	12

<?php

/**
 * XOOPS form radio compo.
 *
 * 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   The XOOPS project http://www.xoops.org/
 * @license     GNU GPL 2 (http://www.gnu.org/licenses/old-licenses/gpl-2.0.html)
 *
 * @since       1.91
 *
 * @author      Kazumi Ono (AKA onokazu) http://www.myweb.ne.jp/, http://jp.xoops.org/
 * @author      Taiwen Jiang <phppp@users.sourceforge.net>
 *
 * @version     $Id: TDMCreateFormRadio.php 12360 2014-12-06 13:18:22Z timgno $
 */
class TDMCreateFormRadio extends XoopsFormRadio

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    /**
     * Prepare HTML for output.
     *
     * @return string HTML
     */
    public function render()
    {
        $ret = '';
        $ele_name = $this->getName();

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

        $ele_title = $this->getTitle();
        $ele_value = $this->getValue();
        $ele_options = $this->getOptions();
        $ele_extra = $this->getExtra();
        $ele_delimeter = empty($this->columns) ? $this->getDelimeter() : '';
        if (!empty($this->columns)) {
            $ret .= '<table class="table table-bordered"><tr>';
        }
        $i = 0;
        $id_ele = 0;
        foreach ($ele_options as $value => $name) {
            ++$id_ele;
            if (!empty($this->columns)) {
                if ($i % $this->columns == 0) {
                    $ret .= '<tr>';
                }
                $ret .= '<td class="radio">';
            }
            $ret .= '<input type="radio" name="'.$ele_name.'" id="'.$ele_name.'['.$value.']'.$id_ele.'" title = "'.htmlspecialchars($ele_title, ENT_QUOTES).'" value="'.htmlspecialchars($value, ENT_QUOTES).'"';
            if (isset($ele_value) && $value == $ele_value) {
                $ret .= ' checked="checked"';
            }
            $ret .= $ele_extra.' />'."<label name='xolb_{$ele_name}' for='".$ele_name.'['.$value.']'.$id_ele."'><span><span></span></span>".$name.'</label>'.$ele_delimeter;
            if (!empty($this->columns)) {
                $ret .= '</td>';
                if (++$i % $this->columns == 0) {
                    $ret .= '</tr>';
                }
            }
        }
        if (!empty($this->columns)) {
            if ($span = $i % $this->columns) {
                $ret .= '<td colspan="'.($this->columns - $span).'"></td></tr>';
            }
            $ret .= '</table>';
        }

        return $ret;
    }
}