Skip to content

Remove obsolete MC-4 workaround - #14273

Merged
Doc94 merged 1 commit into
PaperMC:dev/26.3from
zzzzzyc:fix-mc-4-cleanup
Sep 18, 2026
Merged

Doc94 merged 1 commit into
PaperMC:dev/26.3from
zzzzzyc:fix-mc-4-cleanup

Conversation

@zzzzzyc

@zzzzzyc zzzzzyc commented Sep 17, 2026 •

Copy link
Copy Markdown
Contributor

Description:

As mentioned in #14260, MC-4 is a bug that has existed in the vanilla game for many years. Paper previously added the misc.send-full-pos-for-item-entities configuration, which marks all dropped items to require the sending of precise coordinates as a temporary workaround.

However, in Minecraft 26.2+ (and 26.3), Mojang officially added a check for precision loss during collision detection specifically for ItemEntity in ServerEntity#isFullPrecisionEncodingRequired. When precision loss is detected, a full-precision synchronization packet is automatically sent. The original version has fundamentally fixed MC-4.

Paper’s patch and corresponding configuration are now completely obsolete and redundant and can be safely removed.

This PR makes the following changes:

  • Removed the patch that sets requiresPrecisePosition(true) in ItemEntity.java.
  • Removed the misc.sendFullPosForItemEntities configuration option from GlobalConfiguration.
  • Registered misc.send-full-pos-for-item-entities in RemovedConfigurations.java to ensure that servers upgraded from older versions will automatically clean up this entry when reading paper-global.yml, preventing unknown configuration warnings.

The build passed when running ./gradlew compileJava locally.

Closes #14260

@zzzzzyc
zzzzzyc requested a review from a team as a code owner September 17, 2026 19:29
@github-project-automation github-project-automation Bot moved this to Awaiting review in Paper PR Queue Sep 17, 2026
@Doc94

Doc94 commented Sep 18, 2026

Copy link
Copy Markdown
Member

HI,
can be good also if you make a PR in docs for remove that config mention.

zzzzzyc added a commit to zzzzzyc/docs that referenced this pull request Sep 18, 2026
Removed the configuration option for sending full position for item entities.
as PaperMC/Paper/pull/14273
@zzzzzyc

zzzzzyc commented Sep 18, 2026

Copy link
Copy Markdown
Contributor Author

Docs cleanup: PaperMC/docs#797

@papermc-projects papermc-projects Bot moved this from Awaiting review to Full approval in Paper PR Queue Sep 18, 2026
@Doc94
Doc94 merged commit 3caa2d2 into PaperMC:dev/26.3 Sep 18, 2026
3 checks passed
@github-project-automation github-project-automation Bot moved this from Full approval to Merged in Paper PR Queue Sep 18, 2026
@Doc94

Doc94 commented Sep 18, 2026

Copy link
Copy Markdown
Member

Thanks for the contribution :)

@Doc94 Doc94 added the dev/next Changes related to the snapshot / next branch label Sep 18, 2026
Doc94 pushed a commit to PaperMC/docs that referenced this pull request Sep 18, 2026
Removed the configuration option for sending full position for item entities.
as PaperMC/Paper/pull/14273
granny added a commit to PurpurMC/Purpur that referenced this pull request Sep 21, 2026
Upstream has released updates that appear to apply and compile correctly

Paper Changes:
PaperMC/Paper@1e37b938 [ci/skip] Deprecate PotionBrewer method in Bukkit/Server class
PaperMC/Paper@3caa2d22 Remove obsolete MC-4 workaround PaperMC/Paper#14273
PaperMC/Paper@0fd1cf72 update material tags, fix some methods on item type
PaperMC/Paper@b4c7d168 deprecate PotionEffectTypeCategory and PlayerAnimationEvent
PaperMC/Paper@4e0ccd0f Updatea DataConverter
PaperMC/Paper@48e89d7f Anti xray
PaperMC/Paper@1d60280c Per player mob spawning and more optimization patches
PaperMC/Paper@8d936046 Update remaining feature patches
PaperMC/Paper@9838b408 Add SulfurCube#swallow PaperMC/Paper#14062
PaperMC/Paper@98e06785 tweaks deprecated warnings behavior on event
PaperMC/Paper@a2551859 Call EntityCollideWithEntityEvent for Sulfur Cube pushed by Player PaperMC/Paper#14157
PaperMC/Paper@e6e9780e Initialize lagCompensationTick correctly PaperMC/Paper#14285
PaperMC/Paper@93c9b2ab Fix inlined block state conversion
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dev/next Changes related to the snapshot / next branch

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

3 participants