Fixes #956 (overlapping fragments)

nlplab · Nov 7, 2012 · bada687 · bada687
1 parent d4087c5
commit bada687
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/client/src/annotator_ui.js b/client/src/annotator_ui.js
@@ -1633,14 +1633,15 @@ var AnnotatorUI = (function($, window, undefined) {
 
  var newOffset = [selectedFrom, selectedTo];
  if (reselectedSpan) {
+ var newOffsets = reselectedSpan.offsets.slice(0); // clone
  spanOptions.old_offsets = JSON.stringify(reselectedSpan.offsets);
  if (selectedFragment !== null) {
  if (selectedFragment !== false) {
- reselectedSpan.offsets.splice(selectedFragment, 1);
+ newOffsets.splice(selectedFragment, 1);
  }
- reselectedSpan.offsets.push(newOffset);
- reselectedSpan.offsets.sort(Util.cmpArrayOnFirstElement);
- spanOptions.offsets = reselectedSpan.offsets;
+ newOffsets.push(newOffset);
+ newOffsets.sort(Util.cmpArrayOnFirstElement);
+ spanOptions.offsets = newOffsets;
  } else {
  spanOptions.offsets = [newOffset];
  }