Switch URL based on value

sdetweil

@htilburgs hm… looks ok, I would use the developer window sources tab to debug the code and look at the variables… (mouse over name usage will show contents)

htilburgs

@sdetweil
In the console - sources it says:

Uncaught SyntaxError: Unexpected token 'if'

sdetweil

@htilburgs well, if the code breaks, then …

so, look at that line and see what is missing from before it

a quick way to find syntax errors

in the module folder, run the pre-compiler to check for errors

node -c modulename.js

(or any js file)

htilburgs

@sdetweil
Meanwhile I was doing this, and I found the error (stupid me)

	start: function () {
		Log.info("Starting module: " + this.name);
		requiresVersion: "2.9.0";	
			
		// Set locales

		if (this.config.P1_IP != null) {
			this.urlP1 = "http://" + this.config.P1_IP + "/api/v1/data/";
		} else {
			this.urlP1 = "https://dummyjson.com/c/7e24-36ab-48e0-a96d";
		}

		if (this.config.WM_IP != null) {
			this.urlWM = "http://" + this.config.WM_IP + "/api/v1/data/";
		} else {
			this.urlWM = "https://dummyjson.com/c/704a-9a96-4845-bc72";
		}
//		this.urlP1 = "http://" + this.config.P1_IP + "/api/v1/data/";
//		this.urlWM = "http://" + this.config.WM_IP + "/api/v1/data/";
		
    		this.MHW_P1 = [];	        // <-- Create empty MHW_P1 array
		this.MHW_WM = [];		// <-- Create empty MHW_WM array
		this.scheduleUpdate();       	// <-- When the module updates (see below)
	},

The line with "requiresVersion: “2.9.0”; " was ending with an “,” instead of “;”

without the “if” statement, I don’t get a failure with a “,”
So changed it now into “;” and everythins seems to work.
I continue testing.

@MMRIZE Thanks for the alternative way. I’m going to look into it.
So with the “promise” statement, it looks like I can eleminate the node_helper.js

sdetweil

@htilburgs said in Switch URL based on value:

The line with "requiresVersion: “2.9.0”; " was ending with an “,” instead of “;”

requiresVersion should be outside the start function… (and then needs a trailing comma

sdetweil

@htilburgs said in Switch URL based on value:

So with the “promise” statement, it looks like I can eleminate the node_helper.js

correct, now that electron (and nodejs) have fetch built in(since node 18), you don’t NEED to use node_helper to do it