Correction of the use case: no namespace and xpath find

SaltwaterC · Sep 20, 2011 · 7735c2b · 7735c2b
1 parent 47fb4c0
commit 7735c2b
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 2 deletions.
diff --git a/lib/libxml-to-js.js b/lib/libxml-to-js.js
@@ -148,12 +148,15 @@ module.exports = function (xml, xpath, callback) {
 		callback = xpath;
 		xpath = null;
 	}
-	var xmlDocument, jsDocument, selected = [];
+	var xmlDocument, jsDocument, selected = [], xmlns = null;
 	try {
 		xmlDocument = libxmljs.parseXmlString(xml);
 		jsDocument = libxml2js(xmlDocument);
+		if( jsDocument['@'] && jsDocument['@'].xmlns) {
+		  xmlns = jsDocument['@'].xmlns;
+		}
 		if( !! xpath) {
-			xmlDocument.find(xpath, jsDocument['@'].xmlns).forEach(function(item) {
+			xmlDocument.find(xpath, xmlns).forEach(function(item) {
 				selected.push(libxml2js(item, true).jsobj);
 			});
 			return callback(null, selected);

diff --git a/tests/test-text.js b/tests/test-text.js
@@ -34,3 +34,13 @@ parser(fs.readFileSync('data/text.xml').toString(), function (err, res) {
 	},
 	res);
 });
+
+parser(fs.readFileSync('data/text.xml').toString(), '//nouvelle/news', function (err, res) {
+	assert.ifError(err);
+	assert.deepEqual({
+		"auteur": "Bizzard5",
+		"date": "17 Août 2008",
+		"text": {}
+	},
+	res[0]);
+});