search

Galileo88 / Galileos-Planet-Pack

Galileo's Planet Pack
Other
41 stars 18 forks source link

Syntax error in MM patches for Strategia #57

Open Marandil opened 5 years ago

Marandil commented 5 years ago

File GPP/GPP_Configs/GPP_Strategia.cfg uses double :HAS selector for its last 3 patches, resulting in warnings in the new ModuleManager (and using only the first selector). If I understand this issue correctly, it can be easily fixed by changing two :HAS selectors into a single one with a logical and:

:HAS[~author[Galileo]]:HAS[#name[MoonProgram|PlanetaryProgram|GasGiantProgram]]

should be

:HAS[~author[Galileo]&#name[MoonProgram|PlanetaryProgram|GasGiantProgram]]

Same with the next 2 lines.

Galileo88 commented 5 years ago

Thanks for the heads up! I guess the syntax changed in MM because this used to work fine. Condensing it does seem the logical way to do it, and I’m a bit embarrassed that we didn’t do that to begin with 😄