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 131462 - Diagram not saved after reverse eng causes FileLock exception
Summary: Diagram not saved after reverse eng causes FileLock exception
Status: VERIFIED FIXED
Alias: None
Product: uml
Classification: Unclassified
Component: Diagram Class (show other bugs)
Version: 6.x
Hardware: All All
: P2 blocker (vote)
Assignee: _ jyothivasa
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-03-28 20:41 UTC by Joanne Lau
Modified: 2008-06-24 20:57 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
java.lang.Throwable attached (5.02 KB, text/plain)
2008-03-28 20:46 UTC, Joanne Lau
Details
NPE attached (4.07 KB, text/plain)
2008-03-28 20:47 UTC, Joanne Lau
Details
new npe (2.03 KB, text/plain)
2008-04-10 13:35 UTC, Sergey Petrov
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Joanne Lau 2008-03-28 20:41:51 UTC
Build: 080327

1) Create a UML project and a Java application
2) DnD 2 class elements (c1, c2) and an interface (i1) to the class diagram editor
   Add an operation (c1_p1) to class c1
   Add an operation (c2_p1) to class c2
   Add an operation (p1) to interface i1
   
3) Create an implementation link from c1->i1 and c2->i1
4) Create a generalization link from c1->c2,  select c2 in the operation redefined dialog
5) Right click on the UML project -> Generate code
6) In the Projects tree, right click c1 node -> Navigate to source
   Add a parameter "int dd" to c2_p1 method
7) In the source editor, right click -> Reverse Engineer
   Click "Yes to All" in "the Model Element Overwrite Authorization" popup
8) Click the Class diagram tab

   NPE is thrown followed by java.lang.Throwable

   The issue is that it does not automatically save the diagram after reverse engineering.
Comment 1 Joanne Lau 2008-03-28 20:46:50 UTC
Created attachment 59312 [details]
java.lang.Throwable attached
Comment 2 Joanne Lau 2008-03-28 20:47:16 UTC
Created attachment 59313 [details]
NPE attached
Comment 3 Sergey Petrov 2008-03-29 08:01:21 UTC
will look for first part, may be second will go away after fix for first one.
Comment 4 Sergey Petrov 2008-04-10 13:34:09 UTC
can' reproduce throwble, but see NPE on attempt to save diagram after re but not the same as attached.
may be n issue with steps:
step 1-2: was uml project created with suggested class diagram or class diagram was added later and may be scoped to a
package?
step 4-5: was it saved after step4?

if there is no more throable, it looks like persistence-re integration issue (no save on re)
Comment 5 Sergey Petrov 2008-04-10 13:35:41 UTC
Created attachment 59967 [details]
new npe
Comment 6 Sergey Petrov 2008-04-10 14:07:02 UTC
it seems I got "old" npe after restart and attempt2, I alos fixed Throable for now which is caused by incomlete save
(with npe) without closing of opened streams.
Comment 7 _ jyothivasa 2008-04-11 23:56:41 UTC
Fixed the exception in persistence code. But there is still a bug in update-model that has to send appropriate model
change information to the diagram. Assigning it to Trey.
Comment 8 Sergey Petrov 2008-06-19 09:23:26 UTC
what was the main issue, exception or not saved diagram?
is not saved diagram an issue?
I currently don't see any exceptions or can' reproduce the issue, I see another issue, lost redefined block in c1 after
reverse engineering, but it's in my opinion shoud be separate issue.
Comment 9 Sergey Petrov 2008-06-24 19:19:52 UTC
Jyothi fixed main issues some time ago, if any remains should be tracked by separate issues.
Comment 10 Joanne Lau 2008-06-24 20:57:14 UTC
verified in 080623