Search the Forums BEFORE making a new post.

1.18.2 cannot invoke "net.minecraft.network.FriendlyByteBuf.m_130135_()" because "extraData" is null[SOLVED]

Posted May 1, 20232 yr

I have a working container that displays a screen, however if I try to interact with that screen it causes the error in the title, the container is called from entity#mobInteract using the networkhooks.opengui method. Like I mentioned the screen displays perfectly its just when I click a button it throws this exception

SOLVED - I wasn't properly invoking FriendlyByteBuf in my packets on the screen class, infact I didnt call packets at all like an idiot knowing damn well its client side only.

Log

[02:14:52] [Render thread/FATAL]: Error executing task on Client
java.lang.NullPointerException: Cannot invoke "net.minecraft.network.FriendlyByteBuf.m_130135_()" because "extraData" is null
	at TRANSFORMER/[email protected]/net.xprt.knights.Containers.Container.GuardMenu.<init>(GuardMenu.java:21)
	at TRANSFORMER/[email protected]/net.minecraftforge.network.IContainerFactory.m_39994_(IContainerFactory.java:20)
	at TRANSFORMER/[email protected]/net.minecraft.world.inventory.MenuType.m_39985_(MenuType.java:44)
	at TRANSFORMER/[email protected]/net.minecraft.client.gui.screens.MenuScreens$ScreenConstructor.m_96209_(MenuScreens.java:115)
	at TRANSFORMER/[email protected]/net.minecraft.client.gui.screens.MenuScreens.lambda$create$0(MenuScreens.java:43)
	at java.base/java.util.Optional.ifPresent(Optional.java:178)
	at TRANSFORMER/[email protected]/net.minecraft.client.gui.screens.MenuScreens.m_96201_(MenuScreens.java:43)
	at TRANSFORMER/[email protected]/net.minecraft.client.multiplayer.ClientPacketListener.m_5980_(ClientPacketListener.java:941)
	at TRANSFORMER/[email protected]/net.minecraft.network.protocol.game.ClientboundOpenScreenPacket.m_5797_(ClientboundOpenScreenPacket.java:37)
	at TRANSFORMER/[email protected]/net.minecraft.network.protocol.game.ClientboundOpenScreenPacket.m_5797_(ClientboundOpenScreenPacket.java:11)
	at TRANSFORMER/[email protected]/net.minecraft.network.protocol.PacketUtils.lambda$checkThreadAndEnqueue$0(PacketUtils.java:40)
	at TRANSFORMER/net.optifine/net.optifine.util.PacketRunnable.run(PacketRunnable.java:28)
	at TRANSFORMER/[email protected]/net.minecraft.util.thread.BlockableEventLoop.m_6367_(BlockableEventLoop.java:198)
	at TRANSFORMER/[email protected]/net.minecraft.util.thread.ReentrantBlockableEventLoop.m_6367_(ReentrantBlockableEventLoop.java:23)
	at TRANSFORMER/[email protected]/net.minecraft.util.thread.BlockableEventLoop.m_7245_(BlockableEventLoop.java:163)
	at TRANSFORMER/[email protected]/net.minecraft.util.thread.BlockableEventLoop.m_18699_(BlockableEventLoop.java:140)
	at TRANSFORMER/[email protected]/net.minecraft.client.Minecraft.m_91383_(Minecraft.java:1015)
	at TRANSFORMER/[email protected]/net.minecraft.client.Minecraft.m_91374_(Minecraft.java:665)
	at TRANSFORMER/[email protected]/net.minecraft.client.main.Main.main(Main.java:205)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/java.lang.reflect.Method.invoke(Method.java:568)
	at MC-BOOTSTRAP/[email protected]/net.minecraftforge.fml.loading.targets.CommonClientLaunchHandler.lambda$launchService$0(CommonClientLaunchHandler.java:31)
	at MC-BOOTSTRAP/[email protected]/cpw.mods.modlauncher.LaunchServiceHandlerDecorator.launch(LaunchServiceHandlerDecorator.java:37)
	at MC-BOOTSTRAP/[email protected]/cpw.mods.modlauncher.LaunchServiceHandler.launch(LaunchServiceHandler.java:53)
	at MC-BOOTSTRAP/[email protected]/cpw.mods.modlauncher.LaunchServiceHandler.launch(LaunchServiceHandler.java:71)
	at MC-BOOTSTRAP/[email protected]/cpw.mods.modlauncher.Launcher.run(Launcher.java:106)
	at MC-BOOTSTRAP/[email protected]/cpw.mods.modlauncher.Launcher.main(Launcher.java:77)
	at MC-BOOTSTRAP/[email protected]/cpw.mods.modlauncher.BootstrapLaunchConsumer.accept(BootstrapLaunchConsumer.java:26)
	at MC-BOOTSTRAP/[email protected]/cpw.mods.modlauncher.BootstrapLaunchConsumer.accept(BootstrapLaunchConsumer.java:23)
	at [email protected]/cpw.mods.bootstraplauncher.BootstrapLauncher.main(BootstrapLauncher.java:149)

Container

public class GuardMenu extends AbstractContainerMenu {
    private Knight knight;
    private Player p;
    public GuardMenu(int pContainerId, Inventory inv, Knight k, Player player) {
        this(pContainerId, inv);
        this.knight = k; this.p = player;
    }

    public GuardMenu(int pContainerId, Inventory inv) {
        super(ContainerInit.GUARD_MENU.get(), pContainerId);
    }


    public Player getPlayer(){
        return this.p;
    }

    public Knight getKnight(){
        return this.knight;
    }

    @Override
    public boolean stillValid(Player pPlayer) {
        return true;
    }

}

Screen

public class GuardScreen extends AbstractContainerScreen<GuardMenu> {
    private Knight knight;
    private Player p;

    public GuardScreen(GuardMenu pMenu, Inventory pPlayerInventory, Component pTitle) {
        super(pMenu, pPlayerInventory, pTitle);
        this.knight = pMenu.getKnight(); this.p = pMenu.getPlayer();
        this.init();
    }

    @Override
    public void init() {
        this.addRenderableWidget(new Button(this.width / 2 + 5, this.height / 6 - 12 + 24, 70, 20, Component.nullToEmpty("Stay"), (p_96278_) -> {
            if(knight.getOwner() != null) {
                if (knight.getOwner().equals(p) || knight.getOwner().getId() == p.getId()) {
                    knight.setStay(); knight.getPersistentData().putBoolean(XprtData.KNIGHT_STAYING, true);
                    Utils.MessageEx(p, ChatFormatting.GREEN + "Knight will now stay", true);
                } else {
                    Utils.MessageEx(p, ChatFormatting.RED + "You have not hired this Knight", true);
                }
            } else {
                Utils.MessageEx(p, ChatFormatting.RED + "Knight has not yet been hired", true);
            }
        }));
        this.addRenderableWidget(new Button(this.width / 2 + 5, this.height / 6 - 12 + 60, 70, 20, Component.nullToEmpty("Move Freely"), (p_96278_) -> {
            if(knight.getOwner() != null) {
                if (knight.getOwner().equals(p) || knight.getOwner().getId() == p.getId()) {
                    knight.moveFreely(); knight.getPersistentData().putBoolean(XprtData.KNIGHT_STAYING, false);
                    Utils.MessageEx(p, ChatFormatting.GREEN + "Knight will now move freely", true);
                } else {
                    Utils.MessageEx(p, ChatFormatting.RED + "You have not hired this Knight", true);
                }
            } else {
                Utils.MessageEx(p, ChatFormatting.RED + "Knight has not yet been hired", true);
            }

        }));
        this.addRenderableWidget(new Button(this.width / 2 + 5, this.height / 6 - 12 + 96 , 70, 20, Component.nullToEmpty("Ignore Nearby"), (p_96278_) -> {
            if(knight.getOwner() != null) {
                if (knight.getOwner().equals(p) || knight.getOwner().getId() == p.getId()) {
                    if(!knight.getPersistentData().contains(XprtData.KNIGHT_IGNORE_NEARBY) || !knight.getIgnoreNearby()){
                        knight.setIgnoreNearby(true); knight.getPersistentData().putBoolean(XprtData.KNIGHT_IGNORE_NEARBY, true);
                        Utils.MessageEx(p, ChatFormatting.GREEN + "Knight will now ignore nearby enemies", true);
                    } else{
                        if(knight.getPersistentData().getBoolean(XprtData.KNIGHT_IGNORE_NEARBY) || knight.getIgnoreNearby()){
                            knight.setIgnoreNearby(false); knight.getPersistentData().remove(XprtData.KNIGHT_IGNORE_NEARBY);
                            Utils.MessageEx(p, ChatFormatting.RED + "Knight will no longer ignore nearby enemies", true);
                        }
                    }
                } else {
                    Utils.MessageEx(p, ChatFormatting.RED + "You have not hired this Knight", true);
                }
            } else {
                Utils.MessageEx(p, ChatFormatting.RED + "Knight has not been hired", true);
            }
        }));
        this.addRenderableWidget(new Button(this.width / 2 + 5, this.height / 6 - 12 + 132 , 70, 20, Component.nullToEmpty("Hire Guard"), (p_96278_) -> {
            boolean hasItem = false;
            if(knight.getOwner() == null){
                for(ItemStack items : p.getInventory().items){
                    if(items.getItem().equals(ModItems.GOLD_COIN.get())){
                        hasItem = true;
                        items.setCount(items.getCount()-1);
                    }
                }
                if(hasItem){
                    knight.setOwner(p);
                    Utils.MessageEx(p, ChatFormatting.GREEN + "You have hired a Knight!", true);
                } else{
                    Utils.MessageEx(p, ChatFormatting.RED + "You do not have any gold coins!", true);
                }
            } else {
                Utils.MessageEx(p, ChatFormatting.RED + "This Guard is already hired!", true);
            }
            this.minecraft.setScreen(null);
        }));
        super.init();
    }

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

    @Override
    public void render(PoseStack pPoseStack, int pMouseX, int pMouseY, float pPartialTick) {
        this.renderBackground(pPoseStack);
        super.render(pPoseStack, pMouseX, pMouseY, pPartialTick);
    }

    @Override
    protected void renderBg(PoseStack pPoseStack, float pPartialTick, int pMouseX, int pMouseY) {

    }


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

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


}

Attaching Screen

@Mod.EventBusSubscriber(modid = Main.MOD_ID, bus = Mod.EventBusSubscriber.Bus.MOD, value = Dist.CLIENT)
    public class ClientSetup {
        @SubscribeEvent
        public static void doSetup(FMLClientSetupEvent event) {
            event.enqueueWork( () -> {
                MenuScreens.register(ContainerInit.GUARD_MENU.get(), GuardScreen::new);
            });
            EntityRenderers.register(EntityInit.KNIGHT.get(), KnightRenderer::new);
            EntityRenderers.register(EntityInit.ARCHER.get(), ArcherRenderer::new);
        }
    }

I followed a tutorial perfectly except they are using BlockEntity and im using entity mobInteract and calling player.openMenu

Edited May 2, 20232 yr by sFXprt

Quote

2 yr2 yr sFXprt changed the title to 1.18.2 cannot invoke "net.minecraft.network.FriendlyByteBuf.m_130135_()" because "extraData" is null

May 1, 20232 yr

Quote

ava.lang.NullPointerException: Cannot invoke "net.minecraft.network.FriendlyByteBuf.m_130135_()" because "extraData" is null at TRANSFORMER/[email protected]/net.xprt.knights.Containers.Container.GuardMenu.<init>(GuardMenu.java:21)

I don't see where line 21 could possibly be from what you posted?

In general, if you post random code in the forums most people will just ignore your question unless the problem is obvious from what you post.

So don't post snippets in the forum, put all the relevant code necessary to reproduce the problem on github.

In this case, you don't show 2 of the most important pieces. The code that opens the menu or how you register the MenuType.

https://docs.minecraftforge.net/en/latest/gui/menus/#opening-a-menu

Boilerplate:

If you don't post your logs/debug.log we can't help you. For curseforge you need to enable the forge debug.log in its minecraft settings. You should also post your crash report if you have one.

If there is no error in the log file and you don't have a crash report then post the launcher_log.txt from the minecraft folder. Again for curseforge this will be in your curseforge/minecraft/Install

Large files should be posted to a file sharing site like https://gist.github.com You should also read the support forum sticky post.

Quote

May 2, 20232 yr

Author

@warjort

//Calling 
if(!pPlayer.getLevel().isClientSide())
            NetworkHooks.openGui((ServerPlayer) pPlayer, new SimpleMenuProvider(this.getContainer(), Component.nullToEmpty("Container")));

//Register
public static final RegistryObject<MenuType<GuardMenu>> GUARD_MENU =
            CONTAINERS.register("guard_menu", () -> new MenuType(GuardMenu::new));

Quote

May 2, 20232 yr

Author

13 hours ago, warjort said:

I don't see where line 21 could possibly be from what you posted?

In general, if you post random code in the forums most people will just ignore your question unless the problem is obvious from what you post.

So don't post snippets in the forum, put all the relevant code necessary to reproduce the problem on github.

In this case, you don't show 2 of the most important pieces. The code that opens the menu or how you register the MenuType.

https://docs.minecraftforge.net/en/latest/gui/menus/#opening-a-menu

Im trying to pass an reference to my custom Knight entity to my screen so I can process things

Quote

2 yr2 yr sFXprt changed the title to 1.18.2 cannot invoke "net.minecraft.network.FriendlyByteBuf.m_130135_()" because "extraData" is null[SOLVED]

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.

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

Reply to this topic...

https://forums.minecraftforge.net/topic/123477-1182-cannot-invoke-netminecraftnetworkfriendlybytebufm_130135_-because-extradata-is-nullsolved/

Followers

Go to topic listing

IPS spam blocked by CleanTalk.

Announcements

How to add my mob to another mod's biome in 1.20.1.

I'm on 1.20.1 and was wondering if there was a way to add a mob I am going to make to a other mod's biome (Regions Unexplored in this case). I am trying to figure this out before I start coding the mod. Could someone point me to a good guide on this? I saw some things about biome modifier's but all the resources are either to short, unhelpful or for a different version. Thanks in advance. (i am a beginner but i have java and modding knowledge so this isn't like a "i am new and don't know anything about coding" problem btw)

By ilov3gam3z · Posted 1 hour ago1 hr
incompatible fml modded server and tearing my hair out. Forge 1.16.5 - 36.2.9

Dealing with modded server issues can be a real headache been there! At least you can keep your look stress-free with a fresh low taper fade.

By AriaDawn · 2 hours ago2 hr
Help with file conversion

This is really helpful

By Brialeblanc · 7 hours ago7 hr
Cobblemon invalid player data

Hi everyone, I keep getting "invalid player data" when trying to log onto my friend's server. Log analysis says to remove Night Lights and ImmediatelyFast, but that doesn't work. Any insight is appreciated

Log: https://mclo.gs/uj6pMkd

By CobbleDN · Posted 13 hours ago13 hr
Bad pathfinding for big mobs (1.21.1)

I've struggled with this for way too long now, so I have decided to ask for some assistance here because there is almost no information about this online.

I made a hostile mob and I built it like this:
public static final Supplier<EntityType<Bunyip>> BUNYIP = ENTITY_TYPES.register("bunyip" , () -> EntityType.Builder. of(Bunyip::new, MobCategory.MONSTER).sized(3f, 2.5f).eyeHeight(1.75f).build("bunyip"));

However, I have discovered that a BoundingBox of 3 is too big for the vanilla pathfinding to work well. In fact, anything bigger than a block will not work well. This is especially noticeable when having it chase a player in a negative X and/or negative Z direction. The mob starts spinning, walking backwards, walking diagonally, etc.

I am using the Goal system:
protected void registerGoals() { super.registerGoals(); this.goalSelector.addGoal(1, new MeleeAttackGoal(this, 1.25, true)); this.targetSelector.addGoal(2, new NearestAttackableTargetGoal<>(this, Player.class, true, true)); }

This does not seem to be a me issue as the ravager, one of the biggest vanilla mobs, also struggles with proper pathfinding. I am a bit familiar with how Minecraft's pathfinding works, but not enough to make a pathfinding system from scratch (or at least not in a reasonable amount of time). Is there any way to fix this with some tweaks or will I have to make a custom pathfinding system?

By VictorEvers · Posted 21 hours ago21 hr

There are no registered users currently online

Sign In

1.18.2 cannot invoke "net.minecraft.network.FriendlyByteBuf.m_130135_()" because "extraData" is null[SOLVED]

Featured Replies

Join the conversation

Posts

How to add my mob to another mod's biome in 1.20.1.

incompatible fml modded server and tearing my hair out. Forge 1.16.5 - 36.2.9

Help with file conversion

Cobblemon invalid player data

Bad pathfinding for big mobs (1.21.1)

Who's Online (See full list)

Important Information

Account

Navigation

Search

Configure browser push notifications

Chrome (Android)

Chrome (Desktop)

Safari (iOS 16.4+)

Safari (macOS)

Edge (Android)

Edge (Desktop)

Firefox (Android)

Firefox (Desktop)