2.14 regression, not installed module produces no error

[sdetweil]

I was helping user with MMM-WiFiPassword, it just displays a qr code

he had misspelled the name

to recreate
add this to your config.js

        {
            module: "MMM-WiFiPassword1",
            position: "center",
              config: {
              //See 'Configuration options' for more information.
              network: "xxxxx",
              password: "xxxxxx",
	            showAuthType: false,
	            showNetwork: false,
	            showPassword: false,
	            debug: false
        		}
        }

i got
No helper found for module: MMM-WiFiPassword1.

but not failure, because the module was missing

[rejas]

What was the behaviour pre-2.14 for this case?

[sdetweil]

magicMirror could not validate the config file
will not stop.....
(i think) have to go test

[MichMich]

I think this behavior did not change.

[sdetweil]

does not fail on 2.11...
should fail

[rejas]

Tried as far as v2.7.0 (below wouldnt start anymore on my Ubuntu). No warnings or error besides the " No helper found for module: MMM-WiFiPassword1. "

So not sure when / or if it was changed. But maybe something worth discussing: Shouldnt there be a more obvious warnin/error log that you have a config entry for a module that isnt there?

[sdetweil]

@rejas > Shouldnt there be a more obvious warning/error log that you have a config entry for a module that isnt there?

thats my opinion... users coming from windows make mistakes on letter casing and nothing works.. but they don't know why..

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[rejas]

Re-opening since this might be useful feature (just ran into it myself)

[khassel]

This is the part of app.js

		let loadHelper = true;
		try {
			fs.accessSync(helperPath, fs.R_OK);
		} catch (e) {
			loadHelper = false;
			Log.log(`No helper found for module: ${moduleName}.`);
		}

It is correct to do only a log here because not every module has a node_helper.js.

We could test before this part e.g. if the directory exists (and may its not empty) and otherwise do a Log.error().

Would this solve this issue?
Other ideas?

[sdetweil]

directory and modulename js existing. not node_helper as optional