Start plugins wizard #29 move plugins saving to beforeOkAction webstorm/138.257
authorVassiliy <vassiliy.kudryashov@jetbrains.com>
Wed, 21 May 2014 19:06:28 +0000 (23:06 +0400)
committerVassiliy <vassiliy.kudryashov@jetbrains.com>
Wed, 21 May 2014 19:07:13 +0000 (23:07 +0400)
platform/platform-impl/src/com/intellij/ide/customize/CustomizeIDEWizardDialog.java
platform/platform-impl/src/com/intellij/ide/customize/CustomizePluginsStepPanel.java

index 9afc0ab9f163f0f450d908810da5b405c0b06159..0d62d30fa51187481de05e7c9f3706ac60c4c700 100644 (file)
@@ -15,7 +15,6 @@
  */
 package com.intellij.ide.customize;
 
-import com.intellij.ide.plugins.PluginManager;
 import com.intellij.ide.startup.StartupActionScriptManager;
 import com.intellij.openapi.application.ApplicationNamesInfo;
 import com.intellij.openapi.ui.DialogWrapper;
@@ -27,7 +26,6 @@ import javax.swing.*;
 import java.awt.*;
 import java.awt.event.ActionEvent;
 import java.awt.event.ActionListener;
-import java.io.IOException;
 import java.util.ArrayList;
 import java.util.List;
 
@@ -160,11 +158,6 @@ public class CustomizeIDEWizardDialog extends DialogWrapper implements ActionLis
     for (AbstractCustomizeWizardStep step : mySteps) {
       if (!step.beforeOkAction()) return;
     }
-    try {
-      PluginManager.saveDisabledPlugins(PluginGroups.getInstance().getDisabledPluginIds(), false);
-    }
-    catch (IOException ignored) {
-    }
     super.doOKAction();
   }
 
index 5fa3bbcbbd25de765917cfa30e38d38fb6e3a645..9bd14b69cb477b72e2f1456e579585aad737c20e 100644 (file)
@@ -15,6 +15,7 @@
  */
 package com.intellij.ide.customize;
 
+import com.intellij.ide.plugins.PluginManager;
 import com.intellij.openapi.application.ApplicationNamesInfo;
 import com.intellij.openapi.ui.DialogWrapper;
 import com.intellij.openapi.ui.VerticalFlowLayout;
@@ -33,6 +34,7 @@ import javax.swing.border.CompoundBorder;
 import java.awt.*;
 import java.awt.event.ActionEvent;
 import java.awt.event.ActionListener;
+import java.io.IOException;
 import java.util.List;
 import java.util.Map;
 
@@ -224,6 +226,16 @@ public class CustomizePluginsStepPanel extends AbstractCustomizeWizardStep imple
     return null;
   }
 
+  @Override
+  public boolean beforeOkAction() {
+    try {
+      PluginManager.saveDisabledPlugins(PluginGroups.getInstance().getDisabledPluginIds(), false);
+    }
+    catch (IOException ignored) {
+    }
+    return true;
+  }
+
   private class IdSetPanel extends JPanel implements LinkListener<String> {
     private JLabel myTitleLabel = new JLabel();
     private JPanel myContentPanel = new JPanel(new GridLayout(0, 3, 5, 5));