Re: [apps-discuss] draft-pbryan-json-patch-04 - comments

[Mike Acar <[e-mail]>]

Hi,

On 03/08/2012 09:36 PM, Paul C. Bryan wrote:
> On Thu, 2012-03-08 at 13:46 -0800, Vadim Zaliva wrote:
>> I was looking for JSON patch formats available and found this draft.
>> It looks nice - concise and to the point. I would like to share few
>> comments I have.
>>
>> 1. I am not sure if 'test' operation should be part of it. The purpose
>> of the patch is to modify the document. 'test' operation either
>> succeeds of fails, but it is unclear how this information will be
>> communicated or used. It almost looks like a API operation rather than
>> patch format instruction.

I'm unsure about test as well. It seems strange to me to express "change 
this document if it looks like this" in a patch; if the application 
creating the patch doesn't know how the document looks, how can it 
create a patch?

>> I could see a case when it could be included as a part of conditional
>> syntax. In this case 'test' becomes a container for other operations
>> which would be executed only if the test succeeds.

I'm even more unsure about this. Could you nest them, for example?

> I'd be interested in others' views on this.

Paul, is there an archived discussion of the "test" operation, 
particularly regarding use cases?

-- 
Mike Acar -                                 - macar at cloudmark dot com