[PATCH] Extend / Improve Trigger functionality

[Niek]

With my recent work on Triggerscripting, it appears to me that the Trigger workflow (includes Triggerhelper.js) could use some additional functionality to ease scripting.

This includes (but is not limited to): Pre-written functions for:

• Adding resources to players.
• Pushing GUI notifications.
• Supporting cinematics.
• Enable/disable training of certain templates.
• Changing unit stats.

Enhancing:

• GUI dialog to allow for distinguishing multiple windows (possibly by allowing to set a button id to which can by checked against in the trigger code).
• allowing for AI's and Triggerscripts to communicate with each other (not necessarily the AI code itself, but the ability to do so)

This list is not quite complete and more additions to the list would be appreciated.

[Niek]

A patch for the GUI notification

[sanderd17]

It doesn't allow completely versatile messages with internationalisation support, like those used here: ​http://trac.wildfiregames.com/browser/ps/trunk/binaries/data/mods/public/simulation/components/Player.js#L214

And when it would, I think the function would become too complicated.

So I think either the notification object should be passed directly, or the function should indeed be dumbed down to a certain level, but then it's debatable to what level.

[NiAypa]

Adding resources is already possible with:

var cmpPlayer = TriggerHelper.GetPlayerComponent(playerID);
cmpPlayer.AddResources({
	wood: 100,
	stone: 100,
	metal: 100,
	food: 100
});

And same is

cmpPlayer.SetDisabledTemplates({template1: true, template2: true}); 

for disabling templates

[leper]

Replying to NiAypa:

cmpPlayer.SetDisabledTemplates({template1: true, template2: true}); 

Note that this changed in r16444 and you should now use

cmpPlayer.SetDisabledTemplates(["template1", "template2"]);

[leper]

As already noted by sanderd17 the message change isn't really working.

[elexis]

Backlogging due to lack of progress.

[Imarok]

Move tickets to Simulation as UI & Simulation got some sub components.