holm/egroupware
1
1
Fork 0
forked from extern/egroupware
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fixed template was not prepending parent/top-level name, but search for first loaded template which ends in its name eg. "rows" used in "mail.sieve.index" picked "addressbook.index.rows" instead of "mail.sieve.index.rows"

Browse Source
This commit is contained in:
Ralf Becker 2014-02-25 13:04:28 +00:00
parent ea35b5ce75
commit a021f998e0
2 changed files with 22 additions and 26 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

42
etemplate/js/et2_widget_template.js
View File

@ -22,11 +22,9 @@
* the template class checks whether another template with this id already
* exists. If yes, this template is removed from the DOM tree, copied and
* inserted in place of this template.
*
* TODO: Check whether this widget behaves as it should.
*
*
* @augments et2_DOMWidget
*/
*/
var et2_template = et2_DOMWidget.extend(
{
attributes: {
@ -57,15 +55,17 @@ var et2_template = et2_DOMWidget.extend(
"name": "Content index",
"default": et2_no_init,
"description": "Used for passing in specific content to the template other than what it would get by ID."
},
}
},
createNamespace: true,
/**
* Initializes this template widget as a simple container.
*
*
* @memberOf et2_template
* @param {et2_widget} _parent
* @param {object} _attrs
*/
init: function(_parent, _attrs) {
// Set this early, so it's available for creating namespace
@ -76,7 +76,7 @@ var et2_template = et2_DOMWidget.extend(
this._super.apply(this, arguments);
this.div = document.createElement("div");
// Deferred object so we can load via AJAX
this.loading = jQuery.Deferred();
@ -89,17 +89,11 @@ var et2_template = et2_DOMWidget.extend(
var templates = this.getRoot().getInstanceManager().templates;
if(!(xml = templates[template_name]))
{
// Check to see if ID is short form
// eg: row instead of app.something.row
for(var key in templates)
{
var splitted = key.split('.');
if(splitted[splitted.length-1] == template_name)
{
xml = templates[key];
break;
}
}
// Check to see if ID is short form --> prepend parent/top-level name
var root = _parent ? _parent.getRoot() : null;
var top_name = root && root._inst ? root._inst.name : null;
if (top_name) template_name = top_name+'.'+template_name;
xml = templates[template_name];
if(!xml)
{
// Ask server
@ -119,10 +113,10 @@ var et2_template = et2_DOMWidget.extend(
// Read the XML structure of the requested template
this.loadFromXML(templates[template_name]);
// Update flag
this.loading.resolve();
}, this);
}
return;
@ -134,7 +128,7 @@ var et2_template = et2_DOMWidget.extend(
this.loadFromXML(xml);
// Don't call this here - done by caller, or on whole widget tree
//this.loadingFinished();
// But resolve the promise
this.loading.resolve();
}
@ -169,17 +163,17 @@ var et2_template = et2_DOMWidget.extend(
getDOMNode: function() {
return this.div;
},
/**
* Override to return the promise for deferred loading
*/
doLoadingFinished: function() {
// Apply parent now, which actually puts into the DOM
this._super.apply(this, arguments);
// Fire load event when done loading
this.loading.done(jQuery.proxy(function() {$j(this).trigger("load");},this.div));
// Not done yet, but widget will let you know
return this.loading.promise();
}

6
etemplate/js/etemplate2.js
View File

@ -248,6 +248,8 @@ etemplate2.prototype.unbind_unload = function()
*/
etemplate2.prototype.load = function(_name, _url, _data, _callback)
{
this.name = _name; // store top-level template name to have it available in widgets
egw().debug("info", "Loaded data", _data);
var currentapp = this.app = _data.currentapp || window.egw_appName;
@ -349,7 +351,7 @@ etemplate2.prototype.load = function(_name, _url, _data, _callback)
etemplate2._byTemplate[_name].push(this);
// Read the XML structure of the requested template
this.widgetContainer.loadFromXML(this.templates[_name || missing_name]);
this.widgetContainer.loadFromXML(this.templates[this.name]);
// List of Promises from widgets that are not quite fully loaded
var deferred = [];
@ -425,7 +427,7 @@ etemplate2.prototype.load = function(_name, _url, _data, _callback)
var template = _xmldoc.childNodes[i];
if(template.nodeName.toLowerCase() != "template") continue;
this.templates[template.getAttribute("id")] = template;
if(!_name) missing_name = template.getAttribute("id");
if(!_name) this.name = template.getAttribute("id");
}
_load.apply(this,[]);
}, this);