Delete a person - Githubissues

mmarum-sugarcrm commented 11 years ago

Original author: laurentw...@gmail.com (October 23, 2012 16:01:43)

Ref. Social Server API section 2.1.4.

In OS2.0.1 only “@self” was supported. OMA expanded this support to other Group-Ids, which were also introduced by OS2.5. however, a discrepancy applies now to identify the single “id” of the Person/relationship to be removed. OMA has chosen the way of adding an optional ["/" Related- User-Id] at the end of the request uri with no payload in the body of the DELETE (or ignored if present), whilst OS2.5 now requires a person object as payload. We chose the request uri way because:

some stacks do not easily support DELETE with bodies (although formally valid),
it is a more restful-compliant design,
shindig already supports this mode [@Service(name = "people", path = "/{userId}+/{groupId}/{personId}+")], mainly related to the usage of section 2.1.1.1,
if this part is missing the entire set of relationships of that group can be deleted (a feature that is not supported with OS2.5 solution) Note that for harmonization, the same design would need to be applied to person update (section 2.1.3), although in this case the person object would be needed as payload. See also thread [1] for more information.

[1] https://groups.google.com/d/topic/opensocial-and-gadgets-spec/fpQMG3scvd0/discussion

Original issue: http://code.google.com/p/opensocial-resources/issues/detail?id=1327

mmarum-sugarcrm commented 11 years ago

From mgmarum@gmail.com on March 15, 2013 14:36:22 I think it is reasonable for DELETE to allow the Id to be specified in URI OR in the request payload.

mmarum-sugarcrm commented 11 years ago

From laurentw...@gmail.com on March 26, 2013 16:03:52 attached is a patch that allows the Related-User-Id to be included optionally both in the DELETE and PUT URIs. It also allows DELETE request payloads to be empty and adds some text clarifying the various options further providing a couple of examples.

(btw the patch also fixes minor HTTP headers in other examples)

mmarum-sugarcrm commented 11 years ago

From rbaxte...@gmail.com on March 27, 2013 21:40:20 Walter could you add draft tags around the section you changed?

mmarum-sugarcrm commented 11 years ago

From mgmarum@gmail.com on March 29, 2013 20:28:42 I can add them.

mmarum-sugarcrm commented 11 years ago

From mgmarum@gmail.com on March 29, 2013 21:43:39 Patch applied in r1713.

ryanjbaxter commented 10 years ago

+1

OpenSocial / spec

Delete a person #1297