This is a cool & simple idea. Here's a suggestion though
Code:
@Override
public void moved() {
player.setCanPvp(true);
player.getPVPMinigame().sendInters();
}
this is called every time the player moves. Do you want to make the player be able to attack other people every time they move? (see Player#setCanPvp) No you only want to do it once; so
Code:
player.setCanPvp(true);
should only be sent when the player starts the controller.
Also the interface is an overlay; it won't close when they move regardless. So moved() doesn't need either of those methods.
Pls show a pic of interface 731, dunno what it looks like & kinda interested [i don't have any rs kits open as i type this].