non-melee damage messages

[demonstar55]

So I remember from playing on live that I was never able to parse spell damage, but when I started playing on live it worked fine, and I was able to see other people's spell damage. This was just the case of using the wrong StringID and not sending to to the entity_list. I can verify that on live it will send it to everyone, including people out of your group, so this behavior is correct. The range might be too large, but I am not sure what it is, I just used the same range that critical blast messages use.

Index: EQEmuServer/zone/attack.cpp
===================================================================
--- EQEmuServer/zone/attack.cpp (revision 2145)
+++ EQEmuServer/zone/attack.cpp (working copy)
@@ -3534,7 +3534,7 @@
                                if (((spell_id != SPELL_UNKNOWN)||(FromDamageShield)) && damage>0) {
                                        //special crap for spell damage, looks hackish to me
                                        char val1[20]={0};
-                                       attacker->Message_StringID(MT_NonMelee,OTHER_HIT_NONMELEE,GetCleanName(),ConvertArray(damage,val1));
+                                       entity_list.MessageClose_StringID(this, false, 100, MT_NonMelee, HIT_NON_MELEE, attacker->GetCleanName(), GetCleanName(),ConvertArray(damage,val1));
                            } else {
                                if(damage > 0) {
                                                if(spell_id != SPELL_UNKNOWN)

[demonstar55]

Apparently I missed some things or didn't test that change enough

Index: attack.cpp
===================================================================
--- attack.cpp  (revision 2146)
+++ attack.cpp  (working copy)
@@ -3392,7 +3392,7 @@

                                if(attacker && attacker->IsClient() && (spell_id != SPELL_UNKNOWN) && damage>0) {
                                        char val1[20]={0};
-                                       attacker->Message_StringID(4,OTHER_HIT_NONMELEE,GetCleanName(),ConvertArray(damage,val1));
+                                       entity_list.MessageClose_StringID(this, false, 100, MT_NonMelee, HIT_NON_MELEE, attacker->GetCleanName(), GetCleanName(),ConvertArray(damage,val1));
                                }

                                Death(attacker, damage, spell_id, skill_used);
@@ -3534,7 +3534,10 @@
                                if (((spell_id != SPELL_UNKNOWN)||(FromDamageShield)) && damage>0) {
                                        //special crap for spell damage, looks hackish to me
                                        char val1[20]={0};
-                                       attacker->Message_StringID(MT_NonMelee,OTHER_HIT_NONMELEE,GetCleanName(),ConvertArray(damage,val1));
+                                       if (FromDamageShield)
+                                               attacker->Message_StringID(MT_NonMelee,OTHER_HIT_NONMELEE,GetCleanName(),ConvertArray(damage,val1));
+                                       else
+                                               entity_list.MessageClose_StringID(this, false, 100, MT_NonMelee,HIT_NON_MELEE,attacker->GetCleanName(),GetCleanName(),ConvertArray(damage,val1));
                            } else {
                                if(damage > 0) {
                                                if(spell_id != SPELL_UNKNOWN)

[Secrets]

Originally Posted by demonstar55

Apparently I missed some things or didn't test that change enough

Index: attack.cpp
===================================================================
--- attack.cpp  (revision 2146)
+++ attack.cpp  (working copy)
@@ -3392,7 +3392,7 @@

                                if(attacker && attacker->IsClient() && (spell_id != SPELL_UNKNOWN) && damage>0) {
                                        char val1[20]={0};
-                                       attacker->Message_StringID(4,OTHER_HIT_NONMELEE,GetCleanName(),ConvertArray(damage,val1));
+                                       entity_list.MessageClose_StringID(this, false, 100, MT_NonMelee, HIT_NON_MELEE, attacker->GetCleanName(), GetCleanName(),ConvertArray(damage,val1));
                                }

                                Death(attacker, damage, spell_id, skill_used);
@@ -3534,7 +3534,10 @@
                                if (((spell_id != SPELL_UNKNOWN)||(FromDamageShield)) && damage>0) {
                                        //special crap for spell damage, looks hackish to me
                                        char val1[20]={0};
-                                       attacker->Message_StringID(MT_NonMelee,OTHER_HIT_NONMELEE,GetCleanName(),ConvertArray(damage,val1));
+                                       if (FromDamageShield)
+                                               attacker->Message_StringID(MT_NonMelee,OTHER_HIT_NONMELEE,GetCleanName(),ConvertArray(damage,val1));
+                                       else
+                                               entity_list.MessageClose_StringID(this, false, 100, MT_NonMelee,HIT_NON_MELEE,attacker->GetCleanName(),GetCleanName(),ConvertArray(damage,val1));
                            } else {
                                if(damage > 0) {
                                                if(spell_id != SPELL_UNKNOWN)

Committed along with another change to DS Filters, r2147.

[demonstar55]

Thanks! I also noticed that what live reports for DOTs is different (and UF at least). The Titanium client doesn't have the StringID is 13327 //%1 has taken %2 damage from %3 by %4.
(which is mob, damage, caster, spell)

Not too sure how to proceed since it wouldn't work for Titanium