From df0a6d1502032d8cd9497ea43ee6f9e9cae05cfe Mon Sep 17 00:00:00 2001 From: Oscar Gonzalez Fernandez Date: Tue, 11 Jun 2013 13:42:43 +0200 Subject: [PATCH] Fix potential problem The capture url could be the same if it's generated in the same second. --- .../org/libreplan/web/print/CutyPrint.java | 19 +++++++++---------- 1 file changed, 9 insertions(+), 10 deletions(-) diff --git a/libreplan-webapp/src/main/java/org/libreplan/web/print/CutyPrint.java b/libreplan-webapp/src/main/java/org/libreplan/web/print/CutyPrint.java index 354a005de..143d8f84d 100644 --- a/libreplan-webapp/src/main/java/org/libreplan/web/print/CutyPrint.java +++ b/libreplan-webapp/src/main/java/org/libreplan/web/print/CutyPrint.java @@ -27,7 +27,6 @@ import java.io.IOException; import java.lang.ProcessBuilder.Redirect; import java.net.InetAddress; import java.net.UnknownHostException; -import java.text.SimpleDateFormat; import java.util.Collections; import java.util.Date; import java.util.HashMap; @@ -140,15 +139,6 @@ public class CutyPrint { private static class CutyCaptParameters { - private static String buildCaptureDestination(String extension) { - if (extension == null || extension.equals("")) { - extension = ".pdf"; - } - return "/print/" - + new SimpleDateFormat("yyyyMMddHHmmss").format(new Date()) - + extension; - } - private static final AtomicLong counter = new AtomicLong(); private final HttpServletRequest request = (HttpServletRequest) Executions @@ -190,6 +180,15 @@ public class CutyPrint { return generatedSnapshotServerPath; } + private String buildCaptureDestination(String extension) { + if (extension == null || extension.equals("")) { + extension = ".pdf"; + } + return String.format("/print/%tY%