Using GetLocalPlayer() to show multiboard issues

[LordXavier]

It took me a while to figure out how to successfully use the whole GetLocalPlayer() to show my multiboard for only one player. The first time i tried it, it didn't show up for me (a good thing), but then 1 sec into the game it just closed WC3 without any error messages or anything. The second time I changed so it would show for me, when WC3 opened, screen never got past being black and some "fatal error" thingy popped up. Does anybody have any idea as to how I'd fix this? (I'm a super JASS noob, I had to use WE to change a trigger into JASS then use GUI to run the trigger)

function Trig_Show_Multiboard_Actions takes multiboard Scoreboard returns nothing
    call MultiboardDisplay(Scoreboard,false)
    if GetLocalPlayer() == Player(3) then
        call MultiboardDisplay(Scoreboard,true)
    endif
endfunction

//===========================================================================
function InitTrig_Show_Multiboard takes nothing returns nothing
    set gg_trg_Show_Multiboard = CreateTrigger(  )
    call TriggerAddAction( gg_trg_Show_Multiboard, function Trig_Show_Multiboard_Actions )
endfunction

 

[Trollvottel]

Dont do it (creating and showing the board) on Map Init, do it when time elapsed is 0.01.

 

[LordXavier]

it creates/shows 1 second in. map is fine for one second

 

[saw792]

function Trig_Show_Multiboard_Actions takes multiboard Scoreboard returns nothing

Action functions cannot take arguments. You will need to set Scoreboard as a global variable.

 

[Sevion]

function Trig_Show_Multiboard_Actions takes multiboard Scoreboard returns nothing

Action functions cannot take arguments. You will need to set Scoreboard as a global variable.

call TriggerAddAction( gg_trg_Show_Multiboard, function Trig_Show_Multiboard_Actions )

He's not even trying to pass an argument when he calls it '

That's a flashing red light to me.

If you have something that requires something but you don't give it, doesn't that fire an alarm saying "No, that won't work" in your head?

 

[LordXavier]

BUMP

That bit of trigger was made by WE and me copy and pasting. I'm quite clueless, especially to what you guys are talking about. are you saying it should take nothing return nothing?

 

[Sevion]

Well, how about we start off with that trigger won't even fire. ^_^ No event, and more than likely, no other funcitons call it.

 

[LordXavier]

of course there's no event

I had to use WE to change a trigger into JASS then use GUI to run the trigger)

The trigger I use to make the multiboard has this at the end:

• Trigger - Run Show Multiboard <gen> (checking conditions)

And it does run, it just crashed the game... <-----crashing part is what I need help with

 

[Sevion]

Why don't you just do:

For (Integer A) from 1 to 12 do (Multiple Actions):
    <Hide Multiboard for Player (Integer A)>
Set PLAYERVAR = Player Whatever
Custom Script:    if GetLocalPlayer() == udg_PLAYERVAR then
<Display Multiboard>
Custom Script:    endif

Hand written, you can figure it out though, I bet.

 

[kingkingyyk3]

function Trig_Show_Multiboard_Actions takes multiboard Scoreboard returns nothing
    local boolean show = false
    if GetLocalPlayer() == Player(3) then
        set show = true
    endif
    call MultiboardDisplay(Scoreboard,show)
endfunction

Will this desync?

 

[Sevion]

No.

 

[kingkingyyk3]

Use it. lol

 

[Jesus4Lyf]

call MultiboardDisplay(udg_Scoreboard,GetLocalPlayer()==Player(3))

One liner. Jam it into your GUI. Set a var named "Scoreboard" to your multiboard (case sensitive). Finished.

 

[Terrabull]

And it does run, it just crashed the game...

Yes, and as people stated previously in the thread an action can't take any arguments.
You are doing it wrong anyways. If you want it to be something you call from GUI put it in the text area you get when you click on the map name.
Then call it with "Custom Script", that way you can pass a variable to it without crashing Warcraft.
Right now you are trying to do two different things, and that combination will crash the game every time.

 

[LordXavier]

I don't even know what you all are talking about when you say "arguments". I just know it deals with Jass

@ Jesus4Lyf, it doesn't show up

 

[LordXavier]

I finally figured it out! It worked when I took this

    call MultiboardDisplay(Scoreboard,false)
    if GetLocalPlayer() == Player(3) then
        call MultiboardDisplay(Scoreboard,true)
    endif

and just put it into custom script. Also, the first player is teal, 3 isn't teal in Jass (I'm learning Jass currently)

 

[TriggerHappy]

Players in JASS start at 0, not 1.

 

[kingkingyyk3]

Inline it like this :

call MultiboardDisplay(Scoreboard,GetLocalPlayer() == Player(3))