Skip to content

fix(AutoGap): respect allowEgap setting when choosing enchanted gaps#5637

Merged
Wide-Cat merged 2 commits into
MeteorDevelopment:masterfrom
uxmlen:fix/auto-gap-ignore-allow-egap
Oct 23, 2025
Merged

fix(AutoGap): respect allowEgap setting when choosing enchanted gaps#5637
Wide-Cat merged 2 commits into
MeteorDevelopment:masterfrom
uxmlen:fix/auto-gap-ignore-allow-egap

Conversation

@uxmlen

@uxmlen uxmlen commented Aug 9, 2025

Copy link
Copy Markdown
Contributor

Type of change

  • Bug fix
  • New feature

Description

findSlot() selected enchanted golden apples even when allowEgap was false

preferEGap = this.allowEgap.get() || requiresEGap, so requiresEGap forced EGAP preference and ignored the user setting.

bug demo:

2025-08-09.15-16-45.mp4

Related issues

#5165

How Has This Been Tested?

2025-08-09.15-02-26.mp4

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.

uxmlen and others added 2 commits August 9, 2025 14:47
`findSlot()` selected enchanted golden apples even when allowEgap was false caused by `preferEGap = this.allowEgap.get() || requiresEGap`, so `requiresEGap` forced EGAP preference and ignored the user setting.

MeteorDevelopment#5165
@Wide-Cat Wide-Cat merged commit 9ce8727 into MeteorDevelopment:master Oct 23, 2025
1 check passed
@uxmlen uxmlen deleted the fix/auto-gap-ignore-allow-egap branch November 2, 2025 17:00
MistressOfDNS pushed a commit to MistressOfDNS/meteor-client-fork that referenced this pull request May 7, 2026
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.

2 participants