Fix conflict with Revolution Slider

admin/js/modules/SyncCountriesChoice.js

@@ -7,7 +7,7 @@
 import { FieldOnlyCountries } from './Fields/FieldOnlyCountries.js'
 import { FieldPrefferedCountries } from './Fields/FieldPrefferedCountries.js'
 import { FieldExcludeCountries } from './Fields/FieldExcludeCountries.js'
-import { codesISO2European, codesISO2 } from '../utils.js'
+import { codesISO2European, codesISO2, nfspnRemoveValue } from '../utils.js'
 
 const $ = jQuery
 
@@ -77,7 +77,7 @@ export class SyncCountriesChoice {
           FieldOnly.setFieldValue('all', true)
         } else if (currentValue.includes('european')) {
           let onlyCountries = [...codesISO2European]
-          onlyCountries = onlyCountries.concat(FieldOnly.getOptions('selected', 'value').remove('european'))
+          onlyCountries = onlyCountries.concat(nfspnRemoveValue(FieldOnly.getOptions('selected', 'value'), 'european'))
 
           // Handle Field Only
           FieldOnly.disableOptions(codesISO2European, true)

admin/js/ninja-forms-spn-addon-admin.js

@@ -6,11 +6,6 @@
 
 import { SyncCountriesChoice } from './modules/SyncCountriesChoice.js'
 
-/* eslint no-extend-native: ["error", { "exceptions": ["Array"] }] */
-Array.prototype.remove = function (value) {
-  return this.filter(function (element) { return element !== value })
-};
-
 /**
  * SyncCountriesChoice
  */

admin/js/utils.js

@@ -50,4 +50,8 @@ function propSelected ($selector, value) {
   }
 }
 
-export { propDisabled, propSelected, codesISO2, codesISO2European }
+function nfspnRemoveValue (obj, value) {
+  return obj.filter(function (element) { return element !== value })
+}
+
+export { propDisabled, propSelected, nfspnRemoveValue, codesISO2, codesISO2European }

public/js/modules/ITIInitializer.js

@@ -5,7 +5,7 @@
  * @since 1.2.0
  */
 
-import { codesISO2European } from '../utils.js'
+import { codesISO2European, nfspnRemoveValue } from '../utils.js'
 import $ from 'jquery'
 import intlTelInput from 'intl-tel-input'
 import intlTelInputUtils from 'intl-tel-input/build/js/utils.js'
@@ -109,10 +109,10 @@ class IntlTelInputInitializer {
     })
 
     if (onlyCountries.includes('all')) {
-      onlyCountries = onlyCountries.remove('all')
+      onlyCountries = nfspnRemoveValue(onlyCountries, 'all')
       onlyCountries = onlyCountries.concat(codesISO2)
     } else if (onlyCountries.includes('european')) {
-      onlyCountries = onlyCountries.remove('european')
+      onlyCountries = nfspnRemoveValue(onlyCountries, 'european')
       onlyCountries = onlyCountries.concat(codesISO2European)
     }
     return {
@@ -182,9 +182,4 @@ class IntlTelInputInitializer {
  * @returns {Array} New Array without given value
  */
 
-/* eslint no-extend-native: ["error", { "exceptions": ["Array"] }] */
-Array.prototype.remove = function (value) {
-  return this.filter(function (element) { return element !== value })
-}
-
 export { IntlTelInputInitializer }

public/js/utils.js

@@ -12,4 +12,8 @@ const codesISO2European = [
   'si', 'es', 'se', 'ch', 'ua', 'gb'
 ]
 
-export { codesISO2European }
+function nfspnRemoveValue (obj, value) {
+  return obj.filter(function (element) { return element !== value })
+}
+
+export { codesISO2European, nfspnRemoveValue }