UmbraSpaceIndustries / MKS

MKS/OKS Colonization Systems
Other
221 stars 151 forks source link

Level Up Crew & other EXP modifiers #1243

Open M3tal-Warrior opened 7 years ago

M3tal-Warrior commented 7 years ago

I got Strategia installed and have the Media Circus III strategy active, which gives each Kerbal planting a flag on another body additional EXP on recovery on Kerbin ('Media star from $BODY'). Although the 'Level Up Crew' option on my Minmus Base accounts for stock EXP gains (like 'Plant Flag on Minmus'), but ignores the Strategia modifier.

I think it would be nice if such additional modifiers are accounted for too, if that's possible.

Gnurfos commented 7 years ago

From what I can see, the problem could be in Strategia: ModuleExperienceManagment is a stock one, not from MKS. And the code in Strategia declaring the "media star" log (KerbalRoster.AddExperienceType) is wrapped inside less trickery than was (apparently) necessary for the equivalent in MKS.

I suspect if you try without MKS the problem will be the same (if you can find some part with ModuleExperienceManagment). Or you could compile your own Strategia with a similar fix.