New EVENT_CASTED_ON event

[Bulle]

I have implemented this event to be able to create quests like "go heal this NPC", "go buff this NPC", potentially it could be used to implement a special spell or proc harming an NPC (if the spell is cast on this specific NPC, then some harm is done). This event is received by an NPC each time a spell is cast on him, and it is passed the identity of the caster (usually a PC) and the ID of the casted spell in variable "$spell_id".
Note that it is triggered before any resist or invulnerability is checked. The spell hits the NPC, that is it.

BE CAREFUL : it means an event is fired each tme a spell is cast on an NPC. I have no idea how much CPU overload this can cause. If this proves useless for "Live".like servers and a performance-hog may be this post should be moved to the "Custom" section.

Here is how it is implemented (as an SVN diff from version 1102) :

Index: hero-emulator/zone/event_codes.h
===================================================================
--- hero-emulator/zone/event_codes.h    (revision 135)
+++ hero-emulator/zone/event_codes.h    (working copy)
@@ -23,6 +23,7 @@
        EVENT_ZONE,                     //pc only
        EVENT_LEVEL_UP,         //pc only
        EVENT_KILLED_MERIT,     //received by an NPC once for every PC having merit (would receive XP if mob were not green) to kill the NPC -
basically all the group
+       EVENT_CASTED_ON,        //received by an NPC when a spell is cast on him (whether it sticks, is invuln'd etc or not, when the spell hit
s)

        _LargestEventID
 } QuestEventID;

Index: hero-emulator/zone/embparser.cpp
===================================================================
--- hero-emulator/zone/embparser.cpp    (revision 135)
+++ hero-emulator/zone/embparser.cpp    (working copy)
@@ -57,7 +57,8 @@
        "EVENT_LOOT",
        "EVENT_ZONE",
        "EVENT_LEVEL_UP",
-       "EVENT_KILLED_MERIT"
+       "EVENT_KILLED_MERIT",
+       "EVENT_CASTED_ON"
 };

 PerlembParser::PerlembParser(void) : Parser()
@@ -494,6 +495,11 @@
                        break;
                }

+               case EVENT_CASTED_ON:{
+                       ExportVar(packagename.c_str(), "spell_id", data);
+                       break;
+               }
+
                //nothing special about these events
                case EVENT_DEATH:
                case EVENT_SPAWN:

Index: hero-emulator/zone/spells.cpp
===================================================================
--- hero-emulator/zone/spells.cpp       (revision 132)
+++ hero-emulator/zone/spells.cpp       (working copy)
@@ -2259,6 +2259,13 @@
 // end of action packet


+       /* Send the EVENT_CASTED_ON event */
+       if(spelltar->IsNPC())
+       {       char temp1[100];
+               sprintf(temp1, "%d", spell_id);
+               parse->Event(EVENT_CASTED_ON, spelltar->GetNPCTypeID(), temp1, spelltar->CastToNPC(), this);
+       }
+
        // solar: now check if the spell is allowed to land

        // invuln mobs can't be affected by any spells, good or bad

Finally an example on how to use it :

# a_defective_cleaner
sub EVENT_CASTED_ON
{ if($spell_id == 202)
  { quest::setglobal("DefectiveCleanersBuffed", 1, 5, "F"); }
}

sub EVENT_ITEM
{ plugin::return_items(\%itemcount);
}

[Knightly]

I don't want this to sound trite or anything, but if it's going into the source the past tense of "cast" is just "cast"

[Bulle]

Very true.As you can see English is not my native language. Thank you for pointing it out.

[trevius]

We all know Bulle's native language is C or some other code lol.

Sounds interesting. I am all about having more options! Making unique events is one of the goals I work towards on my server. Nothing spectacular yet, but I am always trying to think up new stuff.

[KLS]

I actually had this and the other thing you posted in when I first did player quests but took them out. The merit one makes a lot more sense on npcs heh, anyway I'll be putting this in soon enough.

[Bulle]

This is great news KLS, thanks. I have been using these two events since I have written them and have had no problem yet. I leave it up to you whether to rename the CASTED by CAST or not, I am a bit puzzled about my English there : you can find reference to both words in many places on the web or in EQEmu.