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
*/
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:
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()));
}