Validate.json - Code Metrics - Inspection of "Use mosquitto server" - vergissberlin/mqtt-resource - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — development ( 6972b1...c2ccf0 )

by André

created 2019-11-26 08:04 UTC

Validate.json A

↳ Parent: Validate

Complexity

Conditions

Size

Total Lines	6
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	4
dl	0
loc	6
rs	10
c	0
b	0
f	0
cc	2

class Validate {

	url (string) {
		/* eslint no-useless-escape: 0 */
		let regex = /(mqtt|mqtts|tcp|tls|ws|wss):\/\/(\w+:{0,1}\w*)?(\S+)(:[0-9]+)?(\/|\/([\w#!:.?+=&%!\-\/]))?/
		var pattern = new RegExp(regex)
		return pattern.test(string)
	}

	json(json)	{
		if (typeof json !== 'object' ) {
			return false
		}
		return true
	}

	sourceConfiguration (input, callback) {
		let error = null

		if (!input.source.url) {
			if (!error) {
				error = new Error('URL for MQTT broker has not been set.')
			}
		}

		if (!this.url(input.source.url)) {
			if (!error) {
				error = new Error('Your defined URL is invalid. It should start with mqtt://')
			}
		}

		let port = Number.parseInt(input.source.port)
		if (Number.isNaN(port)) {
			input.source.port = 1883
		} else {
			input.source.port = port
		}

		callback(input, error)
	}

	paramConfiguration (input, callback) {
		let error = null

		if (!input.hasOwnProperty('params')) {
			if (!error) {
				return error = new Error('The parameters are not been set.')

			}
		} else {
			if (!input.params.topic) {
				if (!error) {
					error = new Error('The parameter topic has not been set.')
				}
			}
			if (!input.params.payload) {
				if (!error) {
					error = new Error('The parameter payload has not been set.')
				}
			}

			if (![0, 1, 2].includes(input.params.qos)) {
				input.params.qos = 1
			} else {
				input.params.qos.toString()
			}
		}

		callback(input, error)
	}


}

module.exports = new Validate


1			class Validate {
2
3			url (string) {
4			/* eslint no-useless-escape: 0 */
5			let regex = /(mqtt\|mqtts\|tcp\|tls\|ws\|wss):\/\/(\w+:{0,1}\w*)?(\S+)(:[0-9]+)?(\/\|\/([\w#!:.?+=&%!\-\/]))?/
6			var pattern = new RegExp(regex)
7			return pattern.test(string)
8			}
9
10			json(json) {
11			if (typeof json !== 'object' ) {
12			return false
13			}
14			return true
15			}
16
17			sourceConfiguration (input, callback) {
18			let error = null
19
20			if (!input.source.url) {
21			if (!error) {
22			error = new Error('URL for MQTT broker has not been set.')
23			}
24			}
25
26			if (!this.url(input.source.url)) {
27			if (!error) {
28			error = new Error('Your defined URL is invalid. It should start with mqtt://')
29			}
30			}
31
32			let port = Number.parseInt(input.source.port)
33			if (Number.isNaN(port)) {
34			input.source.port = 1883
35			} else {
36			input.source.port = port
37			}
38
39			callback(input, error)
40			}
41
42			paramConfiguration (input, callback) {
43			let error = null
44
45			if (!input.hasOwnProperty('params')) {
46			if (!error) {
47			return error = new Error('The parameters are not been set.')
			0 ignored issues – show Comprehensibility introduced 2019-07-03 22:37 UTC by Report Bug Copy Issue Report Are you sure you want to assign to `error` here, or did you intend to make a comparison like `error === new Error("The...ers are not been set.")`? Loading history... Unused Code introduced 2019-07-03 22:37 UTC by Report Bug Copy Issue Report The assignment to variable `error` seems to be never used. Consider removing it. Loading history...
48			}
49			} else {
50			if (!input.params.topic) {
51			if (!error) {
52			error = new Error('The parameter topic has not been set.')
53			}
54			}
55			if (!input.params.payload) {
56			if (!error) {
57			error = new Error('The parameter payload has not been set.')
58			}
59			}
60
61			if (![0, 1, 2].includes(input.params.qos)) {
62			input.params.qos = 1
63			} else {
64			input.params.qos.toString()
65			}
66			}
67
68			callback(input, error)
69			}
			0 ignored issues – show Best Practice introduced 2019-07-03 22:37 UTC by Report Bug Copy Issue Report There is no `return` statement in this branch, but you do return something in other branches. Did you maybe miss it? If you do not want to return anything, consider adding `return undefined;` explicitly. Loading history...
70
71			}
72
73			module.exports = new Validate
74

vergissberlin / mqtt-resource

Push — development ( 6972b1...c2ccf0 )

Validate.json A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like