fix bugs

src/game.cpp

@@ -1196,22 +1196,24 @@ ReturnValue Game::internalMoveItem(Cylinder* fromCylinder, Cylinder* toCylinder,
 		}
 	}
 
-	Item* moveItem = nullptr;
-
-	// remove the item
+	Item* moveItem = item;
 	int32_t itemIndex = fromCylinder->getThingIndex(item);
 	Item* updateItem = nullptr;
-	fromCylinder->removeThing(item, moveCount);
 
+	// update item(s)
 	if (item->isStackable()) {
 		// lets find out how much we need to move
 		uint32_t allowedCount = 0;
 
-		if (item->equals(toItem)) {
-			allowedCount = std::min<uint32_t>(ITEM_STACK_SIZE - toItem->getItemCount(), moveCount);
+		// when item is moved onto another equal item
+		if (item->equals(toItem) && moveCount != MAX_ITEM_COUNT) {
+			allowedCount =
+			    std::min<uint32_t>(static_cast<uint16_t>(MAX_ITEM_COUNT) - toItem->getItemCount(), moveCount);
 			if (allowedCount > 0) {
+				fromCylinder->removeThing(item, allowedCount);
 				toCylinder->updateThing(toItem, toItem->getID(), toItem->getItemCount() + allowedCount);
 				updateItem = toItem;
+				moveItem = nullptr;
 			}
 		}
 
@@ -1222,27 +1224,21 @@ ReturnValue Game::internalMoveItem(Cylinder* fromCylinder, Cylinder* toCylinder,
 				moveItem = item->clone();
 				moveItem->setItemCount(newCount);
 
-				// source item may get deleted if move count is actually the whole source count, so let's release it if
-				// needed
 				if (item->isRemoved()) {
-					ReleaseItem(item);
+					item->onRemoved();
 				}
-			} else {
-				// whole source item is moved
-				moveItem = item;
 			}
 		}
-	} else {
-		moveItem = item;
 	}
 
 	// add item
 	if (moveItem) {
+		fromCylinder->removeThing(item, moveCount);
 		toCylinder->addThing(index, moveItem);
 	}
 
-	if (itemIndex != -1 && !item->hasParent()) {
-		fromCylinder->postRemoveNotification(item, toCylinder, itemIndex);
+	if (itemIndex != -1) {
+		fromCylinder->postRemoveNotification(item, toCylinder, itemIndex, LINK_OWNER, actorPlayer);
 	}
 
 	if (moveItem) {