Arcane Spellpack By colourcloud

[Moderator]

12th Dec 2015
IcemanBo: For long time as NeedsFix. Rejected.

08:40, 1st Jun 2011
Maker:
You should separate the visibility actions from the hashtable creation trigger since they're not part of the spell.
You don't have to use custom script for getting the handle. I'd prefer the GUI way since the spell is done with GUI. But you can leave it as it is.
For Powerslash you could use the handle integer also.
You can abuse Powerslash if you hit stop immediately after casting it, no mana cost or cooldown. You might have to change the event to Starts the effect...
You could store the ability level into hashtable and load it when dealing the damage.
You could use triggering unit instead of casting unit.
Instead of having several values in the or condition, you could have a real value that runs from 0.36 to 0 and check whether it is < 0. Or use an integer for even faster performance.
Leaks a region:

• Set Temp_Loc_2 = (Random point in (Region centered at Temp_Loc_1 with size (320.00, 320.00)))

The originality is a bit lacking, Powerslash is basically Omnislash and we have seen many of those. The Stomp is just a looping circle. The Stomp is triggered well.

 

[Genji Lone]

hmmm going to test, what does spell do?

 

[SpectreD]

besides from ingame boarder ( the green shit) i guess its average
its my first spell

 

[Jazztastic]

Colour Cloud:

1. Post triggers in hidden tags.
2. Improve descriptions
3. It says spell pack on the title, it's a single spell, isn't it?
4. Follow what bribe said. He pretty much said it all

 

[Archangel678]

Well, atleast he gave credits ^^

I'd suggest improving the description and telling us what the spell does. Also, when using [HIDDEN] tags, you must enter text, like [HIDDEN="Spell Description"][/HIDDEN]

===============================================================

You should merge the two loops, as its fewer triggers running at the same time, and it would remove the need for this:

• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
  • If - Conditions
    • (Powerslash_Knockback_Group is empty) Equal to True
  • Then - Actions
    • Trigger - Turn off Powerslash Knockback <gen>
  • Else - Actions

=========================================================

• Set Temp_Loc_2 = (Random point in (Region centered at Temp_Loc_1 with size (320.00, 320.00)))
• //
• //
• //Change that to:
• //
• //Set TempRegion = (Region centered at Temp_Loc_1 with size (320.00, 320.00))
• //Set Temp_Loc_2 = (Random point in (TempRegion))
• //
• //
• Custom script: call RemoveRect(udg_TempRegion)
• Custom script: call RemoveLocation(udg_Temp_Loc_2)

====================================================

I personally don't like using (Key ()), but I guess you can because its GUI.

====================================================

• Untitled Trigger 001
  • Events
    • Map initialization
  • Conditions
  • Actions
    • Set Powerslahs_Hash = (Last created hashtable)
    • Hashtable - Create a hashtable

I'd suggest changing the name or merging it with the other initialization.

====================================================

• Special Effect - Destroy (Load (Key effect_1) of (Key (Picked unit)) in Powerslahs_Hash)

Instead of using (Key (Picked unit)) I'd suggest using the variable...

 

[SpectreD]

ok thanks for the advise, ill be sure to improve