Merge pull request #2517 from DataDog/sbarrio/fix-logging-null-on-add-view-loading-time-first-call

sbarrio · web-flow · commit 323897077d2e · 2025-02-28T14:19:25.000+01:00
Avoid logging initial null viewLoadingTime on first call to addViewLoadingTime
diff --git a/features/dd-sdk-android-rum/src/main/kotlin/com/datadog/android/rum/internal/domain/scope/RumViewScope.kt b/features/dd-sdk-android-rum/src/main/kotlin/com/datadog/android/rum/internal/domain/scope/RumViewScope.kt
@@ -275,7 +275,7 @@ internal open class RumViewScope(
             internalLogger.log(
                 InternalLogger.Level.DEBUG,
                 InternalLogger.Target.USER,
-                { ADDING_VIEW_LOADING_TIME_DEBUG_MESSAGE_FORMAT.format(Locale.US, viewLoadingTime, viewName) }
+                { ADDING_VIEW_LOADING_TIME_DEBUG_MESSAGE_FORMAT.format(Locale.US, newLoadingTime, viewName) }
             )
             internalLogger.logApiUsage {
                 InternalTelemetryEvent.ApiUsage.AddViewLoadingTime(

Original file line number	Diff line number	Diff line change
`@@ -275,7 +275,7 @@ internal open class RumViewScope(`
`275`	`275`	`internalLogger.log(`
`276`	`276`	`InternalLogger.Level.DEBUG,`
`277`	`277`	`InternalLogger.Target.USER,`
`278`		`- { ADDING_VIEW_LOADING_TIME_DEBUG_MESSAGE_FORMAT.format(Locale.US, viewLoadingTime, viewName) }`
	`278`	`+ { ADDING_VIEW_LOADING_TIME_DEBUG_MESSAGE_FORMAT.format(Locale.US, newLoadingTime, viewName) }`
`279`	`279`	`)`
`280`	`280`	`internalLogger.logApiUsage {`
`281`	`281`	`InternalTelemetryEvent.ApiUsage.AddViewLoadingTime(`