SOLVED: My custom item that overrides livingInteract method conflicted with my custom entitys mobInteract method causing weird stuff to happen

 

Ok I have a weird glitch, whenever I first call my screen it works fine and the mouse is stuck to the screen and I cant move my player at all but once I reload the world and try to interact with an entity and call the screen it glitches out and I cant click any of the buttons because whenever I move my keyboard or mouse its moving my player and my camera and whenever I click escape the screen pops up over and over and over and when the screen actually closes for good and I look at an entity it pops up the screen again. I think this is an issue with the interaction method im using, how do I specify on rightClick only?

 

GuardScreen

public GuardScreen(Knight knight, Player p) {
        super(Component.nullToEmpty("Guard"));
        this.minecraft = Minecraft.getInstance();
        this.addRenderableWidget(new Button(this.width / 2 + 5, this.height / 6 - 12 + 24, 70, 20, Component.nullToEmpty("Stay"), (p_96278_) -> {
            knight.setStay();
            Utils.MessageEx(p, ChatFormatting.GREEN + "Knight will now stay", true);
        }));
        this.addRenderableWidget(new Button(this.width / 2 + 5, this.height / 6 - 12 + 60, 70, 20, Component.nullToEmpty("Move Freely"), (p_96278_) -> {
            knight.moveFreely();
            Utils.MessageEx(p, ChatFormatting.GREEN + "Knight will now move freely", true);
        }));
    }
    public GuardScreen(Archer archer, Player p) {
        super(Component.nullToEmpty("Archer"));
        this.minecraft = Minecraft.getInstance();
        this.addRenderableWidget(new Button(this.width / 2 + 5, this.height / 6 - 12 + 24, 70, 20, Component.nullToEmpty("Stay"), (p_96278_) -> {
            archer.setStay();
            Utils.MessageEx(p, ChatFormatting.GREEN + "Archer will now stay", true);
        }));
        this.addRenderableWidget(new Button(this.width / 2 + 5, this.height / 6 - 12 + 60, 70, 20, Component.nullToEmpty("Move Freely"), (p_96278_) -> {
            archer.moveFreely();
            Utils.MessageEx(p, ChatFormatting.GREEN + "Archer will now move freely", true);
        }));
    }

    @Override
    public boolean shouldCloseOnEsc() {
        return true;
    }

    @Override
    public void onClose() {
        this.minecraft.setScreen(null);
    }

 

Calling Method

public void interactedWith(Player pPlayer){
        if(this.getOwner() != null){
            if(this.getOwner().equals(pPlayer) || this.getOwner().getId() == pPlayer.getId()){
                LocalPlayer lPlayer = Minecraft.getInstance().player;
                if(lPlayer.equals(pPlayer) || lPlayer.getId() == pPlayer.getId() || lPlayer.getUUID().equals(pPlayer.getUUID())){
                    Minecraft.getInstance().setScreen(new GuardScreen(this, pPlayer));
                }
            }
        }
    }

Item interactionMethod(referring to the comment below)

public InteractionResult interactLivingEntity(ItemStack pStack, Player pPlayer, LivingEntity pInteractionTarget, InteractionHand pUsedHand) {
        if(pInteractionTarget instanceof Knight k){
            if(k.getOwner() == null){
                k.setOwner(pPlayer);
                if(pPlayer.getMainHandItem() != null){
                    if(pPlayer.getMainHandItem().getItem().equals(ModItems.GOLD_COIN.get())){
                        pPlayer.getMainHandItem().setCount(pPlayer.getMainHandItem().getCount()-1);
                    }
                }
                Utils.MessageEx(pPlayer, ChatFormatting.GREEN + "You have hired a Knight", true);
            }
        } else if(pInteractionTarget instanceof Archer a){
            if(a.getOwner() == null){
                a.setOwner(pPlayer);
                if(pPlayer.getMainHandItem() != null){
                    if(pPlayer.getMainHandItem().getItem().equals(ModItems.GOLD_COIN.get())){
                        pPlayer.getMainHandItem().setCount(pPlayer.getMainHandItem().getCount()-1);
                    }
                }
                Utils.MessageEx(pPlayer, ChatFormatting.GREEN + "You have hired an Archer", true);
            }
        }
        return super.interactLivingEntity(pStack, pPlayer, pInteractionTarget, pUsedHand);
    }

 

Edited May 1, 20232 yr by sFXprt

Cant find a fix, I found a weird fix tho if I have a custom item that overrides interactLivingEntity and I try to open the screen it works perfectly and once I do that I can open the screen with any or no item perfectly.  Its only when I try to open the screen without that item in my hand in the first place

 

The Interaction Method will be posted in the original topic

Edited May 1, 20232 yr by sFXprt