forbid pinning if no persisted state

Author: yann300

apps/remix-ide/src/blockchain/blockchain.tsx

@@ -25,7 +25,7 @@ const profile = {
   name: 'blockchain',
   displayName: 'Blockchain',
   description: 'Blockchain - Logic',
-  methods: ['getCode', 'getTransactionReceipt', 'addProvider', 'removeProvider', 'getCurrentFork', 'getAccounts', 'web3VM', 'web3', 'getProvider', 'getCurrentProvider', 'getCurrentNetworkStatus', 'getAllProviders', 'getPinnedProviders', 'changeExecutionContext'],
+  methods: ['getCode', 'getTransactionReceipt', 'addProvider', 'removeProvider', 'getCurrentFork', 'getAccounts', 'web3VM', 'web3', 'getProvider', 'getCurrentProvider', 'getCurrentNetworkStatus', 'getAllProviders', 'getPinnedProviders', 'changeExecutionContext', 'getProviderObject'],
   version: packageJson.version
 }
 
@@ -569,6 +569,10 @@ export class Blockchain extends Plugin {
     return allProviders[name]
   }
 
+  getProviderObject() {
+    return this.executionContext.getProviderObject()
+  }
+
   getInjectedWeb3Address() {
     return this.executionContext.getSelectedAddress()
   }

libs/remix-ui/run-tab/src/lib/components/universalDappUI.tsx

@@ -129,6 +129,14 @@ export function UniversalDappUI(props: UdappProps) {
   }
 
   const pinContract = async() => {
+    const provider = await props.plugin.call('blockchain', 'getProviderObject')
+    if (!provider.config.statePath && provider.config.isRpcForkedState) {
+      // we can't pin a contract in the following case:
+      // - state is not persisted
+      // - future state is browser stored (e.g it's not just a simple RPC provider)
+      props.plugin.call('notification', 'toast', 'Cannot pin this contract in the current context: state is not persisted. Please fork this provider to start pinning a contract to it.')
+      return
+    }
     const workspace = await props.plugin.call('filePanel', 'getCurrentWorkspace')
     const objToSave = {
       name: props.instance.name,