You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2007/12/02 12:09:56 UTC

svn commit: r600268 - in /tomcat/tc6.0.x/trunk: STATUS.txt java/org/apache/catalina/servlets/WebdavServlet.java webapps/docs/changelog.xml

Author: markt
Date: Sun Dec  2 03:09:55 2007
New Revision: 600268

URL: http://svn.apache.org/viewvc?rev=600268&view=rev
Log:
Fix bug 43706. Copy/move should return 201 if successful. Based on a patch by Panagiotis Astithas.

Modified:
    tomcat/tc6.0.x/trunk/STATUS.txt
    tomcat/tc6.0.x/trunk/java/org/apache/catalina/servlets/WebdavServlet.java
    tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=600268&r1=600267&r2=600268&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Sun Dec  2 03:09:55 2007
@@ -41,12 +41,6 @@
   +1: fhanik, pero, funkman
   -1: 
 
-* Fix http://issues.apache.org/bugzilla/show_bug.cgi?id=43706
-  Copy/move should return 201
-  http://svn.apache.org/viewvc?rev=598783&view=rev
-  +1: markt, jfclere, pero
-  -1:
-
 * Fix http://issues.apache.org/bugzilla/show_bug.cgi?id=43957
   Service.bat doesn't configure logging correctly
   http://svn.apache.org/viewvc?rev=598800&view=rev

Modified: tomcat/tc6.0.x/trunk/java/org/apache/catalina/servlets/WebdavServlet.java
URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/catalina/servlets/WebdavServlet.java?rev=600268&r1=600267&r2=600268&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/java/org/apache/catalina/servlets/WebdavServlet.java (original)
+++ tomcat/tc6.0.x/trunk/java/org/apache/catalina/servlets/WebdavServlet.java Sun Dec  2 03:09:55 2007
@@ -1729,6 +1729,9 @@
 
         }
 
+        // Copy was successful
+        resp.setStatus(WebdavStatus.SC_CREATED);
+        
         // Removing any lock-null resource which would be present at
         // the destination path
         lockNullResources.remove(destinationPath);

Modified: tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml
URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml?rev=600268&r1=600267&r2=600268&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml Sun Dec  2 03:09:55 2007
@@ -58,7 +58,12 @@
   <subsection name="Catalina">
     <changelog>
       <fix>
-        <bug>43675</bug>: Fix a possible logging related classloader leak. (markt)
+        <bug>43675</bug>: Fix a possible logging related classloader leak.
+        (markt)
+      </fix>
+      <fix>
+        <bug>43706</bug>: WebDAV copy/move now returns 201 on success. Based on
+        a patch by Panagiotis Astithas. (markt)
       </fix>
       <fix>
         <bug>43887</bug>: Make error messages much more helpful when illegal



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org