Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 355364

Summary: when hostname is blank the Next button of the wizard must be disabled on server supporting remote
Product: [WebTools] WTP ServerTools Reporter: Petya Sabeva <petya.sabeva>
Component: wst.serverAssignee: Elson Yuen <eyuen7>
Status: RESOLVED FIXED QA Contact: Elson Yuen <eyuen7>
Severity: normal    
Priority: P3 CC: kaloyan, petya.sabeva
Version: 3.3   
Target Milestone: 3.2.5   
Hardware: PC   
OS: Windows Vista   
See Also: https://git.eclipse.org/r/109068
Whiteboard:
Attachments:
Description Flags
Disable Nect button when empty server hostname eyuen7: iplog+

Description Petya Sabeva CLA 2011-08-22 07:09:31 EDT 
Steps to reproduce:
1. Select IBM Websphere server
2. Delete the value in host field

The Next button of the wizard is enabled, although an error message is shown on the wizard...
Comment 1 Petya Sabeva CLA 2011-08-22 07:10:23 EDT 
Created attachment 201907 [details]
Disable Nect button when empty server hostname
Comment 2 Petya Sabeva CLA 2011-08-23 10:49:28 EDT 
This bug comes after fixing bug:

https://bugs.eclipse.org/bugs/show_bug.cgi?id=354330

The reason for that is because before the fix there was only a check that says, if the selected server type doesn't support remote hosts, then the NewServerWizardFragment page is not compete and respectivly the wizard cannot be switched to the next page. But now it must be explicitly checked if the server hostname is an empty string.
Comment 3 Elson Yuen CLA 2011-08-23 17:03:04 EDT 
I cannot reproduce the described problem using IBM WebSphere Server since IBM WebSphere Server does not support remote server.  Having said that, I can reproduce the problem by using the HTTP server which supports remote.

Changes looks good on the patch.  Thanks for submitting.  Targeting to 3.2.5 since the problem can also be reproduced on the 3.2.5 code stream.
Comment 4 Elson Yuen CLA 2011-08-23 17:08:46 EDT 
Code released to 32M, 33M and HEAD
Comment 5 Eclipse Genie CLA 2017-10-11 16:37:09 EDT 
New Gerrit change created: https://git.eclipse.org/r/109068