In DropShortcutContainer accept object with data from constructor.

3 files changed, 34 insertions, 26 deletions

diff --git a/src/game.cpp b/src/game.cpp
index ea006cefc..7c6dcca62 100644
--- a/src/game.cpp
+++ b/src/game.cpp
@@ -29,6 +29,7 @@
 #include "animatedsprite.h"
 #include "client.h"
 #include "configuration.h"
+#include "dropshortcut.h"
 #include "effectmanager.h"
 #include "emoteshortcut.h"
 #include "eventsmanager.h"
@@ -212,7 +213,7 @@ static void createGuiWindows()
         130, 480);
     outfitWindow = new OutfitWindow();
     dropShortcutWindow = new ShortcutWindow("DropShortcut",
-        new DropShortcutContainer(nullptr),
+        new DropShortcutContainer(nullptr, dropShortcut),
         "drops.xml");
 
     spellShortcutWindow = new ShortcutWindow("SpellShortcut", "spells.xml",
diff --git a/src/gui/widgets/dropshortcutcontainer.cpp b/src/gui/widgets/dropshortcutcontainer.cpp
index 6d3d259b0..417cc2df2 100644
--- a/src/gui/widgets/dropshortcutcontainer.cpp
+++ b/src/gui/widgets/dropshortcutcontainer.cpp
@@ -24,6 +24,7 @@
 
 #include "dragdrop.h"
 #include "dropshortcut.h"
+#include "shortcutbase.h"
 #include "settings.h"
 
 #include "being/playerinfo.h"
@@ -45,16 +46,18 @@
 
 #include "debug.h"
 
-DropShortcutContainer::DropShortcutContainer(Widget2 *const widget) :
+DropShortcutContainer::DropShortcutContainer(Widget2 *const widget,
+                                             ShortcutBase *const shortcut) :
     ShortcutContainer(widget),
     mItemClicked(false),
     mEquipedColor(getThemeColor(Theme::ITEM_EQUIPPED)),
     mEquipedColor2(getThemeColor(Theme::ITEM_EQUIPPED_OUTLINE)),
     mUnEquipedColor(getThemeColor(Theme::ITEM_NOT_EQUIPPED)),
-    mUnEquipedColor2(getThemeColor(Theme::ITEM_NOT_EQUIPPED_OUTLINE))
+    mUnEquipedColor2(getThemeColor(Theme::ITEM_NOT_EQUIPPED_OUTLINE)),
+    mShortcut(shortcut)
 {
-    if (dropShortcut)
-        mMaxItems = dropShortcut->getItemCount();
+    if (mShortcut)
+        mMaxItems = mShortcut->getItemCount();
     else
         mMaxItems = 0;
 }
@@ -74,7 +77,7 @@ void DropShortcutContainer::setWidget2(const Widget2 *const widget)
 
 void DropShortcutContainer::draw(Graphics *graphics)
 {
-    if (!dropShortcut)
+    if (!mShortcut)
         return;
 
     BLOCK_START("DropShortcutContainer::draw")
@@ -101,11 +104,11 @@ void DropShortcutContainer::draw(Graphics *graphics)
         const int itemX = (i % mGridWidth) * mBoxWidth;
         const int itemY = (i / mGridWidth) * mBoxHeight;
 
-        if (dropShortcut->getItem(i) < 0)
+        if (mShortcut->getItem(i) < 0)
             continue;
 
-        const Item *const item = inv->findItem(dropShortcut->getItem(i),
-            dropShortcut->getItemColor(i));
+        const Item *const item = inv->findItem(mShortcut->getItem(i),
+            mShortcut->getItemColor(i));
 
         if (item)
         {
@@ -137,7 +140,7 @@ void DropShortcutContainer::draw(Graphics *graphics)
 
 void DropShortcutContainer::mouseDragged(MouseEvent &event)
 {
-    if (!dropShortcut)
+    if (!mShortcut)
         return;
 
     if (event.getButton() == MouseButton::LEFT)
@@ -149,8 +152,8 @@ void DropShortcutContainer::mouseDragged(MouseEvent &event)
             if (index == -1)
                 return;
 
-            const int itemId = dropShortcut->getItem(index);
-            const unsigned char itemColor = dropShortcut->getItemColor(index);
+            const int itemId = mShortcut->getItem(index);
+            const unsigned char itemColor = mShortcut->getItemColor(index);
 
             if (itemId < 0)
                 return;
@@ -164,7 +167,7 @@ void DropShortcutContainer::mouseDragged(MouseEvent &event)
             if (item)
             {
                 dragDrop.dragItem(item, DRAGDROP_SOURCE_DROP);
-                dropShortcut->removeItem(index);
+                mShortcut->removeItem(index);
             }
             else
             {
@@ -176,7 +179,7 @@ void DropShortcutContainer::mouseDragged(MouseEvent &event)
 
 void DropShortcutContainer::mousePressed(MouseEvent &event)
 {
-    if (!dropShortcut || !inventoryWindow)
+    if (!mShortcut || !inventoryWindow)
         return;
 
     const int index = getIndexFromGrid(event.getX(), event.getY());
@@ -189,7 +192,7 @@ void DropShortcutContainer::mousePressed(MouseEvent &event)
     const MouseButton::Type eventButton = event.getButton();
     if (eventButton == MouseButton::LEFT)
     {
-        if (dropShortcut->getItem(index) > 0)
+        if (mShortcut->getItem(index) > 0)
         {
             mItemClicked = true;
         }
@@ -197,7 +200,7 @@ void DropShortcutContainer::mousePressed(MouseEvent &event)
         {
             if (dragDrop.isSelected())
             {
-                dropShortcut->setItems(index, dragDrop.getSelected(),
+                mShortcut->setItems(index, dragDrop.getSelected(),
                     dragDrop.getSelectedColor());
                 dragDrop.deselect();
             }
@@ -209,8 +212,8 @@ void DropShortcutContainer::mousePressed(MouseEvent &event)
         if (!inv)
             return;
 
-        Item *const item = inv->findItem(dropShortcut->getItem(index),
-            dropShortcut->getItemColor(index));
+        Item *const item = inv->findItem(mShortcut->getItem(index),
+            mShortcut->getItemColor(index));
 
         if (popupMenu)
         {
@@ -223,13 +226,13 @@ void DropShortcutContainer::mousePressed(MouseEvent &event)
 
 void DropShortcutContainer::mouseReleased(MouseEvent &event)
 {
-    if (!dropShortcut)
+    if (!mShortcut)
         return;
 
     if (event.getButton() == MouseButton::LEFT)
     {
-        if (dropShortcut->isItemSelected())
-            dropShortcut->setItemSelected(-1);
+        if (mShortcut->isItemSelected())
+            mShortcut->setItemSelected(-1);
 
         const int index = getIndexFromGrid(event.getX(), event.getY());
         if (index == -1)
@@ -241,7 +244,7 @@ void DropShortcutContainer::mouseReleased(MouseEvent &event)
         {
             if (dragDrop.isSourceItemContainer())
             {
-                dropShortcut->setItems(index, dragDrop.getItem(),
+                mShortcut->setItems(index, dragDrop.getItem(),
                     dragDrop.getItemColor());
                 dragDrop.clear();
                 dragDrop.deselect();
@@ -255,7 +258,7 @@ void DropShortcutContainer::mouseReleased(MouseEvent &event)
 // Show ItemTooltip
 void DropShortcutContainer::mouseMoved(MouseEvent &event)
 {
-    if (!dropShortcut)
+    if (!mShortcut)
         return;
 
     const int index = getIndexFromGrid(event.getX(), event.getY());
@@ -263,8 +266,8 @@ void DropShortcutContainer::mouseMoved(MouseEvent &event)
     if (index == -1)
         return;
 
-    const int itemId = dropShortcut->getItem(index);
-    const unsigned char itemColor = dropShortcut->getItemColor(index);
+    const int itemId = mShortcut->getItem(index);
+    const unsigned char itemColor = mShortcut->getItemColor(index);
 
     if (itemId < 0)
         return;
diff --git a/src/gui/widgets/dropshortcutcontainer.h b/src/gui/widgets/dropshortcutcontainer.h
index 84d85c6c0..23addb8c2 100644
--- a/src/gui/widgets/dropshortcutcontainer.h
+++ b/src/gui/widgets/dropshortcutcontainer.h
@@ -26,6 +26,7 @@
 #include "gui/widgets/shortcutcontainer.h"
 
 class ItemPopup;
+class ShortcutBase;
 
 /**
  * An item shortcut container. Used to quickly use items.
@@ -38,7 +39,8 @@ class DropShortcutContainer final : public ShortcutContainer
         /**
          * Constructor. Initializes the graphic.
          */
-        explicit DropShortcutContainer(Widget2 *const widget);
+        DropShortcutContainer(Widget2 *const widget,
+                              ShortcutBase *const shortcut);
 
         A_DELETE_COPY(DropShortcutContainer)
 
@@ -82,6 +84,8 @@ class DropShortcutContainer final : public ShortcutContainer
         Color mEquipedColor2;
         Color mUnEquipedColor;
         Color mUnEquipedColor2;
+
+        ShortcutBase *mShortcut;
 };
 
 #endif  // GUI_WIDGETS_DROPSHORTCUTCONTAINER_H