TrinityCore/TrinityCore — pull request #6962 by Elron103 (2012-06-28 22:21:57+00:00)
Title: Core/Spells: Fix spell casting time reduce / increase effects for players and creatures

### Description
Currently casting time reduce or increase effects do not seem to work for creatures. This issue was noticed in the Anub'Arak encounter in the instance Trial of the Crusader:

Creature: Nerubian Burrower (http://www.wowhead.com/npc=34607)
Spell 1: Spider Frenzy (http://www.wowhead.com/spell=66129)
Spell 2: Shadow Strike (http://www.wowhead.com/spell=66134)

If several Nerubian Burrowers are pulled together, they receive the buff Spider Frenzy, which reduces spell casting time by half. The only ability of Nerubian Burrowers, which even has spell casting time, is the ability Shadow Strike. But currently, the Shadow Strike does not benefit from the Spider Frenzy buff and the casting time is not reduced.

### Fix explanation
Currently, the most general case for spell casting time reduce / increase effects is only applied, if spellProto->SpellFamilyName != 0. But this check is not correct. Shadow Strike for example has spellProto->SpellFamilyName = 0, so it cannot benefit from reduce / increase effects. Only for player abilities a SpellFamilyName is set. This change is meant to allow the following possible situations to benefit from reduce / increase effects:

* Player casting a player ability (=> E.g. quest related spells will NOT benefit from such effects)
* Creature casting any spell

### Thanks
Special thanks to Kandera, Subv and Shauren, who provided some necessary information or ideas for the issue.

### Revs
TDB: 46
TC: f0f68f15a86c5ac