[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
Re: [wtp-dev] Status on RC1 build - assuming delayed 24 hours.
|
Jeff,
thanks for the patch.
Arthur,
with Jeff's patch for 103065, there
are no P1 bugs remaining. I think we'll be OK for RC1, though we do have
several serious bugs that need to be fixed in the days between RC1 and
0.7. Bug 103653 depends on 103424, and is a performance issue that does
not apply to Axis/Tomcat. Bug 102867 pertains to the WebSphere generic
server. Both of these bugs are necessary for WTP 0.7, but would not hold
up RC1.
Craig,
will you please drop David a line to
spin a new build once you've reviewed, committed and released Jeff's patch
to 103065 [1]? I've eyeballed it, and it looks fine insofar as I understand
the inner workings of the WSDL model.
[1] https://bugs.eclipse.org/bugs/show_bug.cgi?id=103065
Cheers - CB.
Chris Brealey
Senior Advisory Technical Manager, Rational Java Web Services, IBM Canada
Ltd.
D3-275, D3/ENX/8200/MKM, 8200 Warden Avenue, Markham, Ontario, Canada,
L6G 1C7
cbrealey@xxxxxxxxxx, 905.413.6038, tieline:969.6038, fax:905.413.4920
Jeffrey Liu/Toronto/IBM@IBMCA
Sent by: wtp-dev-bounces@xxxxxxxxxxx
07/14/2005 12:51 AM
Please respond to
"General discussion of project-wide or architectural issues." |
|
To
| "General discussion
of project-wide or architectural issues." <wtp-dev@xxxxxxxxxxx>
|
cc
|
|
Subject
| Re: [wtp-dev] Status on RC1
build - assuming delayed 24 hours. |
|
For bug 103065 - WSExplorer does not show any input field in the form view,
I believe this is a bug in the WSDL model. The WSDL model has some code
that checks whether the "parameterOrder" attribute exists on
a WSDL operation element or not. If this attribute exists, it will set
the WSDL part ordering base on the value of this attribute. Problem is,
it's using the following code to get the attribute:
changedElement.getAttribute("parameterOrder");
According to the javadoc from w3c.dom.Element, the getAttribute(...) method
will return "" even if the attribute does not exist. As a result,
the WSDL model thought a part ordering was specified even if it is not.
The fix is to check the existence of the attribute using the hasAttribute(...)
method before getting its value. It's a low risk fix. I've attached a patch.
Craig/Kihup/Keith, please review and commit/release if you agree to the
fix.
Thanks,
Jeff
Arthur Ryman/Toronto/IBM@IBMCA
Sent by: wtp-dev-bounces@xxxxxxxxxxx
07/13/2005 11:17 PM
Please respond to
"General discussion of project-wide or architectural issues." |
|
To
| "General discussion
of project-wide or architectural issues." <wtp-dev@xxxxxxxxxxx>
|
cc
|
|
Subject
| Re: [wtp-dev] Status on RC1
build - assuming delayed 24 hours. |
|
Chris,
We need to wait for legal approval for 98340 which may come on Friday,
so let's not hold up RC1 for that.
What is your outlook for resolving 103065, 103653, 102867? Can you fix
these by 8pm Thursday?
Arthur Ryman,
Rational Desktop Tools Development
phone: +1-905-413-3077, TL 969-3077
assistant: +1-905-413-2411, TL 969-2411
fax: +1-905-413-4920, TL 969-4920
mobile: +1-416-939-5063, text: 4169395063@xxxxxxx
intranet: http://labweb.torolab.ibm.com/DRY6/
Chris Brealey/Toronto/IBM@IBMCA
Sent by: wtp-dev-bounces@xxxxxxxxxxx
07/13/2005 03:34 PM
Please respond to
"General discussion of project-wide or architectural issues." |
|
To
| "General discussion
of project-wide or architectural issues." <wtp-dev@xxxxxxxxxxx>
|
cc
|
|
Subject
| Re: [wtp-dev] Status on RC1
build - assuming delayed 24 hours. |
|
Arthur,
thanks. I've committed and released three bug fixes just now, leaving us
with the following bugs we are still working on for WTP 0.7. Some of these
may be more deferrable (eg. Axis on Geronimo?) than others (eg. Axis 1.2?).
As always, the WTP community is more than welcome to chime in with opinions
on which bugs are, or are not, critical for 0.7 - and that can include
bugs that are not on this list.
BTW: At an absolute minimum, bugs 103065, 103653, 98340, 102867 would
be cause for me to vote "no".
jst.ws
100677 blo 1.0
M6 Axis deploy problem with Geronimo server
103065 blo 1.0
M6 WSExplorer does not show any input field
in the form view
103653 cri 1.0
M6 StartServerCommand should check getServerRestartState()
b...
98340 maj 1.0 M6
Support Axis 1.2 instead of Axis 1.1
103068 maj 1.0
M6 EAR project field not pre-filled.
wst.ws
102867 cri 1.0
M6 Can't create a simple top down web service
using WTP0.7 w...
103405 maj 1.0
M6 Check in translated property files for the
WSExplorer
Cheers - CB.
Chris Brealey
Senior Advisory Technical Manager, Rational Java Web Services, IBM Canada
Ltd.
D3-275, D3/ENX/8200/MKM, 8200 Warden Avenue, Markham, Ontario, Canada,
L6G 1C7
cbrealey@xxxxxxxxxx, 905.413.6038, tieline:969.6038, fax:905.413.4920
Arthur Ryman/Toronto/IBM@IBMCA
Sent by: wtp-dev-bounces@xxxxxxxxxxx
07/13/2005 12:06 PM
Please respond to
"General discussion of project-wide or architectural issues." |
|
To
| "General discussion
of project-wide or architectural issues." <wtp-dev@xxxxxxxxxxx>
|
cc
| wtp-dev@xxxxxxxxxxx, wtp-dev-bounces@xxxxxxxxxxx
|
Subject
| Re: [wtp-dev] Status on RC1
build - assuming delayed 24 hours. |
|
David,
Agreed. We have inadequate time for testing today. Also, Chris said he
needed more time for fixes. Let's reset the clock by 1 day.
Chris - go ahead and release your fixes today.
Arthur Ryman,
Rational Desktop Tools Development
phone: +1-905-413-3077, TL 969-3077
assistant: +1-905-413-2411, TL 969-2411
fax: +1-905-413-4920, TL 969-4920
mobile: +1-416-939-5063, text: 4169395063@xxxxxxx
intranet: http://labweb.torolab.ibm.com/DRY6/
David M Williams <david_williams@xxxxxxxxxx>
Sent by: wtp-dev-bounces@xxxxxxxxxxx
07/13/2005 11:13 AM
Please respond to
"General discussion of project-wide or architectural issues." |
|
To
| wtp-dev@xxxxxxxxxxx
|
cc
|
|
Subject
| [wtp-dev] Status on RC1 build
- assuming delayed 24 hours. |
|
Since I have not heard from Naci or Lawrence on cause or potential solutions
to the build hangs, I have backed out to the previous version of resolver.cache
to attempt to revert the "dialog popup" issues until those can
be solved.
However, since the earliest this build would be available (even if this
attempt works) is 1 PM Eastern, I do not think it is a quality process
to then expect team leads to
test and vote by 4 PM as originally planned.
In short, I am assuming RC1 is delayed by at least 24 hours (4 PM Thursday)
as we work through these build issues (and collect photos of comitters
in clown noses :)
[P.S. I did just hear from Lawrence, and he believes the previous version
which is building now should get past the hang problem -- then we'll see
what else pops up (I couldn't resist that pun :) ]
_______________________________________________
wtp-dev mailing list
wtp-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/wtp-dev
_______________________________________________
wtp-dev mailing list
wtp-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/wtp-dev
_______________________________________________
wtp-dev mailing list
wtp-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/wtp-dev
_______________________________________________
wtp-dev mailing list
wtp-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/wtp-dev
_______________________________________________
wtp-dev mailing list
wtp-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/wtp-dev
Index: src-wsdl/org/eclipse/wst/wsdl/internal/impl/OperationImpl.java
===================================================================
RCS file: /home/webtools/wst/components/wsdl/plugins/org.eclipse.wst.wsdl/src-wsdl/org/eclipse/wst/wsdl/internal/impl/OperationImpl.java,v
retrieving revision 1.3
diff -u -r1.3 OperationImpl.java
--- src-wsdl/org/eclipse/wst/wsdl/internal/impl/OperationImpl.java 21 Jun 2005 18:16:33 -0000 1.3
+++ src-wsdl/org/eclipse/wst/wsdl/internal/impl/OperationImpl.java 14 Jul 2005 04:34:51 -0000
@@ -943,12 +943,15 @@
setName(name);
}
- String parameterOrder = changedElement.getAttribute("parameterOrder");
- if (parameterOrder != null)
+ if (changedElement.hasAttribute("parameterOrder"))
{
- String[] array = parameterOrder.split(" ");
- List l = Arrays.asList(array);
- setParameterOrdering(l);
+ String parameterOrder = changedElement.getAttribute("parameterOrder");
+ if (parameterOrder != null)
+ {
+ String[] array = parameterOrder.split(" ");
+ List l = Arrays.asList(array);
+ setParameterOrdering(l);
+ }
}
}