[Libreoffice-commits] core.git: sw/qa sw/source

[Xisco Fauli]

 sw/qa/extras/ooxmlexport/data/tdf118702.odt |binary
 sw/qa/extras/ooxmlexport/ooxmlexport5.cxx   |    9 +++++++++
 sw/qa/extras/ooxmlexport/ooxmlexport9.cxx   |    4 ----
 sw/source/filter/ww8/ww8atr.cxx             |    1 -
 4 files changed, 9 insertions(+), 5 deletions(-)

New commits:
commit b0e2dbca5164c494a669bd48f8ff058f4c1ad710
Author: Xisco Fauli <xiscofauli at libreoffice.org>
Date:   Thu Jul 12 14:12:05 2018 +0200

    tdf#118702: partially revert the fix for tdf#78907
    
    This is a regression from 2df0d9d32d05f08ccf673fd0f61f650438511acb
    
    Reverting it as the unitest from tdf#78907 still passes.
    Besides, adding a new unittest for tdf#118702
    
    Finally, adapt ooxmlexport9 and remove pointless assert which fails
    with:
    
    ooxmlexport9.cxx:557:Assertion
    Test name: testBnc519228OddBreaks::Import_Export_Import
    equality assertion failed
    - Expected: <Any: (void)>
    - Actual  : <Any: (string) Converted2>
    
    Failures !!!
    Run: 155   Failure total: 1   Failures: 1   Errors: 0
    
    
    Change-Id: I486e9489ef6c3978cbcf90f3b9885248f4f92110
    Reviewed-on: https://gerrit.libreoffice.org/57338
    Tested-by: Jenkins
    Reviewed-by: Justin Luth <justin_luth at sil.org>

diff --git a/sw/qa/extras/ooxmlexport/data/tdf118702.odt b/sw/qa/extras/ooxmlexport/data/tdf118702.odt
new file mode 100644
index 000000000000..d55099242d04
Binary files /dev/null and b/sw/qa/extras/ooxmlexport/data/tdf118702.odt differ
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport5.cxx b/sw/qa/extras/ooxmlexport/ooxmlexport5.cxx
index c7a808a24e41..e9b02ed948ef 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport5.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport5.cxx
@@ -569,6 +569,15 @@ DECLARE_OOXMLEXPORT_TEST(testfdo78907,"fdo78907.docx")
     assertXPath ( pXmlDoc1, "/w:ftr[1]/w:tbl[1]/w:tr[1]/w:tc[1]/w:tbl[1]/w:tr[1]/w:tc[1]/w:tbl", 0 );
 }
 
+DECLARE_OOXMLEXPORT_TEST(tdf118702,"tdf118702.odt")
+{
+    xmlDocPtr pXmlDoc = parseExport("word/document.xml");
+    if (!pXmlDoc)
+        return;
+    assertXPath ( pXmlDoc, "/w:document/w:body/w:p[1]/w:pPr/w:sectPr/w:type", "val", "nextPage" );
+    assertXPath ( pXmlDoc, "/w:document/w:body/w:p[1]/w:pPr/w:sectPr/w:pgSz", "orient", "landscape" );
+}
+
 DECLARE_OOXMLEXPORT_TEST(testfdo79822, "fdo79822.docx")
 {
     /* File getting crash while saving in LO.
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport9.cxx b/sw/qa/extras/ooxmlexport/ooxmlexport9.cxx
index 5df206d75a90..f1a260b87057 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport9.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport9.cxx
@@ -551,10 +551,6 @@ DECLARE_OOXMLEXPORT_TEST(testBnc519228OddBreaks, "bnc519228_odd-breaksB.docx")
 
     // Page2 comes from follow of style for page 1 and should be a normal page. Also check the two page style have the same properties,
     // since page style for page1 was created from page style for page 2.
-    uno::Any page2StyleAny = uno::Reference<beans::XPropertySet>(
-        getParagraph(2, "This is page 2, which is obviously an even page."),
-        uno::UNO_QUERY_THROW)->getPropertyValue("PageDescName");
-    CPPUNIT_ASSERT_EQUAL(uno::Any(), page2StyleAny);
     OUString page2StyleName = getProperty<OUString>( page1Style, "FollowStyle" );
     uno::Reference<beans::XPropertySet> page2Style(getStyles("PageStyles")->getByName(page2StyleName), uno::UNO_QUERY);
     CPPUNIT_ASSERT_EQUAL(uno::makeAny(style::PageStyleLayout_ALL), page2Style->getPropertyValue("PageStyleLayout"));
diff --git a/sw/source/filter/ww8/ww8atr.cxx b/sw/source/filter/ww8/ww8atr.cxx
index 2a847e21c09e..f404d9ab2498 100644
--- a/sw/source/filter/ww8/ww8atr.cxx
+++ b/sw/source/filter/ww8/ww8atr.cxx
@@ -493,7 +493,6 @@ void MSWordExportBase::OutputSectionBreaks( const SfxItemSet *pSet, const SwNode
                     }
                 }
             }
-            bNewPageDesc = false; // if next node has RES_BREAK(page break) then bNewPageDesc value should be false.
             bBreakSet = true;
 
             if ( !bRemoveHardBreakInsideTable )