[refactor] Improve WebDAV error messages

eXist-db · Feb 9, 2022 · 96b7964 · 96b7964
1 parent 2aeffb9
commit 96b7964
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/extensions/webdav/src/main/java/org/exist/webdav/MiltonCollection.java b/extensions/webdav/src/main/java/org/exist/webdav/MiltonCollection.java
@@ -215,7 +215,7 @@ public CollectionResource createCollection(String name)
 
         } catch (CollectionExistsException | EXistException ex) {
             LOG.debug(ex.getMessage());
-            throw new ConflictException(this);
+            throw new ConflictException(this, "Create Collection '" + getXmldbUri().append(name) + "' failed: " + ex.getMessage());
 
         }
 
@@ -243,8 +243,7 @@ public Resource createNew(String newName, InputStream is, Long length, String co
 
         } catch (PermissionDeniedException | CollectionDoesNotExistException | IOException e) {
             LOG.debug(e.getMessage());
-            throw new ConflictException(this);
-
+            throw new ConflictException(this, "Create New '" + getXmldbUri().append(newName) + "' failed: " + e.getMessage());
         }
         return resource;
     }
@@ -326,7 +325,7 @@ public void moveTo(CollectionResource rDest, String newName) throws ConflictExce
             existCollection.resourceCopyMove(destCollection, newName, Mode.MOVE);
 
         } catch (EXistException ex) {
-            throw new ConflictException(this);
+            throw new ConflictException(this, "Move '" + getXmldbUri() + "' to '" + destCollection.append(newName) + "' failed: " + ex.getMessage());
         }
     }
 

diff --git a/extensions/webdav/src/main/java/org/exist/webdav/MiltonDocument.java b/extensions/webdav/src/main/java/org/exist/webdav/MiltonDocument.java
@@ -430,7 +430,7 @@ public void moveTo(CollectionResource rDest, String newName) throws ConflictExce
             existDocument.resourceCopyMove(destCollection, newName, Mode.MOVE);
 
         } catch (EXistException ex) {
-            throw new ConflictException(this);
+            throw new ConflictException(this, "Move '" + getXmldbUri() + "' to '" + destCollection.append(newName) + "' failed: " + ex.getMessage());
         }
     }