Earth Spirit Spellpack 1.1

[Moderator]

12th Dec 2015
IcemanBo: Too long as NeedsFix. Rejected.

Required changes

• Boulder Toss is not MUI
• You need to flush child hashtables
• Don't use two unit groups for Earth Power. Set the ability levels based on number of units in group
• Earth Power Aura should check that the unit with the aura is not dead
• Earth Power Aura does not work correctly if there are multiple units using it
• You should retrigger Earth Power Aura, you're doing double checks, units is alive and unit is dead
• Earth Power Aura doesn't have a working disabled icon

Old reviews

Required changes

• Add importing instructions into the map file
• Boulder Toss is not MUI
• Boulder Toss checks for the default Stunned (pause) buff so it can activate on other abilities also
• Trample should check that the caster isn't dead
• You share the hashtable between abilities, so don't flush child hashtable if a unit has data stored for some other spell
• Don't use two unit groups for Earth Power. Set the ability levels based on number of units in group
• Ho to use Crow Form properly

Suggested changes

• Earth Power should not have passive icon as learn icon
• In my opinion Earth Power Aura doesn't need to update every 0.03 seconds. 0.1 should be enough.

 

[Noobsmoke]

The spells are not that impressive but triggering it's good, 3/5

 

[Kazeon]

Actually, I don't like the spells.. But, this is an example of hashtable indexing spells.. I can learn about indexing using hashtable..

 

[mateuspv]

use te tag's [ trigger] you trigger here[ /trigger] to post.