Skip to content

Commit

Permalink
Merge pull request #20362 from wordpress-mobile/issue/20334-fix-incor…
Browse files Browse the repository at this point in the history
…rect-date-range

Fix incorrect date range on the traffic tab
  • Loading branch information
aditi-bhatia authored Mar 2, 2024
2 parents 7508d71 + e280ec1 commit 4dfebc9
Showing 1 changed file with 13 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ import org.wordpress.android.ui.stats.refresh.utils.trackWithGranularity
import org.wordpress.android.util.AppLog
import org.wordpress.android.util.analytics.AnalyticsTrackerWrapper
import org.wordpress.android.viewmodel.ResourceProvider
import java.util.Calendar
import java.util.Date
import javax.inject.Inject
import javax.inject.Named
Expand Down Expand Up @@ -110,9 +111,19 @@ class TrafficOverviewUseCase(
}

private fun getLastDate(model: VisitsAndViewsModel?): Date? {
selectedDateProvider.getSelectedDate(statsGranularity)?.let { return it }
selectedDateProvider.getSelectedDate(statsGranularity)?.let { return dateWithoutHour(it) }

val lastDateString = model?.dates?.lastOrNull()?.period
return lastDateString?.let { statsDateFormatter.parseStatsDate(statsGranularity, it) }
return lastDateString?.let { dateWithoutHour(statsDateFormatter.parseStatsDate(statsGranularity, it)) }
}

// Remove the hour and minute from the date to avoid fetching incorrect dates caused by timezone differences
private fun dateWithoutHour(date: Date): Date {
val calendar = Calendar.getInstance()
calendar.time = date
calendar.set(Calendar.HOUR_OF_DAY, 0)
calendar.set(Calendar.MINUTE, 0)
return calendar.time
}

override suspend fun fetchRemoteData(forced: Boolean): State<TrafficOverviewUiModel> {
Expand Down

0 comments on commit 4dfebc9

Please sign in to comment.