In D4740#207265, @Freagarach wrote:

@Mister any update on this? I can take over if you don't have the time. But we'll still need to know how you want to be contributed in https://trac.wildfiregames.com/browser/ps/trunk/binaries/data/mods/public/gui/credits/texts/programming.json?

Change to use cmpFoundation to get the currentTemplateName, with this, there is no need to use cmpTemplateManager.

In D4740#202201, @Freagarach wrote:

A bit more explanation seems in place.
You use now the template of the foundation (foundation|<sometemplatename>). However, Foundation.js has a property called finalTemplateName, which is the truth about what the foundation will become when finished. E.g. foundation|<sometemplatename> may become foundation|<someothertemplatename> when finished.
As such, when cmpFoundation is present, it ought to be used. Else one can use the current template name (no need for the includes then).

Let me know whether this sounds about right.

In D4740#201764, @Freagarach wrote:

I'll write you some tests when I have some more time on my hands. Feel free to do it yourself though.

In D4740#201774, @Freagarach wrote:

This is how we say a patch is WIP. :) (By prefixing the title.)
Regarding the "cannot repair structures not known", it means that once captured a barracks of the enemy, we cannot repair it. Be aware of that. :) (One can check around that, using the hint provided inline.)

In D4740#201764, @Freagarach wrote:

I'll write you some tests when I have some more time on my hands. Feel free to do it yourself though.

Fixed what suggested on the comments.

Modified the subString condition to be more specific about the regex.

I have made some changes to removing the 'warns' and this is correctly comparing the buildList with the entityName. I don't like how this looks yet because of the subString im doing, will need to fix that.

Thanks for your comments on this diff.
I'll continue working on it and will add that test plan as soon as i can reproduce it.