refactor: apply ktlint recommendations

Author: thunderbiscuit

app/src/main/java/org/bitcoindevkit/devkitwallet/data/Kyoto.kt

@@ -111,18 +111,22 @@ class Kyoto private constructor(
 
         fun create(wallet: Wallet, dataDir: String, network: Network): Kyoto {
             Log.i(TAG, "Starting Kyoto node")
-            val peers: List<Peer> = when(network) {
+            val peers: List<Peer> = when (network) {
                 Network.REGTEST -> {
                     val ip: IpAddress = IpAddress.fromIpv4(10u, 0u, 2u, 2u)
                     val peer1: Peer = Peer(ip, 18444u, false)
                     listOf(peer1)
                 }
+
                 Network.SIGNET -> {
                     val ip: IpAddress = IpAddress.fromIpv4(68u, 47u, 229u, 218u)
                     val peer1: Peer = Peer(ip, null, false)
                     listOf(peer1)
                 }
-                else -> { listOf() }
+
+                else -> {
+                    listOf()
+                }
             }
 
             val (client, node) =

app/src/main/java/org/bitcoindevkit/devkitwallet/domain/BlockchainClientsConfig.kt

@@ -43,14 +43,22 @@ class BlockchainClientsConfig {
                 Network.REGTEST -> {
                     config.addClient(EsploraClient("http://10.0.2.2:3002"), true)
                 }
+
                 Network.TESTNET -> {
                     config.addClient(ElectrumClient("ssl://electrum.blockstream.info:60002"), true)
                 }
-                Network.TESTNET4 -> throw IllegalArgumentException("This app does not support testnet 4 yet")
+
+                Network.TESTNET4 -> {
+                    throw IllegalArgumentException("This app does not support testnet 4 yet")
+                }
+
                 Network.SIGNET -> {
                     config.addClient(ElectrumClient("ssl://mempool.space:60602"), true)
                 }
-                Network.BITCOIN -> throw IllegalArgumentException("This app does not support mainnet")
+
+                Network.BITCOIN -> {
+                    throw IllegalArgumentException("This app does not support mainnet")
+                }
             }
             return config
         }

app/src/main/java/org/bitcoindevkit/devkitwallet/domain/Wallet.kt

@@ -142,13 +142,17 @@ class Wallet private constructor(
 
             val (confirmationBlock, confirmationTimestamp, pending) =
                 when (val position = tx.chainPosition) {
-                    is ChainPosition.Unconfirmed -> Triple(null, null, true)
-                    is ChainPosition.Confirmed ->
+                    is ChainPosition.Unconfirmed -> {
+                        Triple(null, null, true)
+                    }
+
+                    is ChainPosition.Confirmed -> {
                         Triple(
                             ConfirmationBlock(position.confirmationBlockTime.blockId.height),
                             Timestamp(position.confirmationBlockTime.confirmationTime),
                             false,
                         )
+                    }
                 }
             TxDetails(
                 tx.transaction,

app/src/main/java/org/bitcoindevkit/devkitwallet/presentation/DevkitWalletActivity.kt

@@ -8,8 +8,8 @@ package org.bitcoindevkit.devkitwallet.presentation
 import android.content.Context
 import android.os.Bundle
 import android.util.Log
-import androidx.activity.compose.setContent
 import androidx.activity.ComponentActivity
+import androidx.activity.compose.setContent
 import androidx.compose.runtime.getValue
 import androidx.compose.runtime.mutableStateOf
 import androidx.compose.runtime.setValue
@@ -58,24 +58,29 @@ class DevkitWalletActivity : ComponentActivity() {
             try {
                 activeWallet =
                     when (walletCreateType) {
-                        is WalletCreateType.FROMSCRATCH ->
+                        is WalletCreateType.FROMSCRATCH -> {
                             Wallet.createWallet(
                                 newWalletConfig = walletCreateType.newWalletConfig,
                                 internalAppFilesPath = filesDir.absolutePath,
                                 userPreferencesRepository = userPreferencesRepository,
                             )
-                        is WalletCreateType.LOADEXISTING ->
+                        }
+
+                        is WalletCreateType.LOADEXISTING -> {
                             Wallet.loadActiveWallet(
                                 activeWallet = walletCreateType.activeWallet,
                                 internalAppFilesPath = filesDir.absolutePath,
                                 userPreferencesRepository = userPreferencesRepository,
                             )
-                        is WalletCreateType.RECOVER ->
+                        }
+
+                        is WalletCreateType.RECOVER -> {
                             Wallet.recoverWallet(
                                 recoverWalletConfig = walletCreateType.recoverWalletConfig,
                                 internalAppFilesPath = filesDir.absolutePath,
                                 userPreferencesRepository = userPreferencesRepository,
                             )
+                        }
                     }
             } catch (e: Throwable) {
                 Log.i(TAG, "Could not build wallet: $e")

app/src/main/java/org/bitcoindevkit/devkitwallet/presentation/navigation/AppNavigation.kt

@@ -24,15 +24,15 @@ import androidx.navigation.toRoute
 import org.bitcoindevkit.devkitwallet.data.SingleWallet
 import org.bitcoindevkit.devkitwallet.domain.Wallet
 import org.bitcoindevkit.devkitwallet.presentation.WalletCreateType
+import org.bitcoindevkit.devkitwallet.presentation.ui.screens.intro.ActiveWalletsScreen
+import org.bitcoindevkit.devkitwallet.presentation.ui.screens.intro.CreateNewWalletScreen
+import org.bitcoindevkit.devkitwallet.presentation.ui.screens.intro.RecoverWalletScreen
+import org.bitcoindevkit.devkitwallet.presentation.ui.screens.intro.WalletChoiceScreen
 import org.bitcoindevkit.devkitwallet.presentation.ui.screens.settings.AboutScreen
 import org.bitcoindevkit.devkitwallet.presentation.ui.screens.settings.BlockchainClientScreen
 import org.bitcoindevkit.devkitwallet.presentation.ui.screens.settings.LogsScreen
 import org.bitcoindevkit.devkitwallet.presentation.ui.screens.settings.RecoveryDataScreen
 import org.bitcoindevkit.devkitwallet.presentation.ui.screens.settings.SettingsScreen
-import org.bitcoindevkit.devkitwallet.presentation.ui.screens.intro.ActiveWalletsScreen
-import org.bitcoindevkit.devkitwallet.presentation.ui.screens.intro.CreateNewWalletScreen
-import org.bitcoindevkit.devkitwallet.presentation.ui.screens.intro.RecoverWalletScreen
-import org.bitcoindevkit.devkitwallet.presentation.ui.screens.intro.WalletChoiceScreen
 import org.bitcoindevkit.devkitwallet.presentation.ui.screens.wallet.RBFScreen
 import org.bitcoindevkit.devkitwallet.presentation.ui.screens.wallet.ReceiveScreen
 import org.bitcoindevkit.devkitwallet.presentation.ui.screens.wallet.SendScreen

app/src/main/java/org/bitcoindevkit/devkitwallet/presentation/ui/screens/wallet/TransactionScreen.kt

@@ -85,6 +85,7 @@ fun TransactionDetailButton(content: String, navController: NavController, txid:
                 "increase fees" -> {
                     navController.navigate(RbfScreen(txid!!))
                 }
+
                 "back to transaction list" -> {
                     navController.navigateUp()
                 }

app/src/main/java/org/bitcoindevkit/devkitwallet/presentation/ui/screens/wallet/WalletHomeScreen.kt

@@ -141,6 +141,7 @@ internal fun WalletHomeScreen(
                         color = colorScheme.onSurface,
                     )
                 }
+
                 CurrencyUnit.Satoshi -> {
                     Text(
                         text = "${state.balance} sat",
@@ -397,10 +398,12 @@ fun isOnline(context: Context): Boolean {
                 Log.i("Internet", "NetworkCapabilities.TRANSPORT_CELLULAR")
                 return true
             }
+
             capabilities.hasTransport(NetworkCapabilities.TRANSPORT_WIFI) -> {
                 Log.i("Internet", "NetworkCapabilities.TRANSPORT_WIFI")
                 return true
             }
+
             capabilities.hasTransport(NetworkCapabilities.TRANSPORT_ETHERNET) -> {
                 Log.i("Internet", "NetworkCapabilities.TRANSPORT_ETHERNET")
                 return true

app/src/main/java/org/bitcoindevkit/devkitwallet/presentation/viewmodels/SendViewModel.kt

@@ -36,14 +36,18 @@ internal class SendViewModel(private val wallet: Wallet) : ViewModel() {
                 // Create, sign, and broadcast
                 val psbt: Psbt =
                     when (txInfo.transactionType) {
-                        TransactionType.STANDARD ->
+                        TransactionType.STANDARD -> {
                             wallet.createTransaction(
                                 recipientList = txInfo.recipients,
                                 feeRate = FeeRate.fromSatPerVb(txInfo.feeRate),
                                 opReturnMsg = txInfo.opReturnMsg,
                             )
+                        }
+
                         // TransactionType.SEND_ALL -> Wallet.createSendAllTransaction(recipientList[0].address, FeeRate.fromSatPerVb(feeRate), rbfEnabled, opReturnMsg)
-                        TransactionType.SEND_ALL -> throw NotImplementedError("Send all not implemented")
+                        TransactionType.SEND_ALL -> {
+                            throw NotImplementedError("Send all not implemented")
+                        }
                     }
                 val isSigned = wallet.sign(psbt)
                 if (isSigned) {

app/src/main/java/org/bitcoindevkit/devkitwallet/presentation/viewmodels/WalletViewModel.kt

@@ -33,6 +33,7 @@ internal class WalletViewModel(
     private val kyotoCoroutineScope: CoroutineScope = CoroutineScope(Dispatchers.IO)
     private var kyoto: Kyoto? = null
 
+    @Suppress("ktlint:standard:no-multi-spaces")
     fun onAction(action: WalletScreenAction) {
         when (action) {
             WalletScreenAction.SwitchUnit      -> switchUnit()