From 44a7aa6adbace02a74f13ba38e94185c28ce2028 Mon Sep 17 00:00:00 2001
From: Thomas Gelf <thomas@gelf.net>
Date: Thu, 12 Jun 2014 17:33:28 +0000
Subject: [PATCH] JS/IE8: deliver legacy jQuery for IE8

This patch makes IcingaWeb deliver a legacy jQuery version for IE8
as it is no longer supported in jQuery 2.x. JS for IE8 will not be
delivered minified to ease troubleshooting on that buggy platform.

fixes #5866
refs #6417
---
 application/layouts/scripts/layout.phtml |  7 +++++++
 library/Icinga/Application/webrouter.php |  5 +++++
 library/Icinga/Web/JavaScript.php        | 12 +++++++++++-
 3 files changed, 23 insertions(+), 1 deletion(-)

diff --git a/application/layouts/scripts/layout.phtml b/application/layouts/scripts/layout.phtml
index 602a0f0c7..71b233610 100644
--- a/application/layouts/scripts/layout.phtml
+++ b/application/layouts/scripts/layout.phtml
@@ -10,6 +10,8 @@ if (array_key_exists('_dev', $_GET)) {
   $cssfile = 'css/icinga.min.css';
 }
 
+$ie8jsfile  = 'js/icinga.ie8.js';
+
 $isIframe = isset($_GET['_render']) && $_GET['_render'] === 'iframe';
 $iframeClass = $isIframe ? ' iframe' : '';
 
@@ -47,7 +49,12 @@ $iframeClass = $isIframe ? ' iframe' : '';
 <div id="layout" class="default-layout">
 <?= $this->render('body.phtml') ?>
 </div>
+<!--[if IE 8]>
+<script type="text/javascript" src="<?= $this->href($ie8jsfile) ?>"></script>
+<![endif]-->
+<!--[if gt IE 8]><!-->
 <script type="text/javascript" src="<?= $this->href($jsfile) ?>"></script>
+<!--<![endif]-->
 <script type="text/javascript">
 var icinga = new Icinga({
   baseUrl: '<?= $this->href('/') ?>'
diff --git a/library/Icinga/Application/webrouter.php b/library/Icinga/Application/webrouter.php
index bee27ee1d..82c630056 100644
--- a/library/Icinga/Application/webrouter.php
+++ b/library/Icinga/Application/webrouter.php
@@ -47,6 +47,7 @@ $special = array(
     'css/icinga.css',
     'css/icinga.min.css',
     'js/icinga.dev.js',
+    'js/icinga.ie8.js',
     'js/icinga.min.js'
 );
 
@@ -72,6 +73,10 @@ if (in_array($path, $special)) {
             JavaScript::sendMinified();
             break;
 
+        case 'js/icinga.ie8.js':
+            JavaScript::sendForIe8();
+            break;
+
         default:
             return false;
     }
diff --git a/library/Icinga/Web/JavaScript.php b/library/Icinga/Web/JavaScript.php
index 5a2739189..14913a826 100644
--- a/library/Icinga/Web/JavaScript.php
+++ b/library/Icinga/Web/JavaScript.php
@@ -22,11 +22,15 @@ class JavaScript
     );
 
     protected static $vendorFiles = array(
-        // 'js/vendor/jquery-1.11.0',
         'js/vendor/jquery-2.1.0',
         'js/vendor/jquery.sparkline'
     );
 
+    protected static $ie8VendorFiles = array(
+        'js/vendor/jquery-1.11.0',
+        'js/vendor/jquery.sparkline'
+    );
+
     public static function listModuleFiles()
     {
         $list = array();
@@ -43,6 +47,12 @@ class JavaScript
         return self::send(true);
     }
 
+    public static function sendForIe8()
+    {
+        self::$vendorFiles = self::$ie8VendorFiles;
+        return self::send();
+    }
+
     public static function send($minified = false)
     {
         header('Content-Type: application/javascript');