Overriding default key listeners

Trevor · Jul 31st 2018

I have set my plugin to listen for KEY_ESCAPE and call closeGUI() when that happens. That's working fine, however the default game listener still runs and brings up the ESC menu (settings, quit etc.)

Is it possible to tell the game to suspend listening for keys when a plugin is listening? If not, what's the best way to express this in an API change request?

Thanks!

Minotorious · Jul 31st 2018

hmm this must be an error in your code as the Teleport plugin (Teleport) is using the Esc key to close its GUI and the start menu doesn't show up

Trevor · Jul 31st 2018

Unfortunately Galveston hasn't shared his code. Here's my code. Can you spot what's wrong? I've tried each of the commented variations with no apparent difference.

Code

@EventMethod
    public void onPlayerConnect(PlayerConnectEvent event)
    {
        Player player = event.getPlayer();
        player.registerKeys(KeyInput.KEY_ESCAPE);
        player.setListenForKeyInput(true);


    }


@EventMethod
    public static void onPlayerKeyEvent(PlayerKeyEvent event)
    {
        Player player = event.getPlayer();
        //if(event.getKeyCode() == KeyInput.KEY_ESCAPE && event.isPressed())
        //if(event.getKeyCode() == KeyInput.KEY_ESCAPE && !event.isPressed())
        if(event.getKeyCode() == KeyInput.KEY_ESCAPE)
        {
            setVisibility(player,false);
            clearGUI(player);
            player.setMouseCursorVisible(false);
        }
    }

Display More

Minotorious · Jul 31st 2018

got it

you are missing this line when showing the GUI player.disableClientsideKeys(KeyInput.KEY_ESCAPE);

and then ofc this line when hiding it: player.enableClientsideKeys(KeyInput.KEY_ESCAPE);

PS: this line player.setListenForKeyInput(true); is not necessary anymore as the listener always listens

Trevor · Jul 31st 2018

Ah, your blood is worth bottling! (That means thanks!)

Actually it seems that player.setListenForKeyInput(true); is still required. Without it, my plugin doesn't respond to ESC.

Minotorious · Jul 31st 2018

Quote from Trevor

Actually it seems that player.setListenForKeyInput(true); is still required. Without it, my plugin doesn't respond to ESC.

:O @red51 bugfix

Edit: wait no my bad, mixed up the methods the player.setListenForKeyInput(true); is still required, the .setListenForObjectInteractions(boolean) is the one that is not required

PS: sorry for tagging you red

Participate now!