Simple trigger, surprisingly, did not work ?!

Ryuu

I am back with Chocolate (:
Reaction score
64
This does nothing:

JASS:


Please help!
 

Ryuu

I am back with Chocolate (:
Reaction score
64
The whole trigger is way too long, but I will tell you some parts of it:

JASS:
private function AttackInit takes nothing returns nothing
    local integer i=0
    local trigger t=CreateTrigger()
    loop
        exitwhen i>12
            call TriggerRegisterPlayerUnitEvent(t,Player(i),EVENT_PLAYER_UNIT_SPELL_EFFECT,null)
        set i=i+1
    endloop
    call TriggerAddAction(t,function AttackEffects)
endfunction


The thing that isn't working is that nothing is done at all. I haven't done it with a BJDebugMsg yet, because the function is so short.
But the unit isn't created, and yes, I've checked three times that 'h000' does exist with a good model.

That's all I can show you.
 

Akolyt0r

New Member
Reaction score
33
i really dont see any error ...also i would use a local instead of that bj var...

maybe the SpellId is wrong ???
Or some other code of the spell crashs the thread ?
 

quraji

zap
Reaction score
144
Only thing I can think of is that your spell or unit IDs are faulty. Of course, both of those could be ruled out (and the problem most likely solved) by some quick testing on your part..;)

Try putting some debug messages in there, if there is still a problem post again.
 

Builder Bob

Live free or don't
Reaction score
249
It could be that Location(x,y) is actually outside of the map bounds, and most stuff done outside of the map bounds will either not work or crash the game.
 

Ryuu

I am back with Chocolate (:
Reaction score
64
I wrote 20000. because I want my unit to move to the end.
When it reaches the end, he gets killed.

IDs are all correct.
 
General chit-chat
Help Users
  • No one is chatting at the moment.

      The Helper Discord

      Members online

      Affiliates

      Hive Workshop NUON Dome World Editor Tutorials

      Network Sponsors

      Apex Steel Pipe - Buys and sells Steel Pipe.
      Top