Core/Spells: Name SPELL_ATTR9_DO_NOT_ADD_TO_UNLEARN_LIST

Port From (https://github.com/TrinityCore/TrinityCore/commit/88d1078a1256fa9175110b3bec0f7f9fdd6b615d)

Source/Framework/Constants/Spells/SpellConst.cs

@@ -1973,7 +1973,7 @@ namespace Framework.Constants
         MissileSpeedIsDelayInSec = 0x10, // Missile Speed is Delay (in sec)
         IgnoreTotemRequirementsForCasting = 0x20, // Ignore Totem Requirements for Casting
         ItemCastGrantsSkillGain = 0x40, // Item Cast Grants Skill Gain
-        Unk7 = 0x80, // 7
+        DoNotAddToUnlearnList = 0x80, //  NYI - unlearn list not maintained SMSG_SEND_UNLEARN_SPELLS always empty // Do Not Add to Unlearn List
         AimedShot = 0x100, // 8
         NotUsableInArena = 0x200, // 9 Cannot Be Used In Arenas
         Unk10 = 0x400, // 10

Source/Game/Spells/Spell.cs

@@ -2992,7 +2992,7 @@ namespace Game.Spells
             }
 
             // Spells that don't create items can have this attribute - handle here
-            if (m_CastItem && m_spellInfo.HasAttribute(SpellAttr9.ItemCastGrantsSkillGain))
+            if (m_CastItem != null && m_spellInfo.HasAttribute(SpellAttr9.ItemCastGrantsSkillGain))
             {
                 Player playerCaster1 = m_caster.ToPlayer();
                 if (playerCaster1 != null)