Closed Bug 128316 Opened 23 years ago Closed 23 years ago

XML-RPC Fails to create XMLRPC Struct type

Categories

(Core :: XML, defect)

defect
Not set
critical

Tracking

()

RESOLVED FIXED
mozilla1.0

People

(Reporter: frodo.baggins5, Assigned: samuel)

Details

Attachments

(1 file)

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
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
OS: Windows 2000 → All
Hardware: PC → All
Attached patch fixSplinter Review
simple fix. obviously, no one had tried using a struct before :-)
looking for r=/sr= (please)
Severity: normal → critical
Target Milestone: --- → mozilla1.0
Comment on attachment 75917 [details] [diff] [review] fix r=rginda
Attachment #75917 - Flags: review+
Comment on attachment 75917 [details] [diff] [review] fix rs=darin
Attachment #75917 - Flags: superreview+
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+
checked in
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
QA Contact: petersen → rakeshmishra
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: