mod-eluna icon indicating copy to clipboard operation
mod-eluna copied to clipboard

Feat(LuaEngine): Move config reader to ElunaConfig

Open iThorgrim opened this issue 11 months ago • 1 comments

[!WARNING] This PR is the beginning of the porting process for features added to Eluna. Since Multistate is the main objective of my contributions, I am adding each feature that revolves around it to simplify future integration and development.

[!NOTE] Implementation based on this commit Eluna 9806ef7.

📝 Description

This pull request introduces a new configuration system for the Eluna Lua Engine, encapsulating various settings and options into a dedicated ElunaConfig class. The configuration is now managed centrally and can be accessed more easily across the Eluna module.

Key Changes:

  • Introduced the ElunaConfig class to manage various configuration values.
  • Replaced direct use of eConfigMgr with ElunaConfig for better configuration handling.
  • Refactored LuaEngine initialization and reloading to utilize the new configuration system.

iThorgrim avatar Jan 29 '25 15:01 iThorgrim

What's the replacement for the IsInitialized() check, which was called by IsEnabled()? Why is it no longer required in all the hooks of the object files?

55Honey avatar Feb 20 '25 22:02 55Honey