Give Fortress a territory root.
And consequently reduce the territory radius by 20%.
Patch by: @Nescio
Differential Revision: D1762
Reviewed by: @borg-, @ValihrAnt
Comments by: @elexis
Description
Description
Details
Details
- Committed
Freagarach Dec 14 2020, 7:05 PM - Reviewer
- borg-
- Differential Revision
- D1762: [gameplay] give Fortress a territory root
- Parents
- rP24392: Fix JS incremental GC issue (destroyed realms are not collected).
- Branches
- Unknown
- Tags
- Build Status
Buildable 14362 Build 30336: Post-Commit Build Jenkins Build 30335: Post-Commit Build (macOS) Jenkins
Event Timeline
Comment Actions
You broke some trigger maps. JB for sure danubius probably too. You now can destroy your cc once you have a fortress and not get attacked anymore.
Comment Actions
But one could do that prior to this patch also, right? Just put one entity in a fortress and delete your CC?
Comment Actions
Except the ones which can be garrisoned and thus kept alive, but yeah, I do get the point.
Warrants a revert? Or try to fix trigger maps?
Comment Actions
Sounds like you could fix the map to handle territory roots instead of CCs / conquest critical? If it's not trivial / no time, probably best to revert, but if this has a simple solution, I wouldn't call for a revert here
Comment Actions
First of all, I don't really understand map scripts. But if this commit broke only a few maps, then it seems clear to me the problem is with those maps, not with the change per se, and the proper solution is editing the relevant map files, not reverting the patch.
Would introducing a TerritoryRoot class help?