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

[Tibor Nagy (via logerrit)]

 sw/qa/extras/ooxmlexport/data/tdf133924.docx |binary
 sw/qa/extras/ooxmlexport/ooxmlexport6.cxx    |   12 +++++++++++-
 sw/source/filter/ww8/docxattributeoutput.cxx |   11 ++++-------
 3 files changed, 15 insertions(+), 8 deletions(-)

New commits:
commit 9bbc60c0d5e60eacecd624f343aecc94e4219c84
Author:     Tibor Nagy <nagy.tibor2 at nisz.hu>
AuthorDate: Sun Jun 14 16:23:54 2020 +0200
Commit:     László Németh <nemeth at numbertext.org>
CommitDate: Fri Jun 19 14:17:14 2020 +0200

    tdf#133924 DOCX export: fix text wraps around frame
    
    Co-authored-by: Attila Bakos (NISZ)
    
    Change-Id: I08f0c8a8f4f554178548ca5fc8d072998a379b32
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/96290
    Tested-by: László Németh <nemeth at numbertext.org>
    Reviewed-by: László Németh <nemeth at numbertext.org>

diff --git a/sw/qa/extras/ooxmlexport/data/tdf133924.docx b/sw/qa/extras/ooxmlexport/data/tdf133924.docx
new file mode 100644
index 000000000000..3bdbe8d6f1e0
Binary files /dev/null and b/sw/qa/extras/ooxmlexport/data/tdf133924.docx differ
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport6.cxx b/sw/qa/extras/ooxmlexport/ooxmlexport6.cxx
index df0ee671c9c1..123a339c6232 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport6.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport6.cxx
@@ -977,7 +977,17 @@ DECLARE_OOXMLEXPORT_TEST(testTdf133457, "tdf133457.docx")
     if (!pXmlDocument)
         return;
 
-    assertXPath(pXmlDocument, "/w:document/w:body/w:p[4]/w:pPr/w:framePr","vAnchor","text");
+    assertXPath(pXmlDocument, "/w:document/w:body/w:p[4]/w:pPr/w:framePr", "vAnchor", "text");
+}
+
+DECLARE_OOXMLEXPORT_EXPORTONLY_TEST(testTdf133924, "tdf133924.docx")
+{
+    xmlDocUniquePtr pXmlDocument = parseExport("word/document.xml");
+    if (!pXmlDocument)
+        return;
+
+    assertXPath(pXmlDocument, "/w:document/w:body/w:p[2]/w:pPr/w:framePr", "wrap", "around");
+    assertXPath(pXmlDocument, "/w:document/w:body/w:p[3]/w:pPr/w:framePr", "wrap", "notBeside");
 }
 
 CPPUNIT_PLUGIN_IMPLEMENT();
diff --git a/sw/source/filter/ww8/docxattributeoutput.cxx b/sw/source/filter/ww8/docxattributeoutput.cxx
index 0ebda68eed20..3bd7ae8fbf43 100644
--- a/sw/source/filter/ww8/docxattributeoutput.cxx
+++ b/sw/source/filter/ww8/docxattributeoutput.cxx
@@ -583,17 +583,14 @@ void DocxAttributeOutput::PopulateFrameProperties(const SwFrameFormat* pFrameFor
     switch (pFrameFormat->GetSurround().GetValue())
     {
     case css::text::WrapTextMode_NONE:
-        attrList->add( FSNS( XML_w, XML_wrap), "none");
-        break;
-    case css::text::WrapTextMode_THROUGH:
-        attrList->add( FSNS( XML_w, XML_wrap), "through");
-        break;
-    case css::text::WrapTextMode_PARALLEL:
         attrList->add( FSNS( XML_w, XML_wrap), "notBeside");
         break;
     case css::text::WrapTextMode_DYNAMIC:
+        attrList->add(FSNS(XML_w, XML_wrap), "auto");
+        break;
+    case css::text::WrapTextMode_PARALLEL:
     default:
-        attrList->add( FSNS( XML_w, XML_wrap), "auto");
+        attrList->add(FSNS(XML_w, XML_wrap), "around");
         break;
     }
     attrList->add( FSNS( XML_w, XML_vAnchor), relativeFromV );