Skip to content
Snippets Groups Projects
Commit 77cc1eeb authored by srosse's avatar srosse
Browse files

OO-1681: rename the file if it already exists

parent 98b073d6
No related branches found
No related tags found
No related merge requests found
...@@ -31,8 +31,6 @@ import java.text.DateFormat; ...@@ -31,8 +31,6 @@ import java.text.DateFormat;
import java.text.SimpleDateFormat; import java.text.SimpleDateFormat;
import java.util.Date; import java.util.Date;
import org.olat.core.logging.AssertException;
/** /**
* Initial Date: Jul 20, 2004 * Initial Date: Jul 20, 2004
* *
...@@ -54,7 +52,10 @@ public class ExportUtil { ...@@ -54,7 +52,10 @@ public class ExportUtil {
public static File writeContentToFile(String fileName, String content, File exportDirectory, String enc) { public static File writeContentToFile(String fileName, String content, File exportDirectory, String enc) {
File f = new File(exportDirectory, fileName); File f = new File(exportDirectory, fileName);
if (f.exists()) { throw new AssertException("File " + fileName + " already exists!"); } if (f.exists()) {
String newFileName = FileUtils.rename(f);
f = new File(exportDirectory, newFileName);
}
FileUtils.save(f, content, enc); FileUtils.save(f, content, enc);
return f; return f;
} }
......
...@@ -111,7 +111,7 @@ public class ScoreAccountingArchiveController extends BasicController { ...@@ -111,7 +111,7 @@ public class ScoreAccountingArchiveController extends BasicController {
File downloadFile = ExportUtil.writeContentToFile(fileName, result, exportDirectory, charset); File downloadFile = ExportUtil.writeContentToFile(fileName, result, exportDirectory, charset);
vcFeedback = createVelocityContainer("feedback"); vcFeedback = createVelocityContainer("feedback");
vcFeedback.contextPut("body", translate("course.res.feedback", new String[] { fileName })); vcFeedback.contextPut("body", translate("course.res.feedback", new String[] { downloadFile.getName() }));
downloadButton = LinkFactory.createButtonSmall("cmd.download", vcFeedback, this); downloadButton = LinkFactory.createButtonSmall("cmd.download", vcFeedback, this);
downloadButton.setUserObject(downloadFile); downloadButton.setUserObject(downloadFile);
myPanel.setContent(vcFeedback); myPanel.setContent(vcFeedback);
......
...@@ -192,8 +192,8 @@ public class QTIExportManager extends BasicManager{ ...@@ -192,8 +192,8 @@ public class QTIExportManager extends BasicManager{
private String writeContentToFile(String shortTitle, File exportDirectory, String charset, QTIExportFormatter qef, String fileNameSuffix) { private String writeContentToFile(String shortTitle, File exportDirectory, String charset, QTIExportFormatter qef, String fileNameSuffix) {
// defining target filename // defining target filename
String targetFileName = getFilename(shortTitle, qef, fileNameSuffix); String targetFileName = getFilename(shortTitle, qef, fileNameSuffix);
ExportUtil.writeContentToFile(targetFileName, qef.getReport(), exportDirectory, charset); File savedFile = ExportUtil.writeContentToFile(targetFileName, qef.getReport(), exportDirectory, charset);
return targetFileName; return savedFile.getName();
} }
private String getFilename(String shortTitle, QTIExportFormatter qef, String fileNameSuffix) { private String getFilename(String shortTitle, QTIExportFormatter qef, String fileNameSuffix) {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment