Why when i click on the PED it output double message

[Hero192]

Why when i click on the ped it output to the chat a double messages not one

addEventHandler ( 'onClientClick', root, 
    function ( button, state, absoluteX, absoluteY, worldX, worldY, worldZ, clickedElement ) 
        if  ( clickedElement ) and ( getElementType ( clickedElement )  == 'ped' ) then 
        for i=1, #deliverys do 
                if  ( clickedElement == deliverys[i] ) then 
         local x, y, z = getElementPosition(localPlayer) 
        if getDistanceBetweenPoints3D(x, y, z, worldX, worldY, worldZ)<=5 then 
        if ( getElementData ( localPlayer, "Job" ) ~= 'Criminal' ) then  outputChatBox(" You must be criminal.",255,0,0) return end 
                setElementData(deliverys[i], "del", true) 
                if getElementData(deliverys[i],"del") then 
                    if ( guiGetVisible ( Window ) == false ) then 
                     guiSetVisible ( Window,true )   
                    end 
                 end 
              end 
           end 
        end 
    end 
end) 

[TAPL]

1- You may have duplicate row in the table 'deliverys'.

2- You may have duplicate code.

It is unlikely to be the loop cause as you have used return within the output and this will stop the loop and the function together which means there's duplicated code.

[darhal]

check the state

  
addEventHandler ( 'onClientClick', root, 
    function ( button, state, absoluteX, absoluteY, worldX, worldY, worldZ, clickedElement ) 
        if  state and ( clickedElement ) and ( getElementType ( clickedElement )  == 'ped' ) then 
        for i=1, #deliverys do 
                if  ( clickedElement == deliverys[i] ) then 
         local x, y, z = getElementPosition(localPlayer) 
        if getDistanceBetweenPoints3D(x, y, z, worldX, worldY, worldZ)<=5 then 
        if ( getElementData ( localPlayer, "Job" ) ~= 'Criminal' ) then  outputChatBox(" You must be criminal.",255,0,0) return end 
                setElementData(deliverys[i], "del", true) 
                if getElementData(deliverys[i],"del") then 
                    if ( guiGetVisible ( Window ) == false ) then 
                     guiSetVisible ( Window,true )  
                    end 
                 end 
              end 
           end 
        end 
    end 
end)