This Trac instance is not used for development anymore!

We migrated our development workflow to git and Gitea.
To test the future redirection, replace trac by ariadne in the page URL.

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#4195 closed defect (fixed)

[PATCH] Some foundations have more armor than finished buildings

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

Description

r18045 removed the Armour component of all economic buildings, which must be a mistake (reading the commit message that increases the armor). It yields 10% armor for every damage type, while foundations have 10% hack and crush and 40% pierce armor. This violates the principle that arrows ought to be incapable of destroying buildings.

With 10% pierce armor, 20 archers can break economic buildings (including the market) easily.

Bug reported by WarBeast.

Attachments (1)

restore_half_armor_v1.patch (638 bytes ) - added by elexis 8 years ago.

Download all attachments as: .zip

Change History (4)

by elexis, 8 years ago

Attachment: restore_half_armor_v1.patch added

comment:1 by elexis, 8 years ago

Cc: scythetwirler fatherbushido added
Keywords: patch rfc added
Summary: Some foundations have more armor than finished buildings[PATCH] Some foundations have more armor than finished buildings

comment:2 by fatherbushido, 8 years ago

Owner: set to fatherbushido
Resolution: fixed
Status: newclosed

In 18716:

Restores armour entry in economic structure template wich was mistakenly removed in r18045. Slighty reduces stats compares to a20. Fixes #4195. Reported by Warbeast. Agreed by scythetwirler. Reviewed by elexis.

comment:3 by elexis, 8 years ago

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