Skip to content

Commit ec2043f

Browse files
committed
Fix ignored files committed
Spotless breaks because we have classes and files that have been commited that should technically be ignored by our .gitignore settings. See diffplug/spotless#911 (comment) Therefore we have to either remove these files or move them. In this case the DeviceShield stuff was in a package called "report" which we ignore in .gitignore. Therefore I gave it a new package name We want to keep the icon and I guess we want to keep the copyright folder so I made that work in .gitignore
1 parent 3ae0802 commit ec2043f

File tree

9 files changed

+28
-24
lines changed

9 files changed

+28
-24
lines changed

.gitignore

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,9 +41,10 @@ captures/
4141

4242
# Intellij
4343
*.iml
44-
.idea/
45-
.idea/workspace.xml
44+
.idea/*
4645
!.idea/icon.svg
46+
!.idea/copyright/
47+
.idea/workspace.xml
4748

4849
# Keystore files
4950
*.jks

app-tracking-protection/vpn-impl/src/main/AndroidManifest.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@
2828
android:screenOrientation="portrait" />
2929

3030
<activity
31-
android:name="com.duckduckgo.mobile.android.vpn.ui.report.DeviceShieldAppTrackersInfo"
31+
android:name="com.duckduckgo.mobile.android.vpn.ui.privacyreport.DeviceShieldAppTrackersInfo"
3232
android:exported="false"
3333
android:label="@string/atp_ProfilingActivityTitle"
3434
android:parentActivityName="com.duckduckgo.mobile.android.vpn.ui.tracker_activity.DeviceShieldTrackerActivity"

app-tracking-protection/vpn-impl/src/main/java/com/duckduckgo/mobile/android/vpn/ui/newtab/AppTrackingProtectionStateView.kt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -39,9 +39,9 @@ import com.duckduckgo.mobile.android.vpn.pixels.DeviceShieldPixels
3939
import com.duckduckgo.mobile.android.vpn.state.VpnStateMonitor.VpnRunningState.ENABLED
4040
import com.duckduckgo.mobile.android.vpn.state.VpnStateMonitor.VpnStopReason.REVOKED
4141
import com.duckduckgo.mobile.android.vpn.ui.onboarding.VpnStore
42-
import com.duckduckgo.mobile.android.vpn.ui.report.PrivacyReportViewModel
43-
import com.duckduckgo.mobile.android.vpn.ui.report.PrivacyReportViewModel.PrivacyReportView.TrackersBlocked
44-
import com.duckduckgo.mobile.android.vpn.ui.report.PrivacyReportViewModel.PrivacyReportView.ViewState
42+
import com.duckduckgo.mobile.android.vpn.ui.privacyreport.PrivacyReportViewModel
43+
import com.duckduckgo.mobile.android.vpn.ui.privacyreport.PrivacyReportViewModel.PrivacyReportView.TrackersBlocked
44+
import com.duckduckgo.mobile.android.vpn.ui.privacyreport.PrivacyReportViewModel.PrivacyReportView.ViewState
4545
import com.duckduckgo.mobile.android.vpn.ui.tracker_activity.DeviceShieldTrackerActivity
4646
import com.duckduckgo.newtabpage.api.NewTabPageSection
4747
import com.duckduckgo.newtabpage.api.NewTabPageSectionPlugin
Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright (c) 2021 DuckDuckGo
2+
* Copyright (c) 2025 DuckDuckGo
33
*
44
* Licensed under the Apache License, Version 2.0 (the "License");
55
* you may not use this file except in compliance with the License.
@@ -14,7 +14,7 @@
1414
* limitations under the License.
1515
*/
1616

17-
package com.duckduckgo.mobile.android.vpn.ui.report
17+
package com.duckduckgo.mobile.android.vpn.ui.privacyreport
1818

1919
import android.content.Context
2020
import android.content.Intent
Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright (c) 2021 DuckDuckGo
2+
* Copyright (c) 2025 DuckDuckGo
33
*
44
* Licensed under the Apache License, Version 2.0 (the "License");
55
* you may not use this file except in compliance with the License.
@@ -14,7 +14,7 @@
1414
* limitations under the License.
1515
*/
1616

17-
package com.duckduckgo.mobile.android.vpn.ui.report
17+
package com.duckduckgo.mobile.android.vpn.ui.privacyreport
1818

1919
import android.app.ActivityOptions
2020
import android.os.Bundle
@@ -38,7 +38,8 @@ import com.duckduckgo.mobile.android.vpn.R
3838
import com.duckduckgo.mobile.android.vpn.pixels.DeviceShieldPixels
3939
import com.duckduckgo.mobile.android.vpn.state.VpnStateMonitor.VpnRunningState.ENABLED
4040
import com.duckduckgo.mobile.android.vpn.state.VpnStateMonitor.VpnStopReason.REVOKED
41-
import com.duckduckgo.mobile.android.vpn.ui.report.PrivacyReportViewModel.PrivacyReportView.ViewState
41+
import com.duckduckgo.mobile.android.vpn.ui.privacyreport.PrivacyReportViewModel.PrivacyReportView.TrackersBlocked
42+
import com.duckduckgo.mobile.android.vpn.ui.privacyreport.PrivacyReportViewModel.PrivacyReportView.ViewState
4243
import com.duckduckgo.mobile.android.vpn.ui.tracker_activity.DeviceShieldTrackerActivity
4344
import javax.inject.Inject
4445
import kotlinx.coroutines.flow.launchIn
@@ -139,7 +140,7 @@ class DeviceShieldFragment : DuckDuckGoFragment() {
139140
deviceShieldCtaImageView.setImageResource(R.drawable.ic_apptp_warning)
140141
}
141142

142-
private fun renderTrackersBlockedWhenEnabled(trackerBlocked: PrivacyReportViewModel.PrivacyReportView.TrackersBlocked) {
143+
private fun renderTrackersBlockedWhenEnabled(trackerBlocked: TrackersBlocked) {
143144
val trackersBlocked = trackerBlocked.trackers
144145
val lastTrackingApp = trackerBlocked.latestApp
145146
val otherApps = trackerBlocked.otherAppsSize
Lines changed: 11 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright (c) 2021 DuckDuckGo
2+
* Copyright (c) 2025 DuckDuckGo
33
*
44
* Licensed under the Apache License, Version 2.0 (the "License");
55
* you may not use this file except in compliance with the License.
@@ -14,7 +14,7 @@
1414
* limitations under the License.
1515
*/
1616

17-
package com.duckduckgo.mobile.android.vpn.ui.report
17+
package com.duckduckgo.mobile.android.vpn.ui.privacyreport
1818

1919
import androidx.annotation.VisibleForTesting
2020
import androidx.lifecycle.ViewModel
@@ -28,6 +28,8 @@ import com.duckduckgo.mobile.android.vpn.state.VpnStateMonitor
2828
import com.duckduckgo.mobile.android.vpn.state.VpnStateMonitor.VpnState
2929
import com.duckduckgo.mobile.android.vpn.stats.AppTrackerBlockingStatsRepository
3030
import com.duckduckgo.mobile.android.vpn.ui.onboarding.VpnStore
31+
import com.duckduckgo.mobile.android.vpn.ui.privacyreport.PrivacyReportViewModel.PrivacyReportView.TrackersBlocked
32+
import com.duckduckgo.mobile.android.vpn.ui.privacyreport.PrivacyReportViewModel.PrivacyReportView.ViewState
3133
import javax.inject.Inject
3234
import kotlinx.coroutines.flow.Flow
3335
import kotlinx.coroutines.flow.combine
@@ -44,20 +46,20 @@ class PrivacyReportViewModel @Inject constructor(
4446
) : ViewModel() {
4547

4648
val viewStateFlow = vpnStateMonitor.getStateFlow(AppTpVpnFeature.APPTP_VPN).combine(getReport()) { vpnState, trackersBlocked ->
47-
PrivacyReportView.ViewState(vpnState, trackersBlocked, shouldShowCTA())
49+
ViewState(vpnState, trackersBlocked, shouldShowCTA())
4850
}
4951

5052
@VisibleForTesting
51-
fun getReport(): Flow<PrivacyReportView.TrackersBlocked> {
53+
fun getReport(): Flow<TrackersBlocked> {
5254
return repository.getVpnTrackers({ dateOfLastHour() }).map { trackers ->
5355
if (trackers.isEmpty()) {
54-
PrivacyReportView.TrackersBlocked("", 0, 0)
56+
TrackersBlocked("", 0, 0)
5557
} else {
5658
val perApp = trackers.groupBy { it.trackingApp }.toList().sortedByDescending { it.second.sumOf { t -> t.count } }
5759
val otherAppsSize = (perApp.size - 1).coerceAtLeast(0)
5860
val latestApp = perApp.first().first.appDisplayName
5961

60-
PrivacyReportView.TrackersBlocked(latestApp, otherAppsSize, trackers.sumOf { it.count })
62+
TrackersBlocked(latestApp, otherAppsSize, trackers.sumOf { it.count })
6163
}
6264
}
6365
}
@@ -74,9 +76,9 @@ class PrivacyReportViewModel @Inject constructor(
7476

7577
object PrivacyReportView {
7678
data class ViewState(
77-
val vpnState: VpnState,
78-
val trackersBlocked: TrackersBlocked,
79-
val isFeatureEnabled: Boolean,
79+
val vpnState: VpnState,
80+
val trackersBlocked: TrackersBlocked,
81+
val isFeatureEnabled: Boolean,
8082
)
8183

8284
data class TrackersBlocked(

app-tracking-protection/vpn-impl/src/main/java/com/duckduckgo/mobile/android/vpn/ui/tracker_activity/DeviceShieldTrackerActivity.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ import com.duckduckgo.mobile.android.vpn.state.VpnStateMonitor.VpnRunningState
6868
import com.duckduckgo.mobile.android.vpn.state.VpnStateMonitor.VpnState
6969
import com.duckduckgo.mobile.android.vpn.ui.AppBreakageCategory
7070
import com.duckduckgo.mobile.android.vpn.ui.alwayson.AlwaysOnAlertDialogFragment
71-
import com.duckduckgo.mobile.android.vpn.ui.report.DeviceShieldAppTrackersInfo
71+
import com.duckduckgo.mobile.android.vpn.ui.privacyreport.DeviceShieldAppTrackersInfo
7272
import com.duckduckgo.mobile.android.vpn.ui.tracker_activity.DeviceShieldTrackerActivityViewModel.ViewEvent
7373
import com.duckduckgo.mobile.android.vpn.ui.tracker_activity.DeviceShieldTrackerActivityViewModel.ViewEvent.StartVpn
7474
import com.duckduckgo.mobile.android.vpn.ui.tracker_activity.view.DisableVpnDialogOptions

app-tracking-protection/vpn-impl/src/main/res/layout/activity_app_trackers_info.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@
2424
android:layout_width="match_parent"
2525
android:layout_height="match_parent"
2626
android:orientation="vertical"
27-
tools:context=".ui.report.DeviceShieldAppTrackersInfo">
27+
tools:context=".ui.privacyreport.DeviceShieldAppTrackersInfo">
2828

2929
<include
3030
android:id="@+id/include_toolbar"

app-tracking-protection/vpn-impl/src/test/java/com/duckduckgo/mobile/android/vpn/ui/PrivacyReportViewModelTest.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ import com.duckduckgo.mobile.android.vpn.stats.RealAppTrackerBlockingStatsReposi
3232
import com.duckduckgo.mobile.android.vpn.store.VpnDatabase
3333
import com.duckduckgo.mobile.android.vpn.trackers.AppTrackerEntity
3434
import com.duckduckgo.mobile.android.vpn.ui.onboarding.VpnStore
35-
import com.duckduckgo.mobile.android.vpn.ui.report.PrivacyReportViewModel
35+
import com.duckduckgo.mobile.android.vpn.ui.privacyreport.PrivacyReportViewModel
3636
import java.time.LocalDateTime
3737
import kotlin.time.ExperimentalTime
3838
import kotlinx.coroutines.runBlocking

0 commit comments

Comments
 (0)