Page MenuHomeWildfire Games

[gameplay] allow training barracks champions at any barracks
Needs ReviewPublic

Authored by Nescio on Jan 8 2020, 8:37 PM.

Details

Reviewers
None
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Summary

As pointed out by @Stan in D2529, it is unbalanced you can train champion cavalry only at some but not all barracks. This patch inserts the barracks champions in the production list of the shared parent, which means that players who've researched the "unlock_champion_units" technology can train their champions at any captured enemy barracks, regardless the civ who built it, thus improving balance.

Likewise, it's unnecessary to specify the "unlock_champion_units" technology in individual specific barracks rather than the generic parent, because the technology files specify the civs that can research it; cf. D1941/rP22329.

Furthermore, it moves mace_champion_infantry_a_barracks.xml to mace_champion_infantry_barracks.xml and deletes mace_champion_infantry_e_barracks.xml, so Macedon can train them at captured enemy barracks and enemies can train theirs at mace barracks.

Test Plan

Apply the patch, play-test a couple of games, agree this is an improvement.

Event Timeline

Nescio created this revision.Jan 8 2020, 8:37 PM
Vulcan added a comment.Jan 8 2020, 8:39 PM

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/979/display/redirect

Vulcan added a comment.Jan 8 2020, 8:41 PM

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/macos-differential/75/display/redirect

Vulcan added a comment.Jan 8 2020, 8:42 PM

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1497/display/redirect

Nescio updated this revision to Diff 11188.Jan 26 2020, 6:59 PM
Nescio edited the summary of this revision. (Show Details)

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1662/display/redirect

Nescio edited reviewers, added: Restricted Owners Package; removed: Restricted Owners Package.Wed, Mar 25, 6:27 PM
Nescio removed subscribers: ValihrAnt, bb.