Actions
bug #9593
closedChanging and saving a PolytomousKey throws ConcurrentModificationException
Start date:
Due date:
% Done:
90%
Estimated time:
Severity:
normal
Found in Version:
Actions
Added by Andreas Müller almost 3 years ago. Updated almost 3 years ago.
90%
This only seems to happen if the sortindex of the PolytomousKey nodes is not up-to-date. After calling refresh on the key saving works.
So it is related to #5681.
Before closing this ticket can you please debug which collection is concurrently modified and why (propably it is the PKNode.children collection but I am not sure why.
Andreas Müller wrote:
This only seems to happen if the sortindex of the PolytomousKey nodes is not up-to-date. After calling refresh on the key saving works.
So it is related to #5681.
Before closing this ticket can you please debug which collection is concurrently modified and why (propably it is the PKNode.children collection but I am not sure why.
As the problem disappears after updating the sortindex, the children collection seems to cause the problems.
Added the handling of polytomous key in removeNullFromCollection because the editor saves the whole key.
Probably this is ok, though I wonder why the root node is not doing the same job as it is merge-cascaded. But as the incorrect sortindex numbers appeared from time to time in PKs this might be the reason.
We have to watch this.
Generally this ticket can be closed.
Only one issue: the commit 4ce176cc6b710274 also contains code which seems to belong to another ticket. Can you please attache the commit also to the other ticket (what ever it is)?
Andreas Müller wrote:
Probably this is ok, though I wonder why the root node is not doing the same job as it is merge-cascaded. But as the incorrect sortindex numbers appeared from time to time in PKs this might be the reason.
We have to watch this.Generally this ticket can be closed.
Only one issue: the commit 4ce176cc6b710274 also contains code which seems to belong to another ticket. Can you please attache the commit also to the other ticket (what ever it is)?
I removed it