Bug 376654

Description Elson Yuen 2012-04-12 15:16:38 EDT

+++ This bug was initially created as a clone of Bug #371600 +++

Build Identifier: Version: 3.3.1.v201107072200 Build id: 20100921193530

We are developing a custom Server Adapter that performs custom validation when the New Server wizard is finished.
If the validation fails, CoreException is thrown and the user is supposed to correct the problem. 
However if the Server host name field is the one to be edited, an exception is thrown:

!ENTRY org.eclipse.ui 4 0 2012-02-15 11:45:16.314
!MESSAGE Unhandled event loop exception
!STACK 0
java.util.concurrent.RejectedExecutionException
	at java.util.concurrent.ThreadPoolExecutor$AbortPolicy.rejectedExecution(Unknown Source)
	at java.util.concurrent.ThreadPoolExecutor.reject(Unknown Source)
	at java.util.concurrent.ThreadPoolExecutor.execute(Unknown Source)
	at java.util.concurrent.Executors$DelegatedExecutorService.execute(Unknown Source)
	at org.eclipse.wst.server.ui.internal.Timer.runTimer(Timer.java:46)
	at org.eclipse.wst.server.ui.internal.wizard.page.NewManualServerComposite.setHostnameChangeTimer(NewManualServerComposite.java:835)



The problem seems to be that in NewManualServerComposite#dispose the timer is not set to null and it is reused by setHostnameChangeTimer method

Reproducible: Always

Steps to Reproduce:
1. Make sure that New Server wizard will fail on Finish
2. Go back and try to edit Server host name: field