changing updateActivityToTargets to avoid using .includes, which isn'…

…t included in Node 4
GetStream · Jan 23, 2018 · bc51d5f · bc51d5f
1 parent 3a9b9eb
commit bc51d5f
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/src/lib/feed.js b/src/lib/feed.js
@@ -347,9 +347,14 @@ StreamFeed.prototype = {
       if (added_targets && removed_targets) {
           // brute force - iterate through added, check to see if removed contains that element
           for (var i = 0; i < added_targets.length; i++) {
-              if (removed_targets.includes(added_targets[i])) {
-                  throw new Error("Can't have the same feed ID in added_targets and removed_targets.");
+              // would normally use Array.prototype.includes here, but it's not supported in Node.js v4 :(
+              for (var j = 0; j < removed_targets.length; j++) {
+                  if (removed_targets[j] == added_targets[i]) {
+                      throw new Error("Can't have the same feed ID in added_targets and removed_targets.");
+                  }
               }
+              // if (removed_targets.includes(added_targets[i])) {
+              // }
           }
       }