review comment

Hirogen · Hirogen · commit d5510ae486c1 · 2025-11-30T19:43:24.000+01:00
diff --git a/src/LogExpert.Configuration/ConfigManager.cs b/src/LogExpert.Configuration/ConfigManager.cs
@@ -1016,10 +1016,7 @@ private static bool SettingsAreEmptyOrDefault (Settings settings, ExportImportFl
         {
             // For 'Other', we consider the settings valid if Preferences object exists
             // This covers font settings, colors, and other preference data
-            if (settings.Preferences != null)
-            {
-                hasAnyRelevantData = true;
-            }
+            hasAnyRelevantData = true;
         }
 
         // Return true (isEmpty) if no relevant data was found in any checked section

Original file line number	Diff line number	Diff line change
`@@ -1016,10 +1016,7 @@ private static bool SettingsAreEmptyOrDefault (Settings settings, ExportImportFl`
`1016`	`1016`	`{`
`1017`	`1017`	`// For 'Other', we consider the settings valid if Preferences object exists`
`1018`	`1018`	`// This covers font settings, colors, and other preference data`
`1019`		`- if (settings.Preferences != null)`
`1020`		`- {`
`1021`		`- hasAnyRelevantData = true;`
`1022`		`- }`
	`1019`	`+ hasAnyRelevantData = true;`
`1023`	`1020`	`}`
`1024`	`1021`
`1025`	`1022`	`// Return true (isEmpty) if no relevant data was found in any checked section`