summaryrefslogtreecommitdiff
path: root/sw
diff options
context:
space:
mode:
authorVasily Melenchuk <vasily.melenchuk@cib.de>2015-08-25 17:45:11 +0300
committerThorsten Behrens <Thorsten.Behrens@CIB.de>2015-08-28 09:52:08 +0000
commitfd3468024e1ac199f4a2f4108321ef8100d58414 (patch)
tree469bf725ba1978cdb879b7e01c5ae7d59b68c17a /sw
parent9e318c09778f0416143f211c5817536d5f1db3b7 (diff)
tdf#93449 Internal hyperlinks are not exported corectly in HTML
InetURLObject class does not work correctly with internal document links, containing not full url, but just a fragment. This case added as an exception for HTML link export. Change-Id: I44496a1cf186836d4194fc65ce7bf9aeb2f79b65 Reviewed-on: https://gerrit.libreoffice.org/17993 Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de> Tested-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
Diffstat (limited to 'sw')
-rw-r--r--sw/qa/extras/htmlexport/data/tdf90905.odtbin11824 -> 12166 bytes
-rw-r--r--sw/qa/extras/htmlexport/htmlexport.cxx3
-rw-r--r--sw/source/filter/html/wrthtml.cxx3
3 files changed, 4 insertions, 2 deletions
diff --git a/sw/qa/extras/htmlexport/data/tdf90905.odt b/sw/qa/extras/htmlexport/data/tdf90905.odt
index cab8a04fa12e..19a51bdf8fcf 100644
--- a/sw/qa/extras/htmlexport/data/tdf90905.odt
+++ b/sw/qa/extras/htmlexport/data/tdf90905.odt
Binary files differ
diff --git a/sw/qa/extras/htmlexport/htmlexport.cxx b/sw/qa/extras/htmlexport/htmlexport.cxx
index 77db88b1e95d..2ce1040b2d16 100644
--- a/sw/qa/extras/htmlexport/htmlexport.cxx
+++ b/sw/qa/extras/htmlexport/htmlexport.cxx
@@ -250,7 +250,8 @@ DECLARE_HTMLEXPORT_TEST(testExportInternalUrl, "tdf90905.odt")
CPPUNIT_ASSERT(pDoc);
// Internal url should be valid
- assertXPath(pDoc, "/html/body/p/a", "href", "#0.0.1.Text|outline");
+ assertXPath(pDoc, "/html/body/p[1]/a", "href", "#0.0.1.Text|outline");
+ assertXPath(pDoc, "/html/body/p[2]/a", "href", "#bookmark");
}
CPPUNIT_PLUGIN_IMPLEMENT();
diff --git a/sw/source/filter/html/wrthtml.cxx b/sw/source/filter/html/wrthtml.cxx
index 1502441a712f..4e5d57be4920 100644
--- a/sw/source/filter/html/wrthtml.cxx
+++ b/sw/source/filter/html/wrthtml.cxx
@@ -1198,8 +1198,9 @@ OUString SwHTMLWriter::convertHyperlinkHRefValue(const OUString& rURL)
}
}
}
- else
+ else if (!sURL.isEmpty() && sURL[0] != '#')
{
+ // Link is not started from "#", so looks like external link. Encode this URL.
INetURLObject aURL(sURL);
sURL = aURL.GetMainURL(INetURLObject::NO_DECODE);
}