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 140227 - persistence do not maintain elements layering
Summary: persistence do not maintain elements layering
Status: VERIFIED FIXED
Alias: None
Product: uml
Classification: Unclassified
Component: General Diagram (show other bugs)
Version: 6.x
Hardware: All All
: P2 blocker (vote)
Assignee: _ jyothivasa
URL:
Keywords:
Depends on:
Blocks: 140195
  Show dependency tree
 
Reported: 2008-07-16 09:28 UTC by Sergey Petrov
Modified: 2008-08-13 22:40 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
original diagram (178.06 KB, image/png)
2008-08-05 19:15 UTC, Yang Su
Details
reopened diagram (179.08 KB, image/png)
2008-08-05 19:17 UTC, Yang Su
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Sergey Petrov 2008-07-16 09:28:46 UTC
see attached sqd created next way:
create group. create chain of invocations in group and outside.
now deside to add all into partition.
create partition, add column. move to back. expand to  cover group and chain.
save. reload diagram.
some invocations go behind the partition. (group may go too in some cases).
in described scenario partition do not own elements and persistence do not maintain layering.
Comment 1 Sergey Petrov 2008-07-16 09:40:20 UTC
it may be filed as an issue, but we also do not own forks(and other elements) which are placed over separator in
partition/composite state, so only proper layering persistence will restore diagram in this case.
expansion/move of partition do not own elements the same as in 6.1 so may not be fixed in 6.5 but layering works in 6.1
so diagrams are loaded correctly.
in 6.5 it also affect importing where containment isn't handled well for cases of not UMLContainerWidgets
Comment 2 Sergey Petrov 2008-07-25 10:51:06 UTC
ok, it may not affect importing more then 6.5 diagrams any more, I resolve most containment on importing now. but it's
still an issue.
Comment 3 _ jyothivasa 2008-08-01 21:20:28 UTC
The problem is not persistence. Move-to-back is not making partition "contain" the group and other nodes. To verify, try
to move partition after you did "move-to-back".. it doesn't move the group.. which means the containment is NOT
established yet for persistence mechanism to save/load it. hence it is treating them as separate nodes.
Comment 4 _ jyothivasa 2008-08-01 21:42:08 UTC
Move-to-back is not designed to contain the elements before it. This is not a bug, and is working as designed. If you
want to contain the elements, move/drag them on top of the container.
Comment 5 Trey Spiva 2008-08-04 18:08:28 UTC
This is the same as 6.1
Comment 6 Sergey Petrov 2008-08-05 08:32:45 UTC
this issue is not about containment problems which most are the same as in 6.1(but not all) but about layers of elements
which changes after reloading.
Comment 7 _ jyothivasa 2008-08-05 17:53:33 UTC
Sergey, I do not understand what you mean by element layering. Can you please describe in detail and attach before/after
snapshots for clarity.
Comment 8 Yang Su 2008-08-05 19:15:57 UTC
Created attachment 66618 [details]
original diagram
Comment 9 Yang Su 2008-08-05 19:17:55 UTC
Created attachment 66619 [details]
reopened diagram
Comment 10 Yang Su 2008-08-05 19:19:31 UTC
Here are the simple repro steps:

1. create a new class diagram
2. create Class "A" on the diagram, followed by class "B", move class B to partially overlap with A with B on top.
3. save and close the diagram (see screen shot "original")
4. reopen the diagram (see screen shot "after_loading")

As you see, now A is on top, the widget layer index was not preserved.
Comment 12 Trey Spiva 2008-08-08 18:14:33 UTC
Reviewed
Comment 13 Quality Engineering 2008-08-12 04:06:32 UTC
Integrated into 'main-golden', available in build *200808120201* on http://bits.netbeans.org/dev/nightly/
Changeset: http://hg.netbeans.org/main/rev/be5bed728f81
User: jyothivasa@netbeans.org
Log: Partial fix for #140227 - Preserve Element layering
Comment 14 Peter Lam 2008-08-13 22:40:33 UTC
verified in build 20080813.