Set target frame to null when empty string

2 files changed, 10 insertions, 1 deletions

diff --git a/lib/docx/body-reader.js b/lib/docx/body-reader.js
index 71c9855..7b7f6f2 100644
--- a/lib/docx/body-reader.js
+++ b/lib/docx/body-reader.js
@@ -209,7 +209,7 @@ function BodyReader(options) {
             var anchor = element.attributes["w:anchor"];
             return readXmlElements(element.children).map(function(children) {
                 function create(options) {
-                    var targetFrame = element.attributes["w:tgtFrame"];
+                    var targetFrame = element.attributes["w:tgtFrame"] || null;
                     
                     return new documents.Hyperlink(
                         children,
diff --git a/test/docx/body-reader.tests.js b/test/docx/body-reader.tests.js
index f866e9b..4632e3b 100644
--- a/test/docx/body-reader.tests.js
+++ b/test/docx/body-reader.tests.js
@@ -955,6 +955,15 @@ test("w:hyperlink", {
         var result = readXmlElementValue(hyperlinkXml);
         assertThat(result, hasProperties({targetFrame: "_blank"}));
     },
+    
+    "empty target frame is ignored": function() {
+        var hyperlinkXml = new XmlElement("w:hyperlink", {
+            "w:anchor": "Introduction",
+            "w:tgtFrame": ""
+        });
+        var result = readXmlElementValue(hyperlinkXml);
+        assertThat(result, hasProperties({targetFrame: null}));
+    }
 });
 
 test("w:br without explicit type is read as line break", function() {