Added advanced flying data saving, bug fixes and optimization fixes#4
Merged
Conversation
- Implement `DataManager` to load and save flying, boosted states, and chestplate data. - Integrate data persistence with `SpawnBoostListener` to preserve player states across server restarts. - Optimize player tracking by switching from lists to sets and from saving the Player objects to UUIDs. Signed-off-by: SchlangeGoto <kkazimir@protonmail.com>
…t, file saving bug fixes - Fixed bug were flying_players.json file would not be created - Refactor file handling and ensure directory creation during data save operations. - Add detailed logging for loading and saving player flight and boost data. - Prevent duplicate save attempts by introducing a save scheduling mechanism. Signed-off-by: SchlangeGoto <kkazimir@protonmail.com>
… multiple listeners got created Signed-off-by: SchlangeGoto <kkazimir@protonmail.com>
Owner
|
Did you test all functionality, including old Bugs like:
If not, I kindly ask you to do that, so we are not uploaded untested and potentially buggy code! |
Contributor
Author
|
I tested everything were I changed functionality
I did not Test anything else because I did not change any functionality or code there |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Added DataManager class for managing saving/loading and en/decoding
Added methodes in SpawnBoostListener for managing saving and loading of flying data
Optimized all Lists by changing them to Sets and not saving the player object and only the uuids
Added validation to the onEnable methode so it doesnt happend that two listeners get created
Added logging of all kinds of information for better debuging and plugin usage