Fixes #2212 In #2112, we found out that in configurations with disabled cache (like Bungee), this adapter will perform blocking queries on the main thread which impacts the performance. The original code (81cf14f) is in place to preserve the inventory for unregistered and configurations without enforcing a registration (#1830, #1752). Alternatives are: 1. Send the inventory on registration like p.updateInventory() * Still hides it before for unregistered 2. Checking for the enforce registration setting would mean it hides also the inventory for unregistered players 3. Get a notification on player status changes * Requires a complex setup to propagate the changes across spigot servers or different solution for web interfaces * Refresh events, when the status is updated within the plugin itself would be possible, however requires previous queries like registration/login requests. Instant reports about registration will still be complicated. Example: Every time we make queries, save the result locally and fire an event for our components to check for potential changes without making the same query again. Nevertheless this again delays changes if there is no need to that.

Commits

Commits on Jun 18, 2021

Check inventory authentication status only on cache

Commits on Jun 19, 2021

Only leave inventory untouched if registration is not enforced

Allow unregistered players in not enforced config to use tab

Allow unregistered players in not enforced config to use tab

Perform all restriction checks only on cache

Code clean up

Protect unknown data too