I need help disabling Creative Inventory Opening.

Hello, Ive been trying to code a spectating thing for my server gamemode but I'm having trouble will this code disable creative moving items to the inventory? And I also need someway to just flat out cancel the inventory to pop up for spectators in creative mode if thats possible well heres my code.

Code for disabled moving items to inventory in creative

Code:

@EventHandler
public void onClickInventorySpec(InventoryClickEvent event){
    Player player = (Player) event.getClickedInventory();
    if(spectator.contains(player))
    event.setResult(org.bukkit.event.Event.Result.DENY);
    }

Code to disable the inventory open

Code:

@EventHandler
public void onOpenInventorySpec(InventoryOpenEvent event){
    Player player = (Player) event.getInventory();
    if(spectator.contains(player))
    event.setCancelled(true);
    }

Is this the correct way I would do this?

 

@VinexAx789 Check if they have creative mode and cancel opening their own inventory

 

@SuchSwegMuchWow Okay give me a minute so which one do I not need? And what should I keep a check too.

 

@VinexAx789 You're probably not going to need the inventory click, because they won't even get access to that. Btw, why don't you just use like adventure gamemode or spectator gamemode if you're using 1.8

 

Already did that in 1.8 PvP sucks didnt go so well I guess I could use Adventure

 

@VinexAx789 I agree that 1.8 PvP isn't that good

 

@SuchSwegMuchWow I need to use Creative because spectators block arrows and it fixes that.

 

@VinexAx789 Ahh. Well just simply cancel inventory open if they are a spectator or have creative gamemode.

 

So like I had it?

Code:

@EventHandler
public void onOpenInventorySpec(InventoryOpenEvent event){
    Player player = (Player) event.getInventory();
    if(spectator.contains(player))
    event.setCancelled(true);
    }

@SuchSwegMuchWow

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

@VinexAx789 Ya, and btw store player names instead

 

@SuchSwegMuchWow Sorry I do not follow? What do you mean store player names instead? Like p.getPlayer().getName(); ??

I'm sorry

 

@VinexAx789 Ya so your array list will be like

Code:

ArrayList<String> spectator = new ArrayList<>(); 

and when you put someone in the array list you use

Code:

spectator.add(player.getName());

 

Code:

ArrayList<Player> spectator = new ArrayList<Player>();

Thats what I have atm

@SuchSwegMuchWow @SuchSwegMuchWow

@SuchSwegMuchWow Okay one sec let me do that

 

@VinexAx789 You can do that, but it would be better if you used

Code:

ArrayList<String> spectator = new ArrayList<String>();

to store the people who are spectators

 

@SuchSwegMuchWow


Inventory

Code:

@EventHandler
public void onOpenInventorySpec(InventoryOpenEvent event){
    Player player = (Player) event.getInventory();
    if(spectator.contains(player))
    event.setCancelled(true);
    }

Okay here's my spectator part

Code:

    getServer().getScheduler().scheduleSyncDelayedTask(this, new Runnable() {
          public void run() {
             p.spigot().respawn();
             p.teleport(lobby);
             //p.setGameMode(GameMode.SPECTATOR);
             //p.kickPlayer(ChatColor.RED + "You died");
            
              p.getInventory().clear();
              p.updateInventory();
              p.setGameMode(GameMode.CREATIVE);
              p.setAllowFlight(true);
              p.setFlying(true);
            
              for(Player player : Bukkit.getOnlinePlayers()){
                  if(!player.equals(p)){
                      player.hidePlayer(p);
                  }
              }
              spectator.add(p.getName());
          }
    }, 2L);
}

Now my InventoryEvent

Code:

@EventHandler
public void onOpenInventorySpec(InventoryOpenEvent event){
    Player player = (Player) event.getInventory();
    if(spectator.contains(player))
    event.setCancelled(true);
    }

I have a question now will this cancel the event for only Spectators? Or will it to players too?

 

@VinexAx789 Only spectators

 

@SuchSwegMuchWow Okay if I make it like this

Code:

@EventHandler
public void onOpenInventorySpec(InventoryOpenEvent event){
    Player player = (Player) event.getInventory();
    Player player = (Player) event.getPlayer();
    if(spectator.contains(player))
    event.setCancelled(true);
if (player.isOp()); {
event.Cancelled(false);
    }

Will that let ops open the inv?

@SuchSwegMuchWow by any chance do you know how to make a tracking device for the spectators? Or a gui menu?

Sadly testing this nothing worked I have no clue what to do now...

@SuchSwegMuchWow

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

@VinexAx789 The server is unable to detect when the client opens its inventory last time I checked

 

@xTigerRebornx Oh okay thanks I just need this to work

 

Erm, why don't you just use if(p.getGamemode().equals(your gamemode)){ //instead of making an arraylist}

 

@MajorSkillage Because I need only spectators to have the inventory blocked. Because they will be in creative

@SuchSwegMuchWow Any help?

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

@VinexAx789 Just cancel the block place event for spectators.

 

@teej107 Already have that cancel it's the items I'm worried about they can shoot people with bows and spawn in a item to arm a bomb I need to disable the inventorymoveitem something.. For only spectators in creative

@teej107 @xTigerRebornx @MajorSkillage @SuchSwegMuchWow I really need to solve this anyone else know how to do this?

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

@VinexAx789 Cancel the respective events except the inventory open event since that can't be detected.

 

@teej107 I'm sorry I really don't understand what you mean can you tell me what exactly events I need to cancel thanks

 

@VinexAx789 Well you don't want spectators to

• Place blocks
• Fire bows

There are events for those

 

@teej107 I already have blocks disabled but I do not have the bows but I just want it like mineplex has it they have it so when a spectator moves a item out of creative inventory it doesn't work which I want.

 

@VinexAx789 Then listen for an InventoryMoveEvent (or something like that)

 

InventoryClickEvent (though there are methods in spigot-api for managing this).

I'll probably release a plugin for easier spectator mangement

 

Will this work?
@EventHandler
public void onOpenInventorySpec(InventoryItemMoveEvent event){
Player player = (Player) event.getInventory();
if(spectator.contains(player))
event.setCancelled(true);
@1Rogue @teej107