From ebe58344ba4bf6ad7de49564ebf10940d7636176 Mon Sep 17 00:00:00 2001 From: Dennis Francis Date: Tue, 15 Aug 2017 11:21:04 +0530 Subject: tdf#83562: Make the EditTextObject export of automatic colors... ... use the xml token "use-window-font-color" instead of "color" to encode a boolean value of whether an auto color is used or not. This is because the "auto" color is 0xffffffff and cannot be written into "color" xml field which can store only 3 byte-hexcode. This commit also adds a unit test in subsequent_export-test.cxx to ensure the correct export of automatic color. Change-Id: I42aab926f31669c1423bc09b96b45c410c9732de Reviewed-on: https://gerrit.libreoffice.org/41252 Tested-by: Jenkins Reviewed-by: Dennis Francis (cherry picked from commit 06d104ecda085200fb6d22e0bd6d6cb89ca4f66c) Reviewed-on: https://gerrit.libreoffice.org/41358 Reviewed-by: Michael Stahl (cherry picked from commit 1b4c5f66083a92f4bfc9bd20a2a5b1e863ec84c1) Reviewed-on: https://gerrit.libreoffice.org/41467 Reviewed-by: Eike Rathke Reviewed-by: Christian Lohmaier Tested-by: Christian Lohmaier --- sc/qa/unit/subsequent_export-test.cxx | 67 ++++++++++++++++++++++++++++++++++- sc/source/filter/xml/xmlexprt.cxx | 8 ++++- 2 files changed, 73 insertions(+), 2 deletions(-) diff --git a/sc/qa/unit/subsequent_export-test.cxx b/sc/qa/unit/subsequent_export-test.cxx index fb4d496243bf..26ac932a7765 100644 --- a/sc/qa/unit/subsequent_export-test.cxx +++ b/sc/qa/unit/subsequent_export-test.cxx @@ -26,6 +26,7 @@ #include "userdat.hxx" #include "docsh.hxx" #include "patattr.hxx" +#include "docpool.hxx" #include "scitems.hxx" #include "document.hxx" #include "cellform.hxx" @@ -61,6 +62,7 @@ #include #include #include +#include #include #include #include @@ -1029,7 +1031,7 @@ void ScExportTest::testMiscRowHeightExport() namespace { -void setAttribute( ScFieldEditEngine& rEE, sal_Int32 nPara, sal_Int32 nStart, sal_Int32 nEnd, sal_uInt16 nType ) +void setAttribute( ScFieldEditEngine& rEE, sal_Int32 nPara, sal_Int32 nStart, sal_Int32 nEnd, sal_uInt16 nType, sal_uInt32 nColor = COL_BLACK ) { ESelection aSel; aSel.nStartPara = aSel.nEndPara = nPara; @@ -1074,6 +1076,13 @@ void setAttribute( ScFieldEditEngine& rEE, sal_Int32 nPara, sal_Int32 nStart, sa rEE.QuickSetAttribs(aItemSet, aSel); } break; + case EE_CHAR_COLOR: + { + SvxColorItem aItem(nColor, nType); + aItemSet.Put(aItem); + rEE.QuickSetAttribs(aItemSet, aSel); + } + break; default: ; } @@ -1256,6 +1265,23 @@ void ScExportTest::testRichTextExportODS() return false; } + static bool isColor(const editeng::Section& rAttr, sal_uInt32 nColor) + { + if (rAttr.maAttributes.empty()) + return false; + + std::vector::const_iterator it = rAttr.maAttributes.begin(), itEnd = rAttr.maAttributes.end(); + for (; it != itEnd; ++it) + { + const SfxPoolItem* p = *it; + if (p->Which() != EE_CHAR_COLOR) + continue; + + return static_cast(p)->GetValue() == nColor; + } + return false; + } + bool checkB2(const EditTextObject* pText) const { if (!pText) @@ -1482,6 +1508,32 @@ void ScExportTest::testRichTextExportODS() return true; } + bool checkB10(const EditTextObject* pText) const + { + if (!pText) + return false; + + if (pText->GetParagraphCount() != 1) + return false; + + if (pText->GetText(0) != "BLUE AUTO") + return false; + + std::vector aSecAttrs; + pText->GetAllSections(aSecAttrs); + if (aSecAttrs.size() != 2) + return false; + + // auto color + const editeng::Section* pAttr = &aSecAttrs[1]; + if (pAttr->mnParagraph != 0 ||pAttr->mnStart != 5 || pAttr->mnEnd != 9) + return false; + + if (pAttr->maAttributes.size() != 1 || !isColor(*pAttr, COL_AUTO)) + return false; + + return true; + } } aCheckFunc; @@ -1583,6 +1635,17 @@ void ScExportTest::testRichTextExportODS() pEditText = rDoc3.GetEditText(ScAddress(1,8,0)); CPPUNIT_ASSERT_MESSAGE("Incorrect B9 value.", aCheckFunc.checkB9(pEditText)); + ScPatternAttr aCellFontColor(rDoc3.GetPool()); + aCellFontColor.GetItemSet().Put(SvxColorItem(COL_BLUE, ATTR_FONT_COLOR)); + // Set font color of B10 to blue. + rDoc3.ApplyPattern(1, 9, 0, aCellFontColor); + pEE->Clear(); + pEE->SetText("BLUE AUTO"); + // Set the color of the string "AUTO" to automatic color. + setAttribute(*pEE, 0, 5, 9, EE_CHAR_COLOR, COL_AUTO); + rDoc3.SetEditText(ScAddress(1, 9, 0), pEE->CreateTextObject()); + pEditText = rDoc3.GetEditText(ScAddress(1, 9, 0)); + CPPUNIT_ASSERT_MESSAGE("Incorrect B10 value.", aCheckFunc.checkB10(pEditText)); } // Reload the doc again, and check the content of B2, B4, B6 and B7. @@ -1602,6 +1665,8 @@ void ScExportTest::testRichTextExportODS() CPPUNIT_ASSERT_MESSAGE("Incorrect B7 value after save and reload.", aCheckFunc.checkB7(pEditText)); pEditText = rDoc4.GetEditText(ScAddress(1,7,0)); CPPUNIT_ASSERT_MESSAGE("Incorrect B8 value after save and reload.", aCheckFunc.checkB8(pEditText)); + pEditText = rDoc4.GetEditText(ScAddress(1,9,0)); + CPPUNIT_ASSERT_MESSAGE("Incorrect B10 value after save and reload.", aCheckFunc.checkB10(pEditText)); xNewDocSh3->DoClose(); } diff --git a/sc/source/filter/xml/xmlexprt.cxx b/sc/source/filter/xml/xmlexprt.cxx index d7e4ef5b8ce6..7323686bd368 100644 --- a/sc/source/filter/xml/xmlexprt.cxx +++ b/sc/source/filter/xml/xmlexprt.cxx @@ -1101,7 +1101,13 @@ const SvxFieldData* toXMLPropertyStates( if (!static_cast(p)->QueryValue(aAny, pEntry->mnFlag)) continue; - rPropStates.push_back(XMLPropertyState(nIndex, aAny)); + sal_uInt32 nColor = 0; + if ( aAny >>= nColor ) + { + sal_Int32 nIndexColor = ( nColor == COL_AUTO ) ? xMapper->GetEntryIndex( + XML_NAMESPACE_STYLE, GetXMLToken( XML_USE_WINDOW_FONT_COLOR ), 0 ) : nIndex; + rPropStates.push_back( XMLPropertyState( nIndexColor, aAny ) ); + } } break; case EE_CHAR_WLM: -- cgit v1.2.3