[help] Bad argument

[#RooTs]

because you are giving bad argument?

I'm trying to do, if the player is the VIP group, then display the icon in DX

server.lua -- bad argument

local acls = { "VIP"  } 
  
function CarVehiclePainelVip(source) 
    local accName = getAccountName ( getPlayerAccount ( source ) ) 
    for _, k in ipairs ( acls ) do 
        if isObjectInACLGroup ("user."..accName, aclGetGroup(k)) then 
        setElementData(source,"VehiclePainel",true)  
        else 
        setElementData(source,"VehiclePainel",false) 
        end 
    end 
end 
addEventHandler ( "onResourceStart", getRootElement(), CarVehiclePainelVip ) 

client.lua

        if getElementData( source, 'VehiclePainel' ) then 
        dxDrawImage(botX+sizeX+650, 250, 49, 39, "img/icoRepair.png") 
        end 

[neves768]

    
-- Server-side   
local acls = {"VIP"} 
  
function CarVehiclePainelVip() 
    local accName = getAccountName(getPlayerAccount(source)) 
    for _, k in ipairs(acls) do 
        if isObjectInACLGroup("user."..accName, aclGetGroup(k)) then 
            setElementData(source,"VehiclePainel",true) 
        end 
    end 
end 
addEventHandler("onPlayerVehicleEnter", getRootElement(), CarVehiclePainelVip) 
  
-- Client-side 
if getElementData(getLocalPlayer(), "VehiclePainel") then 
    dxDrawImage(botX+sizeX+650, 250, 49, 39, "img/icoRepair.png") 
end 
  

[KariiiM]

Why you looping and you have only one value on table.

[tosfera]

It's a mistake you keep on making #RooTs, you're trying to replace the default source variable with a function variable. Stop doing that since it'll only cause a lot of trouble.

Also, KariiiM is totally right. You're looping for a table which will only take more memory that required. Try something like this;

addEventHandler ( "onPlayerVehicleEnter", getRootElement(), 
    function () 
        local account = getPlayerAccount ( source ); 
        if ( account ) then 
            if ( isObjectInACLGroup ( "user.".. getAccountName ( account ), aclGetGroup ( "VIP" ) ) ) then 
                setElementData ( source, "VehiclePainel", true ); 
            end 
        end 
    end 
); 

[#RooTs]

It's a mistake you keep on making #RooTs, you're trying to replace the default source variable with a function variable. Stop doing that since it'll only cause a lot of trouble.

Also, KariiiM is totally right. You're looping for a table which will only take more memory that required. Try something like this;

addEventHandler ( "onPlayerVehicleEnter", getRootElement(), 
    function () 
        local account = getPlayerAccount ( source ); 
        if ( account ) then 
            if ( isObjectInACLGroup ( "user.".. getAccountName ( account ), aclGetGroup ( "VIP" ) ) ) then 
                setElementData ( source, "VehiclePainel", true ); 
            end 
        end 
    end 
); 

LOL. thanks, guys