Hi,

See
http://cgit.freedesktop.org/libreoffice/core/commit/?id=e8706d7

Regression from 3.4; backported patch attached.

(v1.9.1 of the spec on page 23 states this detail that 0 means
"automatic" color, which is the Writer default.)

Thanks,

Miklos
>From fe203cb75f2529b4bb15a49c8f0b22eba9e55bda Mon Sep 17 00:00:00 2001
From: Miklos Vajna <vmik...@suse.cz>
Date: Wed, 2 May 2012 17:50:55 +0200
Subject: [PATCH] fdo#47764 fix RTF import of automatic paragraph background
 color

Change-Id: I4b3a4bd6ce45cffc5b2c23052c49ff98579c5564
---
 writerfilter/source/rtftok/rtfdocumentimpl.cxx |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/writerfilter/source/rtftok/rtfdocumentimpl.cxx b/writerfilter/source/rtftok/rtfdocumentimpl.cxx
index d65b642..f36e763 100644
--- a/writerfilter/source/rtftok/rtfdocumentimpl.cxx
+++ b/writerfilter/source/rtftok/rtfdocumentimpl.cxx
@@ -2373,6 +2373,7 @@ int RTFDocumentImpl::dispatchValue(RTFKeyword nKeyword, int nParam)
             }
             break;
         case RTF_CBPAT:
+            if (nParam)
             {
                 RTFValue::Pointer_t pValue(new RTFValue(getColorTable(nParam)));
                 lcl_putNestedAttribute(m_aStates.top().aParagraphSprms, NS_sprm::LN_PShd, NS_ooxml::LN_CT_Shd_fill, pValue);
-- 
1.7.7

_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to