Good day.

I'm currently making a coin mod for our server and it has a wallet, which has a security lock that doesn't allow you to open wallets that don't belong to you. I have made it work, except I ran into a big problem. When you create a wallet, you can open it, but when you exit and save to menu and then load up the world again, even tho the NBT is saying the correct name, it doesn't want to open, says it doesn't belong to me!

 

Here's my code for the item where all the action happens:

 

 

package tornith.tmod;

 

import java.util.List;

 

import org.lwjgl.input.Keyboard;

import org.lwjgl.input.Mouse;

 

import net.minecraft.block.Block;

import net.minecraft.entity.Entity;

import net.minecraft.entity.EntityLivingBase;

import net.minecraft.entity.player.EntityPlayer;

import net.minecraft.item.Item;

import net.minecraft.item.ItemStack;

import net.minecraft.nbt.NBTTagCompound;

import net.minecraft.nbt.NBTTagList;

import net.minecraft.util.BlockPos;

import net.minecraft.util.ChatComponentText;

import net.minecraft.util.EnumChatFormatting;

import net.minecraft.util.EnumFacing;

import net.minecraft.world.World;

import net.minecraftforge.fml.relauncher.Side;

import net.minecraftforge.fml.relauncher.SideOnly;

 

public class ItemWallet extends Item {

 

public String walletOwner = "";

 

public ItemWallet(){

// ItemStacks that store an NBT Tag Compound are limited to stack size of 1

this.maxStackSize = 1;

}

 

@Override

public void onCreated(ItemStack itemstack, World world, EntityPlayer player) {

if(itemstack.getTagCompound() == null){

itemstack.setTagCompound(new NBTTagCompound());

}

NBTTagCompound nbt = new NBTTagCompound();

nbt.setString("Owner", player.getName());

itemstack.getTagCompound().setTag("WalletOwner", nbt);

itemstack.setStackDisplayName(EnumChatFormatting.AQUA + player.getName() + "'s Wallet");

//walletOwner = player.getName();

}

 

public String getOwnerName(ItemStack stack, EntityPlayer player){

String ownerName = null;

if(stack.getTagCompound() != null){

if(stack.getTagCompound().hasKey("WalletOwner")){

NBTTagCompound nbt = (NBTTagCompound) stack.getTagCompound().getTag("WalletOwner");

ownerName = nbt.getString("Owner");

}

}

return ownerName;

}

 

@Override

public ItemStack onItemRightClick(ItemStack itemstack, World world, EntityPlayer entity)

    {

EntityPlayer player = (EntityPlayer) entity;

 

System.out.println(getOwnerName(itemstack, player) + ", " + player.getName());

 

if (!world.isRemote && entity instanceof EntityPlayer){

if (getOwnerName(itemstack, entity) == player.getName()){

if(!player.isSneaking()){

player.openGui(TMod.instance, TMod.ItemInventoryGuiIndex, world, (int) player.posX, (int) player.posY, (int) player.posZ);

}

}

 

else if (getOwnerName(itemstack, entity) == null){

if(itemstack.getTagCompound() == null){

itemstack.setTagCompound(new NBTTagCompound());

}

NBTTagCompound nbt = new NBTTagCompound();

nbt.setString("Owner", player.getName());

itemstack.getTagCompound().setTag("WalletOwner", nbt);

itemstack.setStackDisplayName(EnumChatFormatting.AQUA + player.getName() + "'s Wallet");

 

if (!world.isRemote && entity instanceof EntityPlayer){ if(!player.isSneaking()){

player.openGui(TMod.instance, TMod.ItemInventoryGuiIndex, world, (int) player.posX, (int) player.posY, (int) player.posZ);

}}

}

 

else if (getOwnerName(itemstack, entity) != player.getName()){

writeInChat("You cannot open wallets that don't belong to you!", entity);

}

 

else{

System.out.println("Error when openning the inventory: Wallet");

}

}

      return itemstack;

    }

 

public void writeInChat(String string, EntityPlayer player){

ChatComponentText component = new ChatComponentText(string);

player.addChatComponentMessage(component);

}

 

public void writeToNBT(NBTTagCompound tagcompound)

{

 

}

 

public void readfromNBT(NBTTagCompound tagcompound)

{

 

}

 

    @Override

    public int getMaxItemUseDuration(ItemStack itemstack)

    {

            return 1;

    }

}

 

 

1. dont compare strings with == , it will never turn true. Use String1.equals(String2)

2. dont save the players names , those can change. Save the UUID