Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Muting other spectators does not work #372

Closed
ElTyranos opened this issue Jun 7, 2017 · 11 comments
Closed

Muting other spectators does not work #372

ElTyranos opened this issue Jun 7, 2017 · 11 comments
Assignees
Labels
Milestone

Comments

@ElTyranos
Copy link

ElTyranos commented Jun 7, 2017

Arma 3 Version: 1.71
CBA Version: 3.3.1.170504
ACRE2 Version: 2.4.2.962

Mods:
-mod=@CBA_A3;@CUP_Terrains_Core;@BG21_Core;@BG21_IF;@ace;@fow;@IFA3_AIO_LITE;@acre2;

Description:

  • Muting other spectators does not work.

Steps to reproduce:

  • Get in PvP
  • Die, moved to spectator (with onPlayerKilled.sqf > [true] call acre_api_fnc_setSpectator)
  • press the "mute spectator" shortcut
  • Nothing happens and other spectator keep yelling

Where did the issue occur? Dedicated

Placed Modules: None of ACRE. ACRE enabled via init.sqf > [true, true] call acre_api_fnc_setupMission;

RPT log file: https://paste.ubuntu.com/24799931/

NB :
BG21_Core = Bloodsplatter Lite ;L_MOUNT ; SAN_Headlamps ; Zade_boc ;
BG21_IF = WMT mission framework + compatibility files (IF/ACE/FOW)
https://github.com/iEzhuk/WMT_MapTools

@Sniperhid
Copy link
Member

press the "mute spectator" shortcut This sounds like an issue with the spectator system you are using? It's probably blocking the keybind from going through.

Which spectator system are you using?

@jonpas
Copy link
Member

jonpas commented Jun 7, 2017

Is that mute spectator a function of spectator you are using? Vanilla or ACE3?

@ElTyranos
Copy link
Author

image

Indeed @Sniperhid that's the mute spectator shortcut. Unless I did not understand how it works.

We use vanilla spectator mode. Check out https://github.com/ElTyranos/BG21_PvP/blob/master/onPlayerKilled.sqf

@jonpas jonpas added bug and removed need more info labels Jun 7, 2017
@jonpas jonpas added this to the 2.5.0 milestone Jun 7, 2017
@ElTyranos
Copy link
Author

CBATeam/CBA_A3#694
CBATeam/CBA_A3#692

Could be related to these ?

@jonpas
Copy link
Member

jonpas commented Jun 19, 2017

Maybe vanilla spectator doesn't pass keybinds through since it got reworked.

@ElTyranos
Copy link
Author

ElTyranos commented Jul 19, 2017

adding

	[] spawn
	{
		findDisplay 60492 displayAddEventHandler ["KeyDown",
			{
				if (((_this select 1) == 62) && (!dialog)) then
				{
				   [] call acre_sys_core_fnc_toggleHeadset;
				};
				false
			}
		];
	};

to onplayerkilled.sqf fixes the issue (assigns a new key)

@jonpas
Copy link
Member

jonpas commented Jul 19, 2017

What is that display 60492? Vanilla spectator?

@ElTyranos
Copy link
Author

Yup

@jonpas jonpas modified the milestones: 2.5.0, 2.6.0 Sep 7, 2017
@jonpas jonpas modified the milestones: 2.6.0, 2.5.1 Nov 5, 2017
@jonpas
Copy link
Member

jonpas commented Nov 5, 2017

class Extended_DisplayLoad_EventHandlers {
    class RscDisplayEGSpectator {
        ADDON = QUOTE(_this call COMPILE_FILE(XEH_spectatorEGDisplayLoad));
    };
};
params ["_display"];

_display displayAddEventHandler ["KeyDown", {_this call FUNC(handleKeyDown)}];
params ["", "_key", "_shift", "_ctrl", "_alt"];

if ([_key, [_shift, _ctrl, _alt]] in ((["ACRE2", "HeadSet"] call CBA_fnc_getKeybind) select 8)) then {
    [] call acre_sys_core_fnc_toggleHeadset;
};

false

This should solve it for vanilla spectator, and we can make it an API function so modded spectators can also call it if they block keys for some reason. Will PR later/tomorrow.

I am assuming this doesn't happen with ACE3 spectator?

@VyMajoris
Copy link

It seems that this is still an issue. Now pressing the key to to toggle spectators makes triggers two events, one immediately followed by another, muting and unmuting it.

image

@jonpas
Copy link
Member

jonpas commented Dec 22, 2019

Please make a new issue with all the information in the template. Also include which spectator system you are using and any modifications if you have them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants