[GUI] Arcane Circle

[Moderator]

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

10:31, 18th Dec 2012
Magtheridon96:

- This has a lot of leaks. ((some position) offset by x) creates 2 locations, so you need 2 variables. Not 1. You should store (some position) into a point variable and use that point variable to get the other positions like this: (point_variable offset by x)
- You need to make the spell configurable. There needs to be a trigger that runs on map initialization to allow the user to change spell constants like the lightning colors, the damage, the area of effect, the special effects, etc...
- This spell is not Multi-unit-instanceable. There are plenty of methods to make a spell Multi-unit-instanceable (MUI) such as dynamic indexing or hashtable usage.

Here are some links that might help you out:

• Hanky's Dynamic Indexing Method
• Bribe's Unit Indexer
• Simple Method by Tank-Commander
• Hashtable Tutorial by Jazztastic (Complete Beginner's Guide)

 

[Wrda]

1. You must post triggers
2. Your index method isnt efficient, this link can help you how to make MUI: http://www.hiveworkshop.com/forums/spells-569/gui-dynamic-indexing-template-144325/?prev=mmr%3D6

 

[Almia]

Post the triggers so i can put my review.

 

[Magtheridon96]

@Radamantus:

http://www.hiveworkshop.com/forums/apps.php?p=parsetrigger&id=ovdqfk

 

[Almia]

BOOM!
Tons of leaks
Not MUI
No configuration
No Proper Documentation


MAG ! Needs Fix
btw,where did you get that one?

 

[Losam]

...

What is the function of this spell?
I do not understand.

 

[Tank-Commander]

Massively inefficient would be another point on the board, to add to all the other issues Rad raised

 

[doomhammer99]

@Rada
-Very Bad review!

 

[Almia]

Sorry

 

[eubz]

I've seen a wait here.

• Unit - A unit Starts the effect of an ability
• Conditions
  • (Ability being cast) Equal to Arcane Blast
• Actions
  • Wait PS_Duration seconds