<html> <head> <base href="https://bugs.documentfoundation.org/"> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW - FILEOPEN DOCX Footer repeats in sections without footer" href="https://bugs.documentfoundation.org/show_bug.cgi?id=124682#c5">Comment # 5</a> on <a class="bz_bug_link bz_status_NEW " title="NEW - FILEOPEN DOCX Footer repeats in sections without footer" href="https://bugs.documentfoundation.org/show_bug.cgi?id=124682">bug 124682</a> from <a class="email" href="mailto:jluth@mail.com" title="Justin L <jluth@mail.com>"> Justin L</a> <pre>author Justin Luth 2016-06-29 07:35:54 +0000 commit 50bf96d31ab2eb546f6c71cc93c1fa5dd4bf3044 <a class="bz_bug_link bz_status_VERIFIED bz_closed" title="VERIFIED FIXED - FILEOPEN DOCX layout messed up with wrong number of pages (continuous break changed to page break)" href="show_bug.cgi?id=90697">bug 90697</a> docx - don't change continuous break into page break As soon as you set PROP_PAGE_DESC_NAME, you are inserting that style as a page break. Setting a pagebreak via a continuous break was first introduced in commit 50cb1667020494906afaacb68d4163d1eda527cf but the unit test for that commit no longer uses this code. I'm suggesting it be reverted. It really messes up round-tripping when continuous breaks are removed/replaced with hard page breaks. There are a few odd cases where the very first section needs to set the page break via the continuous break, so it hasn't been eliminated completely.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the assignee for the bug.</li> </ul> </body> </html>