EnderContainers

EnderContainers

2.5k Downloads

Duplication Issue

nivcoo opened this issue ยท 17 comments

commented

Hello, i've duplication issue with the latest version in 1.12.2

Place item in your /ec 2 for example
Quit the server and join it and run / ec 2, before the GUI opens, execute the same command, after that you can get the item without editing the enderchest, because the previous gui will be replaced by the new openned gui

commented

I made a fix for my server, in fact we can open an enderchest twice if the load is long so the first time we can modify the enderchest and when the second opens on top then the changes are not saved. To solve this I added a list of users which increments when a player executes / ec or clicks on an auction and is deleted when the loading is finished.
It's not 100% optimized but it works https://pics.revolis.fr/2021-07/31/15-11_P1Mj.png

commented

ugh.. I do not have thinking about that possible issue..! Thank you for the details it will help me a lot to fix that ๐Ÿฅ‡

commented

Yes because at the first /ec the playercontext is loaded and during its loading we can redo a load and therefore have this problem

commented

But it only works when the data to load is big and only with the YML and not the mysql

commented

Hello! I will try to reproduce the issue. As I want to release a new version with Minecraft 1.17 support, I will try to fix that duplication issue before. I'll keep you informed ๐Ÿ‘

commented

In next major version of the plugin (v2.3.x), the chat prefix has been removed ๐Ÿ‘
Check that issue in which we have discussed about it: #127

commented

Also, can you add an otpion to configure the prefix of command ? Because i have to use my fork to edit the prefix

commented

That's a big question.. I am not as active as I would like to be on the development of this plugin... so unfortunately I cannot say a date for now. I will try to release in next days an update with all duplication bugfixes and Minecraft 1.17 support.

commented

Okay thx,when the update will be released ?

commented

I saw your changes and you do the same thing of me, so now i will wait the update :p

commented

That duplication problem has already been reported on holograms (in the issue #125). I have done the fix on them but not on the enderchest manager class. Both systems should be ok now. I have done some tests with a very long loading task and it works well.

commented

Have you reproduce the issue before the fix to compare the situation ?

commented

Yes I have reproduced the exact issue you described in your first post ๐Ÿ‘ And with the bugfix that works fine!

commented

Okay thx :p is it fixed also when we click on enderchest ?

commented

Yes! Loading state of a player context is now handled at the code base. So it benefits holograms, opening of enderchests with the command or by clicking on it. I intentionally increased duration of loading task to test this and it's ok

commented

Okay thx

commented

You must place big item (With multiple datas)