Revert "Inventory: Make addItem for empty ItemStacks respect max stack size" (#8519)
authorANAND ︻气デ═一 <ClobberXD@gmail.com>
Sun, 12 May 2019 17:07:15 +0000 (22:37 +0530)
committerParamat <paramat@users.noreply.github.com>
Sun, 12 May 2019 17:07:15 +0000 (18:07 +0100)
Revert commit e6a9e60

src/inventory.cpp

index 2335136b12e012a6bca9d11179db5dba23d469cd..40dc602d0e855883a1d0a42549bdedb117c1fcda 100644 (file)
@@ -257,17 +257,8 @@ ItemStack ItemStack::addItem(ItemStack newitem, IItemDefManager *itemdef)
        // If this is an empty item, it's an easy job.
        else if(empty())
        {
-               const u16 stackMax = newitem.getStackMax(itemdef);
-
                *this = newitem;
-
-               // If the item fits fully, delete it
-               if (count <= stackMax) {
-                       newitem.clear();
-               } else { // Else the item does not fit fully. Return the rest.
-                       count = stackMax;
-                       newitem.remove(count);
-               }
+               newitem.clear();
        }
        // If item name or metadata differs, bail out
        else if (name != newitem.name
@@ -306,14 +297,7 @@ bool ItemStack::itemFits(ItemStack newitem,
        // If this is an empty item, it's an easy job.
        else if(empty())
        {
-               const u16 stackMax = newitem.getStackMax(itemdef);
-
-               // If the item fits fully, delete it
-               if (newitem.count <= stackMax) {
-                       newitem.clear();
-               } else { // Else the item does not fit fully. Return the rest.
-                       newitem.remove(stackMax);
-               }
+               newitem.clear();
        }
        // If item name or metadata differs, bail out
        else if (name != newitem.name
@@ -335,6 +319,7 @@ bool ItemStack::itemFits(ItemStack newitem,
 
        if(restitem)
                *restitem = newitem;
+
        return newitem.empty();
 }