Nip_Form_Renderer_Bootstrap5::renderRow() - Code Metrics - Inspection of "Merge branch '0.9' into develop" - bytic/form - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( db3995...c553d3 )

by Gabriel

created 2021-03-19 16:13 UTC

Nip_Form_Renderer_Bootstrap5::renderRow() B

↳ Parent: Nip_Form_Renderer_Bootstrap5

Complexity

Conditions	9
Paths	50

Size

Total Lines	39
Code Lines	24

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
eloc	24
c	1
b	0
f	1
dl	0
loc	39
rs	8.0555
cc	9
nc	50
nop	1

<?php

/**
 * Class Bootstrap4
 */
class Nip_Form_Renderer_Bootstrap5 extends Nip_Form_Renderer_Bootstrap
{
    /**
     * @param Nip_Form_Element_Abstract $element
     * @return string
     */
    public function renderRow($element)
    {
        $return = '';
        if (!$element->isRendered()) {
            if ($element->hasCustomRenderer()) {
                return $element->render();
            }

            $formType = $this->getForm()->hasClass('form-horizontal') ? 'form-horizontal' : '';

            $rowClass = 'row';
            $rowClass .= ' row-' . $element->getUniqueId();
            $rowClass .= $element->isError() ? ' has-error' : '';
            $return .= '<div class="' . $rowClass . '">';

            $renderLabel = $element->getOption('render_label');
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
            if ($renderLabel !== false) {

                $return .= $this->renderLabel($element);
            }

            $class = '';
            if ($formType == 'form-horizontal') {
                $class = $element->getType() == 'checkbox' ? 'col-sm-offset-3 col-sm-9' : 'col-sm-9';
            }

            $return .= '<div class="' . $class . '">';
            $return .= $this->renderElement($element);

            $helpBlock = $element->getOption('form-help');
            if ($helpBlock) {
                $return .= '<span class="help-block">' . $helpBlock . '</span>';
            }

            $return .= $element->renderErrors();
            $return .= '</div>';
            $return .= '</div>';
        }

        return $return;
    }

    /**
     * @return string
     */
    public function renderButtons()
    {
        $return = '';
        $buttons = $this->getForm()->getButtons();

        if ($buttons) {

            $return .= '<div class="row">
                            <div class="' . ($this->getForm()->hasClass('form-horizontal') ? 'col-sm-offset-3 col-sm-9' : '') . '">';
            foreach ($buttons as $button) {
                $return .= $button->render() . "\n";
            }
            $return .= '</div>';
            $return .= '</div>';
        }

        return $return;
    }

    /**
     * @param $label
     * @param bool $required
     * @param bool $error
     * @return string
     */
    public function renderLabel($label, $required = false, $error = false)
    {
        if (is_object($label)) {
            $element = $label;
            $label = $element->getLabel();
            $required = $element->isRequired();
            $error = $element->isError();
        }

        $return = '<label class="col-form-label ' . ($this->getForm()->hasClass('form-horizontal') ? ' col-sm-3' : '') . ($error ? '' : '') . '">';
        $return .= $label . ':';

        if ($required) {
            $return .= '<span class="required">*</span>';
        }

        $return .= "</label>";

        return $return;
    }


    /**
     * @inheritDoc
     */
    protected function getLabelClassesForElement($element)
    {
        $classes = parent::getLabelClassesForElement($element);
        $classes[] = 'col-form-label';
        if ($this->getForm()->hasClass('form-horizontal')) {
            $classes[] = 'col-sm-3';
        }
        return $classes;
    }
}


1			<?php
2
3			/**
4			* Class Bootstrap4
5			*/
6			class Nip_Form_Renderer_Bootstrap5 extends Nip_Form_Renderer_Bootstrap
7			{
8			/**
9			* @param Nip_Form_Element_Abstract $element
10			* @return string
11			*/
12			public function renderRow($element)
13			{
14			$return = '';
15			if (!$element->isRendered()) {
16			if ($element->hasCustomRenderer()) {
17			return $element->render();
18			}
19
20			$formType = $this->getForm()->hasClass('form-horizontal') ? 'form-horizontal' : '';
21
22			$rowClass = 'row';
23			$rowClass .= ' row-' . $element->getUniqueId();
24			$rowClass .= $element->isError() ? ' has-error' : '';
25			$return .= '<div class="' . $rowClass . '">';
26
27			$renderLabel = $element->getOption('render_label');
			0 ignored issues – show Bug introduced 2019-02-18 13:03 UTC by Report Bug Copy Issue Report Are you sure the assignment to `$renderLabel` is correct as `$element->getOption('render_label')` targeting `Nip\Form\Elements\AbstractElement::getOption()` seems to always return null. This check looks for function or method calls that always return null and whose return value is assigned to a variable. class A { function getObject() { return null; } } $a = new A(); $object = $a->getObject(); The method `getObject()` can return nothing but null, so it makes no sense to assign that value to a variable. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
28			if ($renderLabel !== false) {
			0 ignored issues – show introduced 2019-02-18 13:03 UTC by Report Bug Copy Issue Report The condition `$renderLabel !== false` is always `true`. Loading history...
29			$return .= $this->renderLabel($element);
30			}
31
32			$class = '';
33			if ($formType == 'form-horizontal') {
34			$class = $element->getType() == 'checkbox' ? 'col-sm-offset-3 col-sm-9' : 'col-sm-9';
35			}
36
37			$return .= '<div class="' . $class . '">';
38			$return .= $this->renderElement($element);
39
40			$helpBlock = $element->getOption('form-help');
41			if ($helpBlock) {
42			$return .= '<span class="help-block">' . $helpBlock . '</span>';
43			}
44
45			$return .= $element->renderErrors();
46			$return .= '</div>';
47			$return .= '</div>';
48			}
49
50			return $return;
51			}
52
53			/**
54			* @return string
55			*/
56			public function renderButtons()
57			{
58			$return = '';
59			$buttons = $this->getForm()->getButtons();
60
61			if ($buttons) {
			0 ignored issues – show Bug Best Practice introduced 2021-03-19 16:26 UTC by Report Bug Copy Issue Report The expression `$buttons` of type `Nip\Form\Buttons\AbstractButton[]` is implicitly converted to a boolean; are you sure this is intended? If so, consider using `! empty($expr)` instead to make it clear that you intend to check for an array without elements. This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using `empty(..)` or `! empty(...)` instead. Loading history...
62			$return .= '<div class="row">
63			<div class="' . ($this->getForm()->hasClass('form-horizontal') ? 'col-sm-offset-3 col-sm-9' : '') . '">';
64			foreach ($buttons as $button) {
65			$return .= $button->render() . "\n";
66			}
67			$return .= '</div>';
68			$return .= '</div>';
69			}
70
71			return $return;
72			}
73
74			/**
75			* @param $label
76			* @param bool $required
77			* @param bool $error
78			* @return string
79			*/
80			public function renderLabel($label, $required = false, $error = false)
81			{
82			if (is_object($label)) {
83			$element = $label;
84			$label = $element->getLabel();
85			$required = $element->isRequired();
86			$error = $element->isError();
87			}
88
89			$return = '<label class="col-form-label ' . ($this->getForm()->hasClass('form-horizontal') ? ' col-sm-3' : '') . ($error ? '' : '') . '">';
90			$return .= $label . ':';
91
92			if ($required) {
93			$return .= '<span class="required">*</span>';
94			}
95
96			$return .= "</label>";
97
98			return $return;
99			}
100
101
102			/**
103			* @inheritDoc
104			*/
105			protected function getLabelClassesForElement($element)
106			{
107			$classes = parent::getLabelClassesForElement($element);
108			$classes[] = 'col-form-label';
109			if ($this->getForm()->hasClass('form-horizontal')) {
110			$classes[] = 'col-sm-3';
111			}
112			return $classes;
113			}
114			}
115

bytic / form

Push — master ( db3995...c553d3 )

Nip_Form_Renderer_Bootstrap5::renderRow() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like