html_url,issue_url,id,node_id,user,created_at,updated_at,author_association,body,reactions,performed_via_github_app,issue
https://github.com/ACEmulator/ACE/issues/1806#issuecomment-494213315,https://api.github.com/repos/ACEmulator/ACE/issues/1806,494213315,MDEyOklzc3VlQ29tbWVudDQ5NDIxMzMxNQ==,10137,2019-05-21T02:14:30Z,2019-05-21T02:25:47Z,NONE,"For Wrath of the Puppeteer spell, ""|| spellID == 2998"" needs to be added to the Volley else if branch in SpellProjectile.GetProjectileSpellType(), as it is a bolt/volley hybrid spell.  Three projectiles are fired but in succession, rather than as a group together, so that effect needs to be accounted for in code.

SpellIDs 1832 through 1838, which Avalanche is a part of, are spell projectile rain spells that are unique to other War Magic projectile spells, with nine or 12 projectiles that slam down from above the target in an area, and should only be usable outdoors.  Those spell lines are a little wonky, as their spell.Power exceeds their item casters ItemSpellcrafts, so that probably needs to be accounted for, also.

Paralyzing Fear spell on Virindi Implant also exceeds the ItemSpellcraft of the caster item.

I imagine that ItemSpellcraft should probably only be used for resist checks and not for if an item has the ""skill"" to cast its spell.  Setting magicSkill = 999, at line 334 in Player_Magic.cs, ensures that a caster item can successfully cast its own spells.

Lastly, the two WarMagic() overloads are probably sending an extra GameEventUseDone(), in the case of ""spell not implemented, yet"", as the cursor goes into perma-hourglass. 
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,437208346
https://github.com/ACEmulator/ACE/issues/1806#issuecomment-494237710,https://api.github.com/repos/ACEmulator/ACE/issues/1806,494237710,MDEyOklzc3VlQ29tbWVudDQ5NDIzNzcxMA==,19214857,2019-05-21T04:46:59Z,2019-05-21T04:46:59Z,CONTRIBUTOR,I'm going to research these spells and see if I can get them added.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,437208346
https://github.com/ACEmulator/ACE/issues/1806#issuecomment-494237719,https://api.github.com/repos/ACEmulator/ACE/issues/1806,494237719,MDEyOklzc3VlQ29tbWVudDQ5NDIzNzcxOQ==,8909245,2019-05-21T04:47:02Z,2019-05-21T04:47:02Z,COLLABORATOR,"""Those spell lines are a little wonky, as their spell.Power exceeds their item casters ItemSpellcrafts, so that probably needs to be accounted for, also.""

i think we saw issues with this for other built-in spells before... even if the ItemSpellcraft was above the spell.Power, it would still sometimes fizzle. The built-in spell code was changed because of this awhile ago, so that built-in spells never fizzle.

The ItemSpellcraft ends up being the 'difficulty' the spell was cast with, ie. if I use a built-in spell on an item with ItemSpellcraft 250, when the target tries to resist, it would be resisting against skill level 250 there","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,437208346
https://github.com/ACEmulator/ACE/issues/1806#issuecomment-494238752,https://api.github.com/repos/ACEmulator/ACE/issues/1806,494238752,MDEyOklzc3VlQ29tbWVudDQ5NDIzODc1Mg==,10137,2019-05-21T04:53:12Z,2019-05-21T04:53:49Z,NONE,"The Avalanche spell (really all of these rain spells) has a spell.Power of 320, but the item, as you mentioned, has a built-in skill of 250. Even with the -50 discount on spell.Power for difficulty calculation, it still ends up being greater than the item's skill level, so the spell automatically fails, and then fizzles.  Currently, the line ""if (isWeaponSpell && caster.ItemSpellcraft != null) magicSkill = caster.ItemSpellcraft;"" is causing the instant spell cast failure for the item's spell.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,437208346
https://github.com/ACEmulator/ACE/issues/1806#issuecomment-494239854,https://api.github.com/repos/ACEmulator/ACE/issues/1806,494239854,MDEyOklzc3VlQ29tbWVudDQ5NDIzOTg1NA==,10137,2019-05-21T04:59:32Z,2019-05-21T04:59:32Z,NONE,"There isn't a lot of information to go on for these spells.  For the Wrath of the Puppeteer, the spell has multiple projectiles but doesn't have a DimsOrigin.X > 1, so SpellProjectile.GetProjectileSpellType() was assigning it a Bolt type.  However, in WarMagic(WorldObject target, Spell spell), the check didn't match any of the current criteria, ie. spell.NumProjectiles == 1 || spellType == SpellProjectile.ProjectileSpellType.Volley || spellType == SpellProjectile.ProjectileSpellType.Blast.  The picture associated with the Puppeteer's Skull weapon shows the three projectiles firing in succession.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,437208346
https://github.com/ACEmulator/ACE/issues/1806#issuecomment-494241632,https://api.github.com/repos/ACEmulator/ACE/issues/1806,494241632,MDEyOklzc3VlQ29tbWVudDQ5NDI0MTYzMg==,8909245,2019-05-21T05:09:51Z,2019-05-21T05:09:51Z,COLLABORATOR,"Hmm, maybe that section should be changed to this:

```
            if (magicSkill > 0 && magicSkill >= (int)difficulty - 50)
            {
                var chance = 1.0f - SkillCheck.GetMagicSkillChance((int)magicSkill, (int)difficulty);
                var rng = ThreadSafeRandom.Next(0.0f, 1.0f);
                if (chance < rng)
                    castingPreCheckStatus = CastingPreCheckStatus.Success;
            }

            if (isWeaponSpell)
                castingPreCheckStatus = CastingPreCheckStatus.Success;
```","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,437208346
https://github.com/ACEmulator/ACE/issues/1806#issuecomment-494243312,https://api.github.com/repos/ACEmulator/ACE/issues/1806,494243312,MDEyOklzc3VlQ29tbWVudDQ5NDI0MzMxMg==,10137,2019-05-21T05:19:08Z,2019-05-21T05:19:08Z,NONE,"Yes, that works.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,437208346