From: Johann-S <johann.servoire@gmail.com>
Date: Sun, 29 Oct 2017 22:51:04 +0000 (+0100)
Subject: Throw error about Popper.js only when it's needed because some of our plugins don... 
X-Git-Tag: v4.0.0-beta.3~156^2
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=2232b6b4d140cf8a01314deb7e4779c7fde8ab05;p=thirdparty%2Fbootstrap.git

Throw error about Popper.js only when it's needed because some of our plugins don't use it (#24573)
---

diff --git a/js/src/dropdown.js b/js/src/dropdown.js
index 45d061c93a..a18f0c28ae 100644
--- a/js/src/dropdown.js
+++ b/js/src/dropdown.js
@@ -12,14 +12,6 @@ import Util from './util'
 
 const Dropdown = (($) => {
 
-  /**
-   * Check for Popper dependency
-   * Popper - https://popper.js.org
-   */
-  if (typeof Popper === 'undefined') {
-    throw new Error('Bootstrap dropdown require Popper.js (https://popper.js.org)')
-  }
-
   /**
    * ------------------------------------------------------------------------
    * Constants
@@ -151,6 +143,14 @@ const Dropdown = (($) => {
         return
       }
 
+      /**
+       * Check for Popper dependency
+       * Popper - https://popper.js.org
+       */
+      if (typeof Popper === 'undefined') {
+        throw new Error('Bootstrap dropdown require Popper.js (https://popper.js.org)')
+      }
+
       let element = this._element
       // for dropup with alignment we use the parent as popper container
       if ($(parent).hasClass(ClassName.DROPUP)) {
diff --git a/js/src/tooltip.js b/js/src/tooltip.js
index a3fc93c913..7cefd0be6a 100644
--- a/js/src/tooltip.js
+++ b/js/src/tooltip.js
@@ -12,15 +12,6 @@ import Util from './util'
 
 const Tooltip = (($) => {
 
-  /**
-   * Check for Popper dependency
-   * Popper - https://popper.js.org
-   */
-  if (typeof Popper === 'undefined') {
-    throw new Error('Bootstrap tooltips require Popper.js (https://popper.js.org)')
-  }
-
-
   /**
    * ------------------------------------------------------------------------
    * Constants
@@ -120,6 +111,13 @@ const Tooltip = (($) => {
   class Tooltip {
 
     constructor(element, config) {
+      /**
+       * Check for Popper dependency
+       * Popper - https://popper.js.org
+       */
+      if (typeof Popper === 'undefined') {
+        throw new Error('Bootstrap tooltips require Popper.js (https://popper.js.org)')
+      }
 
       // private
       this._isEnabled     = true