Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#4011 closed defect (fixed)

[PATCH] New buildings enable players to train wrong units

Reported by: sanderd17 Owned by: fatherbushido
Priority: Release Blocker Milestone: Alpha 21
Component: UI & Simulation Keywords: patch
Cc: scythetwirler Patch:

Description (last modified by sanderd17)

There are several buildings that, when captured by a different civ, allow the production of some unexpected unit:

  • athen_dock: Shows buttons to train the marines, but other civs can never trian the marines due to a missing tech. This can confuse players.
  • athen_royal_stoa: for some reason, the athenians can't train mace_thorakites in their own stoa, but if they capture another stoa (mace or spart), they would be able to train those. Any other civ capturing the stoas would also be able to produce those units. The naming of the templates for the soldiers is also confusing. It's easy to think that mace_* units can only belong to the macedonian civ, but that's not the case here.
  • merc_camp_egyptian and ptol_merc_camp: both allow to train ptolemaic citizen-soldiers that give access to the complete ptolemaic building set (these are atlas-only structures).
  • brit_crannog: allows the capturer to train briton citizen-soldiers and briton ships, which in turn give access to the complete briton building set.
  • other/maur_palace: allows the capturer to train maurian heroes (an atlas-only structure).
  • other/cart_tophet: allows capturer to train cart champions and healers (atlas-only structure).

(templates found by fatherbushido)

Setting it as release blocker, as it would harm gameplay too much to be in a release.

Attachments (5)

civattribute.diff (13.4 KB ) - added by fatherbushido 8 years ago.
civattribute.2.diff (18.9 KB ) - added by fatherbushido 8 years ago.
forget to add some files
civattribute.3.diff (21.4 KB ) - added by fatherbushido 8 years ago.
forget more svn add
civattribute.4.diff (20.8 KB ) - added by fatherbushido 8 years ago.
let mace merc in mace stoa (atlas only) to reward capture and for scenario map
civattribute.5.diff (20.7 KB ) - added by fatherbushido 8 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 by sanderd17, 8 years ago

Description: modified (diff)

by fatherbushido, 8 years ago

Attachment: civattribute.diff added

comment:2 by fatherbushido, 8 years ago

Keywords: patch review added
Summary: New buildings enable players to train wrong units[PATCH] New buildings enable players to train wrong units

by fatherbushido, 8 years ago

Attachment: civattribute.2.diff added

forget to add some files

by fatherbushido, 8 years ago

Attachment: civattribute.3.diff added

forget more svn add

by fatherbushido, 8 years ago

Attachment: civattribute.4.diff added

let mace merc in mace stoa (atlas only) to reward capture and for scenario map

by fatherbushido, 8 years ago

Attachment: civattribute.5.diff added

comment:3 by fatherbushido, 8 years ago

Owner: set to fatherbushido
Resolution: fixed
Status: newclosed

In 18562:

Prevents training of unexpected units in buildings introduced in r18024, lets mace trains 2 mercs in captured stoa or in their own stoa for scenario map, fixes #4011.

comment:4 by fatherbushido, 8 years ago

Keywords: review removed
Note: See TracTickets for help on using tickets.