Opened 8 years ago

Last modified 5 years ago

#4030 new defect

Elephant garrisoning should be specified explicitly by the template

Reported by: elexis Owned by:
Priority: Should Have Milestone: Backlog
Component: Simulation Keywords:
Cc: Patch:

Description (last modified by elexis)

Archer and war elephant can only be garrisoned in the elephant stables, but the support worker elephant can be garrisoned nearly everywhere. Duplicate of #3611, but not reopening that.

It should be consistent:

  • Only elephant stables should allow garrisoning for now.
  • All elephant types (worker, archer, war, hero) can be garrisoned equally. If a building wants to allow garrisoning the support elephant, it should likely allow the other elephant types as well. If it wants to prohibit garrisoning of one type of elephant, it likely wants to prohibit garrisoning of all other elephant types as well. Therefore all GarrisonHolders need either Elephant or !Elephant.

(It's probably not feasible to only add !Elephant to some parent and remove the garrison list from all templates inheriting that.)

Change History (15)

comment:1 by fatherbushido, 8 years ago

Keywords: patch review added
Summary: Elephant garrisoning should be specified explicitly by the template[PATCH] Elephant garrisoning should be specified explicitly by the template

comment:2 by fatherbushido, 8 years ago

Milestone: BacklogAlpha 21

comment:3 by fatherbushido, 8 years ago

Keywords: rfc added; review removed

I move it out of review queue, to avoid issues like #4129, we should get rid of "Support" in garrison list.

Last edited 8 years ago by fatherbushido (previous) (diff)

comment:4 by Imarok, 8 years ago

I think ships should allow garrison too. Otherwise civs with elephants and without rams have a large disadvantage.

comment:5 by elexis, 8 years ago

Keywords: simple removed

It doesn't seem too improbable to transport elephants via those ships and it is indeed very important from the gameplay pov to be able to transport them on naval and semi-naval maps!

comment:6 by fatherbushido, 8 years ago

Indeed, that's why it was decided by the past to allow them to garrison medium and heavy warship.

Last edited 8 years ago by fatherbushido (previous) (diff)

comment:7 by elexis, 8 years ago

Milestone: Alpha 21Alpha 22

comment:8 by elexis, 7 years ago

Milestone: Alpha 22Work In Progress

Moving to the new WIP milestone.

comment:9 by fatherbushido, 7 years ago

Keywords: rfc removed

comment:10 by elexis, 7 years ago

Keywords: rfc added

comment:11 by elexis, 7 years ago

Owner: set to elexis

comment:12 by fatherbushido, 7 years ago

Keywords: patch removed
Summary: [PATCH] Elephant garrisoning should be specified explicitly by the templateElephant garrisoning should be specified explicitly by the template

comment:13 by elexis, 7 years ago

Keywords: rfc removed

Deleting a patch that was proposed for review? Really?

Also the ptolemian elephant hero doesn't have the VisualClass Elephant, so if one doesn't know in advance which hero is the elephant hero, one has to try n error.

comment:14 by elexis, 7 years ago

Description: modified (diff)
Milestone: Work In ProgressBacklog
Owner: elexis removed

comment:15 by Imarok, 5 years ago

Component: UI & SimulationSimulation

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

Note: See TracTickets for help on using tickets.