Fix crash possibility if module: <name> is not defined and on mistake position: <position>

[bugsounet]

Fix #3442

[KristjanESPERANTO]

Just a thought: Wouldn't that be a thing that belongs in check_config.js?

[sdetweil]

yes, also.

[bugsounet]

check_config.js check only syntax but it does not check the contents of the file

should we make some other checks on it too ?

[sdetweil]

well, this one is the first actual content that is important..
altho position should be checked too.. at least valid if specified.. that is another crash..

the user that reporteds the split problem said they ran config:check but it was good..

[bugsounet]

So, we have to check module: <name>and position: <valid position> (if defined)

we check it on check_config.js and MM core ?

Personally, I think it's essential to apply the rules to both

[sdetweil]

I think so.. never crash..

[bugsounet]

ok, I will see to code it

[bugsounet]

Right, I will try to add needed rules with eslint in check_config.js ;)

I have never done this kind of test.
bugsounet, it's time to learn :)

[bugsounet]

module name error:

position error:

No error:

I'm not sure that i can integrate it into eslint

What do you think @sdetweil ?

[bugsounet]

Before approuve some new deps.
Thanks to review this :>