diff --git a/guacamole/src/main/webapp/app/form/templates/timeZoneField.html b/guacamole/src/main/webapp/app/form/templates/timeZoneField.html
index e5db1b8a6..15fd4d6fb 100644
--- a/guacamole/src/main/webapp/app/form/templates/timeZoneField.html
+++ b/guacamole/src/main/webapp/app/form/templates/timeZoneField.html
@@ -9,6 +9,6 @@
+ ng-options="timeZone.value as timeZone.key for timeZone in timeZones[region] | toArray | orderBy: key">
diff --git a/guacamole/src/main/webapp/app/index/filters/arrayFilter.js b/guacamole/src/main/webapp/app/index/filters/arrayFilter.js
index d3a5df25e..861295336 100644
--- a/guacamole/src/main/webapp/app/index/filters/arrayFilter.js
+++ b/guacamole/src/main/webapp/app/index/filters/arrayFilter.js
@@ -19,20 +19,62 @@
/**
* A filter for transforming an object into an array of all non-inherited
- * property values.
+ * property key/value pairs. The resulting array contains one object for each
+ * property in the original object, where the "key" property contains the
+ * original property key and the "value" property contains the original
+ * property value.
*/
angular.module('index').filter('toArray', [function toArrayFactory() {
- return function toArrayFiter(input) {
+ /**
+ * The name of the property to use to store the cached result of converting
+ * an object to an array. This property is added to each object converted,
+ * such that the same array is returned each time unless the original
+ * object has changed.
+ *
+ * @type String
+ */
+ var CACHE_KEY = '_guac_toArray';
+
+ return function toArrayFilter(input) {
// If no object is available, just return an empty array
if (!input) {
return [];
}
- return Object.keys(input).map(function fetchValueByKey(key) {
- return input[key];
+ // Translate object into array of key/value pairs
+ var array = [];
+ angular.forEach(input, function fetchValueByKey(value, key) {
+ array.push({
+ key : key,
+ value : value
+ });
});
+
+ // Sort consistently by key
+ array.sort(function compareKeys(a, b) {
+ if (a.key < b.key) return -1;
+ if (a.key > b.key) return 1;
+ return 0;
+ });
+
+ // Define non-enumerable property for holding cached array
+ if (!input[CACHE_KEY]) {
+ Object.defineProperty(input, CACHE_KEY, {
+ value : [],
+ enumerable : false,
+ configurable : true,
+ writable : true
+ });
+ }
+
+ // Update cache if resulting array is different
+ if (!angular.equals(input[CACHE_KEY], array))
+ input[CACHE_KEY] = array;
+
+ return input[CACHE_KEY];
+
};
}]);
diff --git a/guacamole/src/main/webapp/app/manage/templates/manageConnection.html b/guacamole/src/main/webapp/app/manage/templates/manageConnection.html
index 112f715b0..ed103351d 100644
--- a/guacamole/src/main/webapp/app/manage/templates/manageConnection.html
+++ b/guacamole/src/main/webapp/app/manage/templates/manageConnection.html
@@ -32,7 +32,7 @@