You are viewing a plain text version of this content. The canonical link for it is here.
Posted to wsif-dev@ws.apache.org by an...@apache.org on 2003/03/25 10:17:41 UTC
cvs commit: xml-axis-wsif/java/src/org/apache/wsif/wsdl/extensions/jms JMSBindingSerializer.java
antelder 2003/03/25 01:17:41
Modified: java/src/org/apache/wsif/wsdl/extensions/jms
JMSBindingSerializer.java
Log:
Fix bugzilla 18285 - NPE in JMSBindingSerializer when msgType not set
Revision Changes Path
1.6 +5 -5 xml-axis-wsif/java/src/org/apache/wsif/wsdl/extensions/jms/JMSBindingSerializer.java
Index: JMSBindingSerializer.java
===================================================================
RCS file: /home/cvs/xml-axis-wsif/java/src/org/apache/wsif/wsdl/extensions/jms/JMSBindingSerializer.java,v
retrieving revision 1.5
retrieving revision 1.6
diff -u -r1.5 -r1.6
--- JMSBindingSerializer.java 7 Dec 2002 12:33:48 -0000 1.5
+++ JMSBindingSerializer.java 25 Mar 2003 09:17:41 -0000 1.6
@@ -171,15 +171,15 @@
String msgType = DOMUtils.getAttribute(el, JMSConstants.ATTR_MESSAGE_TYPE);
- if (msgType.equals("ByteMessage")) {
+ if ("ByteMessage".equals(msgType)) {
jmsBinding.setJmsMessageType(JMSConstants.MESSAGE_TYPE_BYTEMESSAGE);
- } else if (msgType.equals("MapMessage")) {
+ } else if ("MapMessage".equals(msgType)) {
jmsBinding.setJmsMessageType(JMSConstants.MESSAGE_TYPE_MAPMESSAGE);
- } else if (msgType.equals("ObjectMessage")) {
+ } else if ("ObjectMessage".equals(msgType)) {
jmsBinding.setJmsMessageType(JMSConstants.MESSAGE_TYPE_OBJECTMESSAGE);
- } else if (msgType.equals("StreamMessage")) {
+ } else if ("StreamMessage".equals(msgType)) {
jmsBinding.setJmsMessageType(JMSConstants.MESSAGE_TYPE_STREAMMESSAGE);
- } else if (msgType.equals("TextMessage")) {
+ } else if ("TextMessage".equals(msgType)) {
jmsBinding.setJmsMessageType(JMSConstants.MESSAGE_TYPE_TEXTMESSAGE);
} else {
jmsBinding.setJmsMessageType(JMSConstants.MESSAGE_TYPE_NOTSET);