_script.js ➔ ... ➔ $.each - Code Metrics - Inspection of "frontend user rbac" - yuncms/framework - Measure and Improve Code Quality continuously with Scrutinizer

Test Setup Failed

Push — master ( 95bd70...48e609 )

by Xu

created 2018-04-11 04:48 UTC

_script.js ➔ ... ➔ $.each A

↳ Parent: _script.js ➔ search

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	6
Ratio	100 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
cc	2
c	2
b	0
f	0
nc	2
nop	2
dl	6
loc	6
rs	9.4285

$('i.glyphicon-refresh-animate').hide();
function updateItems(r) {
    _opts.items.avaliable = r.avaliable;

    _opts.items.assigned = r.assigned;
    search('avaliable');
    search('assigned');
}

$('.btn-assign').click(function (e) {
    e.preventDefault();
    var $this = $(this);
    var target = $this.data('target');
    var items = $('select.list[data-target="' + target + '"]').val();

    if (items.length) {
        $this.children('i.glyphicon-refresh-animate').show();
        $.post($this.attr('href'), {items: items}, function (r) {
            updateItems(r);
        }).always(function () {
            $this.children('i.glyphicon-refresh-animate').hide();
        });
    }
    return false;
});

$('.search[data-target]').keyup(function () {
    search($(this).data('target'));
});

function search(target) {
    var $list = $('select.list[data-target="' + target + '"]');
    $list.html('');
    var q = $('.search[data-target="' + target + '"]').val();

    var groups = {
        role: [$('<optgroup label="Roles">'), false],
        permission: [$('<optgroup label="Permission">'), false],
        route: [$('<optgroup label="Routes">'), false],
    };
    $.each(_opts.items[target], function (name, group) {

        if (name.indexOf(q) >= 0) {
            $('<option>').text(name).val(name).appendTo(groups[group][0]);
            groups[group][1] = true;
        }
    });
    $.each(groups, function () {
        if (this[1]) {
            $list.append(this[0]);
        }
    });
}

// initial
search('avaliable');
search('assigned');


1	View Code Duplication	$('i.glyphicon-refresh-animate').hide();
2		function updateItems(r) {
3		_opts.items.avaliable = r.avaliable;
		0 ignored issues – show Bug introduced 2018-04-11 05:24 UTC by Report Bug Copy Issue Report The variable `_opts` seems to be never declared. If this is a global, consider adding a `/** global: _opts */` 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...
4		_opts.items.assigned = r.assigned;
5		search('avaliable');
6		search('assigned');
7		}
8
9		$('.btn-assign').click(function (e) {
10		e.preventDefault();
11		var $this = $(this);
12		var target = $this.data('target');
13		var items = $('select.list[data-target="' + target + '"]').val();
14
15		if (items.length) {
16		$this.children('i.glyphicon-refresh-animate').show();
17		$.post($this.attr('href'), {items: items}, function (r) {
18		updateItems(r);
19		}).always(function () {
20		$this.children('i.glyphicon-refresh-animate').hide();
21		});
22		}
23		return false;
24		});
25
26		$('.search[data-target]').keyup(function () {
27		search($(this).data('target'));
28		});
29
30		function search(target) {
31		var $list = $('select.list[data-target="' + target + '"]');
32		$list.html('');
33		var q = $('.search[data-target="' + target + '"]').val();
34
35		var groups = {
36		role: [$('<optgroup label="Roles">'), false],
37		permission: [$('<optgroup label="Permission">'), false],
38		route: [$('<optgroup label="Routes">'), false],
39		};
40		$.each(_opts.items[target], function (name, group) {
		0 ignored issues – show Bug introduced 2018-04-11 05:24 UTC by Report Bug Copy Issue Report The variable `_opts` seems to be never declared. If this is a global, consider adding a `/** global: _opts */` 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...
41		if (name.indexOf(q) >= 0) {
42		$('<option>').text(name).val(name).appendTo(groups[group][0]);
43		groups[group][1] = true;
44		}
45		});
46		$.each(groups, function () {
47		if (this[1]) {
48		$list.append(this[0]);
49		}
50		});
51		}
52
53		// initial
54		search('avaliable');
55		search('assigned');
56

yuncms / framework

Push — master ( 95bd70...48e609 )

_script.js ➔ ... ➔ $.each A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like