Handle numbering properties without w:ilvl or w:numId

2 files changed, 27 insertions, 8 deletions

diff --git a/lib/docx/body-reader.js b/lib/docx/body-reader.js
index 44ae038..25a263a 100644
--- a/lib/docx/body-reader.js
+++ b/lib/docx/body-reader.js
@@ -108,13 +108,7 @@ function BodyReader(options) {
             if (alignElement) {
                 properties.alignment = alignElement.attributes["w:val"];
             }
-            var numberingPropertiesElement = element.first("w:numPr");
-            if (numberingPropertiesElement) {
-                var levelElement = numberingPropertiesElement.first("w:ilvl");
-                var level = levelElement.attributes["w:val"];
-                var numId = numberingPropertiesElement.first("w:numId").attributes["w:val"];
-                properties.numbering = numbering.findLevel(numId, level);
-            }
+            properties.numbering = readNumberingProperties(element.firstOrEmpty("w:numPr"));
             
             return elementResult(properties);
         },
@@ -196,9 +190,20 @@ function BodyReader(options) {
     };
     return {
         readXmlElement: readXmlElement,
-        readXmlElements: readXmlElements
+        readXmlElements: readXmlElements,
+        _readNumberingProperties: readNumberingProperties
     };
 
+    function readNumberingProperties(element) {
+        var level = element.firstOrEmpty("w:ilvl").attributes["w:val"];
+        var numId = element.firstOrEmpty("w:numId").attributes["w:val"];
+        if (level === undefined || numId === undefined) {
+            return null;
+        } else {
+            return numbering.findLevel(numId, level);
+        }
+    }
+
     function readDrawingElement(element) {
         var blips = element
             .getElementsByTagName("a:graphic")
diff --git a/lib/xmlreader.js b/lib/xmlreader.js
index f219d2b..551366f 100644
--- a/lib/xmlreader.js
+++ b/lib/xmlreader.js
@@ -85,6 +85,16 @@ function mapObject(input, valueFunc, keyFunc) {
     }, {});
 }
 
+var emptyElement = {
+    first: function() {
+        return null;
+    },
+    firstOrEmpty: function() {
+        return emptyElement;
+    },
+    attributes: {}
+};
+
 function Element(name, attributes, children) {
     this.type = "element";
     this.name = name;
@@ -98,6 +108,10 @@ Element.prototype.first = function(name) {
     });
 };
 
+Element.prototype.firstOrEmpty = function(name) {
+    return this.first(name) || emptyElement;
+};
+
 Element.prototype.getElementsByTagName = function(name) {
     var elements = _.filter(this.children, function(child) {
         return child.name == name;