transfer interface inside jQueryInterface

twbs · Jul 6, 2021 · 3323150 · 3323150
1 parent ef76ac2
commit 3323150
Showing 1 changed file with 14 additions and 18 deletions.
diff --git a/js/src/collapse.js b/js/src/collapse.js
@@ -148,7 +148,7 @@ class Collapse extends BaseComponent {
 
     actives.forEach(elemActive => {
       if (this._element !== elemActive) {
-        Collapse.collapseInterface(elemActive, 'hide')
+        Collapse.getOrCreateInstance(elemActive, { toggle: false }).hide()
       }
 
       if (!activesData) {
@@ -280,26 +280,22 @@ class Collapse extends BaseComponent {
 
   // Static
 
-  static collapseInterface(element, config) {
-    const _config = {}
-    if (typeof config === 'string' && /show|hide/.test(config)) {
-      _config.toggle = false
-    }
-
-    const data = Collapse.getOrCreateInstance(element, _config)
-
-    if (typeof config === 'string') {
-      if (typeof data[config] === 'undefined') {
-        throw new TypeError(`No method named "${config}"`)
+  static jQueryInterface(config) {
+    return this.each(function () {
+      const _config = {}
+      if (typeof config === 'string' && /show|hide/.test(config)) {
+        _config.toggle = false
       }
 
-      data[config]()
-    }
-  }
+      const data = Collapse.getOrCreateInstance(this, _config)
 
-  static jQueryInterface(config) {
-    return this.each(function () {
-      Collapse.collapseInterface(this, config)
+      if (typeof config === 'string') {
+        if (typeof data[config] === 'undefined') {
+          throw new TypeError(`No method named "${config}"`)
+        }
+
+        data[config]()
+      }
     })
   }
 }