Closed
Bug 128316
Opened 23 years ago
Closed 23 years ago
XML-RPC Fails to create XMLRPC Struct type
Categories
(Core :: XML, defect)
Core
XML
Tracking
()
RESOLVED
FIXED
mozilla1.0
People
(Reporter: frodo.baggins5, Assigned: samuel)
Details
Attachments
(1 file)
663 bytes,
patch
|
rginda
:
review+
darin.moz
:
superreview+
asa
:
approval+
|
Details | Diff | Splinter Review |
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:0.9.7+)
Gecko/20020116
BuildID: 20020225
After creating and initializing an xmlrpc object. A call like the following fails:
var xml_data = xmlrpc.createType(xmlrpc.STRUCT, {});
The response here throws an exception. All other xmlrpc data types are
created fine.
Reproducible: Always
Steps to Reproduce:
1.Initialize an xmlrpc object in JavaScript
2.Make a call to xmlrpc.createType(xmlrpc.STRUCT, {});
3.
Actual Results: JavaScript application throws an exception and the required
dictionary
type is not created.
Expected Results: The XmlRpc Object should have returned an object of type
nsIDictionary to
the input variable, so that it can then be populated with data
In nsXmlRpcClient.js inside of the function createType the diff looks as follows
WORKS:
< uuid.value = Components.interfaces.nsIDictionary;
< var xval = Components.classes['@mozilla.org/dictionary;1'].
<
createInstance(Components.interfaces.nsIDictionary);
< return xval;
DOESNT:
---
> return createInstance('@mozilla.org/dictionary;1',
> 'nsIDictionary');
>
Samuel is the module owner of XML-RPC, reassigning.
Assignee: heikki → samuel
Assignee | ||
Updated•23 years ago
|
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
OS: Windows 2000 → All
Hardware: PC → All
Assignee | ||
Comment 2•23 years ago
|
||
simple fix. obviously, no one had tried using a struct before :-)
Assignee | ||
Comment 3•23 years ago
|
||
looking for r=/sr= (please)
Severity: normal → critical
Target Milestone: --- → mozilla1.0
Comment 4•23 years ago
|
||
Comment on attachment 75917 [details] [diff] [review]
fix
r=rginda
Attachment #75917 -
Flags: review+
Comment 5•23 years ago
|
||
Comment on attachment 75917 [details] [diff] [review]
fix
rs=darin
Attachment #75917 -
Flags: superreview+
Comment 6•23 years ago
|
||
Comment on attachment 75917 [details] [diff] [review]
fix
a=asa (on behalf of drivers) for checkin to the 1.0 trunk
Attachment #75917 -
Flags: approval+
Assignee | ||
Comment 7•23 years ago
|
||
checked in
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Updated•22 years ago
|
QA Contact: petersen → rakeshmishra
You need to log in
before you can comment on or make changes to this bug.
Description
•