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

cenchant: Enchants Started and Cancelled still send "Ench Seed Ready" message when enchanting. #72

Closed
leumasme opened this issue Aug 30, 2019 · 5 comments

Comments

@leumasme
Copy link

This can cause multiple messages to show at once after enchanting if you started multiple cenchants. Messages are not cleared by breaking rng status either

@Earthcomputer
Copy link
Owner

What do you mean by a "cancelled" enchant?

@Earthcomputer
Copy link
Owner

Is this still an issue with the new release, 2.3.5?

@leumasme
Copy link
Author

leumasme commented Sep 1, 2019

This issue Directly was fixed, but it came with a new one.
Start a cenchant, let it Finish until it requests a Dummy, then Sprint.
The Enchant Seed message will show, as if you had done the dummy enchantment.
It will also show multiple times if you ran cenchant multiple times without doing anyhing in between.
After that, cenchant will return the <Impossible/Over maxCalculations> message, instead of the message. (As you just broke it by Sprinting.)

@Earthcomputer
Copy link
Owner

I cannot reproduce the enchant seed showing when you invalidate the seed by sprinting. However, I can reproduce two messages showing when you run the command twice. Can you give more detailed reproduction steps for the first issue?

@Earthcomputer
Copy link
Owner

Never mind, I have just managed to reproduce the first bug, that requires enchCrackState to be cracked.

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

No branches or pull requests

2 participants