js/circles.app.settings.js - Code Metrics - Inspection of "moving settings" - nextcloud/circles - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( b17e10...8056b4 )

by Maxence

created 2017-06-09 16:07 UTC

js/circles.app.settings.js A

↳ Parent: Project

Complexity

Total Complexity	2
Complexity/F	2

Size

Lines of Code	13
Function Count	1

Duplication

Duplicated Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
wmc	2
c	1
b	0
f	0
dl	0
loc	13
rs	10
cc	0
nc	1
mnd	1
bc	3
fnc	1
bpm	3
cpm	2
noi	1

1 Function

Rating	Name	Duplication	Size	Complexity
A	settings.displaySettings	0	9	2

/*
 * Circles - Bring cloud-users closer together.
 *
 * This file is licensed under the Affero General Public License version 3 or
 * later. See the COPYING file.
 *
 * @author Maxence Lange <[email protected]>
 * @copyright 2017
 * @license GNU AGPL version 3 or any later version
 *
 * This program is free software: you can redistribute it and/or modify
 * it under the terms of the GNU Affero General Public License as
 * published by the Free Software Foundation, either version 3 of the
 * License, or (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU Affero General Public License for more details.
 *
 * You should have received a copy of the GNU Affero General Public License
 * along with this program.  If not, see <http://www.gnu.org/licenses/>.
 *
 */

/** global: OC */
/** global: OCA */
/** global: Notyf */

/** global: define */
/** global: elements */

var settings = {

	displaySettings: function (display) {
		if (display) {
			elements.mainUIMembers.hide(define.animationSpeed);
			elements.settingsPanel.delay(define.animationSpeed).show(define.animationSpeed);
		} else {
			elements.settingsPanel.hide(define.animationSpeed);
			elements.mainUIMembers.delay(define.animationSpeed).show(define.animationSpeed);
		}
	}

}






1			/*
2			* Circles - Bring cloud-users closer together.
3			*
4			* This file is licensed under the Affero General Public License version 3 or
5			* later. See the COPYING file.
6			*
7			* @author Maxence Lange <[email protected]>
8			* @copyright 2017
9			* @license GNU AGPL version 3 or any later version
10			*
11			* This program is free software: you can redistribute it and/or modify
12			* it under the terms of the GNU Affero General Public License as
13			* published by the Free Software Foundation, either version 3 of the
14			* License, or (at your option) any later version.
15			*
16			* This program is distributed in the hope that it will be useful,
17			* but WITHOUT ANY WARRANTY; without even the implied warranty of
18			* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
19			* GNU Affero General Public License for more details.
20			*
21			* You should have received a copy of the GNU Affero General Public License
22			* along with this program. If not, see <http://www.gnu.org/licenses/>.
23			*
24			*/
25
26			/** global: OC */
27			/** global: OCA */
28			/** global: Notyf */
29
30			/** global: define */
31			/** global: elements */
32
33			var settings = {
34
35			displaySettings: function (display) {
36			if (display) {
37			elements.mainUIMembers.hide(define.animationSpeed);
38			elements.settingsPanel.delay(define.animationSpeed).show(define.animationSpeed);
39			} else {
40			elements.settingsPanel.hide(define.animationSpeed);
41			elements.mainUIMembers.delay(define.animationSpeed).show(define.animationSpeed);
42			}
43			}
44
45			}
			0 ignored issues – show Coding Style introduced 2017-06-09 16:10 UTC by Report Bug Copy Issue Report There should be a semicolon. Requirement of semicolons purely is a coding style issue since JavaScript has specific rules about semicolons which are followed by all browsers. Further Readings: An Open Letter to JavaScript Leaders Regarding Semicolons by Isaac Schlueter JavaScript Semicolon Insertion Loading history...
46
47
48
49

nextcloud / circles

Push — master ( b17e10...8056b4 )

js/circles.app.settings.js A

Complexity

Size

Duplication

Importance

1 Function

Duplication Side-by-Side

Filter issues like