styles names should be null by default

2 files changed, 17 insertions, 1 deletions

diff --git a/lib/docx/styles-reader.js b/lib/docx/styles-reader.js
index dc90bc1..fec1cf2 100644
--- a/lib/docx/styles-reader.js
+++ b/lib/docx/styles-reader.js
@@ -36,6 +36,11 @@ function readStylesXml(xml) {
 function readStyleElement(styleElement) {
     var type = styleElement.attributes["w:type"];
     var styleId = styleElement.attributes["w:styleId"];
-    var name = styleElement.first("w:name").attributes["w:val"];
+    var name = styleName(styleElement);
     return {type: type, styleId: styleId, name: name};
 }
+
+function styleName(styleElement) {
+  var nameElement = styleElement.first("w:name");
+  return nameElement ? nameElement.attributes["w:val"] : null;
+}
diff --git a/tests/docx/styles-reader.tests.js b/tests/docx/styles-reader.tests.js
index 7899b1b..f76dfc2 100644
--- a/tests/docx/styles-reader.tests.js
+++ b/tests/docx/styles-reader.tests.js
@@ -59,6 +59,17 @@ describe('readStylesXml', function() {
         });
         assert.equal(styles.findParagraphStyleById("Heading1").name, "Heading 1");
     });
+
+    test('styles names should be null by default', function() {
+      var styles = readStylesXml({
+        root: new XmlElement("w:styles", {}, [
+          paragraphStyleElement("Heading1", null),
+          characterStyleElement("Heading1Char", null)
+        ])
+      });
+      assert.equal(styles.findParagraphStyleById("Heading1Char"), null);
+      assert.equal(styles.findCharacterStyleById("Heading1"), null);
+    });
 });
 
 function paragraphStyleElement(id, name) {