Fix #8107: Fix incorrect account opening Send from Filecoin asset detail

Sources/BraveWallet/Crypto/Accounts/Add/AddAccountView.swift

@@ -29,13 +29,15 @@ struct AddAccountView: View {
   private var allFilNetworks: [BraveWallet.NetworkInfo]
 
   var preSelectedCoin: BraveWallet.CoinType?
+  var preSelectedFilecoinNetwork: BraveWallet.NetworkInfo?
   var onCreate: (() -> Void)?
   var onDismiss: (() -> Void)?
 
   init(
     keyringStore: KeyringStore,
     networkStore: NetworkStore,
     preSelectedCoin: BraveWallet.CoinType? = nil,
+    preSelectedFilecoinNetwork: BraveWallet.NetworkInfo? = nil,
     onCreate: (() -> Void)? = nil,
     onDismiss: (() -> Void)? = nil
   ) {
@@ -44,7 +46,13 @@ struct AddAccountView: View {
     self.preSelectedCoin = preSelectedCoin
     self.onCreate = onCreate
     self.onDismiss = onDismiss
-    _filNetwork = .init(initialValue: self.allFilNetworks.first ?? .init())
+    if let preSelectedFilecoinNetwork, preSelectedFilecoinNetwork.coin == .fil { // make sure the prefilled
+      // network is a Filecoin network
+      self.preSelectedFilecoinNetwork = preSelectedFilecoinNetwork
+      _filNetwork = .init(initialValue: preSelectedFilecoinNetwork)
+    } else {
+      _filNetwork = .init(initialValue: self.allFilNetworks.first ?? .init())
+    }
   }
 
   private func addAccount(for coin: BraveWallet.CoinType) {
@@ -114,6 +122,7 @@ struct AddAccountView: View {
           Text(Strings.Wallet.transactionDetailsNetworkTitle)
             .foregroundColor(Color(.braveLabel))
         }
+        .disabled(preSelectedFilecoinNetwork != nil)
         .listRowBackground(Color(.secondaryBraveGroupedBackground))
       }
       accountNameSection

Sources/BraveWallet/Crypto/Asset Details/AssetDetailHeaderView.swift

@@ -22,6 +22,7 @@ struct AssetDetailHeaderView: View {
   @ObservedObject var networkStore: NetworkStore
   @Binding var buySendSwapDestination: BuySendSwapDestination?
   @Binding var isShowingBridgeAlert: Bool
+  var onAccountCreationNeeded: (_ savedDestination: BuySendSwapDestination) -> Void
 
   @Environment(\.sizeCategory) private var sizeCategory
   @Environment(\.horizontalSizeClass) private var horizontalSizeClass
@@ -73,10 +74,15 @@ struct AssetDetailHeaderView: View {
       if assetDetailStore.isBuySupported {
         Button(
           action: {
-            buySendSwapDestination = BuySendSwapDestination(
+            let destination = BuySendSwapDestination(
               kind: .buy,
               initialToken: assetDetailStore.assetDetailToken
             )
+            if assetDetailStore.accounts.isEmpty {
+              onAccountCreationNeeded(destination)
+            } else {
+              buySendSwapDestination = destination
+            }
           }
         ) {
           Text(Strings.Wallet.buy)
@@ -85,10 +91,15 @@ struct AssetDetailHeaderView: View {
       if assetDetailStore.isSendSupported {
         Button(
           action: {
-            buySendSwapDestination = BuySendSwapDestination(
+            let destination = BuySendSwapDestination(
               kind: .send,
               initialToken: assetDetailStore.assetDetailToken
             )
+            if assetDetailStore.accounts.isEmpty {
+              onAccountCreationNeeded(destination)
+            } else {
+              buySendSwapDestination = destination
+            }
           }
         ) {
           Text(Strings.Wallet.send)
@@ -97,10 +108,15 @@ struct AssetDetailHeaderView: View {
       if assetDetailStore.isSwapSupported && assetDetailStore.assetDetailToken.isFungibleToken {
         Button(
           action: {
-            buySendSwapDestination = BuySendSwapDestination(
+            let destination = BuySendSwapDestination(
               kind: .swap,
               initialToken: assetDetailStore.assetDetailToken
             )
+            if assetDetailStore.accounts.isEmpty {
+              onAccountCreationNeeded(destination)
+            } else {
+              buySendSwapDestination = destination
+            }
           }
         ) {
           Text(Strings.Wallet.swap)
@@ -251,7 +267,8 @@ struct CurrencyDetailHeaderView_Previews: PreviewProvider {
       keyringStore: .previewStore,
       networkStore: .previewStore,
       buySendSwapDestination: .constant(nil),
-      isShowingBridgeAlert: .constant(false)
+      isShowingBridgeAlert: .constant(false),
+      onAccountCreationNeeded: { _ in }
     )
     .padding(.vertical)
     .previewLayout(.sizeThatFits)

Sources/BraveWallet/Crypto/Asset Details/AssetDetailView.swift

@@ -23,6 +23,9 @@ struct AssetDetailView: View {
   @State private var isShowingAddAccount: Bool = false
   @State private var transactionDetails: TransactionDetailsStore?
   @State private var isShowingAuroraBridgeAlert: Bool = false
+  @State private var isPresentingAddAccount: Bool = false
+  @State private var isPresentingAddAccountConfirmation: Bool = false
+  @State private var savedBSSDestination: BuySendSwapDestination?
 
   @Environment(\.sizeCategory) private var sizeCategory
   /// Reference to the collection view used to back the `List` on iOS 16+
@@ -160,7 +163,11 @@ struct AssetDetailView: View {
           keyringStore: keyringStore,
           networkStore: networkStore,
           buySendSwapDestination: buySendSwapDestination,
-          isShowingBridgeAlert: $isShowingAuroraBridgeAlert
+          isShowingBridgeAlert: $isShowingAuroraBridgeAlert,
+          onAccountCreationNeeded: { savedDestination in
+            isPresentingAddAccountConfirmation = true
+            savedBSSDestination = savedDestination
+          }
         )
         .resetListHeaderStyle()
         .padding(.horizontal, tableInset)  // inset grouped layout margins workaround
@@ -300,6 +307,25 @@ struct AssetDetailView: View {
         isShowingAuroraBridgeAlert = false
       }
     }
+    .addAccount(
+      keyringStore: keyringStore,
+      networkStore: networkStore,
+      accountNetwork: networkStore.network(for: assetDetailStore.assetDetailToken),
+      isShowingConfirmation: $isPresentingAddAccountConfirmation,
+      isShowingAddAccount: $isPresentingAddAccount,
+      onConfirmAddAccount: { isPresentingAddAccount = true },
+      onCancelAddAccount: {},
+      onAddAccountDismissed: {
+        Task { @MainActor in
+          if await assetDetailStore.handleDismissAddAccount() {
+            if let savedBSSDestination {
+              buySendSwapDestination.wrappedValue = savedBSSDestination
+              self.savedBSSDestination = nil
+            }
+          }
+        }
+      }
+    )
   }
 }
 

Sources/BraveWallet/Crypto/BuySendSwap/SendTokenView.swift

@@ -8,6 +8,7 @@ import Strings
 import DesignSystem
 import BigNumber
 import BraveUI
+import BraveCore
 
 struct SendTokenView: View {
   @ObservedObject var keyringStore: KeyringStore
@@ -23,6 +24,7 @@ struct SendTokenView: View {
 
   @Environment(\.appRatingRequestAction) private var appRatingRequest
   @Environment(\.openURL) private var openURL
+  @Environment(\.presentationMode) @Binding private var presentationMode
 
   var completion: ((_ success: Bool) -> Void)?
   var onDismiss: () -> Void
@@ -292,7 +294,7 @@ struct SendTokenView: View {
         }
       }
     }
-    .task {
+    .task { @MainActor in
       if !didAutoShowSelectAccountToken {
         isShowingSelectAccountTokenView = true
         didAutoShowSelectAccountToken = true

Sources/BraveWallet/Crypto/NetworkSelectionView.swift

@@ -53,44 +53,25 @@ struct NetworkSelectionView: View {
         selectNetwork(network)
       }
     )
-    .background(
-      Color.clear
-        .alert(
-          isPresented: $store.isPresentingNextNetworkAlert
-        ) {
-          Alert(
-            title: Text(String.localizedStringWithFormat(Strings.Wallet.createAccountAlertTitle, store.nextNetwork?.shortChainName ?? "")),
-            message: Text(Strings.Wallet.createAccountAlertMessage),
-            primaryButton: .default(Text(Strings.yes), action: {
-              store.handleCreateAccountAlertResponse(shouldCreateAccount: true)
-            }),
-            secondaryButton: .cancel(Text(Strings.no), action: {
-              store.handleCreateAccountAlertResponse(shouldCreateAccount: false)
-            })
-          )
-        }
-    )
-    .background(
-      Color.clear
-        .sheet(
-          isPresented: $store.isPresentingAddAccount
-        ) {
-          NavigationView {
-            AddAccountView(
-              keyringStore: keyringStore,
-              networkStore: networkStore,
-              preSelectedCoin: store.nextNetwork?.coin
-            )
-          }
-          .navigationViewStyle(.stack)
-          .onDisappear {
-            Task { @MainActor in
-              if await store.handleDismissAddAccount() {
-                presentationMode.dismiss()
-              }
-            }
+    .addAccount(
+      keyringStore: keyringStore,
+      networkStore: networkStore,
+      accountNetwork: store.nextNetwork,
+      isShowingConfirmation: $store.isPresentingNextNetworkAlert,
+      isShowingAddAccount: $store.isPresentingAddAccount,
+      onConfirmAddAccount: {
+        store.handleCreateAccountAlertResponse(shouldCreateAccount: true)
+      },
+      onCancelAddAccount: {
+        store.handleCreateAccountAlertResponse(shouldCreateAccount: false)
+      },
+      onAddAccountDismissed: {
+        Task { @MainActor in
+          if await store.handleDismissAddAccount() {
+            presentationMode.dismiss()
           }
         }
+      }
     )
   }
 

Sources/BraveWallet/Crypto/Stores/AssetDetailStore.swift

@@ -326,6 +326,8 @@ class AssetDetailStore: ObservableObject {
           return tokenContractAddress.caseInsensitiveCompare(token.contractAddress) == .orderedSame
         case .erc1155SafeTransferFrom, .solanaDappSignTransaction, .solanaDappSignAndSendTransaction, .solanaSwap:
           return false
+        case .ethFilForwarderTransfer:
+          return false
         @unknown default:
           return false
         }
@@ -357,6 +359,16 @@ class AssetDetailStore: ObservableObject {
       userAssetManager: assetManager
     )
   }
+
+  /// Should be called after dismissing create account. Returns true if an account was created
+  @MainActor func handleDismissAddAccount() async -> Bool {
+    if await keyringService.isKeyringAvailable(for: assetDetailToken.coin) {
+      self.update()
+      return true
+    } else {
+      return false
+    }
+  }
 }
 
 extension AssetDetailStore: BraveWalletKeyringServiceObserver {

Sources/BraveWallet/Crypto/Stores/NetworkStore.swift

@@ -272,6 +272,10 @@ public class NetworkStore: ObservableObject {
   func customNetworkNativeAssetMigration(_ network: BraveWallet.NetworkInfo, completion: (() -> Void)? = nil) {
     assetManager.addUserAsset(network.nativeToken, completion: completion)
   }
+
+  func network(for token: BraveWallet.BlockchainToken) -> BraveWallet.NetworkInfo {
+    return allChains.first { $0.chainId == token.chainId } ?? customChains.first { $0.chainId == token.chainId } ?? allChains.first ?? .init()
+  }
 }
 
 extension NetworkStore: BraveWalletJsonRpcServiceObserver {

Sources/BraveWallet/Extensions/KeyringServiceExtensions.swift

@@ -63,4 +63,9 @@ extension BraveWalletKeyringService {
     )
     return allKeyrings
   }
+
+  /// Check if prefilled token's coin type has an associated keyring created.
+  @MainActor func isKeyringAvailable(for coin: BraveWallet.CoinType) async -> Bool {
+    await !keyrings(for: coin.keyringIds).filter(\.isKeyringCreated).isEmpty
+  }
 }

Sources/BraveWallet/Extensions/ViewExtensions.swift

@@ -5,6 +5,7 @@
 
 import SwiftUI
 import UIKit
+import BraveCore
 
 extension View {
   /// Helper for `hidden()` modifier that accepts a boolean determining if we should hide the view or not.
@@ -30,4 +31,48 @@ extension View {
       vc.navigationItem.backButtonDisplayMode = backButtonDisplayMode
     }
   }
+
+  func addAccount(
+    keyringStore: KeyringStore,
+    networkStore: NetworkStore,
+    accountNetwork: BraveWallet.NetworkInfo?,
+    isShowingConfirmation: Binding<Bool>,
+    isShowingAddAccount: Binding<Bool>,
+    onConfirmAddAccount: @escaping () -> Void,
+    onCancelAddAccount: @escaping () -> Void,
+    onAddAccountDismissed: @escaping () -> Void
+  ) -> some View {
+    self.background(
+      Color.clear
+        .alert(isPresented: isShowingConfirmation) {
+          Alert(
+            title: Text(String.localizedStringWithFormat(Strings.Wallet.createAccountAlertTitle, accountNetwork?.shortChainName ?? "")),
+            message: Text(Strings.Wallet.createAccountAlertMessage),
+            primaryButton: .default(Text(Strings.yes), action: {
+              onConfirmAddAccount()
+            }),
+            secondaryButton: .cancel(Text(Strings.no), action: {
+              onCancelAddAccount()
+            })
+          )
+        }
+    )
+    .background(
+      Color.clear
+        .sheet(
+          isPresented: isShowingAddAccount
+        ) {
+          NavigationView {
+            AddAccountView(
+              keyringStore: keyringStore,
+              networkStore: networkStore,
+              preSelectedCoin: accountNetwork?.coin,
+              preSelectedFilecoinNetwork: accountNetwork
+            )
+          }
+          .navigationViewStyle(.stack)
+          .onDisappear { onAddAccountDismissed() }
+        }
+    )
+  }
 }