Jump to content
Search In
  • More options...
Find results that contain...
Find results in...

[1.16.4] SimplImpl sends packet to client instead server

Recommended Posts

I implementing screen, where user may select option and server should be notified about it. Simple case. I found tutorial and implemented this feature uses SimpleImp. Maybe I missed something - packet from server sends to client successfully, but when I tries to use sendToServer client get message instead server.


My network class

public class ModNetworking {

    private static final String PROTOCOL_VERSION = "1.0";
    private static SimpleChannel INSTANCE;
    private static int ID = 0;

    public static int nextId () { return ID++; }

    public static void registerMessages() {
        INSTANCE = NetworkRegistry.newSimpleChannel(new ResourceLocation(ArtOfFood.MOD_ID, "mod_channel"),
                () -> PROTOCOL_VERSION,

        INSTANCE.messageBuilder(SSetConceptResultSlotPacket.class, nextId())

        INSTANCE.messageBuilder(CUpdateConceptPacket.class, nextId())

    public static void sendToClient(Object packet, ServerPlayerEntity player) {
        INSTANCE.sendTo(packet, player.connection.netManager, NetworkDirection.PLAY_TO_CLIENT);

    public static void sendToServer(Object packet) {


Problem packet is CUpdateConceptPacket:

public class CUpdateConceptPacket {

    private String conceptId;

    public CUpdateConceptPacket(String conceptId) {
        this.conceptId = conceptId;

    public void processPacket(Supplier<NetworkEvent.Context> ctx) {
        PlayerEntity playerEntity = Minecraft.getInstance().player;
        if (playerEntity.openContainer instanceof IUpdateConceptPacketHandler) {
            IUpdateConceptPacketHandler handler = (IUpdateConceptPacketHandler) playerEntity.openContainer;

    public CUpdateConceptPacket(PacketBuffer buf) {

    public void readPacketData(PacketBuffer buf) {
        this.conceptId = buf.readString();

    public void writePacketData(PacketBuffer buf) {

    public String getConceptId() {
        return this.conceptId;


Minimalistic handler:

public interface IUpdateConceptPacketHandler {

    void handleConceptUpdate(String conceptId);


This is how I call it:

public void setConcept(MBConcept concept) {
    this.concept = concept;
    CUpdateConceptPacket packet = new CUpdateConceptPacket(concept.conceptId);


And get client call (see screen, at debug value of playerEntity is ClientPlayerEntity)

P.S. Can't find spoiler for hiding code listings. Is it available here? And ofcouse I tried to investigate Vanila examples few times but didn't get results.


Edited by S-Spirit
Link to post
Share on other sites
7 hours ago, diesieben07 said:
  • Do not use OnlyIn.
  • Minecraft#player is always the client player. By accessing it on a packet thats sent to the server, you are reaching across logical sides, which will just crash on dedicated servers and cause strange behavior in single player.

Thank you very much! You was right - problem was related to code Minecraft.getInstance().player. I Have read page about networking few times and missed this example of common beginners mistakes...

But how I should get opened container at server side? I added list of listeners into ModNetworking 

private static final NonNullList<WeakReference<Object>> serverListeners = NonNullList.create();

public static void addServerListener(Object obj) {
	serverListeners.add(new WeakReference(obj));

public static void removeServerListener(Object obj) {

public static <T> Stream<T> listeners(Class<T> listenerClass) {
    serverListeners.removeIf(r -> r.get() == null);
    return serverListeners.stream()
            .filter(r -> listenerClass.isAssignableFrom(r.get().getClass()))
            .map(r -> ((T) r.get()));


At packet processing ModNetworking provides listeners of necessary type:

public void processPacket(Supplier<NetworkEvent.Context> ctx) {
    Stream<IUpdateConceptPacketHandler> listeners = ModNetworking.listeners(IUpdateConceptPacketHandler.class);
    listeners.forEach( l -> l.handleConceptUpdate(conceptId));


My container add itself like a listener in constructor and remove at the finalize(). This solution works, but is it brilliant idea or ugly implementation of something, which platform already may provide to me?


P.S. Why you notify me to not use @OnlyIn? As I understood, it should be written everywhere, in case when method should be called only from client. Is it wrong vision?

Link to post
Share on other sites

Who told you to use OnlyIn everywhere?

This is my Forum Signature, I am currently attempting to transform it into a small guide for fixing easier issues using spoiler blocks to keep things tidy.


As the most common issue I feel I should put this outside the main bulk:

The only official source for Forge is https://files.minecraftforge.net, and the only site I trust for getting mods is CurseForge.

If you use any site other than these, please take a look at the StopModReposts project and install their browser extension, I would also advise running a virus scan.


For players asking for assistance with Forge please expand the spoiler below and read the appropriate section(s) in its/their entirety.


Logs (Most issues require logs to diagnose):


Please post logs using one of the following sites (Thank you Lumber Wizard for the list):

https://gist.github.com/100MB Requires member (Free)

https://pastebin.com/: 512KB as guest, 10MB as Pro ($$$)

https://hastebin.com/: 400KB

Do NOT use sites like Mediafire, Dropbox, OneDrive, Google Drive, or a site that has a countdown before offering downloads.


What to provide:

...for Crashes and Runtime issues:

Minecraft 1.14.4 and newer:

Post debug.log

Older versions:

Please update...


...for Installer Issues:

Post your installer log, found in the same place you ran the installer

This log will be called either installer.log or named the same as the installer but with .log on the end

Note for Windows users:

Windows hides file extensions by default so the installer may appear without the .jar extension then when the .log is added the log will appear with the .jar extension


Where to get it:

Mojang Launcher: When using the Mojang launcher debug.log is found in .minecraft\logs.


Curse/Overwolf: If you are using the Curse Launcher, their configurations break Forge's log settings, fortunately there is an easier workaround than I originally thought, this works even with Curse's installation of the Minecraft launcher as long as it is not launched THROUGH Twitch:

  1. Make sure you have the correct version of Forge installed (some packs are heavily dependent on one specific build of Forge)
  2. Make a launcher profile targeting this version of Forge.
  3. Set the launcher profile's GameDir property to the pack's instance folder (not the instances folder, the folder that has the pack's name on it).
  4. Now launch the pack through that profile and follow the "Mojang Launcher" instructions above.






or alternately, 


Fallback ("No logs are generated"):

If you don't see logs generated in the usual place, provide the launcher_log.txt from .minecraft


Server Not Starting:


If your server does not start or a command window appears and immediately goes away, run the jar manually and provide the output.


Reporting Illegal/Inappropriate Adfocus Ads:


Get a screenshot of the URL bar or copy/paste the whole URL into a thread on the General Discussion board with a description of the Ad.

Lex will need the Ad ID contained in that URL to report it to Adfocus' support team.


Posting your mod as a GitHub Repo:


When you have an issue with your mod the most helpful thing you can do when asking for help is to provide your code to those helping you. The most convenient way to do this is via GitHub or another source control hub.

When setting up a GitHub Repo it might seem easy to just upload everything, however this method has the potential for mistakes that could lead to trouble later on, it is recommended to use a Git client or to get comfortable with the Git command line. The following instructions will use the Git Command Line and as such they assume you already have it installed and that you have created a repository.


  1. Open a command prompt (CMD, Powershell, Terminal, etc).
  2. Navigate to the folder you extracted Forge’s MDK to (the one that had all the licenses in).
  3. Run the following commands:
    1. git init
    2. git remote add origin [Your Repository's URL]
      • In the case of GitHub it should look like: https://GitHub.com/[Your Username]/[Repo Name].git
    3. git fetch
    4. git checkout --track origin/master
    5. git stage *
    6. git commit -m "[Your commit message]"
    7. git push
  4. Navigate to GitHub and you should now see most of the files.
    • note that it is intentional that some are not synced with GitHub and this is done with the (hidden) .gitignore file that Forge’s MDK has provided (hence the strictness on which folder git init is run from)
  5. Now you can share your GitHub link with those who you are asking for help.

[Workaround line, please ignore]


Link to post
Share on other sites
38 minutes ago, diesieben07 said:

On the server NetworkEvent.Context#getSender gives you the player that sent the packet.

Thanks, got it!


8 hours ago, DaemonUmbra said:

Who told you to use OnlyIn everywhere?

No one. It was my own conclusion from documentation and tutorials. When you asked, I read about @OnlyIn one more time, and see - my conclusion was fully wrong. I even thinked @OnlyIn work with logical sides, but specification says - physical. So I should make a refactorng. Thanks for get my attention.

Link to post
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

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.

  • Recently Browsing

    No registered users viewing this page.

  • Posts

    • Ok so, i think that the problem is i register a ISTER Property of the Item while i initialize the server. But how can I separate item/block registration between client and server?
    • Hi! I had this mod that adds new chest, and when i launch it in a single player world it works good. But when i try to load it on a server, i receive this error: 13.05 11:48:37 [Server] modloading-worker-11/INFO [ne.mi.co.ForgeMod/FORGEMOD]: Forge mod loading, version 36.1.13, for MC 1.16.5 with MCP 20210115.111550 13.05 11:48:37 [Server] modloading-worker-11/INFO [ne.mi.co.MinecraftForge/FORGE]: MinecraftForge v36.1.13 Initialized 13.05 11:48:37 [Server] modloading-worker-2/ERROR [ne.mi.fm.ja.FMLModContainer/LOADING]: Failed to create mod instance. ModID: tenchest, class com.ike.tenchest.TenChest 13.05 11:48:37 [Server] INFO java.lang.BootstrapMethodError: java.lang.IllegalAccessError: no such constructor: com.ike.tenchest.minichest.MiniChest.<init>()void/newInvokeSpecial 13.05 11:48:37 [Server] INFO at com.ike.tenchest.Blocks.<clinit>(Blocks.java:63) ~[tenchest:1.0] {re:classloading,pl:runtimedistcleaner:A} 13.05 11:48:37 [Server] INFO at com.ike.tenchest.TenChest.<init>(TenChest.java:54) ~[tenchest:1.0] {re:classloading} 13.05 11:48:37 [Server] INFO at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method) ~[?:1.8.0_292] {} 13.05 11:48:37 [Server] INFO at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62) ~[?:1.8.0_292] {} 13.05 11:48:37 [Server] INFO at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45) ~[?:1.8.0_292] {} 13.05 11:48:37 [Server] INFO at java.lang.reflect.Constructor.newInstance(Constructor.java:423) ~[?:1.8.0_292] {} 13.05 11:48:37 [Server] INFO at java.lang.Class.newInstance(Class.java:442) ~[?:1.8.0_292] {} 13.05 11:48:37 [Server] INFO at net.minecraftforge.fml.javafmlmod.FMLModContainer.constructMod(FMLModContainer.java:81) ~[forge:36.1] {re:classloading} 13.05 11:48:37 [Server] INFO at net.minecraftforge.fml.ModContainer.lambda$buildTransitionHandler$4(ModContainer.java:120) ~[forge:?] {re:classloading} 13.05 11:48:37 [Server] INFO at java.util.concurrent.CompletableFuture$AsyncRun.run(CompletableFuture.java:1640) [?:1.8.0_292] {} 13.05 11:48:37 [Server] INFO at java.util.concurrent.CompletableFuture$AsyncRun.exec(CompletableFuture.java:1632) [?:1.8.0_292] {} 13.05 11:48:37 [Server] INFO at java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:289) [?:1.8.0_292] {} 13.05 11:48:37 [Server] INFO at java.util.concurrent.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1056) [?:1.8.0_292] {} 13.05 11:48:37 [Server] INFO at java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1692) [?:1.8.0_292] {} 13.05 11:48:37 [Server] INFO at java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:175) [?:1.8.0_292] {} 13.05 11:48:37 [Server] INFO Caused by: java.lang.IllegalAccessError: no such constructor: com.ike.tenchest.minichest.MiniChest.<init>()void/newInvokeSpecial 13.05 11:48:37 [Server] INFO at java.lang.invoke.MethodHandleNatives.linkMethodHandleConstant(MethodHandleNatives.java:483) ~[?:1.8.0_292] {} 13.05 11:48:37 [Server] INFO ... 15 more 13.05 11:48:37 [Server] INFO Caused by: java.lang.NoClassDefFoundError: net/minecraft/tileentity/IChestLid 13.05 11:48:37 [Server] INFO at java.lang.ClassLoader.defineClass1(Native Method) ~[?:1.8.0_292] {} 13.05 11:48:37 [Server] INFO at java.lang.ClassLoader.defineClass(ClassLoader.java:756) ~[?:1.8.0_292] {} 13.05 11:48:37 [Server] INFO at cpw.mods.modlauncher.TransformingClassLoader.loadClass(TransformingClassLoader.java:138) ~[modlauncher-8.0.9.jar:?] {re:classloading} 13.05 11:48:37 [Server] INFO at cpw.mods.modlauncher.TransformingClassLoader.loadClass(TransformingClassLoader.java:98) ~[modlauncher-8.0.9.jar:?] {re:classloading} 13.05 11:48:37 [Server] INFO at java.lang.ClassLoader.loadClass(ClassLoader.java:351) ~[?:1.8.0_292] {} 13.05 11:48:37 [Server] INFO at net.minecraftforge.eventbus.EventSubclassTransformer.buildEvents(EventSubclassTransformer.java:62) ~[eventbus-4.0.0.jar:?] {} 13.05 11:48:37 [Server] INFO at net.minecraftforge.eventbus.EventSubclassTransformer.transform(EventSubclassTransformer.java:44) ~[eventbus-4.0.0.jar:?] {} 13.05 11:48:37 [Multicraft] Skipped 83 lines due to rate limit (30/s) 13.05 11:48:37 [Server] main/FATAL [ne.mi.fm.ModLoader/LOADING]: Failed to complete lifecycle event CONSTRUCT, 1 errors found 13.05 11:48:37 [Server] main/FATAL [ne.mi.co.ForgeMod/]: Preparing crash report with UUID 449cf277-bc9c-4a21-b16a-3f07d0d4e0c4 13.05 11:48:37 [Server] main/INFO [STDOUT/]: [net.minecraft.crash.CrashReport:func_85057_a:196]: Negative index in crash report handler (16/18) 13.05 11:48:37 [Server] main/FATAL [ne.mi.fm.se.ServerModLoader/]: Crash report saved to ./crash-reports/crash-2021-05-13_11.48.37-fml.txt 13.05 11:48:37 [Server] INFO ---- Minecraft Crash Report ---- 13.05 11:48:37 [Server] INFO // Daisy, daisy... 13.05 11:48:37 [Server] INFO Time: 5/13/21 11:48 AM 13.05 11:48:37 [Server] INFO Description: Mod loading error has occurred 13.05 11:48:37 [Server] INFO java.lang.Exception: Mod Loading has failed 13.05 11:48:37 [Server] INFO at net.minecraftforge.fml.CrashReportExtender.dumpModLoadingCrashReport(CrashReportExtender.java:85) ~[forge:?] {re:classloading} 13.05 11:48:37 [Server] INFO at net.minecraftforge.fml.server.ServerModLoader.load(ServerModLoader.java:51) ~[forge:?] {re:classloading} 13.05 11:48:37 [Server] INFO at net.minecraft.server.Main.main(Main.java:95) ~[?:?] {re:classloading} 13.05 11:48:37 [Server] INFO at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:1.8.0_292] {} 13.05 11:48:37 [Server] INFO at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) ~[?:1.8.0_292] {} 13.05 11:48:37 [Server] INFO at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:1.8.0_292] {} 13.05 11:48:37 [Server] INFO at java.lang.reflect.Method.invoke(Method.java:498) ~[?:1.8.0_292] {} 13.05 11:48:37 [Server] INFO at net.minecraftforge.fml.loading.FMLServerLaunchProvider.lambda$launchService$0(FMLServerLaunchProvider.java:51) ~[forge_1.16.5.jar:36.1] {} 13.05 11:48:37 [Server] INFO at cpw.mods.modlauncher.LaunchServiceHandlerDecorator.launch(LaunchServiceHandlerDecorator.java:37) [modlauncher-8.0.9.jar:?] {} 13.05 11:48:37 [Server] INFO at cpw.mods.modlauncher.LaunchServiceHandler.launch(LaunchServiceHandler.java:54) [modlauncher-8.0.9.jar:?] {} 13.05 11:48:37 [Server] INFO at cpw.mods.modlauncher.LaunchServiceHandler.launch(LaunchServiceHandler.java:72) [modlauncher-8.0.9.jar:?] {} 13.05 11:48:37 [Server] INFO at cpw.mods.modlauncher.Launcher.run(Launcher.java:82) [modlauncher-8.0.9.jar:?] {} 13.05 11:48:37 [Server] INFO at cpw.mods.modlauncher.Launcher.main(Launcher.java:66) [modlauncher-8.0.9.jar:?] {} 13.05 11:48:37 [Server] INFO at net.minecraftforge.server.ServerMain$Runner.runLauncher(ServerMain.java:63) [forge_1.16.5.jar:?] {} 13.05 11:48:37 [Server] INFO at net.minecraftforge.server.ServerMain$Runner.access$100(ServerMain.java:60) [forge_1.16.5.jar:?] {} 13.05 11:48:37 [Server] INFO at net.minecraftforge.server.ServerMain.main(ServerMain.java:57) [forge_1.16.5.jar:?] {} 13.05 11:48:37 [Server] INFO A detailed walkthrough of the error, its code path and all known details is as follows: 13.05 11:48:38 [Multicraft] Server shut down (running) 13.05 11:48:38 [Multicraft] Restarting crashed server in 300 seconds 13.05 11:48:38 [Multicraft] Server stopped 13.05 11:50:33 [Multicraft] Received kill command 13.05 11:50:33 [Multicraft] Server stopped What can i do?
    • This is wrong, they still ship J8 for <1.17 So for any of the versions that arnt the latest snapshot use J8.
  • Topics

  • Who's Online (See full list)

  • Create New...

Important Information

By using this site, you agree to our Privacy Policy.