Summary: | ResourceSetSynchronizer not updating ALL related Resources | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | [WebTools] WTP Common Tools | Reporter: | Chuck Bridgham <cbridgha> | ||||||||||
Component: | wst.common | Assignee: | Chuck Bridgham <cbridgha> | ||||||||||
Status: | RESOLVED FIXED | QA Contact: | Konstantin Komissarchik <konstantin> | ||||||||||
Severity: | normal | ||||||||||||
Priority: | P3 | CC: | thatnitind, trungha | ||||||||||
Version: | 3.0 | Keywords: | contributed | ||||||||||
Target Milestone: | 3.0 M6 | Flags: | thatnitind:
review+
|
||||||||||
Hardware: | PC | ||||||||||||
OS: | Windows XP | ||||||||||||
Whiteboard: | |||||||||||||
Attachments: |
|
Description
Chuck Bridgham
2008-03-11 17:57:51 EDT
Fixes touch EMF2DOMSSERenderer as well to capture saves.... Created attachment 92242 [details]
EMF2DOMSSEAdapter patch
Created attachment 92245 [details]
ResourceSetWorkbenchEditSynchronizer patch
Save events now pass the associated EMFResource instance along to be cached in the recently saved cache. This way we can differentiate between saves of each EMFResource instance associated with a certain IFile. The SSEAdapter patch requires the ResourceSetWorkbench... patch applied, as new methods were added. Thanks for the patch, Chuck. What happens to older code that does not call the new method? Created attachment 92728 [details]
For 'org.eclipse.wst.common.emfworkbench.integration'
The patch for ResourceSetWorkbenchEditSynchronizer causes a serious error.
Basically, it unloads all Java related resources whenever there is a resource change in the workspace. This results in the inability to locate and load any java resource (class, java files) by EMF utils classes.
I'm still don't understand why other team (especially the JSP team) can pass the smoke test ?
Anyway, here is the patch to end this nightmare.
Created attachment 92729 [details]
For 'org.eclipse.wst.common.emfworkbench.integration'
this new patch has an additional small optimization that I think will be useful for the synchronizer
Thanks for the patches! I am testing now - *** Bug 222681 has been marked as a duplicate of this bug. *** So the original fix for this bug still works as intended, and now recognizes related changed resources. BUT caused a major regression by unloading all JEM resources causing any scenario depending on the JEM reflectoin api's to fail. After testing, and stepping through the patch - recommend we respin . Carl is doing a build. Found an additional regression in ResourceSetWorkbenchEditSynchronizer.recentlySavedFilesForceRemove() Needed to check for null resource, and remove the file from the list as before... Dropped fix to build mass change to add 'contributed' keyword based on bugzilla query, please correct if that's not accurate (by marking patches as obsolete and removing the 'contributed' keyword. |