]> git.lyx.org Git - lyx.git/blobdiff - src/insets/ExternalSupport.C
* src/LyXAction.C: mark goto-clear-bookmark as working without buffer
[lyx.git] / src / insets / ExternalSupport.C
index 6a64f2867557eddf363ccf46b4fdfe5a4ad5fc9a..d84baf75069b5a48e9ec22d0d8544259387f86d4 100644 (file)
 #include "support/lyxlib.h"
 #include "support/os.h"
 #include "support/package.h"
-#include "support/path.h"
-
-#include "support/std_ostream.h"
-
-namespace support = lyx::support;
 
 using std::endl;
-
-using std::ostream;
 using std::string;
 using std::vector;
 
 
 namespace lyx {
+
+using support::FileName;
+
 namespace external {
 
 Template const * getTemplatePtr(InsetExternalParams const & params)
@@ -55,9 +51,8 @@ Template const * getTemplatePtr(InsetExternalParams const & params)
 
 void editExternal(InsetExternalParams const & params, Buffer const & buffer)
 {
-       string const file_with_path = params.filename.absFilename();
-       formats.edit(buffer, file_with_path,
-                    formats.getFormatFromFile(file_with_path));
+       formats.edit(buffer, params.filename,
+                    formats.getFormatFromFile(params.filename));
 }
 
 
@@ -66,15 +61,17 @@ namespace {
 string const subst_path(string const & input,
                        string const & placeholder,
                        string const & path,
-                        bool use_latex_path,
-                        support::latex_path_extension ext = support::PROTECT_EXTENSION,
-                        support::latex_path_dots dots = support::LEAVE_DOTS)
+                       bool use_latex_path,
+                       support::latex_path_extension ext = support::PROTECT_EXTENSION,
+                       support::latex_path_dots dots = support::LEAVE_DOTS)
 {
        if (input.find(placeholder) == string::npos)
                return input;
+       // Don't use external_path here when use_latex_path is false, as the
+       // path will be compared with another one in internal style later
+       // in Converters::move.
        string const path2 = use_latex_path ?
-               support::latex_path(path, ext, dots) :
-               support::os::external_path(path);
+               support::latex_path(path, ext, dots) : path;
        return support::subst(input, placeholder, path2);
 }
 
@@ -83,9 +80,9 @@ string const subst_path(string const & input,
 
 string const doSubstitution(InsetExternalParams const & params,
                            Buffer const & buffer, string const & s,
-                            bool use_latex_path,
+                           bool use_latex_path,
                            bool external_in_tmpdir,
-                            Substitute what)
+                           Substitute what)
 {
        Buffer const * m_buffer = buffer.getMasterBuffer();
        string const parentpath = external_in_tmpdir ?
@@ -94,60 +91,60 @@ string const doSubstitution(InsetExternalParams const & params,
        string const filename = external_in_tmpdir ?
                params.filename.mangledFilename() :
                params.filename.outputFilename(parentpath);
-       string const basename = support::ChangeExtension(
-                       support::OnlyFilename(filename), string());
-       string const absname = support::MakeAbsPath(filename, parentpath);
+       string const basename = support::changeExtension(
+                       support::onlyFilename(filename), string());
+       string const absname = support::makeAbsPath(filename, parentpath).absFilename();
 
        string result = s;
        if (what != ALL_BUT_PATHS) {
-               string const filepath = support::OnlyPath(filename);
-               string const abspath = support::OnlyPath(absname);
+               string const filepath = support::onlyPath(filename);
+               string const abspath = support::onlyPath(absname);
                string const masterpath = external_in_tmpdir ?
                        m_buffer->temppath() :
                        m_buffer->filePath();
-               string relToMasterPath = support::OnlyPath(
-                               support::MakeRelPath(absname, masterpath));
+               string relToMasterPath = support::onlyPath(
+                               support::makeRelPath(absname, masterpath));
                if (relToMasterPath == "./")
                        relToMasterPath.clear();
-               string relToParentPath = support::OnlyPath(
-                               support::MakeRelPath(absname, parentpath));
+               string relToParentPath = support::onlyPath(
+                               support::makeRelPath(absname, parentpath));
                if (relToParentPath == "./")
                        relToParentPath.clear();
 
                result = subst_path(result, "$$FPath", filepath,
-                                   use_latex_path,
-                                   support::PROTECT_EXTENSION,
-                                   support::ESCAPE_DOTS);
+                                   use_latex_path,
+                                   support::PROTECT_EXTENSION,
+                                   support::ESCAPE_DOTS);
                result = subst_path(result, "$$AbsPath", abspath,
-                                   use_latex_path,
-                                   support::PROTECT_EXTENSION,
-                                   support::ESCAPE_DOTS);
+                                   use_latex_path,
+                                   support::PROTECT_EXTENSION,
+                                   support::ESCAPE_DOTS);
                result = subst_path(result, "$$RelPathMaster",
-                                   relToMasterPath, use_latex_path,
-                                   support::PROTECT_EXTENSION,
-                                   support::ESCAPE_DOTS);
+                                   relToMasterPath, use_latex_path,
+                                   support::PROTECT_EXTENSION,
+                                   support::ESCAPE_DOTS);
                result = subst_path(result, "$$RelPathParent",
-                                   relToParentPath, use_latex_path,
-                                   support::PROTECT_EXTENSION,
-                                   support::ESCAPE_DOTS);
-               if (support::AbsolutePath(filename)) {
+                                   relToParentPath, use_latex_path,
+                                   support::PROTECT_EXTENSION,
+                                   support::ESCAPE_DOTS);
+               if (support::absolutePath(filename)) {
                        result = subst_path(result, "$$AbsOrRelPathMaster",
-                                           abspath, use_latex_path,
-                                           support::PROTECT_EXTENSION,
-                                           support::ESCAPE_DOTS);
+                                           abspath, use_latex_path,
+                                           support::PROTECT_EXTENSION,
+                                           support::ESCAPE_DOTS);
                        result = subst_path(result, "$$AbsOrRelPathParent",
-                                           abspath, use_latex_path,
-                                           support::PROTECT_EXTENSION,
-                                           support::ESCAPE_DOTS);
+                                           abspath, use_latex_path,
+                                           support::PROTECT_EXTENSION,
+                                           support::ESCAPE_DOTS);
                } else {
                        result = subst_path(result, "$$AbsOrRelPathMaster",
-                                           relToMasterPath, use_latex_path,
-                                           support::PROTECT_EXTENSION,
-                                           support::ESCAPE_DOTS);
+                                           relToMasterPath, use_latex_path,
+                                           support::PROTECT_EXTENSION,
+                                           support::ESCAPE_DOTS);
                        result = subst_path(result, "$$AbsOrRelPathParent",
-                                           relToParentPath, use_latex_path,
-                                           support::PROTECT_EXTENSION,
-                                           support::ESCAPE_DOTS);
+                                           relToParentPath, use_latex_path,
+                                           support::PROTECT_EXTENSION,
+                                           support::ESCAPE_DOTS);
                }
        }
 
@@ -155,12 +152,12 @@ string const doSubstitution(InsetExternalParams const & params,
                return result;
 
        result = subst_path(result, "$$FName", filename, use_latex_path,
-                           support::EXCLUDE_EXTENSION);
+                           support::EXCLUDE_EXTENSION);
        result = subst_path(result, "$$Basename", basename, use_latex_path,
-                           support::PROTECT_EXTENSION, support::ESCAPE_DOTS);
+                           support::PROTECT_EXTENSION, support::ESCAPE_DOTS);
        result = subst_path(result, "$$Extension",
-                       '.' + support::GetExtension(filename), use_latex_path);
-       result = subst_path(result, "$$Tempname", params.tempname(), use_latex_path);
+                       '.' + 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);
 
@@ -172,12 +169,10 @@ string const doSubstitution(InsetExternalParams const & params,
                string const file = result.substr(pos + 12, end - (pos + 12));
                string contents;
 
-               string const filepath = support::IsFileReadable(file) ?
-                       buffer.filePath() : m_buffer->temppath();
-               support::Path p(filepath);
-
-               if (support::IsFileReadable(file))
-                       contents = support::GetFileContents(file);
+               FileName const absfile(
+                       support::makeAbsPath(file, m_buffer->temppath()));
+               if (support::isFileReadable(absfile))
+                       contents = support::getFileContents(absfile);
 
                result = support::subst(result,
                                        ("$$Contents(\"" + file + "\")").c_str(),
@@ -197,7 +192,7 @@ namespace {
 void updateExternal(InsetExternalParams const & params,
                    string const & format,
                    Buffer const & buffer,
-                    ExportData & exportdata,
+                   ExportData & exportdata,
                    bool external_in_tmpdir)
 {
        Template const * const et_ptr = getTemplatePtr(params);
@@ -220,14 +215,12 @@ void updateExternal(InsetExternalParams const & params,
        if (from_format.empty())
                return; // NOT_NEEDED
 
-       string abs_from_file = params.filename.absFilename();
-
        if (from_format == "*") {
-               if (abs_from_file.empty())
+               if (params.filename.empty())
                        return; // NOT_NEEDED
 
                // Try and ascertain the file format from its contents.
-               from_format = formats.getFormatFromFile(abs_from_file);
+               from_format = formats.getFormatFromFile(params.filename);
                if (from_format.empty())
                        return; // FAILURE
 
@@ -237,7 +230,7 @@ void updateExternal(InsetExternalParams const & params,
        if (to_format.empty())
                return; // NOT_NEEDED
 
-       if (!converters.isReachable(from_format, to_format)) {
+       if (!theConverters().isReachable(from_format, to_format)) {
                lyxerr[Debug::EXTERNAL]
                        << "external::updateExternal. "
                        << "Unable to convert from "
@@ -251,20 +244,20 @@ void updateExternal(InsetExternalParams const & params,
 
        // We copy the source file to the temp dir and do the conversion
        // there if necessary
-       string const temp_file =
-               support::MakeAbsPath(params.filename.mangledFilename(),
-                                    m_buffer->temppath());
-       if (!abs_from_file.empty()) {
-               unsigned long const from_checksum = support::sum(abs_from_file);
+       FileName const temp_file(
+               support::makeAbsPath(params.filename.mangledFilename(),
+                                    m_buffer->temppath()));
+       if (!params.filename.empty()) {
+               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);
-                       if (!mover.copy(abs_from_file, temp_file)) {
+                       Mover const & mover = getMover(from_format);
+                       if (!mover.copy(params.filename, temp_file)) {
                                lyxerr[Debug::EXTERNAL]
                                        << "external::updateExternal. "
                                        << "Unable to copy "
-                                       << abs_from_file << " to " << temp_file << endl;
+                                       << params.filename << " to " << temp_file << endl;
                                return; // FAILURE
                        }
                }
@@ -273,9 +266,9 @@ void updateExternal(InsetExternalParams const & params,
        // the generated file (always in the temp dir)
        string const to_file = doSubstitution(params, buffer,
                                              outputFormat.updateResult,
-                                             false, true);
-       string const abs_to_file =
-               support::MakeAbsPath(to_file, m_buffer->temppath());
+                                             false, true);
+       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.
@@ -286,10 +279,10 @@ void updateExternal(InsetExternalParams const & params,
                vector<string>::const_iterator fit  = rit->second.begin();
                vector<string>::const_iterator fend = rit->second.end();
                for (; fit != fend; ++fit) {
-                       string const source = support::MakeAbsPath(
+                       FileName const source(support::makeAbsPath(
                                        doSubstitution(params, buffer, *fit,
-                                                      false, true),
-                                       m_buffer->temppath());
+                                                      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
@@ -297,10 +290,10 @@ void updateExternal(InsetExternalParams const & params,
                        string file = support::subst(*fit, "$$FName",
                                        "$$FPath$$Basename$$Extension");
                        file = doSubstitution(params, buffer, file, false, false,
-                                             PATHS);
+                                             PATHS);
                        file = doSubstitution(params, buffer, file,
-                                             false, external_in_tmpdir,
-                                             ALL_BUT_PATHS);
+                                             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);
@@ -311,11 +304,13 @@ void updateExternal(InsetExternalParams const & params,
        // Yes if to_file does not exist or if from_file is newer than to_file
        if (support::compare_timestamps(temp_file, abs_to_file) < 0)
                return; // SUCCESS
-       string const to_file_base =
-               support::ChangeExtension(to_file, string());
+
+       // FIXME (Abdel 12/08/06): Is there a need to show these errors?
+       ErrorList el;
        /* bool const success = */
-               converters.convert(&buffer, temp_file, to_file_base,
-                                  from_format, to_format, true);
+               theConverters().convert(&buffer, temp_file, abs_to_file,
+                                  params.filename, from_format, to_format, el,
+                                  Converters::try_default | Converters::try_cache);
        // return success
 }
 
@@ -331,9 +326,10 @@ string const substituteOptions(InsetExternalParams const & params,
 
 int writeExternal(InsetExternalParams const & params,
                  string const & format,
-                 Buffer const & buffer, ostream & os,
-                  ExportData & exportdata,
-                 bool external_in_tmpdir)
+                 Buffer const & buffer, odocstream & os,
+                 ExportData & exportdata,
+                 bool external_in_tmpdir,
+                 bool external_in_comment)
 {
        Template const * const et_ptr = getTemplatePtr(params);
        if (!et_ptr)
@@ -349,14 +345,17 @@ int writeExternal(InsetExternalParams const & params,
                return 0;
        }
 
-       updateExternal(params, format, buffer, exportdata, external_in_tmpdir);
+       if (!external_in_comment)
+               updateExternal(params, format, buffer, exportdata,
+                              external_in_tmpdir);
 
        bool const use_latex_path = format == "LaTeX";
        string str = doSubstitution(params, buffer, cit->second.product,
-                                   use_latex_path, external_in_tmpdir);
+                                   use_latex_path, external_in_tmpdir);
        str = substituteCommands(params, str, format);
        str = substituteOptions(params, str, format);
-       os << str;
+       // FIXME UNICODE
+       os << from_utf8(str);
        return int(lyx::count(str.begin(), str.end(),'\n'));
 }
 
@@ -479,8 +478,6 @@ string const substituteOption(InsetExternalParams const & params,
                return sanitizeLatexOption(opt);
        if (format == "DocBook")
                return sanitizeDocBookOption(opt);
-       if (format == "LinuxDoc")
-               return sanitizeLinuxDocOption(opt);
        return opt;
 }
 
@@ -509,9 +506,10 @@ string const substituteOptions(InsetExternalParams const & params,
        }
 
        return output;
- }
+}
 
 } // namespace anon
 
 } // namespace external
+
 } // namespace lyx