[Libreoffice-commits] core.git: Branch 'libreoffice-5-0' - sw/qa sw/source

[Mark Hung]

 sw/qa/extras/ww8export/data/tdf92281.doc |binary
 sw/qa/extras/ww8export/ww8export.cxx     |    8 ++++++++
 sw/source/filter/ww8/ww8par6.cxx         |    2 ++
 3 files changed, 10 insertions(+)

New commits:
commit 991dd34d66dada90de0f5eb8f28deb74af8c5851
Author: Mark Hung <marklh9 at gmail.com>
Date:   Mon Jun 29 21:59:23 2015 +0800

    tdf#92281 - FILEOPEN: Only western font is set for symbols in DOC files
    
    Inserting a symbol in Writer set all fonts including CJK and CTL
    to the same one. This patch makes the behavior consistent with that,
    allows CJK and CTL user to see the symbols.
    
    Change-Id: I7a61868e8078c81c68e471760f080b3ce7df589f
    Reviewed-on: https://gerrit.libreoffice.org/16604
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Miklos Vajna <vmiklos at collabora.co.uk>
    Reviewed-on: https://gerrit.libreoffice.org/16871
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/sw/qa/extras/ww8export/data/tdf92281.doc b/sw/qa/extras/ww8export/data/tdf92281.doc
new file mode 100644
index 0000000..2893eb3
Binary files /dev/null and b/sw/qa/extras/ww8export/data/tdf92281.doc differ
diff --git a/sw/qa/extras/ww8export/ww8export.cxx b/sw/qa/extras/ww8export/ww8export.cxx
index e1626cd..cdf6d14 100644
--- a/sw/qa/extras/ww8export/ww8export.cxx
+++ b/sw/qa/extras/ww8export/ww8export.cxx
@@ -452,6 +452,14 @@ DECLARE_WW8EXPORT_TEST(testWw8Cjklist35, "cjklist35.doc")
     CPPUNIT_ASSERT_EQUAL(style::NumberingType::NUMBER_LOWER_ZH, numFormat);
 }
 
+DECLARE_WW8EXPORT_TEST(testTdf92281, "tdf92281.doc")
+{
+        uno::Reference<beans::XPropertySet> xRun(getRun(getParagraph(1),1), uno::UNO_QUERY);
+        CPPUNIT_ASSERT_EQUAL(OUString("Wingdings"), getProperty<OUString>(xRun, "CharFontName"));
+        CPPUNIT_ASSERT_EQUAL(OUString("Wingdings"), getProperty<OUString>(xRun, "CharFontNameAsian"));
+        CPPUNIT_ASSERT_EQUAL(OUString("Wingdings"), getProperty<OUString>(xRun, "CharFontNameComplex"));
+}
+
 DECLARE_WW8EXPORT_TEST(testCommentedTable, "commented-table.doc")
 {
     // Document has a non-trivial commented text range, as the range contains a table.
diff --git a/sw/source/filter/ww8/ww8par6.cxx b/sw/source/filter/ww8/ww8par6.cxx
index 267cd79..86281ac 100644
--- a/sw/source/filter/ww8/ww8par6.cxx
+++ b/sw/source/filter/ww8/ww8par6.cxx
@@ -2779,6 +2779,8 @@ void SwWW8ImplReader::Read_Symbol(sal_uInt16, const sal_uInt8* pData, short nLen
             //themselves so they don't need to add it to the stack either.
             if (SetNewFontAttr(SVBT16ToShort( pData ), false, RES_CHRATR_FONT))
             {
+                SetNewFontAttr(SVBT16ToShort( pData ), false, RES_CHRATR_CJK_FONT);
+                SetNewFontAttr(SVBT16ToShort( pData ), false, RES_CHRATR_CTL_FONT);
                 if( m_bVer67 )
                 {
                     //convert single byte from MS1252 to Unicode