This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 86567 - Not able to customize client for MS based complex wsdl
Summary: Not able to customize client for MS based complex wsdl
Status: VERIFIED FIXED
Alias: None
Product: webservices
Classification: Unclassified
Component: Client (show other bugs)
Version: 5.x
Hardware: All All
: P1 blocker (vote)
Assignee: Roderico Cruz
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-10-05 12:45 UTC by Martin Grebac
Modified: 2006-10-12 15:35 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
stack trace from Edit WS Attributes (4.39 KB, text/plain)
2006-10-05 12:46 UTC, Martin Grebac
Details
Diff file (2.45 KB, text/plain)
2006-10-06 05:31 UTC, Roderico Cruz
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Grebac 2006-10-05 12:45:43 UTC
Using NB 5.5 RC1, when I try to generate client for this wsdl (from MS):

http://131.107.72.15/Security_WsSecurity_Service_Indigo/WsSecurity10.svc?wsdl

I get errors from wsimport, which may be fine, but when I try to open
customization dialog, I receive attached exception, thus no customizer is shown
and I cannot fix errors, cannot do WSIT customization as well.
Comment 1 Martin Grebac 2006-10-05 12:46:57 UTC
Created attachment 34915 [details]
stack trace from Edit WS Attributes
Comment 2 Martin Grebac 2006-10-05 13:10:04 UTC
 Problem in customization is that all files import some other file (and yes, the
last one imports the main wsdl, only with a different namespace), thus the
method in edit customizer never ends. It is allowed wsdl syntax, and can be
found more frequently in MS wsdls. 
 The method in customization should have a better check and in case it already
got the same model it should get out of the recursion.
Comment 3 Roderico Cruz 2006-10-06 05:22:53 UTC
Apparently, WSDLs that have cyclical dependencies with other WSDLs are common in
the Microsoft environment. Changing the priority of this to P1
Comment 4 Roderico Cruz 2006-10-06 05:31:02 UTC
The code for imported models did not consider the case of cyclical importations
of WSDLs. 
Fixed this by checking if an imported model has already been opened. Diffs are
attached.
Comment 5 Roderico Cruz 2006-10-06 05:31:57 UTC
Created attachment 34950 [details]
Diff file
Comment 6 Martin Grebac 2006-10-06 10:04:26 UTC
The fix looks good to me. Please make sure it's commited to release55_dev (and
tested there), then commit to release55 asap.
Comment 7 Lukas Jungmann 2006-10-06 10:19:09 UTC
v. in release55_dev branch, merge the fix also into the release55 branch, please.
Thanks.
Comment 8 Roderico Cruz 2006-10-06 19:27:54 UTC
Fix has been merged to release55 branch. Thanks.
Comment 9 Jaroslav Pospisil 2006-10-12 15:35:35 UTC
VERIFIED