#11344 closed enhancement (fixed)
[patch][cla] dojox.rpc.Service - option to allow sync calls
Reported by: | Daniel Stefaniuk | Owned by: | dylan |
---|---|---|---|
Priority: | low | Milestone: | 1.11 |
Component: | RPC | Version: | 1.5.0b2 |
Keywords: | rpc, service needsreview | Cc: | |
Blocked By: | Blocking: |
Description
Service doesn't have an option to allow synchronized calls back to server. Adding 'sync' option to the second argument of constructor should have an impact how the service behave.
Attachments (1)
Change History (10)
Changed 11 years ago by
Attachment: | Service.diff added |
---|
comment:1 Changed 11 years ago by
comment:2 Changed 9 years ago by
Keywords: | needsreview added |
---|---|
Priority: | high → low |
comment:3 Changed 5 years ago by
Milestone: | tbd → 1.11 |
---|---|
Owner: | changed from Dustin Machi to dylan |
Status: | new → assigned |
comment:4 Changed 5 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
comment:9 Changed 5 years ago by
Landed the patch provided, thanks danstefan. Apologies that it took six years to land a one-line patch.
If you're still interested in the setSync method, we would accept a pull request ( https://github.com/dojo/dojo/blob/master/CONTRIBUTING.md ), and I'll review it right away this time.
Note: See
TracTickets for help on using
tickets.
As it is not very common to use synchronized calls, also a "method" setSync(/*bool*/value) could be added to the implementation of Service. This will give a possibility to switch synchronization off later on, if declared with {sync:true}.