Fix text direction issue for InsetInfo in RTL context

[lyx.git] / src / Exporter.cpp

diff --git a/src/Exporter.cpp b/src/Exporter.cpp
index 32fd07c45399f11078a21b044115c65f2804d20c..073842f2806213c9deeec12add2b61ba351a7c6f 100644 (file)
--- a/src/Exporter.cpp
+++ b/src/Exporter.cpp
@@ -40,31 +40,22 @@ namespace Alert = frontend::Alert;
 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()));
-       int choice = Alert::prompt(_("Overwrite file?"),
-                               text, 0, 2,
+                                  makeDisplayPath(filename.absFileName()));
+       return Alert::prompt(_("Overwrite file?"),
+                               text, 0, 3,
                                _("&Keep file"), _("&Overwrite"),
-                               _("&Cancel export"));
-
-       if (choice == 0)
-               return -1;
-
-       if (choice == 1) {
-               text = _("Should I continue asking for overwriting files?");
-               return Alert::prompt(_("Overwrite all files?"),
-                               text, 0, 0,
-                               _("Continue &asking"), _("&Overwrite all"));
-       }
-       return choice;
+                               _("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
@@ -73,28 +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.
        // Also don't overwrite files that already exist and are identical
        // to the source files.
-       if (!prefixIs(onlyPath(sourceFile.absFilename()), package().temp_dir().absFilename())
+       if ((only_tmp && !prefixIs(onlyPath(sourceFile.absFileName()), package().temp_dir().absFileName()))
            || sourceFile.checksum() == destFile.checksum())
                return ret;
 
        if (!force) {
                switch(checkOverwrite(destFile)) {
-               case -1:
-                       return SUCCESS;
                case 0:
+                       return SUCCESS;
+               case 1:
                        ret = SUCCESS;
                        break;
-               case 1:
+               case 2:
                        ret = FORCE;
                        break;
                default:
@@ -106,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;
 }
@@ -142,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()));
 }