This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 63307 - remove a module from suite removes all modules in suite
Summary: remove a module from suite removes all modules in suite
Status: VERIFIED FIXED
Alias: None
Product: apisupport
Classification: Unclassified
Component: Project (show other bugs)
Version: 5.x
Hardware: All All
: P2 blocker (vote)
Assignee: Martin Krauskopf
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-08-30 12:23 UTC by pzajac
Modified: 2005-09-13 10:29 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:

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 pzajac 2005-08-30 12:23:30 UTC 
Steps to reproduce:
1)Create a Module suite
2)Create modules (m1,m2,m3) and place it to module suite 
3)Remove module m2 from module suite

All modules are removed from module suite
Comment 1 Martin Krauskopf 2005-08-30 15:58:03 UTC 
They are just wrongly written into project.properties (without separator - ':').
Not sure who is the culprit.
Comment 2 Martin Krauskopf 2005-08-30 17:01:39 UTC 
Started - I'll write a test and fix it tomorrow.
Comment 3 Martin Krauskopf 2005-08-31 15:05:01 UTC 
For some reason I though that EditableProperties.setProperty(String key,
String[] value) saves also separators. Should read javadoc next time.

Fixed.

Checking in test/TestBase.java; 1.15 --> 1.16
Checking in ui/customizer/SuiteUtils.java; 1.5 --> 1.6
Checking in test/ui/customizer/SuitePropertiesTest.java; 1.7 --> 1.8
Comment 4 pzajac 2005-09-05 09:23:41 UTC 
It sometimes removes two modules:
1) add modules a,b,c,d to site
2) remove c module
....
Comment 5 Martin Krauskopf 2005-09-08 12:13:13 UTC 
It seems to be random. Sometime it also happens to me. I wrote two tests which
pass 100% successfully.
Are you able to reproduce every time?
Comment 6 Martin Krauskopf 2005-09-08 14:39:10 UTC 
Fixed again - other problem (with refreshing). Not radnom. Tested.

Checking in test/project/ui/customizer/SuitePropertiesTest.java; 1.11 --> 1.12
Checking in ui/customizer/SuiteProperties.java; 1.11 --> 1.12
Checking in ui/customizer/SuiteUtils.java; 1.7 --> 1.8
Comment 7 pzajac 2005-09-13 10:29:14 UTC 
v