Skip to content

Commit c95717d

Browse files
committed
Fix detekt ReturnCount in NetworkConnectionMonitor
Collapse hasActiveConnection's guard clauses into a safe-call chain so it has two returns instead of four. No behavior change.
1 parent 32a6852 commit c95717d

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

WordPress/src/main/java/org/wordpress/android/networking/NetworkConnectionMonitor.kt

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -69,8 +69,7 @@ class NetworkConnectionMonitor @Inject constructor() {
6969
*/
7070
private fun hasActiveConnection(): Boolean {
7171
val manager = connectivityManager ?: return false
72-
val activeNetwork = manager.activeNetwork ?: return false
73-
val capabilities = manager.getNetworkCapabilities(activeNetwork) ?: return false
74-
return capabilities.hasCapability(NetworkCapabilities.NET_CAPABILITY_INTERNET)
72+
val capabilities = manager.activeNetwork?.let { manager.getNetworkCapabilities(it) }
73+
return capabilities?.hasCapability(NetworkCapabilities.NET_CAPABILITY_INTERNET) == true
7574
}
7675
}

0 commit comments

Comments
 (0)