safebox/guacamole-client
4
0
Fork 0
mirror of https://github.com/gyurix1968/guacamole-client.git synced 2025-09-06 21:27:40 +00:00
Code Issues Packages Projects Releases Wiki Activity

GUAC-1378: Cache previous calls to $templateRequest() to avoid duplicating patch processing work.

Browse Source
This commit is contained in:
Michael Jumper
2016-02-18 17:49:48 -08:00
parent f2a767f045
commit 8f17e9ed0a
1 changed files with 23 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
guacamole/src/main/webapp/app/index/config/templateRequestDecorator.js
View File

@@ -31,6 +31,16 @@ angular.module('index').config(['$provide', function($provide) {
// Required services
var $q = $injector.get('$q');
/**
* A map of previously-returned promises from past calls to
* $templateRequest(). Future calls to $templateRequest() will return
* new promises chained to the first promise returned for a given URL,
* rather than redo patch processing for every request.
*
* @type Object.<String, Promise.<String>>
*/
var promiseCache = {};
/**
* Array of the raw HTML of all patches which should be applied to the
* HTML of retrieved templates.
@@ -224,14 +234,24 @@ angular.module('index').config(['$provide', function($provide) {
* applying all HTML patches from any installed Guacamole extensions
* to the HTML of the requested template.
*
* @param {String} url
* The URL of the template being requested.
*
* @returns {Promise.<String>}
* A Promise which resolves with the patched HTML contents of the
* requested template if retrieval of the template is successful.
*/
var decoratedTemplateRequest = function decoratedTemplateRequest() {
var decoratedTemplateRequest = function decoratedTemplateRequest(url) {
var deferred = $q.defer();
// Chain to cached promise if it already exists
var cachedPromise = promiseCache[url];
if (cachedPromise) {
cachedPromise.then(deferred.resolve, deferred.reject);
return deferred.promise;
}
// Resolve promise with patched template HTML
$delegate.apply(this, arguments).then(function patchTemplate(data) {
@@ -281,6 +301,8 @@ angular.module('index').config(['$provide', function($provide) {
}, deferred.reject);
// Cache this promise for future results
promiseCache[url] = deferred.promise;
return deferred.promise;
};