Mana Void

[Moderator]

20:27, 13th Nov 2015
BPower: No update done in years. Rejected.

09:45, 5th Apr 2010
TriggerHappy:

Spell is far too simple and could even be made in the object editor.

 

[matej]

it seems i made a little mistake when was writing triger in description, second special effect is destroyed inside the loop...my bad =(

 

[DevilDuke1995]

Description plz?

 

[Dr. Boom]

Seas =)

Ok I tested it - hmm nothing difficult I think - but it still works.

Question:
- Why you use so many! variables?
- MV_Group variable - now it pick dead unit - buildings - magic immune units - and so on - too. Maybe you should "fix" that.

 

[matej]

Seas =)

Ok I tested it - hmm nothing difficult I think - but it still works.

Question:
- Why you use so many! variables?
- MV_Group variable - now it pick dead unit - buildings - magic immune units - and so on - too. Maybe you should "fix" that.

wow....ok thanks for info...i compleatly forget for buildings and imune units. thx =)

 

[Catch_ya]

Too simple + I really think their is too many globals for such a simple spell. I mean 3/4 of the spell is variable settings. It is unoriginal, I don't like DotA spells. 2/5.

 

[Squiggy]

Use (Triggering Unit) instead of setting a var :/

Also, place the Special Effect - Destroy last created special effect inside the loop.

In additon, why don't you just use

• Unit - Cause (Triggering Unit) to damage (Picked unit), dealing ((0.35 * (Level of Mana Void for (Triggering Unit))) * (Mana of (Triggering Unit) - Current Mana of (Triggering Unit)) of attack type Spells and damage type Sonic

instead of setting thousands of variables?

Furthermore, as the spell's being cast instantly, you don't need a Set MV_Target = (Targeted unit of ability being cast)

I rewrote your code a bit, look below:

Lurk

• Set MV_TargetPoint = (Position of Target unit of ability being cast)
• Set MV_Group = (Units whithin 250 of MV_TargetPoint matching ((((Matching unit) belongs to an enemy of (Owner of (Triggering Unit)) Equal to True)))
• Unit Group - Pick every unit in MV_Group and do (Actions)
  • Unit - Cause (Triggering Unit) to damage (Picked unit), dealing ((0.35 * (Level of Mana Void for (Triggering Unit))) * (Mana of (Triggering Unit) - Current Mana of (Triggering Unit)) of attack type Spells and damage type Sonic
• Special Effect - Create a special effect attached to origin of MV_Target using Abilities/Spells/NightElf/Blink/BlinkTarget.mdl
• Special Effect - Destroy last created special effect
• Special Effect - Create a special effect attached to origin of MV_Target using Abilities/Spells/NightElf/ManaBurn/ManaBurnTarget.mdl
• Special Effect - Destroy last created special effect
• Custom script: call RemoveLocation (udg_MV_TargetPoint)
• Custom script: call DestroyGroup (udg_MV_Group)

That's a bit too easy to even be cosidered a spell

 

[watermelon_1234]

Also, place the Special Effect - Destroy last created special effect inside the loop.

The creator says it's a typo on the description, but I don't know if it's like that in the map.

Just by looking at the code, I'm not too impressed because of its simplicity. It's basically a custom Frost Nova without the cold effect.