Unable to create dynamic interface for google wsdl.

RESOLVED FIXED

Status

()

Core
XML
RESOLVED FIXED
16 years ago
16 years ago

People

(Reporter: harishd, Assigned: harishd)

Tracking

Trunk
x86
Windows XP
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment, 1 obsolete attachment)

1.91 KB, patch
harishd
: review+
harishd
: superreview+
Details | Diff | Splinter Review
(Assignee)

Description

16 years ago
When I tried to create a proxy for the google service I got the following error:
"Couldn't find interface info for port".

I dug into the code and found out that the type stored in mArrayInfo - a place
holder type - was never resolved and caused the interface generation code to fail.
That is, the type DirectoryCategory stored in DirectoryCategoryArray was not
resloved into a complext type.
(Assignee)

Comment 1

16 years ago
Created attachment 113702 [details] [diff] [review]
patch v1.0 

Make sure to resolve the array type stored in mArrayInfo.
(Assignee)

Updated

16 years ago
Attachment #113702 - Flags: superreview?(jband)
Attachment #113702 - Flags: review?(vidur)
(Assignee)

Updated

16 years ago
Blocks: 190182
(Assignee)

Updated

16 years ago
Attachment #113702 - Flags: superreview?(vidur)
Attachment #113702 - Flags: superreview?(jband)
Attachment #113702 - Flags: review?(vidur)
Attachment #113702 - Flags: review?(rayw)

Comment 2

16 years ago
Comment on attachment 113702 [details] [diff] [review]
patch v1.0 

This looks correct.
Attachment #113702 - Flags: review?(rayw) → review+

Comment 3

16 years ago
r=rayw

Comment 4

16 years ago
Comment on attachment 113702 [details] [diff] [review]
patch v1.0 

The Resolve() method should be called on the type even if it isn't a
placeholder. Other than that sr=vidur.
Attachment #113702 - Flags: superreview?(vidur) → superreview+
(Assignee)

Comment 5

16 years ago
Created attachment 115583 [details] [diff] [review]
patch v1.1

Addressing vidur's concern.
Attachment #113702 - Attachment is obsolete: true
(Assignee)

Comment 6

16 years ago
Comment on attachment 115583 [details] [diff] [review]
patch v1.1

carrying forward r/sr=.
Attachment #115583 - Flags: superreview+
Attachment #115583 - Flags: review+
(Assignee)

Updated

16 years ago
Attachment #113702 - Flags: superreview+
Attachment #113702 - Flags: review+
(Assignee)

Comment 7

16 years ago
Fix landed.
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.