Enable `unicorn/prefer-prototype-methods` rule

diff --git a/.eslintrc.json b/.eslintrc.json
index 9bd404dfa31a99ed325f502aa1c0f5d829469ea1..72b267691cd0212347e6cff5514f0c784f0d0f03 100644 (file)
--- a/.eslintrc.json
+++ b/.eslintrc.json
@@ -60,7 +60,6 @@
     "unicorn/prefer-array-flat": "off",
     "unicorn/prefer-dom-node-dataset": "off",
     "unicorn/prefer-module": "off",
-    "unicorn/prefer-prototype-methods": "off",
     "unicorn/prefer-query-selector": "off",
     "unicorn/prefer-spread": "off",
     "unicorn/prevent-abbreviations": "off"

diff --git a/js/src/util/index.js b/js/src/util/index.js
index 7e9e9b0464aecadddd3a19e3716ecddf89a888e3..b99496301848f8dd2fd436a9674f999c6f237848 100644 (file)
--- a/js/src/util/index.js
+++ b/js/src/util/index.js
@@ -15,7 +15,7 @@ const toType = obj => {
     return `${obj}`
   }
 
-  return {}.toString.call(obj).match(/\s([a-z]+)/i)[1].toLowerCase()
+  return Object.prototype.toString.call(obj).match(/\s([a-z]+)/i)[1].toLowerCase()
 }
 
 /**

diff --git a/js/tests/visual/modal.html b/js/tests/visual/modal.html
index ac0a931af551dc4444415eb89cbbfc9baece7de2..eeb0aa567a768349c9ffa6422d53aee2ee9910c1 100644 (file)
--- a/js/tests/visual/modal.html
+++ b/js/tests/visual/modal.html
@@ -220,12 +220,12 @@
         }
       }
 
-      [].slice.call(document.querySelectorAll('[data-bs-toggle="popover"]'))
+      Array.prototype.slice.call(document.querySelectorAll('[data-bs-toggle="popover"]'))
         .forEach(function (popover) {
           new Popover(popover)
         })
 
-      var tooltipList = [].slice.call(document.querySelectorAll('[data-bs-toggle="tooltip"]'))
+      var tooltipList = Array.prototype.slice.call(document.querySelectorAll('[data-bs-toggle="tooltip"]'))
       tooltipList.forEach(function (tooltip) {
         new Tooltip(tooltip)
       })

diff --git a/js/tests/visual/popover.html b/js/tests/visual/popover.html
index 986ac571e3e8ba1ecd1add9dcc50b60f8a09cb7c..f914f2b3be5f3ea429d2f5ed9d090b9d6b71c97a 100644 (file)
--- a/js/tests/visual/popover.html
+++ b/js/tests/visual/popover.html
@@ -40,7 +40,7 @@
     <script src="../../dist/tooltip.js"></script>
     <script src="../../dist/popover.js"></script>
     <script>
-      [].slice.call(document.querySelectorAll('[data-bs-toggle="popover"]'))
+      Array.prototype.slice.call(document.querySelectorAll('[data-bs-toggle="popover"]'))
         .forEach(function (popover) {
           new Popover(popover)
         })

diff --git a/js/tests/visual/tooltip.html b/js/tests/visual/tooltip.html
index ab604092050baa103b09f9f3ccc9423b89f1467a..29248f9daf68bd9226a5cc24d4838b8ee23869ba 100644 (file)
--- a/js/tests/visual/tooltip.html
+++ b/js/tests/visual/tooltip.html
@@ -92,7 +92,7 @@
         })
       }
 
-      [].slice.call(document.querySelectorAll('[data-bs-toggle="tooltip"]'))
+      Array.prototype.slice.call(document.querySelectorAll('[data-bs-toggle="tooltip"]'))
         .forEach(function (tooltip) {
           new Tooltip(tooltip)
         })

diff --git a/site/.eslintrc.json b/site/.eslintrc.json
index 0660ebbaf68232ea01665a7bc3447539b4fc9f60..656465c39e9808d4e4f65ea1cf5e5a636b0d913f 100644 (file)
--- a/site/.eslintrc.json
+++ b/site/.eslintrc.json
@@ -54,7 +54,6 @@
     "unicorn/prefer-array-flat": "off",
     "unicorn/prefer-dom-node-dataset": "off",
     "unicorn/prefer-module": "off",
-    "unicorn/prefer-prototype-methods": "off",
     "unicorn/prefer-query-selector": "off",
     "unicorn/prevent-abbreviations": "off"
   }

diff --git a/site/content/docs/5.1/components/alerts.md b/site/content/docs/5.1/components/alerts.md
index 6127279f5667f63fd9d3e1525bb7349618c88b27..d54f7b9cee4568593a59a82cbc5664a4b7aee18b 100644 (file)
--- a/site/content/docs/5.1/components/alerts.md
+++ b/site/content/docs/5.1/components/alerts.md
@@ -182,7 +182,7 @@ Initialize elements as alerts
 
 ```js
 var alertList = document.querySelectorAll('.alert')
-var alerts =  [].slice.call(alertList).map(function (element) {
+var alerts = Array.prototype.slice.call(alertList).map(function (element) {
   return new bootstrap.Alert(element)
 })
 ```

diff --git a/site/content/docs/5.1/components/collapse.md b/site/content/docs/5.1/components/collapse.md
index 1e1b58c29324fb8cecd4dc045fdf3caf703245e0..75e4b20a3ff5da446e2e2fd0c0424fe9511aa82e 100644 (file)
--- a/site/content/docs/5.1/components/collapse.md
+++ b/site/content/docs/5.1/components/collapse.md
@@ -133,7 +133,7 @@ To add accordion-like group management to a collapsible area, add the data attri
 Enable manually with:
 
 ```js
-var collapseElementList = [].slice.call(document.querySelectorAll('.collapse'))
+var collapseElementList = Array.prototype.slice.call(document.querySelectorAll('.collapse'))
 var collapseList = collapseElementList.map(function (collapseEl) {
   return new bootstrap.Collapse(collapseEl)
 })

diff --git a/site/content/docs/5.1/components/dropdowns.md b/site/content/docs/5.1/components/dropdowns.md
index 738940fc9595a3c038042b9af9ed82f2750fd9b6..2750ce540ab0e46eb66d5b85d5f983254542a65f 100644 (file)
--- a/site/content/docs/5.1/components/dropdowns.md
+++ b/site/content/docs/5.1/components/dropdowns.md
@@ -1003,7 +1003,7 @@ Add `data-bs-toggle="dropdown"` to a link or button to toggle a dropdown.
 Call the dropdowns via JavaScript:
 
 ```js
-var dropdownElementList = [].slice.call(document.querySelectorAll('.dropdown-toggle'))
+var dropdownElementList = Array.prototype.slice.call(document.querySelectorAll('.dropdown-toggle'))
 var dropdownList = dropdownElementList.map(function (dropdownToggleEl) {
   return new bootstrap.Dropdown(dropdownToggleEl)
 })

diff --git a/site/content/docs/5.1/components/list-group.md b/site/content/docs/5.1/components/list-group.md
index 71e45245c5fd43ec24426be2d568f26aee40933a..af0237a07761cfef3d4847bcddf6e5dee1ee21b4 100644 (file)
--- a/site/content/docs/5.1/components/list-group.md
+++ b/site/content/docs/5.1/components/list-group.md
@@ -397,7 +397,7 @@ You can activate a list group navigation without writing any JavaScript by simpl
 Enable tabbable list item via JavaScript (each list item needs to be activated individually):
 
 ```js
-var triggerTabList = [].slice.call(document.querySelectorAll('#myTab a'))
+var triggerTabList = Array.prototype.slice.call(document.querySelectorAll('#myTab a'))
 triggerTabList.forEach(function (triggerEl) {
   var tabTrigger = new bootstrap.Tab(triggerEl)
 

diff --git a/site/content/docs/5.1/components/navs-tabs.md b/site/content/docs/5.1/components/navs-tabs.md
index 353d03d2ddd76e09fa6d1022662dface68ded217..68e744b1a238cd3deb83efef8a27d647b14d09fe 100644 (file)
--- a/site/content/docs/5.1/components/navs-tabs.md
+++ b/site/content/docs/5.1/components/navs-tabs.md
@@ -525,7 +525,7 @@ You can activate a tab or pill navigation without writing any JavaScript by simp
 Enable tabbable tabs via JavaScript (each tab needs to be activated individually):
 
 ```js
-var triggerTabList = [].slice.call(document.querySelectorAll('#myTab button'))
+var triggerTabList = Array.prototype.slice.call(document.querySelectorAll('#myTab button'))
 triggerTabList.forEach(function (triggerEl) {
   var tabTrigger = new bootstrap.Tab(triggerEl)
 

diff --git a/site/content/docs/5.1/components/offcanvas.md b/site/content/docs/5.1/components/offcanvas.md
index 6be2cd64d4aac72ecf7c2947c026f3dc272e9d0d..a5bf9426fea8be432b278fc2be2f4464b13631ba 100644 (file)
--- a/site/content/docs/5.1/components/offcanvas.md
+++ b/site/content/docs/5.1/components/offcanvas.md
@@ -211,7 +211,7 @@ While both ways to dismiss an offcanvas are supported, keep in mind that dismiss
 Enable manually with:
 
 ```js
-var offcanvasElementList = [].slice.call(document.querySelectorAll('.offcanvas'))
+var offcanvasElementList = Array.prototype.slice.call(document.querySelectorAll('.offcanvas'))
 var offcanvasList = offcanvasElementList.map(function (offcanvasEl) {
   return new bootstrap.Offcanvas(offcanvasEl)
 })

diff --git a/site/content/docs/5.1/components/popovers.md b/site/content/docs/5.1/components/popovers.md
index 4511645cee355a4efb5ff1f75b1a114b6710b6bd..dc1c985d3f1dc3e7f63bfd7666ad54d3a4e91079 100644 (file)
--- a/site/content/docs/5.1/components/popovers.md
+++ b/site/content/docs/5.1/components/popovers.md
@@ -36,7 +36,7 @@ Keep reading to see how popovers work with some examples.
 One way to initialize all popovers on a page would be to select them by their `data-bs-toggle` attribute:
 
 ```js
-var popoverTriggerList = [].slice.call(document.querySelectorAll('[data-bs-toggle="popover"]'))
+var popoverTriggerList = Array.prototype.slice.call(document.querySelectorAll('[data-bs-toggle="popover"]'))
 var popoverList = popoverTriggerList.map(function (popoverTriggerEl) {
   return new bootstrap.Popover(popoverTriggerEl)
 })

diff --git a/site/content/docs/5.1/components/scrollspy.md b/site/content/docs/5.1/components/scrollspy.md
index bcd9fb78a6d27dcc42c8d4091484a5c4c60e36f0..aeabb81857ac1191f2036ccfed1c96cf51a1015c 100644 (file)
--- a/site/content/docs/5.1/components/scrollspy.md
+++ b/site/content/docs/5.1/components/scrollspy.md
@@ -278,7 +278,7 @@ Target elements that are not visible will be ignored and their corresponding nav
 When using scrollspy in conjunction with adding or removing of elements from the DOM, you'll need to call the refresh method like so:
 
 ```js
-var dataSpyList = [].slice.call(document.querySelectorAll('[data-bs-spy="scroll"]'))
+var dataSpyList = Array.prototype.slice.call(document.querySelectorAll('[data-bs-spy="scroll"]'))
 dataSpyList.forEach(function (dataSpyEl) {
   bootstrap.ScrollSpy.getInstance(dataSpyEl)
     .refresh()

diff --git a/site/content/docs/5.1/components/toasts.md b/site/content/docs/5.1/components/toasts.md
index 449f0275ce4658456a4512c498c181be94c2bb42..10093b6bc6df68a86d4aca23183c900d38c95bd9 100644 (file)
--- a/site/content/docs/5.1/components/toasts.md
+++ b/site/content/docs/5.1/components/toasts.md
@@ -335,7 +335,7 @@ While technically it's possible to add focusable/actionable controls (such as ad
 Initialize toasts via JavaScript:
 
 ```js
-var toastElList = [].slice.call(document.querySelectorAll('.toast'))
+var toastElList = Array.prototype.slice.call(document.querySelectorAll('.toast'))
 var toastList = toastElList.map(function (toastEl) {
   return new bootstrap.Toast(toastEl, option)
 })

diff --git a/site/content/docs/5.1/components/tooltips.md b/site/content/docs/5.1/components/tooltips.md
index 58da2076b3ebae36a21e6b758b718dd377b6672c..caa2a2d0c0fc99a5b27ef3a1566a562c989022bd 100644 (file)
--- a/site/content/docs/5.1/components/tooltips.md
+++ b/site/content/docs/5.1/components/tooltips.md
@@ -35,7 +35,7 @@ Got all that? Great, let's see how they work with some examples.
 One way to initialize all tooltips on a page would be to select them by their `data-bs-toggle` attribute:
 
 ```js
-var tooltipTriggerList = [].slice.call(document.querySelectorAll('[data-bs-toggle="tooltip"]'))
+var tooltipTriggerList = Array.prototype.slice.call(document.querySelectorAll('[data-bs-toggle="tooltip"]'))
 var tooltipList = tooltipTriggerList.map(function (tooltipTriggerEl) {
   return new bootstrap.Tooltip(tooltipTriggerEl)
 })

diff --git a/site/content/docs/5.1/examples/sidebars/sidebars.js b/site/content/docs/5.1/examples/sidebars/sidebars.js
index 68384c1633e88970593660b87763b1c937cb12a6..d00bc0db1629dc24cfaaa73da47858bee8619d9b 100644 (file)
--- a/site/content/docs/5.1/examples/sidebars/sidebars.js
+++ b/site/content/docs/5.1/examples/sidebars/sidebars.js
@@ -1,7 +1,7 @@
 /* global bootstrap: false */
 (function () {
   'use strict'
-  var tooltipTriggerList = [].slice.call(document.querySelectorAll('[data-bs-toggle="tooltip"]'))
+  var tooltipTriggerList = Array.prototype.slice.call(document.querySelectorAll('[data-bs-toggle="tooltip"]'))
   tooltipTriggerList.forEach(function (tooltipTriggerEl) {
     new bootstrap.Tooltip(tooltipTriggerEl)
   })