addEventHandler warning

[.:HyPeX:.]

Hello guys i've done this way to check something, wich was handy, but i'm not sure how to avoid the debug error (It still works perfectly thought)

  
addCommandHandler("turnOffMapInfo", function() 
local event = addEventHandler("onClientRender",root, MapInfo) 
    if event ~= true then 
    removeEventHandler("onClientRender", root, MapInfo) 
    end 
end) 
bindKey("F3", "down", "turnOffMapInfo") 

[WASSIm.]

you have us this:

function isEventHandlerAdded( sEventName, pElementAttachedTo, func ) 
    if type( sEventName ) == 'string' and isElement( pElementAttachedTo ) and type( func ) == 'function' then 
        local aAttachedFunctions = getEventHandlers( sEventName, pElementAttachedTo ) 
        if type( aAttachedFunctions ) == 'table' and #aAttachedFunctions > 0 then 
            for i, v in ipairs( aAttachedFunctions ) do 
                if v == func then 
                    return true 
                end 
            end 
        end 
    end 
    return false 
end 

[Feche1320]

  
local event = false 
  
addCommandHandler("turnOffMapInfo",  
    function() 
        if not event then 
            addEventHandler("onClientRender", root, MapInfo) 
        else 
            removeEventHandler("onClientRender", root, MapInfo) 
        end 
        event = not event 
    end 
) 
bindKey("F3", "down", "turnOffMapInfo") 

[novo]

  
function isEventHandlerAdded (name, element, func) 
    if type(name) ~= "string" or not isElement(element) or type(func) ~= "function" then return end 
    for i, v in ipairs(getEventHandlers(name, element)) do 
        if v == func then return true end 
    end 
end 
-- 
addCommandHandler("turnOffMapInfo", 
    function() 
        local name = "onClientRender" 
        local element = root 
        local func = MapInfo 
        return ((isEventHandlerAdded(name, element, func) and removeEventHandler(name, element, func)) or addEventHandler(name, element, func)) 
    end 
) 
bindKey("F3", "down", "turnOffMapInfo") 
  

Either way, you shouldn't have used "event ~= true" that way since addEventHandler returns true if the event handler was attached successfully or returns false if the specified event could not be found or any parameters were invalid.

[Dealman]

There's no need to go that far over-the-top for such a simple thing, just use a variable - simple.

local isMapInfoActive = false; 
  
function mapInfo_Handler() 
    if(isMapInfoActive == false) then 
        addEventHandler("onClientRender", root, MapInfo); 
        isMapInfoActive = true; 
    else 
        removeEventHandler("onClientRender", root, MapInfo); 
        isMapInfoActive = false; 
    end 
end 
addCommandHandler("TurnOffMapInfo", mapInfo_Handler, false); -- False means it will not be case-sensitive 
bindKey("F3", "down", mapInfo_Handler); 

[Feche1320]

There's no need to go that far over-the-top for such a simple thing, just use a variable - simple.

local isMapInfoActive = false; 
  
function mapInfo_Handler() 
    if(isMapInfoActive == false) then 
        addEventHandler("onClientRender", root, MapInfo); 
        isMapInfoActive = true; 
    else 
        removeEventHandler("onClientRender", root, MapInfo); 
        isMapInfoActive = false; 
    end 
end 
addCommandHandler("TurnOffMapInfo", mapInfo_Handler, false); -- False means it will not be case-sensitive 
bindKey("F3", "down", mapInfo_Handler); 

That's what I did lol