Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Playerside sign update #697

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

Intybyte
Copy link
Contributor

@Intybyte Intybyte commented Aug 20, 2024

Describe in detail what your pull request accomplishes

Use a bukkit method to force the player to update the signs instead

Related issues:

Checklist

  • Tested
  • Performance tested

@TylerS1066
Copy link
Contributor

I ran a test and this does not fully work yet. It's better than before (without any update packets), but not fully working.

@Intybyte
Copy link
Contributor Author

not fully working.
What do you mean with not fully working? like sometimes it works sometimes it doesn't?

@TylerS1066
Copy link
Contributor

What do you mean with not fully working? like sometimes it works sometimes it doesn't?

Correct, it worked some of the time but not all. I didn't try it for long, but there didn't seem to be a pattern (I was just cruising a ship a bit and sometimes the sign would update, sometimes not).

@oh-noey
Copy link
Collaborator

oh-noey commented Aug 30, 2024

the api you're using is marked as deprecated in recent javadoc, which could signal why its broken. Try using the alternative method they recommend instead.

@Intybyte
Copy link
Contributor Author

the api you're using is marked as deprecated in recent javadoc, which could signal why its broken. Try using the alternative method they recommend instead.

Ok just read the java docs

Use sendBlockUpdate(Location, TileState) by creating a new virtual Sign block state via BlockData.createBlockState() (constructed e.g.

I will try using the suggested method, I hope the createBlockState() calls aren't as expensive as the original server update call otherwise it would be a waste of time

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

Successfully merging this pull request may close these issues.

Remove unnecessary sign updates
3 participants