validate encodable characters in href inset (#8357)

[lyx.git] / src / insets / InsetHyperlink.cpp

diff --git a/src/insets/InsetHyperlink.cpp b/src/insets/InsetHyperlink.cpp
index e85fb4558a39615a0a7f174e70fcc9b1f554b91a..37692af97a634c65244245908861036357a14206 100644 (file)
--- a/src/insets/InsetHyperlink.cpp
+++ b/src/insets/InsetHyperlink.cpp
@@ -13,15 +13,24 @@
 
 #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_xhtml.h"
 
 #include "support/docstream.h"
+#include "support/FileName.h"
+#include "support/filetools.h"
 #include "support/gettext.h"
 #include "support/lstrings.h"
 
+#include "frontends/alert.h"
+
 using namespace std;
 using namespace lyx::support;
 
@@ -29,7 +38,7 @@ namespace lyx {
 
 
 InsetHyperlink::InsetHyperlink(Buffer * buf, InsetCommandParams const & p)
-       : InsetCommand(buf, p, "href")
+       : InsetCommand(buf, p)
 {}
 
 
@@ -63,9 +72,47 @@ docstring InsetHyperlink::screenLabel() const
        return temp + url;
 }
 
+void InsetHyperlink::doDispatch(Cursor & cur, FuncRequest & cmd)
+{
+       switch (cmd.action()) {
+
+       case LFUN_INSET_EDIT:
+               viewTarget();
+               break;
+
+       default:
+               InsetCommand::doDispatch(cur, cmd);
+               break;
+       }
+}
+
 
-int InsetHyperlink::latex(odocstream & os,
-                         OutputParams const & runparams) const
+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:");
+               return true;
+
+       default:
+               return InsetCommand::getStatus(cur, cmd, flag);
+       }
+}
+
+
+void InsetHyperlink::viewTarget() const
+{
+       if (getParam("type") == "file:") {
+               FileName url = makeAbsPath(to_utf8(getParam("target")), buffer().filePath());
+               string const format = formats.getFormatFromFile(url);
+               formats.view(buffer(), url, format);
+       }
+}
+
+
+void InsetHyperlink::latex(otexstream & os,
+                          OutputParams const & runparams) const
 {
        docstring url = getParam("target");
        docstring name = getParam("name");
@@ -141,7 +188,17 @@ int InsetHyperlink::latex(odocstream & os,
                        (pos = name.find('~', i)) != string::npos;
                        i = pos + 1)
                        name.replace(pos, 1, sim);
-
+               pair<docstring, docstring> name_latexed =
+                       runparams.encoding->latexString(name, runparams.dryrun);
+               name = name_latexed.first;
+               if (!name_latexed.second.empty()) {
+                       // 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));
+               }
        }  // end if (!name.empty())
        
        if (runparams.moving_arg)
@@ -149,8 +206,6 @@ int InsetHyperlink::latex(odocstream & os,
 
        // output the ready \href command
        os << "\\href{" << getParam("type") << url << "}{" << name << '}';
-
-       return 0;
 }
 
 
@@ -183,7 +238,8 @@ int InsetHyperlink::docbook(odocstream & os, OutputParams const &) const
 
 docstring InsetHyperlink::xhtml(XHTMLStream & xs, OutputParams const &) const
 {
-       docstring const & target = getParam("target");
+       docstring const & target = 
+               html::htmlize(getParam("target"), XHTMLStream::ESCAPE_AND);
        docstring const & name   = getParam("name");
        xs << html::StartTag("a", to_utf8("href=\"" + target + "\""));
        xs << (name.empty() ? target : name);
@@ -192,12 +248,23 @@ docstring InsetHyperlink::xhtml(XHTMLStream & xs, OutputParams const &) const
 }
 
 
-void InsetHyperlink::tocString(odocstream & os) const
+void InsetHyperlink::toString(odocstream & os) const
 {
        plaintext(os, OutputParams(0));
 }
 
 
+void InsetHyperlink::forToc(docstring & os, size_t) const
+{
+       docstring const & n = getParam("name");
+       if (!n.empty()) {
+               os += n;
+               return;
+       }
+       os += getParam("target");
+}
+
+
 docstring InsetHyperlink::toolTip(BufferView const & /*bv*/, int /*x*/, int /*y*/) const
 {
        docstring url = getParam("target");
@@ -217,9 +284,9 @@ void InsetHyperlink::validate(LaTeXFeatures & features) const
 }
 
 
-docstring InsetHyperlink::contextMenu(BufferView const &, int, int) const
+string InsetHyperlink::contextMenuName() const
 {
-       return from_ascii("context-hyperlink");
+       return "context-hyperlink";
 }