You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by en...@apache.org on 2010/06/27 22:27:55 UTC
svn commit: r958417 -
/sling/trunk/bundles/servlets/post/src/main/java/org/apache/sling/servlets/post/impl/operations/ImportOperation.java
Author: enorman
Date: Sun Jun 27 20:27:55 2010
New Revision: 958417
URL: http://svn.apache.org/viewvc?rev=958417&view=rev
Log:
SLING-1172 fixed response code for missing required request parameters
Modified:
sling/trunk/bundles/servlets/post/src/main/java/org/apache/sling/servlets/post/impl/operations/ImportOperation.java
Modified: sling/trunk/bundles/servlets/post/src/main/java/org/apache/sling/servlets/post/impl/operations/ImportOperation.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/servlets/post/src/main/java/org/apache/sling/servlets/post/impl/operations/ImportOperation.java?rev=958417&r1=958416&r2=958417&view=diff
==============================================================================
--- sling/trunk/bundles/servlets/post/src/main/java/org/apache/sling/servlets/post/impl/operations/ImportOperation.java (original)
+++ sling/trunk/bundles/servlets/post/src/main/java/org/apache/sling/servlets/post/impl/operations/ImportOperation.java Sun Jun 27 20:27:55 2010
@@ -95,7 +95,7 @@ public class ImportOperation extends Abs
String contentType = request.getParameter(SlingPostConstants.RP_CONTENT_TYPE);
if (contentType == null) {
- response.setStatus(HttpServletResponse.SC_INTERNAL_SERVER_ERROR,
+ response.setStatus(HttpServletResponse.SC_PRECONDITION_FAILED,
"Required :contentType parameter is missing");
return;
}
@@ -126,7 +126,7 @@ public class ImportOperation extends Abs
}
if (contentStream == null) {
- response.setStatus(HttpServletResponse.SC_INTERNAL_SERVER_ERROR,
+ response.setStatus(HttpServletResponse.SC_PRECONDITION_FAILED,
"Missing content for import");
return;
} else {