Hi,
for CRUD I need the returncode 201 (created).
In RESTServer.inc every unknown code generates an watchdog_expression, even it is known in hook_rest_server_execute_errors_alter()
I need something like this:
diff --git a/sites/all/modules/contrib/services/servers/rest_server/includes/RESTServer.inc b/sites/all/modules/contrib/services/servers/rest_server/includes/RESTServer.inc
index 1f72ce8..4c3e227 100644
--- a/sites/all/modules/contrib/services/servers/rest_server/includes/RESTServer.inc
+++ b/sites/all/modules/contrib/services/servers/rest_server/includes/RESTServer.inc
@@ -683,6 +683,9 @@ class RESTServer {
case 200:
$error_header_status_message = $this->formatHttpHeaderStatusMessage('200', $error_message);
break;
+ case 201:
+ $error_header_status_message = $this->formatHttpHeaderStatusMessage('201', $error_message);
+ break;
default:
if ($error_code >= 400 && $error_code < 600) {
$error_header_status_message = $this->formatHttpHeaderStatusMessage($error_code, $error_message);
Thanks
Yusuf
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | 2945362-5.patch | 621 bytes | tyler.frankenstein |
Comments
Comment #2
yusuf.fidan commentedComment #3
yusuf.fidan commentedComment #4
yusuf.fidan commentedComment #5
tyler.frankenstein commentedRe-rolled patch to work out of services module, instead of doc root.
Comment #7
tyler.frankenstein commentedThanks for the patch, I've committed it to the latest dev version, it will also become available when Services 7.x-3.26 is released.
Comment #8
yusuf.fidan commentedThank you. Last year I forgot to set status on "needs review" :-(