Can someone Tell me what i did wrong in this spell ? I'm just started making one so be easy on me

[KiriyaNozomi]

 

[Rheiko]

First of all, you're posting in the wrong forum. You should start a thread here: Triggers & Scripts

To answer your question, local variables should be the very first actions before everything else. Move it to the very top. It should work.

 

[Pyrogasm]

All of your custom script stuff is entirely pointless for two reasons:

1. TSA (wait) will not sleep 0.03s on every iteration of the loop. The minimum is something like 0.27s so that loop will take about 10x longer to complete than you think it does. People use timers or periodic triggers for this sort of thing: Visualize: Dynamic Indexing

2. You stored a bunch of information in global variables before your cs wait loop bologna, and then refer to that data multiple times after multiple further waits. Even though your loop counter variable (i) is properly insulated from being overwritten by multiple simultaneous casts… none of your other variables are. This spell will break completely if multiple casts overlap temporally (not positionally).

 

[Nichilus]

local variables must be declared first in function body. In terms of GUI triggers, that means as first actions in the "Actions" block.