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

[Libreoffice Gerrit user]

 sw/qa/extras/odfimport/data/tdf113289.odt          |binary
 sw/qa/extras/odfimport/odfimport.cxx               |    9 +++++++++
 xmloff/source/style/XMLFootnoteSeparatorImport.cxx |    5 ++++-
 3 files changed, 13 insertions(+), 1 deletion(-)

New commits:
commit 4aa8a6ab08b755e3d82860e8dbc294f854336477
Author:     David Vogt <david.vogt at adfinis-sygroup.ch>
AuthorDate: Wed Mar 13 13:11:39 2019 +0100
Commit:     Miklos Vajna <vmiklos at collabora.com>
CommitDate: Thu Mar 14 18:13:24 2019 +0100

    tdf#113289: Default value for footnote separator
    
    In old versions of LO/AOO, separator style was defaulting to a solid
    line. When you have an old document that doesn't have the style set
    explicitly, the line would disappear.
    
    Since newer versions explicitly set the style, we should set the default
    to what it was before.
    
    Change-Id: I8dacea75fcf2f95f9bc145442b22ab0d173e7c5a
    Reviewed-on: https://gerrit.libreoffice.org/69167
    Tested-by: Jenkins
    Reviewed-by: Miklos Vajna <vmiklos at collabora.com>

diff --git a/sw/qa/extras/odfimport/data/tdf113289.odt b/sw/qa/extras/odfimport/data/tdf113289.odt
new file mode 100644
index 000000000000..01aa5700c6d5
Binary files /dev/null and b/sw/qa/extras/odfimport/data/tdf113289.odt differ
diff --git a/sw/qa/extras/odfimport/odfimport.cxx b/sw/qa/extras/odfimport/odfimport.cxx
index a911635d3a6d..0eaae316a22c 100644
--- a/sw/qa/extras/odfimport/odfimport.cxx
+++ b/sw/qa/extras/odfimport/odfimport.cxx
@@ -14,6 +14,7 @@
 #include <com/sun/star/drawing/BitmapMode.hpp>
 #include <com/sun/star/lang/IndexOutOfBoundsException.hpp>
 #include <com/sun/star/style/PageStyleLayout.hpp>
+#include <com/sun/star/style/FootnoteLineStyle.hpp>
 #include <com/sun/star/table/XCell.hpp>
 #include <com/sun/star/table/XCellRange.hpp>
 #include <com/sun/star/table/BorderLine.hpp>
@@ -927,5 +928,13 @@ DECLARE_ODFIMPORT_TEST(testTdf123829, "tdf123829.odt")
         pDoc->getIDocumentSettingAccess().get(DocumentSettingId::COLLAPSE_EMPTY_CELL_PARA));
 }
 
+DECLARE_ODFIMPORT_TEST(testTdf113289, "tdf113289.odt")
+{
+    uno::Any aPageStyle = getStyles("PageStyles")->getByName("Standard");
+    CPPUNIT_ASSERT_EQUAL(static_cast<sal_Int8>(style::FootnoteLineStyle::SOLID),
+                         getProperty<sal_Int8>(aPageStyle, "FootnoteLineStyle"));
+}
+
+
 CPPUNIT_PLUGIN_IMPLEMENT();
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/xmloff/source/style/XMLFootnoteSeparatorImport.cxx b/xmloff/source/style/XMLFootnoteSeparatorImport.cxx
index a49852afc9c4..5e928fe196f8 100644
--- a/xmloff/source/style/XMLFootnoteSeparatorImport.cxx
+++ b/xmloff/source/style/XMLFootnoteSeparatorImport.cxx
@@ -81,7 +81,10 @@ void XMLFootnoteSeparatorImport::StartElement(
     text::HorizontalAdjust eLineAdjust = text::HorizontalAdjust_LEFT;
     sal_Int32 nLineTextDistance = 0;
     sal_Int32 nLineDistance = 0;
-    sal_Int8 nLineStyle = 0;
+
+    // Default separator line style should be SOLID (used to be default before
+    // the choice selector was available)
+    sal_Int8 nLineStyle = 1;
 
     // iterate over xattribute list and fill values
     sal_Int16 nLength = xAttrList->getLength();