roller-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From snoopd...@apache.org
Subject svn commit: r1070937 - /roller/trunk/weblogger-business/src/main/java/org/apache/roller/weblogger/business/FileContentManagerImpl.java
Date Tue, 15 Feb 2011 15:32:04 GMT
Author: snoopdave
Date: Tue Feb 15 15:32:04 2011
New Revision: 1070937

URL: http://svn.apache.org/viewvc?rev=1070937&view=rev
Log:
Changing file delete to warning rather than exception to enable tests to run on Windows.

At runtime the effect should be the same as before, the problem will be logged. 
The difference is that the user won't see an error message.

Modified:
    roller/trunk/weblogger-business/src/main/java/org/apache/roller/weblogger/business/FileContentManagerImpl.java

Modified: roller/trunk/weblogger-business/src/main/java/org/apache/roller/weblogger/business/FileContentManagerImpl.java
URL: http://svn.apache.org/viewvc/roller/trunk/weblogger-business/src/main/java/org/apache/roller/weblogger/business/FileContentManagerImpl.java?rev=1070937&r1=1070936&r2=1070937&view=diff
==============================================================================
--- roller/trunk/weblogger-business/src/main/java/org/apache/roller/weblogger/business/FileContentManagerImpl.java
(original)
+++ roller/trunk/weblogger-business/src/main/java/org/apache/roller/weblogger/business/FileContentManagerImpl.java
Tue Feb 15 15:32:04 2011
@@ -136,9 +136,8 @@ public class FileContentManagerImpl impl
         // get path to delete file, checks that path exists and is readable
         File delFile = this.getRealFile(weblog, fileId);
         
-        if(!delFile.delete()) {
-            throw new FileIOException("Delete failed for ["+fileId+"], "+
-                    "possibly a non-empty directory?");
+        if (!delFile.delete()) {
+            log.warn("Delete appears to have failed for ["+fileId+"]");
         }
     }
     



Mime
View raw message