fix(main/targets): fix clearing targets when its no longer needed

- only need to clear targeting on the end of the broadcast

- don't set radio/call channels on removal, its already set elsewhere (whoops), fixes race condition with net events changing the channel

Author: AvarianKnight

client/main.lua

@@ -76,18 +76,15 @@ end
 ---@param targets table expects multiple tables to be sent over
 function playerTargets(...)
 	local targets = {...}
-	local addedPlayers = {}
-
-	MumbleClearVoiceTarget(1)
-    MumbleClearVoiceTargetPlayers(1)
-    MumbleSetVoiceTarget(0)
-    MumbleSetVoiceTarget(1)
-	currentGrid = 0
+	local addedPlayers = {
+		[playerServerId] = true
+	}
 
 	for i = 1, #targets do
 		for id, _ in pairs(targets[i]) do
-			if id == playerServerId or addedPlayers[id] then
-				debug.verbose(('[main] %s is already target (or self) don\'t re-add'):format(id))
+			-- we don't want to log ourself, or listen to ourself
+			if addedPlayers[id] and id ~= playerServerId then
+				debug.verbose(('[main] %s is already target don\'t re-add'):format(id))
 				goto skip_loop
 			end
 			if not addedPlayers[id] then

client/module/phone.lua

@@ -46,7 +46,6 @@ RegisterNetEvent('pma-voice:removePlayerFromCall', function(plySource)
 				toggleVoice(tgt, false, 'phone')
 			end
 		end
-		callChannel = 0
 		callData = {}
 		playerTargets(radioPressed and radioData or {}, callData)
 	else

client/module/radio.lua

@@ -54,7 +54,6 @@ function removePlayerFromRadio(plySource)
 			end
 		end
 		radioData = {}
-		radioChannel = 0
 		playerTargets({}, NetworkIsPlayerTalking(PlayerId()) and callData or {})
 	else
 		radioData[plySource] = nil