♻️ Replaced all Debug.WriteLine for Trace.WriteLine (#1311)

replaced all Debug.WriteLine for Trace.WriteLine

CONTRIBUTING.md

@@ -119,6 +119,9 @@ Here we will have some:
 
 ## Contributing Code - Best Practices
 
+### Debug Logging
+* Always use `Trace.WriteLine()` instead of `Debug.WriteLine` for debug logging because `Debug.WriteLine` is removed by the compiler in Release builds
+
 ### Enums
 * Always use `Unknown` at index 0 for return types that may have a value that is not known
 * Always use `Default` at index 0 for option types that can use the system default option

src/CommunityToolkit.Maui.Core/Essentials/Badge/Android/DefaultBadgeProvider.android.cs

@@ -13,20 +13,20 @@ public void SetCount(uint count)
 		var packageName = Application.Context.PackageName;
 		if (packageName is null)
 		{
-			Debug.WriteLine("Unable to get package name");
+			Trace.WriteLine("Unable to get package name");
 			return;
 		}
 
 		var component = Application.Context.PackageManager?.GetLaunchIntentForPackage(packageName)?.Component;
 		if (component is null)
 		{
-			Debug.WriteLine($"Unable to get launch intent component for package {packageName}");
+			Trace.WriteLine($"Unable to get launch intent component for package {packageName}");
 			return;
 		}
 
 		if (!CanSetBadgeCounter())
 		{
-			Debug.WriteLine("Current launcher doesn't support badge counter");
+			Trace.WriteLine("Current launcher doesn't support badge counter");
 			return;
 		}
 
@@ -40,7 +40,7 @@ public void SetCount(uint count)
 		}
 		catch (Exception ex)
 		{
-			Debug.WriteLine($"{nameof(DefaultBadgeProvider)} unable to set badge count: " + ex.Message);
+			Trace.WriteLine($"{nameof(DefaultBadgeProvider)} unable to set badge count: " + ex.Message);
 		}
 	}
 

src/CommunityToolkit.Maui.Core/Essentials/Badge/BadgeImplementation.macios.cs

@@ -13,7 +13,7 @@ public void SetCount(uint count)
 		{
 			if (error is not null)
 			{
-				Debug.WriteLine($"Error Requesting Authorization to Set Badge Count: {error.Description}");
+				Trace.WriteLine($"Error Requesting Authorization to Set Badge Count: {error.Description}");
 			}
 		});
 

src/CommunityToolkit.Maui.Core/Platform/StatusBar/StatusBar.android.cs

@@ -65,7 +65,7 @@ static bool IsSupported()
 			return true;
 		}
 
-		System.Diagnostics.Debug.WriteLine($"This functionality is not available. Minimum supported API is {(int)BuildVersionCodes.M}");
+		System.Diagnostics.Trace.WriteLine($"This functionality is not available. Minimum supported API is {(int)BuildVersionCodes.M}");
 		return false;
 	}
 }

src/CommunityToolkit.Maui.Core/Views/DrawingView/PlatformView/MauiDrawingView.windows.cs

@@ -27,7 +27,7 @@ public void Initialize()
 		}
 		else
 		{
-			System.Diagnostics.Debug.WriteLine("DrawingView requires Windows 10.0.18362 or higher.");
+			System.Diagnostics.Trace.WriteLine("DrawingView requires Windows 10.0.18362 or higher.");
 		}
 
 		Lines.CollectionChanged += OnLinesCollectionChanged;

src/CommunityToolkit.Maui.SourceGenerators/Generators/TextColorToGenerator.cs

@@ -145,7 +145,7 @@ namespace {{textStyleClassMetadata.Namespace}};
 		{
 			//When creating an Animation too early in the lifecycle of the Page, i.e. in the OnAppearing method,
 			//the Page might not have an 'IAnimationManager' yet, resulting in an ArgumentException.
-			System.Diagnostics.Debug.WriteLine($"{aex.GetType().Name} thrown in {typeof(ColorAnimationExtensions_{{textStyleClassMetadata.ClassName}}).FullName}: {aex.Message}");
+			System.Diagnostics.Trace.WriteLine($"{aex.GetType().Name} thrown in {typeof(ColorAnimationExtensions_{{textStyleClassMetadata.ClassName}}).FullName}: {aex.Message}");
 			animationCompletionSource.SetResult(false);
 		}
 

src/CommunityToolkit.Maui.UnitTests/Layouts/StateContainerTests.cs

@@ -567,7 +567,7 @@ public bool CanChangeState
 			}
 		}
 
-		Command ChangeStateCommand => changeStateCommand ??= new Command(() => Debug.WriteLine("Command Tapped"), () => CanChangeState);
+		Command ChangeStateCommand => changeStateCommand ??= new Command(() => Trace.WriteLine("Command Tapped"), () => CanChangeState);
 
 		public event PropertyChangedEventHandler? PropertyChanged;
 

src/CommunityToolkit.Maui/Behaviors/AnimationBehavior.shared.cs

@@ -99,7 +99,7 @@ Task OnAnimate()
 		}
 		catch (Exception ex) when (Options.ShouldSuppressExceptionsInAnimations)
 		{
-			Debug.WriteLine(ex);
+			Trace.WriteLine(ex);
 			return Task.CompletedTask;
 		}
 	}

src/CommunityToolkit.Maui/Behaviors/BaseBehavior.shared.cs

@@ -116,7 +116,7 @@ void OnViewPropertyChanged(object? sender, PropertyChangedEventArgs e)
 		}
 		catch (Exception ex) when (Options.ShouldSuppressExceptionsInBehaviors)
 		{
-			Debug.WriteLine(ex);
+			Trace.WriteLine(ex);
 		};
 	}
 }

src/CommunityToolkit.Maui/Behaviors/ImpliedOrderGridBehavior.shared.cs

@@ -34,7 +34,7 @@ protected override void OnDetachingFrom(Grid bindable)
 
 	static void LogWarning(string warning, bool shouldThrowException)
 	{
-		System.Diagnostics.Debug.WriteLine(warning);
+		System.Diagnostics.Trace.WriteLine(warning);
 
 		if (shouldThrowException)
 		{

src/CommunityToolkit.Maui/Behaviors/Validators/ValidationBehavior.shared.cs

@@ -399,7 +399,7 @@ async ValueTask UpdateStateAsync(VisualElement? view, ValidationFlags flags, boo
 				IsValid = false;
 				IsRunning = false;
 
-				Debug.WriteLine(ex);
+				Trace.WriteLine(ex);
 			}
 		}
 

src/CommunityToolkit.Maui/Converters/BaseConverter.shared.cs

@@ -70,7 +70,7 @@ public abstract class BaseConverter<TFrom, TTo, TParam> : ValueConverterExtensio
 		}
 		catch (Exception ex) when (Options.ShouldSuppressExceptionsInConverters)
 		{
-			Debug.WriteLine(ex);
+			Trace.WriteLine(ex);
 			return DefaultConvertBackReturnValue;
 		}
 	}
@@ -89,7 +89,7 @@ public abstract class BaseConverter<TFrom, TTo, TParam> : ValueConverterExtensio
 		}
 		catch (Exception ex) when (Options.ShouldSuppressExceptionsInConverters)
 		{
-			Debug.WriteLine(ex);
+			Trace.WriteLine(ex);
 			return DefaultConvertReturnValue;
 		}
 	}
@@ -152,7 +152,7 @@ public abstract class BaseConverter<TFrom, TTo> : ValueConverterExtension, IComm
 		}
 		catch (Exception ex) when (Options.ShouldSuppressExceptionsInConverters)
 		{
-			Debug.WriteLine(ex);
+			Trace.WriteLine(ex);
 			return DefaultConvertBackReturnValue;
 		}
 	}
@@ -170,7 +170,7 @@ public abstract class BaseConverter<TFrom, TTo> : ValueConverterExtension, IComm
 		}
 		catch (Exception ex) when (Options.ShouldSuppressExceptionsInConverters)
 		{
-			Debug.WriteLine(ex);
+			Trace.WriteLine(ex);
 			return DefaultConvertReturnValue;
 		}
 	}

src/CommunityToolkit.Maui/Extensions/ColorAnimationExtensions.shared.cs

@@ -42,7 +42,7 @@ public static Task<bool> BackgroundColorTo(this VisualElement element, Color col
 		{
 			//When creating an Animation too early in the lifecycle of the Page, i.e. in the OnAppearing method,
 			//the Page might not have an 'IAnimationManager' yet, resulting in an ArgumentException.
-			System.Diagnostics.Debug.WriteLine($"{aex.GetType().Name} thrown in {typeof(ColorAnimationExtensions).FullName}: {aex.Message}");
+			System.Diagnostics.Trace.WriteLine($"{aex.GetType().Name} thrown in {typeof(ColorAnimationExtensions).FullName}: {aex.Message}");
 			animationCompletionSource.SetResult(false);
 		}
 

src/CommunityToolkit.Maui/Extensions/HttpClientExtensions.shared.cs

@@ -21,7 +21,7 @@ public static async Task<Stream> DownloadStreamAsync(this HttpClient client, Uri
 		}
 		catch (Exception ex)
 		{
-			Debug.WriteLine($"Error getting stream for {uri}: {ex}");
+			Trace.WriteLine($"Error getting stream for {uri}: {ex}");
 			return Stream.Null;
 		}
 	}