Details
- Reviewers
elexis Freagarach - Commits
- rP24437: Display header from rP21310 also for upgrades.
Agree the tooltip is better this way
Wonder why rP20046 changed this like this
Notice that similar code (training, construction etc.) uses this function
Diff Detail
- Repository
- rP 0 A.D. Public Repository
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Event Timeline
@elexis provided the hint why the line is changed: if (template.wallSet) will always be false and thus it is not necessary to go through that?
Don't see why one will never have upgradable wallsets. Yes wallsets are never realised as global entities. But even during placement, in theory, there could be upgrades (note that I really mean upgrades, not technologies or whatever). Not that I see a direct usecase (surely it is not implemented), but I could imagine some mod wanting to up the wallset on a player command to build gates in some segments directly, or any other modification allowed by upgrades.