Move document conversion into document-to-html module

4 files changed, 83 insertions, 85 deletions

diff --git a/lib/document-to-html.js b/lib/document-to-html.js
new file mode 100644
index 0000000..caccb6f
--- /dev/null
+++ b/lib/document-to-html.js
@@ -0,0 +1,80 @@
+var q = require("q");
+
+var styles = require("./styles");
+var HtmlGenerator = require("./html-generator").HtmlGenerator;
+
+exports.DocumentConverter = DocumentConverter;
+
+
+function DocumentConverter(options) {
+    options = options || {};
+    var defaultParagraphStyle = options.defaultParagraphStyle || styles.topLevelElement("p");
+    
+    var paragraphStyleMap = options.paragraphStyleMap || {};
+    var runStyleMap = options.runStyleMap || {};
+    
+    function convertToHtml(document) {
+        var html = new HtmlGenerator();
+        elementToHtml(document, html);
+        html.closeAll();
+        
+        return q.when({
+            html: html.asString()
+        }); 
+    }
+    
+    function convertElements(elements, html) {
+        elements.forEach(function(element) {
+            elementToHtml(element, html);
+        });
+    }
+
+    function elementToHtml(element, html) {
+        var handler = elementConverters[element.type];
+        if (handler) {
+            handler(element, html);
+        }
+    }
+    
+    function convertParagraph(element, html) {
+        html.style(styleForParagraph(element));
+        convertElements(element.children, html);
+    }
+    
+    function styleForParagraph(element) {
+        return paragraphStyleMap[element.properties.styleName] || defaultParagraphStyle;
+    }
+    
+    function convertRun(run, html) {
+        var runHtml = new HtmlGenerator();
+        if (run.properties.styleName) {
+            var style = runStyleMap[run.properties.styleName];
+            if (style) {
+                runHtml.style(style);
+            }
+        }
+        if (run.properties.isBold) {
+            runHtml.push(styles.element("strong"));
+        }
+        if (run.properties.isItalic) {
+            runHtml.push(styles.element("em"));
+        }
+        convertElements(run.children, runHtml);
+        runHtml.closeAll();
+        html.append(runHtml);
+    }
+
+    var elementConverters = {
+        "document": function(document, html) {
+            convertElements(document.children, html);
+        },
+        "paragraph": convertParagraph,
+        "run": convertRun,
+        "text": function(element, html) {
+            html.text(element.value);
+        }
+    };
+    return {
+        convertToHtml: convertToHtml
+    }
+}
diff --git a/lib/documents.js b/lib/documents.js
index 88f95ba..27190aa 100644
--- a/lib/documents.js
+++ b/lib/documents.js
@@ -1,86 +1,8 @@
-var q = require("q");
-
-var styles = require("./styles");
-var HtmlGenerator = require("./html-generator").HtmlGenerator;
-
-exports.DocumentConverter = DocumentConverter;
 exports.Document = Document;
 exports.Paragraph = Paragraph;
 exports.Run = Run;
 exports.Text = Text;
 
-function DocumentConverter(options) {
-    options = options || {};
-    var defaultParagraphStyle = options.defaultParagraphStyle || styles.topLevelElement("p");
-    
-    var paragraphStyleMap = options.paragraphStyleMap || {};
-    var runStyleMap = options.runStyleMap || {};
-    
-    function convertToHtml(document) {
-        var html = new HtmlGenerator();
-        elementToHtml(document, html);
-        html.closeAll();
-        
-        return q.when({
-            html: html.asString()
-        }); 
-    }
-    
-    function convertElements(elements, html) {
-        elements.forEach(function(element) {
-            elementToHtml(element, html);
-        });
-    }
-
-    function elementToHtml(element, html) {
-        var handler = elementConverters[element.type];
-        if (handler) {
-            handler(element, html);
-        }
-    }
-    
-    function convertParagraph(element, html) {
-        html.style(styleForParagraph(element));
-        convertElements(element.children, html);
-    }
-    
-    function styleForParagraph(element) {
-        return paragraphStyleMap[element.properties.styleName] || defaultParagraphStyle;
-    }
-    
-    function convertRun(run, html) {
-        var runHtml = new HtmlGenerator();
-        if (run.properties.styleName) {
-            var style = runStyleMap[run.properties.styleName];
-            if (style) {
-                runHtml.style(style);
-            }
-        }
-        if (run.properties.isBold) {
-            runHtml.push(styles.element("strong"));
-        }
-        if (run.properties.isItalic) {
-            runHtml.push(styles.element("em"));
-        }
-        convertElements(run.children, runHtml);
-        runHtml.closeAll();
-        html.append(runHtml);
-    }
-
-    var elementConverters = {
-        "document": function(document, html) {
-            convertElements(document.children, html);
-        },
-        "paragraph": convertParagraph,
-        "run": convertRun,
-        "text": function(element, html) {
-            html.text(element.value);
-        }
-    };
-    return {
-        convertToHtml: convertToHtml
-    }
-}
 
 function Document(children) {
     return {
diff --git a/lib/index.js b/lib/index.js
index b124b81..146b507 100644
--- a/lib/index.js
+++ b/lib/index.js
@@ -1,9 +1,5 @@
-var q = require("q");
-var _ = require("underscore");
-var ZipFile = require("zipfile").ZipFile;
-
 var docxReader = require("./docx-reader");
-var documents = require("./documents");
+var DocumentConverter = require("./document-to-html").DocumentConverter;
 var styles = require("./styles");
 
 exports.Converter = Converter;
@@ -17,7 +13,7 @@ exports.standardOptions = {
 };
 
 function Converter(options) {
-    this._documentConverter = new documents.DocumentConverter(options);
+    this._documentConverter = new DocumentConverter(options);
 }
 
 Converter.prototype.convertToHtml = function(docxFile) {
diff --git a/tests/documents.tests.js b/tests/document-to-html.tests.js
index d08500d..a53039b 100644
--- a/tests/documents.tests.js
+++ b/tests/document-to-html.tests.js
@@ -1,7 +1,7 @@
 var assert = require("assert");
 
 var documents = require("../lib/documents");
-var DocumentConverter = documents.DocumentConverter;
+var DocumentConverter = require("../lib/document-to-html").DocumentConverter;
 var test = require("./testing").test;
 var styles = require("../lib/styles");