Fixes wrong implementation of repeat header

EvotecIT · Jun 20, 2024 · 7b895e3 · 7b895e3
1 parent 63cfda9
commit 7b895e3
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 19 deletions.
diff --git a/OfficeIMO.Tests/Word.Tables.cs b/OfficeIMO.Tests/Word.Tables.cs
@@ -26,7 +26,6 @@ public void Test_CreatingWordDocumentWithTables() {
 
                 Assert.True(wordTable.RepeatHeaderRowAtTheTopOfEachPage == false);
 
-                Assert.True(wordTable.LastRow.RepeatHeaderRowAtTheTopOfEachPage == false);
                 foreach (var row in wordTable.Rows) {
                     Assert.True(row.RepeatHeaderRowAtTheTopOfEachPage == false);
                 }
@@ -35,10 +34,8 @@ public void Test_CreatingWordDocumentWithTables() {
 
                 Assert.True(wordTable.RepeatHeaderRowAtTheTopOfEachPage == true);
 
-                Assert.True(wordTable.LastRow.RepeatHeaderRowAtTheTopOfEachPage == true);
-
-                foreach (var row in wordTable.Rows) {
-                    Assert.True(row.RepeatHeaderRowAtTheTopOfEachPage == true);
+                foreach (var row in wordTable.Rows.Skip(1)) {
+                    Assert.True(row.RepeatHeaderRowAtTheTopOfEachPage == false);
                 }
 
                 Assert.True(wordTable.Rows.Count == 3);
@@ -47,9 +44,7 @@ public void Test_CreatingWordDocumentWithTables() {
 
                 Assert.True(wordTable.Rows.Count == 5);
 
-                foreach (var row in wordTable.Rows) {
-                    Assert.True(row.RepeatHeaderRowAtTheTopOfEachPage == true);
-                }
+                Assert.True(wordTable.Rows[0].RepeatHeaderRowAtTheTopOfEachPage == true);
 
                 Assert.True(wordTable.Rows[2].Cells[0].Paragraphs[0].Text == "Test 3", "Text in table matches. Actual text: " + wordTable.Rows[2].Cells[0].Paragraphs[0].Text);
                 Assert.True(wordTable.Paragraphs.Count == 16, "Number of paragraphs during creation in table is wrong. Current: " + wordTable.Paragraphs.Count);

diff --git a/OfficeIMO.Word/WordTable.cs b/OfficeIMO.Word/WordTable.cs
@@ -237,11 +237,7 @@ public bool? ConditionalFormattingNoVerticalBand {
         /// </summary>
         public bool RepeatHeaderRowAtTheTopOfEachPage {
             get => Rows[0].RepeatHeaderRowAtTheTopOfEachPage;
-            set {
-                foreach (var row in Rows) {
-                    row.RepeatHeaderRowAtTheTopOfEachPage = value;
-                }
-            }
+            set => Rows[0].RepeatHeaderRowAtTheTopOfEachPage = value;
         }
 
         public int RowsCount => this.Rows.Count;
@@ -418,12 +414,7 @@ internal WordTable(WordDocument document, Header header, int rows, int columns,
         /// </summary>
         /// <param name="cellsCount"></param>
         public WordTableRow AddRow(int cellsCount = 0) {
-            // when adding a row to the table, we need to check if the last row has RepeatHeaderRowAtTheTopOfEachPage set
-            // if it does, we need to set it for the new row as well
-            var repeatHeaders = this.LastRow.RepeatHeaderRowAtTheTopOfEachPage;
-            WordTableRow row = new WordTableRow(_document, this) {
-                RepeatHeaderRowAtTheTopOfEachPage = repeatHeaders
-            };
+            WordTableRow row = new WordTableRow(_document, this);
             _table.Append(row._tableRow);
             AddCells(row, cellsCount);
             return row;