#4125 closed enhancement (fixed)
[PATCH] Wooden Tower upgrade to Defense Tower
Reported by: | fatherbushido | Owned by: | elexis |
---|---|---|---|
Priority: | Nice to Have | Milestone: | Alpha 21 |
Component: | UI & Simulation | Keywords: | patch |
Cc: | Patch: |
Description
Attachments (2)
Change History (14)
by , 8 years ago
Attachment: | towerupgrade.diff added |
---|
comment:1 by , 8 years ago
comment:2 by , 8 years ago
Milestone: | Backlog → Alpha 21 |
---|
comment:3 by , 8 years ago
Keywords: | review removed |
---|
comment:4 by , 8 years ago
- There should be a more creative string than "Upgrade to a defense tower." (immersion). Perhaps "Enforce with stone and upgrade to a defense tower."?
- There should be either a link to some forum thread or a ticket description somewhere here.
- I agree on the idea. Defensive structures have been improved throughout the decades, so this upgrade could express that.
- My only concern is that this upgrade might incentivize building many towers more and thus make the current turtle-issue worse. In particular it would reduce the window of opportunity for attacks in the first 10min significantly.
Thus I suggest so penalize this upgrade in comparison to avoiding the wooden tower + building a stone tower.
A wooden tower costs 100 wood. A regular stone tower costs 100 wood + 100 stone. Thus the upgrade should cost more than 100 stone IMO. Otherwise everyone will build towers earlier, even without urgency. It should be
cost(stone tower) < cost(wooden tower + upgrade) < cost(wooden tower + stone tower)
. So increasing the upgrade cost from your patch by50 wood
should solve that issue. That wood could represent materials that had to be broken up to be replaced. The upgrade time 110 (iber:160), wooden tower time 40, stone tower time 150 (iber:200) fulfil the equation if we replace "<" with "<=", which seems ok. If it's too harsh, we can correct it later. I'd rather have something underpowered than overpowered.
- Reducing the iberian stone tower costs by 50 might be noticeable in some games where wood is more abundant than stone, but that's not really an issue. So I agree with that proposal.
comment:5 by , 8 years ago
Correction: The upgrade time can't be really compared to the build time, since usually more than one unit builds a tower. So it's a significant penalty too. Hence I grant to reduce the upgrade time from 110 to 100 (iber: 160 to 120).
by , 8 years ago
Attachment: | towerupgrade_v2.diff added |
---|
comment:6 by , 8 years ago
Keywords: | review added |
---|
comment:7 by , 8 years ago
That sounds good. (And it will look nice if lordgood commits his new small tower models).
comment:8 by , 8 years ago
Keywords: | rfc added; review removed |
---|---|
Priority: | Should Have → Nice to Have |
Type: | defect → enhancement |
Or should we rather close it as won't fix since minDistance for wooden tower is 40, maxDistance for the stone tower 60?
comment:9 by , 8 years ago
Indeed. I m lazy to check if upgrade component check those requirement. If so, it's ok else we should perhaps make both 60 ?
comment:10 by , 8 years ago
Keywords: | rfc removed |
---|
BuildRestrictions only apply to building buildings. 60 distance seems more than fine since the wooden towers have 60m range too, thus can cover each other. Notice romans can't build wooden towers anymore as of r18024. Thanks for the review, you helped this release a lot!
Can be nice individual upgrade from outpost upgrade to wooden to watch tower. It's my idea. But upgrade individual.