Jump to content

Recommended Posts

Posted

Title explains it, but when a key is pressed it sets this method in motion. The method goes through and the itemstack visually decreases, but when you right click or open the inventory the itemstack is reset back to what it was previously.

 

 

@Override
public void onUpdate(ItemStack itemstack, World world, Entity entity, int metadata, boolean bool)
{
currentShot++;
currentReload++;

if (entity instanceof EntityPlayer)
{
	EntityPlayer player = (EntityPlayer) entity;
	FalloutPlayer props = FalloutPlayer.get(player);

	if (clipCount < clipSize && props.getReloadingState() == 1 && player.inventory.hasItem(ammoType))
	{
		player.playSound(reloadSound, 1.0F, 1);

		for (int i = 0; i < clipSize; i++)
		{
			if (player.inventory.hasItem(ammoType) && clipCount < clipSize)
			{
				player.inventory.consumeInventoryItem(ammoType);
				player.inventory.markDirty();
				player.inventoryContainer.detectAndSendChanges();
				clipCount++;
			}
		}

		currentReload = 0;
		props.setReloadingState(0);
	}
}
}

 

Posted

Yeah, I'm aware that values like that are applied on a global scale. I've been using NBT tags in other versions, but they don't always seem to work, so while I'm figuring that out I'm using item values.

On another note, in the past I've used !world.isRemote checks, but the method doesn't fire. Here's an example of what I used to have:

public void onUpdate(ItemStack itemstack, World world, Entity entity, int metadata, boolean bool)
{
	if (itemstack.getTagCompound() == null)
	{
		itemstack.setTagCompound(new NBTTagCompound()); // = new NBTTagCompound();
		itemstack.getTagCompound().setInteger("clipCount", 0);
		itemstack.getTagCompound().setInteger("currentReloadTime", 0);
		itemstack.getTagCompound().setInteger("currentShotTime", 0);
	}

	if (itemstack.getTagCompound() != null)
	{
		//itemstack.getTagCompound().setInteger("currentShotTime", itemstack.getTagCompound().getInteger("currentShotTime") + 1);
		//itemstack.getTagCompound().setInteger("currentReloadTime", itemstack.getTagCompound().getInteger("currentReloadTime") + 1);

		currentShot++;
		currentReload++;

		if (entity instanceof EntityPlayer)
		{
			EntityPlayer player = (EntityPlayer) entity;
			FalloutPlayer props = FalloutPlayer.get(player);

			//if (clipCount < clipSize && Fallout.isReloading && player.inventory.hasItem(ammoType))
			if (clipCount < clipSize && props.getReloadingState() == 1 && player.inventory.hasItem(ammoType))
			{
				player.playSound(reloadSound, 1.0F, 1);

				for (int i = 0; i < clipSize; i++)
				{
					if (player.inventory.hasItem(ammoType) && clipCount < clipSize)
					{
						if (!world.isRemote)
						{
							player.inventory.consumeInventoryItem(ammoType);
							player.inventory.markDirty();
							player.inventoryContainer.detectAndSendChanges();
							clipCount++;
							//int currentClip = itemstack.getTagCompound().getInteger("clipCount");
							//itemstack.getTagCompound().setInteger("clipCount", currentClip + 1);
						}
					}
				}

				//itemstack.getTagCompound().setInteger("currentReloadTime", 0);
				currentReload = 0;
				props.setReloadingState(0);
				//Fallout.isReloading = false;
			}
		}
	}
}

 

 

I've experimented by moving !world.isRemote to different places within the method, but anything below it won't be activated.

Posted

That's what I had in the last block and is what I'm currently using:

if (!world.isRemote)
						{
							player.inventory.consumeInventoryItem(ammoType);
							player.inventory.markDirty();
							player.inventoryContainer.detectAndSendChanges();
							clipCount++;
							//int currentClip = itemstack.getTagCompound().getInteger("clipCount");
							//itemstack.getTagCompound().setInteger("clipCount", currentClip + 1);
						}

 

This is the final check, but it still won't update. As shown above, a sound plays and then the bullets will load into the weapon. On screen it shows the current clip count out of clip size, ie 4/6 loaded, and this number is never updated as well.

Posted

Switched it to this:

public void onUpdate(ItemStack itemstack, World world, Entity entity, int metadata, boolean bool)
{
	if (itemstack.getTagCompound() == null)
	{
		itemstack.setTagCompound(new NBTTagCompound());
		itemstack.getTagCompound().setInteger("clipCount", 0);
		itemstack.getTagCompound().setInteger("currentReloadTime", 0);
		itemstack.getTagCompound().setInteger("currentShotTime", 0);
		System.out.println("onUpdate tag created");
	}

	if (itemstack.getTagCompound() != null)
	{
		NBTTagCompound tag = itemstack.getTagCompound();

		tag.setInteger("currentShotTime", tag.getInteger("currentShotTime") + 1);
		tag.setInteger("currentReloadTime", tag.getInteger("currentReloadTime") + 1);

		if (entity instanceof EntityPlayer)
		{
			EntityPlayer player = (EntityPlayer) entity;
			FalloutPlayer props = FalloutPlayer.get(player);

			if (tag.getInteger("clipcount") < clipSize && props.getReloadingState() == 1 && player.inventory.hasItem(ammoType))
			{
				player.playSound(reloadSound, 1.0F, 1);

				for (int i = 0; i < clipSize; i++)
				{
					if (player.inventory.hasItem(ammoType) && tag.getInteger("clipcount") < clipSize)
					{
						if (!world.isRemote)
						{
							player.inventory.consumeInventoryItem(ammoType);
							player.inventory.markDirty();
							//player.inventoryContainer.detectAndSendChanges();

							int currentClip = tag.getInteger("clipcount");
							tag.setInteger("clipCount", currentClip + 1);
						}
					}
				}

				tag.setInteger("currentReloadTime", 0);
				props.setReloadingState(0);
			}
		}
	}
}

 

 

When I don't have the worldRemote check the ammo is consumed (and stays consumed), but the tag is not updated. If the check is there all it does is play the sound.

Posted

Kay, looks like this now:

public void onUpdate(ItemStack itemstack, World world, Entity entity, int metadata, boolean bool)
{
	if (!world.isRemote)
	{
		if (itemstack.getTagCompound() == null)
		{
			itemstack.setTagCompound(new NBTTagCompound());
			itemstack.getTagCompound().setInteger("clipCount", 0);
			itemstack.getTagCompound().setInteger("currentReloadTime", 0);
			itemstack.getTagCompound().setInteger("currentShotTime", 0);
			System.out.println("onUpdate tag created");
		}

		if (itemstack.getTagCompound() != null)
		{
			NBTTagCompound tag = itemstack.getTagCompound();

			tag.setInteger("currentShotTime", tag.getInteger("currentShotTime") + 1);
			tag.setInteger("currentReloadTime", tag.getInteger("currentReloadTime") + 1);

			if (entity instanceof EntityPlayer)
			{
				EntityPlayer player = (EntityPlayer) entity;
				FalloutPlayer props = FalloutPlayer.get(player);

				if (tag.getInteger("clipcount") < clipSize && props.getReloadingState() == 1 && player.inventory.hasItem(ammoType))
				{
					player.playSound(reloadSound, 1.0F, 1);

					for (int i = 0; i < clipSize; i++)
					{
						if (player.inventory.hasItem(ammoType) && tag.getInteger("clipcount") < clipSize)
						{
							player.inventory.consumeInventoryItem(ammoType);
							player.inventory.markDirty();
							//player.inventoryContainer.detectAndSendChanges();

							int currentClip = tag.getInteger("clipcount");
							tag.setInteger("clipCount", currentClip + 1);
						}
					}

					tag.setInteger("currentReloadTime", 0);
					props.setReloadingState(0);
				}
			}
		}
	}
}

 

 

Now nothing at all happens. There's a keybind I'm using that tells an IExtendedEntityProperties class that a player is reloading and sets a certain integer mark that. After the player is done reloading, it sets the int back to 0, but I think it's all on the client side. Would that be the reason this isn't communicating? I inserted println checks and it wont stop printing after it reached the IExtendedEntityProperties check.

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • Hi everyone, I'm currently developing a Forge 1.21 mod for Minecraft and I want to display a custom HUD overlay for a minigame. My goal: When the game starts, all players should see an item/block icon (from the base game, not a custom texture) plus its name/text in the HUD – similar to how the bossbar overlay works. The HUD should appear centered above the hotbar (or at a similar prominent spot), and update dynamically (icon and name change as the target item changes). What I've tried: I looked at many online tutorials and several GitHub repos (e.g. SeasonHUD, MiniHUD), but most of them use NeoForge or Forge versions <1.20 that provide the IGuiOverlay API (e.g. implements IGuiOverlay, RegisterGuiOverlaysEvent). In Forge 1.21, it seems that neither IGuiOverlay nor RegisterGuiOverlaysEvent exist anymore – at least, I can't import them and they are missing from the docs and code completion. I tried using RenderLevelStageEvent as a workaround but it is probably not intended for custom HUDs. I am not using NeoForge, and switching the project to NeoForge is currently not an option for me. I tried to look at the original minecraft source code to see how elements like hearts, hotbar etc are drawn on the screen but I am too new to Minecraft modding to understand. What I'm looking for: What is the correct way to add a custom HUD element (icon + text) in Forge 1.21, given that the previous overlay API is missing? Is there a new recommended event, callback, or method in Forge 1.21 for custom HUD overlays, or is everyone just using a workaround? Is there a minimal open-source example repo for Forge 1.21 that demonstrates a working HUD overlay without relying on NeoForge or deprecated Forge APIs? My ideal solution: Centered HUD element with an in-game item/block icon (from the base game's assets, e.g. a diamond or any ItemStack / Item) and its name as text, with a transparent background rectangle. It should be visible to the players when the mini game is running. Easy to update the item (e.g. static variable or other method), so it can change dynamically during the game. Any help, code snippets, or up-to-date references would be really appreciated! If this is simply not possible right now in Forge 1.21, it would also help to know that for sure. Thank you very much in advance!
    • The simple answer is there is not an easy way. You would need to know how to program in Java, as well as at least some familiarity with how Forge works so you could port the differences. You would also need the sourcecode for the original mod, and permission from the author to modify it, if they did not use some sort of open source license. So it's not impossible, but it would take some effort, but doing so would open up a whole new world of possibilities for you!
    • Does it still crash if you remove holdmyitems? Looks like that mod doesn't work on a server as far as I can tell from the error.  
    • Crashes the server when trying to start. Error code -1. Log  
  • Topics

  • Who's Online (See full list)

    • There are no registered users currently online
×
×
  • Create New...

Important Information

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