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.
Summary: | Work with the properties table causes data loss | ||
---|---|---|---|
Product: | utilities | Reporter: | Milan Kuchtiak <mkuchtiak> |
Component: | Properties | Assignee: | issues@utilities <issues> |
Status: | CLOSED FIXED | ||
Severity: | blocker | CC: | pzavadsky |
Priority: | P1 | ||
Version: | 3.x | ||
Hardware: | PC | ||
OS: | Windows ME/2000 | ||
Issue Type: | DEFECT | Exception Reporter: |
Description
Milan Kuchtiak
2001-12-03 09:29:47 UTC
fixed in [maintrunk] Rolling back previous changes /cvs/properties/src/org/netbeans/modules/properties/Element.java,v <-- Element.java new revision: 1.25; previous revision: 1.24 /cvs/properties/src/org/netbeans/modules/properties/KeyNode.java,v <-- KeyNode.java new revision: 1.24; previous revision: 1.23 /cvs/properties/src/org/netbeans/modules/properties/PropertiesFileEntry.java,v <-- PropertiesFileEntry.java new revision: 1.38; previous revision: 1.37 /cvs/properties/src/org/netbeans/modules/properties/PropertiesParser.java,v <-- PropertiesParser.java new revision: 1.26; previous revision: 1.25 /cvs/properties/src/org/netbeans/modules/properties/PropertiesStructure.java,v <-- PropertiesStructure.java new revision: 1.31; previous revision: 1.30 fixed in [release33] too. Rolling back previous changes /cvs/properties/src/org/netbeans/modules/properties/Element.java,v <-- Element.java new revision: 1.24.2.1; previous revision: 1.24 /cvs/properties/src/org/netbeans/modules/properties/KeyNode.java,v <-- KeyNode.java new revision: 1.23.2.1; previous revision: 1.23 /cvs/properties/src/org/netbeans/modules/properties/PropertiesFileEntry.java,v <-- PropertiesFileEntry.java new revision: 1.37.2.1; previous revision: 1.37 /cvs/properties/src/org/netbeans/modules/properties/PropertiesParser.java,v <-- PropertiesParser.java new revision: 1.25.2.1; previous revision: 1.25 /cvs/properties/src/org/netbeans/modules/properties/PropertiesStructure.java,v <-- PropertiesStructure.java new revision: 1.30.2.1; previous revision: 1.30 (Build 20011204-1123) -verified, discussed with bug reporter (Milan) fix has been integreated into [release330] /cvs/properties/src/org/netbeans/modules/properties/Element.java,v <-- Element.java new revision: 1.24.4.1; previous revision: 1.24 /cvs/properties/src/org/netbeans/modules/properties/KeyNode.java,v <-- KeyNode.java new revision: 1.23.4.1; previous revision: 1.23 /cvs/properties/src/org/netbeans/modules/properties/PropertiesFileEntry.java,v <-- PropertiesFileEntry.java new revision: 1.37.4.1; previous revision: 1.37 /cvs/properties/src/org/netbeans/modules/properties/PropertiesParser.java,v <-- PropertiesParser.java new revision: 1.25.4.1; previous revision: 1.25 /cvs/properties/src/org/netbeans/modules/properties/PropertiesStructure.java,v <-- PropertiesStructure.java new revision: 1.30.4.1; previous revision: 1.30 The fix of the issue #14634 again caused this problem. I used the previous fix except the changes in PropertiesParser. I believed that the changes in PropertiesParser caused this bug. I have tested the fix and because I didn't reproduced this bug I commited it. Unfortunatelly the problem wasn't only in PropertiesParser. It seems, that it is global problem, mainly caused by changes in PropertiesStructure, but I am not sure. I think the best solution will be roll back it again :-( There is no time to investigate the problem. fixed in [maintrunk] Rolled back the changes made by bugfixing the issue #14634. Peter, could you please review this fix? I would like to request for an integration to release 34. Thanks. DIFFs: PropertiesStructure: -------------------- fix of this issue: http://properties.netbeans.org/source/browse/properties/src/org/netbeans/modules/properties/PropertiesStructure.java.diff?r1=1.34&r2=1.35 previous fix of #14634: http://properties.netbeans.org/source/browse/properties/src/org/netbeans/modules/properties/PropertiesStructure.java.diff?r1=1.31&r2=1.32 KeyNode: -------- fix of this issue: http://properties.netbeans.org/source/browse/properties/src/org/netbeans/modules/properties/KeyNode.java.diff?r1=1.25&r2=1.26 previous fix of #14634: http://properties.netbeans.org/source/browse/properties/src/org/netbeans/modules/properties/KeyNode.java.diff?r1=1.24&r2=1.25 Element: -------- fix of this issue: http://properties.netbeans.org/source/browse/properties/src/org/netbeans/modules/properties/Element.java.diff?r1=1.26&r2=1.27 previous fix of #14634: http://properties.netbeans.org/source/browse/properties/src/org/netbeans/modules/properties/Element.java.diff?r1=1.25&r2=1.26 PropertiesFileEntry: --------------------- fix of this issue: http://properties.netbeans.org/source/browse/properties/src/org/netbeans/modules/properties/PropertiesFileEntry.java.diff?r1=1.39&r2=1.40 previous fix of #14634: http://properties.netbeans.org/source/browse/properties/src/org/netbeans/modules/properties/PropertiesFileEntry.java.diff?r1=1.38&r2=1.39 It's just a rollback, so it should be fine. Verified in #200207190200. It works as expected. Only comment : After change in value field (in editor form), the value is not visible cca 1-2 seconds. This is not so user friendly. integrated into [release34] /cvs/properties/src/org/netbeans/modules/properties/Element.java,v <-- Element.java new revision: 1.26.8.1; previous revision: 1.26 /cvs/properties/src/org/netbeans/modules/properties/KeyNode.java,v <-- KeyNode.java new revision: 1.25.8.1; previous revision: 1.25 /cvs/properties/src/org/netbeans/modules/properties/PropertiesFileEntry.java,v <-- PropertiesFileEntry.java new revision: 1.39.8.1; previous revision: 1.39 /cvs/properties/src/org/netbeans/modules/properties/PropertiesStructure.java,v <-- PropertiesStructure.java new revision: 1.33.8.2; previous revision: 1.33.8.1 Resolved for 3.4.x or earlier, no new info since then -> closing. |