[Solved] How to cancel PlayerEvent.ItemCraftedEvent?

[DJ1TJOO]

How to cancel PlayerEvent.ItemCraftedEvent?

Edited February 2, 2020 by DJ1TJOO

[Cadiboo]

You can call Event#setCancelled on cancellable events.

IIRC you should not be cancelling this event though because it results in the items in the crafting matrix being consumed but the item not being given to the player. What are you trying to achieve, from an end-user’s (a player) perspective?

[DJ1TJOO]

  On 2/1/2020 at 12:07 PM, Cadiboo said:

You can call Event#setCancelled on cancellable events.

IIRC you should not be cancelling this event though because it results in the items in the crafting matrix being consumed but the item not being given to the player. What are you trying to achieve, from an end-user’s (a player) perspective?

Expand  

The player has todo something first before they can craft the item. I tried making a custom condition, but I can't seem how todo that because what the docs says isn't working for me 

Github: https://github.com/DJ1TJOO/fantasy20/

[Cadiboo]

  On 2/1/2020 at 12:10 PM, DJ1TJOO said:

I tried making a custom condition

Expand  

This is what you’re meant to do. What is the exact issue that you’re having? Forge provides custom conditions too so even if you’re having trouble with the docs, you can look at Forge’s code to see how it works

[DJ1TJOO]

  On 2/1/2020 at 12:12 PM, Cadiboo said:

This is what you’re meant to do. What is the exact issue that you’re having? Forge provides custom conditions too so even if you’re having trouble with the docs, you can look at Forge’s code to see how it works

Expand  

I read the ItemExsistsCondition.class and it has a constructor with a variable and in the docs they say that the constructor must be empty so I don't understand what I need todo

[Cadiboo]

What is the condition that you’re trying to implement meant to do?

[DJ1TJOO]

  On 2/1/2020 at 12:23 PM, Cadiboo said:

What is the condition that you’re trying to implement meant to do?

Expand  

I have a capability on each player with the Items they have "learned" if the item is not in the capability list then the player cannot craft it

[Cadiboo]

Ok so you would want a condition called something like “has_learned_item”. I’m not sure if you get a player object in the conditions, let me check

[DJ1TJOO]

  On 2/1/2020 at 12:28 PM, Cadiboo said:

Ok so you would want a condition called something like “has_learned_item”. I’m not sure if you get a player object in the conditions, let me check

Expand  

Yes I called it researched because the way you do it is in the research table

[DJ1TJOO]

  On 2/1/2020 at 12:28 PM, Cadiboo said:

Ok so you would want a condition called something like “has_learned_item”. I’m not sure if you get a player object in the conditions, let me check

Expand  

Al ready have an idea?

[DJ1TJOO]

  On 2/1/2020 at 6:28 PM, diesieben07 said:

You already asked about recipe conditions here:

 

However they won't help you make player-specific conditions. For that you need a custom recipe type (check IRecipeSerializer) and then check the player inside the recipe implementation.

Expand  

I tried to make it but I think I need to register it somewhere. I've updated the github and the doorlock recipe has my type (for now just the copied normal shapedrecipe) as recipe. 

The error I'm getting when joining a world is:
com.google.gson.JsonSyntaxException: Invalid or unsupported recipe type 'fantasy20:shaped_researched_recipe'
 

[Draco18s]

...You do know that there's a game rule that already covers this, right? If you don't have the recipe in your recipe book, you can't craft it when the rule is enabled.

[DJ1TJOO]

  On 2/1/2020 at 9:00 PM, diesieben07 said:

IRecipeSerializer is a registry entry, just like blocks or items.

Expand  

It works now but how would I get the player in my IRecipe because the CraftingInventory doesn't hold the owner of the inv. And its all server side so I can't use the Minecraft.getInstance()

[DJ1TJOO]

  On 2/2/2020 at 12:32 PM, diesieben07 said:

CraftingInventory has a reference to the Container (field_70465_c, you'll need reflection).

You can then look through Container#listeners which should contain the player using this inventory.

Expand  

 

@SuppressWarnings("rawtypes")
		Class invClass = inv.getClass();
		try {
			Field field = invClass.getField("field_70465_c");
			Container container = (Container) field.get(this);
			@SuppressWarnings("rawtypes")
			Class containerClass = container.getClass();
			@SuppressWarnings("unchecked")
			List<IContainerListener> listeners = (List<IContainerListener>) containerClass.getField("listeners").get(this);
			for (IContainerListener iContainerListener : listeners) {
				iContainerListener.
			}
		} catch (Exception e) {
			e.printStackTrace();
		}

I have this now but I don't see how I can get the player from IContainerListener.
Edit: Oh I can cast it to a serverplayer 
Changed this -> inv

Edited February 2, 2020 by DJ1TJOO

[DJ1TJOO]

  On 2/2/2020 at 12:52 PM, diesieben07 said:

ServerPlayerEntity implements IContainerListener. You do an instanceof check.

 

Also, why are you using raw types?

Expand  

How would I do it else then I've just searched reflection on the internet it's new for me

[DJ1TJOO]

  On 2/2/2020 at 12:56 PM, diesieben07 said:

https://docs.oracle.com/javase/tutorial/java/generics/rawTypes.html

Expand  

what do I need todo it to avoid raw types?

Edited February 2, 2020 by DJ1TJOO

[DJ1TJOO]

  On 2/2/2020 at 1:00 PM, diesieben07 said:

Please learn about Java generics. This is not a Java school.

Expand  

Okay

[DJ1TJOO]

  On 2/2/2020 at 1:00 PM, diesieben07 said:

Please learn about Java generics. This is not a Java school.

Expand  

I've this code now and I get the error that the field field_70465_c does not exist
 

try {
			Class<? extends CraftingInventory> invClass = inv.getClass();
			Field field = invClass.getField("field_70465_c");
			Container container = (Container) field.get(inv);
			Class<? extends Container> containerClass = container.getClass();
			Field field2 = containerClass.getField("listeners");
			@SuppressWarnings("unchecked")
			List<IContainerListener> listeners = (List<IContainerListener>) field2.get(inv);
			for (IContainerListener iContainerListener : listeners) {
				if(iContainerListener instanceof ServerPlayerEntity) {
					ServerPlayerEntity p = (ServerPlayerEntity) iContainerListener;
					if(!p.getCapability(CapabilityResearchProvider.RESEARCH_CAPABILITY, p.getHorizontalFacing()).map(r -> {
						if(!r.getResearched().contains(getCraftingResult(inv).getItem())) {
							return false;
						}
						return true;
					}).orElse(Boolean.FALSE)) {
						return false;
					}
				}
			}
		} catch (Exception e) {
			e.printStackTrace();
		}

 

[Cadiboo]

You can use a single line of code with ObfuscationReflectionHelper. You can also use an AT (Access Transformer)