Changeset 1268 for trunk/autoquest-htmlmonitor/src/main/java/de/ugoe/cs/autoquest/htmlmonitor/HtmlMonitorOutputWriter.java

Timestamp:

07/25/13 15:08:58 (11 years ago)

Author:

pharms

Message:

• removed find bugs warnings

File:

• trunk/autoquest-htmlmonitor/src/main/java/de/ugoe/cs/autoquest/htmlmonitor/HtmlMonitorOutputWriter.java (modified) (1 diff)

trunk/autoquest-htmlmonitor/src/main/java/de/ugoe/cs/autoquest/htmlmonitor/HtmlMonitorOutputWriter.java

@@ -415 +415 @@
      * </p>
      *
-     * @param file
-     * @param clientLogDir
+     * @param oldLogDir the old log directory
+     * @param newLogDir the new log directory
      */
     private void handleOldLogFiles(File oldLogDir, File newLogDir) {
         if (oldLogDir.exists() && oldLogDir.isDirectory()) {
+            boolean allFilesRenamed = true;
             for (File oldLogFile : oldLogDir.listFiles()) {
-                oldLogFile.renameTo(new File(newLogDir, oldLogFile.getName()));
+                allFilesRenamed &= oldLogFile.renameTo(new File(newLogDir, oldLogFile.getName()));
             }
             
-            oldLogDir.delete();
+            if (allFilesRenamed) {
+                if (!oldLogDir.delete()) {
+                    Console.printerrln("could not move old file directory structure to new one");
+                }
+            }
         }
     }