Opened 14 years ago
Closed 14 years ago
#3237 closed defect (fixed)
Rename JsonPService to JsonpService
Reported by: | James Burke | Owned by: | Dustin Machi |
---|---|---|---|
Priority: | high | Milestone: | 0.9beta |
Component: | IO | Version: | 0.9 |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
Worked with Dustin on this. The JsonPService should be called JsonpService? to fit with the naming convention. JsonP was used because in dojo.io.script, the parameter jsonpParam was a bit deceptive: the double p's caused lots of typos, and it is a bit unclear (is it the parameter name or value?).
Dustin and I agreed to change the dojo.io.script parameter name from jsonpParam to callbackParamName, and once this is done, convert JsonPService to JsonpService?.
Change History (4)
comment:1 Changed 14 years ago by
comment:2 Changed 14 years ago by
comment:3 Changed 14 years ago by
Owner: | changed from James Burke to Dustin Machi |
---|
Reassigning to dmachi for the JsonPService renaming part.
comment:4 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
(In [8991]) Rename JsonPService.js to JsonpService?.js for consistency with the io system. fixes #3237
(In [8873]) Refs #3237: changes dojo.io.script's jsonpParam to callbackParamName