Data Store Improvements

[x3rt]

Breaking Changes

• CustomDataStore<T>.Get no longer automatically creates an instances if it does not exist, and instead returns null. The previous naming was misleading

Changes

• Stop using reflection for Adding/Removing Players to Data Stores and Destroying instances
• Stop registering un-registered Data Stores in GetOrAdd as it can lead to undesired behavior
• Added CustomDataStore.GetAll<T>() to get all instances of a specific store. Idea from @SlejmUr
• Added CustomDataStore.Exists<T>(Player) to check whether a player has the specified DataStore. Implemented by @xCynDev
• Added CustomDataStore.Get<T>(Player) which only returns an existing instance or null if one does not exist.
• Added CustomDataStore.TryGet<T>(Player, out T?)
• Made CustomDataStore.Destroy<T>(Player) public as it is now possible to use datastores without ever registering them, meaning developers can manage the lifetimes themselves

[x3rt]

Going to add a few things later today/tomorrow.

[LumiFae]

@Jesus-QC @x3rt any chance we can get an update on this one? would be really useful to have in the next update, or later.

[Jesus-QC]

@Jesus-QC @x3rt any chance we can get an update on this one? would be really useful to have in the next update, or later.

Yes of course, as this contains breaking changes it is being paused until our next breaking update