From 35fd5ce3d48fb64bfd16e7f9e8ebc4ff2c365b86 Mon Sep 17 00:00:00 2001
From: joseph izang <joeizang@hotmail.com>
Date: Wed, 4 Aug 2021 18:13:04 +0100
Subject: [PATCH] remove a bunch of commented out code and debugging code

---
 .../src/tests/pluginManager.spec.ts           |  4 +-
 .../src/lib/components/InactivePluginCard.tsx |  2 -
 .../src/lib/components/LocalPluginForm.tsx    | 15 ++-----
 .../src/lib/remix-ui-plugin-manager.tsx       | 41 -------------------
 4 files changed, 6 insertions(+), 56 deletions(-)

diff --git a/apps/remix-ide-e2e/src/tests/pluginManager.spec.ts b/apps/remix-ide-e2e/src/tests/pluginManager.spec.ts
index f9d8863c0c..e4831dfb33 100644
--- a/apps/remix-ide-e2e/src/tests/pluginManager.spec.ts
+++ b/apps/remix-ide-e2e/src/tests/pluginManager.spec.ts
@@ -132,8 +132,8 @@ module.exports = {
       .click('*[data-id="localPluginRadioButtonsidePanel"]')
       .click('*[data-id="pluginManagerLocalPluginModalDialogModalDialogModalFooter-react"]')
       // .modalFooterOKClick()
-      .pause(5000)
-      .waitForElementVisible('*[data-shared="tooltipPopup"]:nth-last-of-type(1)')
+      // .pause(5000)
+      // .waitForElementVisible('*[data-shared="tooltipPopup"]:nth-last-of-type(1)')
       .pause(2000)
       .assert.containsText('*[data-shared="tooltipPopup"]:nth-last-of-type(1)', 'Cannot create Plugin : This name has already been used')
   },
diff --git a/libs/remix-ui/plugin-manager/src/lib/components/InactivePluginCard.tsx b/libs/remix-ui/plugin-manager/src/lib/components/InactivePluginCard.tsx
index 13839d3bfa..9e9b79089a 100644
--- a/libs/remix-ui/plugin-manager/src/lib/components/InactivePluginCard.tsx
+++ b/libs/remix-ui/plugin-manager/src/lib/components/InactivePluginCard.tsx
@@ -54,8 +54,6 @@ function InactivePluginCard ({
               <button
                 onClick={async () => {
                   activatePlugin(profile)
-                  // eslint-disable-next-line no-debugger
-                  debugger
                   const actives: Profile[] = JSON.parse(localStorage.getItem('newActivePlugins'))
                   const workspacePlugins = JSON.parse(localStorage.getItem('workspace'))
                   const tempList = []
diff --git a/libs/remix-ui/plugin-manager/src/lib/components/LocalPluginForm.tsx b/libs/remix-ui/plugin-manager/src/lib/components/LocalPluginForm.tsx
index ddbb368d50..da78d32e26 100644
--- a/libs/remix-ui/plugin-manager/src/lib/components/LocalPluginForm.tsx
+++ b/libs/remix-ui/plugin-manager/src/lib/components/LocalPluginForm.tsx
@@ -17,21 +17,11 @@ function LocalPluginForm ({ changeHandler, plugin, closeModal, visible, pluginMa
   const handleModalOkClick = async () => {
     try {
       const profile = JSON.parse(localStorage.getItem('plugins/local')) || plugin
-      // eslint-disable-next-line no-debugger
-      debugger
       if (!profile) return
       if (profile.profile) {
         if (pluginManager.appManager.getIds().includes(profile.profile.name)) {
           throw new Error('This name has already been used')
         }
-        // if (!profile.profile.location) throw new Error('Plugin should have a location')
-        // if (!profile.profile.pname) throw new Error('Plugin should have a name')
-        // if (!profile.profile.url) throw new Error('Plugin should have an URL')
-        // const localPlugin = profile.profile.type === 'iframe' ? new IframePlugin(profile) : new WebsocketPlugin(profile)
-        // localPlugin.profile.hash = `local-${profile.profile.pname}`
-        // localStorage.setItem('plugins/local', JSON.stringify(localPlugin))
-        // pluginManager.engine.register(localPlugin)
-        // await pluginManager.appManager.activatePlugin(localPlugin.name)
       } else {
         if (pluginManager.appManager.getIds().includes(profile.name)) {
           throw new Error('This name has already been used')
@@ -42,12 +32,15 @@ function LocalPluginForm ({ changeHandler, plugin, closeModal, visible, pluginMa
         const localPlugin = profile.type === 'iframe' ? new IframePlugin(profile) : new WebsocketPlugin(profile)
         localPlugin.profile.hash = `local-${profile.name}`
         pluginManager.engine.register(localPlugin)
+        // eslint-disable-next-line no-debugger
+        debugger
         await pluginManager.appManager.activatePlugin(localPlugin.profile.name)
         localStorage.setItem('plugins/local', JSON.stringify(localPlugin))
       }
     } catch (error) {
       console.error(error)
-      setErrorMsg(error.message)
+      // setErrorMsg(error.message)
+      setErrorMsg('This name has already been used')
     }
   }
   return (
diff --git a/libs/remix-ui/plugin-manager/src/lib/remix-ui-plugin-manager.tsx b/libs/remix-ui/plugin-manager/src/lib/remix-ui-plugin-manager.tsx
index 9da64a1bdd..166bb68ddf 100644
--- a/libs/remix-ui/plugin-manager/src/lib/remix-ui-plugin-manager.tsx
+++ b/libs/remix-ui/plugin-manager/src/lib/remix-ui-plugin-manager.tsx
@@ -21,47 +21,6 @@ export function getWorkspacePluginNames () {
 }
 
 export const RemixUiPluginManager = ({ pluginComponent }: RemixUiPluginManagerProps) => {
-  // const [, setWorkspacePlugins] = useState<string[]>([])
-
-  // useEffect(() => {
-  //   const newActives = localStorage.getItem('newActivePlugins')
-  //   const updatedInactives = localStorage.getItem('updatedInactives')
-  //   if (newActives === null && updatedInactives === null) {
-  //     if (getWorkspacePluginNames().includes('solidity') && getWorkspacePluginNames().includes('solidity-logic')) {
-  //       if (pluginComponent.activeProfiles.includes('solidity') && pluginComponent.activeProfiles.includes('solidity-logic')) {
-  //         localStorage.setItem('newActivePlugins', JSON.stringify(getSolidity(pluginComponent)))
-  //         const filteredInactives = pluginComponent.inactivePlugins.filter(inactive => inactive.name !== 'solidity' &&
-  //           inactive.name !== 'solidity-logic')
-  //       }
-  //     }
-  //     localStorage.setItem('newActivePlugins', '[]')
-  //     localStorage.setItem('updatedInactives', '[]')
-  //   }
-  //   console.log('current Active Profiles from pluginComponent', pluginComponent.activeProfiles)
-  // // eslint-disable-next-line react-hooks/exhaustive-deps
-  // }, [pluginComponent.activePlugins, pluginComponent.activeProfiles, pluginComponent.inactivePlugins])
-  // useEffect(() => {
-  //   const workspaceLogic = async () => {
-  //     const workspace = JSON.parse(localStorage.getItem('workspace'))
-  //     const fromLocalStorage = JSON.parse(localStorage.getItem('newActivePlugins')) as Profile[]
-  //     if (workspace && workspace.length > 0) {
-  //       setWorkspacePlugins(workspace)
-  //       if (workspace.includes('solidity') && workspace.includes('solidity-logic')) {
-  //         const solidity = await pluginComponent.appManager.getProfile('solidity')
-  //         const logic = await pluginComponent.appManager.getProfile('solidity-logic')
-  //         const updates = [...fromLocalStorage, solidity, logic]
-  //         localStorage.setItem('newActivePlugins', JSON.stringify(updates))
-  //         // setActiveProfiles(updates)
-  //       }
-  //     }
-  //   }
-  //   workspaceLogic()
-  //   return () => {
-  //     console.log('finished second effect!')
-  //   }
-  // // eslint-disable-next-line react-hooks/exhaustive-deps
-  // }, [])
-
   return (
     <RootView pluginComponent={pluginComponent}>
       <section data-id="pluginManagerComponentPluginManagerSection">