Refactored logging methods to use more descriptive names for improved clarity

Author: TavstalDev

TLibrary/Helpers/Unturned/UChatHelper.cs

@@ -61,7 +61,7 @@ public static void SendCommandReply(this IPlugin plugin, object toPlayer, string
                     player.SteamPlayer());
             }
             else
-                plugin.GetLogger().LogRichCommand(plugin.Localize(false, translation, args));
+                plugin.GetLogger().RichCommand(plugin.Localize(false, translation, args));
         }
 
         /// <summary>
@@ -79,7 +79,7 @@ public static void SendPlainCommandReply(this IPlugin plugin, object toPlayer, s
             else if (toPlayer is UnturnedPlayer player)
                 ServerSendChatMessage(FormatHelper.FormatTextV2(string.Format(translation, args)), icon, null, player.SteamPlayer());
             else
-                plugin.GetLogger().LogRichCommand(string.Format(translation, args));
+                plugin.GetLogger().RichCommand(string.Format(translation, args));
         }
 
         /// <summary>

TLibrary/Managers/DatabaseManagerBase.cs

@@ -45,7 +45,7 @@ public MySqlConnection CreateConnection()
             }
             catch (Exception ex)
             {
-                _plugin.GetLogger().LogException(ex.ToString());
+                _plugin.GetLogger().Exception(ex.ToString());
             }
             return mySqlConnection;
         }

TLibrary/Managers/HookManager.cs

@@ -29,20 +29,20 @@ public void Load(Type type)
         {
             if (!typeof(Hook).IsAssignableFrom(type))
             {
-                _logger.LogException("'{0}' is not a hook.");
+                _logger.Exception("'{0}' is not a hook.");
                 return;
             }
 
             if (type.IsAbstract)
             {
-                _logger.LogException($"Cannot register {type.Name} because it is abstract.");
+                _logger.Exception($"Cannot register {type.Name} because it is abstract.");
                 return;
             }
 
             var hook = CreateInstance<Hook>(type);
             if (_hooks.ContainsKey(hook.Name))
             {
-                _logger.LogException("Hook with '{0}' name already exists.");
+                _logger.Exception("Hook with '{0}' name already exists.");
                 return;
             }
 
@@ -64,7 +64,7 @@ public void LoadAll(Assembly pluginAssembly, bool ignoreDuplicate = false)
                 {
                     if (_hooks.ContainsKey(hook.Name))
                     {
-                        _logger.LogException($"Hook with '{hook.Name}' name already exists.");
+                        _logger.Exception($"Hook with '{hook.Name}' name already exists.");
                         continue;
                     }
 
@@ -80,8 +80,8 @@ public void LoadAll(Assembly pluginAssembly, bool ignoreDuplicate = false)
                 }
                 catch (Exception ex)
                 {
-                    _logger.LogException($"Failed to add the '{hook.Name}' hook to the library.");
-                    _logger.LogError(ex);
+                    _logger.Exception($"Failed to add the '{hook.Name}' hook to the library.");
+                    _logger.Error(ex);
                 }
             }
         }
@@ -94,20 +94,20 @@ public void Unload(Type type)
         {
             if (type != typeof(Hook))
             {
-                _logger.LogException($"'{type.Name}' is not a hook.");
+                _logger.Exception($"'{type.Name}' is not a hook.");
                 return;
             }
 
             if (!_hooks.Any(x => x.Value.GetType() == type))
             {
-                _logger.LogException($"'{type.Name}' is not loaded.");
+                _logger.Exception($"'{type.Name}' is not loaded.");
                 return;
             }
 
             var hook = _hooks.Values.FirstOrDefault(x => x.GetType() == type);
             if (hook == null)
             {
-                _logger.LogException($"'{type.Name}' is not found.");
+                _logger.Exception($"'{type.Name}' is not found.");
                 return;
             }
 
@@ -186,8 +186,8 @@ private T CreateInstance<T>(Type type)
             }
             catch (Exception ex)
             {
-                _logger.LogException($"Failed to create instance for '{type.Name}' hook.");
-                _logger.LogError(ex);
+                _logger.Exception($"Failed to create instance for '{type.Name}' hook.");
+                _logger.Error(ex);
                 return default;
             }
         }

TLibrary/Models/Hooks/Hook.cs

@@ -50,8 +50,8 @@ internal void Load()
             }
             catch (Exception ex)
             {
-                Plugin.GetLogger().LogError($"Failed to load '{Name}' hook.");
-                Plugin.GetLogger().LogException(ex.ToString());
+                Plugin.GetLogger().Error($"Failed to load '{Name}' hook.");
+                Plugin.GetLogger().Exception(ex.ToString());
             }
         }
 
@@ -69,8 +69,8 @@ internal void Unload()
             catch (Exception ex)
             {
                 IsLoaded = false;
-                Plugin.GetLogger().LogError($"Failed to unload '{Name}' hook.");
-                Plugin.GetLogger().LogException(ex.ToString());
+                Plugin.GetLogger().Error($"Failed to unload '{Name}' hook.");
+                Plugin.GetLogger().Exception(ex.ToString());
             }
         }