| Summary: | [dnd][reorg] moving source files excluded from build path [refactoring] | ||
|---|---|---|---|
| Product: | [Eclipse Project] JDT | Reporter: | Tibor Sulyán <sulyan.tibor> |
| Component: | UI | Assignee: | JDT-UI-Inbox <jdt-ui-inbox> |
| Status: | CLOSED WONTFIX | QA Contact: | |
| Severity: | normal | ||
| Priority: | P3 | CC: | dirk_baeumer |
| Version: | 3.1 | ||
| Target Milestone: | --- | ||
| Hardware: | PC | ||
| OS: | Windows 2000 | ||
| Whiteboard: | stalebug | ||
|
Description
Tibor Sulyán
Tobias, what API are we using to move the CU. If this is core API then I would agrue that core keeps the two data structures in sync. If we use native resource API we have to do it. Can you please investigate the failure when moving the file back ? Since the compilation unit to be moved is not on the build path (in package a), the reorg policy uses a MoveResourceChange to move the compilation unit to the other package. The policy factory receives an IFile from DelegatingDropAdapter. This explains why it reappears on the build path after the move. The failure when moving the file back to package a is actually due to MoveResourceElementsOperation which tries to acquire a document for the cu A.java in package a to store the moved back content. If the build path status is correctly set on moves (eg. exclude A.java in both packages from the build path, the core API works as exspected for both moves) We should therefore fix d'n'd to return a compilation unit even if the cu is not on the build path. This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant. -- The automated Eclipse Genie. |