namespace Alert = frontend::Alert;
-/// Ask the user what to do if the destination file already exists
-/// and is different from the source file.
-static int checkOverwrite(FileName const & src_file, FileName const & dst_file)
+/// Ask the user what to do if a file already exists
+static int checkOverwrite(FileName const & filename)
{
- if (!dst_file.exists())
- return 0;
-
- if (src_file.checksum() == dst_file.checksum())
- return -1;
+ if (!filename.exists())
+ return 1;
docstring text = bformat(_("The file %1$s already exists.\n\n"
"Do you want to overwrite that file?"),
- makeDisplayPath(dst_file.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
*/
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(sourceFile, destFile)) {
- case -1:
- return SUCCESS;
+ switch(checkOverwrite(destFile)) {
case 0:
+ return SUCCESS;
+ case 1:
ret = SUCCESS;
break;
- case 1:
+ case 2:
ret = FORCE;
break;
default:
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;
}
void ExportData::addExternalFile(string const & format,
FileName const & sourceName)
{
- addExternalFile(format, sourceName, onlyFilename(sourceName.absFilename()));
+ addExternalFile(format, sourceName, onlyFileName(sourceName.absFileName()));
}