To reproduce bug:
1. create endpoint (REST) enable node resource index call.
2. create node
3. go to http://_HOST_/_ENDPOINT_PATH_/node

You will not receive any errors but you will not see your created node either.

CommentFileSizeAuthor
#2 services-985312.patch732 byteskylebrowning
#1 services-node-resource-index-985312.patch2.39 KBygerasimov

Comments

ygerasimov’s picture

ygerasimov commented
Status: Active » Needs review
StatusFileSize
new services-node-resource-index-985312.patch2.39 KB

The problem is in _node_resource_index() function.

When it gets called with argument $fields = ''; and current user does have 'administer nodes' the SQL query formed wrong.

Please review attached patch that solves this problem + removes trailing spaces.

kylebrowning’s picture

kylebrowning commented
StatusFileSize
new services-985312.patch732 bytes

Actually this error is happening because the default value isnt set correctly in the resource definition, try this patch instead.

Status: Needs review » Needs work

The last submitted patch, services-985312.patch, failed testing.

kylebrowning’s picture

kylebrowning commented
Status: Needs work » Needs review

#2: services-985312.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, services-985312.patch, failed testing.

kylebrowning’s picture

kylebrowning commented
Status: Needs work » Fixed

Its failing against head b/c I actually committed it by accident. Dev has the fix, you can verify that way.

Status: Fixed » Closed (fixed)

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