Author: max.andersen(a)jboss.com Date: 2007-11-28 18:12:19 -0500 (Wed, 28 Nov 2007) New Revision: 5144 Modified: trunk/as/plugins/org.jboss.ide.eclipse.as.ui/jbossui/org/jboss/ide/eclipse/as/ui/wizards/JBossServerWizardFragment.java Log: fixed NPE caused by isComplete() lying (or to be more correct WTP calls enter() before ui is setup so things go foobar if not validated) Modified: trunk/as/plugins/org.jboss.ide.eclipse.as.ui/jbossui/org/jboss/ide/eclipse/as/ui/wizards/JBossServerWizardFragment.java =================================================================== --- trunk/as/plugins/org.jboss.ide.eclipse.as.ui/jbossui/org/jboss/ide/eclipse/as/ui/wizards/JBossServerWizardFragment.java 2007-11-28 16:21:16 UTC (rev 5143) +++ trunk/as/plugins/org.jboss.ide.eclipse.as.ui/jbossui/org/jboss/ide/eclipse/as/ui/wizards/JBossServerWizardFragment.java 2007-11-28 23:12:19 UTC (rev 5144) @@ -339,7 +339,15 @@ private String getErrorString() { if( findServer(name) != null ) return Messages.swf_NameInUse; + + if(deployVal==null || deployVal.trim().length()==0) { + return "Deploy folder not specified"; + } + if(deployTmpFolderVal==null || deployTmpFolderVal.trim().length()==0) { + return "Temporary deploy folder not specified"; + } + return null; }