You are viewing a plain text version of this content. The canonical link for it is here.
Posted to slide-dev@jakarta.apache.org by bu...@apache.org on 2003/01/31 15:50:59 UTC

DO NOT REPLY [Bug 16642] New: - Check for 207 (Multistatus) breaks webdavResource.exists()

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=16642>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=16642

Check for 207 (Multistatus) breaks webdavResource.exists()

           Summary: Check for 207 (Multistatus) breaks
                    webdavResource.exists()
           Product: Slide
           Version: Nightly
          Platform: Other
        OS/Version: Other
            Status: NEW
          Severity: Normal
          Priority: Other
         Component: WebDAV client
        AssignedTo: slide-dev@jakarta.apache.org
        ReportedBy: lachiewicz@plusnet.pl


When I call WebdavResource with not found url (404 code) unknown exception is 
throw and it's not possible to call WebdavResource.exists()

Everything works fine without status check (changes done with last cvs update).
http://cvs.apache.org/viewcvs.cgi/jakarta-
slide/src/webdav/client/src/org/apache/webdav/lib/WebdavResource.java.diff?
r1=1.53&r2=1.54&diff_format=h

Index: WebdavResource.java
===================================================================
RCS file: /home/cvspublic/jakarta-slide/src/webdav/client/src/org/apache/webdav/
lib/WebdavResource.java,v
retrieving revision 1.54
diff -u -r1.54 WebdavResource.java
--- WebdavResource.java 16 Dec 2002 15:17:29 -0000      1.54
+++ WebdavResource.java 31 Jan 2003 14:45:49 -0000
@@ -2839,11 +2839,13 @@
         if (thisResource == true) {
             setStatusCode(status);
         }
+/*
         if (status != HttpStatus.SC_MULTI_STATUS) {
             HttpException ex = new HttpException();
             ex.setReasonCode(status);
             throw ex;
         }
+*/
         thisResource = false;

         return method.getResponses();
@@ -2906,13 +2908,16 @@
         // Set status code for this resource.
         if (thisResource == true) {
             // Set the status code.
-            setStatusCode(method.getStatusLine().getStatusCode());
+            setStatusCode(status);
         }
+/*
+        // why???
         if (status != HttpStatus.SC_MULTI_STATUS) {
             HttpException ex = new HttpException();
             ex.setReasonCode(status);
             throw ex;
         }
+*/
         thisResource = false;

         return method.getResponses();

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