Spell Jump Spell (GUI & JASS)

[shadowcon]

Posted in the wrong section, could any moderator please move it to "member's projects"?

If it is completed or not, it is up to you...

Leakhelp and comments please.
Dummy ability for jump is Channel, icon detonate.
Crow is a spellbook with the Crow Form ability in it

Jump Init

Spoiler

Jump Init GUI
Events
Unit - A unit Learns a skill
Conditions
(Learned Hero Skill) Equal to Jump
Actions
Set Owner_of_JU = (Owner of Jump_Unit)
Set Jump_Unit = (Triggering unit)
Unit - Add Crow to Jump_Unit
Player - Disable Crow for (Owner of Jump_Unit)

The maintrigger:

Spoiler

Jump GUI
Events
Unit - A unit Starts the effect of an ability
Conditions
(Ability being cast) Equal to Jump
Actions
Set Jump_Unit = (Triggering unit)
Set Loc_JU = (Position of Jump_Unit)
Set Loc_PoAbc = (Target point of ability being cast)
Set Value[0] = ((Distance between Loc_JU and Loc_PoAbc) / 1200.00)
Selection - Remove Jump_Unit from selection for Owner_of_JU
Trigger - Turn on Unselection GUI <gen>
Trigger - Turn on Check Test Selec <gen>
Unit - Turn collision for Jump_Unit Off
Unit - Set Jump_Unit movement speed to 300.00
Unit - Make Jump_Unit Invulnerable
Animation - Change Jump_Unit flying height to (400.00 x Value[0]) at 450.00
Wait (0.20 x Value[0]) seconds
Unit - Order Jump_Unit to Move To Loc_PoAbc
Wait (0.80 x Value[0]) seconds
Animation - Change Jump_Unit flying height to (600.00 x Value[0]) at 150.00
Wait (1.00 x Value[0]) seconds
Animation - Change Jump_Unit flying height to (400.00 x Value[0]) at 150.00
Wait (1.00 x Value[0]) seconds
Animation - Change Jump_Unit flying height to 0.00 at 450.00
Wait (1.00 x Value[0]) seconds
Unit - Set Jump_Unit movement speed to (Default movement speed of Jump_Unit)
Unit - Turn collision for Jump_Unit On
Animation - Reset Jump_Unit's animation
Unit - Cause Jump_Unit to damage circular area after 0.00 seconds of radius (50.00 + (50.00 x (Real((Level of Jump for Jump_Unit))))) at Loc_PoAbc, dealing (400.00 x Value[0]) damage of attack type Spells and damage type Magic
Unit - Make Jump_Unit Vulnerable
Trigger - Turn off Unselection GUI <gen>
Custom script: call RemoveLocation(udg_Loc_PoAbc)
Custom script: call RemoveLocation(udg_Loc_JU)
Trigger - Turn off Check Test Selec <gen>
If (All Conditions are True) then do (Then Actions) else do (Else Actions)
If - Conditions
Sel Equal to True
Then - Actions
------------------Do nothing
Else - Actions
-------------------------Selection - Add Jump_Unit to selection for Owner_of_JU
Set Sel = False

Unselection Trigger

Spoiler

Unselection GUI
*Initially off*
Events
Player - Player 1 (Red) Selects a unit
Player - Player 2 (Blue) Selects a unit
Player - Player 3 (Teal) Selects a unit
Player - Player 4 (Purple) Selects a unit
Player - Player 5 (Yellow) Selects a unit
Player - Player 6 (Orange) Selects a unit
Player - Player 7 (Green) Selects a unit
Player - Player 8 (Pink) Selects a unit
Player - Player 9 (Gray) Selects a unit
Player - Player 12 (Brown) Selects a unit
Player - Player 10 (Light Blue) Selects a unit
Player - Player 11 (Dark Green) Selects a unit
Conditions
Actions
Selection - Remove Jump_Unit from selection for Owner_of_JU

Check Test Selec Trigger

Spoiler

*Initially Off*
Check Test Selec
Events
Player - Player 1 (Red) Selects a unit
Conditions
(Triggering Unit) (Not equal to) Jump_Unit
Actions
Set Sel = True

Made by me and my cousin, credit us if you use it in your map.
Changelog:
21 July 2008:
Posted it.
22 July 2008:
Fixed the leaks that Draphoelix showed me (I think)

 

[manofsteel]

That's not Jass it's just the converted GUI script. If you replace the globals with locals and fix the BJ's this should be better.

 

[Draphoelix]

Leaks

Jump GUI
Events
Unit - A unit Starts the effect of an ability
Conditions
(Ability being cast) Equal to Jump
Actions
Set Jump_Unit = (Triggering unit)
Set Value[0] = ((Distance between [B](Position of Jump_Unit)[/B] and [B](Target point of ability being cast))[/B] / 1200.00)
Selection - Remove Jump_Unit from selection for Owner_of_JU
Trigger - Turn on Unselection JASS <gen>
Unit - Turn collision for Jump_Unit Off
Unit - Set Jump_Unit movement speed to 300.00
Unit - Make Jump_Unit Invulnerable
Animation - Change Jump_Unit flying height to (400.00 x Value[0]) at 450.00
Wait (0.20 x Value[0]) seconds
Unit - Order Jump_Unit to Move To [B](Target point of ability being cast)[/B]
Wait (0.80 x Value[0]) seconds
Animation - Change Jump_Unit flying height to (600.00 x Value[0]) at 150.00
Wait (1.00 x Value[0]) seconds
Animation - Change Jump_Unit flying height to (400.00 x Value[0]) at 150.00
Wait (1.00 x Value[0]) seconds
Animation - Change Jump_Unit flying height to 0.00 at 450.00
Wait (1.00 x Value[0]) seconds
Unit - Set Jump_Unit movement speed to (Default movement speed of Jump_Unit)
Unit - Turn collision for Jump_Unit On
Animation - Reset Jump_Unit's animation
Unit - Cause Jump_Unit to damage circular area after 0.00 seconds of radius (50.00 + (50.00 x (Real((Level of Jump for Jump_Unit))))) at [B](Position of Jump_Unit)[/B], dealing (400.00 x Value[0]) damage of attack type Spells and damage type Magic
Unit - Make Jump_Unit Vulnerable
Trigger - Turn off Unselection JASS <gen>
Selection - Add Jump_Unit to selection for Owner_of_JU

And why did you make it JASS?

 

[shadowcon]

I don't have time to fix that today, no computer time left will have to fix that tomorrow
Removing the Converted GUI until fixed...

 

[Ultramen]

Cool spell.

 

[emjlr3]

this must look terrible in game

 

[Forty]

this must look terrible in game

does

 

[Fluffball]

I bet I could probably make a better version of this spell easily, all you have to do is make 3 For Each Integer A from 1 to 33 (???) and make the waits and flying hight different, to make it look a lot better, also making another integer variable that increases, then decreases, to symbolize the ascent, climax, and descent. You will have to do plus, then for the second one you will make it + a little bit, then a couple of waits, then - off, the third one will be minusing till you get to 0, or default flying height.

 

[Larcenist]

I bet I could probably make a better version of this spell easily, all you have to do is make 3 For Each Integer A from 1 to 33 (???) and make the waits and flying hight different, to make it look a lot better, also making another integer variable that increases, then decreases, to symbolize the ascent, climax, and descent. You will have to do plus, then for the second one you will make it + a little bit, then a couple of waits, then - off, the third one will be minusing till you get to 0, or default flying height.

The bolded underlined part made you fail already...

You can never ever use waits for sliding, period.

 

[Fluffball]

If you dont move waits in a for each integer, then won't it occur instantly?

 

[Forty]

thats the reason for using periodic events/timers