Closed
Bug 132753
Opened 22 years ago
Closed 22 years ago
nsIException ignored in script invoked from SOAP callback handler.
Categories
(Core :: XPConnect, defect)
Core
XPConnect
Tracking
()
VERIFIED
DUPLICATE
of bug 132713
People
(Reporter: rayw, Assigned: dbradley)
Details
Attachments
(1 file)
932 bytes,
patch
|
Details | Diff | Splinter Review |
The SOAP module has implemented nsIException to give better diagnostic errors to a javascript developer. These work great in the normal script. But if the script is invoked from the SOAP response listener callback, then instead of generating the message, it says that the component threw a native object that was not an exception. My test case was soapisprimeproxy.js, which is invoked every time you press a key in the text field of soapisprimenumber.html (both of these located in mozilla/extensions/xmlextras/tests). If you edit line 22 of soapisprimeproxy.js and change "boolean" to "float", this causes an error, because the service returns a boolean (when you press a key in the text field as described above). But instead of the correct message related to xsi:type, you get the message about throwing an object that was not an exception. jband was able to quickly identify the problem, and I have verified that it works and makes the error be correctly reported.
Reporter | ||
Comment 1•22 years ago
|
||
This patch seems obvious, direct, and would save would-be SOAP developers a fair amount of hassle. In my opinion, if we cannot land this for Mozilla 1.0, then we should have a note in the release notes telling developers how to convert their calls to synchronous calls to see the real error message whenever they get this bogus message about throwing a non-exception.
Reporter | ||
Comment 2•22 years ago
|
||
In the time it took me to get a good build to get a good build -- a couple of failed pulls -- to test the fix, another bug was entered on this, which has more info. *** This bug has been marked as a duplicate of 132713 ***
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•