@xmldom/xmldom 0.8.9 → 0.8.11

package/CHANGELOG.md

@@ -4,6 +4,24 @@ All notable changes to this project will be documented in this file.
 
 This project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.8.11](https://github.com/xmldom/xmldom/compare/0.8.10...0.8.11)
+
+### Fixed
+
+- update `ownerDocument` when moving nodes between documents
+
+Thank you, [@shunkica](https://github.com/shunkica), for your contributions
+
+
+## [0.8.10](https://github.com/xmldom/xmldom/compare/0.8.9...0.8.10)
+
+### Fixed
+
+- dom: prevent iteration over deleted items [`#514`](https://github.com/xmldom/xmldom/pull/514)/ [`#499`](https://github.com/xmldom/xmldom/issues/499)
+
+Thank you, [@qtow](https://github.com/qtow), for your contributions
+
+
 ## [0.8.9](https://github.com/xmldom/xmldom/compare/0.8.8...0.8.9)
 
 ### Fixed

package/lib/dom.js

@@ -169,7 +169,7 @@ NodeList.prototype = {
 	 * 	The node at the indexth position in the NodeList, or null if that is not a valid index.
 	 */
 	item: function(index) {
-		return this[index] || null;
+		return index >= 0 && index < this.length ? this[index] : null;
 	},
 	toString:function(isHTML,nodeFilter){
 		for(var buf = [], i = 0;i<this.length;i++){
@@ -202,17 +202,23 @@ function LiveNodeList(node,refresh){
 }
 function _updateLiveList(list){
 	var inc = list._node._inc || list._node.ownerDocument._inc;
-	if(list._inc != inc){
+	if (list._inc !== inc) {
 		var ls = list._refresh(list._node);
-		//console.log(ls.length)
 		__set__(list,'length',ls.length);
+		if (!list.$$length || ls.length < list.$$length) {
+			for (var i = ls.length; i in list; i++) {
+				if (Object.prototype.hasOwnProperty.call(list, i)) {
+					delete list[i];
+				}
+			}
+		}
 		copy(ls,list);
 		list._inc = inc;
 	}
 }
 LiveNodeList.prototype.item = function(i){
 	_updateLiveList(this);
-	return this[i];
+	return this[i] || null;
 }
 
 _extends(LiveNodeList,NodeList);
@@ -974,6 +980,9 @@ function _insertBefore(parent, node, child, _inDocumentAssertion) {
 	}
 	do{
 		newFirst.parentNode = parent;
+		// Update ownerDocument for each node being inserted
+		var targetDoc = parent.ownerDocument || parent;
+		_updateOwnerDocument(newFirst, targetDoc);
 	}while(newFirst !== newLast && (newFirst= newFirst.nextSibling))
 	_onUpdateChild(parent.ownerDocument||parent, parent);
 	//console.log(parent.lastChild.nextSibling == null)
@@ -983,6 +992,37 @@ function _insertBefore(parent, node, child, _inDocumentAssertion) {
 	return node;
 }
 
+/**
+ * Recursively updates the ownerDocument property for a node and all its descendants
+ * @param {Node} node
+ * @param {Document} newOwnerDocument
+ * @private
+ */
+function _updateOwnerDocument(node, newOwnerDocument) {
+	if (node.ownerDocument === newOwnerDocument) {
+		return;
+	}
+	
+	node.ownerDocument = newOwnerDocument;
+	
+	// Update attributes if this is an element
+	if (node.nodeType === ELEMENT_NODE && node.attributes) {
+		for (var i = 0; i < node.attributes.length; i++) {
+			var attr = node.attributes.item(i);
+			if (attr) {
+				attr.ownerDocument = newOwnerDocument;
+			}
+		}
+	}
+	
+	// Recursively update child nodes
+	var child = node.firstChild;
+	while (child) {
+		_updateOwnerDocument(child, newOwnerDocument);
+		child = child.nextSibling;
+	}
+}
+
 /**
  * Appends `newChild` to `parentNode`.
  * If `newChild` is already connected to a `parentNode` it is first removed from it.
@@ -1008,6 +1048,11 @@ function _appendSingleChild (parentNode, newChild) {
 	}
 	parentNode.lastChild = newChild;
 	_onUpdateChild(parentNode.ownerDocument, parentNode, newChild);
+	
+	// Update ownerDocument for the new child and all its descendants
+	var targetDoc = parentNode.ownerDocument || parentNode;
+	_updateOwnerDocument(newChild, targetDoc);
+	
 	return newChild;
 }
 
@@ -1036,7 +1081,7 @@ Document.prototype = {
 			return newChild;
 		}
 		_insertBefore(this, newChild, refChild);
-		newChild.ownerDocument = this;
+		_updateOwnerDocument(newChild, this);
 		if (this.documentElement === null && newChild.nodeType === ELEMENT_NODE) {
 			this.documentElement = newChild;
 		}
@@ -1052,7 +1097,7 @@ Document.prototype = {
 	replaceChild: function (newChild, oldChild) {
 		//raises
 		_insertBefore(this, newChild, oldChild, assertPreReplacementValidityInDocument);
-		newChild.ownerDocument = this;
+		_updateOwnerDocument(newChild, this);
 		if (oldChild) {
 			this.removeChild(oldChild);
 		}

package/package.json

@@ -1,71 +1,71 @@
 {
-  "name": "@xmldom/xmldom",
-  "version": "0.8.9",
-  "description": "A pure JavaScript W3C standard-based (XML DOM Level 2 Core) DOMParser and XMLSerializer module.",
-  "keywords": [
-    "w3c",
-    "dom",
-    "xml",
-    "parser",
-    "javascript",
-    "DOMParser",
-    "XMLSerializer",
-    "ponyfill"
-  ],
-  "homepage": "https://github.com/xmldom/xmldom",
-  "repository": {
-    "type": "git",
-    "url": "git://github.com/xmldom/xmldom.git"
-  },
-  "main": "lib/index.js",
-  "types": "index.d.ts",
-  "files": [
-    "CHANGELOG.md",
-    "LICENSE",
-    "readme.md",
-    "SECURITY.md",
-    "index.d.ts",
-    "lib"
-  ],
-  "scripts": {
-    "lint": "eslint lib test",
-    "format": "prettier --write test",
-    "changelog": "auto-changelog --unreleased-only",
-    "start": "nodemon --watch package.json --watch lib --watch test --exec 'npm --silent run test && npm --silent run lint'",
-    "stryker": "stryker run",
-    "stryker:dry-run": "stryker run -m '' --reporters progress",
-    "test": "jest",
-    "testrelease": "npm test && eslint lib",
-    "version": "./changelog-has-version.sh",
-    "release": "np --no-yarn --test-script testrelease --branch release-0.8.x patch"
-  },
-  "engines": {
-    "node": ">=10.0.0"
-  },
-  "dependencies": {},
-  "devDependencies": {
-    "@stryker-mutator/core": "5.6.1",
-    "auto-changelog": "2.4.0",
-    "eslint": "8.25.0",
-    "eslint-config-prettier": "8.5.0",
-    "eslint-plugin-es5": "1.5.0",
-    "eslint-plugin-prettier": "4.2.1",
-    "get-stream": "6.0.1",
-    "jest": "27.5.1",
-    "nodemon": "2.0.20",
-    "np": "7.6.2",
-    "prettier": "2.7.1",
-    "xmltest": "1.5.0",
-    "yauzl": "2.10.0"
-  },
-  "bugs": {
-    "url": "https://github.com/xmldom/xmldom/issues"
-  },
-  "license": "MIT",
-  "auto-changelog": {
-    "prepend": true,
-    "remote": "upstream",
-    "tagPrefix": "",
-    "template": "./auto-changelog.hbs"
-  }
+	"name": "@xmldom/xmldom",
+	"version": "0.8.11",
+	"description": "A pure JavaScript W3C standard-based (XML DOM Level 2 Core) DOMParser and XMLSerializer module.",
+	"keywords": [
+		"w3c",
+		"dom",
+		"xml",
+		"parser",
+		"javascript",
+		"DOMParser",
+		"XMLSerializer",
+		"ponyfill"
+	],
+	"homepage": "https://github.com/xmldom/xmldom",
+	"repository": {
+		"type": "git",
+		"url": "git://github.com/xmldom/xmldom.git"
+	},
+	"main": "lib/index.js",
+	"types": "index.d.ts",
+	"files": [
+		"CHANGELOG.md",
+		"LICENSE",
+		"readme.md",
+		"SECURITY.md",
+		"index.d.ts",
+		"lib"
+	],
+	"scripts": {
+		"lint": "eslint lib test",
+		"format": "prettier --write test",
+		"changelog": "auto-changelog --unreleased-only",
+		"start": "nodemon --watch package.json --watch lib --watch test --exec 'npm --silent run test && npm --silent run lint'",
+		"stryker": "stryker run",
+		"stryker:dry-run": "stryker run -m '' --reporters progress",
+		"test": "jest",
+		"testrelease": "npm test && eslint lib",
+		"version": "./changelog-has-version.sh",
+		"release": "np --no-yarn --test-script testrelease --branch release-0.8.x patch"
+	},
+	"engines": {
+		"node": ">=10.0.0"
+	},
+	"dependencies": {},
+	"devDependencies": {
+		"@stryker-mutator/core": "5.6.1",
+		"auto-changelog": "2.4.0",
+		"eslint": "8.25.0",
+		"eslint-config-prettier": "8.5.0",
+		"eslint-plugin-es5": "1.5.0",
+		"eslint-plugin-prettier": "4.2.1",
+		"get-stream": "6.0.1",
+		"jest": "27.5.1",
+		"nodemon": "2.0.20",
+		"np": "9.2.0",
+		"prettier": "2.7.1",
+		"xmltest": "1.5.0",
+		"yauzl": "2.10.0"
+	},
+	"bugs": {
+		"url": "https://github.com/xmldom/xmldom/issues"
+	},
+	"license": "MIT",
+	"auto-changelog": {
+		"prepend": true,
+		"remote": "upstream",
+		"tagPrefix": "",
+		"template": "./auto-changelog.hbs"
+	}
 }