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

Bug 370832

Summary: Auto publish will not triggered for structure changes
Product: [WebTools] WTP ServerTools Reporter: Elson Yuen <eyuen7>
Component: wst.serverAssignee: Elson Yuen <eyuen7>
Status: RESOLVED FIXED QA Contact: Elson Yuen <eyuen7>
Severity: normal    
Priority: P3 CC: david_williams, jasonxd
Version: 3.2   
Target Milestone: 3.2.5 P   
Hardware: PC   
OS: Windows XP   
See Also: https://git.eclipse.org/r/109093
Whiteboard:
Bug Depends on: 370766    
Bug Blocks:    
Attachments:
Description Flags
v1.0 none

Description Elson Yuen CLA 2012-02-07 09:58:01 EST
+++ This bug was initially created as a clone of Bug #370766 +++

Build Identifier: 3.2.maintance

I am looking at the code below in the Server.ResouceChangeJob#run().  It appears to me that a publish will not happen after the structure change if there is no module file changed.  A structure change can be recorded in a .XXX file in the project but it doesn't need to be a part of the module.  I think it should be published.


			// run the visitor
			visit(visitor, null);
			
			if (getServerPublishInfo().hasStructureChanged(modules2)) {
				int newState = getServerPublishState() == IServer.PUBLISH_STATE_FULL ? IServer.PUBLISH_STATE_FULL : IServer.PUBLISH_STATE_INCREMENTAL;
				setServerPublishState(newState);
			}
			
			if (!changed[0])
				return Status.OK_STATUS;
			
			if (getServerState() != IServer.STATE_STOPPED && behaviourDelegate != null)
				behaviourDelegate.handleResourceChange();
			
			if (getServerState() == IServer.STATE_STARTED)
				autoPublish(event);
			
			return Status.OK_STATUS;


Reproducible: Always
Comment 1 Elson Yuen CLA 2012-02-07 09:59:23 EST
This defect is to check in changes on bug 370766 to 325P code stream.
Comment 2 Elson Yuen CLA 2012-02-07 10:09:44 EST
Created attachment 210649 [details]
v1.0
Comment 3 Elson Yuen CLA 2012-02-07 10:31:56 EST
Changes released to HEAD
Comment 4 Elson Yuen CLA 2012-02-09 18:20:50 EST
Code released to 325P
Comment 5 Eclipse Genie CLA 2017-10-11 16:37:50 EDT
New Gerrit change created: https://git.eclipse.org/r/109093