|
 |
a9add1 |
From b5d3739516032e8863ee60e0c614dd1ec9e262a8 Mon Sep 17 00:00:00 2001
|
|
|
a9add1 |
From: =?UTF-8?q?Caol=C3=A1n=20McNamara?= <caolanm@redhat.com>
|
|
|
a9add1 |
Date: Wed, 27 Apr 2016 14:50:05 +0100
|
|
|
a9add1 |
Subject: [PATCH] Resolves: tdf#99498 don't overwrite trwWidth value if set on
|
|
|
a9add1 |
a table
|
|
|
a9add1 |
|
|
|
a9add1 |
Change-Id: Ia39da999a2de1ca2b8bec5fc7f35d2f9ffe2dd19
|
|
|
a9add1 |
---
|
|
|
a9add1 |
sw/qa/extras/rtfimport/rtfimport.cxx | 10 ++++++++++
|
|
|
a9add1 |
writerfilter/source/rtftok/rtfdocumentimpl.cxx | 18 ++++++++++++++----
|
|
|
a9add1 |
2 files changed, 24 insertions(+), 4 deletions(-)
|
|
|
a9add1 |
|
|
|
a9add1 |
diff --git a/sw/qa/extras/rtfimport/rtfimport.cxx b/sw/qa/extras/rtfimport/rtfimport.cxx
|
|
|
a9add1 |
index 4236a48..07dd0da 100644
|
|
|
a9add1 |
--- a/sw/qa/extras/rtfimport/rtfimport.cxx
|
|
|
a9add1 |
+++ b/sw/qa/extras/rtfimport/rtfimport.cxx
|
|
|
a9add1 |
@@ -2388,6 +2388,16 @@ DECLARE_RTFIMPORT_TEST(testTdf96308Tabpos, "tdf96308-tabpos.rtf")
|
|
|
a9add1 |
CPPUNIT_ASSERT(!aTabStops.hasElements());
|
|
|
a9add1 |
}
|
|
|
a9add1 |
|
|
|
a9add1 |
+DECLARE_RTFIMPORT_TEST(testTdf99498, "tdf99498.rtf")
|
|
|
a9add1 |
+{
|
|
|
a9add1 |
+ uno::Reference<text::XTextTablesSupplier> xTextTablesSupplier(mxComponent, uno::UNO_QUERY);
|
|
|
a9add1 |
+ uno::Reference<container::XIndexAccess> xTables(xTextTablesSupplier->getTextTables(), uno::UNO_QUERY);
|
|
|
a9add1 |
+
|
|
|
a9add1 |
+ // Table width should be 7056
|
|
|
a9add1 |
+ CPPUNIT_ASSERT_EQUAL(sal_Int32(7056), getProperty<sal_Int32>(xTables->getByIndex(0), "Width"));
|
|
|
a9add1 |
+}
|
|
|
a9add1 |
+
|
|
|
a9add1 |
+
|
|
|
a9add1 |
DECLARE_RTFIMPORT_TEST(testTdf87034, "tdf87034.rtf")
|
|
|
a9add1 |
{
|
|
|
a9add1 |
// This was A1BC34D, i.e. the first "super" text portion was mis-imported,
|
|
|
a9add1 |
diff --git a/writerfilter/source/rtftok/rtfdocumentimpl.cxx b/writerfilter/source/rtftok/rtfdocumentimpl.cxx
|
|
|
a9add1 |
index 799186d..f292e9d 100644
|
|
|
a9add1 |
--- a/writerfilter/source/rtftok/rtfdocumentimpl.cxx
|
|
|
a9add1 |
+++ b/writerfilter/source/rtftok/rtfdocumentimpl.cxx
|
|
|
a9add1 |
@@ -1368,10 +1368,14 @@ void RTFDocumentImpl::prepareProperties(
|
|
|
a9add1 |
}
|
|
|
a9add1 |
|
|
|
a9add1 |
// Table width.
|
|
|
a9add1 |
- auto pUnitValue = std::make_shared<RTFValue>(3);
|
|
|
a9add1 |
- lcl_putNestedAttribute(rState.aTableRowSprms, NS_ooxml::LN_CT_TblPrBase_tblW, NS_ooxml::LN_CT_TblWidth_type, pUnitValue);
|
|
|
a9add1 |
- auto pWValue = std::make_shared<RTFValue>(nCurrentCellX);
|
|
|
a9add1 |
- lcl_putNestedAttribute(rState.aTableRowSprms, NS_ooxml::LN_CT_TblPrBase_tblW, NS_ooxml::LN_CT_TblWidth_w, pWValue);
|
|
|
a9add1 |
+ RTFValue::Pointer_t const pTableWidthProps = rState.aTableRowSprms.find(NS_ooxml::LN_CT_TblPrBase_tblW);
|
|
|
a9add1 |
+ if (!pTableWidthProps.get())
|
|
|
a9add1 |
+ {
|
|
|
a9add1 |
+ auto pUnitValue = std::make_shared<RTFValue>(3);
|
|
|
a9add1 |
+ lcl_putNestedAttribute(rState.aTableRowSprms, NS_ooxml::LN_CT_TblPrBase_tblW, NS_ooxml::LN_CT_TblWidth_type, pUnitValue);
|
|
|
a9add1 |
+ auto pWValue = std::make_shared<RTFValue>(nCurrentCellX);
|
|
|
a9add1 |
+ lcl_putNestedAttribute(rState.aTableRowSprms, NS_ooxml::LN_CT_TblPrBase_tblW, NS_ooxml::LN_CT_TblWidth_w, pWValue);
|
|
|
a9add1 |
+ }
|
|
|
a9add1 |
|
|
|
a9add1 |
auto pRowValue = std::make_shared<RTFValue>(1);
|
|
|
a9add1 |
if (nCells > 0)
|
|
|
a9add1 |
@@ -4783,6 +4787,12 @@ RTFError RTFDocumentImpl::dispatchValue(RTFKeyword nKeyword, int nParam)
|
|
|
a9add1 |
lcl_putNestedSprm(m_aStates.top().aTableRowSprms, NS_ooxml::LN_CT_TblPrBase_tblCellMar, NS_ooxml::LN_CT_TblCellMar_right, std::make_shared<RTFValue>(aAttributes));
|
|
|
a9add1 |
}
|
|
|
a9add1 |
break;
|
|
|
a9add1 |
+ case RTF_TRFTSWIDTH:
|
|
|
a9add1 |
+ lcl_putNestedAttribute(m_aStates.top().aTableRowSprms, NS_ooxml::LN_CT_TblPrBase_tblW, NS_ooxml::LN_CT_TblWidth_type, pIntValue);
|
|
|
a9add1 |
+ break;
|
|
|
a9add1 |
+ case RTF_TRWWIDTH:
|
|
|
a9add1 |
+ lcl_putNestedAttribute(m_aStates.top().aTableRowSprms, NS_ooxml::LN_CT_TblPrBase_tblW, NS_ooxml::LN_CT_TblWidth_w, pIntValue);
|
|
|
a9add1 |
+ break;
|
|
|
a9add1 |
default:
|
|
|
a9add1 |
{
|
|
|
a9add1 |
SAL_INFO("writerfilter", "TODO handle value '" << lcl_RtfToString(nKeyword) << "'");
|
|
|
a9add1 |
--
|
|
|
a9add1 |
2.7.3
|
|
|
a9add1 |
|