You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by dk...@apache.org on 2014/05/28 18:35:50 UTC

svn commit: r1598064 - /sling/trunk/testing/tools/src/main/java/org/apache/sling/testing/tools/sling/SlingClient.java

Author: dklco
Date: Wed May 28 16:35:49 2014
New Revision: 1598064

URL: http://svn.apache.org/r1598064
Log:
Fixing SLING-3561 : Updating to fix an error when attempting to call SlingClient.mkdirs where a portion of the parent path exists

Modified:
    sling/trunk/testing/tools/src/main/java/org/apache/sling/testing/tools/sling/SlingClient.java

Modified: sling/trunk/testing/tools/src/main/java/org/apache/sling/testing/tools/sling/SlingClient.java
URL: http://svn.apache.org/viewvc/sling/trunk/testing/tools/src/main/java/org/apache/sling/testing/tools/sling/SlingClient.java?rev=1598064&r1=1598063&r2=1598064&view=diff
==============================================================================
--- sling/trunk/testing/tools/src/main/java/org/apache/sling/testing/tools/sling/SlingClient.java (original)
+++ sling/trunk/testing/tools/src/main/java/org/apache/sling/testing/tools/sling/SlingClient.java Wed May 28 16:35:49 2014
@@ -177,16 +177,16 @@ public class SlingClient {
     
     /** Create path and all its parent folders, using MKCOL */
     public void mkdirs(String path) throws IOException {
-        // Call mkdir on all parent path, starting at the topmost one
+        // Call mkdir on all parent paths, starting at the topmost one
         final Stack<String> parents = new Stack<String>();
         path = getParentPath(path);
-        while(path.length() > 0) {
+        while(path.length() > 0 && !exists(path)) {
             parents.push(path);
             path = getParentPath(path);
         }
         
         while(!parents.isEmpty()) {
-            mkdir(parents.pop());
+        	mkdir(parents.pop());
         }
     }