Account for old versions of Pygments

[lyx.git] / src / Exporter.cpp

diff --git a/src/Exporter.cpp b/src/Exporter.cpp
index 85d5eb3d5551862f7791fc21897651da4124a5bc..073842f2806213c9deeec12add2b61ba351a7c6f 100644 (file)
--- a/src/Exporter.cpp
+++ b/src/Exporter.cpp
@@ -36,23 +36,26 @@ namespace lyx {
 
 namespace Alert = frontend::Alert;
 
-/// ask the user what to do if a file already exists
+/// Ask the user what to do if a file already exists
 static int checkOverwrite(FileName const & filename)
 {
        if (!filename.exists())
-               return 0;
+               return 1;
+
        docstring text = bformat(_("The file %1$s already exists.\n\n"
-                                                        "Do you want to overwrite that file?"),
-                                               makeDisplayPath(filename.absFilename()));
+                                  "Do you want to overwrite that file?"),
+                                  makeDisplayPath(filename.absFileName()));
        return Alert::prompt(_("Overwrite file?"),
-                                        text, 0, 2,
-                                        _("&Overwrite"), _("Overwrite &all"),
-                                        _("&Cancel export"));
+                               text, 0, 3,
+                               _("&Keep file"), _("&Overwrite"),
+                               _("Overwrite &all"), _("&Cancel export"));
 }
 
 
 /** copy file \p sourceFile to \p destFile. If \p force is false, the user
- *  will be asked before existing files are overwritten.
+ *  will be asked before existing files are overwritten. If \p only_tmp
+ *  is true, then only copy files that are in our tmp dir (to avoid other files
+ *  overwriting themselves).
  *  \return
  *  - SUCCESS if this file got copied
  *  - FORCE   if subsequent calls should not ask for confirmation before
@@ -61,23 +64,27 @@ static int checkOverwrite(FileName const & filename)
  */
 CopyStatus copyFile(string const & format,
                    FileName const & sourceFile, FileName const & destFile,
-                   string const & latexFile, bool force)
+                   string const & latexFile, bool force, bool only_tmp)
 {
        CopyStatus ret = force ? FORCE : SUCCESS;
 
-       // Only copy files that are in our tmp dir, all other files would
-       // overwrite themselves. This check could be changed to
+       // This check could be changed to
        // boost::filesystem::equivalent(sourceFile, destFile) if export to
        // other directories than the document directory is desired.
-       if (!prefixIs(onlyPath(sourceFile.absFilename()), package().temp_dir().absFilename()))
+       // Also don't overwrite files that already exist and are identical
+       // to the source files.
+       if ((only_tmp && !prefixIs(onlyPath(sourceFile.absFileName()), package().temp_dir().absFileName()))
+           || sourceFile.checksum() == destFile.checksum())
                return ret;
 
        if (!force) {
                switch(checkOverwrite(destFile)) {
                case 0:
+                       return SUCCESS;
+               case 1:
                        ret = SUCCESS;
                        break;
-               case 1:
+               case 2:
                        ret = FORCE;
                        break;
                default:
@@ -89,8 +96,8 @@ CopyStatus copyFile(string const & format,
        if (!mover.copy(sourceFile, destFile, latexFile))
                Alert::error(_("Couldn't copy file"),
                             bformat(_("Copying %1$s to %2$s failed."),
-                                    makeDisplayPath(sourceFile.absFilename()),
-                                    makeDisplayPath(destFile.absFilename())));
+                                    makeDisplayPath(sourceFile.absFileName()),
+                                    makeDisplayPath(destFile.absFileName())));
 
        return ret;
 }
@@ -125,7 +132,7 @@ void ExportData::addExternalFile(string const & format,
 void ExportData::addExternalFile(string const & format,
                                 FileName const & sourceName)
 {
-       addExternalFile(format, sourceName, onlyFilename(sourceName.absFilename()));
+       addExternalFile(format, sourceName, onlyFileName(sourceName.absFileName()));
 }