You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@roller.apache.org by el...@apache.org on 2006/10/16 19:30:33 UTC

svn commit: r464575 - /incubator/roller/trunk/src/org/apache/roller/business/FileManagerImpl.java

Author: eliast
Date: Mon Oct 16 10:30:31 2006
New Revision: 464575

URL: http://svn.apache.org/viewvc?view=rev&rev=464575
Log:
- Fixed bug where we were using String.replaceAll() and File.separator in Windows becomes the beginnings of a escape sequence and throws an outofbounds exception. I'm now using replace(char,char).

Modified:
    incubator/roller/trunk/src/org/apache/roller/business/FileManagerImpl.java

Modified: incubator/roller/trunk/src/org/apache/roller/business/FileManagerImpl.java
URL: http://svn.apache.org/viewvc/incubator/roller/trunk/src/org/apache/roller/business/FileManagerImpl.java?view=diff&rev=464575&r1=464574&r2=464575
==============================================================================
--- incubator/roller/trunk/src/org/apache/roller/business/FileManagerImpl.java (original)
+++ incubator/roller/trunk/src/org/apache/roller/business/FileManagerImpl.java Mon Oct 16 10:30:31 2006
@@ -461,7 +461,7 @@
         
         // convert "/" to filesystem specific file separator
         if(relPath != null) {
-            relPath.replaceAll("/", File.separator);
+            relPath = relPath.replace('/', File.separatorChar);
         }
         
         // now form the absolute path