X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2Finsets%2FInsetHyperlink.cpp;h=3f1131c4ad3a86cb12a882a909026e5f89e61e52;hb=997f0621c5346bb623cf86713c4fe1be0c941104;hp=8b3433c944b0b105afb84800cc1245b470df561e;hpb=670efa8f646218f2a378f0cc614c4c37a9f6b89a;p=lyx.git diff --git a/src/insets/InsetHyperlink.cpp b/src/insets/InsetHyperlink.cpp index 8b3433c944..3f1131c4ad 100644 --- a/src/insets/InsetHyperlink.cpp +++ b/src/insets/InsetHyperlink.cpp @@ -10,28 +10,31 @@ */ #include - #include "InsetHyperlink.h" #include "Buffer.h" -#include "DispatchResult.h" -#include "Encoding.h" #include "Format.h" #include "FuncRequest.h" #include "FuncStatus.h" #include "LaTeXFeatures.h" -#include "OutputParams.h" +#include "output_docbook.h" #include "output_xhtml.h" +#include "xml.h" #include "texstream.h" +#include "support/debug.h" #include "support/docstream.h" #include "support/FileName.h" #include "support/filetools.h" #include "support/gettext.h" #include "support/lstrings.h" +#include "support/qstring_helpers.h" #include "frontends/alert.h" +#include +#include + using namespace std; using namespace lyx::support; @@ -47,9 +50,11 @@ ParamInfo const & InsetHyperlink::findInfo(string const & /* cmdName */) { static ParamInfo param_info_; if (param_info_.empty()) { - param_info_.add("name", ParamInfo::LATEX_OPTIONAL); + param_info_.add("name", ParamInfo::LATEX_OPTIONAL, + ParamInfo::HANDLING_LATEXIFY); param_info_.add("target", ParamInfo::LATEX_REQUIRED); param_info_.add("type", ParamInfo::LATEX_REQUIRED); + param_info_.add("literal", ParamInfo::LYX_INTERNAL); } return param_info_; } @@ -57,23 +62,33 @@ ParamInfo const & InsetHyperlink::findInfo(string const & /* cmdName */) docstring InsetHyperlink::screenLabel() const { + // TODO: replace with unicode hyperlink character = U+1F517 docstring const temp = _("Hyperlink: "); docstring url; url += getParam("name"); - if (url.empty()) + if (url.empty()) { url += getParam("target"); - // elide if long - if (url.length() > 30) { - docstring end = url.substr(url.length() - 17, url.length()); - support::truncateWithEllipsis(url, 13); - url += end; + // elide if long and no name was provided + if (url.length() > 30) { + docstring end = url.substr(url.length() - 17, url.length()); + support::truncateWithEllipsis(url, 13); + url += end; + } + } else { + // elide if long (approx number of chars in line of article class) + if (url.length() > 80) { + docstring end = url.substr(url.length() - 67, url.length()); + support::truncateWithEllipsis(url, 13); + url += end; + } } return temp + url; } + void InsetHyperlink::doDispatch(Cursor & cur, FuncRequest & cmd) { switch (cmd.action()) { @@ -93,9 +108,13 @@ bool InsetHyperlink::getStatus(Cursor & cur, FuncRequest const & cmd, FuncStatus & flag) const { switch (cmd.action()) { - case LFUN_INSET_EDIT: - flag.setEnabled(getParam("type").empty() || getParam("type") == "file:"); + case LFUN_INSET_EDIT: { + QUrl url(toqstr(getParam("target")),QUrl::StrictMode); + bool url_valid = getParam("type").empty() && url.isValid(); + + flag.setEnabled(url_valid || getParam("type") == "file:"); return true; + } default: return InsetCommand::getStatus(cur, cmd, flag); @@ -105,10 +124,15 @@ bool InsetHyperlink::getStatus(Cursor & cur, FuncRequest const & cmd, void InsetHyperlink::viewTarget() const { - if (getParam("type") == "file:") { + if (getParam("type").empty()) { //==Web + QUrl url(toqstr(getParam("target")),QUrl::StrictMode); + if (!QDesktopServices::openUrl(url)) + LYXERR0("Unable to open URL!"); + + } else if (getParam("type") == "file:") { FileName url = makeAbsPath(to_utf8(getParam("target")), buffer().filePath()); - string const format = formats.getFormatFromFile(url); - formats.view(buffer(), url, format); + string const format = theFormats().getFormatFromFile(url); + theFormats().view(buffer(), url, format); } } @@ -118,23 +142,21 @@ void InsetHyperlink::latex(otexstream & os, { docstring url = getParam("target"); docstring name = getParam("name"); - static docstring const backslash = from_ascii("\\"); - static docstring const braces = from_ascii("{}"); static char_type const chars_url[2] = {'%', '#'}; - static char_type const chars_name[6] = { - '&', '_', '$', '%', '#', '^'}; // For the case there is no name given, the target is set as name. // Do this before !url.empty() and !name.empty() to handle characters - // like the "%" correctly. + // such as % correctly. if (name.empty()) name = url; if (!url.empty()) { - // Replace the "\" character by its ASCII code according to the - // URL specifications because "\" is not allowed in URLs and by - // \href. Only do this when the following character is not also - // a "\", because "\\" is valid code + // Use URI/URL-style percent-encoded string (hexadecimal). + // We exclude some characters that must not be transformed + // in hrefs: % # / : ? = & ! * ' ( ) ; @ + $ , [ ] + // or that we need to treat manually: \. + url = to_percent_encoding(url, from_ascii("%#\\/:?=&!*'();@+$,[]")); + // We handle \ manually since \\ is valid for (size_t i = 0, pos; (pos = url.find('\\', i)) != string::npos; i = pos + 2) { @@ -146,12 +168,13 @@ void InsetHyperlink::latex(otexstream & os, // field because otherwise LaTeX will fail when the hyperlink is // within an argument of another command, e.g. in a \footnote. It // is important that they are escaped as "\#" and not as "\#{}". + // FIXME this is not necessary in outside of commands. for (int k = 0; k < 2; k++) for (size_t i = 0, pos; (pos = url.find(chars_url[k], i)) != string::npos; i = pos + 2) - url.replace(pos, 1, backslash + chars_url[k]); - + url.replace(pos, 1, from_ascii("\\") + chars_url[k]); + // add "http://" when the type is web (type = empty) // and no "://" or "run:" is given docstring type = getParam("type"); @@ -162,47 +185,20 @@ void InsetHyperlink::latex(otexstream & os, } // end if (!url.empty()) - // The characters in chars_name[] need to be changed to a command when - // they are in the name field. if (!name.empty()) { - // handle the "\" character, but only when the following character - // is not also a "\", because "\\" is valid code - docstring const textbackslash = from_ascii("\\textbackslash{}"); - for (size_t i = 0, pos; - (pos = name.find('\\', i)) != string::npos; - i = pos + 2) { - if (name[pos + 1] != '\\') - name.replace(pos, 1, textbackslash); - } - // The characters in chars_name[] need to be changed to a command - // when they are in the name field. - // Therefore the treatment of "\" must be the first thing - for (int k = 0; k < 6; k++) - for (size_t i = 0, pos; - (pos = name.find(chars_name[k], i)) != string::npos; - i = pos + 2) - name.replace(pos, 1, backslash + chars_name[k] + braces); - + name = params().prepareCommand(runparams, name, + ParamInfo::HANDLING_LATEXIFY); // replace the tilde by the \sim character as suggested in the // LaTeX FAQ for URLs - docstring const sim = from_ascii("$\\sim$"); - for (size_t i = 0, pos; - (pos = name.find('~', i)) != string::npos; - i = pos + 1) - name.replace(pos, 1, sim); - pair name_latexed = - runparams.encoding->latexString(name, runparams.dryrun); - name = name_latexed.first; - if (!name_latexed.second.empty() && !runparams.silent) { - // issue a warning about omitted characters - // FIXME: should be passed to the error dialog - frontend::Alert::warning(_("Uncodable characters"), - bformat(_("The following characters that are used in the href inset are not\n" - "representable in the current encoding and therefore have been omitted:\n%1$s."), - name_latexed.second)); + if (getParam("literal") != from_ascii("true")) { + docstring const sim = from_ascii("$\\sim$"); + for (size_t i = 0, pos; + (pos = name.find('~', i)) != string::npos; + i = pos + 1) + name.replace(pos, 1, sim); } - } // end if (!name.empty()) - + } + if (runparams.moving_arg) os << "\\protect"; @@ -228,25 +224,22 @@ int InsetHyperlink::plaintext(odocstringstream & os, } -int InsetHyperlink::docbook(odocstream & os, OutputParams const &) const +void InsetHyperlink::docbook(XMLStream & xs, OutputParams const &) const { - os << "" - << getParam("name") - << ""; - return 0; + xs << xml::StartTag("link", "xlink:href=\"" + subst(getParam("target"), from_ascii("&"), from_ascii("&")) + "\""); + xs << xml::escapeString(getParam("name")); + xs << xml::EndTag("link"); } -docstring InsetHyperlink::xhtml(XHTMLStream & xs, OutputParams const &) const +docstring InsetHyperlink::xhtml(XMLStream & xs, OutputParams const &) const { - docstring const & target = - html::htmlize(getParam("target"), XHTMLStream::ESCAPE_AND); + docstring const & target = + xml::escapeString(getParam("target"), XMLStream::ESCAPE_AND); docstring const & name = getParam("name"); - xs << html::StartTag("a", to_utf8("href=\"" + target + "\"")); + xs << xml::StartTag("a", to_utf8("href=\"" + target + "\"")); xs << (name.empty() ? target : name); - xs << html::EndTag("a"); + xs << xml::EndTag("a"); return docstring(); } @@ -286,6 +279,14 @@ docstring InsetHyperlink::toolTip(BufferView const & /*bv*/, int /*x*/, int /*y* void InsetHyperlink::validate(LaTeXFeatures & features) const { features.require("hyperref"); + InsetCommand::validate(features); +} + + +pair InsetHyperlink::isWords() const +{ + docstring const label = getParam("name"); + return pair(label.size(), wordCount(label)); }