BugFix: Kasaroro Northern sandy

[xworndownx]

original script was mising an actual call to get the pNation from the player. Added that in to the triggers to prevent the npc from crashing the server.

Code: Select all

-----------------------------------
-- Area: Northern San d'Oria
-- NPC:  Kasaroro
-- Type: Consulate Representative
-- Involved in Mission: 2-3 Windurst
-- @pos -72 -3 34 231
-----------------------------------
package.loaded["scripts/zones/Northern_San_dOria/TextIDs"] = nil;
-----------------------------------

require("scripts/globals/keyitems");
require("scripts/globals/missions");
require("scripts/zones/Northern_San_dOria/TextIDs");

-----------------------------------
-- onTrade Action
-----------------------------------

function onTrade(player,npc,trade)
	
	if(player:getQuestStatus(SANDORIA,FLYERS_FOR_REGINE) == QUEST_ACCEPTED) then
		if(trade:hasItemQty(532,1) and trade:getItemCount() == 1) then -- Trade Magicmart_flyer
			player:messageSpecial(FLYER_REFUSED);
		end
	end
	
end;

-----------------------------------
-- onTrigger Action
-----------------------------------

function onTrigger(player,npc)
	pNation = player:getNation();
	currentMission = player:getCurrentMission(pNation);
	MissionStatus = player:getVar("MissionStatus");
	
	if(currentMission == THE_THREE_KINGDOMS) then
		if(MissionStatus == 2) then
			player:startEvent(0x0222);
		elseif(MissionStatus == 6) then
			player:showText(npc,KASARORO_DIALOG + 7);
		elseif(MissionStatus == 7) then
			player:startEvent(0x0223);
		elseif(MissionStatus == 11) then
			player:showText(npc,KASARORO_DIALOG + 20);
		end
	elseif(currentMission == THE_THREE_KINGDOMS_SANDORIA) then
		if(MissionStatus == 3) then
			player:showText(npc,KASARORO_DIALOG);
		elseif(MissionStatus == 4) then
			player:startEvent(0x0225);
		elseif(MissionStatus == 5) then
			player:startEvent(0x0226); -- done with Sandy first path, now go to bastok
		end
	elseif(currentMission == THE_THREE_KINGDOMS_SANDORIA2) then
		if(MissionStatus == 8) then
			player:showText(npc,KASARORO_DIALOG);
		elseif(MissionStatus == 10) then
			player:startEvent(0x0227);
		end
	else
		player:startEvent(0x0224);
	end
	
end;

-----------------------------------
-- onEventUpdate
-----------------------------------

function onEventUpdate(player,csid,option)
-- printf("CSID: %u",csid);
-- printf("RESULT: %u",option);
end;

-----------------------------------
-- onEventFinish
-----------------------------------

function onEventFinish(player,csid,option)
-- printf("CSID: %u",csid);
-- printf("RESULT: %u",option);
	
	if(csid == 0x0222) then
		player:addMission(WINDURST,THE_THREE_KINGDOMS_SANDORIA);
		player:delKeyItem(LETTER_TO_THE_CONSULS_WINDURST);
		player:setVar("MissionStatus",3);
	elseif(csid == 0x0226) then
		player:addMission(WINDURST,THE_THREE_KINGDOMS);
		player:setVar("MissionStatus",6);
	elseif(csid == 0x0223) then
		player:addMission(WINDURST,THE_THREE_KINGDOMS_SANDORIA2);
		player:setVar("MissionStatus",8);
	elseif(csid == 0x0227) then
		player:addMission(WINDURST,THE_THREE_KINGDOMS);
		player:delKeyItem(KINDRED_CREST);
		player:addKeyItem(KINDRED_REPORT);
		player:messageSpecial(KEYITEM_OBTAINED,KINDRED_REPORT);
		player:setVar("MissionStatus",11);
	end
	
end;

[bluekirby0]

Fixed: http://code.google.com/p/onetimexi/source/detail?r=1586

[xworndownx]

Fix for 1586: currentMission = player:getCurrentMission(SANDORIA);
Should be currentMission = player:getCurrentMission(WINDURST);

This is a quest for Windy players. Setting it to SANDORIA will break the quest as the script needs to read the windy mission line from the player. Should be set to WINDURST instead. Tested and confirmed by players on my server.

Thanks!

[bluekirby0]

Took me a while to figure out what you were talking about since that line isn't anywhere in the script. At any rate, I figured it out and fixed: http://code.google.com/p/onetimexi/source/detail?r=1601