Services Core returns entities for its create and update resources, eg for nodes.

We should do the same.

CommentFileSizeAuthor
#1 1960118.services_entity.return-entity-not-id.patch1.96 KBjoachim

Comments

joachim’s picture

joachim commented
Status: Active » Fixed
StatusFileSize
new 1960118.services_entity.return-entity-not-id.patch1.96 KB
JulienF’s picture

JulienF commented

Actually,

Line 42 of file services_entity_resource_clean needs to be:

return $this->get_data($wrapper, '*');

for it to return the entity.

Cheers,

joachim’s picture

joachim commented
Status: Fixed » Active

Oops!

Any chance you could roll a patch for that?

wodenx’s picture

wodenx commented
Status: Active » Closed (duplicate)

See #1855042: Return an entity rather than entity id on create

joachim’s picture

joachim commented
Status: Closed (duplicate) » Active

Flipping these, as this has the patch.

Also, there's the follow-up in #2.

wodenx’s picture

wodenx commented
Status: Active » Fixed

I think the follow-up was already committed, no? http://drupalcode.org/project/services_entity.git/blob/refs/heads/7.x-2....

joachim’s picture

joachim commented

Yup, looks like you took care of this in another issue -- git blame says:

commit 9dbf6b5d82e2d88c3f4bf08175aadc31effcea6b
Author: wodenx
Date: Thu Aug 1 09:05:44 2013 +0100

Issue #2051147 by wodenx: Fixed clean controller returning no data on create or update.

Status: Fixed » Closed (fixed)

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