From: Johann-S <johann.servoire@gmail.com>
Date: Tue, 28 Mar 2017 22:10:27 +0000 (+0200)
Subject: Collapse - Fix check to not prevent event for input and textarea
X-Git-Tag: v4.0.0-beta~147^2~70
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=fb42d6e0435bb101c0505090055e8034cb101dc4;p=thirdparty%2Fbootstrap.git

Collapse - Fix check to not prevent event for input and textarea
---

diff --git a/js/src/collapse.js b/js/src/collapse.js
index dc2e2a67d7..88428310d3 100644
--- a/js/src/collapse.js
+++ b/js/src/collapse.js
@@ -357,7 +357,7 @@ const Collapse = (($) => {
    */
 
   $(document).on(Event.CLICK_DATA_API, Selector.DATA_TOGGLE, function (event) {
-    if (/input|textarea/i.test(event.target.tagName)) {
+    if (!/input|textarea/i.test(event.target.tagName)) {
       event.preventDefault()
     }
 
diff --git a/js/tests/unit/collapse.js b/js/tests/unit/collapse.js
index 01bc5adb96..e7083f56dd 100644
--- a/js/tests/unit/collapse.js
+++ b/js/tests/unit/collapse.js
@@ -515,13 +515,14 @@ $(function () {
   })
 
   QUnit.test('should not prevent event for input', function (assert) {
-    assert.expect(2)
+    assert.expect(3)
     var done = assert.async()
     var $target = $('<input type="checkbox" data-toggle="collapse" data-target="#collapsediv1" />').appendTo('#qunit-fixture')
 
     $('<div id="collapsediv1"/>')
       .appendTo('#qunit-fixture')
       .on('shown.bs.collapse', function () {
+        assert.ok($(this).hasClass('show'))
         assert.ok($target.attr('aria-expanded') === 'true')
         assert.ok($target.prop('checked'))
         done()