[1.12.2] Packet to play sound in world

[Naturix]

I am fairly new to the whole networking side of forge, i am unsure why my mod is crashing when the keybind is pushed. The sound works in /playsound but something about my packet sending is off. 

https://github.com/NicosaurusRex99/farts

[Draco18s]

https://github.com/NicosaurusRex99/farts/blob/1.12.2/src/main/java/naturix/farts/network/PacketSendKey.java#L38-L47

 

Did you miss the warning?

 

 

Quote

 

Warning

As of Minecraft 1.8 packets are by default handled on the network thread.

That means that your IMessageHandler can not interact with most game objects directly. Minecraft provides a convenient way to make your code execute on the main thread instead using IThreadListener.addScheduledTask.

The way to obtain an IThreadListener is using either the Minecraft instance (client side) or a WorldServer instance (server side). The code above shows an example of this by getting a WorldServer instance from an EntityPlayerMP.

 

 

[Naturix]

6 hours ago, Draco18s said:

https://github.com/NicosaurusRex99/farts/blob/1.12.2/src/main/java/naturix/farts/network/PacketSendKey.java#L38-L47

 

Did you miss the warning?

 

 

 

Turns out yes, i did miss it.

Where i have nullified something causes this error, i am not sure what to put in that slot

https://github.com/NicosaurusRex99/farts/blob/1.12.2/src/main/java/naturix/farts/network/PacketSendKey.java#L50

https://gist.github.com/NicosaurusRex99/230851c4c0411485a67e10d008061a3f

 

Edited January 6, 2018 by Naturix
Not full post

[Draco18s]

You put your code in there.

 

Quote

    serverPlayer.getServerWorld().addScheduledTask(() -> {
      serverPlayer.inventory.addItemStackToInventory(new ItemStack(Items.DIAMOND, amount));
    });

 

 

[Naturix]

50 minutes ago, Draco18s said:

You put your code in there.

 

 

Thanks for your help, its not crashing anymore atleast

[Naturix]

Incase you were wondering (probs not) it works now Github