X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2FPDFOptions.cpp;h=644f26e738bcfe49523dfd5991c8ece4d0b6c559;hb=99f2abf59fc88faeac69df6a64d0d2a3615c8972;hp=8abb0d98989bf0a13871782f2686c3bdee020f3b;hpb=eb885931548124e98dbfe72bdc83666aec681559;p=lyx.git

diff --git a/src/PDFOptions.cpp b/src/PDFOptions.cpp
index 8abb0d9898..644f26e738 100644
--- a/src/PDFOptions.cpp
+++ b/src/PDFOptions.cpp
@@ -12,20 +12,20 @@
 
 #include "PDFOptions.h"
 
+#include "Lexer.h"
+
 #include "support/convert.h"
+#include "support/debug.h"
 #include "support/lstrings.h"
-#include "debug.h"
-#include "Lexer.h"
 
 #include <sstream>
 #include <string>
 
-namespace lyx {
+using namespace std;
+using namespace lyx::support;
 
+namespace lyx {
 
-using std::ostream;
-using std::ostringstream;
-using std::string;
 
 const string PDFOptions::pagemode_fullscreen("FullScreen");
 
@@ -51,6 +51,7 @@ bool PDFOptions::empty() const
 		&& pdfusetitle == x.pdfusetitle;
 }
 
+
 void PDFOptions::writeFile(ostream & os) const
 {
 	os << "\\use_hyperref " << convert<string>(use_hyperref) << '\n';
@@ -86,14 +87,11 @@ void PDFOptions::writeFile(ostream & os) const
 		os << "\\pdf_quoted_options \"" << quoted_options << "\"\n";
 }
 
-void PDFOptions::writeLaTeX(odocstringstream &os) const
+
+void PDFOptions::writeLaTeX(odocstream & os, bool hyperref_already_provided) const
 {
-	if (!use_hyperref)
-		return;
-	
 	string opt;
 	
-	opt = "\\usepackage[";
 	// since LyX uses unicode, also set the PDF strings to unicode strings with the
 	// hyperref option "unicode"
 	opt += "unicode=true, ";
@@ -125,8 +123,7 @@ void PDFOptions::writeLaTeX(odocstringstream &os) const
 	if (!pagemode.empty())
 		opt += "pdfpagemode=" + pagemode + ',';
 	
-	opt = support::rtrim(opt,",");
-	opt += "]\n {hyperref}\n";
+
 
 	// load the pdftitle etc. as hypersetup, otherwise you'll get
 	// LaTeX-errors when using non-latin characters
@@ -141,11 +138,28 @@ void PDFOptions::writeLaTeX(odocstringstream &os) const
 		hyperset += "\n pdfkeywords={" + keywords + "},";
 	if (!quoted_options.empty()){
 		hyperset += "\n ";
-		hyperset += quoted_options_get();
+		hyperset += quoted_options;
 	}
-	hyperset = support::rtrim(hyperset,",");
-	if (!hyperset.empty())
-		opt += "\\hypersetup{" + hyperset + "}\n ";
+	hyperset = rtrim(hyperset,",");
+
+
+	// use in \\usepackage parameter as not all options can be handled inside \\hypersetup
+	if (!hyperref_already_provided) {
+		opt = rtrim(opt,",");
+		opt = "\\usepackage[" + opt + "]\n {hyperref}\n";
+
+		if (!hyperset.empty())
+			opt += "\\hypersetup{" + hyperset + "}\n ";
+	} else
+		// only in case hyperref is already loaded by the current text class
+		// try to put it into hyperset
+		//
+		// FIXME: this still does not fix the cases where hyperref is loaded
+		//        and the option is active only when part of usepackage parameter
+		//        (e.g. pdfusetitle).
+		{
+			opt = "\\hypersetup{" + opt + hyperset + "}\n ";
+		}
 	
 	// FIXME UNICODE
 	os << from_utf8(opt);
@@ -188,8 +202,6 @@ string PDFOptions::readToken(Lexer &lex, string const & token)
 		lex >> pagemode;
 	} else if (token == "\\pdf_quoted_options") {
 		lex >> quoted_options;
-	} else if (token == "\\pdf_store_options") {
-		//leftover from previous option; intended for later removal
 	} else {
 		return token;
 	}
@@ -197,12 +209,13 @@ string PDFOptions::readToken(Lexer &lex, string const & token)
 }
 
 
-// prepared for check
-string PDFOptions::quoted_options_get() const
+// check the string from UI
+string PDFOptions::quoted_options_check(string const str) const
 {
-	return quoted_options;
+	return subst(str, "\n", "");
 }
 
+
 // set implicit settings for hyperref
 void PDFOptions::clear()
 {