Solved I can't change inventory called on InventoryClickEvent

Hey, Bukkit, I need your help again I have this simple code:

Code:java

@EventHandler
    public void CGinvClick(InventoryClickEvent event) {
        Player p = (Player) event.getWhoClicked();
        if (event.getInventory().getType() == InventoryType.CHEST) {
            ItemStack book = new ItemStack(Material.WRITTEN_BOOK);
            book.setAmount(1);
            book.setItemMeta(CGmodifyEmptyBook(book, "my chest").getItemMeta());
            event.getInventory().setItem(0,book);
            p.sendMessage("Bah!");
        }
    }

The code works almost fine, plugin prints me "Bah!" message, but it doesn't change the inventory like I want to. Here's the video where I'm showing you, how bugged is it: http://rbg.wz.cz/mc.avi

Is there anyone who knows how to fix this problem?

 

@PDKnight I think you need to call something like p.updateInventory();

 

@sander_blaadjes I know, but event.getInventory().update() or event.updateInventory() doesn't exist I tried your p.updateInventory(), but it didn't help me

 

@PDKnight What are you trying to do? Because I think you should be using something other than InventoryClickEvent.

InventoryClickEvent Is good if you are trying to create some kind of GUI like in most minigames, where players can't take the item out of the inventory / chest.

 

@sander_blaadjes I'm just trying to set item to the first slot in chest right when player grabs it (so then player has item in cursor and it's in the first slot, too), nothing else. I didn't show you the complete code, because I just want to solve this problem

 

Seems strange, The way I would do it is.

Code:

ItemStack book = new ItemStack(Material.WRITTEN_BOOK, 1);
book.setItemMeta(CGmodifyEmptyBook(book, "my chest").getItemMeta());
event.getInventory().addItem(book);

If you tried that and it does not work can you show us the CGmodifyEmptyBook class
and p.getInventory.setItem(stuff goes here) You are getting the players inventory and setting it to an item? I think you're supposed to use .addItem(stuff goes here)
This is my 100th msg

 

@_Error
CGmodifyEmptyBook is a function:

Code:java

private ItemStack CGmodifyEmptyBook(ItemStack book, String chest_name) {
    BookMeta bm = (BookMeta) book.getItemMeta();
    List<String> found = getConfig().getStringList("found." + chest_name);
    String book_log_str = "";
    for (Object o : found)
        book_log_str += (found.indexOf(o) + 1) + ". " + o + "\n";
    bm.setAuthor(ChatColor.AQUA + "CGeo_");
    bm.setTitle(ChatColor.AQUA + "GeoChest Log");
    bm.addPage(parseColors(getConfig().getString("log_who_found"))
            + "\u00a70\n\n" + book_log_str);
    book.setItemMeta(bm);
 
    return book;
}

EDIT: Anyway, this function works great, it gives me a book, the problem isn't caused by it

 

Maybe you have to at first get the Chests Inventory, then Change it and at last set the Chests Inventory to the changed one. I dont know if that works, but you can try

 

@PDKnight This is not perfect and can deffinitly be improved a lot, but this should help you.

Code:java

 
    @SuppressWarnings("deprecation")
    @EventHandler
    public void CGinvClick(InventoryClickEvent event) {
        final int slot = 0;
        Player p = (Player) event.getWhoClicked();
        p.sendMessage(event.getInventory().getName());
        if (event.getRawSlot() < event.getInventory().getSize() && event.getRawSlot() == slot) {
            event.setCancelled(true);
            ItemStack book = new ItemStack(Material.WRITTEN_BOOK);
            book.setAmount(1);
            event.getInventory().clear();
            event.getInventory().setItem(slot,book);
            p.updateInventory();
            p.setItemOnCursor(book);
 
 
        }
    }

 

@Doomfighter I tried it, but without any effect.

@sander_blaadjes Okay, I'll try it when I arrive home, but I'll repair it first

Code:java

    @SuppressWarnings("deprecation")
    @EventHandler
    public void CGinvClick(InventoryClickEvent event) {
        Player p = (Player) event.getWhoClicked();
        InventoryType it = event.getInventory().getType();
        if (it == InventoryType.CHEST
                && event.getRawSlot() < event.getInventory().getSize()
                && event.getRawSlot() == 0) {
            event.setCancelled(true); // <-- why do you cancel it?
            ItemStack book = new ItemStack(Material.WRITTEN_BOOK,1);
            book.setItemMeta(CGmodifyEmptyBook(book, "my chest").getItemMeta());
            event.getInventory().setItem(0,book);
            p.updateInventory();
            p.setItemOnCursor(book);
        }
    }

EDIT: @sander_blaadjes, MANY THANKS! You don't know, but you propably saved me several days!

SOLVED

 

@PDKnight Sorry, I also had a version with comments:

Same code (might be more clear to see) http://pastebin.com/BabF6Gha

Code:java

@SuppressWarnings("deprecation")
    @EventHandler
    public void CGinvClick(InventoryClickEvent event) {
        final int slot = 0; //The first slot of the chest
        Player p = (Player) event.getWhoClicked();
        p.sendMessage(event.getInventory().getName());
        if (event.getRawSlot() < event.getInventory().getSize() && //This makes sure the player clicked in the chest, and not the player's inventory.
                event.getRawSlot() == slot) // This makes sure it only gives the player the item in his hand if he clicks the item. (Could be replaced with event.getItem() == book)
            {
            event.setCancelled(true); //If you don't cancel it, it will do this odd thing you showed in your video, where it creates ghost books.
            ItemStack book = new ItemStack(Material.WRITTEN_BOOK);
            book.setAmount(1);
            event.getInventory().clear(); //Clearing it, in case the player placed the book in the chest.
            event.getInventory().setItem(slot,book); //Because we cancelled the event, we have to manually give the player the item on his cursor.
            p.updateInventory();
            p.setItemOnCursor(book); //Can't explain, remove it to see the difference :p
 
 
        }
    }