131952 - New string variant types not known as strings to SOAP.

Status: VERIFIED FIXED

(Core :: XML, defect, P2)

Description

[Ray Whitmer]

New string types were added to nsIVariant, that need to be added to SOAP.

After review of the default encoding, it appears that the only thing wrong with
these types is that they will be automatically identified as AnySimpleType
instead of as String.  I do not have an exact test case, because I do not know
in what cases these strings are produced.

Comment 1

[Ray Whitmer]

I am assuming that this does not affect arrays -- that I will still only get
arrays of type STR or WSTR.

Comment 2

[Ray Whitmer]

Attachment: Patch to recognize new native types as strings.

This should fix the case where a typeless native string is passed to the
encoder, where it would otherwise be identified as AnyType.  There was probably
never any problem when the schema type is explicitly passed.  This is
speculation, since we have no test case, but it seems like a fairly safe thing
to do.

Comment 3

[Ray Whitmer]

I request review and superreview on this, and I will need to NSBETA1+ this.

Comment 4

[John Bandhauer]

Comment on attachment 74934 [details] [diff] [review]
Patch to recognize new native types as strings.

sr=jband

Comment 5

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Comment on attachment 74934 [details] [diff] [review]
Patch to recognize new native types as strings.

r=heikki

Comment 6

[Heikki Toivonen (remove -bugzilla when emailing directly)]

->nsbeta1+. Mail drivers for approval.

Comment 7

[Asa Dotzler [:asa]]

Comment on attachment 74934 [details] [diff] [review]
Patch to recognize new native types as strings.

a=asa (on behalf of drivers) for checkin to the 1.0 trunk

Comment 8

[Ray Whitmer]

Patch was checkin in to tree.

Comment 9

[Rakesh Mishra]

Changing Q/A contact

Comment 10

[Rakesh Mishra]

Verified on the trunk build 2002-05-28-09-trunk .

Marking verified

Comment 11

[Hixie (not reading bugmail)]

Rakesh: Just out of interest, how did you verify this?