https://bz.apache.org/bugzilla/show_bug.cgi?id=69362

--- Comment #8 from Michael Osipov <micha...@apache.org> ---
My counter-proposal for code clarity:
> diff --git a/java/org/apache/catalina/servlets/WebdavServlet.java 
> b/java/org/apache/catalina/servlets/WebdavServlet.java
> index e8cb718295..22df7343af 100644
> --- a/java/org/apache/catalina/servlets/WebdavServlet.java
> +++ b/java/org/apache/catalina/servlets/WebdavServlet.java
> @@ -1765,9 +1765,11 @@ public class WebdavServlet extends DefaultServlet 
> implements PeriodicEventListen
>                  }
> 
>                  if (!childResource.delete()) {
> +                    /* According to RFC 4918, section 9.6.1, last paragraph 
> the multi-status response
> +                     * SHOULD NOT contain the failed ancestor resource 
> collection because the client
> +                     * will know that already.
> +                     */
>                      if (!childResource.isDirectory()) {
> -                        // If it's not a collection, then it's an unknown
> -                        // error
>                          errorList.put(childName, 
> Integer.valueOf(WebdavStatus.SC_METHOD_NOT_ALLOWED));

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to