From 842a76cfd0ef8965de92908b33b7809c3788e61f Mon Sep 17 00:00:00 2001
From: Doug Woos <doug_woos@brown.edu>
Date: Sun, 8 Aug 2021 11:24:36 -0700
Subject: [PATCH] Add a field to armor to ignore dexterity

The dexterity modifier, whether it's a bonus or a penalty, should be ignored
entirely when computing AC for heavy armor.
---
 dungeonsheets/armor.py | 6 ++----
 dungeonsheets/stats.py | 9 +++++----
 2 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/dungeonsheets/armor.py b/dungeonsheets/armor.py
index 443abfe..f4ec83f 100644
--- a/dungeonsheets/armor.py
+++ b/dungeonsheets/armor.py
@@ -77,6 +77,7 @@ class Armor:
     cost = "0 gp"
     base_armor_class = 10
     dexterity_mod_max = None
+    dexterity_applied = True
     strength_required = None
     stealth_disadvantage = False
     weight = 0  # In lbs
@@ -112,6 +113,7 @@ class MediumArmor(Armor):
 
 class HeavyArmor(Armor):
     name = "Heavy Armor"
+    dexterity_applied = False
 
 
 class PaddedArmor(LightArmor):
@@ -182,7 +184,6 @@ class RingMail(HeavyArmor):
     name = "Ring Mail"
     cost = "30 gp"
     base_armor_class = 14
-    dexterity_mod_max = 0
     stealth_disadvantage = True
     weight = 40
 
@@ -191,7 +192,6 @@ class ChainMail(HeavyArmor):
     name = "Chain Mail"
     cost = "75 gp"
     base_armor_class = 16
-    dexterity_mod_max = 0
     strength_required = 13
     stealth_disadvantage = True
     weight = 55
@@ -201,7 +201,6 @@ class SplintArmor(HeavyArmor):
     name = "Splint Armor"
     cost = "200 gp"
     base_armor_class = 17
-    dexterity_mod_max = 0
     strength_required = 15
     stealth_disadvantage = True
     weight = 60
@@ -211,7 +210,6 @@ class PlateMail(HeavyArmor):
     name = "Plate Mail"
     cost = "1,500 gp"
     base_armor_class = 18
-    dexterity_mod_max = 0
     strength_required = 15
     stealth_disadvantage = True
     weight = 65
diff --git a/dungeonsheets/stats.py b/dungeonsheets/stats.py
index 568553f..640eea8 100644
--- a/dungeonsheets/stats.py
+++ b/dungeonsheets/stats.py
@@ -194,10 +194,11 @@ class ArmorClass:
         armor = actor.armor or NoArmor()
         ac = armor.base_armor_class
         # calculate and apply modifiers
-        if armor.dexterity_mod_max is None:
-            ac += actor.dexterity.modifier
-        else:
-            ac += min(actor.dexterity.modifier, armor.dexterity_mod_max)
+        if armor.dexterity_applied:
+            if armor.dexterity_mod_max is None:
+                ac += actor.dexterity.modifier
+            else:
+                ac += min(actor.dexterity.modifier, armor.dexterity_mod_max)
         if actor.has_feature(NaturalArmor):
             ac = max(ac, 13 + actor.dexterity.modifier)
         shield = actor.shield or NoShield()