Would be nice to have more context available.

CommentFileSizeAuthor
#1 2206661_restws_hook_restws_response_alter_param.patch4.17 KBdasjo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dasjo’s picture

dasjo
he/him
Primary language German
Location Zurich
CreditAttribution: dasjo commented
Status: Active » Needs review
FileSize
2206661_restws_hook_restws_response_alter_param.patch4.17 KB

attached patch

fago’s picture

fago
Primary language German
Location Vienna
CreditAttribution: fago commented
Status: Needs review » Needs work 
+++ a/restws.api.php
@@ -140,8 +140,11 @@ function hook_restws_request_alter(array &$request) {
+

Extraneous new line.

dasjo’s picture

dasjo
he/him
Primary language German
Location Zurich
CreditAttribution: dasjo commented

that's just how currently parameters are specified, with newlines:
http://drupalcode.org/project/restws.git/blob/refs/heads/7.x-2.x:/restws...

dasjo’s picture

dasjo
he/him
Primary language German
Location Zurich
CreditAttribution: dasjo commented
Status: Needs work » Needs review
fago’s picture

fago
Primary language German
Location Vienna
CreditAttribution: fago commented
Status: Needs review » Fixed

That has been probably a mistake then, as it violates coding standards. Fixed that for this hook + committed.

Status: Fixed » Closed (fixed)

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