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

Fix trade signs not updating with full inventories #5574

Merged
merged 1 commit into from
Nov 11, 2023

Conversation

JRoy
Copy link
Member

@JRoy JRoy commented Nov 11, 2023

Starting with 1.19, sign block states are no longer persistent causing them to become dead after any call to Sign#update. To avoid this, ensure we either
always get the latest block state or update the block state after any call to Sign#update.

Fixes #5550

Starting with 1.19, sign block states are no longer
persistent causing them to become dead after any call
to Sign#update. To avoid this, ensure we either
always get the latest block state or update the block
state after any call to Sign#update.
@JRoy JRoy added type: bugfix PRs that fix bugs in EssentialsX. module: main Issues or PRs for the main Essentials module labels Nov 11, 2023
@JRoy JRoy added this to the 2.20.2 milestone Nov 11, 2023
@JRoy JRoy enabled auto-merge (squash) November 11, 2023 02:02
@JRoy JRoy closed this Nov 11, 2023
auto-merge was automatically disabled November 11, 2023 02:19

Pull request was closed

@JRoy JRoy reopened this Nov 11, 2023
@JRoy JRoy merged commit c68b277 into EssentialsX:2.x Nov 11, 2023
1 check passed
@JRoy JRoy deleted the fix/trade-sign-updates branch November 11, 2023 02:23
HarvelsX pushed a commit to HarvelsX/Essentials that referenced this pull request Jun 2, 2024
Starting with 1.19, sign block states are no longer persistent causing them to become dead after any call to Sign#update. 
To avoid this, ensure we either always get the latest block state or update the block state after any call to Sign#update.
Starmism pushed a commit to valence-smp/Essentials that referenced this pull request Jan 8, 2025
Starting with 1.19, sign block states are no longer persistent causing them to become dead after any call to Sign#update. 
To avoid this, ensure we either always get the latest block state or update the block state after any call to Sign#update.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module: main Issues or PRs for the main Essentials module type: bugfix PRs that fix bugs in EssentialsX.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Money duplication
2 participants