From e5fc65264bf0d2174bf29f3512d105297a6c739f Mon Sep 17 00:00:00 2001 From: 12xx12 <44411062+12xx12@users.noreply.github.com> Date: Sat, 29 May 2021 18:28:57 +0200 Subject: Added standardised way to Log with plugin name (#5227) * added logging functions to each plugin * added documentation * modified the global LOG macro * updated the way of string composition * removed cloumn * removed capital v --- Server/Plugins/APIDump/Writing-a-Cuberite-plugin.html | 6 +++--- src/Bindings/ManualBindings.cpp | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/Server/Plugins/APIDump/Writing-a-Cuberite-plugin.html b/Server/Plugins/APIDump/Writing-a-Cuberite-plugin.html index 6d53ccaab..8c8008530 100644 --- a/Server/Plugins/APIDump/Writing-a-Cuberite-plugin.html +++ b/Server/Plugins/APIDump/Writing-a-Cuberite-plugin.html @@ -42,12 +42,12 @@ function Initialize(Plugin) -- Command Bindings - LOG("Initialised " .. Plugin:GetName() .. " v." .. Plugin:GetVersion()) + LOG("Initialised version " .. Plugin:GetVersion()) return true end function OnDisable() - LOG(PLUGIN:GetName() .. " is shutting down...") + LOG("Shutting down...") end

@@ -56,7 +56,7 @@ end

  • function Initialize is called on plugin startup. It is the place where the plugin is set up.
  • Plugin:SetName sets the name of the plugin.
  • Plugin:SetVersion sets the revision number of the plugin. This must be an integer.
  • -
  • LOG logs to console a message, in this case, it prints that the plugin was initialised.
  • +
  • LOG logs to console a message, in this case, it prints that the plugin was initialised. This will add a prefix with the name of your plugin.
  • The PLUGIN variable just stores this plugin's object, so GetName() can be called in OnDisable (as no Plugin parameter is passed there, contrary to Initialize). This global variable is only needed if you want to know the plugin details (name, etc.) when shutting down.
  • function OnDisable is called when the plugin is disabled, commonly when the server is shutting down. Perform cleanup and logging here.
  • diff --git a/src/Bindings/ManualBindings.cpp b/src/Bindings/ManualBindings.cpp index fe5a1e6cb..1d0519708 100644 --- a/src/Bindings/ManualBindings.cpp +++ b/src/Bindings/ManualBindings.cpp @@ -362,7 +362,7 @@ static void LogFromLuaStack(lua_State * tolua_S, eLogLevel a_LogLevel) size_t len = 0; const char * str = lua_tolstring(tolua_S, 1, &len); - Logger::LogSimple(std::string_view(str, len), a_LogLevel); + Logger::LogSimple(fmt::format("[{}] {}", cManualBindings::GetLuaPlugin(tolua_S)->GetName(), std::string_view(str, len)), a_LogLevel); } -- cgit v1.2.3