You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-dev@axis.apache.org by "JC (JIRA)" <ji...@apache.org> on 2017/06/21 15:40:00 UTC

[jira] [Created] (AXIS2-5856) Wrong null checker

JC created AXIS2-5856:
-------------------------

             Summary: Wrong null checker
                 Key: AXIS2-5856
                 URL: https://issues.apache.org/jira/browse/AXIS2-5856
             Project: Axis2
          Issue Type: Bug
          Components: XMPP transport
            Reporter: JC
            Priority: Trivial


Hi 

In a recent GitHub snapshot, I've found a following code smell.

Path: modules/transport/xmpp/src/org/apache/axis2/transport/xmpp/XMPPListener.java

{code}
165     public EndpointReference[] getEPRsForService(String serviceName, String ip) throws AxisFault {
166         String domainName = serverCredentials.getDomainName() == null? serverCredentials.getDomainName()
167                         : serverCredentials.getServerUrl();
168         return new EndpointReference[]{new EndpointReference(XMPPConstants.XMPP_PREFIX +
169                         serverCredentials.getAccountName() +"@"+ domainName +"/services/" + serviceName)};
170     }
{code}

In Line 166, serverCredentials.getDomainName() == null should be serverCredentials.getDomainName() != null ? This might be a trivial issue but wanted to report just in case.

Thanks!



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: java-dev-unsubscribe@axis.apache.org
For additional commands, e-mail: java-dev-help@axis.apache.org