diff --git a/libraries/cms/html/bootstrap.php b/libraries/cms/html/bootstrap.php index 83a553258e063..3166580a19357 100644 --- a/libraries/cms/html/bootstrap.php +++ b/libraries/cms/html/bootstrap.php @@ -256,36 +256,12 @@ public static function framework($debug = null) * @return void * * @since 3.0 - * @deprecated 4.0 Unused, JS Not working + * @deprecated 4.0 This method was used by the old renderModal() implementation. + * Since the new implementation it is unneeded and the broken JS it was injecting could create issues + * As a case, please see: https://github.com/joomla/joomla-cms/pull/6918 */ public static function modal($selector = 'modal', $params = array()) { - $sig = md5(serialize(array($selector, $params))); - - if (!isset(static::$loaded[__METHOD__][$sig])) - { - // Include Bootstrap framework - static::framework(); - - // Setup options object - $opt['backdrop'] = isset($params['backdrop']) ? (boolean) $params['backdrop'] : true; - $opt['keyboard'] = isset($params['keyboard']) ? (boolean) $params['keyboard'] : true; - $opt['show'] = isset($params['show']) ? (boolean) $params['show'] : true; - $opt['remote'] = isset($params['remote']) ? $params['remote'] : ''; - - $options = JHtml::getJSObject($opt); - - // Attach the modal to document - JFactory::getDocument()->addScriptDeclaration( - "(function($){ - $('#$selector').modal($options); - })(jQuery);" - ); - - // Set static array - static::$loaded[__METHOD__][$sig] = true; - } - return; }