Solved Problems with getInvetory().getContents()

So basically I'm trying to get the inventory that a player has just closed and saving the contents to a file to be reloaded later. I have everything working fine unless there is an empty space in the inventory. When there is one the getContents() method seems to stop at the space and not get anything after that. Any help would be appreciated, and if you need any other snippets of code ask and i'll get them

Code:java

 
    @EventHandler
    public void onECClose(InventoryCloseEvent e)
    {
        //only checking for enderchest inventory types
        if(e.getInventory().getType() == InventoryType.ENDER_CHEST)
        {
            Player p = (Player) e.getViewers().get(0);
            Yaml pf = PDPmain.getOfflinePlayerYaml(p.getName());
            String p2 = pf.getString(shareSelect);
 
            //checks the inventory name to make sure im looking at correct inventory
            if(e.getInventory().getName() == p2 + "'s Ender Chest")
                plugin.saveInv(e.getInventory().getContents(), "Enderchest", p2);
 
            //For debugging, prints out contents of the item stack
            for(ItemStack itm : e.getInventory().getContents())
                p.sendMessage(itm.toString());
        }
    }

 

save it by encoding it in base64(store the string) and decode it when you need it again

 

because you can read the encoded string, silly.

 

yes.

 

@timtower No npe's. nothing shows up in console at all and @FrostedSnowman I've already checked and the storing isnt an issue

 

Alright, so after a day or two of not looking at it i came back with fresh eyes and found they problem. Looks like it was the loading of the inventory from file, whenever there was a space it would stop loading and be finished(got the code online and was too tired to realize it) Ill post the original and fixed code below so that anybody curious can see what changed.

Original code from a 3 year old post

Code:java

public ItemStack[] loadInv(String inventoryName) {
            //Creates a blank list
            List<ItemStack> itemstackList = new ArrayList<ItemStack>();
            //Preparing
            boolean done = false;
            //Starts the counting
            int i = -1;
            while (done == false) {
                i++;
                //Checks if the config contains that slot
                if (getConfig().contains("Inventory"+"."+inventoryName+"."+i)) {
                    //Adds the itemstack to the list
                    itemstackList.add(getConfig().getItemStack("Inventory"+"."+inventoryName+"."+i));
                } else {
                    //No more items, the list is complete
                    done = true;
                }
            }
            //Some converting and returning
            ItemStack[] toReturn = itemstackList.toArray(new ItemStack[itemstackList.size()]);
            return toReturn;
        }

Fixed code

Code:java

public ItemStack[] loadInv(String inventoryName, String playerName)
    {
        //Opens players PlayerDataPlus config file
        Yaml p = PDPmain.getOfflinePlayerYaml(playerName);
        //Creates a blank list
        List<ItemStack> itemstackList = new ArrayList<ItemStack>();
 
        //Preparing
        int i = 0, total = 0;
 
        //Checks amount of items in inventory
        for(String s : p.getConfigurationSection("Inventory." + inventoryName).getKeys(false))
        {
            //checks if the section is an int(should be the item location in the inventory
            if(isInt(s))
            {
                int n = Integer.parseInt(s);
                //makes total the highest inventory location
                if(n > total)
                    total = n;
            }
        }
 
        //Begin loading
        while (i <= total)
        {
            //Checks if the config contains that slot
            if (p.contains("Inventory."+inventoryName+"."+i))
                //Adds the itemstack to the list
                itemstackList.add(p.getItemStack("Inventory." + inventoryName + "." + i));
            else
                //If it doesnt exist adds air to prevent EC crunching
                itemstackList.add(new ItemStack(Material.AIR));
 
            i++;
        }
        //Some converting and returning
        ItemStack[] toReturn = itemstackList.toArray(new ItemStack[itemstackList.size()]);
 
        return toReturn;
    }