LineGraph: Add an existingItemsMap to check if items are new or not b…

…efore skipping
almende · May 19, 2017 · f7d59fc · f7d59fc
1 parent f67d16c
commit f7d59fc
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/lib/timeline/component/LineGraph.js b/lib/timeline/component/LineGraph.js
@@ -453,13 +453,15 @@ LineGraph.prototype._updateAllGroupData = function (ids, groupIds) {
     }
 
     //Pre-load arrays from existing groups if items are not changed (not in ids)
+    var existingItemsMap = {};
     if (!groupIds && ids) {
       for (var groupId in this.groups) {
         if (this.groups.hasOwnProperty(groupId)) {
           var group = this.groups[groupId];
           var existing_items = group.getItems();
 
           groupsContent[groupId] = existing_items.filter(function (item) {
+            existingItemsMap[fieldId] = fieldId;
             return (item[fieldId] !== idMap[item[fieldId]]);
           });
           var newLength = groupCounts[groupId];
@@ -478,7 +480,7 @@ LineGraph.prototype._updateAllGroupData = function (ids, groupIds) {
       if (groupId === null || groupId === undefined) {
         groupId = UNGROUPED;
       }
-      if (!groupIds && ids && (item[fieldId] !== idMap[item[fieldId]])) {
+      if (!groupIds && ids && (item[fieldId] !== idMap[item[fieldId]]) && existingItemsMap.hasOwnProperty(fieldId)) {
         continue;
       }
       if (!groupsContent.hasOwnProperty(groupId)) {