XoopsModules\Smartobject\Form\Elements\SmartFormSectionClose

Complexity

Total Complexity

3

Size/Duplication

Total Lines	40
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	5	5	1
A	getValue()	4	4	1
A	render()	4	4	1

<?php namespace XoopsModules\Smartobject\Form\Elements;
/*
 * 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    XOOPS Project https://xoops.org/
 * @license      GNU GPL 2 or later (http://www.gnu.org/licenses/gpl-2.0.html)
 * @package
 * @since
 * @author       XOOPS Development Team, Kazumi Ono (AKA onokazu)
 */

use XoopsModules\Smartobject;

// defined('XOOPS_ROOT_PATH') || die('Restricted access');

/**
 * Class SmartFormSectionClose
 */
class SmartFormSectionClose extends \XoopsFormElement

This class seems to be duplicated in your project.

{
    /**
     * Text
     * @var string
     * @access  private
     */
    public $_value;

    /**
     * SmartFormSectionClose constructor.
     * @param      $sectionname
     * @param bool $value
     */
    public function __construct($sectionname, $value = false)
    {
        $this->setName($sectionname);
        $this->_value = $value;

The property $_value was declared of type string, but $value is of type boolean. Maybe add a type cast?

    }

    /**
     * Get the text
     *
     * @return string
     */
    public function getValue()
    {
        return $this->_value;
    }

    /**
     * Prepare HTML for output
     *
     * @return string
     */
    public function render()
    {
        return $this->getValue();
    }
}