Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#3770 closed defect (fixed)

[PATCH] Ptolemies Mercenary templates

Reported by: fatherbushido Owned by: elexis
Priority: Must Have Milestone: Alpha 20
Component: UI & Simulation Keywords: patch
Cc: Patch:

Description

Some mercenary ptolemies are not in mercenary classes. So they are not affected by Ptolemy_1 hero aura. I join the edited templates.

Attachments (2)

patch_mercenary.diff​ (3.3 KB ) - added by fatherbushido 8 years ago.
patch_mercenary_v2.diff​ (22.0 KB ) - added by fatherbushido 8 years ago.
fix all mercenaries templates

Download all attachments as: .zip

Change History (7)

by fatherbushido, 8 years ago

Attachment: patch_mercenary.diff​ added

comment:1 by s0600204, 8 years ago

Keywords: simple patch added

To quote leper on #0ad-dev:

should probably be VisibleClasses

If we're having the Mercenary class in VisibleClasses, then it will also need to be moved in ptol_infantry_swordsman_merc_b.xml (Celt can probably stay where it is)

comment:2 by Palaxin, 8 years ago

Summary: [Patch] Ptolemies Mercenary templates → [PATCH] Ptolemies Mercenary templates

comment:3 by fatherbushido, 8 years ago

Keywords: review added; simple removed

by fatherbushido, 8 years ago

Attachment: patch_mercenary_v2.diff​ added

fix all mercenaries templates

comment:4 by elexis, 8 years ago

Owner: set to elexis
Resolution: → fixed
Status: new → closed

In 17782:

Add/move the Mercenary tag to VisibleClasses, so that all mercenaries are influenced by auras and techs.
Patch by fatherbushido, fixes #3770.

comment:5 by elexis, 8 years ago

Keywords: review removed

Thanks for the patch!

Note: See TracTickets for help on using tickets.