CVS: check authorization inside doNext() before actually doNext() for default project CVS browsing to work

plugins/cvs/cvs-plugin/src/com/intellij/cvsSupport2/ui/experts/CvsWizard.java

@@ -41,6 +41,17 @@ public class CvsWizard extends AbstractWizard<WizardStep> {
     return null;
   }
 
+  @Override
+  protected void doNextAction() {
+    if ((myCurrentStep + 1) >= mySteps.size()) return;
+    final WizardStep nextStep = mySteps.get(myCurrentStep + 1);
+    if (! nextStep.preNextCheck()) {
+      return;
+    }
+
+    super.doNextAction();
+  }
+
   public void updateStep() {
     super.updateStep();
     if (getNumberOfSteps() == 0) return;

plugins/cvs/cvs-plugin/src/com/intellij/cvsSupport2/ui/experts/SelectCvsElementStep.java

@@ -77,18 +77,24 @@ public class SelectCvsElementStep extends WizardStep {
           errors.set(Boolean.TRUE);
         }
       });
-    final boolean logged = performer.loginAll(
-      new ModalityContextImpl(ModalityState.stateForComponent(mySelectCVSConfigurationStep.getComponent()), false), false);
+    /*final boolean logged = performer.loginAll(
+      new ModalityContextImpl(ModalityState.stateForComponent(mySelectCVSConfigurationStep.getComponent()), false), false);*/
+    final boolean logged = performer.loginAll(new ModalityContextImpl(ModalityState.current(), false), false);
     if ((! logged) || (! errors.isNull())) {
       return false;
     }
     return true;
   }
 
+  @Override
+  public boolean preNextCheck() {
+    CvsRootConfiguration selectedConfiguration = mySelectCVSConfigurationStep.getSelectedConfiguration();
+    return isLogged(selectedConfiguration);
+  }
+
   public boolean setActive() {
     CvsRootConfiguration selectedConfiguration =
       mySelectCVSConfigurationStep.getSelectedConfiguration();
-    if (! isLogged(selectedConfiguration)) return false;
 
     if (myCvsTree == null || !Comparing.equal(myConfiguration, selectedConfiguration)) {
       myConfiguration = selectedConfiguration;

plugins/cvs/cvs-plugin/src/com/intellij/cvsSupport2/ui/experts/WizardStep.java

@@ -53,6 +53,10 @@ public abstract class WizardStep extends StepAdapter{
 
   }
 
+  public boolean preNextCheck() {
+    return true;
+  }
+
   public abstract boolean nextIsEnabled();
   public abstract boolean setActive();
   protected abstract JComponent createComponent();