Skip to content
This repository has been archived by the owner on Feb 27, 2024. It is now read-only.

[1.8.9] Thaumcraft selecting focus on wand not work behind BungeeCord #731

Closed
ghost opened this issue Jul 5, 2016 · 23 comments
Closed

[1.8.9] Thaumcraft selecting focus on wand not work behind BungeeCord #731

ghost opened this issue Jul 5, 2016 · 23 comments

Comments

@ghost
Copy link

ghost commented Jul 5, 2016

Tested with pure forge server - no problems. When add spongeforge (no another plugins) then cannot change focuses on the Thaumcraft wand wand. Looks like spongeforge break clicking in the selecting gui.

Steps to reproduce:

  • give to inventory some focuses and wand.
    • hold wand
    • press F
    • then try to click focus

expected result is change focus to clicked.

SF(build 1538)

@LuxusDarkangel
Copy link

Probably an issue with packets, as happened with BungeeCord. Please, post your server log.

@gabizou
Copy link
Member

gabizou commented Jul 5, 2016

I've got a fairly decent idea as to what's going on and how to handle it for 1.10.2+. @bloodmc basically, when a foreign packet is received, we should be checking the content changes for the player's inventories and throwing events accordingly. In this case, Thaumcraft is changing the itemstack in the player's main hand and modifying the tag.

@ryantheleach
Copy link
Contributor

But couldn't that foreign packet be doing a number of different things? Won't some foreign packets get spammed a lot?

@gabizou
Copy link
Member

gabizou commented Jul 6, 2016

Yes, @ryantheleach but without being able to tell when something happens, since itemstacks are interacted with directly, i's not really well possible unless we basically flag to capture anything that does change during the packet being handled.

@ghost
Copy link
Author

ghost commented Jul 6, 2016

In the server is no exceptions, errors etc. Yes its behind bungeecord.

@ghost
Copy link
Author

ghost commented Jul 6, 2016

Yay, without bungeecord it works. Im using SpongePls plugin for forwarding in the BungeeCord. In the logs of sponge and bungee see no errors or exceptions.

@ghost ghost changed the title [1.8.9] SpongeForge break Thaumcraft selecting focus on wand [1.8.9] Thaumcraft selecting focus on wand not work behind BungeeCord Jul 6, 2016
@SilverCory
Copy link

This is probably something to contact the thaumcraft author about. I've no clue of it's protocol or how it works, so the chances of me fixing this are slim.

@LuxusDarkangel
Copy link

LuxusDarkangel commented Jul 6, 2016

The issue is with BungeeCord, not SpongeForge, SpongePls or Thaumcraft. See this.

So... SpongeCoord when?

@bloodmc
Copy link
Contributor

bloodmc commented Jul 6, 2016

@dualspiral Can you look into this with your BungeeCord fork?

@dualspiral
Copy link
Contributor

@bloodmc When I get chance I'll take a look - though @hunter688, have you tried this using Waterfall instead of Bungee yet?

@Zidane
Copy link
Member

Zidane commented Jul 6, 2016

Closing this but discussion can continue as it is not a Sponge issue and is a bungee issue.

@Zidane Zidane closed this as completed Jul 6, 2016
@ghost
Copy link
Author

ghost commented Jul 6, 2016

Found source of problem. Lobby server is 1.10.2 with protocolsupport plugin. When connect through this server, got issue. When remove protocolsupport and join to modded server as fallback, all is ok.

@thomas15v
Copy link

Took me a while to figure out this issue. But the problem isn't bungeecord nor thaumcraft. The problem lies in the minecraft protocol and the challenge bungeecord brings.

When a player join a server on bungeecord, it gets asigned a entity id on that specific server. However when the client switches to another server the client id on the new server will be different. Bungee itself counteracts this problem by rewriting all know packets and changing their id's.

However thaumcraft is one of those mods that uses the entity id of the player for some of it's packets. What causes everything to go haywire when a player switches servers. As a proof of concept I also fixed the problem in the form of a bungeecord plugin: https://github.com/densitycraft/ForgePacketFix

@ghost
Copy link
Author

ghost commented Sep 27, 2016

Hey Thanks a lot!

@ghost
Copy link
Author

ghost commented Sep 27, 2016

Looks like it does not work for me. :(

Latest Waterfall, ForgePacketFix, lobby server with 1.10.2 spigot and latest protocolsupport, latest 1.8.9 sponge with thaumcraft. Still cannot change focuses.

@thomas15v
Copy link

Yeah I tested this for 1.7.10. I guess the packet id's for the new thaumcraft are different. Ill add the id's for 1.8.9 tho. See: https://github.com/densitycraft/ForgePacketFix/issues/1.

@ghost
Copy link
Author

ghost commented Sep 27, 2016

Great!

@RhythmNz
Copy link

@thomas15v https://github.com/densitycraft/ForgePacketFix/ is 404
Do you have a file?

@thomas15v
Copy link

@RhythmNz I removed that repo with the densitycraft group. Also I am not re-uploading that repo since it is not the correct way to fix it, see: MinecraftForge/MinecraftForge#3284.

By modifying bungeecord to send the correct packet on server switch, you can easily fix the problem. However I am pretty sure that change will never get pulled in the original bungeecord since it breaks the team api inside bungee. Wich is also the reason why I never send a PR.

@ghost
Copy link
Author

ghost commented Dec 12, 2016

@RhythmNz
Copy link

@thomas15v You mean I can write a plugin to fix?
I will try it thanks

@thomas15v
Copy link

@RhythmNz Ehm no. You have to path your bungee in order to support it. This is the path I applied on my server: https://gist.github.com/thomas15v/3768613bce327caa2f46b319eadea615.

@RhythmNz
Copy link

RhythmNz commented Jan 1, 2017

@thomas15v Do you have the finished version of the Jar. For 1.7.10
I still don't learn how to use Git

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

9 participants