You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pivot.apache.org by sm...@apache.org on 2011/06/24 02:18:31 UTC

svn commit: r1139128 - /pivot/trunk/demos-server/src/org/apache/pivot/demos/rest/server/RESTDemoServlet.java

Author: smartini
Date: Fri Jun 24 00:18:31 2011
New Revision: 1139128

URL: http://svn.apache.org/viewvc?rev=1139128&view=rev
Log:
small fix, as seen by FindBugs

Modified:
    pivot/trunk/demos-server/src/org/apache/pivot/demos/rest/server/RESTDemoServlet.java

Modified: pivot/trunk/demos-server/src/org/apache/pivot/demos/rest/server/RESTDemoServlet.java
URL: http://svn.apache.org/viewvc/pivot/trunk/demos-server/src/org/apache/pivot/demos/rest/server/RESTDemoServlet.java?rev=1139128&r1=1139127&r2=1139128&view=diff
==============================================================================
--- pivot/trunk/demos-server/src/org/apache/pivot/demos/rest/server/RESTDemoServlet.java (original)
+++ pivot/trunk/demos-server/src/org/apache/pivot/demos/rest/server/RESTDemoServlet.java Fri Jun 24 00:18:31 2011
@@ -131,7 +131,9 @@ public class RESTDemoServlet extends Que
             throw new QueryException(Query.Status.NOT_FOUND);
         }
 
-        file.delete();
+        boolean deleted = file.delete();
+        if (!deleted)
+            throw new QueryException(Query.Status.INTERNAL_SERVER_ERROR);
     }
 
     @Override