Fixes issue with read only documents crashing

CHANGELOG.MD

@@ -1,5 +1,6 @@
 #### 0.3.1 - 2022.10.14
 - Adds BMP, GIF, PNG, TIFF image formats support [#42](https://github.com/EvotecIT/OfficeIMO/pull/42) by rstm-sf
+- Fixes issue with read only documents crashing
 
 #### 0.3.0 - 2022.10.11
 - Update DocumentFormat.OpenXml from 2.16.0 to 2.18.0

OfficeIMO.Examples/Program.cs

@@ -95,13 +95,8 @@ static void Main(string[] args) {
             //filePath = System.IO.Path.Combine(folderPath, "AdvancedParagraphs.docx");
             //Example_MultipleParagraphsViaDifferentWays(filePath, false);
 
-            var openWordWithImages = false;
-            Images.Example_AddingImages(folderPath, openWordWithImages);
-            if (openWordWithImages != true) {
-                // this example is based on the example above, so opening word would break the example
-                // This requires additional fixes to work properly, different PR will be needed
-                //Images.Example_ReadWordWithImages();
-            }
+            Images.Example_AddingImages(folderPath, true);
+            Images.Example_ReadWordWithImages();
             Images.Example_AddingImagesMultipleTypes(folderPath, true);
             //Console.WriteLine("[*] Creating standard document with page breaks and removing them");
             //filePath = System.IO.Path.Combine(folderPath, "Basic Document with some page breaks.docx");

OfficeIMO.Tests/Word.Images.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.IO;
 using DocumentFormat.OpenXml.Drawing;
 using OfficeIMO.Word;

OfficeIMO.Word/WordDocument.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.IO;
 using System.IO.Packaging;
@@ -29,8 +29,7 @@ public WordTableOfContent TableOfContent {
             get {
                 var sdtBlocks = _document.Body?.ChildElements.OfType<SdtBlock>() ?? Enumerable.Empty<SdtBlock>();
 
-                foreach (var sdtBlock in sdtBlocks)
-                {
+                foreach (var sdtBlock in sdtBlocks) {
                     var sdtProperties = sdtBlock?.ChildElements.OfType<SdtProperties>().FirstOrDefault();
                     var docPartObject = sdtProperties?.ChildElements.OfType<SdtContentDocPartObject>().FirstOrDefault();
                     var docPartGallery = docPartObject?.ChildElements.OfType<DocPartGallery>().FirstOrDefault();
@@ -48,8 +47,7 @@ public WordCoverPage CoverPage {
             get {
                 var sdtBlocks = _document.Body?.ChildElements.OfType<SdtBlock>() ?? Enumerable.Empty<SdtBlock>();
 
-                foreach (var sdtBlock in sdtBlocks)
-                {
+                foreach (var sdtBlock in sdtBlocks) {
                     var sdtProperties = sdtBlock?.ChildElements.OfType<SdtProperties>().FirstOrDefault();
                     var docPartObject = sdtProperties?.ChildElements.OfType<SdtContentDocPartObject>().FirstOrDefault();
                     var docPartGallery = docPartObject?.ChildElements.OfType<DocPartGallery>().FirstOrDefault();
@@ -487,7 +485,7 @@ public static WordDocument Load(string filePath, bool readOnly = false, bool aut
 
             var wordDocument = WordprocessingDocument.Open(memoryStream, !readOnly, openSettings);
 
-            InitialiseStyleDefinitions(wordDocument);
+            InitialiseStyleDefinitions(wordDocument, readOnly);
 
             word.FilePath = filePath;
             word._wordprocessingDocument = wordDocument;
@@ -496,17 +494,15 @@ public static WordDocument Load(string filePath, bool readOnly = false, bool aut
             return word;
         }
 
-        public static WordDocument Load(Stream stream, bool readOnly = false, bool autoSave = false)
-        {
+        public static WordDocument Load(Stream stream, bool readOnly = false, bool autoSave = false) {
             var document = new WordDocument();
 
-            var openSettings = new OpenSettings
-            {
+            var openSettings = new OpenSettings {
                 AutoSave = autoSave
             };
 
             var wordDocument = WordprocessingDocument.Open(stream, !readOnly, openSettings);
-            InitialiseStyleDefinitions(wordDocument);
+            InitialiseStyleDefinitions(wordDocument, readOnly);
 
             document._wordprocessingDocument = wordDocument;
             document._document = wordDocument.MainDocumentPart.Document;
@@ -638,12 +634,10 @@ public void Save(bool openWord) {
             this.Save("", openWord);
         }
 
-        public void Save(Stream outputStream)
-        {
+        public void Save(Stream outputStream) {
             this._wordprocessingDocument.Clone(outputStream);
 
-            if (outputStream.CanSeek)
-            {
+            if (outputStream.CanSeek) {
                 outputStream.Seek(0, SeekOrigin.Begin);
             }
         }
@@ -678,17 +672,19 @@ public void Dispose() {
             }
         }
 
-        private static void InitialiseStyleDefinitions(WordprocessingDocument wordDocument)
-        {
-            var styleDefinitionsPart = wordDocument.MainDocumentPart.GetPartsOfType<StyleDefinitionsPart>().FirstOrDefault();
-            if (styleDefinitionsPart != null)
-            {
-                AddStyleDefinitions(styleDefinitionsPart);
-            }
-            else
-            {
-                var styleDefinitionsPart1 = wordDocument.MainDocumentPart.AddNewPart<StyleDefinitionsPart>("rId1");
-                GenerateStyleDefinitionsPart1Content(styleDefinitionsPart1);
+        private static void InitialiseStyleDefinitions(WordprocessingDocument wordDocument, bool readOnly) {
+            // if document is read only we shouldn't be doing any new styles, hopefully it doesn't break anything
+            if (readOnly == false) {
+                var styleDefinitionsPart = wordDocument.MainDocumentPart.GetPartsOfType<StyleDefinitionsPart>()
+                    .FirstOrDefault();
+                if (styleDefinitionsPart != null) {
+                    AddStyleDefinitions(styleDefinitionsPart);
+                } else {
+
+                    var styleDefinitionsPart1 = wordDocument.MainDocumentPart.AddNewPart<StyleDefinitionsPart>("rId1");
+                    GenerateStyleDefinitionsPart1Content(styleDefinitionsPart1);
+
+                }
             }
         }
 
@@ -719,4 +715,4 @@ public List<ValidationErrorInfo> DocumentValidationErrors {
             }
         }
     }
-}
+}