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: | New > Java Class doesn't open new file in Editor | ||
---|---|---|---|
Product: | java | Reporter: | Adam Sotona <asotona> |
Component: | Unsupported | Assignee: | Jan Pokorsky <jpokorsky> |
Status: | RESOLVED FIXED | ||
Severity: | blocker | CC: | jtulach |
Priority: | P3 | ||
Version: | 4.x | ||
Hardware: | All | ||
OS: | All | ||
Issue Type: | DEFECT | Exception Reporter: |
Description
Adam Sotona
2005-03-23 14:21:09 UTC
IMO the right place to fix this is MultiDataObject.createMultiObject since the same may be the case of others than JDO. To workaround this in the java module I will have to invalidate DO obtained from MDO.handleCreateFromTemplate inside JDO.handleCreateFromTemplate which seems to be really ugly. > To workaround this in the java module I will have to invalidate DO > obtained from MDO.handleCreateFromTemplate Or not call MDO.handleCreateFromTemplate at all. > IMO the right place to fix this is MultiDataObject.createMultiObject It is one of the places where it would be possible to fix this, but also one of the most dangerous as a lot of things could go wrong. In my discussion with Adam I wanted to say two things: 1. the fact that somebody has to subclass JDO is flaw in the design and should be replaced 2. if we have to do it, there is a way fixed in /cvs/java/src/org/netbeans/modules/java/JavaDataObject.java new revision: 1.203; previous revision: 1.202 I wanted to be sure you do not consider this as a defect of MDO. I agree that the design is not good. The java module api would deserve redesign in some future release. It would be great if Adam or someone from the mobility team could list their requirements so they do not need to subclass JDO or possibly other classes (JavaEditor?). - FYI: we also subclass JavaDataObject in Creator, in order to allow the Java editor to be opened as a sub-tab of a page in designer - we should see if there is a better way to do this |