I've noticed that "Delete a node" operation doesn't have description at all and "Upload and attach file(s)" has unclear descriptions for 'field_name' param.

CommentFileSizeAuthor
#5 services-minor-changes-to-descriptions-of-node-resource-operations-1981084-5.patch866 bytesm1r1k
#4 services-minor-changes-to-descriptions-of-node-resource-operations-1981084-4.patch601.65 KBm1r1k
#1 services-minor-changes-to-descriptions-of-node-resource-operations-some-code-styling-1981084-2.patch10.78 KBm1r1k

Comments

m1r1k’s picture

m1r1k commented
StatusFileSize
new services-minor-changes-to-descriptions-of-node-resource-operations-some-code-styling-1981084-2.patch10.78 KB

Here is a patch.

marcingy’s picture

marcingy commented
Status: Active » Needs review

Thanks for this.

Can this patch just deal with fixing up description, rather than changing layout. The current approach in code of simply one space after the array key before => is correct in terms of drupal coding standards. (As is lining up all => items)

Whether we want to change is a matter for further discussion however fixing descriptions is totally and utter ++ :)

Setting needs review for the bot.

marcingy’s picture

marcingy commented
Priority: Minor » Normal
Status: Needs review » Needs work
m1r1k’s picture

m1r1k commented
StatusFileSize
new services-minor-changes-to-descriptions-of-node-resource-operations-1981084-4.patch601.65 KB

Remove code styling:

m1r1k’s picture

m1r1k commented
StatusFileSize
new services-minor-changes-to-descriptions-of-node-resource-operations-1981084-5.patch866 bytes

I'm sorry, it was totally wrong patch :)
Here is proper one:

kylebrowning’s picture

kylebrowning commented
Status: Needs work » Reviewed & tested by the community
ygerasimov’s picture

ygerasimov commented
Status: Reviewed & tested by the community » Fixed

Committed.

Status: Fixed » Closed (fixed)

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