Frost Slide v0.2

[Moderator]

12.12
IcemanBo: For long time as NeedsFix. Rejected.

04:00, 12th Dec 2015
IcemanBo:

 

[light bolt30]

First spell.. Please give more information about it, and maybe post the triggers. I believe people will be expecting leaks and poor use of triggers. The idea isn't really that creative as well.

 

[Adiktuz]

don't really like the visuals... anyway, not a prob though...

and as for the idea, I kinda don't think its useful (sliding to slow enemies)...

but please post triggers so we could know if this is worth downloading...

 

[Patio_111]

I expected this, as light bolt30 said, to be full of leaks, non-MUI, etc. However, if it is your first spell, I must say that you have done a pretty good job. The triggering is very simple. The spell is completely MUI, and you used the "Slow Aura" spell (which I didn't even know existed) ingeniously to save the complication of having to enumerate all nearby units and manually slow them (which I thought was the part you were going to mess up at).

Overall, I think the spell is very simple and the idea is not too original (nor can it be applied to any kind of map as a normal spell), but it is well done and works fine. I vote for approval.

Do, however, update the description and attach the triggers in the post, because it is more attractive, and more people will download it that way. Also, a useful little detail to add is to include two heroes in the test map for MUI testing purposes.

EDIT: I didn't notice the leak Adikutz pointed out...

• Set Frost_Move = ((Position of (Picked unit)) offset by 25.00 towards (Facing of (Picked unit)) degrees)

Better fix that soon

Triggers

Frost Cast

• Frost Cast
  • Events
    • Unit - A unit Starts the effect of an ability
  • Conditions
    • (Ability being cast) Equal to Frost Slide
  • Actions
    • Unit Group - Add (Triggering unit) to Frost_Group
    • Unit - Add Slow Frost to (Triggering unit)
    • Unit - Set level of Slow Frost for (Triggering unit) to (Level of Frost Slide for (Triggering unit))

Frost face

• Frost face
  • Events
    • Unit - A unit Is issued an order targeting a point
  • Conditions
    • ((Triggering unit) has buff Frost Slide ) Equal to True
  • Actions
    • Set Temp_Point = (Target point of issued order)
    • Unit - Make (Triggering unit) face Temp_Point over 0.00 seconds
    • Custom script: call RemoveLocation(udg_Temp_Point)

Frost Move

• Frost Move
  • Events
    • Time - Every 0.05 seconds of game time
  • Conditions
  • Actions
    • Unit Group - Pick every unit in Frost_Group and do (Actions)
      • Loop - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • ((Picked unit) has buff Frost Slide ) Equal to True
            • ((Picked unit) is alive) Equal to True
            • (Mana of (Picked unit)) Greater than 5.00
          • Then - Actions
            • Set Frost_Move = ((Position of (Picked unit)) offset by 25.00 towards (Facing of (Picked unit)) degrees)
            • Unit - Move (Picked unit) instantly to Frost_Move
            • Custom script: call RemoveLocation(udg_Frost_Move)
            • Unit - Set mana of (Picked unit) to ((Mana of (Picked unit)) - 0.50)
          • Else - Actions
            • Unit Group - Remove (Picked unit) from Frost_Group
            • Unit - Remove Slow Frost from (Picked unit)
            • Unit - Remove Frost Slide buff from (Picked unit)

 

[Adiktuz]

this (Position of (Picked unit)) leaks...

anyway, pretty simple and doesn't even support damage... (and its hard for the user to add damage since you don't pick the units...)

and it does not support multiple spell levels...

For now:

-fix the leak
-add support for multiple spell levels
-(not really needed but a good addition)support for damage

until the first two are done, I currently Vote for rejection

 

[tigago]

How can i upload the triggers??
I think the leak is fixed now :B

 

[Patio_111]

How can i upload the triggers??
I think the leak is fixed now :B

You can post the triggers using the [TRIGGER.][/TRIGGER.] tags, and maybe hide them with [HIDDEN.][/HIDDEN.] tags (both without the period at the end). Just copy the trigger as text an paste them within the tags: select the trigger title in the trigger editor, above "Events", and go to Edit -> Copy as Text, or simply press Shift+Ctrl+C.

EDIT: Err... you posted a different spell.

 

[tigago]

• function Trig_Frost_Cast_Conditions takes nothing returns boolean
  • if ( not ( GetSpellAbilityId() == 'A000' ) ) then
    • return false
  • endif
  • return true
• endfunction
• function Trig_Frost_Cast_Actions takes nothing returns nothing
  • call GroupAddUnitSimple( GetTriggerUnit(), udg_Frost_Group )
  • call UnitAddAbilityBJ( 'A001', GetTriggerUnit() )
  • call SetUnitAbilityLevelSwapped( 'A001', GetTriggerUnit(), GetUnitAbilityLevelSwapped('A000', GetTriggerUnit()) )
• endfunction
• //===========================================================================
• function InitTrig_Frost_Cast takes nothing returns nothing
  • set gg_trg_Frost_Cast = CreateTrigger( )
  • call TriggerRegisterAnyUnitEventBJ( gg_trg_Frost_Cast, EVENT_PLAYER_UNIT_SPELL_EFFECT )
  • call TriggerAddCondition( gg_trg_Frost_Cast, Condition( function Trig_Frost_Cast_Conditions ) )
  • call TriggerAddAction( gg_trg_Frost_Cast, function Trig_Frost_Cast_Actions )
• endfunction

 

[tigago]

Fail? =D

 

[Patio_111]

You don't have to convert the trigger to custom text when posting it. Post them in the spell description.

Fail? =D

Hehehe, much better now

I think the leak is fixed now :B

Yes, the leak is fixed now.

PS: Use the Edit button.

 

[xorkatoss]

the spell is good but maybe when unit begins to cast the ability...
you should remove collision and maybe make the unit get a height (for example 500)
im only suggesting lol
anyway the spell is pretty good 3/5 for now, keep up the good work

 

[tigago]

the spell is good but maybe when unit begins to cast the ability...
you should remove collision and maybe make the unit get a height (for example 500)
im only suggesting lol
anyway the spell is very good, leakless and MUI 5/5 !!

About the height, it will look like Goku's flyng cloud

 

[Archangel678]

I'd add a check to see if the pathing is walkable, as if you run into a boundary it has an unique effect.

It could potentially be used by a tank-like unit, slowing them and/or forcing them to attack it before targeting weaker ranged units.


I'll give you a 3/5 for it being a first ability.
As well as a Vote for Approval

 

[Bribe]

I like it a lot.

Store (Picked unit) into a variable at the beginning of the periodic loop (so as to reference that instead) and this thing is good to go!