Merge pull request bootstrap-vue#111 from SirLamer/array-includes-remove

GregPeden · web-flow · commit d1c75acec68d · 2017-02-17T20:35:12.000-07:00
Replace Array.prototype.includes with Array.prototype.indexOf
diff --git a/components/popover.vue b/components/popover.vue
@@ -39,7 +39,7 @@
                 type: String,
                 default: 'top',
                 validator(value) {
-                    return ['top', 'bottom', 'left', 'right'].includes(value);
+                    return ['top', 'bottom', 'left', 'right'].indexOf(value) != -1;
                 }
             },
             triggers: {
@@ -52,11 +52,11 @@
                     if (value === false) {
                         return true;
                     } else if (typeof value === 'string') {
-                        return Object.keys(triggerListeners).includes(value);
+                        return Object.keys(triggerListeners).indexOf(value) != -1;
                     } else if (Array.isArray(value)) {
                         const keys = Object.keys(triggerListeners);
                         value.forEach(item => {
-                            if (!keys.includes(item)) {
+                            if (keys.indexOf(item) == -1) {
                                 return false;
                             }
                         });
@@ -288,14 +288,14 @@
 
                 // Look for new events not yet mapped (all of them on first load)
                 triggers.forEach(item => {
-                    if (!appliedTriggers.includes(item)) {
+                    if (appliedTriggers.indexOf(item) == -1) {
                         newTriggers.push(item);
                     }
                 });
 
                 // Disable any removed event triggers
                 appliedTriggers.forEach(item => {
-                    if (!triggers.includes(item)) {
+                    if (triggers.indexOf(item) == -1) {
                         removeTriggers.push(item);
                     }
                 });
