HomeWildfire Games

Rename Armour to Resistance and change the way it is processed.

Description

Rename Armour to Resistance and change the way it is processed.

  • Renames Armour-node to Resistance.
  • Support resistance against Capture.
  • Puts resistance against effects in separate nodes.
  • Some cleaning.

Differential Revision: D2229
Reviewed by: @bb (accepted), @wraitii.
Comments by: @Stan, @Nescio.

Event Timeline

Nescio added inline comments.Aug 27 2020, 12:38 PM
/ps/trunk/binaries/data/mods/public/simulation/data/auras/structures/kush_pyramids_military.json
6โ€“8

Aura modifications are supposed to be sorted alphabetically.

/ps/trunk/binaries/data/mods/public/simulation/data/technologies/unit_elite.json
6โ€“8

Technology modifications are supposed to be sorted alphabetically.

/ps/trunk/binaries/data/mods/public/simulation/templates/units/rome_champion_infantry_sword_gladiator.xml
3โ€“10

Template nodes are supposed to be sorted alphabetically.

Freagarach added inline comments.Aug 27 2020, 5:55 PM
/ps/trunk/binaries/data/mods/public/simulation/helpers/Attacking.js
199

Should be "ApplyStatus". (See D2808.)

Freagarach added inline comments.Aug 28 2020, 8:36 AM
/ps/trunk/binaries/data/mods/public/simulation/helpers/Attacking.js
199

D2977 Has the fix.

Given that Armo(u)r was renamed to Resistance/Damage Resistance in the GUI tooltip, it should probably be renamed in the aura descriptions and technology tooltips too, and the https://trac.wildfiregames.com/wiki/EnglishStyleGuide ought to be updated accordingly.

Freagarach marked 3 inline comments as done.EditedAug 28 2020, 10:09 AM

That is a good one! Do you have any proposition how to properly refer to that? I'll make a minimalistic diff to discuss it there. [EDIT]: D2978.

(I can't raise a concern on my own commit ^^ )

/ps/trunk/binaries/data/mods/public/simulation/helpers/Attacking.js
346

While this may technically correct, it breaks capturing invulnerable targets (e.g. relics),,, See D1268 for a proper fix.

bb raised a concern with this commit.Aug 28 2020, 9:16 PM
This commit now has outstanding concerns.Aug 28 2020, 9:16 PM
Silier raised a concern with this commit.Aug 28 2020, 10:40 PM
Silier added a subscriber: Silier.
Silier added inline comments.
/ps/trunk/binaries/data/mods/public/simulation/components/Health.js
190

you just removed safe guard of multiple killing

myou5e added a subscriber: myou5e.Aug 29 2020, 12:16 AM
Freagarach added inline comments.Aug 29 2020, 7:15 AM
/ps/trunk/binaries/data/mods/public/simulation/components/Health.js
190
Silier resigned from this commit.Aug 29 2020, 12:22 PM
Silier removed an auditor: Silier.
This commit no longer requires audit.Dec 28 2020, 10:47 AM