Xoops\Form\ThemeForm

Complexity

Total Complexity

11

Size/Duplication

Total Lines	81
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
B	render()	0	51	8
A	insertBreak()	0	12	3

<?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.
*/

namespace Xoops\Form;

/**
 * ThemeForm - Form that will output as a theme-enabled HTML table
 *
 * Also adds JavaScript to validate required fields
 *
 * @category  Xoops\Form\ThemeForm
 * @package   Xoops\Form
 * @author    Xoops Team
 * @copyright 2001-2016 XOOPS Project (http://xoops.org)
 * @license   GNU GPL 2 or later (http://www.gnu.org/licenses/gpl-2.0.html)
 * @link      http://xoops.org
 */
class ThemeForm extends Form
{
    /**
     * Insert an empty row in the table to serve as a separator.
     *
     * @param string $extra HTML to be displayed in the empty row.
     * @param string $class CSS class name for <td> tag
     *
     * @return void
     */
    public function insertBreak($extra = '', $class = '')
    {
        $class = ($class != '' ? " class=\"" . $class . "\"" : " class=\"break\"");
        // Fix for $extra tag not showing
        if ($extra) {
            $value = '<div' . $class . '>' . $extra . '</div>';
            $ele = new Raw($value);
            $this->addElement($ele);
        } else {
            $value = '<div' . $class . '>&nbsp;</div>';
            $ele = new Raw($value);
            $this->addElement($ele);
        }
    }

    /**
     * create HTML to output the form as a theme-enabled table with validation.
     *
     * @return string rendered form
     */
    public function render()
    {
        $xoops = \Xoops::getInstance();
        $xoops->theme()->addStylesheet('media/xoops/css/form.css');
        switch ($this->getDisplay()) {
            case '':
            case 'horizontal':
            default:
                $xoops->tpl()->assign('type', 'horizontal');
                break;

            case 'vertical':
                $xoops->tpl()->assign('type', 'vertical');
                break;

            case 'inline':
                $xoops->tpl()->assign('type', 'inline');
                break;

            case 'personalized':
                $xoops->tpl()->assign('type', 'personalized');
                break;
        }
        $xoops->tpl()->assign('title', $this->getTitle());
        $xoops->tpl()->assign('name', $this->getName());
        $xoops->tpl()->assign('action', $this->getAction());
        $xoops->tpl()->assign('method', $this->getMethod());
        $xoops->tpl()->assign('extra', $this->getExtra());
        $hidden = '';
        foreach ($this->getElements() as $ele) {
            /* @var $ele Element */
            if (!$ele->isHidden()) {
                $input['name'] = $ele->getName();
                $input['caption'] = $ele->getCaption();
                $input['description'] = $ele->getDescription();
                $input['ele'] = $ele->render();
                $input['required'] = $ele->isRequired();
                $input['pattern_description'] = $ele->getPatternDescription();
                $input['datalist'] = $ele->renderDatalist();
                $xoops->tpl()->appendByRef('xo_input', $input);

The method appendByRef() does not exist on Xoops\Core\XoopsTpl. Did you maybe mean append()?

The variable $input seems to be defined later in this foreach loop on line 88. Are you sure it is defined here?

                unset($input);
            } else {
                $hidden .= $ele->render(). "\n";
            }

        }
        $xoops->tpl()->assign('hidden', $hidden);
        $xoops->tpl()->assign('validationJS', $this->renderValidationJS(true));
        $ret = $xoops->tpl()->fetch('module:system/system_form.tpl');
        $xoops->tpl()->clearAssign('xo_input');

The method clearAssign() does not exist on Xoops\Core\XoopsTpl. Did you maybe mean clear_assign()?

        return $ret;

    }
}