Always getting error + not working at same time

[DarkByte]

Hello...i wonder why this script is giving me errors and :~..It seems fine to me. Maybe you can help me out  

Spoiler

 local gs_ArmedVehicles = {[425] = true,[447] = true,[520] = true,[430] = false,[464] = false,[432] = false}
 
function checkNoDMGreen()
    local vehicles = getElementsByType ( "vehicle" )
	local Players = getElementsByType ( "player" )
    for _, vehicle in ipairs(vehicles) do
	for i, thePlayer in ipairs(Players) do
			                local player = getVehicleOccupant(vehicle,0)
		local noDm = getElementData(thePlayer,"invincible",true) or getElementData(thePlayer,"green",true)
        if gs_ArmedVehicles[getElementModel(vehicle)] then
		if noDm or green then
		
                    -- check elementData here then usetoggleControl
                    toggleControl(player, 'vehicle_fire', false)
                    toggleControl(player, 'vehicle_secondary_fire', false)
					else
					toggleControl(player, 'vehicle_fire', true)
                    toggleControl(player, 'vehicle_secondary_fire', true)
            end
        end
    end
end
end
setTimer(checkNoDMGreen,100,0)

 

 

Edited May 2, 2017 by DarkByte

[NeXuS™]

local gs_ArmedVehicles = {[425] = true,[447] = true,[520] = true,[430] = false,[464] = false,[432] = false}
 
function checkNoDMGreen()
	local playerT = getElementsByType("player")
	for i, thePlayer in ipairs(playerT) do
		if getPedOccupiedVehicle(thePlayer) and getPedOccupiedVehicleSeat(thePlayer) == 0 then
		local noDm = getElementData(thePlayer,"invincible") or getElementData(thePlayer,"green")
		if gs_ArmedVehicles[getElementModel(getPedOccupiedVehicle(thePlayer))] then
			if noDm or green then
				-- check elementData here then usetoggleControl
				toggleControl(thePlayer, 'vehicle_fire', false)
				toggleControl(thePlayer, 'vehicle_secondary_fire', false)
			else
				toggleControl(thePlayer, 'vehicle_fire', true)
				toggleControl(thePlayer, 'vehicle_secondary_fire', true)
			end
		end
	end
end
setTimer(checkNoDMGreen,100,0)

I think you want this to happen.

Edited May 2, 2017 by NeXuS™

[DarkByte]

Thanks buddeh