diff options
| author | Tibor Nagy <nagy.tibor2@nisz.hu> | 2020-12-29 08:36:49 +0100 |
|---|---|---|
| committer | Eike Rathke <erack@redhat.com> | 2021-01-05 14:33:16 +0100 |
| commit | 9165d5f834a6c831e8683a430bf5911eb4316b68 (patch) | |
| tree | 6e2933e86fd6f21801b24c6bcb73840db4b2a7f4 /sc | |
| parent | 8708529e1ba6f85388722d1441f0eef025018e33 (diff) | |
tdf#139394 XLSX export: remove extra quotation marks
This fixes commit 583e2bfba2d72ac8afe7261c23f380daf5486889
(tdf#139021 XLSX export: fix "contains" conditional formatting).
Change-Id: Idae3190b5f87dac551251da19c511f501bdaaa21
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108452
Tested-by: László Németh <nemeth@numbertext.org>
Reviewed-by: László Németh <nemeth@numbertext.org>
Reviewed-by: Eike Rathke <erack@redhat.com>
Diffstat (limited to 'sc')
| -rw-r--r-- | sc/qa/unit/data/xlsx/tdf139394.xlsx | bin | 0 -> 9778 bytes | |||
| -rw-r--r-- | sc/qa/unit/subsequent_export-test.cxx | 27 | ||||
| -rw-r--r-- | sc/source/filter/excel/xeextlst.cxx | 20 |
3 files changed, 37 insertions, 10 deletions
diff --git a/sc/qa/unit/data/xlsx/tdf139394.xlsx b/sc/qa/unit/data/xlsx/tdf139394.xlsx Binary files differnew file mode 100644 index 000000000000..eac83fb2f3a8 --- /dev/null +++ b/sc/qa/unit/data/xlsx/tdf139394.xlsx diff --git a/sc/qa/unit/subsequent_export-test.cxx b/sc/qa/unit/subsequent_export-test.cxx index 0261ea9f2dca..8f471b175148 100644 --- a/sc/qa/unit/subsequent_export-test.cxx +++ b/sc/qa/unit/subsequent_export-test.cxx @@ -89,6 +89,7 @@ public: virtual void tearDown() override; void test(); + void testTdf139394(); void testExtCondFormatXLSX(); void testTdf90104(); void testTdf111876(); @@ -280,6 +281,7 @@ public: CPPUNIT_TEST_SUITE(ScExportTest); CPPUNIT_TEST(test); + CPPUNIT_TEST(testTdf139394); CPPUNIT_TEST(testExtCondFormatXLSX); CPPUNIT_TEST(testTdf90104); CPPUNIT_TEST(testTdf111876); @@ -519,6 +521,31 @@ void ScExportTest::test() xDocSh->DoClose(); } +void ScExportTest::testTdf139394() +{ + ScDocShellRef xShell = loadDoc(u"tdf139394.", FORMAT_XLSX); + CPPUNIT_ASSERT(xShell.is()); + + ScDocShellRef xDocSh = saveAndReload(&(*xShell), FORMAT_XLSX); + CPPUNIT_ASSERT(xDocSh.is()); + + std::shared_ptr<utl::TempFile> pXPathFile = ScBootstrapFixture::exportTo(&(*xDocSh), FORMAT_XLSX); + xmlDocUniquePtr pDoc = XPathHelper::parseExport(pXPathFile, m_xSFactory, "xl/worksheets/sheet1.xml"); + CPPUNIT_ASSERT(pDoc); + + assertXPathContent(pDoc, + "/x:worksheet/x:extLst/x:ext/x14:conditionalFormattings/x14:conditionalFormatting[1]/" + "x14:cfRule/xm:f", "LEFT(A1,LEN(\"+\"))=\"+\""); + assertXPathContent(pDoc, + "/x:worksheet/x:extLst/x:ext/x14:conditionalFormattings/x14:conditionalFormatting[2]/" + "x14:cfRule/xm:f", "RIGHT(A2,LEN(\"-\"))=\"-\""); + assertXPathContent(pDoc, + "/x:worksheet/x:extLst/x:ext/x14:conditionalFormattings/x14:conditionalFormatting[3]/" + "x14:cfRule/xm:f", "LEFT(A3,LEN($B$3))=$B$3"); + + xDocSh->DoClose(); +} + void ScExportTest::testExtCondFormatXLSX() { ScDocShellRef xShell = loadDoc(u"tdf139021.", FORMAT_XLSX); diff --git a/sc/source/filter/excel/xeextlst.cxx b/sc/source/filter/excel/xeextlst.cxx index 972a9010b342..1f0d8fcc942d 100644 --- a/sc/source/filter/excel/xeextlst.cxx +++ b/sc/source/filter/excel/xeextlst.cxx @@ -175,16 +175,16 @@ OString GetFixedFormula(ScConditionMode eMode, const ScAddress& rAddress, std::s OString aPos = aBuffer.makeStringAndClear(); switch (eMode) { - case ScConditionMode::BeginsWith: - return OString("LEFT(" + aPos + ",LEN(" + rText + "))=\"" + rText + "\""); - case ScConditionMode::EndsWith: - return OString("RIGHT(" + aPos + ",LEN(" + rText + "))=\"" + rText + "\""); - case ScConditionMode::ContainsText: - return OString(OString::Concat("NOT(ISERROR(SEARCH(") + rText + "," + aPos + ")))"); - case ScConditionMode::NotContainsText: - return OString(OString::Concat("ISERROR(SEARCH(") + rText + "," + aPos + "))"); - default: - break; + case ScConditionMode::BeginsWith: + return OString("LEFT(" + aPos + ",LEN(" + rText + "))=" + rText); + case ScConditionMode::EndsWith: + return OString("RIGHT(" + aPos + ",LEN(" + rText + "))=" + rText); + case ScConditionMode::ContainsText: + return OString(OString::Concat("NOT(ISERROR(SEARCH(") + rText + "," + aPos + ")))"); + case ScConditionMode::NotContainsText: + return OString(OString::Concat("ISERROR(SEARCH(") + rText + "," + aPos + "))"); + default: + break; } return ""; |