Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.

Review: AmirAli SeyedZadegan #3

Open
alirezaopmc opened this issue Jun 30, 2022 · 2 comments
Open

Review: AmirAli SeyedZadegan #3

alirezaopmc opened this issue Jun 30, 2022 · 2 comments

Comments

@alirezaopmc
Copy link
Contributor

Dear @L000Pz

Thanks for submitting your awesome plugin here and providing a delightful README. 🌷

Check this one out and I will share any issues I find. We can discuss on them to fix them. πŸ™‹πŸ»β€β™‚οΈ

Note that it would be great if you ensure me that this repository will be up to date. (As I checked, this repository contains your plugin's source code. If I am wrong please provide a view access to me.)

@alirezaopmc
Copy link
Contributor Author

alirezaopmc commented Jun 30, 2022

Hi again @L000Pz

I was testing your plugin that I just encountered an issue.

Challenge counter will be very fast after repeating the /challenge
At the first time I used /challenge, the counter was fine. It was counting normally. Exactly 1 per second. But for the second and third time counter was counting faster. I guess it grows linearly because of storing the history of challenges in a data-structure without destroying the unsuccessful challenges.

To Reproduce
Steps to reproduce the behavior:

  1. Start the server with the plain plugin with no data saved.
  2. Connect two players. Let call them p1 and p2.
  3. p1 command: /challenge p2
  4. Wait until the counter ends.
  5. Do the 3rd step again.

Expected behavior
I expected normal behavior to remain the same.

Screenshots
g1

@L000Pz
Copy link
Contributor

L000Pz commented Jun 30, 2022

HELLOO and Thank you so much for submitting your problem.πŸ™Œ
I am so sorry you faced this bug. I kinda was aware of the problem but I don't know why I forgot to fix it.
The problem was because I used an old code (Minecraft 1.8 old πŸ€¦β€β™‚οΈ), it's now fixed and I'll update it as soon as possible.
Thank you again for sharing this problem and if you happen to face more bugs I'd be happy to fix them for you.πŸ˜‰
P.S: happy you liked the README file :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants