fix(FE): delete global plugin failed (#1170)

* fix: delete global plugin failed * fix: filter disable plugins
apache · Dec 31, 2020 · 011634f · 011634f
1 parent c0241ee
commit 011634f
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 14 deletions.
diff --git a/web/src/pages/Plugin/List.tsx b/web/src/pages/Plugin/List.tsx
@@ -34,15 +34,17 @@ const Page: React.FC = () => {
   const [name, setName] = useState('');
 
   useEffect(() => {
-    fetchList().then(({ data }) => {
-      const plugins: any = {};
-      // eslint-disable-next-line no-shadow
-      data.forEach(({ name, value }) => {
-        plugins[name] = value;
+    if (!name) {
+      fetchList().then(({ data }) => {
+        const plugins: any = {};
+        // eslint-disable-next-line no-shadow
+        data.forEach(({ name, value }) => {
+          plugins[name] = value;
+        });
+        setInitialData(plugins);
       });
-      setInitialData(plugins);
-    });
-  }, []);
+    }
+  }, [name]);
 
   const columns: ProColumns<PluginModule.TransformedPlugin>[] = [
     {
@@ -72,6 +74,7 @@ const Page: React.FC = () => {
                 const plugins = omit(initialData, [`${record.name}`]);
                 createOrUpdate({ plugins }).then(() => {
                   ref.current?.reload();
+                  setName('');
                 });
               }}
               okText={formatMessage({ id: 'component.global.confirm' })}
@@ -99,13 +102,16 @@ const Page: React.FC = () => {
         setVisible(false);
       }}
       onChange={({ formData, codemirrorData }) => {
+        const disable = !formData.disable;
         createOrUpdate({
           plugins: {
             ...initialData,
-            [name]: { ...codemirrorData, ...formData },
+            [name]: { ...codemirrorData, disable },
           },
         }).then(() => {
           setVisible(false);
+          setName('');
+          ref.current?.reload();
         });
       }}
     />

diff --git a/web/src/pages/Plugin/PluginMarket.tsx b/web/src/pages/Plugin/PluginMarket.tsx
@@ -24,14 +24,18 @@ import { fetchList, createOrUpdate } from './service';
 const PluginMarket: React.FC = () => {
   const [initialData, setInitialData] = useState({});
 
-  useEffect(() => {
+  const initPageData = () => {
     fetchList().then(({ data }) => {
       const plugins: any = {};
       data.forEach(({ name, value }) => {
         plugins[name] = value;
       });
       setInitialData(plugins);
     });
+  }
+
+  useEffect(() => {
+    initPageData();
   }, []);
 
   return (
@@ -48,8 +52,7 @@ const PluginMarket: React.FC = () => {
                 ...pluginsData,
               },
             }).then(() => {
-              // TODO:
-              window.location.reload();
+              initPageData();
             });
           }}
         />

diff --git a/web/src/pages/Plugin/service.ts b/web/src/pages/Plugin/service.ts
@@ -24,10 +24,10 @@ export const fetchList = (): Promise<{
 }> =>
   request<{
     data: {
-      plugins: Record<string, object>;
+      plugins: Record<string, any>;
     };
   }>(`/global_rules/${DEFAULT_GLOBAL_RULE_ID}`).then(({ data }) => {
-    const plugins = Object.entries(data.plugins || {}).map(([name, value]) => ({
+    const plugins = Object.entries(data.plugins || {}).filter(([, value]) => !value.disable).map(([name, value]) => ({
       id: name,
       name,
       value,