Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 391054 - New Server Wizard non-default name fails [regression]
Summary: New Server Wizard non-default name fails [regression]
Status: VERIFIED FIXED
Alias: None
Product: WTP ServerTools
Classification: WebTools
Component: wst.server (show other bugs)
Version: 3.3.2   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 3.3.2 P   Edit
Assignee: Elson Yuen CLA
QA Contact: Elson Yuen CLA
URL:
Whiteboard:
Keywords:
Depends on: 390993
Blocks:
  Show dependency tree
 
Reported: 2012-10-03 14:02 EDT by Elson Yuen CLA
Modified: 2012-10-31 15:21 EDT (History)
2 users (show)

See Also:

Attachments
v1.0 (767 bytes, patch)
2012-10-03 14:06 EDT, Elson Yuen CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Elson Yuen CLA 2012-10-03 14:02:26 EDT 
+++ This bug was initially created as a clone of Bug #390993 +++

This is a regression of bug 376506. 

The code clearly returns (ie does nothing) if the name IS valid, rather than if it isn't. Please check 

./plugins/org.eclipse.wst.server.ui/serverui/org/eclipse/wst/server/ui/internal/wizard/page/NewManualServerComposite.java

	if (validate(selectedServerType)) {
		// Do not set the server name if it is invalid
		return;			
	}
Comment 1 Elson Yuen CLA 2012-10-03 14:06:23 EDT 
Created attachment 221862 [details]
v1.0

Porting changes from bug 390993
Comment 2 Elson Yuen CLA 2012-10-31 15:21:30 EDT 
Changes released to 332P