From 98a94a032f08e9f1d0ad6a07ea45ef539dd8eac9 Mon Sep 17 00:00:00 2001
From: Michael Williamson <mike@zwobble.org>
Date: Sat, 2 Jan 2016 22:32:32 +0000
Subject: Use Style class in Paragraph

---
 .../zwobble/mammoth/documents/ParagraphElement.java | 10 +++++-----
 .../java/org/zwobble/mammoth/documents/Style.java   | 21 +++++++++++++++++++++
 src/main/java/org/zwobble/mammoth/docx/BodyXml.java | 12 +++++-------
 src/main/java/org/zwobble/mammoth/docx/Style.java   | 15 ---------------
 src/main/java/org/zwobble/mammoth/docx/Styles.java  |  2 ++
 .../java/org/zwobble/mammoth/docx/StylesXml.java    |  8 +++++---
 6 files changed, 38 insertions(+), 30 deletions(-)
 create mode 100644 src/main/java/org/zwobble/mammoth/documents/Style.java
 delete mode 100644 src/main/java/org/zwobble/mammoth/docx/Style.java

(limited to 'src/main/java/org')

diff --git a/src/main/java/org/zwobble/mammoth/documents/ParagraphElement.java b/src/main/java/org/zwobble/mammoth/documents/ParagraphElement.java
index dcf8dad..22d9306 100644
--- a/src/main/java/org/zwobble/mammoth/documents/ParagraphElement.java
+++ b/src/main/java/org/zwobble/mammoth/documents/ParagraphElement.java
@@ -4,16 +4,16 @@ import java.util.List;
 import java.util.Optional;
 
 public class ParagraphElement implements DocumentElement, HasChildren {
-    private final Optional<String> styleId;
+    private final Optional<Style> style;
     private final List<DocumentElement> children;
 
-    public ParagraphElement(Optional<String> styleId, List<DocumentElement> children) {
-        this.styleId = styleId;
+    public ParagraphElement(Optional<Style> style, List<DocumentElement> children) {
+        this.style = style;
         this.children = children;
     }
 
-    public Optional<String> getStyleId() {
-        return styleId;
+    public Optional<Style> getStyle() {
+        return style;
     }
 
     public List<DocumentElement> getChildren() {
diff --git a/src/main/java/org/zwobble/mammoth/documents/Style.java b/src/main/java/org/zwobble/mammoth/documents/Style.java
new file mode 100644
index 0000000..82a25de
--- /dev/null
+++ b/src/main/java/org/zwobble/mammoth/documents/Style.java
@@ -0,0 +1,21 @@
+package org.zwobble.mammoth.documents;
+
+import java.util.Optional;
+
+public class Style {
+    private final String styleId;
+    private final Optional<String> name;
+
+    public Style(String styleId, Optional<String> name) {
+        this.styleId = styleId;
+        this.name = name;
+    }
+
+    public String getStyleId() {
+        return styleId;
+    }
+
+    public Optional<String> getName() {
+        return name;
+    }
+}
diff --git a/src/main/java/org/zwobble/mammoth/docx/BodyXml.java b/src/main/java/org/zwobble/mammoth/docx/BodyXml.java
index 8aba487..df98fbc 100644
--- a/src/main/java/org/zwobble/mammoth/docx/BodyXml.java
+++ b/src/main/java/org/zwobble/mammoth/docx/BodyXml.java
@@ -1,10 +1,7 @@
 package org.zwobble.mammoth.docx;
 
 import com.google.common.collect.ImmutableList;
-import org.zwobble.mammoth.documents.DocumentElement;
-import org.zwobble.mammoth.documents.ParagraphElement;
-import org.zwobble.mammoth.documents.RunElement;
-import org.zwobble.mammoth.documents.TextElement;
+import org.zwobble.mammoth.documents.*;
 import org.zwobble.mammoth.xml.XmlElement;
 import org.zwobble.mammoth.xml.XmlElementLike;
 import org.zwobble.mammoth.xml.XmlNode;
@@ -39,12 +36,13 @@ public class BodyXml {
     }
 
     private static ParagraphElement readParagraph(XmlElement element) {
-        return new ParagraphElement(readParagraphStyleId(element), readElements(element.children()));
+        return new ParagraphElement(readParagraphStyle(element), readElements(element.children()));
     }
 
-    private static Optional<String> readParagraphStyleId(XmlElement paragraph) {
+    private static Optional<Style> readParagraphStyle(XmlElement paragraph) {
         XmlElementLike properties = paragraph.findChildOrEmpty("w:pPr");
-        return properties.findChildOrEmpty("w:pStyle").getAttributeOrNone("w:val");
+        return properties.findChildOrEmpty("w:pStyle").getAttributeOrNone("w:val")
+            .map(styleId -> new Style(styleId, Optional.empty()));
     }
 
     private static List<DocumentElement> readElements(Iterable<XmlNode> nodes) {
diff --git a/src/main/java/org/zwobble/mammoth/docx/Style.java b/src/main/java/org/zwobble/mammoth/docx/Style.java
deleted file mode 100644
index 551a93f..0000000
--- a/src/main/java/org/zwobble/mammoth/docx/Style.java
+++ /dev/null
@@ -1,15 +0,0 @@
-package org.zwobble.mammoth.docx;
-
-import java.util.Optional;
-
-public class Style {
-    private final Optional<String> name;
-
-    public Style(Optional<String> name) {
-        this.name = name;
-    }
-
-    public Optional<String> getName() {
-        return name;
-    }
-}
diff --git a/src/main/java/org/zwobble/mammoth/docx/Styles.java b/src/main/java/org/zwobble/mammoth/docx/Styles.java
index 89098de..d2311fe 100644
--- a/src/main/java/org/zwobble/mammoth/docx/Styles.java
+++ b/src/main/java/org/zwobble/mammoth/docx/Styles.java
@@ -1,5 +1,7 @@
 package org.zwobble.mammoth.docx;
 
+import org.zwobble.mammoth.documents.Style;
+
 import java.util.Map;
 import java.util.Optional;
 
diff --git a/src/main/java/org/zwobble/mammoth/docx/StylesXml.java b/src/main/java/org/zwobble/mammoth/docx/StylesXml.java
index 18061d1..f7a955d 100644
--- a/src/main/java/org/zwobble/mammoth/docx/StylesXml.java
+++ b/src/main/java/org/zwobble/mammoth/docx/StylesXml.java
@@ -1,9 +1,11 @@
 package org.zwobble.mammoth.docx;
 
+import org.zwobble.mammoth.documents.Style;
 import org.zwobble.mammoth.xml.XmlElement;
 import org.zwobble.mammoth.xml.XmlElementList;
 
 import java.util.Map;
+import java.util.Optional;
 
 import static com.google.common.collect.Iterables.filter;
 import static com.google.common.collect.Maps.immutableEntry;
@@ -24,8 +26,8 @@ public class StylesXml {
     }
 
     private static Map.Entry<String, Style> readStyle(XmlElement element) {
-        return immutableEntry(
-            element.getAttribute("w:styleId"),
-            new Style(element.findChildOrEmpty("w:name").getAttributeOrNone("w:val")));
+        String styleId = element.getAttribute("w:styleId");
+        Optional<String> styleName = element.findChildOrEmpty("w:name").getAttributeOrNone("w:val");
+        return immutableEntry(styleId, new Style(styleId, styleName));
     }
 }
-- 
cgit v1.2.3