Nip_Form_Renderer_Elements_Dateselect

Complexity

Total Complexity

6

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	generateElement()	0	14	3
A	generateElement2()	0	32	3

<?php

class Nip_Form_Renderer_Elements_Dateselect extends Nip_Form_Renderer_Elements_MultiElement

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    public function generateElement()
    {
        $return = '<div class="row">';

        $elements = $this->getElement()->getElements();

The method getElements() does not exist on Nip_Form_Element_Abstract. It seems like you code against a sub-type of Nip_Form_Element_Abstract such as Nip_Form_Element_MultiElement or Nip_Form_Element_Input_Group.

        foreach ($elements as $key => $element) {
            $element->addClass('form-control');
            $returnElements[] = '<div class="col-xs-4" style="max-width:' . ($key == 'day' ? 95 : 110) . 'px;">' .
                $element->render() . '</div>';
        }

        $return .= implode(' ', $returnElements);

The variable $returnElements seems to be defined by a foreach iteration on line 10. Are you sure the iterator is never empty, otherwise this variable is not defined?

        $return .= '</div>';
        return $return;
    }

    public function generateElement2()
    {
        if (!$this->getElement()->getAttrib('id')) {

Are you sure the usage of $this->getElement()->getAttrib('id') targeting Nip\Form\Elements\AbstractElement::getAttrib() seems to always return null.

            $this->getElement()->setAttrib('id', $this->getElement()->getJSID());
            $this->getElement()->addClass('datepicker');
        }
        $return = parent::generateElement();
        $return .= '<script type="text/javascript">';
        $return .= 'document.addEventListener("DOMContentLoaded", function() {';

        $options = [];
        $options[] = 'changeMonth: true';
        $options[] = 'changeYear: true';

        $yearRange = $this->getElement()->getOption('yearRange');

Are you sure the assignment to $yearRange is correct as $this->getElement()->getOption('yearRange') targeting Nip\Form\Elements\AbstractElement::getOption() seems to always return null.

        if ($yearRange) {

$yearRange is of type null, thus it always evaluated to false.

            $options[] = 'yearRange: "' . $yearRange . '"';
        }
        $format = $this->getElement()->getFormat();

The method getFormat() does not exist on Nip_Form_Element_Abstract. Did you maybe mean getForm()?

        $format = strtr($format, [
            'Y' => 'yy',
            'd' => 'dd',
            'm' => 'mm',
        ]);
        $options[] = 'dateFormat: "' . $format . '"';

        $return .= "    jQuery('#{$this->getElement()->getAttrib('id')}').datepicker({

Are you sure the usage of $this->getElement()->getAttrib('id') targeting Nip\Form\Elements\AbstractElement::getAttrib() seems to always return null.

			" . implode(',', $options) . "
		});";
        $return .= '});';
        $return .= '</script>';
        return $return;
    }
}