Skip to content

Commit 358d290

Browse files
committed
Merge pull request #1 from ajaxorg/master
updates
2 parents 145f65b + b04a8cd commit 358d290

File tree

5 files changed

+11
-6
lines changed

5 files changed

+11
-6
lines changed

core/baseclasses/basetree.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -744,7 +744,8 @@ apf.BaseTree = function(){
744744
pContainer = htmlNode.parentNode;
745745

746746
//Remove htmlNodes from tree
747-
containerNode.parentNode.removeChild(containerNode);
747+
if (containerNode)
748+
containerNode.parentNode.removeChild(containerNode);
748749
pContainer.removeChild(htmlNode);
749750

750751
//Datagrid??

core/class.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -945,7 +945,7 @@ apf.Class.prototype = new (function(){
945945
//Remove id from global js space
946946
try {
947947
if (this.id || this.name)
948-
self[this.id || this.name] = null;
948+
delete self[this.id || this.name];
949949
}
950950
catch (ex) {}
951951
return;

core/lib/xmldb.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -536,7 +536,7 @@ apf.xmldb = new (function(){
536536

537537
// @todo: only do this once! - should store on the undo object
538538
if (oldNode.ownerDocument.importNode && newNode.ownerDocument != oldNode.ownerDocument) {
539-
var oldNodeS = xmlNode;
539+
var oldNodeS = newNode;
540540
newNode = oldNode.ownerDocument.importNode(newNode, true); //Safari issue not auto importing nodes
541541
if (oldNodeS.parentNode)
542542
oldNodeS.parentNode.removeChild(oldNodeS);

core/markup/aml/element.js

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -479,8 +479,12 @@ apf.AmlElement = function(struct, tagName){
479479
if (options && options.clear)
480480
include.setAttribute("clear", true);
481481
include.options = options;
482-
include.callback = options && options.callback || function(){
482+
include.callback = function(){
483483
_self.dispatchEvent("afteramlinserted", {src: amlDefNode});
484+
options && options.callback && options.callback();
485+
setTimeout(function(){
486+
include.destroy(true, true);
487+
});
484488
};
485489
this.appendChild(include);
486490
};

elements/skin.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ apf.aml.setElement("skin", apf.skin);
7878
}
7979

8080
this.$propHandlers["name"] = function(value){
81-
if (!this.attributes.getNamedItem("src")) {
81+
if (!this.src && !this.attributes.getNamedItem("src")) {
8282
this.$path = apf.getAbsolutePath(apf.hostPath, value) + "/index.xml";
8383
getSkin.call(this, this.$path);
8484
}
@@ -289,4 +289,4 @@ apf.aml.setElement("skin", apf.skin);
289289
});
290290
}).call(apf.skin.prototype = new apf.AmlElement());
291291

292-
// #endif
292+
// #endif

0 commit comments

Comments
 (0)