plan-player-analytics / Plan

Player Analytics plugin for Minecraft Server platforms - View player activity of your server with ease. :calendar:
https://www.spigotmc.org/resources/plan-player-analytics.32536/
GNU Lesser General Public License v3.0
826 stars 167 forks source link

Rewrite Import system #1882

Open AuroraLS3 opened 3 years ago

AuroraLS3 commented 3 years ago

Describe the issue

Import system is broken.

Additional information

The import functionality dates back to 3.x.x and a lot has changed since then. The code has been changed during those changes to match changes in other parts of the code, but it has not been kept up to date.

Main issues with the importing code is that it attempts to do too many things at once, leading to massive data type classes that don't translate well into the internal data structures used in the plugin.

Optimal solution would be to use all of the gained knowledge in building data classes and functions to create something modular that can be expanded more easily.

AuroraLS3 commented 2 years ago

TODO

This is a bit vague description since it needs understanding the current import system. It should be easier to add more importers after this is completed.