From: Enrico Forestieri <forenr@lyx.org>
Date: Sun, 28 Sep 2014 19:28:16 +0000 (+0200)
Subject: Fix export of xfig external insets (bug #9244).
X-Git-Tag: 2.1.3~122
X-Git-Url: https://git.lyx.org/gitweb/?a=commitdiff_plain;h=a291c7065865cb95c5bfe75466ded5c86443583e;p=features.git

Fix export of xfig external insets (bug #9244).

The check for a latex format is very fragile. Both libmagic and our
guessing from contents cannot distinguish the pstex and pdftex formats
used by the xfig external inset. Moreover, it may also happen that
lyx files are mistaken as latex ones. Thus, when the guessed format
is latex, the only solution is to give precedence to the format
determined by the file extension.
---

diff --git a/src/Format.cpp b/src/Format.cpp
index 71437de3ea..9e221be352 100644
--- a/src/Format.cpp
+++ b/src/Format.cpp
@@ -408,11 +408,11 @@ string Formats::getFormatFromFile(FileName const & filename) const
 	if (filename.empty())
 		return string();
 
+	string format;
 #ifdef HAVE_MAGIC_H
 	if (filename.exists()) {
 		magic_t magic_cookie = magic_open(MAGIC_MIME);
 		if (magic_cookie) {
-			string format;
 			if (magic_load(magic_cookie, NULL) != 0) {
 				LYXERR(Debug::GRAPHICS, "Formats::getFormatFromFile\n"
 					<< "\tCouldn't load magic database - "
@@ -442,24 +442,38 @@ string Formats::getFormatFromFile(FileName const & filename) const
 				}
 			}
 			magic_close(magic_cookie);
-			if (!format.empty())
+			// libmagic recognizes as latex also some formats of ours
+			// such as pstex and pdftex. Therefore we have to perform
+			// additional checks in this case (bug 9244).
+			if (!format.empty() && format != "latex")
 				return format;
 		}
 	}
 #endif
 
-	string const format = guessFormatFromContents(filename);
 	string const ext = getExtension(filename.absFileName());
-	if (isZippedFileFormat(format) && !ext.empty()) {
-		string const & fmt_name = formats.getFormatFromExtension(ext);
-		if (!fmt_name.empty()) {
-			Format const * p_format = formats.getFormat(fmt_name);
-			if (p_format && p_format->zippedNative())
-				return p_format->name();
+	if (format.empty()) {
+		string const format = guessFormatFromContents(filename);
+		if (isZippedFileFormat(format) && !ext.empty()) {
+			string const & fmt_name = formats.getFormatFromExtension(ext);
+			if (!fmt_name.empty()) {
+				Format const * p_format = formats.getFormat(fmt_name);
+				if (p_format && p_format->zippedNative())
+					return p_format->name();
+			}
 		}
+		// Don't simply return latex (bug 9244).
+		if (!format.empty() && format != "latex")
+			return format;
+	}
+
+	// Both libmagic and our guessing from contents may return as latex
+	// also lyx files and our pstex and pdftex formats. In this case we
+	// give precedence to the format determined by the extension.
+	if (format == "latex") {
+		format = getFormatFromExtension(ext);
+		return format.empty() ? "latex" : format;
 	}
-	if (!format.empty())
-		return format;
 
 	// try to find a format from the file extension.
 	return getFormatFromExtension(ext);
diff --git a/status.21x b/status.21x
index 0b5973089d..1ea3e612e0 100644
--- a/status.21x
+++ b/status.21x
@@ -60,6 +60,8 @@ What's new
 
 - Fix baseline calculation in last paragraph (bug 9231).
 
+- Fix export of xfig external insets (bug 9244).
+
 
 * LYX2LYX