[Trigger] Not showing floating text, GetLocalPlayer + Loop

[Airandius]

Oke, long title but the question still is. Why wont it work?
It's this part that's being annoying.

• For each (Integer A) from 1 to 4, do (Actions)
  • Loop - Actions
    • Set Point_Hero[(Integer A)] = (Position of Unit_PlayerHero[(Integer A)])
    • Custom script: if (GetLocalPlayer()==Player(GetForLoopIndexA())) then
    • Floating Text - Create floating text that reads ((PlayerColor[(Integer A)] + ((Proper name of Unit_PlayerHero[(Integer A)]) + :|r Where am I?)) + RandomTextCasual[(Random integer number between 1 and 8)]) at TalkTargetPoint[(Player number of (Owner of (Triggering unit)))] with Z offset 0.00, using font size 8.00, color (100.00%, 100.00%, 100.00%), and 0.00% transparency
    • Floating Text - Change (Last created floating text): Disable permanence
    • Floating Text - Change the lifespan of (Last created floating text) to 5.00 seconds
    • Floating Text - Change the fading age of (Last created floating text) to 3.00 seconds
    • Floating Text - Set the velocity of (Last created floating text) to 15.00 towards (Random real number between 50.00 and 90.00) degrees
    • Custom script: endif

I think I made the GetLocalPlayer wrong, but I'm not sure.
Can somebody tell me why it ain't working?

 

[Dr Super Good]

Works fine for player 2 blue to player 5. Make sure you are testing as them since it will not work for player 1 red (Player(0)).

 

[Airandius]

Ah dang, it needs for player 1 to 4.
I need, getconvertedplayer or something then right?

Could you tell me how that one looks like, as every time I try to use it I end up with errors.

 

[Dr Super Good]

Loop from 0 to 3 then?

 

[Airandius]

Would be possible for this trigger yes, that I didn't think about it. Problem is that I it's often mixed with integers that are stored with 1 through 4.
There was a function to add 1, GetConvertedPlayer, but I don't know how to spell the exact function withing this function. =/

Still thanks though.

 

[Eleandor]

Don't use a function if you can simply subtract one...

Custom script: if (GetLocalPlayer()==Player(GetForLoopIndexA()-1)) then

The best solution still is looping from 0 to 3, like DSG said...

 

[Airandius]

I'm a sucker at jass.
Thanks, that's of great help to me.