diff options
| author | Attila Szűcs <szucs.attila3@nisz.hu> | 2020-12-21 23:54:43 +0100 |
|---|---|---|
| committer | László Németh <nemeth@numbertext.org> | 2020-12-23 11:16:03 +0100 |
| commit | 610153dd28b57ea2670f5cf240e4da9655fb7093 (patch) | |
| tree | 9b9fd8cc9b08b53aa220e59b43273ab6ec37dba6 /sc/qa | |
| parent | cf5715da45ddce8b667f1b999d41c4e6e7e65659 (diff) | |
tdf#121472 XLSX export: fix external Linux path
Removed the extra 4th '/' after 'file:///' from
aData.maBasePath at exportDocument().
In Linux, the root path could start with '/'.. like /home/...
in that case 'file:///'+'/home/..' resulted 'file:////home/ that
is invalid.
Co-authored-by: Tibor Nagy (NISZ)
Change-Id: Ia37a11def80df2747578807ee021fafe38d48ec0
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108131
Tested-by: Jenkins
Reviewed-by: László Németh <nemeth@numbertext.org>
Diffstat (limited to 'sc/qa')
| -rw-r--r-- | sc/qa/unit/subsequent_export-test.cxx | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/sc/qa/unit/subsequent_export-test.cxx b/sc/qa/unit/subsequent_export-test.cxx index bcfa926c9314..e29e7a5ee648 100644 --- a/sc/qa/unit/subsequent_export-test.cxx +++ b/sc/qa/unit/subsequent_export-test.cxx @@ -5636,11 +5636,9 @@ void ScExportTest::testTdf138824_linkToParentDirectory() pXPathFile, m_xSFactory, "xl/externalLinks/_rels/externalLink1.xml.rels"); CPPUNIT_ASSERT(pDoc); - // it should be "../tdf138824_externalSource.ods" but because of another bug, - // on linux some other directory names may be added into the middle - OUString aValue = getXPath(pDoc, "/r:Relationships/r:Relationship", "Target"); - CPPUNIT_ASSERT_EQUAL(0, static_cast<int>(aValue.indexOf("../"))); - CPPUNIT_ASSERT(aValue.indexOf("/tdf138824_externalSource.ods") > 0); + // test also the Linux specific bug tdf#121472 + assertXPath(pDoc, "/r:Relationships/r:Relationship", "Target", + "../tdf138824_externalSource.ods"); xDocSh->DoClose(); } |