]> git.lyx.org Git - lyx.git/blobdiff - src/insets/InsetHyperlink.cpp
Well, it turns out that we need a different return value for the xhtml
[lyx.git] / src / insets / InsetHyperlink.cpp
index d38509b60e7943527ed7b3a3cfb0df960d8337c1..aa4ff3af7d784655d07bacb73bb8e753945067b6 100644 (file)
@@ -3,8 +3,8 @@
  * This file is part of LyX, the document processor.
  * Licence details can be found in the file COPYING.
  *
- * \author José Matos
- * \author Uwe Stöhr
+ * \author José Matos
+ * \author Uwe Stöhr
  *
  * Full author contact details are available in file CREDITS.
  */
 #include "DispatchResult.h"
 #include "FuncRequest.h"
 #include "LaTeXFeatures.h"
-#include "gettext.h"
 #include "OutputParams.h"
 
-#include "support/lstrings.h"
 #include "support/docstream.h"
+#include "support/gettext.h"
+#include "support/lstrings.h"
 
-using std::string;
-using std::find;
-using std::replace;
+using namespace std;
+using namespace lyx::support;
 
 namespace lyx {
 
-using support::subst;
-
 
 InsetHyperlink::InsetHyperlink(InsetCommandParams const & p)
        : InsetCommand(p, "href")
 {}
 
 
-CommandInfo const * InsetHyperlink::findInfo(string const & /* cmdName */)
+ParamInfo const & InsetHyperlink::findInfo(string const & /* cmdName */)
 {
-       static const char * const paramnames[] =
-               {"name", "target", "type", ""};
-       static const bool isoptional[] = {true, false};
-       static const CommandInfo info = {3, paramnames, isoptional};
-       return &info;
+       static ParamInfo param_info_;
+       if (param_info_.empty()) {
+               param_info_.add("name", ParamInfo::LATEX_OPTIONAL);
+               param_info_.add("target", ParamInfo::LATEX_REQUIRED);
+               param_info_.add("type", ParamInfo::LATEX_REQUIRED);
+       }
+       return param_info_;
 }
 
 
-docstring const InsetHyperlink::getScreenLabel(Buffer const &) const
+docstring InsetHyperlink::screenLabel() const
 {
        docstring const temp = from_ascii("Hyperlink: ");
 
@@ -65,55 +64,80 @@ docstring const InsetHyperlink::getScreenLabel(Buffer const &) const
 }
 
 
-int InsetHyperlink::latex(Buffer const &, odocstream & os,
-                   OutputParams const & runparams) const
+int InsetHyperlink::latex(odocstream & os,
+                                                 OutputParams const & runparams) const
 {
        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] = {
                '&', '_', '$', '%', '#', '^'};
 
-       // The characters in chars_url[] need to be changed to a command when
-       // they are in the url field.
+       // 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.
+       if (name.empty())
+               name = url;
+
        if (!url.empty()) {
-               // the chars_url[] characters must be handled for both, url and href
-               for (int k = 0; k < 2; k++) {
+               // 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
+               for (size_t i = 0, pos;
+                       (pos = url.find('\\', i)) != string::npos;
+                       i = pos + 2) {
+                       if (url[pos + 1] != '\\')
+                               url.replace(pos, 1, from_ascii("%5C"));
+               }
+
+               // The characters in chars_url[] need to be escaped in the url
+               // 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 "\#{}".
+               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) {
+                               i = pos + 2)
                                url.replace(pos, 1, backslash + chars_url[k]);
-                       }
-               }
-       } // end if (!url.empty())
+               
+               // add "http://" when the type is web (type = empty)
+               // and no "://" or "run:" is given
+               docstring type = getParam("type");
+               if (url.find(from_ascii("://")) == string::npos
+                       && url.find(from_ascii("run:")) == string::npos
+                       && type.empty())
+                       url = from_ascii("http://") + url;
 
-       docstring name = getParam("name");
+       } // 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] != '\\')
+                       if (name[pos + 1] != '\\')
                                name.replace(pos, 1, textbackslash);
                }
-               for (int k = 0; k < 6; k++) {
+               // 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) {
+                               i = pos + 2)
                                name.replace(pos, 1, backslash + chars_name[k] + braces);
-                       }
-               }
-               // replace the tilde by the \sim character as suggested in the LaTeX FAQ
-               // for URLs
+
+               // replace the tilde by the \sim character as suggested in the
+               // LaTeX FAQ for URLs
                docstring const sim = from_ascii("$\\sim$");
-               for (int i = 0, pos;
+               for (size_t i = 0, pos;
                        (pos = name.find('~', i)) != string::npos;
                        i = pos + 1)
                        name.replace(pos, 1, sim);
@@ -123,16 +147,14 @@ int InsetHyperlink::latex(Buffer const &, odocstream & os,
        if (runparams.moving_arg)
                os << "\\protect";
 
-       //for the case there is no name given, the target is set as name
-       os << "\\href{" << getParam("type") << url << "}{"
-               << (name.empty()? url : name) << '}';
+       // output the ready \href command
+       os << "\\href{" << getParam("type") << url << "}{" << name << '}';
 
        return 0;
 }
 
 
-int InsetHyperlink::plaintext(Buffer const &, odocstream & os,
-                       OutputParams const &) const
+int InsetHyperlink::plaintext(odocstream & os, OutputParams const &) const
 {
        odocstringstream oss;
 
@@ -148,8 +170,7 @@ int InsetHyperlink::plaintext(Buffer const &, odocstream & os,
 }
 
 
-int InsetHyperlink::docbook(Buffer const &, odocstream & os,
-                     OutputParams const &) const
+int InsetHyperlink::docbook(odocstream & os, OutputParams const &) const
 {
        os << "<ulink url=\""
           << subst(getParam("target"), from_ascii("&"), from_ascii("&amp;"))
@@ -160,10 +181,21 @@ int InsetHyperlink::docbook(Buffer const &, odocstream & os,
 }
 
 
-int InsetHyperlink::textString(Buffer const & buf, odocstream & os,
-                      OutputParams const & op) const
+docstring InsetHyperlink::xhtml(odocstream & os, OutputParams const &) const
+{
+       os << "<a href=\""
+                       // FIXME Do we need to do more escaping than this?
+          << subst(getParam("target"), from_ascii("&"), from_ascii("&amp;"))
+          << "\">"
+          << getParam("name")
+          << "</a>";
+       return docstring();
+}
+
+
+void InsetHyperlink::tocString(odocstream & os) const
 {
-       return plaintext(buf, os, op);
+       plaintext(os, OutputParams(0));
 }