Jump to content

[1.7.10] Inventory not updating when changed


Ben_

Recommended Posts

I'm trying to write a questing mod, and I'm trying to write the completion.

 

What I need to do, is give some items to the player and then remove some of them.

The relevant code is near the bottom:

	public static Boolean finishQuest(Quest quest, EntityPlayer player) {
	if(player.worldObj.isRemote == true) {
		//On the client side
		System.out.println("Sending quest completion request");
		EngineerCraft.channel.sendToServer(new QuestCompleteMessage(player, quest.getName()));



		return null;
	} else {
		//On the server side
		System.out.println("Receiving quest completion request");






		//Get the list of inputs to be checked
		@SuppressWarnings("unchecked")
		ArrayList<ItemStack> toBeChecked = (ArrayList<ItemStack>) quest.getInputs().clone();

		//Go through every single input
		while(toBeChecked.size() > 0) {
			ItemStack currentStack = toBeChecked.get(0);

			//The amount of items of this type in the inventory
			int itemCount = 0;

			//Go through each stack in the inventory and add together the total amount of the same item
			for(int i = 0; i < player.inventory.mainInventory.length; i++) {
				ItemStack checkStack = player.inventory.mainInventory[i];
				if(checkStack != null) {
					if(checkStack.isItemEqual(currentStack)) {
						itemCount += checkStack.stackSize;
					}
				}
			}

			if(itemCount >= currentStack.stackSize) {
				toBeChecked.remove(0);
			} else {
				return false;
			}
		}

		//If the game has reached this point, the player's inventory has everything the quest needs








		//Get the list of inputs to be taken
		@SuppressWarnings("unchecked")
		ArrayList<ItemStack> toBeTaken = (ArrayList<ItemStack>) quest.getInputs().clone();

		//Go through every single input and remove them
		while(toBeTaken.size() > 0) {
			ItemStack currentStack = toBeTaken.get(0);

			//The amount of items of this type in the inventory
			int itemCount = currentStack.stackSize;

			//Go through each stack in the inventory and add together the total amount of the same item
			for(int i = 0; i < player.inventory.mainInventory.length; i++) {
				ItemStack checkStack = player.inventory.mainInventory[i];
				if(checkStack != null) {
					if(checkStack.isItemEqual(currentStack)) {
						if(checkStack.stackSize > itemCount) {
							player.inventory.mainInventory[i] = player.inventory.decrStackSize(i, itemCount);
							itemCount = 0;
						} else {
							itemCount -= checkStack.stackSize;
							player.inventory.mainInventory[i] = null;
						}
					}
				}

			}
			toBeTaken.remove(0);
		}






		//Reward the player
		//Get the list of rewards to be given
		@SuppressWarnings("unchecked")
		ArrayList<ItemStack> toBeRewarded = (ArrayList<ItemStack>) quest.getInputs().clone();

		for(int i = 0; i < toBeRewarded.size(); i++) {

			//If adding an item stack to the inventory fails
			if(player.inventory.addItemStackToInventory(toBeRewarded.get(i).copy()) == false) {

				//Spawn an item on the ground
				player.worldObj.loadedEntityList.add(new EntityItem(player.worldObj, player.posX, player.posY + 1.0, player.posZ, toBeRewarded.get(i).copy()));
			}
		}

		player.inventory.inventoryChanged = true;

		player.inventory.markDirty();



		//Save the quest completion
		saveQuestCompletion(quest, player);

		return true;
	}
}

I have debugged the code and seen that it does indeed get changed, but then outside of the scope it gets brought back to its original value.

 

Is there something I'm missing? This code is only server-side, but that shouldn't matter, because when I reload the world it doesn't change.

 

Does inventory.inventoryChanged need to be changed back to false next tick?

 

I have been incredibly frustrated by this issue, because none of my solutions work.

Link to comment
Share on other sites

Guest
This topic is now closed to further replies.

Announcements



  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • Akun Pro Kamboja adalah pilihan terbaik bagi Anda yang mencari pengalaman bermain slot Maxwin dengan transaksi mudah menggunakan Bank Lampung. Berikut adalah beberapa alasan mengapa Anda harus memilih Akun Pro Kamboja: Slot Maxwin Terbaik Kami menyajikan koleksi slot Maxwin terbaik yang menawarkan kesenangan bermain dan peluang kemenangan besar. Dengan fitur-fitur unggulan dan tema-tema menarik, setiap putaran permainan akan memberikan Anda pengalaman yang tak terlupakan. Transaksi Mudah dengan Bank Lampung Kami menyediakan layanan transaksi mudah melalui Bank Lampung untuk kenyamanan dan keamanan Anda. Dengan proses yang cepat dan efisien, Anda dapat melakukan deposit dan penarikan dana dengan lancar dan tanpa hambatan. Anti Rungkat Akun Pro Kamboja memberikan jaminan "anti rungkat" kepada para pemainnya. Dengan fitur ini, Anda dapat merasakan sensasi bermain dengan percaya diri, karena kami memastikan pengalaman bermain yang adil dan menyenangkan bagi semua pemain.  
    • BINGO188: Destinasi Terbaik untuk Pengalaman Slot yang Terjamin Selamat datang di BINGO188, tempat terbaik bagi para pecinta slot yang mencari pengalaman bermain yang terjamin dan penuh kemenangan. Di sini, kami menawarkan fitur unggulan yang dirancang untuk memastikan kepuasan dan keamanan Anda. Situs Slot Garansi Kekalahan 100 Kami memahami bahwa kadang-kadang kekalahan adalah bagian dari permainan. Namun, di BINGO188, kami memberikan jaminan keamanan dengan fitur garansi kekalahan 100. Jika Anda mengalami kekalahan, kami akan mengembalikan saldo Anda secara penuh. Kemenangan atau uang kembali, kami memastikan Anda tetap merasa aman dan nyaman. Bebas IP Tanpa TO Nikmati kebebasan bermain tanpa batasan IP dan tanpa harus khawatir tentang TO (Turn Over) di BINGO188. Fokuslah pada permainan Anda dan rasakan sensasi kemenangan tanpa hambatan. Server Thailand Paling Gacor Hari Ini Bergabunglah dengan server terbaik di Thailand hanya di BINGO188! Dengan tingkat kemenangan yang tinggi dan pengalaman bermain yang lancar, server kami dijamin akan memberikan Anda pengalaman slot yang tak tertandingi. Kesimpulan BINGO188 adalah pilihan terbaik bagi Anda yang menginginkan pengalaman bermain slot yang terjamin dan penuh kemenangan. Dengan fitur situs slot garansi kekalahan 100, bebas IP tanpa TO, dan server Thailand paling gacor hari ini, kami siap memberikan Anda pengalaman bermain yang aman, nyaman, dan menguntungkan. Bergabunglah sekarang dan mulailah petualangan slot Anda di BINGO188!
    • Mengapa Memilih AlibabaSlot? AlibabaSlot adalah pilihan terbaik bagi Anda yang mencari slot gacor dari Pgsoft dengan transaksi mudah menggunakan Bank Panin. Berikut adalah beberapa alasan mengapa Anda harus memilih AlibabaSlot: Slot Gacor dari Pgsoft Kami menyajikan koleksi slot gacor terbaik dari Pgsoft. Dengan fitur-fitur unggulan dan peluang kemenangan yang tinggi, setiap putaran permainan akan memberikan Anda kesenangan dan keuntungan yang maksimal. Transaksi Mudah dengan Bank Panin Kami menyediakan layanan transaksi mudah melalui Bank Panin untuk kenyamanan dan keamanan Anda. Dengan proses yang cepat dan efisien, Anda dapat melakukan deposit dan penarikan dana dengan lancar dan tanpa masalah.  
    • Delete the jei-server.toml file in your config folder and test it again
    • java.util.ConcurrentModificationException at java.base/java.util.HashMap.computeIfAbsent(Unknown Source) at Genesis//com.moonsworth.lunar. ... Looks like an issue with the Launcher - Make a test with another Launcher like MultiMC, AT Launcher or Technic Launcher or try the original one
  • Topics

×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.