Hi all, in the function TCustomJSONRPCDispatcher.FormatResult the Result:=TJSONObject.Create(['result',Return,'error',TJSonNull.Create,transactionproperty,ID.Clone]); i think that should be checked if jdoJSONRPC2 is enabled and remove the 'error' member. From the http://jsonrpc.org/spec.html "error This member is REQUIRED on error. This member MUST NOT exist if there was no error triggered during invocation. The value for this member MUST be an Object as defined in section 5.1." Also i forgot to add that at the end of the same paragraph titled "5 Responce object" stated that "Either the result member or error member MUST be included, but both members MUST NOT be included.".

On Wed, 30 Nov 2011, Dimitrios Chr. Ioannidis wrote: > Hi all, > > in the function TCustomJSONRPCDispatcher.FormatResult the > Result:=TJSONObject.Create(['result',Return,'error',TJSonNull.Create,transactionproperty,ID.Clone]); > i think that should be checked if jdoJSONRPC2 is enabled and remove > the 'error' member. > > From the http://jsonrpc.org/spec.html "error This member is REQUIRED > on error. This member MUST NOT exist if there was no error triggered > during invocation. The value for this member MUST be an Object as > defined in section 5.1." > > Also i forgot to add that at the end of the same paragraph titled "5 > Responce object" stated that "Either the result member or error > member MUST be included, but both members MUST NOT be included.". I will have a implementation was tested mostly with an ExtJS Ext.Direct client, and there the rules are slightly different.

Hi Michael, On 30/11/2011 6:38 μμ, wrote: > > > On Wed, 30 Nov 2011, Dimitrios Chr. Ioannidis wrote: > >> Hi all, >> >> in the function TCustomJSONRPCDispatcher.FormatResult the >> Result:=TJSONObject.Create(['result',Return,'error',TJSonNull.Create,transactionproperty,ID.Clone]); >> i think that should be checked if jdoJSONRPC2 is enabled and remove the >> 'error' member. >> >> From the http://jsonrpc.org/spec.html "error This member is REQUIRED on >> error. This member MUST NOT exist if there was no error triggered during >> invocation. The value for this member MUST be an Object as defined in >> section 5.1." >> >> Also i forgot to add that at the end of the same paragraph titled "5 >> Responce object" stated that "Either the result member or error member >> MUST be included, but both members MUST NOT be included.". > > I will have a implementation was tested mostly with an > ExtJS > Ext.Direct client, and there the rules are slightly different. if i 'fixed' the json 2.0 implementation will you accept the patches for the 2.6 release ? I'm asking cause i need to code a cgi app api for multiple json rpc clients which will be written in different languages, and i want to work with the most latest featured a stable fpc release i can.