#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 9 years ago by
| Attachment: | Service.diff added |
|---|
comment:1 Changed 9 years ago by
comment:2 Changed 7 years ago by
| Keywords: | needsreview added |
|---|---|
| Priority: | high → low |
comment:3 Changed 3 years ago by
| Milestone: | tbd → 1.11 |
|---|---|
| Owner: | changed from Dustin Machi to dylan |
| Status: | new → assigned |
comment:4 Changed 3 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
comment:9 Changed 3 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}.