[Trigger] Moeveing unit instantly to point

[ALiEN95]

• move unit for 500 range
  • Events
    • Unit - A unit Is issued an order targeting a point
  • Conditions
  • Actions
    • Unit - Move (Ordered unit) instantly to (+500 range to target point of issued order)

i just want this work like blink ability with cast range 500

 

[Anti_Bodies]

what do you mean? do you want to create another blink?

 

[UreDe4D]

• Events
  • Unit - A unit Is issued an order targeting a point
• Conditions
• Actions
  • Set temppoint = ((Position of (Triggering unit)) offset by 500.00 towards (Angle from (Position of (Triggering unit)) to (Target point of issued order)) degrees)
  • Unit - Move (Ordered unit) instantly to (temppoint)
  • Custom script: call RemoveLocation(udg_temppoint)

like this?

 

[darkrider]

urede4d: you are leaking position of triggering unit and target point of issued order

 

[NoAmmoSrry]

is he? I thought it was solid

 

[UreDe4D]

i'm not sure about that either, but if so you better clean the leaks as i guess the trigger fires pretty often in your map, solution:

• Events
  • Unit - A unit Is issued an order targeting a point
• Conditions
• Actions
  • Set temppoint1 = ((Position of (Triggering unit))
  • Set temppoint2 = ((Target point of issued order))
  • Set temppoint = ((temppoint1) offset by 500.00 towards (Angle from (temppoint1) to (temppoint2)) degrees)
  • Unit - Move (Ordered unit) instantly to (temppoint)
  • Custom script: call RemoveLocation(udg_temppoint)
  • Custom script: call RemoveLocation(udg_temppoint1)
  • Custom script: call RemoveLocation(udg_temppoint2)

i hate leaks :/

 

[darkrider]

yeah... they are tricky :/

Now it is leakless (Y)

 

[ALiEN95]

• Events
  • Unit - A unit Is issued an order targeting a point
• Conditions
• Actions
  • Set temppoint = ((Position of (Triggering unit)) offset by 500.00 towards (Angle from (Position of (Triggering unit)) to (Target point of issued order)) degrees)
  • Unit - Move (Ordered unit) instantly to (temppoint)
  • Custom script: call RemoveLocation(udg_temppoint)

like this?

y..i think this trigger has no leaks

i don´t know do leak exist at all

i mean,do you really think when you have blink ability and that (blizzard) trigger will set 3 points,move unit,and remove them? -.-

 

[Saia_Djinn]

No just Post #6 is location leakfree the others leak 2 locs

 

[BaraBura]

i'm not sure about that either, but if so you better clean the leaks as i guess the trigger fires pretty often in your map, solution:

• Events
  • Unit - A unit Is issued an order targeting a point
• Conditions
• Actions
  • Set temppoint1 = ((Position of (Triggering unit))
  • Set temppoint2 = ((Target point of issued order))
  • Set temppoint = ((temppoint1) offset by 500.00 towards (Angle from (temppoint1) to (temppoint2)) degrees)
  • Unit - Move (Ordered unit) instantly to (temppoint)
  • Custom script: call RemoveLocation(udg_temppoint)
  • Custom script: call RemoveLocation(udg_temppoint1)
  • Custom script: call RemoveLocation(udg_temppoint2)

i hate leaks :/

Dude you hate leaks, but this also leaks...
Check this out:

• Events
  • Unit - A unit Is issued an order targeting a point
• Conditions
• Actions
  • Set temppoint1 = ((Position of (Triggering unit))
  • Set temppoint2 = ((Target point of issued order))
  • Set temppoint = ((temppoint1) offset by 500.00 towards (Angle from (temppoint1) to (temppoint2)) degrees)
  • Unit - Pause (Ordered Unit)
  • Unit - Move (Ordered unit) instantly to (temppoint)
  • Unit - Unpause (Ordered Unit)
  • Custom script: call RemoveLocation(udg_temppoint)
  • Custom script: call RemoveLocation(udg_temppoint1)
  • Custom script: call RemoveLocation(udg_temppoint2)

You must first Pause the unit before you move it, else ability cooldowns will malfunction and be instantly refreshed. Once you moved the unit, unpause it.

 

[Saia_Djinn]

his function is except the pause not any other then yours and tthe pause dont leaks, his was already leakless
but i never got your bug till yet i think the pause thing is not necessary

 

[darkrider]

y..i think this trigger has no leaks

i don´t know do leak exist at all

i mean,do you really think when you have blink ability and that (blizzard) trigger will set 3 points,move unit,and remove them? -.-

Every time you refer to a location, if you dont remove it, it leaks... always. And yes, i do else it will leak

 

[ALiEN95]

Every time you refer to a location, if you dont remove it, it leaks... always. And yes, i do else it will leak

well i once tested do locations leak,i had about 500 locations not removing it or setting to a variable and nothing happens.

i only saw special effect can leak if you have 500+ not destroyed special effects in SAME place,in that place you will have "big sky" of that effect and you will not see because of that "sky",but it makes no lags

 

[Super-Sheep]

The ability cooldown malfunction is a lie.

 

[ALiEN95]

The ability cooldown malfunction is a lie.

yes i tough so

 

[toofless]

BTW if you want to replicate the original blink you will want to add a pathing check which cancels actions and returns a message if not pathable

I just spent 1 hour trying to do this in Jass and replacing locations with x and y. Now I have a headache and a malfunctioning trigger. LOL fun:

JASS

function Blink takes nothing returns nothing
    local unit u = GetTriggerUnit()
    local location lt = GetSpellTargetLoc()
    local real xt = GetLocationX(lt)
    local real yt = GetLocationY(lt)
    local real xu = GetUnitX(u)
    local real yu = GetUnitY(u)
    local real xd = xt - xu
    local real yd = yt - yu
    local real d = SquareRoot(xd * xd + yd * yd)
    local real a = (180 / 3.14159) * Atan2(yt - yu, xt - xu)
    local real xa = Cos(a) * 500 + xu //Probably the problem
    local real ya = Sin(a) * 500 + yu //Probably the problem
        if (GetSpellAbilityId() == 'A000' ) then
            if (IsTerrainPathable( xt, yt, PATHING_TYPE_WALKABILITY) == false) then
                call DisplayTextToPlayer( GetOwningPlayer(u), 0, 0, "Cannot target there")
            elseif ( d > 500 ) then
                if (IsTerrainPathable( xa, ya, PATHING_TYPE_WALKABILITY) == false) then
                    call DisplayTextToPlayer( GetOwningPlayer(u), 0, 0, "Cannot target there")
                else
                    call SetUnitX( u, xa)
                    call SetUnitY( u, ya)
                endif
            else
                call SetUnitX( u, xt)
                call SetUnitY( u, yt)
            endif
        endif
    call RemoveLocation(lt)
    set lt = null
    set u = null
endfunction

 

[Saia_Djinn]

Also with set Instantly a unit will not be moved into deep water etc. coz the collision is still activated and the unit will then be setted to the nearest walkable point