Opened 7 years ago

Closed 6 years ago

#16153 closed defect (fixed)

[patch][cla] Missing "namedParams: true" in JsonRpc.js for JSON-RPC-2.0

Reported by: js402882 Owned by: dylan
Priority: low Milestone: 1.9
Component: RPC Version: 1.8.0
Keywords: Cc:
Blocked By: Blocking:

Description

In JsonRPC.js:51 should be

dojo.mixin({ namedParams:true }, jsonRpcEnvelope()) // 2.0 supports named params

Otherwise in Service.js:87 named parameters are not recognized.

Attachments (1)

patch-16153.diff (663 bytes) - added by dylan 6 years ago.
Possible patch

Download all attachments as: .zip

Change History (5)

Changed 6 years ago by dylan

Attachment: patch-16153.diff added

Possible patch

comment:1 Changed 6 years ago by dylan

Milestone: tbd1.9
Priority: undecidedlow

I've added a possible patch, based on your feedback.

Please test and let me know if this works.

comment:2 Changed 6 years ago by dylan

Summary: Missing "namedParams: true" in JsonRpc.js for JSON-RPC-2.0[patch][cla] Missing "namedParams: true" in JsonRpc.js for JSON-RPC-2.0

comment:3 Changed 6 years ago by dylan

Owner: changed from Dustin Machi to dylan
Status: newassigned

Reviewed and approved by dmachi, will commit and close.

comment:4 Changed 6 years ago by dylan

Resolution: fixed
Status: assignedclosed

In [30745]:

fixes #16153, handling of named parameters for json-rpc 2.0

Note: See TracTickets for help on using tickets.