]> git.lyx.org Git - lyx.git/blobdiff - src/insets/ExternalSupport.C
* In the process of fixing the math background color bug, this commit transfer backgr...
[lyx.git] / src / insets / ExternalSupport.C
index 7b769cd9565664d28ee323f6bb4de90308ac2ff0..1f7669b7045a1fed3984387c3598f52962a519d3 100644 (file)
 #include "support/os.h"
 #include "support/package.h"
 
+#include <boost/filesystem/operations.hpp>
+
 using std::endl;
 using std::string;
 using std::vector;
 
+using boost::filesystem::is_directory;
+
 
 namespace lyx {
 
@@ -102,12 +106,16 @@ string const doSubstitution(InsetExternalParams const & params,
                string const masterpath = external_in_tmpdir ?
                        m_buffer->temppath() :
                        m_buffer->filePath();
+               // FIXME UNICODE
                string relToMasterPath = support::onlyPath(
-                               support::makeRelPath(absname, masterpath));
+                               to_utf8(support::makeRelPath(from_utf8(absname),
+                                                            from_utf8(masterpath))));
                if (relToMasterPath == "./")
                        relToMasterPath.clear();
+               // FIXME UNICODE
                string relToParentPath = support::onlyPath(
-                               support::makeRelPath(absname, parentpath));
+                               to_utf8(support::makeRelPath(from_utf8(absname),
+                                                            from_utf8(parentpath))));
                if (relToParentPath == "./")
                        relToParentPath.clear();
 
@@ -159,14 +167,19 @@ string const doSubstitution(InsetExternalParams const & params,
                        '.' + support::getExtension(filename), use_latex_path);
        result = subst_path(result, "$$Tempname", params.tempname().absFilename(), use_latex_path);
        result = subst_path(result, "$$Sysdir",
-                               support::package().system_support(), use_latex_path);
+                               support::package().system_support().absFilename(), use_latex_path);
 
        // Handle the $$Contents(filename) syntax
        if (support::contains(result, "$$Contents(\"")) {
-
-               string::size_type const pos = result.find("$$Contents(\"");
-               string::size_type const end = result.find("\")", pos);
-               string const file = result.substr(pos + 12, end - (pos + 12));
+               // Since use_latex_path may be true we must extract the file
+               // name from s instead of result and do the substitutions
+               // again, this time with use_latex_path false.
+               string::size_type const spos = s.find("$$Contents(\"");
+               string::size_type const send = s.find("\")", spos);
+               string const file_template = s.substr(spos + 12, send - (spos + 12));
+               string const file = doSubstitution(params, buffer,
+                                                  file_template, false,
+                                                  external_in_tmpdir, what);
                string contents;
 
                FileName const absfile(
@@ -174,9 +187,9 @@ string const doSubstitution(InsetExternalParams const & params,
                if (support::isFileReadable(absfile))
                        contents = support::getFileContents(absfile);
 
-               result = support::subst(result,
-                                       ("$$Contents(\"" + file + "\")").c_str(),
-                                       contents);
+               string::size_type const pos = result.find("$$Contents(\"");
+               string::size_type const end = result.find("\")", pos);
+               result.replace(pos, end + 2, contents);
        }
 
        return result;
@@ -193,7 +206,8 @@ void updateExternal(InsetExternalParams const & params,
                    string const & format,
                    Buffer const & buffer,
                    ExportData & exportdata,
-                   bool external_in_tmpdir)
+                    bool external_in_tmpdir,
+                    bool dryrun)
 {
        Template const * const et_ptr = getTemplatePtr(params);
        if (!et_ptr)
@@ -230,14 +244,6 @@ void updateExternal(InsetExternalParams const & params,
        if (to_format.empty())
                return; // NOT_NEEDED
 
-       if (!converters.isReachable(from_format, to_format)) {
-               lyxerr[Debug::EXTERNAL]
-                       << "external::updateExternal. "
-                       << "Unable to convert from "
-                       << from_format << " to " << to_format << endl;
-               return; // FAILURE
-       }
-
        // The master buffer. This is useful when there are multiple levels
        // of include files
        Buffer const * m_buffer = buffer.getMasterBuffer();
@@ -247,14 +253,14 @@ void updateExternal(InsetExternalParams const & params,
        FileName const temp_file(
                support::makeAbsPath(params.filename.mangledFilename(),
                                     m_buffer->temppath()));
-       if (!params.filename.empty()) {
+       if (!params.filename.empty() && !is_directory(params.filename.toFilesystemEncoding())) {
                unsigned long const from_checksum = support::sum(params.filename);
                unsigned long const temp_checksum = support::sum(temp_file);
 
                if (from_checksum != temp_checksum) {
-                       Mover const & mover = movers(from_format);
+                       Mover const & mover = getMover(from_format);
                        if (!mover.copy(params.filename, temp_file)) {
-                               lyxerr[Debug::EXTERNAL]
+                               LYXERR(Debug::EXTERNAL)
                                        << "external::updateExternal. "
                                        << "Unable to copy "
                                        << params.filename << " to " << temp_file << endl;
@@ -270,33 +276,35 @@ void updateExternal(InsetExternalParams const & params,
        FileName const abs_to_file(
                support::makeAbsPath(to_file, m_buffer->temppath()));
 
-       // Record the referenced files for the exporter.
-       // The exporter will copy them to the export dir.
-       typedef Template::Format::FileMap FileMap;
-       FileMap::const_iterator rit  = outputFormat.referencedFiles.begin();
-       FileMap::const_iterator rend = outputFormat.referencedFiles.end();
-       for (; rit != rend; ++rit) {
-               vector<string>::const_iterator fit  = rit->second.begin();
-               vector<string>::const_iterator fend = rit->second.end();
-               for (; fit != fend; ++fit) {
-                       FileName const source(support::makeAbsPath(
-                                       doSubstitution(params, buffer, *fit,
-                                                      false, true),
-                                       m_buffer->temppath()));
-                       // The path of the referenced file is never the
-                       // temp path, but the filename may be the mangled
-                       // or the real name. Therefore we substitute the
-                       // paths and names separately.
-                       string file = support::subst(*fit, "$$FName",
-                                       "$$FPath$$Basename$$Extension");
-                       file = doSubstitution(params, buffer, file, false, false,
-                                             PATHS);
-                       file = doSubstitution(params, buffer, file,
-                                             false, external_in_tmpdir,
-                                             ALL_BUT_PATHS);
-                       // if file is a relative name, it is interpreted
-                       // relative to the master document.
-                       exportdata.addExternalFile(rit->first, source, file);
+       if (!dryrun) {
+               // Record the referenced files for the exporter.
+               // The exporter will copy them to the export dir.
+               typedef Template::Format::FileMap FileMap;
+               FileMap::const_iterator rit  = outputFormat.referencedFiles.begin();
+               FileMap::const_iterator rend = outputFormat.referencedFiles.end();
+               for (; rit != rend; ++rit) {
+                       vector<string>::const_iterator fit  = rit->second.begin();
+                       vector<string>::const_iterator fend = rit->second.end();
+                       for (; fit != fend; ++fit) {
+                               FileName const source(support::makeAbsPath(
+                                               doSubstitution(params, buffer, *fit,
+                                                              false, true),
+                                               m_buffer->temppath()));
+                               // The path of the referenced file is never the
+                               // temp path, but the filename may be the mangled
+                               // or the real name. Therefore we substitute the
+                               // paths and names separately.
+                               string file = support::subst(*fit, "$$FName",
+                                               "$$FPath$$Basename$$Extension");
+                               file = doSubstitution(params, buffer, file, false, false,
+                                                     PATHS);
+                               file = doSubstitution(params, buffer, file,
+                                                     false, external_in_tmpdir,
+                                                     ALL_BUT_PATHS);
+                               // if file is a relative name, it is interpreted
+                               // relative to the master document.
+                               exportdata.addExternalFile(rit->first, source, file);
+                       }
                }
        }
 
@@ -307,10 +315,17 @@ void updateExternal(InsetExternalParams const & params,
 
        // FIXME (Abdel 12/08/06): Is there a need to show these errors?
        ErrorList el;
-       /* bool const success = */
-               converters.convert(&buffer, temp_file, abs_to_file,
+       bool const success =
+               theConverters().convert(&buffer, temp_file, abs_to_file,
                                   params.filename, from_format, to_format, el,
                                   Converters::try_default | Converters::try_cache);
+
+       if (!success)
+               LYXERR(Debug::EXTERNAL)
+                       << "external::updateExternal. "
+                       << "Unable to convert from "
+                       << from_format << " to " << to_format << endl;
+
        // return success
 }
 
@@ -329,7 +344,7 @@ int writeExternal(InsetExternalParams const & params,
                  Buffer const & buffer, odocstream & os,
                  ExportData & exportdata,
                  bool external_in_tmpdir,
-                 bool external_in_comment)
+                 bool dryrun)
 {
        Template const * const et_ptr = getTemplatePtr(params);
        if (!et_ptr)
@@ -338,16 +353,16 @@ int writeExternal(InsetExternalParams const & params,
 
        Template::Formats::const_iterator cit = et.formats.find(format);
        if (cit == et.formats.end()) {
-               lyxerr[Debug::EXTERNAL]
+               LYXERR(Debug::EXTERNAL)
                        << "External template format '" << format
                        << "' not specified in template "
                        << params.templatename() << endl;
                return 0;
        }
 
-       if (!external_in_comment)
+       if (!dryrun || support::contains(cit->second.product, "$$Contents"))
                updateExternal(params, format, buffer, exportdata,
-                              external_in_tmpdir);
+                              external_in_tmpdir, dryrun);
 
        bool const use_latex_path = format == "LaTeX";
        string str = doSubstitution(params, buffer, cit->second.product,