Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ModuleManager error on TranquilityTemplates.cfg #8

Closed
wants to merge 2 commits into from

Conversation

Li0n-0
Copy link

@Li0n-0 Li0n-0 commented Jan 8, 2018

@Angel-125 I got a little error thrown by Module Manger on the last version of DSEV :

[ModuleManager] Error - pass specifier detected on an insert node (not a patch): WildBlueIndustries/DSEV/Templates/Common/TranquilityTemplates/TRANQ:FOR[DSEV]

You'll need to add a "@" so MM understand you modify, and not insert, a node. A few others "@" are also needed to edit the existing variable instead of duplicating them.

This pull request should take care of it. (I've tested it).

@Aelfhe1m
Copy link

Aelfhe1m commented Jan 8, 2018

Sorry, @Li0n-0 but I disagree. In this case there are multiple TRANQ nodes and the intent of the patch is to create a new one for the EL Survey Station.

Since a new node is being created with all fixed values it doesn't matter which pass it is processed on, the correct fix would be to drop the :FOR[DSEV] on line 158 but keep the patch otherwise the same.

@Li0n-0
Copy link
Author

Li0n-0 commented Jan 8, 2018

@Aelfhe1m thanks. I correct it, hopefully get it right this time.

@Angel-125
Copy link
Owner

@Aelfhe1m is correct, these are new templates that are added to the Tranquility module. @Li0n-0 thanks for the pull request, I've added it to DSEV. :)

@Angel-125 Angel-125 closed this Jan 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants