From 7438e20ee1a8e6771c71dfa5beb0901f89ea12a8 Mon Sep 17 00:00:00 2001
From: Andrei Kortunov <andrei.kortunov@yandex.ru>
Date: Sat, 17 Nov 2018 22:19:52 +0400
Subject: [PATCH] Fix shield displaying on inventory avatar (bug #4720)

---
 CHANGELOG.md                              |  1 +
 apps/openmw/mwrender/characterpreview.cpp | 27 +++++++++++++++++------
 2 files changed, 21 insertions(+), 7 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 2bb06874b..cbea8d324 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -6,6 +6,7 @@
     Bug #4701: PrisonMarker record is not hardcoded like other markers
     Bug #4714: Crash upon game load in the repair menu while the "Your repair failed!" message is active
     Bug #4715: "Cannot get class of an empty object" exception after pressing ESC in the dialogue mode
+    Bug #4720: Inventory avatar has shield with two-handed weapon during [un]equipping animation
     Feature #2229: Improve pathfinding AI
     Feature #3442: Default values for fallbacks from ini file
     Feature #4673: Weapon sheathing
diff --git a/apps/openmw/mwrender/characterpreview.cpp b/apps/openmw/mwrender/characterpreview.cpp
index 075d2e03c..ce6a27389 100644
--- a/apps/openmw/mwrender/characterpreview.cpp
+++ b/apps/openmw/mwrender/characterpreview.cpp
@@ -305,24 +305,37 @@ namespace MWRender
                    type == ESM::Weapon::LongBladeOneHand ||
                    type == ESM::Weapon::BluntOneHand ||
                    type == ESM::Weapon::AxeOneHand ||
-                   type == ESM::Weapon::MarksmanThrown ||
-                   type == ESM::Weapon::MarksmanCrossbow ||
-                   type == ESM::Weapon::MarksmanBow)
+                   type == ESM::Weapon::MarksmanThrown)
+                {
+                    groupname = "inventoryweapononehand";
+                }
+                else if(type == ESM::Weapon::MarksmanCrossbow ||
+                        type == ESM::Weapon::MarksmanBow)
+                {
                     groupname = "inventoryweapononehand";
+                    showCarriedLeft = false;
+                }
                 else if(type == ESM::Weapon::LongBladeTwoHand ||
                         type == ESM::Weapon::BluntTwoClose ||
                         type == ESM::Weapon::AxeTwoHand)
+                {
                     groupname = "inventoryweapontwohand";
+                    showCarriedLeft = false;
+                }
                 else if(type == ESM::Weapon::BluntTwoWide ||
                         type == ESM::Weapon::SpearTwoWide)
+                {
                     groupname = "inventoryweapontwowide";
+                    showCarriedLeft = false;
+                }
                 else
+                {
                     groupname = "inventoryhandtohand";
-
-                showCarriedLeft = (iter->getClass().canBeEquipped(*iter, mCharacter).first != 2);
+                    showCarriedLeft = false;
+                }
            }
-            else
-                groupname = "inventoryhandtohand";
+           else
+               groupname = "inventoryhandtohand";
         }
 
         mAnimation->showCarriedLeft(showCarriedLeft);