Issue 14471 - Configuration: cannot reinsert removed set elements without intervening commit
Summary: Configuration: cannot reinsert removed set elements without intervening commit
Status: ACCEPTED
Alias: None
Product: utilities
Classification: Unclassified
Component: code (show other issues)
Version: OOo 1.1 Beta2
Hardware: All All
: P3 Trivial (vote)
Target Milestone: OOo Later
Assignee: AOO issues mailing list
QA Contact: issues@tools
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-05-15 14:02 UTC by joerg.barfurth
Modified: 2017-05-20 11:33 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description joerg.barfurth 2003-05-15 14:02:27 UTC 
The following configuration API (pseudo)code:

  aSetnode = (an updatable view onto a set node)
  aElement = aSetnode.getByName(name)
  aSetnode.removeByName(name)
  aSetNode.insertByName(othername)
  aSetnode.commitChanges

does not work. 

In 1.1 Beta2 it causes a crash (internal bug #109342#).
In later (srx645) versions, it causes a RuntimeException.

The reason is, that the view layer does not handle insertion of non-detached
data (and removal to non-detached elements) propely during commit.

Workaround: call commitChanges between the removeByName and the insertByName
operations.
Comment 1 joerg.barfurth 2003-05-15 14:03:56 UTC 
Accepting
Comment 2 joerg.barfurth 2004-11-17 08:16:55 UTC 
As this change is rather complex for limited benefit, this can only be fixed for
OOo 2.0, if there is spare time (this also needs thorough testing).
Comment 3 joerg.barfurth 2005-10-10 08:34:08 UTC 
Handing over:

@sb: This bug indicates a rather deep problem in the handling of set elements in
the 'view' layer (source directory 'treemgr'). This problem is best addressed
with a restructuring of some of these things. Thus a fix will take some time and
careful regression testing. I can assist with that and explain what changes I
had planned when you get around to this.

I also have a Basic testcase [non-public :-( ]
Comment 4 Stephan Bergmann 2005-10-10 14:24:42 UTC 
.
Comment 5 Marcus 2017-05-20 11:33:25 UTC 
Reset assigne to the default "issues@openoffice.apache.org".