Fixed issue: 8140

http://hercules.ws/board/tracker/issue-8140-toucan-npc-error-message/
That error usually happens when the player clicked on a NPC that has the view id
of a mob, to activate this kind of npc it's needed to be in a 2,2 range
from it. If the OnTouch area of a npc, coincides with the 2,2 range of
another it's expected that the OnTouch event be put first in stack, because
unit_walktoxy_timer is executed before any other function in this case.
So it's best practice to put an 'end;' before OnTouch events in npcs that
have view ids of mobs to avoid this kind of error.
Also updated script_commands.txt to add this recomendation.

1 files changed, 7 insertions, 1 deletions

diff --git a/npc/re/quests/quests_brasilis.txt b/npc/re/quests/quests_brasilis.txt
index b6aa85ce9..33758fe9a 100644
--- a/npc/re/quests/quests_brasilis.txt
+++ b/npc/re/quests/quests_brasilis.txt
@@ -3,7 +3,7 @@
 //===== By ===================================================
 //= L0ne_W0lf
 //===== Version ==============================================
-//= 1.2a
+//= 1.2c
 //===== Description ==========================================
 //= [Official Conversion]
 //= Lost Puppies (Repeatable, 24 hours.)
@@ -18,6 +18,8 @@
 //= 1.2 Optimization. [Euphy]
 //= 1.2a Added 'consumeitem' command. [Euphy]
 //= 1.2b Updated RE EXP. [Michieru]
+//= 1.2c Put end; before OnTouch events in npcs that
+//= have view id of a mob [Panikon]
 //============================================================
 
 // Lost Puppies :: dogdog.sc
@@ -862,6 +864,8 @@ brasilis,59,226,3	script	Poring#bra	4_PORING,{
 }
 
 bra_fild01,75,83,5	script	Toucan#bra	TOUCAN,2,2,{
+end;
+
 OnTouch:
 	if (brazil_gua == 6) {
 		mes "[Toucan]";
@@ -922,6 +926,8 @@ OnTouch:
 }
 
 bra_fild01,34,184,5	script	Jaguar#bra	JAGUAR,2,2,{
+end;
+
 OnTouch_:
 	if (brazil_gua == 7) {
 		mes "[Jaguar]";