From 7a2ff1a9784f032815007790cabf07d86098ff1b Mon Sep 17 00:00:00 2001 From: Jonathan Jenne Date: Tue, 12 Nov 2019 10:57:55 +0100 Subject: [PATCH] remove ClassNLog in favor of logging module, remove calls to logger.fatal --- DDUserManager/DDUserManager/ClassNLog.vb | 27 ------------------- .../DDUserManager/DDUserManager.vbproj | 2 -- DDUserManager/DDUserManager/frmMain.vb | 6 ++--- 3 files changed, 3 insertions(+), 32 deletions(-) delete mode 100644 DDUserManager/DDUserManager/ClassNLog.vb diff --git a/DDUserManager/DDUserManager/ClassNLog.vb b/DDUserManager/DDUserManager/ClassNLog.vb deleted file mode 100644 index 6e7d425..0000000 --- a/DDUserManager/DDUserManager/ClassNLog.vb +++ /dev/null @@ -1,27 +0,0 @@ -Public Class ClassNLog - Public Shared Function GetLogPathFor(moduleName As String) - Dim localAppData = Environment.GetFolderPath(Environment.SpecialFolder.LocalApplicationData) - Return IO.Path.Combine(localAppData, "Digital Data", "UserManager", "Log") - End Function - - Public Shared Function GetLoggerConfigFor(moduleName As String) - Dim loggerConfig = New NLog.Config.LoggingConfiguration() - Dim logFileName = Environment.UserName & "-${date:format=yyyy-MM-dd}.log" - Dim logPath As String = IO.Path.Combine(GetLogPathFor(moduleName), logFileName) - - Dim logFileInfo As New NLog.Targets.FileTarget("logFileInfo") With { - .FileName = logPath, - .Layout = "${longdate}|${level:uppercase=true}|${logger}|${message}" - } - - Dim logFileError As New NLog.Targets.FileTarget("logFileError") With { - .FileName = logPath, - .Layout = "${longdate}|${level:uppercase=true}|${logger}|${message}|${exception:format=message}" & vbCrLf & "${exception:format=toString}" - } - - loggerConfig.AddRule(NLog.LogLevel.Debug, NLog.LogLevel.Warn, logFileInfo) - loggerConfig.AddRule(NLog.LogLevel.Error, NLog.LogLevel.Fatal, logFileError) - - Return loggerConfig - End Function -End Class diff --git a/DDUserManager/DDUserManager/DDUserManager.vbproj b/DDUserManager/DDUserManager/DDUserManager.vbproj index 2db510a..f9954f7 100644 --- a/DDUserManager/DDUserManager/DDUserManager.vbproj +++ b/DDUserManager/DDUserManager/DDUserManager.vbproj @@ -55,7 +55,6 @@ - @@ -111,7 +110,6 @@ - True True diff --git a/DDUserManager/DDUserManager/frmMain.vb b/DDUserManager/DDUserManager/frmMain.vb index cbc083f..0157713 100644 --- a/DDUserManager/DDUserManager/frmMain.vb +++ b/DDUserManager/DDUserManager/frmMain.vb @@ -125,7 +125,7 @@ Public Class frmMain End If Catch ex As Exception MsgBox("Unexpected Error while loading. Please check the log.", MsgBoxStyle.Critical, "User Manager") - MyLogger.Fatal(ex, "Unexpected Error while loading.") + MyLogger.Error(ex, "Unexpected Error while loading.") End Try End Sub @@ -1072,7 +1072,7 @@ Public Class frmMain End Sub Private Sub btnOpenLogDir_Click(sender As Object, e As EventArgs) Handles btnOpenLogDir.Click - Process.Start(ClassNLog.GetLogPathFor(MODULE_NAME)) + Process.Start(MyLogConfig.LogDirectory) End Sub Private Sub txtADRootNode_Leave(sender As Object, e As EventArgs) Handles txtADRootNode.Leave, txtLDAPGroupQuery.Leave, txtLDAPUserQuery.Leave @@ -1135,7 +1135,7 @@ Public Class frmMain End Sub Private Sub Button3_Click(sender As Object, e As EventArgs) Handles Button3.Click, Button4.Click - MyConfig.Config.AdGroupFilter = txtLDAPGroupQuery.Text + MyConfig.Config.AdUserFilter = txtLDAPUserQuery.Text MyConfig.Save() End Sub