Sun Ray

[Flame_Phoenix]

wrong test map...

Arrghh, damn ! Sorry for that, well, here is CORRECT test map if I am not mistaken.
Please enjoy and give opinion.

[chobibo]

Yo Flame_Phoenix don't get offended please, here are my comments; The spell looks typical, A nova spell that works like holy light, Nova-like spells are common nowadays so it doesn't give you a " Yay cool! XD " feeling when you cast it. Now about your code, it looks very organized, lovely indeed. Also, handles stored on locals need only be nulled if they are dynamic.

Good Luck creating more spells dude, you've improved so much now.

[Flame_Phoenix]

The spell looks typical, A nova spell that works like holy light, Nova-like spells are common nowadays so it doesn't give you a " Yay cool! XD "

This is very far away from my idea of a Nova Spell.
Anyway, most spells are outdated and use Kattanas Handle Vars, so I guess this one can be used (and the main reason why I am submitting it) to do more stuff than just a Light Nova =)

Also, handles stored on locals need only be nulled if they are dynamic.

I don't get it, is there something I should update on this version to see it approved ?

Good Luck creating more spells dude, you've improved so much now.

Yey !! Thx xD I know I agree. Check out my Dark Lightning too, it is a very cool spell and it is more difficult that this one =)

Now about your code, it looks very organized, lovely indeed.

I am really happy you think that, because that is how I tried to make the code. Take that Anitarf and Griffen you newbs !!! =P

Anyway thx for your opinion, positive feedback is always appreciated.

PS: when will some one mod this spell again ?

[chobibo]

I don't get it, is there something I should update on this version to see it approved ?

lol no, I just wanted to tell you about those handles that are permanent during the game need not be nulled, it's not a big deal actually, it's good practice to null all local handles after use. Anyways forget about it lol.

EDIT: just to clear things up, try running these scripts:

an example of a dynamic handle:

scope Test initializer Init

private function Display takes nothing returns nothing
    local trigger t=CreateTrigger()
    call BJDebugMsg(I2S(H2I(t)))
    call DestroyTrigger(t)
endfunction

private function Init takes nothing returns nothing
    local trigger trig=CreateTrigger()

    call TriggerRegisterPlayerEvent(trig, Player(0), EVENT_PLAYER_END_CINEMATIC)
    call TriggerAddAction(trig, function Display)
endfunction

endscope

an example of a static\constant handle:

scope Test initializer Init

globals
    private constant trigger Trig=CreateTrigger()
endglobals

private function Display takes nothing returns nothing
    local trigger t=Trig
    call BJDebugMsg(I2S(H2I(t)))
endfunction

private function Init takes nothing returns nothing
    local trigger trig=CreateTrigger()

    call TriggerRegisterPlayerEvent(trig, Player(0), EVENT_PLAYER_END_CINEMATIC)
    call TriggerAddAction(trig, function Display)
endfunction

endscope

The first example will leak a reference id because it was not nulled and also because a new handle is being referenced each time it is ran. The second on the other hand will always return a constant id since it also refers to a constant handle regardless of the local variable being used to reference it. In short, handles that will be used throughout the game are safe to be referenced with local variables without nulling it after using it inside functions.

[Flame_Phoenix]

AHh thx for your enthusiasm in my work =)
+rep for being a nice guy!

[Anitarf]

An interesting option to add would be a boolean that allows the spell directions to adapt to the caster's facing, that would be particularly useful for versions of the spell with few long rays, like 4.

Anyway, the spell looks nice, approved.

[Flame_Phoenix]

This is in deed interesting, except for the fact I have no idea on how to do it lol...
I will try updating this as you requested, more news soon.

Anyway, thx fro approval =)

EDIT EDIT EDIT

Ok guys, this spell was updated to Version 1.5 !!
Now Anitarfs suggestion was added ! I am sure you will like the new changed, and I hope you all use this spell. =D
Credits if use please =P

[Flame_Phoenix]

Updated for patch 1.24. Only changed updated TimerUtils.

[KySoto]

the map appears to be corrupted.