fix: check quest reward nil value on setStorage (#3403)

Author: dudantas

data-otservbr-global/scripts/actions/system/quest_reward_common.lua

@@ -56,7 +56,7 @@ local achievementTable = {
 	[6088] = "Annihilator",
 }
 
-local function playerAddItem(params, item)
+local function playerAddItem(params, item, rewardIndex)
 	local player = params.player
 	if not checkWeightAndBackpackRoom(player, params.weight, params.message) then
 		return false
@@ -92,6 +92,11 @@ local function playerAddItem(params, item)
 			player:questKV(params.questName):set("params.questName", os.time() + params.time * 3600) -- multiplicação por hora
 		end
 	else
+		if params.storage == nil then
+			logger.warn("Storage key is nil for reward index {}, itemid {}", rewardIndex, params.itemid)
+			return false
+		end
+
 		player:setStorageValue(params.storage, 1)
 		if params.timer then
 			player:setStorageValue(params.timer, os.time() + params.time * 3600) -- multiplicação por hora
@@ -100,7 +105,7 @@ local function playerAddItem(params, item)
 	return true
 end
 
-local function playerAddContainerItem(params, item)
+local function playerAddContainerItem(params, item, rewardIndex)
 	local player = params.player
 
 	local reward = params.containerReward
@@ -130,6 +135,11 @@ local function playerAddContainerItem(params, item)
 			player:questKV(params.questName):set("params.questName", os.time() + params.time * 3600) -- multiplicação por hora
 		end
 	else
+		if params.storage == nil then
+			logger.warn("Storage key is nil for reward index {}, itemid {} in container", rewardIndex, params.itemid)
+			return false
+		end
+
 		player:setStorageValue(params.storage, 1)
 		if params.timer then
 			player:setStorageValue(params.timer, os.time() + params.time * 3600) -- multiplicação por hora
@@ -223,7 +233,7 @@ function questReward.onUse(player, item, fromPosition, itemEx, toPosition)
 			else
 				addItemParams.message = "You have found " .. itemArticle .. " " .. itemName
 			end
-			if not playerAddItem(addItemParams, item) then
+			if not playerAddItem(addItemParams, item, i) then
 				return true
 			end
 		end
@@ -242,7 +252,7 @@ function questReward.onUse(player, item, fromPosition, itemEx, toPosition)
 				useKV = setting.useKV,
 			}
 
-			if not playerAddContainerItem(addContainerItemParams, item) then
+			if not playerAddContainerItem(addContainerItemParams, item, i) then
 				return true
 			end
 		end