Skip to content

fix xray only shows exposed ores#6287

Merged
MineGame159 merged 4 commits intoMeteorDevelopment:masterfrom
mofetaxd:master
Apr 5, 2026
Merged

fix xray only shows exposed ores#6287
MineGame159 merged 4 commits intoMeteorDevelopment:masterfrom
mofetaxd:master

Conversation

@mofetaxd
Copy link
Copy Markdown
Contributor

@mofetaxd mofetaxd commented Apr 5, 2026

Type of change

  • Bug fix
  • New feature

Description

SodiumBlockOcclusionCacheMixin was applying a sodium class called BlockOcclusionCache which was removed and replaced with AbstractBlockRenderContext causing the mixin to not inject correctly

Related issues

#6280

How Has This Been Tested?

2026-04-05_16 35 28 2026-04-05_16 35 43 2026-04-05_16 35 48

Checklist:

  • My code follows the style guidelines of this project.
  • I have added comments to my code in more complex areas.
  • I have tested the code in both development and production environments.

@Big-Iron-Cheems
Copy link
Copy Markdown
Collaborator

Big-Iron-Cheems commented Apr 5, 2026

For this PR to be valid it should:

  1. Update the Sodium version we depend on: https://github.com/MeteorDevelopment/meteor-client/blob/master/gradle/libs.versions.toml#L16 (The target class still exists up until 0.8.6)
  2. Update the breaking version os Sodium here: https://github.com/MeteorDevelopment/meteor-client/blob/master/src/main/resources/fabric.mod.json#L60 (Require at least 0.8.7 as older versions would now be invalid targets)
  3. Properly import the shadowed fields like:
@Shadow 
protected BlockState state;

This would also mean updating any other Sodium mixins that may have broken in the 0.8.0 to 0.8.7 range.

@MineGame159 MineGame159 merged commit 1c7927f into MeteorDevelopment:master Apr 5, 2026
1 check passed
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.

4 participants