From 63460ea00d944ccb0c1ab7735010c90c028d1e84 Mon Sep 17 00:00:00 2001
From: David Markowitz <39972741+EmosewaMC@users.noreply.github.com>
Date: Wed, 30 Nov 2022 01:04:46 -0800
Subject: [PATCH] Fix bricks not creating new stacks (#860)

Unintentionally, bricks were not creating new stacks if you tried to get another stack.  This prevents some missions from being completed.  This issue is now fixed
---
 dGame/dComponents/InventoryComponent.cpp | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/dGame/dComponents/InventoryComponent.cpp b/dGame/dComponents/InventoryComponent.cpp
index 77f3f035..acf1ae6a 100644
--- a/dGame/dComponents/InventoryComponent.cpp
+++ b/dGame/dComponents/InventoryComponent.cpp
@@ -209,8 +209,10 @@ void InventoryComponent::AddItem(
 
 	auto stack = static_cast<uint32_t>(info.stackSize);
 
+	bool isBrick = inventoryType == eInventoryType::BRICKS || (stack == 0 && info.itemType == 1);
+
 	// info.itemType of 1 is item type brick
-	if (inventoryType == eInventoryType::BRICKS || (stack == 0 && info.itemType == 1)) {
+	if (isBrick) {
 		stack = UINT32_MAX;
 	} else if (stack == 0) {
 		stack = 1;
@@ -233,7 +235,7 @@ void InventoryComponent::AddItem(
 	}
 
 	// If we have some leftover and we aren't bricks, make a new stack
-	while (left > 0 && !(inventoryType == eInventoryType::BRICKS || (stack == 0 && info.itemType == 1))) {
+	while (left > 0 && (!isBrick || (isBrick && !existing))) {
 		const auto size = std::min(left, stack);
 
 		left -= size;