Use the size of the AllItemsList vector instead of IDI_LAST for getting how many items are there

Source/items.cpp

@@ -1351,8 +1351,8 @@ _item_indexes GetItemIndexForDroppableItem(bool considerDropRate, tl::function_r
 	ril.clear();
 
 	unsigned cumulativeWeight = 0;
-	for (std::underlying_type_t<_item_indexes> i = IDI_GOLD; i <= IDI_LAST; i++) {
+	for (size_t i = 0; i < AllItemsList.size(); i++) {
-		if (!IsItemAvailable(i))
+		if (!IsItemAvailable(static_cast<int>(i)))
 			continue;
 		const ItemData &item = AllItemsList[i];
 		if (item.dropRate == 0)
@@ -2369,7 +2369,7 @@ std::string GetTranslatedItemNameMagical(const Item &item, bool hellfireItem, bo
 
 bool IsItemAvailable(int i)
 {
-	if (i < 0 || i > IDI_LAST)
+	if (i < 0 || i >= static_cast<int>(AllItemsList.size()))
 		return false;
 
 	if (gbIsSpawn) {

Source/lua/modules/dev/items.cpp

@@ -135,8 +135,8 @@ std::string DebugSpawnUniqueItem(std::string itemName)
 	if (!foundUnique) return "No unique item found!";
 
 	_item_indexes uniqueBaseIndex = IDI_GOLD;
-	for (std::underlying_type_t<_item_indexes> j = IDI_GOLD; j <= IDI_LAST; j++) {
+	for (size_t j = 0; j < AllItemsList.size(); j++) {
-		if (!IsItemAvailable(j))
+		if (!IsItemAvailable(static_cast<int>(j)))
 			continue;
 		if (AllItemsList[j].iItemId == uniqueItem.UIItemId) {
 			uniqueBaseIndex = static_cast<_item_indexes>(j);

Source/lua/modules/items.cpp

@@ -346,7 +346,6 @@ void RegisterItemIndexEnum(sol::state_view &lua)
 	        { "RuneOfStone", IDI_RUNEOFSTONE },
 	        { "SorcererDiablo", IDI_SORCERER_DIABLO },
 	        { "ArenaPotion", IDI_ARENAPOT },
-	        { "Last", IDI_LAST },
 	        { "None", IDI_NONE },
 	    });
 }

Source/pack.cpp

@@ -436,7 +436,7 @@ bool UnPackNetItem(const Player &player, const ItemNetPack &packedItem, Item &it
 {
 	item = {};
 	const _item_indexes idx = static_cast<_item_indexes>(SDL_SwapLE16(packedItem.def.wIndx));
-	if (idx < 0 || idx > IDI_LAST)
+	if (idx < 0 || idx >= static_cast<_item_indexes>(AllItemsList.size()))
 		return true;
 	if (idx == IDI_EAR) {
 		RecreateEar(item, SDL_SwapLE16(packedItem.ear.wCI), SDL_SwapLE32(packedItem.ear.dwSeed), packedItem.ear.bCursval, packedItem.ear.heroname);

test/items_test.cpp

@@ -62,8 +62,8 @@ void GenerateAllUniques(bool hellfire, const size_t expectedUniques)
 			continue;
 
 		_item_indexes uniqueBaseIndex = IDI_GOLD;
-		for (std::underlying_type_t<_item_indexes> j = IDI_GOLD; j <= IDI_LAST; j++) {
+		for (size_t j = 0; j < AllItemsList.size(); j++) {
-			if (!IsItemAvailable(j))
+			if (!IsItemAvailable(static_cast<int>(j)))
 				continue;
 			if (AllItemsList[j].iItemId != uniqueItem.UIItemId)
 				continue;