poqdavid / VirtualTool

A plugin for minecraft to provide virtual gui for all possible items
https://poqdavid.github.io/VirtualTool/
MIT License
5 stars 5 forks source link

SpongeUpdate broke Backpack Files #13

Closed SeanStoves closed 6 years ago

SeanStoves commented 6 years ago

We have recently updated from Sponge2477 2731 to Sponge2477 2793. And at this moment, our Backpack files are being emptied whenever you access the backpack.

There is no error in the logs. It just, empties the file and leaves us with {} in it.

poqdavid commented 6 years ago

that's odd but if i recall once there was a problem with one of the sponge apis well try 1.10.2-2477-5.2.0-BETA-2790 and tell me if same thing happens.

SeanStoves commented 6 years ago

We have rolled back to 2790 Sponge. It now empties when you modify the contents of the inventory. However, it does not empty it if you just view it like it does on 2793.

poqdavid commented 6 years ago

yah its a problem with api so yap

SeanStoves commented 6 years ago

Is this something you are able to fix, or should we look into doing a forked fix for ourselves?

Daniel-I-Am commented 6 years ago

Same thing happened here after an update to the latest version of sponge. Downgrading is not possible due to other incompatibilities. As far as we could downgrade, we could not find a working version.

poqdavid commented 6 years ago

Well give me error logs?

Daniel-I-Am commented 6 years ago

This is what it gives me in the server's FML-latest.log first /bp is open and the second /bp is reopen after putting something in it (at which point it became empty):

[22:14:24] [Server thread/INFO] [nucleus/]: Daniel_I_Am ran the command: /bp 
[22:14:26] [Server thread/INFO] [nucleus/]: Daniel_I_Am ran the command: /bp 

There is no error there; console reads nothing different. When opening up the file it reads nothing more than: {} That is all the clues I can give you. If there is some kind of debug mode that can somehow be enabled, I am willing to try it out. As far as I can see, there is no such thing in the configs.

poqdavid commented 6 years ago

no well its confusing as its not showing any error

Daniel-I-Am commented 6 years ago

My guessing leads me to believe it was some kind of bug being exploited to make them work in the first place (since the inventory API shouldn't work like this) that now got fixed... I'd have to jump into the code to really know. Only thing I can say is that it was working on sponge 2731 and patch between that version and 2793 broke it..

SeanStoves commented 6 years ago

We are seeing the same complete lack of logs, Sadly.

Also, is there any plans of pushing out a version for API7 MC 1.12.2 ?

VirtualTools is much better then the SpongyBackups alternative.. So we would like to continue using VT.

poqdavid commented 6 years ago

@SolaoBajiuik Here for 1.12.2 https://github.com/poqdavid/VirtualTool/releases/tag/v4.0 and let me know if anything is wrong and please give me logs if there is any.

poqdavid commented 6 years ago

@SolaoBajiuik so i checked everything and here https://github.com/poqdavid/VirtualTool/releases/tag/v4.5 that must fix it and i tested that my self.

SeanStoves commented 6 years ago

@poqdavid will this work on API5/6 still for 1.10 servers?

poqdavid commented 6 years ago

no this is api 7.1.0

Brend-Smits commented 6 years ago

Could you perhaps also release a version for API 5 that fixes this issue? We are also having issues with peoples backpack voiding on 1.10 =/

poqdavid commented 6 years ago

@Rubbertjuh the thing is i don't know when i get the time to do that but i would recommend you guys to update to 1.12.2

Brend-Smits commented 6 years ago

@poqdavid that's a shame =/ Unfortunately updating these modpacks to 1.12 is not possible as we don't own the modpacks. We use the 1.12 version for our 1.12 servers and they work fine. Hopefully you're able to find some time <3 That would be amazing!