Help me in syntax

[goofie88]

function allowORnot (player, key) 
    if isObjectInACLGroup("user."..getAccountName( getPlayerAccount( player ) ), aclGetGroup( "Pirate" ) ) then 
        if (key = nil) then 
        key = 0 
        removeEventHandler("onVehicleStartEnter", getRootElement(), pirateBoat) 
        end 
        elseif (key = 0) then 
        addEventHandler ( "onVehicleStartEnter", getRootElement(), pirateBoat) 
        key = nil 
    end 
end 
addCommandHandler ("cannon", allowORnot) 

I don't see what I'm doing wrong, is it my syntax?

Error: Loading script failed restriction.lua:3: ')' expected near '=' 

[csiguusz]

    function allowORnot (player, key) 
        if isObjectInACLGroup("user."..getAccountName( getPlayerAccount( player ) ), aclGetGroup( "Pirate" ) ) then 
            if (key == nil) then 
            key = 0 
            removeEventHandler("onVehicleStartEnter", getRootElement(), pirateBoat) 
            end 
            elseif (key == 0) then 
            addEventHandler ( "onVehicleStartEnter", getRootElement(), pirateBoat) 
            key = nil 
        end 
    end 
    addCommandHandler ("cannon", allowORnot) 

[goofie88]

thanks(no errors now) but the command seems to not respond, added outputChatBox after key = 0 and key = nil and nothing

im trying to figure out my mistake but i dont even imagine what the error could be

[فاّرس]

What you want to doing?

[goofie88]

i want it with a command to remove and add events

removeEventHandler("onVehicleStartEnter", getRootElement(), pirateBoat) 
addEventHandler ( "onVehicleStartEnter", getRootElement(), pirateBoat) 

like a toggable command not cannonopen, cannonclose

[فاّرس]

Can you tell me (key = ?)

[goofie88]

key is nothing its just so i can use it as a toggle

[فاّرس]

I'm don't know what is pirateBoat ?

post your code.

[goofie88]

key is like cur, if you know what i mean. I wont post full code coz it wont rly fix. All im trying to do is a toggable command to add and remove the handler

[PaiN^]

csiguusz's code is fine .

Show us the 'pirateBoat' function .

[goofie88]

    function allowORnot (player, key) 
outputChatBox("check") 
        if isObjectInACLGroup("user."..getAccountName( getPlayerAccount( player ) ), aclGetGroup( "Pirate" ) ) then 
            if (key == nil) then 
            key = 0 
outputChatBox("go") 
            removeEventHandler("onVehicleStartEnter", getRootElement(), pirateBoat) 
            end 
            elseif (key == 0) then 
            addEventHandler ( "onVehicleStartEnter", getRootElement(), pirateBoat) 
            key = nil 
outputChatBox("stop") 
        end 
    end 
    addCommandHandler ("cannon", allowORnot) 

csiguusz's code isn't fine , tried with checks, it "check" then "go" but doesnt go to "stop" and after that it just "check"

[Castillo]

The second argument of addCommandHandler is the command name, not "nil" nor a number.

[goofie88]

so is there a way I can make it toggable? I understood what you said Solidsnake14 but it didnt rly help me

[Castillo]

You can use a table.

local state = { } 
  
function allowORnot ( player ) 
    outputChatBox ( "check" ) 
    if isObjectInACLGroup ( "user.".. getAccountName ( getPlayerAccount ( player ) ), aclGetGroup( "Pirate" ) ) then 
        if ( state [ player ] ) then 
            state [ player ] = nil 
            outputChatBox ( "go" ) 
            removeEventHandler ( "onVehicleStartEnter", getRootElement(), pirateBoat ) 
        else 
            state [ player ] = true 
            addEventHandler ( "onVehicleStartEnter", getRootElement(), pirateBoat ) 
            outputChatBox ( "stop" ) 
        end 
    end 
end 
addCommandHandler ( "cannon", allowORnot ) 

[goofie88]

Wow thank you Solidsnake that's exactly what i wanted thanks for you help!!

[Castillo]

You're welcome.