Preventing the player from punching/attacking on left click

[Eilux]

I need a way to temporarily make it so that a player is not able to punch/attack when they hit left click. I don't have any code yet because I'm unsure of where exactly to start.

[Animefan8888]

  On 8/30/2019 at 5:02 PM, Eilux said:

not able to punch/attack when they hit left click.

Expand  

Define not able to. Do you mean no animation or no actual damage or knockback.

[Eilux]

no damage, or knockback

[Animefan8888]

  On 8/30/2019 at 5:07 PM, Eilux said:

no damage, or knockback

Expand  

I think you can just cancel the LivingDamageEvent(maybe another one) and no damage will be applied. If that doesnt work for knockback then you can use the KnockBack event.

[Eilux]

the function says that it will still consume absorption hearts and armor durability I need a way to prevent this.

* Also note that appropriate resources (like armor durability and absorption extra hearths) have already been consumed.<br>
...
* If this event is canceled, the Entity is not hurt. Used resources WILL NOT be restored.

 

[DaemonUmbra]

What version of Minecraft are we looking at?

[Eilux]

1.12.2

[DaemonUmbra]

Are we talking about stopping the player attacking all entities, specific entities, entities and blocks, etc.?

[Eilux]

Everything basically so it ignores the click entirely .

[DaemonUmbra]

I think you'll want to listen for both PlayerInteractEvent.LeftClickBlock and AttackEntityEvent

[Eilux]

how do you actually cancel the event.

[Animefan8888]

  On 9/1/2019 at 1:59 PM, Eilux said:

how do you actually cancel the event.

Expand  

event.setCanceled(true);

[Eilux]

I created some test code in my Main however it doesn't seem to work.

 

    @SubscribeEvent
    public void AttackEntityEvent(AttackEntityEvent event){
        event.setCanceled(true);
    }

 

[felinoid]

Try making it static?

[Eilux]

heres the rest of my main if that helps:

package eilux.mod;



import net.minecraftforge.event.entity.player.AttackEntityEvent;
import net.minecraftforge.fml.common.Mod;
import net.minecraftforge.fml.common.event.FMLInitializationEvent;
import net.minecraftforge.fml.common.event.FMLPostInitializationEvent;
import net.minecraftforge.fml.common.event.FMLPreInitializationEvent;
import net.minecraftforge.fml.common.eventhandler.SubscribeEvent;
import org.apache.logging.log4j.LogManager;
import org.apache.logging.log4j.Logger;

@Mod(modid = Main.MODID, name = Main.NAME, version = Main.VERSION, acceptedMinecraftVersions = Main.MC_VERSION)
public class Main {
    public static final String MODID = "mc_general_expansion";
    public static final String NAME = "Minecraft General Expansion";
    public static final String VERSION = "0.0";
    public static final String MC_VERSION = "[1.12.2]";


    public static final Logger LOGGER = LogManager.getLogger(Main.MODID);



    @Mod.EventHandler
    public void preInit(FMLPreInitializationEvent event){

    }

    @Mod.EventHandler
    public void init(FMLInitializationEvent event){
        LOGGER.info(Main.NAME + " says it wants functionality.");
    }

    @Mod.EventHandler
    public void postInit(FMLPostInitializationEvent event){

    }

    @SubscribeEvent
    public void AttackEntityEvent(AttackEntityEvent event) {
        event.setCanceled(true);
    }
}

 

[Animefan8888]

  On 9/2/2019 at 7:49 PM, Eilux said:

public void AttackEntityEvent(AttackEntityEvent event) {

Expand  

You never register this event. Read this.

[Eilux]

I have tried putting the event in an event handler class and registering it but it still isn't working.

package eilux.mod.util;


import net.minecraftforge.event.entity.player.AttackEntityEvent;
import net.minecraftforge.fml.common.Mod;
import net.minecraftforge.fml.common.eventhandler.SubscribeEvent;

@Mod.EventBusSubscriber
public class EventHandler {

    @SubscribeEvent
    public void AttackEntityEvent(AttackEntityEvent event) {
        event.setCanceled(true);
    }
}

 

[Animefan8888]

  On 9/2/2019 at 8:07 PM, Eilux said:

I have tried putting the event in an event handler class and registering it but it still isn't working.

Expand  

You must not have read it thoroughly enough. Because you need to make your method static if you use the EventBusSubscriber

[Eilux]

It's working now. I was wondering if there is also a way to prevent the animation from happening and the attack speed from reseting. Thank you everyone for your help so far.

 

[Differentiation]

  On 9/2/2019 at 8:18 PM, Eilux said:

It's working now. I was wondering if there is also a way to prevent the animation from happening and the attack speed from reseting. Thank you everyone for your help so far.

 

Expand  

I think you can cancel MouseEvent when the player left clicks and that solves your issues. Since the client needs to be notified

Edited September 2, 2019 by Differentiation

[Eilux]

That might work but players could work around it by assigning the attack button to a different key.

[Eilux]

That sounds like it will work I am however having trouble finding where Minecraft is instantiated so I can reflect it.

[Eilux]

I found the method that dose this however it returns Minecraft instead of Class.

[Eilux]

/** The instance of the Minecraft Client, set in the constructor. */
private static Minecraft instance;

...

public static Minecraft getMinecraft()
    {
        return instance;
    }

Here is the code in the Minecraft class.

[Eilux]

when I try to use it with reflection I get the following error: 

code:

            Class<?> minecraftClass = Minecraft.getMinecraft();

error:

Incompatible Types.

required: java.lang.Class

found: net.minecraft.client.Minecraft