BlockForm::render() - Code Metrics - XoopsModules25x/about - Measure and Improve Code Quality continuously with Scrutinizer

BlockForm::render() A
last analyzed 2022-02-20 18:48 UTC

↳ Parent: BlockForm

Complexity

Conditions	6
Paths	6

Size

Total Lines	21
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	14
dl	0
loc	21
rs	9.2222
c	0
b	0
f	0
cc	6
nc	6
nop	0

<?php

namespace XoopsModules\About;

/**
 * About
 *
 * 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 Co.Ltd. http://www.xoops.com.cn
 * @copyright      XOOPS Project (https://xoops.org)
 * @license        GNU GPL 2 or later (https://www.gnu.org/licenses/gpl-2.0.html)
 * @package        about
 * @since          1.0.0
 * @author         Susheng Yang <[email protected]>
 */

require_once XOOPS_ROOT_PATH . '/class/xoopsform/form.php';

/**
 * Class BlockForm
 */
class BlockForm extends \XoopsForm
{
    /**
     * BlockForm constructor.
     */
    public function __construct()
    {
    }

    /**
     * create HTML to output the form as a table
     *
     * @return string
     */
    public function render()
    {
        $ret    = '<div>';
        $hidden = '';
        foreach ($this->getElements() as $ele) {
            if (!\is_object($ele)) {
                $ret .= $ele;
            } elseif (!$ele->isHidden()) {
                if ('' !== ($caption = $ele->getCaption())) {
                    $ret .= "<div class='xoops-form-element-caption" . ($ele->isRequired() ? '-required' : '') . "'>" . "<span class='caption-text'><strong>{$caption}</strong></span>" . "<span class='caption-marker'>*</span>" . '</div>';
                }

                $ret .= "<div style='margin:5px 0 8px 0; '>" . $ele->render() . "</div>\n";
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
            } else {
                $hidden .= $ele->render();
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
            }
        }
        $ret .= '</div>';
        $ret .= $this->renderValidationJS(true);

        return $ret;
    }
}


1			<?php
2
3			namespace XoopsModules\About;
4
5			/**
6			* About
7			*
8			* You may not change or alter any portion of this comment or credits
9			* of supporting developers from this source code or any supporting source code
10			* which is considered copyrighted (c) material of the original comment or credit authors.
11			* This program is distributed in the hope that it will be useful,
12			* but WITHOUT ANY WARRANTY; without even the implied warranty of
13			* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
14			*
15			* @copyright The XOOPS Co.Ltd. http://www.xoops.com.cn
16			* @copyright XOOPS Project (https://xoops.org)
17			* @license GNU GPL 2 or later (https://www.gnu.org/licenses/gpl-2.0.html)
18			* @package about
19			* @since 1.0.0
20			* @author Susheng Yang <[email protected]>
21			*/
22
23			require_once XOOPS_ROOT_PATH . '/class/xoopsform/form.php';
24
25			/**
26			* Class BlockForm
27			*/
28			class BlockForm extends \XoopsForm
29			{
30			/**
31			* BlockForm constructor.
32			*/
33			public function __construct()
34			{
35			}
36
37			/**
38			* create HTML to output the form as a table
39			*
40			* @return string
41			*/
42			public function render()
43			{
44			$ret = '<div>';
45			$hidden = '';
46			foreach ($this->getElements() as $ele) {
47			if (!\is_object($ele)) {
48			$ret .= $ele;
49			} elseif (!$ele->isHidden()) {
50			if ('' !== ($caption = $ele->getCaption())) {
51			$ret .= "<div class='xoops-form-element-caption" . ($ele->isRequired() ? '-required' : '') . "'>" . "<span class='caption-text'><strong>{$caption}</strong></span>" . "<span class='caption-marker'>*</span>" . '</div>';
52			}
53
54			$ret .= "<div style='margin:5px 0 8px 0; '>" . $ele->render() . "</div>\n";
			0 ignored issues – show Bug introduced 2019-02-12 05:23 UTC by Report Bug Copy Issue Report Are you sure the usage of `$ele->render()` targeting `XoopsFormElement::render()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
55			} else {
56			$hidden .= $ele->render();
			0 ignored issues – show Bug introduced 2019-02-12 05:23 UTC by Report Bug Copy Issue Report Are you sure the usage of `$ele->render()` targeting `XoopsFormElement::render()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
57			}
58			}
59			$ret .= '</div>';
60			$ret .= $this->renderValidationJS(true);
61
62			return $ret;
63			}
64			}
65

XoopsModules25x / about

BlockForm::render() A last analyzed 2022-02-20 18:48 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

BlockForm::render() A
last analyzed 2022-02-20 18:48 UTC