From ff96f9a7bf3b8f0ee57ab7bb8ed954ad8db93b9e Mon Sep 17 00:00:00 2001
From: Jonathan Barda <jonathan.barda@gmail.com>
Date: Thu, 26 Nov 2015 13:58:54 +0100
Subject: [PATCH 1/3] Fixed missing HTML header

- Added missing HTML Doctype
- Added missing HTML header
---
 index.html | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/index.html b/index.html
index bf317f4..6e52552 100644
--- a/index.html
+++ b/index.html
@@ -1,3 +1,5 @@
+<!DOCTYPE html>
+<html lang="en">
 <head>
 	<script src="js/ext/jquery.min.js"></script>
 	<script src="js/ext/jquery-ui.min.js"></script>
@@ -421,4 +423,4 @@ <h4 class="modal-title" id="Settings">Settings</h4>
 
 }
 </script>
-
+</html>

From 8d726ec84ae50d663afb5d12aa26c7059b461160 Mon Sep 17 00:00:00 2001
From: Jonathan Barda <jonathan.barda@gmail.com>
Date: Thu, 26 Nov 2015 14:06:13 +0100
Subject: [PATCH 2/3] Fixed character encoding

- Added missing character charset meta tag
---
 index.html | 1 +
 1 file changed, 1 insertion(+)

diff --git a/index.html b/index.html
index 6e52552..361dd24 100644
--- a/index.html
+++ b/index.html
@@ -1,6 +1,7 @@
 <!DOCTYPE html>
 <html lang="en">
 <head>
+	<meta charset="utf-8">
 	<script src="js/ext/jquery.min.js"></script>
 	<script src="js/ext/jquery-ui.min.js"></script>
 	<link rel="stylesheet" href="css/ext/jquery-ui.min.css" type="text/css" media="all" />

From f47a631a4f869fb13c4ba0aabc107219f566fa76 Mon Sep 17 00:00:00 2001
From: Jonathan Barda <jonathan.barda@gmail.com>
Date: Thu, 26 Nov 2015 14:27:55 +0100
Subject: [PATCH 3/3] Last little fixes

- Added missing </div> tag
- Moved in the right place the <script></script> content
---
 index.html | 546 ++++++++++++++++++++++++++---------------------------
 1 file changed, 272 insertions(+), 274 deletions(-)

diff --git a/index.html b/index.html
index 361dd24..92c5e65 100644
--- a/index.html
+++ b/index.html
@@ -107,6 +107,7 @@
 					</span>
 				</div>
 			</div>
+		</div>
 			
 		<!-- Populated by API call -->
 		<div id="hosts-list" class="row"></div>
@@ -137,291 +138,288 @@ <h4 class="modal-title" id="Settings">Settings</h4>
 				</div>
 			</div>
 		</div>
-</body>
-
-
-
-
-<script>
-var zabbix;
-$(document).ready(function() {
-
-	$('#view-select-grid').click(function() {
-		$('.view-grid-cols').attr('disabled', false);
-	});
-	$('#view-select-list').click(function() {
-		$('.view-grid-cols').attr('checked', false);
-		$('.view-grid-cols').attr('disabled', true);
-	});
-	$('.settings').click(function() {
-		filterHostsAndOutput(hosts); //Any settings change should trigger a redraw.
+	</div>
+	<script>
+	var zabbix;
+	$(document).ready(function() {
+	
+		$('#view-select-grid').click(function() {
+			$('.view-grid-cols').attr('disabled', false);
+		});
+		$('#view-select-list').click(function() {
+			$('.view-grid-cols').attr('checked', false);
+			$('.view-grid-cols').attr('disabled', true);
+		});
+		$('.settings').click(function() {
+			filterHostsAndOutput(hosts); //Any settings change should trigger a redraw.
+		});
+		
+		zabbix = new $.zabbix ('http://zabbix.org/zabbix/api_jsonrpc.php', 'guest', ''); //Change zabbix.org to your server's IP, guest to a Zabbix user and the blank to the password
+		if(zabbix.getApiVersion() >= '2.5.1') {
+			throw new Error('Zabbix has been updated, check the new documentation then update the code and Zabbix api version');
+			//API changes can be problematic. The current code seems to work well with 2.2, 2.4 and 2.5 (unreleased, used by zabbix.org/zabbix) version
+		}
+		zabbix.authenticate(); //this gives us our API key, which gets assigned to the zabbix object
+	   var hosts = getHostsFromZabbix(zabbix);
+	   filterHostsAndOutput(hosts);
+	   
+	   //TODO fix this. If the next API call starts before the last one finishes, the dashboard explodes and it's not pretty.
+	   setInterval(function() { var hosts = getHostsFromZabbix(zabbix); filterHostsAndOutput(hosts); }, 54000);
+	
 	});
 	
-	zabbix = new $.zabbix ('http://zabbix.org/zabbix/api_jsonrpc.php', 'guest', ''); //Change zabbix.org to your server's IP, guest to a Zabbix user and the blank to the password
-	if(zabbix.getApiVersion() >= '2.5.1') {
-		throw new Error('Zabbix has been updated, check the new documentation then update the code and Zabbix api version');
-		//API changes can be problematic. The current code seems to work well with 2.2, 2.4 and 2.5 (unreleased, used by zabbix.org/zabbix) version
+	
+	
+	function getHostsFromZabbix(zabbix) {
+		return zabbix.call('host.get', {
+			"selectInventory": true,
+			"selectItems": [
+				"name",
+				"lastvalue",
+				"units",
+				"itemid",
+				"key_",
+				"lastclock",
+				"value_type",
+				"status",
+				"itemid"
+			],//Next line is thanks to eburgueno, disabled unless you want to use it
+	//		"groupids": "11", //Only display hosts in this groupID #.  You should create a new one with all the hosts you want, and templates
+			"output": "extend",
+			"search" : {"host" : ""},
+		"sortfield" : "host",
+			"expandDescription": 1,
+			"expandData": 1
+		});
 	}
-	zabbix.authenticate(); //this gives us our API key, which gets assigned to the zabbix object
-   var hosts = getHostsFromZabbix(zabbix);
-   filterHostsAndOutput(hosts);
-   
-   //TODO fix this. If the next API call starts before the last one finishes, the dashboard explodes and it's not pretty.
-   setInterval(function() { var hosts = getHostsFromZabbix(zabbix); filterHostsAndOutput(hosts); }, 54000);
-
-});
-
-
-
-function getHostsFromZabbix(zabbix) {
-	return zabbix.call('host.get', {
-		"selectInventory": true,
-		"selectItems": [
-			"name",
-			"lastvalue",
-			"units",
-			"itemid",
-			"key_",
-			"lastclock",
-			"value_type",
-			"status",
-			"itemid"
-		],//Next line is thanks to eburgueno, disabled unless you want to use it
-//		"groupids": "11", //Only display hosts in this groupID #.  You should create a new one with all the hosts you want, and templates
-		"output": "extend",
-		"search" : {"host" : ""},
-	"sortfield" : "host",
-		"expandDescription": 1,
-		"expandData": 1
-	});
-}
-function Disk(Name, pctFree, used, total) {
-	this.Name = Name;
-	this.pctFree = pctFree;
-	this.used = used;
-	this.total = total;
-}
-function findIndexOfObjectWithAttributeInArray(array, attr, value) {
-    for(var i = 0; i < array.length; i += 1) {
-        if(array[i][attr] === value) {
-            return i;
-        }
-    }
-    return false;
-}
-//This process takes the hosts retrieved from the API call and creates the view on the webpage.
-function filterHostsAndOutput(hosts) {
-	var anError = 0;
-	var hostsCount = 0;
-	$('#hosts-list').empty();
-	$.each(hosts, function() { 
-		var thisHost = $(this); 
-		var thisHostObj = thisHost[0];
-		var processorCount = 1; //default to one CPU (or thread) in case system.cpu.num is not being gathered for the host
-		var loadTotals = 0;
-		var name_index = false;
-		thisHostObj.agentPing = '';
-		thisHostObj.totalMem = 0;
-		thisHostObj.freeMem = 0;
-		thisHostObj.numProcesses = 0;
-		thisHostObj.networkTraffic = 0;
-		thisHostObj.Disks = [];
-		$.each(thisHostObj.items, function() { //Parse the statistics to human-readable.
-			switch ($(this)[0].key_) {
-				case 'system.cpu.num': //You may have to add this to your Zabbix templates.
-					processorCount = $(this)[0].lastvalue;
-					break;
-				case 'system.cpu.load': //You may have to add this to your Zabbix templates (as float)
-					loadTotals += $(this)[0].lastvalue;
-					break;
-				case 'agent.ping':
-					thisHostObj.agentPing = $(this)[0].lastvalue;
-					break;
-				case 'vm.memory.size[total]': //You may have to add these as well
-				   thisHostObj.totalMem = $(this)[0].lastvalue / 1073741824; //1024^3 => GB
-					break;
-				case 'vm.memory.size[free]':
-					thisHostObj.freeMem += $(this)[0].lastvalue / 1073741824; //free(avail)=free+cached+inactive on linux, free on Windows
-					break;
-				case 'vm.memory.size[cached]':
-					thisHostObj.freeMem += $(this)[0].lastvalue / 1073741824;
-					break;
-				case 'vm.memory.size[inactive]':
-					thisHostObj.freeMem += $(this)[0].lastvalue / 1073741824;
-					break;
-				case 'proc.num[]':
-					thisHostObj.numProcesses = $(this)[0].lastvalue;
-					break;
-				//as seen in http://stackoverflow.com/a/22099079 for case substring matching. There may be a better way to do this...
-				case ( ($(this)[0].key_.match(/vfs.fs.size/) )? $(this)[0].key_: undefined ):
-					diskname = $(this)[0].key_.substring(12,$(this)[0].key_.indexOf(",")); //12 is where the name of the disk starts
-					name_index = findIndexOfObjectWithAttributeInArray(thisHostObj.Disks, 'Name', diskname);
-					if (name_index === false) {
-						thisHostObj.Disks.push(new Disk(diskname));
+	function Disk(Name, pctFree, used, total) {
+		this.Name = Name;
+		this.pctFree = pctFree;
+		this.used = used;
+		this.total = total;
+	}
+	function findIndexOfObjectWithAttributeInArray(array, attr, value) {
+	    for(var i = 0; i < array.length; i += 1) {
+	        if(array[i][attr] === value) {
+	            return i;
+	        }
+	    }
+	    return false;
+	}
+	//This process takes the hosts retrieved from the API call and creates the view on the webpage.
+	function filterHostsAndOutput(hosts) {
+		var anError = 0;
+		var hostsCount = 0;
+		$('#hosts-list').empty();
+		$.each(hosts, function() { 
+			var thisHost = $(this); 
+			var thisHostObj = thisHost[0];
+			var processorCount = 1; //default to one CPU (or thread) in case system.cpu.num is not being gathered for the host
+			var loadTotals = 0;
+			var name_index = false;
+			thisHostObj.agentPing = '';
+			thisHostObj.totalMem = 0;
+			thisHostObj.freeMem = 0;
+			thisHostObj.numProcesses = 0;
+			thisHostObj.networkTraffic = 0;
+			thisHostObj.Disks = [];
+			$.each(thisHostObj.items, function() { //Parse the statistics to human-readable.
+				switch ($(this)[0].key_) {
+					case 'system.cpu.num': //You may have to add this to your Zabbix templates.
+						processorCount = $(this)[0].lastvalue;
+						break;
+					case 'system.cpu.load': //You may have to add this to your Zabbix templates (as float)
+						loadTotals += $(this)[0].lastvalue;
+						break;
+					case 'agent.ping':
+						thisHostObj.agentPing = $(this)[0].lastvalue;
+						break;
+					case 'vm.memory.size[total]': //You may have to add these as well
+					   thisHostObj.totalMem = $(this)[0].lastvalue / 1073741824; //1024^3 => GB
+						break;
+					case 'vm.memory.size[free]':
+						thisHostObj.freeMem += $(this)[0].lastvalue / 1073741824; //free(avail)=free+cached+inactive on linux, free on Windows
+						break;
+					case 'vm.memory.size[cached]':
+						thisHostObj.freeMem += $(this)[0].lastvalue / 1073741824;
+						break;
+					case 'vm.memory.size[inactive]':
+						thisHostObj.freeMem += $(this)[0].lastvalue / 1073741824;
+						break;
+					case 'proc.num[]':
+						thisHostObj.numProcesses = $(this)[0].lastvalue;
+						break;
+					//as seen in http://stackoverflow.com/a/22099079 for case substring matching. There may be a better way to do this...
+					case ( ($(this)[0].key_.match(/vfs.fs.size/) )? $(this)[0].key_: undefined ):
+						diskname = $(this)[0].key_.substring(12,$(this)[0].key_.indexOf(",")); //12 is where the name of the disk starts
 						name_index = findIndexOfObjectWithAttributeInArray(thisHostObj.Disks, 'Name', diskname);
-					}
-					
-					switch ($(this)[0].key_) {
-						case ( ($(this)[0].key_.match(/,pfree/) )? $(this)[0].key_: undefined ):
-							thisHostObj.Disks[name_index].pctFree = 100 - $(this)[0].lastvalue;
-							break;
-						case ( ($(this)[0].key_.match(/,used/) )? $(this)[0].key_: undefined ):
-							thisHostObj.Disks[name_index].used = $(this)[0].lastvalue / 1073741824;
-							break;
-						case ( ($(this)[0].key_.match(/,total/) )? $(this)[0].key_: undefined ):
-							thisHostObj.Disks[name_index].total = $(this)[0].lastvalue / 1073741824;
-							break;
-					   }
-					break;
-					
-				case ( ($(this)[0].key_.match(/net.if.in/) || $(this)[0].key_.match(/net.if.out/) ) ? $(this)[0].key_ : undefined ): //FIXME: filter by network interface
-					thisHostObj.networkTraffic += $(this)[0].lastvalue / 8192; // KBps
-					break;
-				case 'net.tcp.service[http]':
-					//Shame we need a second API call, but using 'selectTriggers' in the host.get method doesn't return anything we can use to check the status of the trigger
-					//See https://www.zabbix.com/forum/showthread.php?t=24137
-					var webtrigger = zabbix.call('trigger.get', {
-						"host": thisHostObj.name,
-						"itemids": $(this)[0].itemid,
-						});
-					if(webtrigger[0] && webtrigger[0].status == 0 && webtrigger[0].value == 1) {
-						thisHostObj.error = 1;
-						thisHostObj.errorText = 'Web service is down (httpd or IIS)';
-					}
-					break;
+						if (name_index === false) {
+							thisHostObj.Disks.push(new Disk(diskname));
+							name_index = findIndexOfObjectWithAttributeInArray(thisHostObj.Disks, 'Name', diskname);
+						}
+						
+						switch ($(this)[0].key_) {
+							case ( ($(this)[0].key_.match(/,pfree/) )? $(this)[0].key_: undefined ):
+								thisHostObj.Disks[name_index].pctFree = 100 - $(this)[0].lastvalue;
+								break;
+							case ( ($(this)[0].key_.match(/,used/) )? $(this)[0].key_: undefined ):
+								thisHostObj.Disks[name_index].used = $(this)[0].lastvalue / 1073741824;
+								break;
+							case ( ($(this)[0].key_.match(/,total/) )? $(this)[0].key_: undefined ):
+								thisHostObj.Disks[name_index].total = $(this)[0].lastvalue / 1073741824;
+								break;
+						   }
+						break;
+						
+					case ( ($(this)[0].key_.match(/net.if.in/) || $(this)[0].key_.match(/net.if.out/) ) ? $(this)[0].key_ : undefined ): //FIXME: filter by network interface
+						thisHostObj.networkTraffic += $(this)[0].lastvalue / 8192; // KBps
+						break;
+					case 'net.tcp.service[http]':
+						//Shame we need a second API call, but using 'selectTriggers' in the host.get method doesn't return anything we can use to check the status of the trigger
+						//See https://www.zabbix.com/forum/showthread.php?t=24137
+						var webtrigger = zabbix.call('trigger.get', {
+							"host": thisHostObj.name,
+							"itemids": $(this)[0].itemid,
+							});
+						if(webtrigger[0] && webtrigger[0].status == 0 && webtrigger[0].value == 1) {
+							thisHostObj.error = 1;
+							thisHostObj.errorText = 'Web service is down (httpd or IIS)';
+						}
+						break;
+				}
+			});
+	
+			if (thisHostObj.error != '' && thisHostObj.error != 1) {
+				thisHostObj.errorText = new String;
+				thisHostObj.errorText = thisHostObj.error;
 			}
-		});
-
-		if (thisHostObj.error != '' && thisHostObj.error != 1) {
-			thisHostObj.errorText = new String;
-			thisHostObj.errorText = thisHostObj.error;
-		}
-		if(thisHostObj.errorText && thisHostObj.errorText.indexOf('Interrupted system call') > -1) {
-			thisHostObj.errorText = 'Couldn\'t reach agent - check firewall settings';
-		}
-		if(thisHostObj.errorText && thisHostObj.errorText.indexOf('Connection refused') > -1) {
-			thisHostObj.errorText = 'Connection refused - check agent settings';
-		}
-		if(thisHostObj.errorText && thisHostObj.errorText.indexOf('timed out') > -1) {
-			thisHostObj.errorText = 'Connection timed out - SERVER MAY BE DOWN';
-			anError = 1;
-		}
-		if(thisHostObj.inventory.os) {
-			if(thisHostObj.inventory.os.indexOf('Windows Server 2008 R2') > -1) {
-				thisHostObj.os = 'Win Server 2008 R2';
-			} else if (thisHostObj.inventory.os.indexOf('Windows Server 2012') > -1) {
-				thisHostObj.os = 'Win Server 2012';
-			} else if (thisHostObj.inventory.os.indexOf('Windows Server 2003') > -1) {
-				thisHostObj.os = 'Win Server 2003';
-			} else if (thisHostObj.inventory.os.indexOf('el7') > -1) {
-				thisHostObj.os = 'CentOS 7';
-			} else if (thisHostObj.inventory.os.indexOf('el6') > -1) {
-				thisHostObj.os = 'CentOS 6.5';
-			} else if (thisHostObj.inventory.os.indexOf('Ubuntu') > -1) {//Ubuntu's uname -a does not give us the OS version
-		thisHostObj.os = 'Ubuntu';
-			} else if (thisHostObj.inventory.os.indexOf('Version 10') > -1) { //Does this conflict with any other OS?
-		thisHostObj.os = 'Mac OS 10.6';
-			} else if (thisHostObj.inventory.os.indexOf('Version 11') > -1) {
-				thisHostObj.os = 'Mac OS 10.7';
-			} else if (thisHostObj.inventory.os.indexOf('Version 12') > -1) {
-		thisHostObj.os = 'Mac OS 10.8';
-			} else if (thisHostObj.inventory.os.indexOf('Version 13') > -1) {
-		thisHostObj.os = 'Mac OS 10.9';
-			} else if (thisHostObj.inventory.os.indexOf('Version 14') > -1) {
-		thisHostObj.os = 'Mac OS 10.10';
-			} else if (thisHostObj.inventory.os.indexOf('Version 15') > -1) {
-		thisHostObj.os = 'Mac OS 10.11'; //Even though it doesn't exist yet....
-				//Feel free to roll your own OS stuff here as well! Don't feel restrained by my life choices!
-			} else if (thisHostObj.inventory.os.indexOf('Linux') > -1) {
+			if(thisHostObj.errorText && thisHostObj.errorText.indexOf('Interrupted system call') > -1) {
+				thisHostObj.errorText = 'Couldn\'t reach agent - check firewall settings';
+			}
+			if(thisHostObj.errorText && thisHostObj.errorText.indexOf('Connection refused') > -1) {
+				thisHostObj.errorText = 'Connection refused - check agent settings';
+			}
+			if(thisHostObj.errorText && thisHostObj.errorText.indexOf('timed out') > -1) {
+				thisHostObj.errorText = 'Connection timed out - SERVER MAY BE DOWN';
+				anError = 1;
+			}
+			if(thisHostObj.inventory.os) {
+				if(thisHostObj.inventory.os.indexOf('Windows Server 2008 R2') > -1) {
+					thisHostObj.os = 'Win Server 2008 R2';
+				} else if (thisHostObj.inventory.os.indexOf('Windows Server 2012') > -1) {
+					thisHostObj.os = 'Win Server 2012';
+				} else if (thisHostObj.inventory.os.indexOf('Windows Server 2003') > -1) {
+					thisHostObj.os = 'Win Server 2003';
+				} else if (thisHostObj.inventory.os.indexOf('el7') > -1) {
+					thisHostObj.os = 'CentOS 7';
+				} else if (thisHostObj.inventory.os.indexOf('el6') > -1) {
+					thisHostObj.os = 'CentOS 6.5';
+				} else if (thisHostObj.inventory.os.indexOf('Ubuntu') > -1) {//Ubuntu's uname -a does not give us the OS version
+			thisHostObj.os = 'Ubuntu';
+				} else if (thisHostObj.inventory.os.indexOf('Version 10') > -1) { //Does this conflict with any other OS?
+			thisHostObj.os = 'Mac OS 10.6';
+				} else if (thisHostObj.inventory.os.indexOf('Version 11') > -1) {
+					thisHostObj.os = 'Mac OS 10.7';
+				} else if (thisHostObj.inventory.os.indexOf('Version 12') > -1) {
+			thisHostObj.os = 'Mac OS 10.8';
+				} else if (thisHostObj.inventory.os.indexOf('Version 13') > -1) {
+			thisHostObj.os = 'Mac OS 10.9';
+				} else if (thisHostObj.inventory.os.indexOf('Version 14') > -1) {
+			thisHostObj.os = 'Mac OS 10.10';
+				} else if (thisHostObj.inventory.os.indexOf('Version 15') > -1) {
+			thisHostObj.os = 'Mac OS 10.11'; //Even though it doesn't exist yet....
+					//Feel free to roll your own OS stuff here as well! Don't feel restrained by my life choices!
+				} else if (thisHostObj.inventory.os.indexOf('Linux') > -1) {
+					thisHostObj.os = 'Linux';
+				} else {
+					//Return the unfriendly string if no patterns were matched
+					thisHostObj.os = thisHost[0].inventory.os;
+				}
+			} else if (thisHostObj.inventory.os_full) { //Per eburgueno, his install uses an item for Hostname to populate OS (Full Details) in Inventory
+					if(thisHostObj.inventory.os_full.indexOf('Linux') > -1) {
 				thisHostObj.os = 'Linux';
+			}
 			} else {
-				//Return the unfriendly string if no patterns were matched
-				thisHostObj.os = thisHost[0].inventory.os;
+				thisHostObj.os = 'OS Unavailable';
 			}
-		} else if (thisHostObj.inventory.os_full) { //Per eburgueno, his install uses an item for Hostname to populate OS (Full Details) in Inventory
-				if(thisHostObj.inventory.os_full.indexOf('Linux') > -1) {
-			thisHostObj.os = 'Linux';
-		}
-		} else {
-			thisHostObj.os = 'OS Unavailable';
-		}
-		//Math
-		thisHostObj.usedMem = thisHostObj.totalMem - thisHostObj.freeMem;
-		thisHostObj.avgCpuLoad = loadTotals / processorCount;
-		thisHostObj.memPct = thisHostObj.usedMem / thisHostObj.totalMem;
-		var disks = new String;
-		var diskTopUsage = 0;
-		$.each(thisHostObj.Disks, function(i, thisDisk) {
-			disks += '<small>' + thisDisk.Name + ": " + thisDisk.used.toFixed(1) + 'G / ' + thisDisk.total.toFixed(1) + 'G (' + thisDisk.pctFree.toFixed(1) + '%)</small><br/>';
-			if (diskTopUsage < thisDisk.pctFree) {
-				diskTopUsage = thisDisk.pctFree;
+			//Math
+			thisHostObj.usedMem = thisHostObj.totalMem - thisHostObj.freeMem;
+			thisHostObj.avgCpuLoad = loadTotals / processorCount;
+			thisHostObj.memPct = thisHostObj.usedMem / thisHostObj.totalMem;
+			var disks = new String;
+			var diskTopUsage = 0;
+			$.each(thisHostObj.Disks, function(i, thisDisk) {
+				disks += '<small>' + thisDisk.Name + ": " + thisDisk.used.toFixed(1) + 'G / ' + thisDisk.total.toFixed(1) + 'G (' + thisDisk.pctFree.toFixed(1) + '%)</small><br/>';
+				if (diskTopUsage < thisDisk.pctFree) {
+					diskTopUsage = thisDisk.pctFree;
+				}
+			});
+			var diskUsageClass = new String;
+			if (diskTopUsage && diskTopUsage > 90) {
+				diskUsageClass = 'panel-danger';
+			} else if (diskTopUsage && diskTopUsage > 75) {
+				diskUsageClass = 'panel-warning';
+			} else {
+				diskUsageClass = 'panel-default';
+			}
+			//TODO Move this outside of the loop; it should only be set once.
+			
+		if($("input:radio[name=view-grid-cols]:checked").val()) {
+				columnWidth = $('input:radio[name=view-grid-cols]:checked').val(); //If a value has been selected for the grid view
+			} else if ($('input:radio[name=view-select]:checked').val() == 'grid') {
+				columnWidth = 3; //If the grid view is selected but no value has been assigned for view-grid-cols
+			}
+			//TODO Check hostsCount % columnWidth to add <div class="row"></div> surrounding each row for better layout.
+			hostsCount++;
+			
+			if(!thisHostObj.error) {
+				//Build the server entry.
+				$('#hosts-list').append('<div class="col-md-' + columnWidth + '" id="' + thisHostObj.hostid +
+				'"><h4 style="text-align: center">' + thisHostObj.name + '<small> - ' + thisHostObj.os + '</small></h4><span class="circle" data-pct="' +
+				thisHostObj.avgCpuLoad + '"><strong></strong><span>' + thisHostObj.numProcesses +
+				' PROCS</span></span><div class="pull-right"><span class="circle" data-pct="' + thisHostObj.memPct +
+				'"><strong></strong><span>USED: ' + thisHostObj.usedMem.toFixed(1) + 'G / ' +
+				thisHostObj.totalMem.toFixed(1) + 'G' + '</span></span></div><br/><div class="row"><div class="col-md-6">PING: ' +
+				thisHostObj.agentPing + 'ms<br/>TRAFFIC: ' + thisHostObj.networkTraffic.toFixed(1) +
+				'KBps</div><div class="col-md-6"><span class="pull-right"><div class="panel-group" id="accordion-' + thisHostObj.hostid + '"><div class="panel ' + diskUsageClass + '"><div class="panel-heading"><div class="panel-title"><a data-toggle="collapse" data-parent="#accordion-' + thisHostObj.hostid + '" href="#collapser-' + thisHostObj.hostid + '">DISKS</a></div></div><div id="collapser-' + thisHostObj.hostid + '" class="panel-collapse collapse"><div class="panel-body">' + disks +
+				'</div></div></div></div></span></div></div></div>');
+			} else {
+				$('#hosts-list').append('<div class="col-md-' + columnWidth + ' blink" id="' + thisHostObj.hostid + '"><h3 style="text-align: center; font-weight: bolder">' +
+					thisHostObj.name + '</h3><h4 style="font-weight: bolder">' + thisHostObj.errorText + '</h4></div>');
 			}
+	
+	
+	
+	
 		});
-		var diskUsageClass = new String;
-		if (diskTopUsage && diskTopUsage > 90) {
-			diskUsageClass = 'panel-danger';
-		} else if (diskTopUsage && diskTopUsage > 75) {
-			diskUsageClass = 'panel-warning';
-		} else {
-			diskUsageClass = 'panel-default';
-		}
-		//TODO Move this outside of the loop; it should only be set once.
-		
-	if($("input:radio[name=view-grid-cols]:checked").val()) {
-			columnWidth = $('input:radio[name=view-grid-cols]:checked').val(); //If a value has been selected for the grid view
-		} else if ($('input:radio[name=view-select]:checked').val() == 'grid') {
-			columnWidth = 3; //If the grid view is selected but no value has been assigned for view-grid-cols
-		}
-		//TODO Check hostsCount % columnWidth to add <div class="row"></div> surrounding each row for better layout.
-		hostsCount++;
-		
-		if(!thisHostObj.error) {
-			//Build the server entry.
-			$('#hosts-list').append('<div class="col-md-' + columnWidth + '" id="' + thisHostObj.hostid +
-			'"><h4 style="text-align: center">' + thisHostObj.name + '<small> - ' + thisHostObj.os + '</small></h4><span class="circle" data-pct="' +
-			thisHostObj.avgCpuLoad + '"><strong></strong><span>' + thisHostObj.numProcesses +
-			' PROCS</span></span><div class="pull-right"><span class="circle" data-pct="' + thisHostObj.memPct +
-			'"><strong></strong><span>USED: ' + thisHostObj.usedMem.toFixed(1) + 'G / ' +
-			thisHostObj.totalMem.toFixed(1) + 'G' + '</span></span></div><br/><div class="row"><div class="col-md-6">PING: ' +
-			thisHostObj.agentPing + 'ms<br/>TRAFFIC: ' + thisHostObj.networkTraffic.toFixed(1) +
-			'KBps</div><div class="col-md-6"><span class="pull-right"><div class="panel-group" id="accordion-' + thisHostObj.hostid + '"><div class="panel ' + diskUsageClass + '"><div class="panel-heading"><div class="panel-title"><a data-toggle="collapse" data-parent="#accordion-' + thisHostObj.hostid + '" href="#collapser-' + thisHostObj.hostid + '">DISKS</a></div></div><div id="collapser-' + thisHostObj.hostid + '" class="panel-collapse collapse"><div class="panel-body">' + disks +
-			'</div></div></div></div></span></div></div></div>');
-		} else {
-			$('#hosts-list').append('<div class="col-md-' + columnWidth + ' blink" id="' + thisHostObj.hostid + '"><h3 style="text-align: center; font-weight: bolder">' +
-				thisHostObj.name + '</h3><h4 style="font-weight: bolder">' + thisHostObj.errorText + '</h4></div>');
-		}
-
-
-
-
-	});
-	//Play the tornado siren if a server is unreachable
-	if(anError === 1) { //We do this outside the for loop because we only want the sound to play...once.
-		var times = 3;
-		function repeat() {
-			times--;
-			if (times === 0) {
-				clearInterval(loop);
+		//Play the tornado siren if a server is unreachable
+		if(anError === 1) { //We do this outside the for loop because we only want the sound to play...once.
+			var times = 3;
+			function repeat() {
+				times--;
+				if (times === 0) {
+					clearInterval(loop);
+				}
+				var audio = document.createElement("audio");
+				audio.src = "audio/tornado2.ogg";
+				audio.play();
 			}
-			var audio = document.createElement("audio");
-			audio.src = "audio/tornado2.ogg";
-			audio.play();
+			var loop = setInterval(repeat, 6000);
+			repeat();
 		}
-		var loop = setInterval(repeat, 6000);
-		repeat();
-	}
-
-	$('.circle').each(function() {
-		$(this).circleProgress({
-			value: $(this).attr('data-pct'),
-			fill: {gradient: ['#1EB33C', '#F72323']},
-			thickness: 15
-		}).on('circle-animation-progress', function (event, progress, stepValue) {
-			$(this).find('strong').html(String((100 * stepValue).toFixed(1)) + '<i>%</i>');
+	
+		$('.circle').each(function() {
+			$(this).circleProgress({
+				value: $(this).attr('data-pct'),
+				fill: {gradient: ['#1EB33C', '#F72323']},
+				thickness: 15
+			}).on('circle-animation-progress', function (event, progress, stepValue) {
+				$(this).find('strong').html(String((100 * stepValue).toFixed(1)) + '<i>%</i>');
+			});
 		});
-	});
-
-
-}
-</script>
+	
+	
+	}
+	</script>
+</body>
 </html>
