Skip to content

Make GTUtil.canSeeSunClearly work for all dimensions even if they have fixed time enabled - #5239

Open
screret wants to merge 2 commits into
GregTechCEu:1.20.1from
screret:sc/make-day-check-more-generic
Open

Make GTUtil.canSeeSunClearly work for all dimensions even if they have fixed time enabled#5239
screret wants to merge 2 commits into
GregTechCEu:1.20.1from
screret:sc/make-day-check-more-generic

Conversation

@screret

@screret screret commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

What

Remove the special case for JAVD in GTUtil.canSeeSunClearly() in favor of a fully generic solution

Implementation Details

copied Level#isDay's logic and removed the check that made fixed time be neither day or night.
Also removed the duplicate Level.getBiome(pos.above()) call by storing the biome's holder in a variable instead of the biome itself

AI Usage

  • No AI driven tools were used for this pull request.

Outcome

void dimension mods other than JAVD will also work now

How Was This Tested

ran game, went into Just Another Void Dimension, placed solar boiler, filled with water, waited.

@screret
screret requested a review from a team as a code owner August 4, 2026 12:44
@github-actions github-actions Bot added the 1.20.1 label Aug 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants