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

Bug: Skyroot Beds (and /weather command) not clearing weather #324

Open
4 tasks done
VaraHunter682 opened this issue Jun 16, 2024 · 3 comments · May be fixed by #346
Open
4 tasks done

Bug: Skyroot Beds (and /weather command) not clearing weather #324

VaraHunter682 opened this issue Jun 16, 2024 · 3 comments · May be fixed by #346
Labels
type/bug The issue describes a bug or error in the project.

Comments

@VaraHunter682
Copy link

What Feature Types Apply to This Bug?

World-Gen

Other Type

No response

What Type of Bug Is This?

Unexpected Behavior

Forge Version

20.4.234

The Aether II Version

1.20.4-001

Is This Bug a Conflict With Another Mod?

No response

Client Log

No response

Crash Report (if applicable)

No response

Steps to Reproduce

wait for it to rain/snow, then type "/weather clear" in chat or sleep in a Skyroot Bed

What You Expect To Happen

Based on Vanilla behavior, this should've cleared the weather.

What Actually Happened

Both do nothing, it still rains.

Additional Details

No response

Please Read and Confirm The Following

  • I have confirmed this bug can be replicated without the use of Optifine.
  • I have confirmed this bug is on the most recently supported version of Minecraft.
  • I have confirmed the details provided in this report are concise as possible and does not contained vague information (ie. Versions are properly recorded, answers to questions are clear).
  • I have confirmed this issue is unique and has not been reported already.
@VaraHunter682 VaraHunter682 added the type/bug The issue describes a bug or error in the project. label Jun 16, 2024
@Builderdog841
Copy link
Contributor

Builderdog841 commented Jun 16, 2024

You need to write /execute in minecraft:overworld run weather clear. This seems to be a hardcoded Vanilla flaw (I think) so we might need to add a new command

@VaraHunter682
Copy link
Author

ah ok

@bconlon1
Copy link
Contributor

bconlon1 commented Jul 4, 2024

code to add

    public static Long finishSleep(LevelAccessor level, long newTime) {
        if (level instanceof ServerLevel && level.dimensionType().effectsLocation().equals(AetherDimensions.AETHER_DIMENSION_TYPE.location())) {
            ServerLevelAccessor serverLevelAccessor = (ServerLevelAccessor) level;
            serverLevelAccessor.aether$getServerLevelData().setRainTime(0);
            serverLevelAccessor.aether$getServerLevelData().setRaining(false);
            serverLevelAccessor.aether$getServerLevelData().setThunderTime(0);
            serverLevelAccessor.aether$getServerLevelData().setThundering(false);

            long time = newTime + 48000L;
            return time - time % (long) AetherDimensions.AETHER_TICKS_PER_DAY;
        }
        return null;
    }```

@bconlon1 bconlon1 linked a pull request Jul 6, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/bug The issue describes a bug or error in the project.
Projects
Status: Review Requested
Development

Successfully merging a pull request may close this issue.

3 participants