Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Избавиться от загрузки мира при необходимости получить UUID владельца #35

Closed
Dymeth opened this issue Mar 4, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@Dymeth
Copy link
Collaborator

Dymeth commented Mar 4, 2024

Сейчас для проверки владения уровнем прогружается мир полностью, хотя стоило бы прогружать только его мета-данные один раз при запуске сервера (+ при создании мира)

@Dymeth Dymeth added the enhancement New feature or request label Mar 4, 2024
@DrupalDoesNotExists
Copy link
Contributor

Как я упоминал в своём ответе на #36, кажется, что для таких задач уже целесообразно использовать какую-то простейшую базу данных. Иначе мы просто будем "писать свою" внутри плагина.

@Dymeth
Copy link
Collaborator Author

Dymeth commented Mar 19, 2024

Как я упоминал в своём ответе на #36, кажется, что для таких задач уже целесообразно использовать какую-то простейшую базу данных. Иначе мы просто будем "писать свою" внутри плагина.

В данной задаче идёт речь про организацию бизнес-логики, а не про способ хранения и загрузки данных.
Именно эта задача и была решена в рамках данного issue. Теперь данные уровней на загружаются и не выгружаются без реальной необходимости.
При этом легковестные мета-данные всех уровней теперь загружаются при запуске плагина, поскольку они всегда нужны "на горячую" в рантайме - например, для таб-комплита команды /play

Что касается самого хранения данных, то по нему при желании всегда можно создать отдельный issue. Задача будет заключаться в том, чтобы написать новую имплементацию интерфейса LevelSettingDAO.

Но я считаю, что на текущий момент острой необходимости в этом нет. Тем более, что миры по-прежнему нет возможности хранить в базах данных (см. issue со SlimeWorldManager).

Поскольку текущая задача выполнена в полном объёме - закрываю

@Dymeth Dymeth closed this as completed Mar 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants