Bug 38450 - Delete: Removing default package removes source folder
Summary: Delete: Removing default package removes source folder
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 2.1   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.0 M2   Edit
Assignee: Jerome Lanneluc CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-06-05 04:45 EDT by Martin Aeschlimann CLA
Modified: 2003-07-16 05:39 EDT (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Aeschlimann CLA 2003-06-05 04:45:12 EDT 
20020604

1. Create new Java project, add source folder 'src' with a file 'A.java'
2. In the package explorer select the 'default package' in 'src'
2. press delete

Note that the source folder is removed as well. But looking at the project's
build path you see that the folder is still on the class path (but not existing)

I'd expect that removing the default package does not remove the source folder
(only the package's content)
Comment 1 Adam Kiezun CLA 2003-06-05 09:05:05 EDT 
jcore bug - it deletes the source folder when i call ISourceManipulation:delete 
on the package

i'll disable the action for M1

in the old world we lived with the workaround of deleting the cus ourselves
we think this workaround should be possible to remove

moving to jcore for further action
Comment 2 Jerome Lanneluc CLA 2003-07-07 07:18:48 EDT 
DeleteResourceElementsOperation.deletePackageFragment(...) didn't check for the 
default package case.

Fixed and added regression test DeleteTests.testDeletePackageFragment2()
Comment 3 Adam Kiezun CLA 2003-07-11 08:45:12 EDT 
i don't see it in 0710 - should it be there?
Comment 4 Jerome Lanneluc CLA 2003-07-11 09:04:05 EDT 
See build notes for I20030710:
http://fullmoon.torolab.ibm.com/downloads/drops/I-I20030710-
200307100800/buildnotes/buildnotes_jdt-core.html
Comment 5 David Audel CLA 2003-07-16 05:39:28 EDT 
Verified.