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

Comments

Yusuf.Fidan created an issue. See original summary.

yusuf.fidan’s picture

Issue summary: View changes
yusuf.fidan’s picture

yusuf.fidan’s picture

Status: Active » Needs review
tyler.frankenstein’s picture

StatusFileSize
new621 bytes

Re-rolled patch to work out of services module, instead of doc root.

tyler.frankenstein’s picture

Status: Needs review » Fixed

Thanks 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.

yusuf.fidan’s picture

Thank you. Last year I forgot to set status on "needs review" :-(

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.