FormStatic::getHTML() - Code Metrics - Inspection of "shiftleft-analyses fix" - Stefanius67/Formgenerator - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — main ( 2e1a62...22ca80 )

by Stefan

created 2021-06-25 09:56 UTC

FormStatic::getHTML() D

↳ Parent: FormStatic

Complexity

Conditions	11
Paths	384

Size

Total Lines	43
Code Lines	32

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	32
dl	0
loc	43
rs	4.1833
c	1
b	0
f	0
cc	11
nc	384
nop	0

How to fix Complexity

<?php
declare(strict_types=1);

namespace SKien\Formgenerator;


/**
 * Static Text element.
 *
 * The FormStatic element can be used for different purposes:
 * - just for the normal output of a text
 * - as a label that is assigned to a specific input element
 *   - set the `$strLabelFor` param in the constructor or use the `setLabelFor()` method
 * - As a hint
 *   - set the `FormFlags::HINT` flag
 * - To display an error message
 *   - set the `FormFlags::ERROR` flag
 * - As an information field
 *   - set the `FormFlags::INFO` flag
 *
 * @SKienImage FormStatic.png
 *
 * @package Formgenerator
 * @author Stefanius <[email protected]>
 * @copyright MIT License - see the LICENSE file for details
 */
class FormStatic extends FormElement
{
    /** @var string the text to display     */
    protected string $strText;
    /** @var string the if set, label with for attribute is created     */
    protected string $strLabelFor = '';

    /**
     * Create a static text element.
     * @param string $strText       text to display
     * @param int $wFlags           any combination of FormFlag constants
     * @param string $strLabelFor   crate label for this element
     */
    public function __construct(string $strText, int $wFlags = 0, string $strLabelFor = '')
    {
        $this->strText = $strText;
        $this->strLabelFor = $strLabelFor;
        parent::__construct($wFlags);
    }

    /**
     * {@inheritDoc}
     * @see \SKien\Formgenerator\FormElement::fromXML()
     * @internal
     */
    static public function fromXML(\DOMElement $oXMLElement, FormCollection $oFormParent) : ?FormElement
    {
        $strText = self::getAttribString($oXMLElement, 'text', '');
        $wFlags = self::getAttribFlags($oXMLElement);
        $strLabelFor = self::getAttribString($oXMLElement, 'for', '');
        $oFormElement = new self($strText, $wFlags, $strLabelFor);
        $oFormParent->add($oFormElement);
        $oFormElement->readAdditionalXML($oXMLElement);

        return $oFormElement;
    }

    /**
     * Set the control for which a label should be created.
     * @param string $strLabelFor
     */
    public function setLabelFor(string $strLabelFor) : void
    {
        $this->strLabelFor = $strLabelFor;
    }

    /**
     * Build the HTML-notation for the static text
     * @return string
     * @internal
     */
    public function getHTML() : string
    {
        // no container div!
        $this->addStyle('float', 'left');
        $strWidth = ($this->oParent ? $this->oParent->getColWidth($this->iCol) : '');
        if (!empty($strWidth)) {
            $this->addStyle('width', $strWidth);
        }
        if ($this->oFlags->isSet(FormFlags::ALIGN_RIGHT)) {
            $this->addStyle('text-align', 'right');
        } else if ($this->oFlags->isSet(FormFlags::ALIGN_CENTER)) {
            $this->addStyle('text-align', 'center');
        }
        if ($this->oFlags->isSet(FormFlags::BOLD)) {
            $this->addStyle('font-weight', 'bold');
        }
        $strSep = '';
        if (strlen($this->strClass) !== 0) {
            $strSep = ' ';
        }
        if ($this->oFlags->isSet(FormFlags::ERROR)) {
            $this->strClass .= $strSep . 'error';
        } else if ($this->oFlags->isSet(FormFlags::HINT)) {
            $this->strClass = $strSep . 'hint';
        } else if ($this->oFlags->isSet(FormFlags::INFO)) {
            $this->strClass = $strSep . 'forminfo';
        }

        $strHTML  = '';
        $strHTML .= '<div';
        $strHTML .= $this->buildID();
        $strHTML .= $this->buildClass();
        $strHTML .= $this->buildStyle();
        $strHTML .= $this->buildAttributes();
        $strHTML .= '>';
        if (strlen($this->strLabelFor) > 0 ) {
            $strHTML .= '<label for="' . $this->strLabelFor . '">' . $this->strText . '</label>';
        } else {
            $strHTML .= $this->strText;
        }
        $strHTML .= '</div>' . PHP_EOL;

        return $strHTML;
    }
}


1			<?php
2			declare(strict_types=1);
3
4			namespace SKien\Formgenerator;
5
6
7			/**
8			* Static Text element.
9			*
10			* The FormStatic element can be used for different purposes:
11			* - just for the normal output of a text
12			* - as a label that is assigned to a specific input element
13			* - set the `$strLabelFor` param in the constructor or use the `setLabelFor()` method
14			* - As a hint
15			* - set the `FormFlags::HINT` flag
16			* - To display an error message
17			* - set the `FormFlags::ERROR` flag
18			* - As an information field
19			* - set the `FormFlags::INFO` flag
20			*
21			* @SKienImage FormStatic.png
22			*
23			* @package Formgenerator
24			* @author Stefanius <[email protected]>
25			* @copyright MIT License - see the LICENSE file for details
26			*/
27			class FormStatic extends FormElement
28			{
29			/** @var string the text to display */
30			protected string $strText;
31			/** @var string the if set, label with for attribute is created */
32			protected string $strLabelFor = '';
33
34			/**
35			* Create a static text element.
36			* @param string $strText text to display
37			* @param int $wFlags any combination of FormFlag constants
38			* @param string $strLabelFor crate label for this element
39			*/
40			public function __construct(string $strText, int $wFlags = 0, string $strLabelFor = '')
41			{
42			$this->strText = $strText;
43			$this->strLabelFor = $strLabelFor;
44			parent::__construct($wFlags);
45			}
46
47			/**
48			* {@inheritDoc}
49			* @see \SKien\Formgenerator\FormElement::fromXML()
50			* @internal
51			*/
52			static public function fromXML(\DOMElement $oXMLElement, FormCollection $oFormParent) : ?FormElement
53			{
54			$strText = self::getAttribString($oXMLElement, 'text', '');
55			$wFlags = self::getAttribFlags($oXMLElement);
56			$strLabelFor = self::getAttribString($oXMLElement, 'for', '');
57			$oFormElement = new self($strText, $wFlags, $strLabelFor);
58			$oFormParent->add($oFormElement);
59			$oFormElement->readAdditionalXML($oXMLElement);
60
61			return $oFormElement;
62			}
63
64			/**
65			* Set the control for which a label should be created.
66			* @param string $strLabelFor
67			*/
68			public function setLabelFor(string $strLabelFor) : void
69			{
70			$this->strLabelFor = $strLabelFor;
71			}
72
73			/**
74			* Build the HTML-notation for the static text
75			* @return string
76			* @internal
77			*/
78			public function getHTML() : string
79			{
80			// no container div!
81			$this->addStyle('float', 'left');
82			$strWidth = ($this->oParent ? $this->oParent->getColWidth($this->iCol) : '');
83			if (!empty($strWidth)) {
84			$this->addStyle('width', $strWidth);
85			}
86			if ($this->oFlags->isSet(FormFlags::ALIGN_RIGHT)) {
87			$this->addStyle('text-align', 'right');
88			} else if ($this->oFlags->isSet(FormFlags::ALIGN_CENTER)) {
89			$this->addStyle('text-align', 'center');
90			}
91			if ($this->oFlags->isSet(FormFlags::BOLD)) {
92			$this->addStyle('font-weight', 'bold');
93			}
94			$strSep = '';
95			if (strlen($this->strClass) !== 0) {
96			$strSep = ' ';
97			}
98			if ($this->oFlags->isSet(FormFlags::ERROR)) {
99			$this->strClass .= $strSep . 'error';
100			} else if ($this->oFlags->isSet(FormFlags::HINT)) {
101			$this->strClass = $strSep . 'hint';
102			} else if ($this->oFlags->isSet(FormFlags::INFO)) {
103			$this->strClass = $strSep . 'forminfo';
104			}
105
106			$strHTML = '';
107			$strHTML .= '<div';
108			$strHTML .= $this->buildID();
109			$strHTML .= $this->buildClass();
110			$strHTML .= $this->buildStyle();
111			$strHTML .= $this->buildAttributes();
112			$strHTML .= '>';
113			if (strlen($this->strLabelFor) > 0 ) {
114			$strHTML .= '<label for="' . $this->strLabelFor . '">' . $this->strText . '</label>';
115			} else {
116			$strHTML .= $this->strText;
117			}
118			$strHTML .= '</div>' . PHP_EOL;
119
120			return $strHTML;
121			}
122			}
123

Stefanius67 / Formgenerator

Push — main ( 2e1a62...22ca80 )

FormStatic::getHTML() D

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like