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

[Miklos Vajna]

 sw/CppunitTest_sw_rtfexport.mk              |    1 +
 sw/qa/extras/rtfexport/data/fdo82858.docx   |binary
 sw/qa/extras/rtfexport/rtfexport.cxx        |    6 ++++++
 sw/source/filter/ww8/rtfattributeoutput.cxx |    3 +++
 4 files changed, 10 insertions(+)

New commits:
commit f824b1b575dbdb2bc515656a66cbb94764031a44
Author: Miklos Vajna <vmiklos at collabora.co.uk>
Date:   Sat Oct 25 12:15:16 2014 +0200

    fdo#82858 RTF export: fix "none" line style of TextFrames
    
    Regression from commit 9e6a5b94e00f0054b058dbb42c2c0b6c75236c9d
    (RtfAttributeOutput::FormatBox: export line{Color,Width} shape
    properties, 2013-04-15), in case lineColor is written, but there is no
    border style, then an explicit fLine=0 property is needed, otherwise a
    black border shows up.
    
    Change-Id: I6d6eedbc8d3ee4bee0f2aadb51a376c7734c6640

diff --git a/sw/CppunitTest_sw_rtfexport.mk b/sw/CppunitTest_sw_rtfexport.mk
index fc692d7..33f366b 100644
--- a/sw/CppunitTest_sw_rtfexport.mk
+++ b/sw/CppunitTest_sw_rtfexport.mk
@@ -76,6 +76,7 @@ $(eval $(call gb_CppunitTest_use_components,sw_rtfexport,\
 	uui/util/uui \
     writerfilter/util/writerfilter \
     xmloff/util/xo \
+	oox/util/oox \
 ))
 
 $(eval $(call gb_CppunitTest_use_configuration,sw_rtfexport))
diff --git a/sw/qa/extras/rtfexport/data/fdo82858.docx b/sw/qa/extras/rtfexport/data/fdo82858.docx
new file mode 100644
index 0000000..e0b1a13
Binary files /dev/null and b/sw/qa/extras/rtfexport/data/fdo82858.docx differ
diff --git a/sw/qa/extras/rtfexport/rtfexport.cxx b/sw/qa/extras/rtfexport/rtfexport.cxx
index 2ffbb38..4cbb975 100644
--- a/sw/qa/extras/rtfexport/rtfexport.cxx
+++ b/sw/qa/extras/rtfexport/rtfexport.cxx
@@ -719,6 +719,12 @@ DECLARE_RTFEXPORT_TEST(testFdo82860, "fdo82860.odt")
     CPPUNIT_ASSERT_EQUAL(OUString("hello"), getParagraphOfText(1, xText)->getString());
 }
 
+DECLARE_RTFEXPORT_TEST(testFdo82858, "fdo82858.docx")
+{
+    // This was table::BorderLineStyle::SOLID, exporter failed to write explicit no line when line color was written.
+    CPPUNIT_ASSERT_EQUAL(table::BorderLineStyle::NONE, getProperty<table::BorderLine2>(getShape(1), "TopBorder").LineStyle);
+}
+
 CPPUNIT_PLUGIN_IMPLEMENT();
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/sw/source/filter/ww8/rtfattributeoutput.cxx b/sw/source/filter/ww8/rtfattributeoutput.cxx
index bb0e6f5..2d3a882 100644
--- a/sw/source/filter/ww8/rtfattributeoutput.cxx
+++ b/sw/source/filter/ww8/rtfattributeoutput.cxx
@@ -3104,6 +3104,9 @@ void RtfAttributeOutput::FormatBox(const SvxBoxItem& rBox)
                 sal_Int32 nWidth = sal_Int32(fConverted * 635); // Twips -> EMUs
                 m_aFlyProperties.push_back(std::make_pair<OString, OString>("lineWidth", OString::number(nWidth)));
             }
+            else
+                // No border: no line.
+                m_aFlyProperties.push_back(std::make_pair<OString, OString>("fLine", "0"));
         }
 
         return;