DatePicker::getElementJS() - Code Metrics - Inspection of "Merge branch 'master' of github.com:chamilo/chamil..." - chamilo/chamilo-lms - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( ad6b2a...ec8fd5 )

by Julito

created 2021-04-20 15:06 UTC

DatePicker::getElementJS() A

↳ Parent: DatePicker

Complexity

Conditions	1
Paths	1

Size

Total Lines	62
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	18
nc	1
nop	0
dl	0
loc	62
rs	9.6666
c	0
b	0
f	0

How to fix Long Method

<?php

/* For licensing terms, see /license.txt */

/**
 * Form element to select a date.
 *
 * Class DatePicker
 */
class DatePicker extends HTML_QuickForm_text
{
    /**
     * @param string       $elementName
     * @param string|array $elementLabel
     * @param array        $attributes
     */
    public function __construct($elementName, $elementLabel = null, $attributes = null)
    {
        if (!isset($attributes['id'])) {
            $attributes['id'] = $elementName;
        }
        $attributes['class'] = 'form-control';

        parent::__construct($elementName, $elementLabel, $attributes);
        $this->_appendName = true;
    }

    /**
     * HTML code to display this datepicker.
     *
     * @return string
     */
    public function toHtml()
    {
        if ($this->_flagFrozen) {
            return $this->getFrozenHtml();
        }

        $id = $this->getAttribute('id');
        $value = $this->getValue();

        if (!empty($value)) {
            $value = api_format_date($value, DATE_FORMAT_LONG_NO_DAY);
        }

        return '
            <div id="'.$id.'" class="input-group mb-3">
                <input '.$this->_getAttrString($this->_attributes).'
                    class="form-control" type="text" value="'.$value.'" data-input>
                <div class="input-group-prepend" id="button-addon3">
                    <button class="btn btn-outline-secondary"  type="button" data-toggle>
                        <i class="fas fa-calendar-alt"></i>
                    </button>
                    <button class="btn btn-outline-secondary" type="button" data-clear>
                        <i class="fas fa-times"></i>
                    </button>
              </div>
            </div>
        '.$this->getElementJS();
    }

    /**
     * @param string $value
     */
    public function setValue($value)
    {
        $value = substr($value, 0, 16);
        $this->updateAttributes(
            [
                'value' => $value,
            ]
        );
    }

    /**
     * Get the necessary javascript for this datepicker.
     *
     * @return string
     */
    private function getElementJS()
    {
        $js = null;
        $id = $this->getAttribute('id');
        //timeFormat: 'hh:mm'
        $js .= "<script>
            $(function() {
                var config = {
                    altInput: true,
                    altFormat: '".get_lang('F d, Y')."',
                    enableTime: false,
                    dateFormat: 'Y-m-d',
                    wrap: true
                };
                $('#{$id}').flatpickr(config);
             });
        </script>";

        return $js;

        $js .= "<script>
function fx() {
    try {
        doSomething();
        return true;
    }
    catch (\Exception $e) {
        return false;
    }

    return false;
}
            $(function() {
                var txtDate = $('#$id'),
                    inputGroup = txtDate.parents('.input-group'),
                    txtDateAlt = $('#{$id}_alt'),
                    txtDateAltText = $('#{$id}_alt_text');

                txtDate
                    .hide()
                    .datepicker({
                        defaultDate: '".$this->getValue()."',
                        dateFormat: 'yy-mm-dd',
                        altField: '#{$id}_alt',
                        altFormat: \"".get_lang('MM dd, yy')."\",
                        showOn: 'both',
                        buttonImage: '".Display::return_icon('attendance.png', null, [], ICON_SIZE_TINY, true, true)."',
                        buttonImageOnly: true,
                        buttonText: '".get_lang('Select date')."',
                        changeMonth: true,
                        changeYear: true,
                        yearRange: 'c-60y:c+5y'
                    })
                    .on('change', function (e) {
                        txtDateAltText.text(txtDateAlt.val());
                    });

                txtDateAltText.on('click', function () {
                    txtDate.datepicker('show');
                });

                inputGroup
                    .find('button')
                    .on('click', function (e) {
                        e.preventDefault();

                        $('#$id, #{$id}_alt').val('');
                        $('#{$id}_alt_text').html('');
                    });
            });
        </script>";

        return $js;
    }
}


1			<?php
2
3			/* For licensing terms, see /license.txt */
4
5			/**
6			* Form element to select a date.
7			*
8			* Class DatePicker
9			*/
10			class DatePicker extends HTML_QuickForm_text
11			{
12			/**
13			* @param string $elementName
14			* @param string\|array $elementLabel
15			* @param array $attributes
16			*/
17			public function __construct($elementName, $elementLabel = null, $attributes = null)
18			{
19			if (!isset($attributes['id'])) {
20			$attributes['id'] = $elementName;
21			}
22			$attributes['class'] = 'form-control';
23
24			parent::__construct($elementName, $elementLabel, $attributes);
25			$this->_appendName = true;
26			}
27
28			/**
29			* HTML code to display this datepicker.
30			*
31			* @return string
32			*/
33			public function toHtml()
34			{
35			if ($this->_flagFrozen) {
36			return $this->getFrozenHtml();
37			}
38
39			$id = $this->getAttribute('id');
40			$value = $this->getValue();
41
42			if (!empty($value)) {
43			$value = api_format_date($value, DATE_FORMAT_LONG_NO_DAY);
44			}
45
46			return '
47			<div id="'.$id.'" class="input-group mb-3">
48			<input '.$this->_getAttrString($this->_attributes).'
49			class="form-control" type="text" value="'.$value.'" data-input>
50			<div class="input-group-prepend" id="button-addon3">
51			<button class="btn btn-outline-secondary" type="button" data-toggle>
52			<i class="fas fa-calendar-alt"></i>
53			</button>
54			<button class="btn btn-outline-secondary" type="button" data-clear>
55			<i class="fas fa-times"></i>
56			</button>
57			</div>
58			</div>
59			'.$this->getElementJS();
60			}
61
62			/**
63			* @param string $value
64			*/
65			public function setValue($value)
66			{
67			$value = substr($value, 0, 16);
68			$this->updateAttributes(
69			[
70			'value' => $value,
71			]
72			);
73			}
74
75			/**
76			* Get the necessary javascript for this datepicker.
77			*
78			* @return string
79			*/
80			private function getElementJS()
81			{
82			$js = null;
83			$id = $this->getAttribute('id');
84			//timeFormat: 'hh:mm'
85			$js .= "<script>
86			$(function() {
87			var config = {
88			altInput: true,
89			altFormat: '".get_lang('F d, Y')."',
90			enableTime: false,
91			dateFormat: 'Y-m-d',
92			wrap: true
93			};
94			$('#{$id}').flatpickr(config);
95			});
96			</script>";
97
98			return $js;
99
100			$js .= "<script>
			0 ignored issues – show Unused Code introduced 2020-09-08 12:00 UTC by Report Bug Copy Issue Report `AssignConcatNode` is not reachable. This check looks for unreachable code. It uses sophisticated control flow analysis techniques to find statements which will never be executed. Unreachable code is most often the result of `return`, `die` or `exit` statements that have been added for debug purposes. function fx() { try { doSomething(); return true; } catch (\Exception $e) { return false; } return false; } In the above example, the last `return false` will never be executed, because a return statement has already been met in every possible execution path. Loading history...
101			$(function() {
102			var txtDate = $('#$id'),
103			inputGroup = txtDate.parents('.input-group'),
104			txtDateAlt = $('#{$id}_alt'),
105			txtDateAltText = $('#{$id}_alt_text');
106
107			txtDate
108			.hide()
109			.datepicker({
110			defaultDate: '".$this->getValue()."',
111			dateFormat: 'yy-mm-dd',
112			altField: '#{$id}_alt',
113			altFormat: \"".get_lang('MM dd, yy')."\",
114			showOn: 'both',
115			buttonImage: '".Display::return_icon('attendance.png', null, [], ICON_SIZE_TINY, true, true)."',
116			buttonImageOnly: true,
117			buttonText: '".get_lang('Select date')."',
118			changeMonth: true,
119			changeYear: true,
120			yearRange: 'c-60y:c+5y'
121			})
122			.on('change', function (e) {
123			txtDateAltText.text(txtDateAlt.val());
124			});
125
126			txtDateAltText.on('click', function () {
127			txtDate.datepicker('show');
128			});
129
130			inputGroup
131			.find('button')
132			.on('click', function (e) {
133			e.preventDefault();
134
135			$('#$id, #{$id}_alt').val('');
136			$('#{$id}_alt_text').html('');
137			});
138			});
139			</script>";
140
141			return $js;
142			}
143			}
144

chamilo / chamilo-lms

Push — master ( ad6b2a...ec8fd5 )

DatePicker::getElementJS() A

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like