assets/js/gaaf.js - Code Metrics - Inspection of "Merge pull request #84 from mdrahul/master" - AyeCode/invoicing - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 169dfd...f76276 )

by Stiofan

created 2017-08-21 17:06 UTC

assets/js/gaaf.js A

↳ Parent: Project

Complexity

Total Complexity	19
Complexity/F	3.17

Size

Lines of Code	107
Function Count	6

Duplication

Duplicated Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	2

Metric	Value
cc	0
wmc	19
c	2
b	0
f	2
nc	2
mnd	5
bc	15
fnc	6
dl	0
loc	107
rs	10
bpm	2.5
cpm	3.1666
noi	17

jQuery(function ($) {
    var autocomplete = new google.maps.places.Autocomplete(

        document.getElementById('wpinv_address'),
        {types: ['geocode']}
    );
    autocomplete.addListener('place_changed', fillInAddress);
    function fillInAddress() {
        var place = autocomplete.getPlace();
        var temp_state = '';
        
        for (var i = 0; i < place.address_components.length; i++) {

            var address_type = place.address_components[i].types[0];

            switch (address_type) {
                case 'route':
                    var val = place.address_components[i]['short_name'];
                    document.getElementById('wpinv_address').value = val;
                    break;
                case 'locality':
                    var val = place.address_components[i]['long_name'];

                    document.getElementById('wpinv_city').value = val;
                    break;
                case 'administrative_area_level_1':
                    var val = place.address_components[i]['short_name'];

                    temp_state = val;
                    break;
                case 'country':
                    var val = place.address_components[i]['short_name'];


                    $('#wpinv_country').val(val);

                    var elB = $('#wpinv-address');
                    var elF = $('#wpinv-fields');
                    data = {

                        action: 'wpinv_get_states_field',
                        country: val,
                        field_name: 'wpinv_state',
                    };

                    if(elB.length){
                        var $this = $('#wpinv_country', elB);

                        $this.closest('.gdmbx-row').find('.wpi-loader').show();
                        $('#wpinv_state', elB).css({
                            'opacity': '.5'
                        });
                        $.post(ajaxurl, data, function(response) {

                            if ('nostates' === response || '' == temp_state) {

                                var text_field = '<input type="text" value="' + temp_state + '" id="wpinv_state" name="wpinv_state" />';
                                $('#wpinv_state', elB).replaceWith(text_field);

                            } else {
                                $('#wpinv_state', elB).replaceWith(response);
                                $('#wpinv_state', elB).find('option[value="' + temp_state + '"]').attr('selected', 'selected');
                                $('#wpinv_state', elB).find('option[value=""]').remove();
                            }

                            $('#wpinv_state', elB).addClass('gdmbx2-text-large');
                            if (typeof $this.data('change') === '1') {

                                $('#wpinv_state', elB).change();
                            } else {
                                window.wpiConfirmed = true;
                                $('#wpinv-recalc-totals').click();
                                window.wpiConfirmed = false;
                            }

                            $this.closest('.gdmbx-row').find('.wpi-loader').hide();
                            $('#wpinv_state', elB).css({
                                'opacity': '1'
                            });
                        });
                    } else if(elF.length){
                        $('.wpinv_errors').remove();
                        wpinvBlock(jQuery('#wpinv_state_box'));
                        var $this = $('#wpinv_country', elF);

                        $.post(ajaxurl, data, function(response) {

                            if ('nostates' === response) {
                                var text_field = '<input type="text" required="required" class="wpi-input required" id="wpinv_state" name="wpinv_state">';
                                $('#wpinv_state', elF).replaceWith(text_field);

                            } else {
                                $('#wpinv_state', elF).replaceWith(response);
                                $('#wpinv_state', elF).find('option[value="' + temp_state + '"]').attr('selected', 'selected');

                                var changeState = function() {
                                    wpinv_recalculate_taxes(temp_state);

                                };
                                $("#wpinv_state", elF).unbind("change", changeState);

                                $("#wpinv_state", elF).bind("change", changeState);
                            }
                            $('#wpinv_state', elF).find('option[value=""]').remove();
                            $('#wpinv_state', elF).addClass('form-control wpi-input required');
                        }).done(function(data) {

                            jQuery('#wpinv_state_box').unblock();
                            wpinv_recalculate_taxes();
                        });
                    }

                    break;
                case 'postal_code':
                    var val = place.address_components[i]['short_name'];

                    document.getElementById('wpinv_zip').value = val;
                    break;
                default:
                    break;
            }
        }
    }
});

1			jQuery(function ($) {
2			var autocomplete = new google.maps.places.Autocomplete(
			0 ignored issues – show Bug introduced 2017-08-19 14:16 UTC by Report Bug Copy Issue Report The variable `google` seems to be never declared. If this is a global, consider adding a `/** global: google */` comment. This checks looks for references to variables that have not been declared. This is most likey a typographical error or a variable has been renamed. To learn more about declaring variables in Javascript, see the MDN. Loading history...
3			document.getElementById('wpinv_address'),
4			{types: ['geocode']}
5			);
6			autocomplete.addListener('place_changed', fillInAddress);
7			function fillInAddress() {
8			var place = autocomplete.getPlace();
9			var temp_state = '';
10
11			for (var i = 0; i < place.address_components.length; i++) {
12
13			var address_type = place.address_components[i].types[0];
14
15			switch (address_type) {
16			case 'route':
17			var val = place.address_components[i]['short_name'];
18			document.getElementById('wpinv_address').value = val;
19			break;
20			case 'locality':
21			var val = place.address_components[i]['long_name'];
			0 ignored issues – show Comprehensibility Naming Best Practice introduced 2017-08-19 14:16 UTC by Report Bug Copy Issue Report The variable `val` already seems to be declared on line `17`. Consider using another variable name or omitting the `var` keyword. This check looks for variables that are declared in multiple lines. There may be several reasons for this. In the simplest case the variable name was reused by mistake. This may lead to very hard to locate bugs. If you want to reuse a variable for another purpose, consider declaring it at or near the top of your function and just assigning to it subsequently so it is always declared. Loading history...
22			document.getElementById('wpinv_city').value = val;
23			break;
24			case 'administrative_area_level_1':
25			var val = place.address_components[i]['short_name'];
			0 ignored issues – show Comprehensibility Naming Best Practice introduced 2017-08-19 14:16 UTC by Report Bug Copy Issue Report The variable `val` already seems to be declared on line `17`. Consider using another variable name or omitting the `var` keyword. This check looks for variables that are declared in multiple lines. There may be several reasons for this. In the simplest case the variable name was reused by mistake. This may lead to very hard to locate bugs. If you want to reuse a variable for another purpose, consider declaring it at or near the top of your function and just assigning to it subsequently so it is always declared. Loading history...
26			temp_state = val;
27			break;
28			case 'country':
29			var val = place.address_components[i]['short_name'];
			0 ignored issues – show Comprehensibility Naming Best Practice introduced 2017-08-19 14:16 UTC by Report Bug Copy Issue Report The variable `val` already seems to be declared on line `17`. Consider using another variable name or omitting the `var` keyword. This check looks for variables that are declared in multiple lines. There may be several reasons for this. In the simplest case the variable name was reused by mistake. This may lead to very hard to locate bugs. If you want to reuse a variable for another purpose, consider declaring it at or near the top of your function and just assigning to it subsequently so it is always declared. Loading history...
30
31			$('#wpinv_country').val(val);
32
33			var elB = $('#wpinv-address');
34			var elF = $('#wpinv-fields');
35			data = {
			0 ignored issues – show Bug introduced 2017-08-19 14:16 UTC by Report Bug Copy Issue Report The variable `data` seems to be never declared. Assigning variables without defining them first makes them global. If this was intended, consider making it explicit like using `window.data`. Loading history...
36			action: 'wpinv_get_states_field',
37			country: val,
38			field_name: 'wpinv_state',
39			};
40
41			if(elB.length){
42			var $this = $('#wpinv_country', elB);
43
44			$this.closest('.gdmbx-row').find('.wpi-loader').show();
45			$('#wpinv_state', elB).css({
46			'opacity': '.5'
47			});
48			$.post(ajaxurl, data, function(response) {
			0 ignored issues – show Bug introduced 2017-08-19 14:16 UTC by Report Bug Copy Issue Report The variable `ajaxurl` seems to be never declared. If this is a global, consider adding a `/** global: ajaxurl */` comment. This checks looks for references to variables that have not been declared. This is most likey a typographical error or a variable has been renamed. To learn more about declaring variables in Javascript, see the MDN. Loading history...
49			if ('nostates' === response \|\| '' == temp_state) {
			0 ignored issues – show Bug introduced 2017-08-19 14:16 UTC by Report Bug Copy Issue Report The variable `temp_state` is changed as part of the for loop for example by `val` on line `26`. Only the value of the last iteration will be visible in this function if it is called after the loop. Loading history...
50			var text_field = '<input type="text" value="' + temp_state + '" id="wpinv_state" name="wpinv_state" />';
51			$('#wpinv_state', elB).replaceWith(text_field);
			0 ignored issues – show Bug introduced 2017-08-21 09:38 UTC by Report Bug Copy Issue Report The variable `elB` is changed as part of the for loop for example by `$("#wpinv-address")` on line `33`. Only the value of the last iteration will be visible in this function if it is called after the loop. Loading history...
52			} else {
53			$('#wpinv_state', elB).replaceWith(response);
54			$('#wpinv_state', elB).find('option[value="' + temp_state + '"]').attr('selected', 'selected');
55			$('#wpinv_state', elB).find('option[value=""]').remove();
56			}
57
58			$('#wpinv_state', elB).addClass('gdmbx2-text-large');
59			if (typeof $this.data('change') === '1') {
			0 ignored issues – show Bug introduced 2017-08-21 09:38 UTC by Report Bug Copy Issue Report The variable `$this` is changed as part of the for loop for example by `$("#wpinv_country", elB)` on line `42`. Only the value of the last iteration will be visible in this function if it is called after the loop. Loading history...
60			$('#wpinv_state', elB).change();
61			} else {
62			window.wpiConfirmed = true;
63			$('#wpinv-recalc-totals').click();
64			window.wpiConfirmed = false;
65			}
66
67			$this.closest('.gdmbx-row').find('.wpi-loader').hide();
68			$('#wpinv_state', elB).css({
69			'opacity': '1'
70			});
71			});
72			} else if(elF.length){
73			$('.wpinv_errors').remove();
74			wpinvBlock(jQuery('#wpinv_state_box'));
75			var $this = $('#wpinv_country', elF);
			0 ignored issues – show Comprehensibility Naming Best Practice introduced 2017-08-21 09:38 UTC by Report Bug Copy Issue Report The variable `$this` already seems to be declared on line `42`. Consider using another variable name or omitting the `var` keyword. This check looks for variables that are declared in multiple lines. There may be several reasons for this. In the simplest case the variable name was reused by mistake. This may lead to very hard to locate bugs. If you want to reuse a variable for another purpose, consider declaring it at or near the top of your function and just assigning to it subsequently so it is always declared. Loading history...
76			$.post(ajaxurl, data, function(response) {
			0 ignored issues – show Bug introduced 2017-06-23 11:56 UTC by Report Bug Copy Issue Report The variable `ajaxurl` seems to be never declared. If this is a global, consider adding a `/** global: ajaxurl */` comment. This checks looks for references to variables that have not been declared. This is most likey a typographical error or a variable has been renamed. To learn more about declaring variables in Javascript, see the MDN. Loading history...
77			if ('nostates' === response) {
78			var text_field = '<input type="text" required="required" class="wpi-input required" id="wpinv_state" name="wpinv_state">';
79			$('#wpinv_state', elF).replaceWith(text_field);
			0 ignored issues – show Bug introduced 2017-08-21 09:38 UTC by Report Bug Copy Issue Report The variable `elF` is changed as part of the for loop for example by `$("#wpinv-fields")` on line `34`. Only the value of the last iteration will be visible in this function if it is called after the loop. Loading history...
80			} else {
81			$('#wpinv_state', elF).replaceWith(response);
82			$('#wpinv_state', elF).find('option[value="' + temp_state + '"]').attr('selected', 'selected');
			0 ignored issues – show Bug introduced 2017-08-21 09:38 UTC by Report Bug Copy Issue Report The variable `temp_state` is changed as part of the for loop for example by `val` on line `26`. Only the value of the last iteration will be visible in this function if it is called after the loop. Loading history...
83			var changeState = function() {
84			wpinv_recalculate_taxes(temp_state);
			0 ignored issues – show Bug introduced 2017-08-21 09:38 UTC by Report Bug Copy Issue Report The variable `temp_state` is changed as part of the for loop for example by `val` on line `26`. Only the value of the last iteration will be visible in this function if it is called after the loop. Loading history...
85			};
86			$("#wpinv_state", elF).unbind("change", changeState);
			0 ignored issues – show Bug introduced 2017-08-21 09:38 UTC by Report Bug Copy Issue Report The variable `elF` is changed as part of the for loop for example by `$("#wpinv-fields")` on line `34`. Only the value of the last iteration will be visible in this function if it is called after the loop. Loading history...
87			$("#wpinv_state", elF).bind("change", changeState);
88			}
89			$('#wpinv_state', elF).find('option[value=""]').remove();
90			$('#wpinv_state', elF).addClass('form-control wpi-input required');
91			}).done(function(data) {
			0 ignored issues – show Unused Code introduced 2017-06-23 11:56 UTC by Report Bug Copy Issue Report The parameter `data` is not used and could be removed. This check looks for parameters in functions that are not used in the function body and are not followed by other parameters which are used inside the function. Loading history...
92			jQuery('#wpinv_state_box').unblock();
93			wpinv_recalculate_taxes();
94			});
95			}
96
97			break;
98			case 'postal_code':
99			var val = place.address_components[i]['short_name'];
			0 ignored issues – show Comprehensibility Naming Best Practice introduced 2017-08-19 14:16 UTC by Report Bug Copy Issue Report The variable `val` already seems to be declared on line `17`. Consider using another variable name or omitting the `var` keyword. This check looks for variables that are declared in multiple lines. There may be several reasons for this. In the simplest case the variable name was reused by mistake. This may lead to very hard to locate bugs. If you want to reuse a variable for another purpose, consider declaring it at or near the top of your function and just assigning to it subsequently so it is always declared. Loading history...
100			document.getElementById('wpinv_zip').value = val;
101			break;
102			default:
103			break;
104			}
105			}
106			}
107			});

AyeCode / invoicing

Push — master ( 169dfd...f76276 )

assets/js/gaaf.js A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like