You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by dk...@apache.org on 2015/04/03 18:02:56 UTC
[1/3] cxf git commit: The new PMD plugin is strict about simplifying
boolean returns
Repository: cxf
Updated Branches:
refs/heads/master 4387055bb -> 3309231e4
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/customization/CustomizationParser.java
----------------------------------------------------------------------
diff --git a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/customization/CustomizationParser.java b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/customization/CustomizationParser.java
index 7352010..fbac01b 100644
--- a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/customization/CustomizationParser.java
+++ b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/customization/CustomizationParser.java
@@ -598,26 +598,12 @@ public final class CustomizationParser {
}
private boolean isValidJaxbBindingFile(XMLStreamReader reader) {
- if (ToolConstants.JAXB_BINDINGS.equals(reader.getName())
- || ToolConstants.SCHEMA.equals(reader.getName())) {
- return true;
- }
- return false;
+ return ToolConstants.JAXB_BINDINGS.equals(reader.getName())
+ || ToolConstants.SCHEMA.equals(reader.getName());
}
private boolean isValidJaxwsBindingFile(String bindingLocation, XMLStreamReader reader) {
- if (ToolConstants.JAXWS_BINDINGS.equals(reader.getName())) {
- // Comment this check , by default wsdlLocation value will be the
- // user input wsdl url
- /*
- * String wsdlLocation = reader.getAttributeValue(null,
- * "wsdlLocation"); if (!StringUtils.isEmpty(wsdlLocation)) { return
- * true; }
- */
- return true;
- }
- return false;
-
+ return ToolConstants.JAXWS_BINDINGS.equals(reader.getName());
}
protected void setWSDLNode(final Element node) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/customization/JAXWSBindingParser.java
----------------------------------------------------------------------
diff --git a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/customization/JAXWSBindingParser.java b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/customization/JAXWSBindingParser.java
index 02c895a..b0d3487 100644
--- a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/customization/JAXWSBindingParser.java
+++ b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/customization/JAXWSBindingParser.java
@@ -163,12 +163,8 @@ public class JAXWSBindingParser {
private boolean isPackageElement(Node node) {
- if (ToolConstants.NS_JAXWS_BINDINGS.equals(node.getNamespaceURI())
- && "package".equals(node.getLocalName())) {
- return true;
- }
- return false;
-
+ return ToolConstants.NS_JAXWS_BINDINGS.equals(node.getNamespaceURI())
+ && "package".equals(node.getLocalName());
}
private boolean isJAXWSParameterElement(Node node) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/AntGenerator.java
----------------------------------------------------------------------
diff --git a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/AntGenerator.java b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/AntGenerator.java
index d5e3f25..9935306 100644
--- a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/AntGenerator.java
+++ b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/AntGenerator.java
@@ -46,12 +46,9 @@ public class AntGenerator extends AbstractJAXWSGenerator {
}
public boolean passthrough() {
- if (env.optionSet(ToolConstants.CFG_ANT)
+ return !(env.optionSet(ToolConstants.CFG_ANT)
|| env.optionSet(ToolConstants.CFG_ALL)
- || env.optionSet(ToolConstants.CFG_GEN_ANT)) {
- return false;
- }
- return true;
+ || env.optionSet(ToolConstants.CFG_GEN_ANT));
}
public void generate(ToolContext penv) throws ToolException {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/ClientGenerator.java
----------------------------------------------------------------------
diff --git a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/ClientGenerator.java b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/ClientGenerator.java
index d8b222f..f7fc307 100644
--- a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/ClientGenerator.java
+++ b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/ClientGenerator.java
@@ -46,19 +46,8 @@ public class ClientGenerator extends AbstractJAXWSGenerator {
}
public boolean passthrough() {
- if (env.optionSet(ToolConstants.CFG_GEN_CLIENT) || env.optionSet(ToolConstants.CFG_CLIENT)
- || env.optionSet(ToolConstants.CFG_ALL)) {
- return false;
- }
- if (env.optionSet(ToolConstants.CFG_GEN_ANT) || env.optionSet(ToolConstants.CFG_GEN_TYPES)
- || env.optionSet(ToolConstants.CFG_GEN_IMPL) || env.optionSet(ToolConstants.CFG_GEN_SEI)
- || env.optionSet(ToolConstants.CFG_GEN_SERVER) || env.optionSet(ToolConstants.CFG_GEN_SERVICE)
- || env.optionSet(ToolConstants.CFG_GEN_FAULT)) {
- return true;
- }
-
- return true;
-
+ return !(env.optionSet(ToolConstants.CFG_GEN_CLIENT) || env.optionSet(ToolConstants.CFG_CLIENT)
+ || env.optionSet(ToolConstants.CFG_ALL));
}
public void generate(ToolContext penv) throws ToolException {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/FaultGenerator.java
----------------------------------------------------------------------
diff --git a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/FaultGenerator.java b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/FaultGenerator.java
index b830ac4..35830ea 100644
--- a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/FaultGenerator.java
+++ b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/FaultGenerator.java
@@ -49,17 +49,13 @@ public class FaultGenerator extends AbstractJAXWSGenerator {
|| env.optionSet(ToolConstants.CFG_ALL)) {
return false;
}
- if (env.optionSet(ToolConstants.CFG_GEN_ANT)
+ return env.optionSet(ToolConstants.CFG_GEN_ANT)
|| env.optionSet(ToolConstants.CFG_GEN_TYPES)
|| env.optionSet(ToolConstants.CFG_GEN_CLIENT)
|| env.optionSet(ToolConstants.CFG_GEN_IMPL)
|| env.optionSet(ToolConstants.CFG_GEN_SEI)
|| env.optionSet(ToolConstants.CFG_GEN_SERVER)
- || env.optionSet(ToolConstants.CFG_GEN_SERVICE)) {
- return true;
- }
-
- return false;
+ || env.optionSet(ToolConstants.CFG_GEN_SERVICE);
}
public void generate(ToolContext penv) throws ToolException {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/ImplGenerator.java
----------------------------------------------------------------------
diff --git a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/ImplGenerator.java b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/ImplGenerator.java
index a99a6ea..77137df 100644
--- a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/ImplGenerator.java
+++ b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/ImplGenerator.java
@@ -47,24 +47,9 @@ public class ImplGenerator extends AbstractJAXWSGenerator {
}
public boolean passthrough() {
- if (env.optionSet(ToolConstants.CFG_GEN_IMPL)
+ return !(env.optionSet(ToolConstants.CFG_GEN_IMPL)
|| env.optionSet(ToolConstants.CFG_IMPL)
- || env.optionSet(ToolConstants.CFG_ALL)) {
- return false;
- }
- if (env.optionSet(ToolConstants.CFG_GEN_ANT)
- || env.optionSet(ToolConstants.CFG_GEN_TYPES)
- || env.optionSet(ToolConstants.CFG_GEN_CLIENT)
- || env.optionSet(ToolConstants.CFG_GEN_SEI)
- || env.optionSet(ToolConstants.CFG_GEN_SERVER)
- || env.optionSet(ToolConstants.CFG_GEN_SERVICE)
- || env.optionSet(ToolConstants.CFG_GEN_FAULT)) {
- return true;
- }
-
- return true;
-
-
+ || env.optionSet(ToolConstants.CFG_ALL));
}
public void generate(ToolContext penv) throws ToolException {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/SEIGenerator.java
----------------------------------------------------------------------
diff --git a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/SEIGenerator.java b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/SEIGenerator.java
index 06675a5..87e7c97 100644
--- a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/SEIGenerator.java
+++ b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/SEIGenerator.java
@@ -51,15 +51,10 @@ public class SEIGenerator extends AbstractJAXWSGenerator {
if (env.optionSet(ToolConstants.CFG_GEN_SEI) || env.optionSet(ToolConstants.CFG_ALL)) {
return false;
}
- if (env.optionSet(ToolConstants.CFG_GEN_ANT) || env.optionSet(ToolConstants.CFG_GEN_TYPES)
+ return env.optionSet(ToolConstants.CFG_GEN_ANT) || env.optionSet(ToolConstants.CFG_GEN_TYPES)
|| env.optionSet(ToolConstants.CFG_GEN_CLIENT) || env.optionSet(ToolConstants.CFG_GEN_IMPL)
|| env.optionSet(ToolConstants.CFG_GEN_SERVER) || env.optionSet(ToolConstants.CFG_GEN_SERVICE)
- || env.optionSet(ToolConstants.CFG_GEN_FAULT)) {
- return true;
- }
-
- return false;
-
+ || env.optionSet(ToolConstants.CFG_GEN_FAULT);
}
private boolean hasHandlerConfig(JavaInterface intf) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/ServerGenerator.java
----------------------------------------------------------------------
diff --git a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/ServerGenerator.java b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/ServerGenerator.java
index 4311640..555efe4 100644
--- a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/ServerGenerator.java
+++ b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/ServerGenerator.java
@@ -48,17 +48,8 @@ public class ServerGenerator extends AbstractJAXWSGenerator {
}
public boolean passthrough() {
- if (env.optionSet(ToolConstants.CFG_GEN_SERVER) || env.optionSet(ToolConstants.CFG_SERVER)
- || env.optionSet(ToolConstants.CFG_ALL)) {
- return false;
- }
- if (env.optionSet(ToolConstants.CFG_GEN_ANT) || env.optionSet(ToolConstants.CFG_GEN_TYPES)
- || env.optionSet(ToolConstants.CFG_GEN_CLIENT) || env.optionSet(ToolConstants.CFG_GEN_IMPL)
- || env.optionSet(ToolConstants.CFG_GEN_SEI) || env.optionSet(ToolConstants.CFG_GEN_SERVICE)
- || env.optionSet(ToolConstants.CFG_GEN_FAULT)) {
- return true;
- }
- return true;
+ return !(env.optionSet(ToolConstants.CFG_GEN_SERVER) || env.optionSet(ToolConstants.CFG_SERVER)
+ || env.optionSet(ToolConstants.CFG_ALL));
}
public void generate(ToolContext penv) throws ToolException {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/ServiceGenerator.java
----------------------------------------------------------------------
diff --git a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/ServiceGenerator.java b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/ServiceGenerator.java
index 22aefa4..1715d4c 100644
--- a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/ServiceGenerator.java
+++ b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/generators/ServiceGenerator.java
@@ -51,16 +51,13 @@ public class ServiceGenerator extends AbstractJAXWSGenerator {
|| env.optionSet(ToolConstants.CFG_ALL)) {
return false;
}
- if (env.optionSet(ToolConstants.CFG_GEN_ANT)
+ return env.optionSet(ToolConstants.CFG_GEN_ANT)
|| env.optionSet(ToolConstants.CFG_GEN_TYPES)
|| env.optionSet(ToolConstants.CFG_GEN_CLIENT)
|| env.optionSet(ToolConstants.CFG_GEN_IMPL)
|| env.optionSet(ToolConstants.CFG_GEN_SEI)
|| env.optionSet(ToolConstants.CFG_GEN_SERVER)
- || env.optionSet(ToolConstants.CFG_GEN_FAULT)) {
- return true;
- }
- return false;
+ || env.optionSet(ToolConstants.CFG_GEN_FAULT);
}
public void generate(ToolContext penv) throws ToolException {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/processor/internal/ParameterProcessor.java
----------------------------------------------------------------------
diff --git a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/processor/internal/ParameterProcessor.java b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/processor/internal/ParameterProcessor.java
index 4bf2145..2d28713 100644
--- a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/processor/internal/ParameterProcessor.java
+++ b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/processor/internal/ParameterProcessor.java
@@ -522,10 +522,7 @@ public class ParameterProcessor extends AbstractProcessor {
return false;
}
- if (!in.getNamespaceURI().equals(out.getNamespaceURI())) {
- return false;
- }
- return true;
+ return in.getNamespaceURI().equals(out.getNamespaceURI());
}
private JavaParameter getParameterFromQName(QName wrapperElement, QName item, JavaType.Style style,
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/processor/internal/ServiceProcessor.java
----------------------------------------------------------------------
diff --git a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/processor/internal/ServiceProcessor.java b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/processor/internal/ServiceProcessor.java
index c941aba..e2b7e5b 100644
--- a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/processor/internal/ServiceProcessor.java
+++ b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/processor/internal/ServiceProcessor.java
@@ -537,11 +537,8 @@ public class ServiceProcessor extends AbstractProcessor {
private boolean isOutOfBandHeader(BindingMessageInfo bmi, ExtensibilityElement ext) {
SoapHeader soapHeader = SOAPBindingUtil.getSoapHeader(ext);
- if (soapHeader.getMessage() != null
- && !bmi.getMessageInfo().getName().equals(soapHeader.getMessage())) {
- return true;
- }
- return false;
+ return soapHeader.getMessage() != null
+ && !bmi.getMessageInfo().getName().equals(soapHeader.getMessage());
}
private void processSoapHeader(JavaMethod jm, BindingOperationInfo operation, ExtensibilityElement ext) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/processor/internal/annotator/BindingAnnotator.java
----------------------------------------------------------------------
diff --git a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/processor/internal/annotator/BindingAnnotator.java b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/processor/internal/annotator/BindingAnnotator.java
index 7a48f29..36fc246 100644
--- a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/processor/internal/annotator/BindingAnnotator.java
+++ b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/processor/internal/annotator/BindingAnnotator.java
@@ -132,12 +132,9 @@ public final class BindingAnnotator implements Annotator {
intf.setSOAPUse(SOAPBinding.Use.ENCODED);
}
- if (intf.getSOAPStyle() == SOAPBinding.Style.DOCUMENT
+ return !(intf.getSOAPStyle() == SOAPBinding.Style.DOCUMENT
&& intf.getSOAPUse() == SOAPBinding.Use.LITERAL
- && intf.getSOAPParameterStyle() == SOAPBinding.ParameterStyle.WRAPPED) {
- return false;
- }
- return true;
+ && intf.getSOAPParameterStyle() == SOAPBinding.ParameterStyle.WRAPPED);
}
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/validator/WrapperStyleNameCollisionValidator.java
----------------------------------------------------------------------
diff --git a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/validator/WrapperStyleNameCollisionValidator.java b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/validator/WrapperStyleNameCollisionValidator.java
index aaecf44..5d3b700 100644
--- a/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/validator/WrapperStyleNameCollisionValidator.java
+++ b/tools/wsdlto/frontend/jaxws/src/main/java/org/apache/cxf/tools/wsdlto/frontend/jaxws/validator/WrapperStyleNameCollisionValidator.java
@@ -81,10 +81,7 @@ public class WrapperStyleNameCollisionValidator extends ServiceValidator {
}
private boolean checkBare(ToolContext context, String opName) {
String o[] = context.getArray(ToolConstants.CFG_BAREMETHODS);
- if (checkArray(o, opName)) {
- return true;
- }
- return false;
+ return checkArray(o, opName);
}
private boolean isValidOperation(OperationInfo operation) {
ToolContext context = service.getProperty(ToolContext.class.getName(), ToolContext.class);
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/wsdlto/misc/src/main/java/org/apache/cxf/tools/misc/processor/WSDLToSoapProcessor.java
----------------------------------------------------------------------
diff --git a/tools/wsdlto/misc/src/main/java/org/apache/cxf/tools/misc/processor/WSDLToSoapProcessor.java b/tools/wsdlto/misc/src/main/java/org/apache/cxf/tools/misc/processor/WSDLToSoapProcessor.java
index f6b02ad..08458af 100644
--- a/tools/wsdlto/misc/src/main/java/org/apache/cxf/tools/misc/processor/WSDLToSoapProcessor.java
+++ b/tools/wsdlto/misc/src/main/java/org/apache/cxf/tools/misc/processor/WSDLToSoapProcessor.java
@@ -142,11 +142,8 @@ public class WSDLToSoapProcessor extends AbstractWSDLToProcessor {
}
private boolean nameSpaceCheck() {
- if (WSDLConstants.RPC.equalsIgnoreCase((String)env.get(ToolConstants.CFG_STYLE))
- && !env.optionSet(ToolConstants.CFG_NAMESPACE)) {
- return false;
- }
- return true;
+ return !(WSDLConstants.RPC.equalsIgnoreCase((String)env.get(ToolConstants.CFG_STYLE))
+ && !env.optionSet(ToolConstants.CFG_NAMESPACE));
}
protected void init() throws ToolException {
[2/3] cxf git commit: The new PMD plugin is strict about simplifying
boolean returns
Posted by dk...@apache.org.
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/AlgorithmSuitePolicyValidator.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/AlgorithmSuitePolicyValidator.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/AlgorithmSuitePolicyValidator.java
index 5488764..ac8a701 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/AlgorithmSuitePolicyValidator.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/AlgorithmSuitePolicyValidator.java
@@ -52,13 +52,9 @@ public class AlgorithmSuitePolicyValidator extends AbstractSecurityPolicyValidat
* policy defined by the AssertionInfo parameter
*/
public boolean canValidatePolicy(AssertionInfo assertionInfo) {
- if (assertionInfo.getAssertion() != null
+ return assertionInfo.getAssertion() != null
&& (SP12Constants.ALGORITHM_SUITE.equals(assertionInfo.getAssertion().getName())
- || SP11Constants.ALGORITHM_SUITE.equals(assertionInfo.getAssertion().getName()))) {
- return true;
- }
-
- return false;
+ || SP11Constants.ALGORITHM_SUITE.equals(assertionInfo.getAssertion().getName()));
}
/**
@@ -136,11 +132,7 @@ public class AlgorithmSuitePolicyValidator extends AbstractSecurityPolicyValidat
return false;
}
- if (!checkKeyLengths(result, algorithmPolicy, ai, true)) {
- return false;
- }
-
- return true;
+ return checkKeyLengths(result, algorithmPolicy, ai, true);
}
/**
@@ -214,11 +206,7 @@ public class AlgorithmSuitePolicyValidator extends AbstractSecurityPolicyValidat
}
}
- if (!checkKeyLengths(result, algorithmPolicy, ai, false)) {
- return false;
- }
-
- return true;
+ return checkKeyLengths(result, algorithmPolicy, ai, false);
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/AsymmetricBindingPolicyValidator.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/AsymmetricBindingPolicyValidator.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/AsymmetricBindingPolicyValidator.java
index 2c12a30..bb5c6ec 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/AsymmetricBindingPolicyValidator.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/AsymmetricBindingPolicyValidator.java
@@ -48,13 +48,9 @@ public class AsymmetricBindingPolicyValidator extends AbstractBindingPolicyValid
* policy defined by the AssertionInfo parameter
*/
public boolean canValidatePolicy(AssertionInfo assertionInfo) {
- if (assertionInfo.getAssertion() != null
+ return assertionInfo.getAssertion() != null
&& (SP12Constants.ASYMMETRIC_BINDING.equals(assertionInfo.getAssertion().getName())
- || SP11Constants.ASYMMETRIC_BINDING.equals(assertionInfo.getAssertion().getName()))) {
- return true;
- }
-
- return false;
+ || SP11Constants.ASYMMETRIC_BINDING.equals(assertionInfo.getAssertion().getName()));
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/ConcreteSupportingTokenPolicyValidator.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/ConcreteSupportingTokenPolicyValidator.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/ConcreteSupportingTokenPolicyValidator.java
index 89517d5..3a1c979 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/ConcreteSupportingTokenPolicyValidator.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/ConcreteSupportingTokenPolicyValidator.java
@@ -46,13 +46,9 @@ public class ConcreteSupportingTokenPolicyValidator extends AbstractSupportingTo
* policy defined by the AssertionInfo parameter
*/
public boolean canValidatePolicy(AssertionInfo assertionInfo) {
- if (assertionInfo.getAssertion() != null
+ return assertionInfo.getAssertion() != null
&& (SP12Constants.SUPPORTING_TOKENS.equals(assertionInfo.getAssertion().getName())
- || SP11Constants.SUPPORTING_TOKENS.equals(assertionInfo.getAssertion().getName()))) {
- return true;
- }
-
- return false;
+ || SP11Constants.SUPPORTING_TOKENS.equals(assertionInfo.getAssertion().getName()));
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/EncryptedTokenPolicyValidator.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/EncryptedTokenPolicyValidator.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/EncryptedTokenPolicyValidator.java
index 4a52304..adffac4 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/EncryptedTokenPolicyValidator.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/EncryptedTokenPolicyValidator.java
@@ -45,12 +45,8 @@ public class EncryptedTokenPolicyValidator extends AbstractSupportingTokenPolicy
* policy defined by the AssertionInfo parameter
*/
public boolean canValidatePolicy(AssertionInfo assertionInfo) {
- if (assertionInfo.getAssertion() != null
- && SP12Constants.ENCRYPTED_SUPPORTING_TOKENS.equals(assertionInfo.getAssertion().getName())) {
- return true;
- }
-
- return false;
+ return assertionInfo.getAssertion() != null
+ && SP12Constants.ENCRYPTED_SUPPORTING_TOKENS.equals(assertionInfo.getAssertion().getName());
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/EndorsingEncryptedTokenPolicyValidator.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/EndorsingEncryptedTokenPolicyValidator.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/EndorsingEncryptedTokenPolicyValidator.java
index 9e0ca22..8118289 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/EndorsingEncryptedTokenPolicyValidator.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/EndorsingEncryptedTokenPolicyValidator.java
@@ -49,12 +49,8 @@ public class EndorsingEncryptedTokenPolicyValidator extends AbstractSupportingTo
*/
public boolean canValidatePolicy(AssertionInfo assertionInfo) {
QName sp12QName = SP12Constants.ENDORSING_ENCRYPTED_SUPPORTING_TOKENS;
- if (assertionInfo.getAssertion() != null
- && sp12QName.equals(assertionInfo.getAssertion().getName())) {
- return true;
- }
-
- return false;
+ return assertionInfo.getAssertion() != null
+ && sp12QName.equals(assertionInfo.getAssertion().getName());
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/EndorsingTokenPolicyValidator.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/EndorsingTokenPolicyValidator.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/EndorsingTokenPolicyValidator.java
index 354ec8b..47cdceb 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/EndorsingTokenPolicyValidator.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/EndorsingTokenPolicyValidator.java
@@ -47,13 +47,9 @@ public class EndorsingTokenPolicyValidator extends AbstractSupportingTokenPolicy
* policy defined by the AssertionInfo parameter
*/
public boolean canValidatePolicy(AssertionInfo assertionInfo) {
- if (assertionInfo.getAssertion() != null
+ return assertionInfo.getAssertion() != null
&& (SP12Constants.ENDORSING_SUPPORTING_TOKENS.equals(assertionInfo.getAssertion().getName())
- || SP11Constants.ENDORSING_SUPPORTING_TOKENS.equals(assertionInfo.getAssertion().getName()))) {
- return true;
- }
-
- return false;
+ || SP11Constants.ENDORSING_SUPPORTING_TOKENS.equals(assertionInfo.getAssertion().getName()));
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/IssuedTokenPolicyValidator.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/IssuedTokenPolicyValidator.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/IssuedTokenPolicyValidator.java
index dcac606..aed16f9 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/IssuedTokenPolicyValidator.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/IssuedTokenPolicyValidator.java
@@ -59,13 +59,9 @@ public class IssuedTokenPolicyValidator extends AbstractSamlPolicyValidator {
* policy defined by the AssertionInfo parameter
*/
public boolean canValidatePolicy(AssertionInfo assertionInfo) {
- if (assertionInfo.getAssertion() != null
+ return assertionInfo.getAssertion() != null
&& (SP12Constants.ISSUED_TOKEN.equals(assertionInfo.getAssertion().getName())
- || SP11Constants.ISSUED_TOKEN.equals(assertionInfo.getAssertion().getName()))) {
- return true;
- }
-
- return false;
+ || SP11Constants.ISSUED_TOKEN.equals(assertionInfo.getAssertion().getName()));
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/KerberosTokenPolicyValidator.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/KerberosTokenPolicyValidator.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/KerberosTokenPolicyValidator.java
index e8cb852..af7474c 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/KerberosTokenPolicyValidator.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/KerberosTokenPolicyValidator.java
@@ -55,13 +55,9 @@ public class KerberosTokenPolicyValidator extends AbstractSecurityPolicyValidato
* policy defined by the AssertionInfo parameter
*/
public boolean canValidatePolicy(AssertionInfo assertionInfo) {
- if (assertionInfo.getAssertion() != null
+ return assertionInfo.getAssertion() != null
&& (SP12Constants.KERBEROS_TOKEN.equals(assertionInfo.getAssertion().getName())
- || SP11Constants.KERBEROS_TOKEN.equals(assertionInfo.getAssertion().getName()))) {
- return true;
- }
-
- return false;
+ || SP11Constants.KERBEROS_TOKEN.equals(assertionInfo.getAssertion().getName()));
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/LayoutPolicyValidator.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/LayoutPolicyValidator.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/LayoutPolicyValidator.java
index b74025a..c2a3ab9 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/LayoutPolicyValidator.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/LayoutPolicyValidator.java
@@ -57,13 +57,9 @@ public class LayoutPolicyValidator extends AbstractSecurityPolicyValidator {
* policy defined by the AssertionInfo parameter
*/
public boolean canValidatePolicy(AssertionInfo assertionInfo) {
- if (assertionInfo.getAssertion() != null
+ return assertionInfo.getAssertion() != null
&& (SP12Constants.LAYOUT.equals(assertionInfo.getAssertion().getName())
- || SP11Constants.LAYOUT.equals(assertionInfo.getAssertion().getName()))) {
- return true;
- }
-
- return false;
+ || SP11Constants.LAYOUT.equals(assertionInfo.getAssertion().getName()));
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SamlTokenPolicyValidator.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SamlTokenPolicyValidator.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SamlTokenPolicyValidator.java
index 27f61c8..46cfb1c 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SamlTokenPolicyValidator.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SamlTokenPolicyValidator.java
@@ -48,13 +48,9 @@ public class SamlTokenPolicyValidator extends AbstractSamlPolicyValidator {
* policy defined by the AssertionInfo parameter
*/
public boolean canValidatePolicy(AssertionInfo assertionInfo) {
- if (assertionInfo.getAssertion() != null
+ return assertionInfo.getAssertion() != null
&& (SP12Constants.SAML_TOKEN.equals(assertionInfo.getAssertion().getName())
- || SP11Constants.SAML_TOKEN.equals(assertionInfo.getAssertion().getName()))) {
- return true;
- }
-
- return false;
+ || SP11Constants.SAML_TOKEN.equals(assertionInfo.getAssertion().getName()));
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SecurityContextTokenPolicyValidator.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SecurityContextTokenPolicyValidator.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SecurityContextTokenPolicyValidator.java
index 9c6444e..6cbbd14 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SecurityContextTokenPolicyValidator.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SecurityContextTokenPolicyValidator.java
@@ -41,13 +41,9 @@ public class SecurityContextTokenPolicyValidator extends AbstractSecurityPolicyV
* policy defined by the AssertionInfo parameter
*/
public boolean canValidatePolicy(AssertionInfo assertionInfo) {
- if (assertionInfo.getAssertion() != null
+ return assertionInfo.getAssertion() != null
&& (SP12Constants.SECURITY_CONTEXT_TOKEN.equals(assertionInfo.getAssertion().getName())
- || SP11Constants.SECURITY_CONTEXT_TOKEN.equals(assertionInfo.getAssertion().getName()))) {
- return true;
- }
-
- return false;
+ || SP11Constants.SECURITY_CONTEXT_TOKEN.equals(assertionInfo.getAssertion().getName()));
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SignedEncryptedTokenPolicyValidator.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SignedEncryptedTokenPolicyValidator.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SignedEncryptedTokenPolicyValidator.java
index 1126c20..e8d58c7 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SignedEncryptedTokenPolicyValidator.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SignedEncryptedTokenPolicyValidator.java
@@ -45,12 +45,8 @@ public class SignedEncryptedTokenPolicyValidator extends AbstractSupportingToken
* policy defined by the AssertionInfo parameter
*/
public boolean canValidatePolicy(AssertionInfo assertionInfo) {
- if (assertionInfo.getAssertion() != null
- && SP12Constants.SIGNED_ENCRYPTED_SUPPORTING_TOKENS.equals(assertionInfo.getAssertion().getName())) {
- return true;
- }
-
- return false;
+ return assertionInfo.getAssertion() != null
+ && SP12Constants.SIGNED_ENCRYPTED_SUPPORTING_TOKENS.equals(assertionInfo.getAssertion().getName());
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SignedEndorsingEncryptedTokenPolicyValidator.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SignedEndorsingEncryptedTokenPolicyValidator.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SignedEndorsingEncryptedTokenPolicyValidator.java
index f7bb086..b71d14e 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SignedEndorsingEncryptedTokenPolicyValidator.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SignedEndorsingEncryptedTokenPolicyValidator.java
@@ -49,12 +49,8 @@ public class SignedEndorsingEncryptedTokenPolicyValidator extends AbstractSuppor
*/
public boolean canValidatePolicy(AssertionInfo assertionInfo) {
QName suppTokens12 = SP12Constants.SIGNED_ENDORSING_ENCRYPTED_SUPPORTING_TOKENS;
- if (assertionInfo.getAssertion() != null
- && suppTokens12.equals(assertionInfo.getAssertion().getName())) {
- return true;
- }
-
- return false;
+ return assertionInfo.getAssertion() != null
+ && suppTokens12.equals(assertionInfo.getAssertion().getName());
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SignedEndorsingTokenPolicyValidator.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SignedEndorsingTokenPolicyValidator.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SignedEndorsingTokenPolicyValidator.java
index a207715..77e7ebd 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SignedEndorsingTokenPolicyValidator.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SignedEndorsingTokenPolicyValidator.java
@@ -47,13 +47,9 @@ public class SignedEndorsingTokenPolicyValidator extends AbstractSupportingToken
* policy defined by the AssertionInfo parameter
*/
public boolean canValidatePolicy(AssertionInfo assertionInfo) {
- if (assertionInfo.getAssertion() != null
+ return assertionInfo.getAssertion() != null
&& (SP12Constants.SIGNED_ENDORSING_SUPPORTING_TOKENS.equals(assertionInfo.getAssertion().getName())
- || SP11Constants.SIGNED_ENDORSING_SUPPORTING_TOKENS.equals(assertionInfo.getAssertion().getName()))) {
- return true;
- }
-
- return false;
+ || SP11Constants.SIGNED_ENDORSING_SUPPORTING_TOKENS.equals(assertionInfo.getAssertion().getName()));
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SignedTokenPolicyValidator.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SignedTokenPolicyValidator.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SignedTokenPolicyValidator.java
index ea80db1..c04f4b6 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SignedTokenPolicyValidator.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SignedTokenPolicyValidator.java
@@ -45,13 +45,9 @@ public class SignedTokenPolicyValidator extends AbstractSupportingTokenPolicyVal
* policy defined by the AssertionInfo parameter
*/
public boolean canValidatePolicy(AssertionInfo assertionInfo) {
- if (assertionInfo.getAssertion() != null
+ return assertionInfo.getAssertion() != null
&& (SP12Constants.SIGNED_SUPPORTING_TOKENS.equals(assertionInfo.getAssertion().getName())
- || SP11Constants.SIGNED_SUPPORTING_TOKENS.equals(assertionInfo.getAssertion().getName()))) {
- return true;
- }
-
- return false;
+ || SP11Constants.SIGNED_SUPPORTING_TOKENS.equals(assertionInfo.getAssertion().getName()));
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SymmetricBindingPolicyValidator.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SymmetricBindingPolicyValidator.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SymmetricBindingPolicyValidator.java
index 08b1699..f724f5e 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SymmetricBindingPolicyValidator.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/SymmetricBindingPolicyValidator.java
@@ -46,13 +46,9 @@ public class SymmetricBindingPolicyValidator extends AbstractBindingPolicyValida
* policy defined by the AssertionInfo parameter
*/
public boolean canValidatePolicy(AssertionInfo assertionInfo) {
- if (assertionInfo.getAssertion() != null
+ return assertionInfo.getAssertion() != null
&& (SP12Constants.SYMMETRIC_BINDING.equals(assertionInfo.getAssertion().getName())
- || SP11Constants.SYMMETRIC_BINDING.equals(assertionInfo.getAssertion().getName()))) {
- return true;
- }
-
- return false;
+ || SP11Constants.SYMMETRIC_BINDING.equals(assertionInfo.getAssertion().getName()));
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/TransportBindingPolicyValidator.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/TransportBindingPolicyValidator.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/TransportBindingPolicyValidator.java
index 869804d..2e049f3 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/TransportBindingPolicyValidator.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/TransportBindingPolicyValidator.java
@@ -42,13 +42,9 @@ public class TransportBindingPolicyValidator extends AbstractBindingPolicyValida
* policy defined by the AssertionInfo parameter
*/
public boolean canValidatePolicy(AssertionInfo assertionInfo) {
- if (assertionInfo.getAssertion() != null
+ return assertionInfo.getAssertion() != null
&& (SP12Constants.TRANSPORT_BINDING.equals(assertionInfo.getAssertion().getName())
- || SP11Constants.TRANSPORT_BINDING.equals(assertionInfo.getAssertion().getName()))) {
- return true;
- }
-
- return false;
+ || SP11Constants.TRANSPORT_BINDING.equals(assertionInfo.getAssertion().getName()));
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/UsernameTokenPolicyValidator.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/UsernameTokenPolicyValidator.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/UsernameTokenPolicyValidator.java
index 2e06b02..37ceeaa 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/UsernameTokenPolicyValidator.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/UsernameTokenPolicyValidator.java
@@ -48,13 +48,9 @@ public class UsernameTokenPolicyValidator extends AbstractSecurityPolicyValidato
* policy defined by the AssertionInfo parameter
*/
public boolean canValidatePolicy(AssertionInfo assertionInfo) {
- if (assertionInfo.getAssertion() != null
+ return assertionInfo.getAssertion() != null
&& (SP12Constants.USERNAME_TOKEN.equals(assertionInfo.getAssertion().getName())
- || SP11Constants.USERNAME_TOKEN.equals(assertionInfo.getAssertion().getName()))) {
- return true;
- }
-
- return false;
+ || SP11Constants.USERNAME_TOKEN.equals(assertionInfo.getAssertion().getName()));
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/WSS11PolicyValidator.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/WSS11PolicyValidator.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/WSS11PolicyValidator.java
index 14e4180..4c86c82 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/WSS11PolicyValidator.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/WSS11PolicyValidator.java
@@ -45,13 +45,9 @@ public class WSS11PolicyValidator extends AbstractSecurityPolicyValidator {
* policy defined by the AssertionInfo parameter
*/
public boolean canValidatePolicy(AssertionInfo assertionInfo) {
- if (assertionInfo.getAssertion() != null
+ return assertionInfo.getAssertion() != null
&& (SP12Constants.WSS11.equals(assertionInfo.getAssertion().getName())
- || SP11Constants.WSS11.equals(assertionInfo.getAssertion().getName()))) {
- return true;
- }
-
- return false;
+ || SP11Constants.WSS11.equals(assertionInfo.getAssertion().getName()));
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/X509TokenPolicyValidator.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/X509TokenPolicyValidator.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/X509TokenPolicyValidator.java
index 20ffd2a..8a3feb2 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/X509TokenPolicyValidator.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/X509TokenPolicyValidator.java
@@ -62,13 +62,9 @@ public class X509TokenPolicyValidator extends AbstractSecurityPolicyValidator {
* policy defined by the AssertionInfo parameter
*/
public boolean canValidatePolicy(AssertionInfo assertionInfo) {
- if (assertionInfo.getAssertion() != null
+ return assertionInfo.getAssertion() != null
&& (SP12Constants.X509_TOKEN.equals(assertionInfo.getAssertion().getName())
- || SP11Constants.X509_TOKEN.equals(assertionInfo.getAssertion().getName()))) {
- return true;
- }
-
- return false;
+ || SP11Constants.X509_TOKEN.equals(assertionInfo.getAssertion().getName()));
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/wsdl/src/main/java/org/apache/cxf/wsdl11/PartialWSDLProcessor.java
----------------------------------------------------------------------
diff --git a/rt/wsdl/src/main/java/org/apache/cxf/wsdl11/PartialWSDLProcessor.java b/rt/wsdl/src/main/java/org/apache/cxf/wsdl11/PartialWSDLProcessor.java
index 6dd5c03..389dbcd 100644
--- a/rt/wsdl/src/main/java/org/apache/cxf/wsdl11/PartialWSDLProcessor.java
+++ b/rt/wsdl/src/main/java/org/apache/cxf/wsdl11/PartialWSDLProcessor.java
@@ -60,7 +60,7 @@ public final class PartialWSDLProcessor {
}
- public static boolean isPortTypeExisted(Definition wsdlDefinition, QName name) {
+ public static boolean isPortTypeExisted(Definition wsdlDefinition, QName name) {
Map<QName, PortType> portTypes = CastUtils.cast(wsdlDefinition.getAllPortTypes());
if (portTypes == null || portTypes.isEmpty()) {
return false;
@@ -78,10 +78,7 @@ public final class PartialWSDLProcessor {
} catch (Exception e) {
portType = null;
}
- if (portType == null) {
- return false;
- }
- return true;
+ return portType != null;
}
public static boolean isBindingExisted(Definition wsdlDefinition, QName name) {
@@ -100,22 +97,12 @@ public final class PartialWSDLProcessor {
} catch (Exception e) {
binding = null;
}
- if (binding == null) {
- return false;
- }
- return true;
+ return binding != null;
}
public static boolean isServiceExisted(Definition wsdlDefinition, QName name) {
- if (wsdlDefinition.getService(name) == null) {
- return false;
- }
- return true;
+ return wsdlDefinition.getService(name) != null;
}
-
-
-
-
public static Binding doAppendBinding(Definition wsdlDefinition, String name, PortType portType,
ExtensionRegistry extReg) throws Exception {
Binding binding = wsdlDefinition.createBinding();
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/wsdl/src/main/java/org/apache/cxf/wsdl11/WSDLServiceBuilder.java
----------------------------------------------------------------------
diff --git a/rt/wsdl/src/main/java/org/apache/cxf/wsdl11/WSDLServiceBuilder.java b/rt/wsdl/src/main/java/org/apache/cxf/wsdl11/WSDLServiceBuilder.java
index 0fff006..4f9f481 100644
--- a/rt/wsdl/src/main/java/org/apache/cxf/wsdl11/WSDLServiceBuilder.java
+++ b/rt/wsdl/src/main/java/org/apache/cxf/wsdl11/WSDLServiceBuilder.java
@@ -784,10 +784,7 @@ public class WSDLServiceBuilder {
private static boolean hasAttributes(XmlSchemaComplexType complexType) {
// Now lets see if we have any attributes...
// This should probably look at the restricted and substitute types too.
- if (complexType.getAnyAttribute() != null || !complexType.getAttributes().isEmpty()) {
- return true;
- }
- return false;
+ return complexType.getAnyAttribute() != null || !complexType.getAttributes().isEmpty();
}
private static boolean isWrappableSequence(XmlSchemaComplexType type, String namespaceURI,
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/services/sts/sts-core/src/main/java/org/apache/cxf/sts/token/provider/SAMLTokenProvider.java
----------------------------------------------------------------------
diff --git a/services/sts/sts-core/src/main/java/org/apache/cxf/sts/token/provider/SAMLTokenProvider.java b/services/sts/sts-core/src/main/java/org/apache/cxf/sts/token/provider/SAMLTokenProvider.java
index bad2009..4a16a75 100644
--- a/services/sts/sts-core/src/main/java/org/apache/cxf/sts/token/provider/SAMLTokenProvider.java
+++ b/services/sts/sts-core/src/main/java/org/apache/cxf/sts/token/provider/SAMLTokenProvider.java
@@ -83,11 +83,8 @@ public class SAMLTokenProvider extends AbstractSAMLTokenProvider implements Toke
if (realm != null && !realmMap.containsKey(realm)) {
return false;
}
- if (WSConstants.WSS_SAML2_TOKEN_TYPE.equals(tokenType) || WSConstants.SAML2_NS.equals(tokenType)
- || WSConstants.WSS_SAML_TOKEN_TYPE.equals(tokenType) || WSConstants.SAML_NS.equals(tokenType)) {
- return true;
- }
- return false;
+ return WSConstants.WSS_SAML2_TOKEN_TYPE.equals(tokenType) || WSConstants.SAML2_NS.equals(tokenType)
+ || WSConstants.WSS_SAML_TOKEN_TYPE.equals(tokenType) || WSConstants.SAML_NS.equals(tokenType);
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/services/sts/sts-core/src/main/java/org/apache/cxf/sts/token/provider/SCTProvider.java
----------------------------------------------------------------------
diff --git a/services/sts/sts-core/src/main/java/org/apache/cxf/sts/token/provider/SCTProvider.java b/services/sts/sts-core/src/main/java/org/apache/cxf/sts/token/provider/SCTProvider.java
index 1024c17..0d80a63 100644
--- a/services/sts/sts-core/src/main/java/org/apache/cxf/sts/token/provider/SCTProvider.java
+++ b/services/sts/sts-core/src/main/java/org/apache/cxf/sts/token/provider/SCTProvider.java
@@ -78,11 +78,8 @@ public class SCTProvider implements TokenProvider {
* token provider.
*/
public boolean canHandleToken(String tokenType, String realm) {
- if (STSUtils.TOKEN_TYPE_SCT_05_02.equals(tokenType)
- || STSUtils.TOKEN_TYPE_SCT_05_12.equals(tokenType)) {
- return true;
- }
- return false;
+ return STSUtils.TOKEN_TYPE_SCT_05_02.equals(tokenType)
+ || STSUtils.TOKEN_TYPE_SCT_05_12.equals(tokenType);
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/services/sts/sts-core/src/main/java/org/apache/cxf/sts/token/validator/UsernameTokenValidator.java
----------------------------------------------------------------------
diff --git a/services/sts/sts-core/src/main/java/org/apache/cxf/sts/token/validator/UsernameTokenValidator.java b/services/sts/sts-core/src/main/java/org/apache/cxf/sts/token/validator/UsernameTokenValidator.java
index 7b51669..04fc346 100644
--- a/services/sts/sts-core/src/main/java/org/apache/cxf/sts/token/validator/UsernameTokenValidator.java
+++ b/services/sts/sts-core/src/main/java/org/apache/cxf/sts/token/validator/UsernameTokenValidator.java
@@ -102,10 +102,7 @@ public class UsernameTokenValidator implements TokenValidator {
* ReceivedToken argument. The realm is ignored in this token Validator.
*/
public boolean canHandleToken(ReceivedToken validateTarget, String realm) {
- if (validateTarget.getToken() instanceof UsernameTokenType) {
- return true;
- }
- return false;
+ return validateTarget.getToken() instanceof UsernameTokenType;
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/services/sts/sts-core/src/test/java/org/apache/cxf/sts/operation/DummyTokenProvider.java
----------------------------------------------------------------------
diff --git a/services/sts/sts-core/src/test/java/org/apache/cxf/sts/operation/DummyTokenProvider.java b/services/sts/sts-core/src/test/java/org/apache/cxf/sts/operation/DummyTokenProvider.java
index 14b98e8..87b7ea3 100644
--- a/services/sts/sts-core/src/test/java/org/apache/cxf/sts/operation/DummyTokenProvider.java
+++ b/services/sts/sts-core/src/test/java/org/apache/cxf/sts/operation/DummyTokenProvider.java
@@ -39,10 +39,7 @@ public class DummyTokenProvider implements TokenProvider {
WSConstants.SOAPMESSAGE_NS + "#Base64Binary";
public boolean canHandleToken(String tokenType) {
- if (TOKEN_TYPE.equals(tokenType)) {
- return true;
- }
- return false;
+ return TOKEN_TYPE.equals(tokenType);
}
public boolean canHandleToken(String tokenType, String realm) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/services/sts/sts-core/src/test/java/org/apache/cxf/sts/operation/DummyTokenValidator.java
----------------------------------------------------------------------
diff --git a/services/sts/sts-core/src/test/java/org/apache/cxf/sts/operation/DummyTokenValidator.java b/services/sts/sts-core/src/test/java/org/apache/cxf/sts/operation/DummyTokenValidator.java
index 78dd303..13f4424 100644
--- a/services/sts/sts-core/src/test/java/org/apache/cxf/sts/operation/DummyTokenValidator.java
+++ b/services/sts/sts-core/src/test/java/org/apache/cxf/sts/operation/DummyTokenValidator.java
@@ -37,11 +37,8 @@ public class DummyTokenValidator implements TokenValidator {
public boolean canHandleToken(ReceivedToken validateTarget) {
Object token = validateTarget.getToken();
- if ((token instanceof BinarySecurityTokenType)
- && TOKEN_TYPE.equals(((BinarySecurityTokenType)token).getValueType())) {
- return true;
- }
- return false;
+ return (token instanceof BinarySecurityTokenType)
+ && TOKEN_TYPE.equals(((BinarySecurityTokenType)token).getValueType());
}
public boolean canHandleToken(ReceivedToken validateTarget, String realm) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/services/sts/systests/advanced/src/test/java/org/apache/cxf/systest/sts/deployment/CustomBSTTokenProvider.java
----------------------------------------------------------------------
diff --git a/services/sts/systests/advanced/src/test/java/org/apache/cxf/systest/sts/deployment/CustomBSTTokenProvider.java b/services/sts/systests/advanced/src/test/java/org/apache/cxf/systest/sts/deployment/CustomBSTTokenProvider.java
index ab1f39a..4d5dfd2 100644
--- a/services/sts/systests/advanced/src/test/java/org/apache/cxf/systest/sts/deployment/CustomBSTTokenProvider.java
+++ b/services/sts/systests/advanced/src/test/java/org/apache/cxf/systest/sts/deployment/CustomBSTTokenProvider.java
@@ -40,10 +40,7 @@ public class CustomBSTTokenProvider implements TokenProvider {
WSConstants.SOAPMESSAGE_NS + "#Base64Binary";
public boolean canHandleToken(String tokenType) {
- if (TOKEN_TYPE.equals(tokenType)) {
- return true;
- }
- return false;
+ return TOKEN_TYPE.equals(tokenType);
}
public boolean canHandleToken(String tokenType, String realm) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/services/sts/systests/advanced/src/test/java/org/apache/cxf/systest/sts/deployment/CustomBSTTokenValidator.java
----------------------------------------------------------------------
diff --git a/services/sts/systests/advanced/src/test/java/org/apache/cxf/systest/sts/deployment/CustomBSTTokenValidator.java b/services/sts/systests/advanced/src/test/java/org/apache/cxf/systest/sts/deployment/CustomBSTTokenValidator.java
index 6879ec2..67f5b57 100644
--- a/services/sts/systests/advanced/src/test/java/org/apache/cxf/systest/sts/deployment/CustomBSTTokenValidator.java
+++ b/services/sts/systests/advanced/src/test/java/org/apache/cxf/systest/sts/deployment/CustomBSTTokenValidator.java
@@ -39,11 +39,8 @@ public class CustomBSTTokenValidator implements TokenValidator {
public boolean canHandleToken(ReceivedToken validateTarget) {
Object token = validateTarget.getToken();
- if ((token instanceof BinarySecurityTokenType)
- && TOKEN_TYPE.equals(((BinarySecurityTokenType)token).getValueType())) {
- return true;
- }
- return false;
+ return (token instanceof BinarySecurityTokenType)
+ && TOKEN_TYPE.equals(((BinarySecurityTokenType)token).getValueType());
}
public boolean canHandleToken(ReceivedToken validateTarget, String realm) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/services/sts/systests/advanced/src/test/java/org/apache/cxf/systest/sts/distributed_caching/CustomUsernameTokenProvider.java
----------------------------------------------------------------------
diff --git a/services/sts/systests/advanced/src/test/java/org/apache/cxf/systest/sts/distributed_caching/CustomUsernameTokenProvider.java b/services/sts/systests/advanced/src/test/java/org/apache/cxf/systest/sts/distributed_caching/CustomUsernameTokenProvider.java
index 1023c1f..b03bd95 100644
--- a/services/sts/systests/advanced/src/test/java/org/apache/cxf/systest/sts/distributed_caching/CustomUsernameTokenProvider.java
+++ b/services/sts/systests/advanced/src/test/java/org/apache/cxf/systest/sts/distributed_caching/CustomUsernameTokenProvider.java
@@ -39,10 +39,7 @@ public class CustomUsernameTokenProvider implements TokenProvider {
"http://docs.oasis-open.org/wss/2004/01/oasis-200401-wss-username-token-profile-1.0#UsernameToken";
public boolean canHandleToken(String tokenType) {
- if (TOKEN_TYPE.equals(tokenType)) {
- return true;
- }
- return false;
+ return TOKEN_TYPE.equals(tokenType);
}
public boolean canHandleToken(String tokenType, String realm) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/services/sts/systests/advanced/src/test/java/org/apache/cxf/systest/sts/secure_conv/SCTSAMLTokenProvider.java
----------------------------------------------------------------------
diff --git a/services/sts/systests/advanced/src/test/java/org/apache/cxf/systest/sts/secure_conv/SCTSAMLTokenProvider.java b/services/sts/systests/advanced/src/test/java/org/apache/cxf/systest/sts/secure_conv/SCTSAMLTokenProvider.java
index 4087ca1..81f257e 100644
--- a/services/sts/systests/advanced/src/test/java/org/apache/cxf/systest/sts/secure_conv/SCTSAMLTokenProvider.java
+++ b/services/sts/systests/advanced/src/test/java/org/apache/cxf/systest/sts/secure_conv/SCTSAMLTokenProvider.java
@@ -75,11 +75,8 @@ public class SCTSAMLTokenProvider implements TokenProvider {
* that corresponds to the given TokenType.
*/
public boolean canHandleToken(String tokenType) {
- if (WSConstants.WSS_SAML2_TOKEN_TYPE.equals(tokenType) || WSConstants.SAML2_NS.equals(tokenType)
- || WSConstants.WSS_SAML_TOKEN_TYPE.equals(tokenType) || WSConstants.SAML_NS.equals(tokenType)) {
- return true;
- }
- return false;
+ return WSConstants.WSS_SAML2_TOKEN_TYPE.equals(tokenType) || WSConstants.SAML2_NS.equals(tokenType)
+ || WSConstants.WSS_SAML_TOKEN_TYPE.equals(tokenType) || WSConstants.SAML_NS.equals(tokenType);
}
public boolean canHandleToken(String tokenType, String realm) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/services/sts/systests/basic/src/test/java/org/apache/cxf/systest/sts/username_onbehalfof/UsernameTokenValidator.java
----------------------------------------------------------------------
diff --git a/services/sts/systests/basic/src/test/java/org/apache/cxf/systest/sts/username_onbehalfof/UsernameTokenValidator.java b/services/sts/systests/basic/src/test/java/org/apache/cxf/systest/sts/username_onbehalfof/UsernameTokenValidator.java
index dfad100..ecd662f 100644
--- a/services/sts/systests/basic/src/test/java/org/apache/cxf/systest/sts/username_onbehalfof/UsernameTokenValidator.java
+++ b/services/sts/systests/basic/src/test/java/org/apache/cxf/systest/sts/username_onbehalfof/UsernameTokenValidator.java
@@ -41,10 +41,7 @@ public class UsernameTokenValidator implements TokenValidator {
* ReceivedToken argument. The realm is ignored in this token Validator.
*/
public boolean canHandleToken(ReceivedToken validateTarget, String realm) {
- if (validateTarget.getToken() instanceof UsernameTokenType) {
- return true;
- }
- return false;
+ return validateTarget.getToken() instanceof UsernameTokenType;
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/services/ws-discovery/ws-discovery-api/src/main/java/org/apache/cxf/ws/discovery/internal/WSDiscoveryServiceImpl.java
----------------------------------------------------------------------
diff --git a/services/ws-discovery/ws-discovery-api/src/main/java/org/apache/cxf/ws/discovery/internal/WSDiscoveryServiceImpl.java b/services/ws-discovery/ws-discovery-api/src/main/java/org/apache/cxf/ws/discovery/internal/WSDiscoveryServiceImpl.java
index de8a0e6..713034b 100644
--- a/services/ws-discovery/ws-discovery-api/src/main/java/org/apache/cxf/ws/discovery/internal/WSDiscoveryServiceImpl.java
+++ b/services/ws-discovery/ws-discovery-api/src/main/java/org/apache/cxf/ws/discovery/internal/WSDiscoveryServiceImpl.java
@@ -594,10 +594,7 @@ public class WSDiscoveryServiceImpl implements WSDiscoveryService {
return false;
}
if (addr.equals(addc)) {
- if (snr != null && !snr.equals(snc)) {
- return false;
- }
- return true;
+ return !(snr != null && !snr.equals(snc));
}
return false;
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/security/AllowAllHostnameVerifier.java
----------------------------------------------------------------------
diff --git a/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/security/AllowAllHostnameVerifier.java b/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/security/AllowAllHostnameVerifier.java
index f440240..b62c950 100644
--- a/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/security/AllowAllHostnameVerifier.java
+++ b/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/security/AllowAllHostnameVerifier.java
@@ -34,10 +34,7 @@ class AllowAllHostnameVerifier implements javax.net.ssl.HostnameVerifier {
public boolean verify(String host, SSLSession session) {
try {
Certificate[] certs = session.getPeerCertificates();
- if (certs != null && certs[0] instanceof X509Certificate) {
- return true;
- }
- return false;
+ return certs != null && certs[0] instanceof X509Certificate;
} catch (SSLException e) {
return false;
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/systests/uncategorized/src/test/java/org/apache/cxf/systest/type_test/AbstractTypeTestClient.java
----------------------------------------------------------------------
diff --git a/systests/uncategorized/src/test/java/org/apache/cxf/systest/type_test/AbstractTypeTestClient.java b/systests/uncategorized/src/test/java/org/apache/cxf/systest/type_test/AbstractTypeTestClient.java
index be285e4..6c1aa2b 100644
--- a/systests/uncategorized/src/test/java/org/apache/cxf/systest/type_test/AbstractTypeTestClient.java
+++ b/systests/uncategorized/src/test/java/org/apache/cxf/systest/type_test/AbstractTypeTestClient.java
@@ -60,12 +60,9 @@ public abstract class AbstractTypeTestClient
}
public boolean shouldRunTest(String name) {
- if (System.getProperty("java.vendor").contains("IBM")
- && "GMonth".equals(name)) {
- //the validator in ibm doesn't like this type.
- return false;
- }
- return true;
+ //the validator in ibm doesn't like this type.
+ return !(System.getProperty("java.vendor").contains("IBM")
+ && "GMonth".equals(name));
}
public static void initClient(Class<?> clz, QName serviceName,
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/systests/uncategorized/src/test/java/org/apache/cxf/systest/type_test/AbstractTypeTestClient3.java
----------------------------------------------------------------------
diff --git a/systests/uncategorized/src/test/java/org/apache/cxf/systest/type_test/AbstractTypeTestClient3.java b/systests/uncategorized/src/test/java/org/apache/cxf/systest/type_test/AbstractTypeTestClient3.java
index e66865e..a55f6b4 100644
--- a/systests/uncategorized/src/test/java/org/apache/cxf/systest/type_test/AbstractTypeTestClient3.java
+++ b/systests/uncategorized/src/test/java/org/apache/cxf/systest/type_test/AbstractTypeTestClient3.java
@@ -1580,12 +1580,9 @@ public abstract class AbstractTypeTestClient3 extends AbstractTypeTestClient2 {
protected boolean equals(ChoiceWithGroups x, ChoiceWithGroups y) {
if (x.getVarInt() != null && x.getVarString() != null
&& x.getVarFloat() != null) {
- if (x.getVarInt().compareTo(y.getVarInt()) == 0
+ return x.getVarInt().compareTo(y.getVarInt()) == 0
&& x.getVarString().equals(y.getVarString())
- && x.getVarFloat().compareTo(y.getVarFloat()) == 0) {
- return true;
- }
- return false;
+ && x.getVarFloat().compareTo(y.getVarFloat()) == 0;
} else {
if (x.getVarOtherFloat() != null && y.getVarOtherFloat() != null) {
return x.getVarOtherFloat().compareTo(y.getVarOtherFloat()) == 0;
@@ -2193,10 +2190,7 @@ public abstract class AbstractTypeTestClient3 extends AbstractTypeTestClient2 {
if (x.getVarString() != null && !x.getVarString().equals(y.getVarString())) {
return false;
}
- if (x.getVarInt() != null && x.getVarInt() != y.getVarInt()) {
- return false;
- }
- return true;
+ return !(x.getVarInt() != null && x.getVarInt() != y.getVarInt());
}
@Test
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/systests/ws-rm/src/test/java/org/apache/cxf/systest/ws/util/MessageFlow.java
----------------------------------------------------------------------
diff --git a/systests/ws-rm/src/test/java/org/apache/cxf/systest/ws/util/MessageFlow.java b/systests/ws-rm/src/test/java/org/apache/cxf/systest/ws/util/MessageFlow.java
index 7d17d7b..03919a5 100644
--- a/systests/ws-rm/src/test/java/org/apache/cxf/systest/ws/util/MessageFlow.java
+++ b/systests/ws-rm/src/test/java/org/apache/cxf/systest/ws/util/MessageFlow.java
@@ -516,10 +516,7 @@ public class MessageFlow extends Assert {
break;
}
}
- if (null != bodyElement && bodyElement.hasChildNodes()) {
- return false;
- }
- return true;
+ return !(null != bodyElement && bodyElement.hasChildNodes());
}
private String dump(List<byte[]> streams) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/systests/ws-security/src/test/java/org/apache/cxf/systest/ws/wssec11/WSSecurity11Common.java
----------------------------------------------------------------------
diff --git a/systests/ws-security/src/test/java/org/apache/cxf/systest/ws/wssec11/WSSecurity11Common.java b/systests/ws-security/src/test/java/org/apache/cxf/systest/ws/wssec11/WSSecurity11Common.java
index 27327e2..e565871 100644
--- a/systests/ws-security/src/test/java/org/apache/cxf/systest/ws/wssec11/WSSecurity11Common.java
+++ b/systests/ws-security/src/test/java/org/apache/cxf/systest/ws/wssec11/WSSecurity11Common.java
@@ -113,10 +113,6 @@ public class WSSecurity11Common extends AbstractBusClientServerTestBase {
} else {
javaVersionNum = new Double(javaVersion).doubleValue();
}
- if (javaVersionNum < 1.6) {
- return false;
- }
-
- return true;
+ return !(javaVersionNum < 1.6);
}
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/util/MessageFlow.java
----------------------------------------------------------------------
diff --git a/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/util/MessageFlow.java b/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/util/MessageFlow.java
index 7d17d7b..03919a5 100644
--- a/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/util/MessageFlow.java
+++ b/systests/ws-specs/src/test/java/org/apache/cxf/systest/ws/util/MessageFlow.java
@@ -516,10 +516,7 @@ public class MessageFlow extends Assert {
break;
}
}
- if (null != bodyElement && bodyElement.hasChildNodes()) {
- return false;
- }
- return true;
+ return !(null != bodyElement && bodyElement.hasChildNodes());
}
private String dump(List<byte[]> streams) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/testutils/src/main/java/org/apache/cxf/testutil/common/AbstractBusClientServerTestBase.java
----------------------------------------------------------------------
diff --git a/testutils/src/main/java/org/apache/cxf/testutil/common/AbstractBusClientServerTestBase.java b/testutils/src/main/java/org/apache/cxf/testutil/common/AbstractBusClientServerTestBase.java
index fc1bfcf..17987fd 100644
--- a/testutils/src/main/java/org/apache/cxf/testutil/common/AbstractBusClientServerTestBase.java
+++ b/testutils/src/main/java/org/apache/cxf/testutil/common/AbstractBusClientServerTestBase.java
@@ -114,9 +114,6 @@ public abstract class AbstractBusClientServerTestBase extends AbstractClientServ
tpe.execute(client);
tpe.shutdown();
tpe.awaitTermination(timeOut, timeUnit);
- if (!client.isDone()) {
- return false;
- }
- return true;
+ return client.isDone();
}
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/common/src/main/java/org/apache/cxf/tools/common/AbstractCXFToolContainer.java
----------------------------------------------------------------------
diff --git a/tools/common/src/main/java/org/apache/cxf/tools/common/AbstractCXFToolContainer.java b/tools/common/src/main/java/org/apache/cxf/tools/common/AbstractCXFToolContainer.java
index 905688c..96fc51c 100644
--- a/tools/common/src/main/java/org/apache/cxf/tools/common/AbstractCXFToolContainer.java
+++ b/tools/common/src/main/java/org/apache/cxf/tools/common/AbstractCXFToolContainer.java
@@ -62,10 +62,7 @@ public abstract class AbstractCXFToolContainer extends AbstractToolContainer {
if (commandDocument == null) {
return false;
}
- if ((commandDocument.hasParameter("help")) || (commandDocument.hasParameter("version"))) {
- return true;
- }
- return false;
+ return commandDocument.hasParameter("help") || commandDocument.hasParameter("version");
}
public void execute(boolean exitOnFinish) throws ToolException {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/common/src/main/java/org/apache/cxf/tools/common/ToolContext.java
----------------------------------------------------------------------
diff --git a/tools/common/src/main/java/org/apache/cxf/tools/common/ToolContext.java b/tools/common/src/main/java/org/apache/cxf/tools/common/ToolContext.java
index c3535dc..735b202 100644
--- a/tools/common/src/main/java/org/apache/cxf/tools/common/ToolContext.java
+++ b/tools/common/src/main/java/org/apache/cxf/tools/common/ToolContext.java
@@ -174,20 +174,14 @@ public class ToolContext {
if (s instanceof String && ((String)s).length() > 0 && ((String)s).charAt(0) == '=') {
s = ((String)s).substring(1);
}
- if (s == null || "none".equals(s) || "false".equals(s) || "basic".equals(s)) {
- return false;
- }
- return true;
+ return !(s == null || "none".equals(s) || "false".equals(s) || "basic".equals(s));
}
public boolean basicValidateWSDL() {
Object s = get(ToolConstants.CFG_VALIDATE_WSDL);
if (s instanceof String && ((String)s).length() > 0 && ((String)s).charAt(0) == '=') {
s = ((String)s).substring(1);
}
- if ("none".equals(s) || "false".equals(s)) {
- return false;
- }
- return true;
+ return !("none".equals(s) || "false".equals(s));
}
public void addNamespacePackageMap(String namespace, String pn) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/AttributeVisitor.java
----------------------------------------------------------------------
diff --git a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/AttributeVisitor.java b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/AttributeVisitor.java
index d53ed6d..a7acbd2 100644
--- a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/AttributeVisitor.java
+++ b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/AttributeVisitor.java
@@ -85,11 +85,8 @@ public class AttributeVisitor extends VisitorBase {
}
public static boolean accept(AST node) {
- if (node.getType() == IDLTokenTypes.LITERAL_readonly
- || node.getType() == IDLTokenTypes.LITERAL_attribute) {
- return true;
- }
- return false;
+ return node.getType() == IDLTokenTypes.LITERAL_readonly
+ || node.getType() == IDLTokenTypes.LITERAL_attribute;
}
public void visit(AST attributeNode) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/ConstVisitor.java
----------------------------------------------------------------------
diff --git a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/ConstVisitor.java b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/ConstVisitor.java
index c56e14e..ba542e1 100644
--- a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/ConstVisitor.java
+++ b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/ConstVisitor.java
@@ -39,10 +39,7 @@ public class ConstVisitor extends VisitorBase {
}
public static boolean accept(AST node) {
- if (node.getType() == IDLTokenTypes.LITERAL_const) {
- return true;
- }
- return false;
+ return node.getType() == IDLTokenTypes.LITERAL_const;
}
public void visit(AST constNode) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/EnumVisitor.java
----------------------------------------------------------------------
diff --git a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/EnumVisitor.java b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/EnumVisitor.java
index 20e8d80..792b9c1 100644
--- a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/EnumVisitor.java
+++ b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/EnumVisitor.java
@@ -42,10 +42,7 @@ public class EnumVisitor extends VisitorBase {
}
public static boolean accept(AST node) {
- if (node.getType() == IDLTokenTypes.LITERAL_enum) {
- return true;
- }
- return false;
+ return node.getType() == IDLTokenTypes.LITERAL_enum;
}
public void visit(AST enumNode) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/ExceptionVisitor.java
----------------------------------------------------------------------
diff --git a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/ExceptionVisitor.java b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/ExceptionVisitor.java
index 74c8b47..40b24b1 100644
--- a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/ExceptionVisitor.java
+++ b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/ExceptionVisitor.java
@@ -47,10 +47,7 @@ public class ExceptionVisitor extends VisitorBase {
}
public static boolean accept(AST node) {
- if (node.getType() == IDLTokenTypes.LITERAL_exception) {
- return true;
- }
- return false;
+ return node.getType() == IDLTokenTypes.LITERAL_exception;
}
public void visit(AST node) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/FixedPtConstVisitor.java
----------------------------------------------------------------------
diff --git a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/FixedPtConstVisitor.java b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/FixedPtConstVisitor.java
index f8904c2..767b91e 100644
--- a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/FixedPtConstVisitor.java
+++ b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/FixedPtConstVisitor.java
@@ -49,10 +49,7 @@ public class FixedPtConstVisitor implements Visitor {
public static boolean accept(AST node) {
- if (node.getType() == IDLTokenTypes.LITERAL_fixed) {
- return true;
- }
- return false;
+ return node.getType() == IDLTokenTypes.LITERAL_fixed;
}
public void visit(AST fixedNode) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/FixedVisitor.java
----------------------------------------------------------------------
diff --git a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/FixedVisitor.java b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/FixedVisitor.java
index eb4f1a9..6b108cf 100644
--- a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/FixedVisitor.java
+++ b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/FixedVisitor.java
@@ -46,10 +46,7 @@ public class FixedVisitor extends VisitorBase {
}
public static boolean accept(AST node) {
- if (node.getType() == IDLTokenTypes.LITERAL_fixed) {
- return true;
- }
- return false;
+ return node.getType() == IDLTokenTypes.LITERAL_fixed;
}
public void visit(AST fixedNode) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/PortTypeVisitor.java
----------------------------------------------------------------------
diff --git a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/PortTypeVisitor.java b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/PortTypeVisitor.java
index d443847..c82aa25 100644
--- a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/PortTypeVisitor.java
+++ b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/PortTypeVisitor.java
@@ -60,10 +60,7 @@ public class PortTypeVisitor extends VisitorBase {
}
public static boolean accept(AST node) {
- if (node.getType() == IDLTokenTypes.LITERAL_interface) {
- return true;
- }
- return false;
+ return node.getType() == IDLTokenTypes.LITERAL_interface;
}
public void visit(AST node) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/ScopedNameVisitor.java
----------------------------------------------------------------------
diff --git a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/ScopedNameVisitor.java b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/ScopedNameVisitor.java
index 2d1fb67..a1a2585 100644
--- a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/ScopedNameVisitor.java
+++ b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/ScopedNameVisitor.java
@@ -562,13 +562,10 @@ public class ScopedNameVisitor extends VisitorBase {
}
protected static boolean isFullyScopedName(AST node) {
- if (node.getType() == IDLTokenTypes.IDENT
+ return node.getType() == IDLTokenTypes.IDENT
&& node.getFirstChild() != null
&& ((node.getFirstChild().getType() == IDLTokenTypes.SCOPEOP)
- || (node.getFirstChild().getType() == IDLTokenTypes.IDENT))) {
- return true;
- }
- return false;
+ || (node.getFirstChild().getType() == IDLTokenTypes.IDENT));
}
protected static Scope getFullyScopedName(Scope currentScope, AST node) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/SequenceVisitor.java
----------------------------------------------------------------------
diff --git a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/SequenceVisitor.java b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/SequenceVisitor.java
index 384583c..4666d31 100644
--- a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/SequenceVisitor.java
+++ b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/SequenceVisitor.java
@@ -51,10 +51,7 @@ public class SequenceVisitor extends VisitorBase {
}
public static boolean accept(AST node) {
- if (node.getType() == IDLTokenTypes.LITERAL_sequence) {
- return true;
- }
- return false;
+ return node.getType() == IDLTokenTypes.LITERAL_sequence;
}
public void visit(AST seq) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/StringVisitor.java
----------------------------------------------------------------------
diff --git a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/StringVisitor.java b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/StringVisitor.java
index 726f43e..1a06fa0 100644
--- a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/StringVisitor.java
+++ b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/StringVisitor.java
@@ -56,18 +56,12 @@ public class StringVisitor extends VisitorBase {
}
public static boolean accept(AST node) {
- if ((node.getType() == IDLTokenTypes.LITERAL_string)
- || (node.getType() == IDLTokenTypes.LITERAL_wstring)) {
- return true;
- }
- return false;
+ return (node.getType() == IDLTokenTypes.LITERAL_string)
+ || (node.getType() == IDLTokenTypes.LITERAL_wstring);
}
public static boolean isBounded(AST node) {
- if (node.getFirstChild() == null) {
- return false;
- }
- return true;
+ return node.getFirstChild() != null;
}
public void visit(AST node) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/StructVisitor.java
----------------------------------------------------------------------
diff --git a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/StructVisitor.java b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/StructVisitor.java
index cb2d23f..aa26056 100644
--- a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/StructVisitor.java
+++ b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/StructVisitor.java
@@ -47,10 +47,7 @@ public class StructVisitor extends VisitorBase {
}
public static boolean accept(AST node) {
- if (node.getType() == IDLTokenTypes.LITERAL_struct) {
- return true;
- }
- return false;
+ return node.getType() == IDLTokenTypes.LITERAL_struct;
}
public void visit(AST node) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/TypedefVisitor.java
----------------------------------------------------------------------
diff --git a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/TypedefVisitor.java b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/TypedefVisitor.java
index 97bf6f5..b300f70 100644
--- a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/TypedefVisitor.java
+++ b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/TypedefVisitor.java
@@ -41,10 +41,7 @@ public class TypedefVisitor extends VisitorBase {
}
public static boolean accept(AST node) {
- if (node.getType() == IDLTokenTypes.LITERAL_typedef) {
- return true;
- }
- return false;
+ return node.getType() == IDLTokenTypes.LITERAL_typedef;
}
public void visit(AST typedefNode) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/UnionVisitor.java
----------------------------------------------------------------------
diff --git a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/UnionVisitor.java b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/UnionVisitor.java
index 1b06f0b..8f56ded 100644
--- a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/UnionVisitor.java
+++ b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/idl/UnionVisitor.java
@@ -48,10 +48,7 @@ public class UnionVisitor extends VisitorBase {
}
public static boolean accept(AST node) {
- if (node.getType() == IDLTokenTypes.LITERAL_union) {
- return true;
- }
- return false;
+ return node.getType() == IDLTokenTypes.LITERAL_union;
}
public void visit(AST unionNode) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/wsdl/WSDLParameter.java
----------------------------------------------------------------------
diff --git a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/wsdl/WSDLParameter.java b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/wsdl/WSDLParameter.java
index 6b93661..f55882f 100644
--- a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/wsdl/WSDLParameter.java
+++ b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/wsdl/WSDLParameter.java
@@ -614,10 +614,7 @@ public final class WSDLParameter {
private boolean hasAttributes(XmlSchemaComplexType complexType) {
// Now lets see if we have any attributes...
// This should probably look at the restricted and substitute types too.
- if (complexType.getAnyAttribute() != null || complexType.getAttributes().size() > 0) {
- return true;
- }
- return false;
+ return complexType.getAnyAttribute() != null || complexType.getAttributes().size() > 0;
}
private boolean isWrappableSequence(XmlSchemaComplexType type) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/wsdl/WSDLToCorbaHelper.java
----------------------------------------------------------------------
diff --git a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/wsdl/WSDLToCorbaHelper.java b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/wsdl/WSDLToCorbaHelper.java
index ad5fdfb..9f2fed0 100644
--- a/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/wsdl/WSDLToCorbaHelper.java
+++ b/tools/corba/src/main/java/org/apache/cxf/tools/corba/processors/wsdl/WSDLToCorbaHelper.java
@@ -1594,11 +1594,7 @@ public class WSDLToCorbaHelper {
}
private boolean isIDLObjectType(QName typeName) {
- if (typeName.equals(ReferenceConstants.WSADDRESSING_TYPE)) {
- return true;
- }
-
- return false;
+ return typeName.equals(ReferenceConstants.WSADDRESSING_TYPE);
}
private boolean isAddressingNamespace(QName typeName) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/FrontendFactory.java
----------------------------------------------------------------------
diff --git a/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/FrontendFactory.java b/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/FrontendFactory.java
index eb2fde4..0814227 100644
--- a/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/FrontendFactory.java
+++ b/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/FrontendFactory.java
@@ -70,10 +70,7 @@ public final class FrontendFactory {
return true;
}
}
- if (isJAXWSAnnotationExists()) {
- return true;
- }
- return false;
+ return isJAXWSAnnotationExists();
}
private boolean isJAXWSAnnotationExists() {
@@ -108,10 +105,7 @@ public final class FrontendFactory {
private boolean isExcluced(Method method) {
WebMethod webMethod = AnnotationUtil.getPrivMethodAnnotation(method, WebMethod.class);
- if (webMethod != null && webMethod.exclude()) {
- return true;
- }
- return false;
+ return webMethod != null && webMethod.exclude();
}
public Style discoverStyle() {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/AntGenerator.java
----------------------------------------------------------------------
diff --git a/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/AntGenerator.java b/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/AntGenerator.java
index 80c2d89..f8d0094 100644
--- a/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/AntGenerator.java
+++ b/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/AntGenerator.java
@@ -41,10 +41,7 @@ public class AntGenerator extends AbstractGenerator {
}
public boolean passthrough() {
- if (env.optionSet(ToolConstants.CFG_ANT)) {
- return false;
- }
- return true;
+ return !env.optionSet(ToolConstants.CFG_ANT);
}
public void generate(ToolContext penv) throws ToolException {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/JAXWSFrontEndProcessor.java
----------------------------------------------------------------------
diff --git a/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/JAXWSFrontEndProcessor.java b/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/JAXWSFrontEndProcessor.java
index 7027085..e4ab80a 100644
--- a/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/JAXWSFrontEndProcessor.java
+++ b/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/JAXWSFrontEndProcessor.java
@@ -182,10 +182,7 @@ public class JAXWSFrontEndProcessor implements Processor {
getInfClass(returnClass);
}
}
- if (infList.contains("java.rmi.Remote")) {
- return true;
- }
- return false;
+ return infList.contains("java.rmi.Remote");
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/generator/JaxwsClientGenerator.java
----------------------------------------------------------------------
diff --git a/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/generator/JaxwsClientGenerator.java b/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/generator/JaxwsClientGenerator.java
index 590071d..8494c60 100644
--- a/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/generator/JaxwsClientGenerator.java
+++ b/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/generator/JaxwsClientGenerator.java
@@ -36,12 +36,9 @@ public class JaxwsClientGenerator extends AbstractJaxwsGenerator {
public boolean passthrough() {
- if (env.optionSet(ToolConstants.CFG_CLIENT)) {
- return false;
- }
- return true;
+ return !env.optionSet(ToolConstants.CFG_CLIENT);
}
-
+
public void generate(ToolContext penv) throws ToolException {
this.env = penv;
JavaModel javaModel = env.get(JavaModel.class);
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/generator/JaxwsImplGenerator.java
----------------------------------------------------------------------
diff --git a/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/generator/JaxwsImplGenerator.java b/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/generator/JaxwsImplGenerator.java
index 6f69f49..2f439a1 100644
--- a/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/generator/JaxwsImplGenerator.java
+++ b/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/generator/JaxwsImplGenerator.java
@@ -38,11 +38,8 @@ public class JaxwsImplGenerator extends AbstractJaxwsGenerator {
public boolean passthrough() {
Boolean genFromSei = (Boolean)env.get(ToolConstants.GEN_FROM_SEI);
- if (genFromSei && env.optionSet(ToolConstants.CFG_SERVER)
- && (!env.optionSet(ToolConstants.IMPL_CLASS))) {
- return false;
- }
- return true;
+ return !(genFromSei && env.optionSet(ToolConstants.CFG_SERVER)
+ && (!env.optionSet(ToolConstants.IMPL_CLASS)));
}
public void generate(ToolContext penv) throws ToolException {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/generator/JaxwsSEIGenerator.java
----------------------------------------------------------------------
diff --git a/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/generator/JaxwsSEIGenerator.java b/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/generator/JaxwsSEIGenerator.java
index 3abf04f..e345119 100644
--- a/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/generator/JaxwsSEIGenerator.java
+++ b/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/generator/JaxwsSEIGenerator.java
@@ -37,11 +37,8 @@ public class JaxwsSEIGenerator extends AbstractJaxwsGenerator {
public boolean passthrough() {
Boolean genFromSei = (Boolean)env.get(ToolConstants.GEN_FROM_SEI);
- if (!genFromSei && env.optionSet(ToolConstants.CFG_CLIENT)
- && (!env.optionSet(ToolConstants.SEI_CLASS))) {
- return false;
- }
- return true;
+ return !(!genFromSei && env.optionSet(ToolConstants.CFG_CLIENT)
+ && (!env.optionSet(ToolConstants.SEI_CLASS)));
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/generator/JaxwsServerGenerator.java
----------------------------------------------------------------------
diff --git a/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/generator/JaxwsServerGenerator.java b/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/generator/JaxwsServerGenerator.java
index 1852099..9e92152 100644
--- a/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/generator/JaxwsServerGenerator.java
+++ b/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/generator/JaxwsServerGenerator.java
@@ -39,10 +39,7 @@ public class JaxwsServerGenerator extends AbstractJaxwsGenerator {
}
public boolean passthrough() {
- if (env.optionSet(ToolConstants.CFG_SERVER)) {
- return false;
- }
- return true;
+ return !env.optionSet(ToolConstants.CFG_SERVER);
}
public void generate(ToolContext penv) throws ToolException {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/simple/generator/SimpleClientGenerator.java
----------------------------------------------------------------------
diff --git a/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/simple/generator/SimpleClientGenerator.java b/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/simple/generator/SimpleClientGenerator.java
index 6f0ede3..e19db77 100644
--- a/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/simple/generator/SimpleClientGenerator.java
+++ b/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/simple/generator/SimpleClientGenerator.java
@@ -35,10 +35,7 @@ public class SimpleClientGenerator extends AbstractSimpleGenerator {
}
public boolean passthrough() {
- if (env.optionSet(ToolConstants.CFG_CLIENT)) {
- return false;
- }
- return true;
+ return !env.optionSet(ToolConstants.CFG_CLIENT);
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/simple/generator/SimpleImplGenerator.java
----------------------------------------------------------------------
diff --git a/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/simple/generator/SimpleImplGenerator.java b/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/simple/generator/SimpleImplGenerator.java
index 71d6a20..8c92676 100644
--- a/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/simple/generator/SimpleImplGenerator.java
+++ b/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/simple/generator/SimpleImplGenerator.java
@@ -36,13 +36,8 @@ public class SimpleImplGenerator extends AbstractSimpleGenerator {
public boolean passthrough() {
Boolean genFromSei = (Boolean)env.get(ToolConstants.GEN_FROM_SEI);
- if (genFromSei && env.optionSet(ToolConstants.CFG_SERVER)
- && (!env.optionSet(ToolConstants.IMPL_CLASS))) {
- return false;
- }
-
- return true;
-
+ return !(genFromSei && env.optionSet(ToolConstants.CFG_SERVER)
+ && (!env.optionSet(ToolConstants.IMPL_CLASS)));
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/simple/generator/SimpleSEIGenerator.java
----------------------------------------------------------------------
diff --git a/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/simple/generator/SimpleSEIGenerator.java b/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/simple/generator/SimpleSEIGenerator.java
index 87c6601..a985355 100644
--- a/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/simple/generator/SimpleSEIGenerator.java
+++ b/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/simple/generator/SimpleSEIGenerator.java
@@ -36,12 +36,8 @@ public class SimpleSEIGenerator extends AbstractSimpleGenerator {
public boolean passthrough() {
Boolean genFromSei = (Boolean)env.get(ToolConstants.GEN_FROM_SEI);
- if (!genFromSei && env.optionSet(ToolConstants.CFG_CLIENT)
- && (!env.optionSet(ToolConstants.SEI_CLASS))) {
- return false;
- }
- return true;
-
+ return !(!genFromSei && env.optionSet(ToolConstants.CFG_CLIENT)
+ && (!env.optionSet(ToolConstants.SEI_CLASS)));
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/simple/generator/SimpleServerGenerator.java
----------------------------------------------------------------------
diff --git a/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/simple/generator/SimpleServerGenerator.java b/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/simple/generator/SimpleServerGenerator.java
index ad1783a..7ab74df 100644
--- a/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/simple/generator/SimpleServerGenerator.java
+++ b/tools/javato/ws/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/simple/generator/SimpleServerGenerator.java
@@ -39,10 +39,7 @@ public class SimpleServerGenerator extends AbstractSimpleGenerator {
}
public boolean passthrough() {
- if (env.optionSet(ToolConstants.CFG_SERVER)) {
- return false;
- }
- return true;
+ return !env.optionSet(ToolConstants.CFG_SERVER);
}
public void generate(ToolContext penv) throws ToolException {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/wadlto/jaxrs/src/main/java/org/apache/cxf/tools/wadlto/jaxb/CustomizationParser.java
----------------------------------------------------------------------
diff --git a/tools/wadlto/jaxrs/src/main/java/org/apache/cxf/tools/wadlto/jaxb/CustomizationParser.java b/tools/wadlto/jaxrs/src/main/java/org/apache/cxf/tools/wadlto/jaxb/CustomizationParser.java
index 1f2164e..22bd3e4 100644
--- a/tools/wadlto/jaxrs/src/main/java/org/apache/cxf/tools/wadlto/jaxb/CustomizationParser.java
+++ b/tools/wadlto/jaxrs/src/main/java/org/apache/cxf/tools/wadlto/jaxb/CustomizationParser.java
@@ -164,11 +164,7 @@ public final class CustomizationParser {
}
private boolean isValidJaxbBindingFile(XMLStreamReader reader) {
- if (ToolConstants.JAXB_BINDINGS.equals(reader.getName())) {
-
- return true;
- }
- return false;
+ return ToolConstants.JAXB_BINDINGS.equals(reader.getName());
}
private String[] getBindingFiles(ToolContext env) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/tools/wsdlto/core/src/main/java/org/apache/cxf/tools/wsdlto/WSDLToJavaContainer.java
----------------------------------------------------------------------
diff --git a/tools/wsdlto/core/src/main/java/org/apache/cxf/tools/wsdlto/WSDLToJavaContainer.java b/tools/wsdlto/core/src/main/java/org/apache/cxf/tools/wsdlto/WSDLToJavaContainer.java
index 3b57251..1ba192a 100644
--- a/tools/wsdlto/core/src/main/java/org/apache/cxf/tools/wsdlto/WSDLToJavaContainer.java
+++ b/tools/wsdlto/core/src/main/java/org/apache/cxf/tools/wsdlto/WSDLToJavaContainer.java
@@ -711,10 +711,7 @@ public class WSDLToJavaContainer extends AbstractCXFToolContainer {
|| context.optionSet(ToolConstants.CFG_GEN_FAULT)) {
return true;
}
- if (context.optionSet(ToolConstants.CFG_NO_TYPES)) {
- return true;
- }
- return false;
+ return context.optionSet(ToolConstants.CFG_NO_TYPES);
}
public void generateTypes() throws ToolException {
[3/3] cxf git commit: The new PMD plugin is strict about simplifying
boolean returns
Posted by dk...@apache.org.
The new PMD plugin is strict about simplifying boolean returns
Project: http://git-wip-us.apache.org/repos/asf/cxf/repo
Commit: http://git-wip-us.apache.org/repos/asf/cxf/commit/3309231e
Tree: http://git-wip-us.apache.org/repos/asf/cxf/tree/3309231e
Diff: http://git-wip-us.apache.org/repos/asf/cxf/diff/3309231e
Branch: refs/heads/master
Commit: 3309231e467225b18b24d90d77153a0c572a17e0
Parents: 4387055
Author: Daniel Kulp <dk...@apache.org>
Authored: Fri Apr 3 12:00:25 2015 -0400
Committer: Daniel Kulp <dk...@apache.org>
Committed: Fri Apr 3 12:02:47 2015 -0400
----------------------------------------------------------------------
.../apache/cxf/common/util/PropertyUtils.java | 12 +---
.../org/apache/cxf/common/util/StringUtils.java | 5 +-
.../cxf/interceptor/FIStaxInInterceptor.java | 7 +--
.../org/apache/cxf/staxutils/StaxUtils.java | 7 +--
.../staxutils/transform/OutTransformWriter.java | 7 +--
.../cxf/jca/cxf/ManagedConnectionImpl.java | 5 +-
.../cxf/jca/outbound/CXFConnectionSpec.java | 6 +-
.../outbound/ManagedConnectionFactoryImpl.java | 6 +-
.../org/apache/cxf/jca/servant/EJBEndpoint.java | 10 +---
parent/pom.xml | 4 +-
.../cxf/binding/corba/utils/CorbaUtils.java | 5 +-
.../cxf/aegis/type/basic/BeanTypeInfo.java | 6 +-
.../apache/cxf/aegis/inheritance/ws1/BeanA.java | 6 +-
.../apache/cxf/aegis/inheritance/ws1/BeanB.java | 6 +-
.../apache/cxf/aegis/inheritance/ws1/BeanC.java | 6 +-
.../cxf/aegis/inheritance/ws1/ResultBean.java | 6 +-
.../cxf/aegis/inheritance/ws1/RootBean.java | 6 +-
.../cxf/aegis/inheritance/ws1/WS1Exception.java | 6 +-
.../inheritance/ws1/WS1ExtendedException.java | 6 +-
.../inheritance/ws2/common/ParentBean.java | 6 +-
.../exception/AlreadyExistsException.java | 6 +-
.../ws2/common/exception/NotFoundException.java | 6 +-
.../ws2/common/pack1/ContentBean1.java | 6 +-
.../ws2/common/pack2/ContentBean2.java | 6 +-
.../org/apache/cxf/jaxb/JAXBEncoderDecoder.java | 5 +-
.../cxf/jaxrs/provider/MultipartProvider.java | 7 +--
.../provider/RequestDispatcherProvider.java | 5 +-
.../apache/cxf/jaxrs/utils/ExceptionUtils.java | 6 +-
.../org/apache/cxf/jaxrs/utils/JAXRSUtils.java | 14 ++---
.../jaxrs/utils/multipart/AttachmentUtils.java | 5 +-
.../apache/cxf/frontend/WSDLGetInterceptor.java | 6 +-
.../javascript/JavascriptGetInterceptor.java | 5 +-
.../client/service/settings/Subscription.java | 6 +-
.../web/logging/atom/AtomPushEngine.java | 6 +-
.../AbstractMessageResponseTimeInterceptor.java | 5 +-
.../cxf/jaxrs/client/ClientProxyImpl.java | 7 +--
.../jaxrs/provider/atom/AtomPojoProvider.java | 5 +-
.../search/AbstractSearchConditionParser.java | 5 +-
.../cxf/rs/security/oauth/data/Client.java | 6 +-
.../security/saml/sso/AbstractSSOSpHandler.java | 6 +-
.../apache/cxf/transport/http/HTTPConduit.java | 5 +-
.../https/AllowAllHostnameVerifier.java | 7 +--
.../ws/addressing/impl/MAPAggregatorImpl.java | 12 +---
.../addressing/soap/DecoupledFaultHandler.java | 5 +-
.../external/URIDomainExpression.java | 45 ++++-----------
.../apache/cxf/ws/rm/AbstractRMInterceptor.java | 5 +-
.../cxf/ws/security/policy/PolicyUtils.java | 6 +-
.../apache/cxf/ws/security/trust/STSUtils.java | 6 +-
.../wss4j/AbstractTokenInterceptor.java | 5 +-
.../wss4j/PolicyBasedWSS4JInInterceptor.java | 6 +-
.../wss4j/StaxCryptoCoverageChecker.java | 60 ++++++--------------
.../wss4j/StaxSecurityContextInInterceptor.java | 16 ++----
.../wss4j/UsernameTokenInterceptor.java | 7 +--
.../ws/security/wss4j/WSS4JInInterceptor.java | 15 +----
.../policyhandlers/SymmetricBindingHandler.java | 8 +--
.../AbstractBindingPolicyValidator.java | 15 +----
.../AbstractSupportingTokenPolicyValidator.java | 60 +++++---------------
.../AlgorithmSuitePolicyValidator.java | 20 ++-----
.../AsymmetricBindingPolicyValidator.java | 8 +--
.../ConcreteSupportingTokenPolicyValidator.java | 8 +--
.../EncryptedTokenPolicyValidator.java | 8 +--
.../EndorsingEncryptedTokenPolicyValidator.java | 8 +--
.../EndorsingTokenPolicyValidator.java | 8 +--
.../IssuedTokenPolicyValidator.java | 8 +--
.../KerberosTokenPolicyValidator.java | 8 +--
.../policyvalidators/LayoutPolicyValidator.java | 8 +--
.../SamlTokenPolicyValidator.java | 8 +--
.../SecurityContextTokenPolicyValidator.java | 8 +--
.../SignedEncryptedTokenPolicyValidator.java | 8 +--
...dEndorsingEncryptedTokenPolicyValidator.java | 8 +--
.../SignedEndorsingTokenPolicyValidator.java | 8 +--
.../SignedTokenPolicyValidator.java | 8 +--
.../SymmetricBindingPolicyValidator.java | 8 +--
.../TransportBindingPolicyValidator.java | 8 +--
.../UsernameTokenPolicyValidator.java | 8 +--
.../policyvalidators/WSS11PolicyValidator.java | 8 +--
.../X509TokenPolicyValidator.java | 8 +--
.../apache/cxf/wsdl11/PartialWSDLProcessor.java | 21 ++-----
.../apache/cxf/wsdl11/WSDLServiceBuilder.java | 5 +-
.../sts/token/provider/SAMLTokenProvider.java | 7 +--
.../cxf/sts/token/provider/SCTProvider.java | 7 +--
.../token/validator/UsernameTokenValidator.java | 5 +-
.../cxf/sts/operation/DummyTokenProvider.java | 5 +-
.../cxf/sts/operation/DummyTokenValidator.java | 7 +--
.../sts/deployment/CustomBSTTokenProvider.java | 5 +-
.../sts/deployment/CustomBSTTokenValidator.java | 7 +--
.../CustomUsernameTokenProvider.java | 5 +-
.../sts/secure_conv/SCTSAMLTokenProvider.java | 7 +--
.../UsernameTokenValidator.java | 5 +-
.../internal/WSDiscoveryServiceImpl.java | 5 +-
.../security/AllowAllHostnameVerifier.java | 5 +-
.../type_test/AbstractTypeTestClient.java | 9 +--
.../type_test/AbstractTypeTestClient3.java | 12 +---
.../apache/cxf/systest/ws/util/MessageFlow.java | 5 +-
.../systest/ws/wssec11/WSSecurity11Common.java | 6 +-
.../apache/cxf/systest/ws/util/MessageFlow.java | 5 +-
.../common/AbstractBusClientServerTestBase.java | 5 +-
.../tools/common/AbstractCXFToolContainer.java | 5 +-
.../apache/cxf/tools/common/ToolContext.java | 10 +---
.../corba/processors/idl/AttributeVisitor.java | 7 +--
.../corba/processors/idl/ConstVisitor.java | 5 +-
.../tools/corba/processors/idl/EnumVisitor.java | 5 +-
.../corba/processors/idl/ExceptionVisitor.java | 5 +-
.../processors/idl/FixedPtConstVisitor.java | 5 +-
.../corba/processors/idl/FixedVisitor.java | 5 +-
.../corba/processors/idl/PortTypeVisitor.java | 5 +-
.../corba/processors/idl/ScopedNameVisitor.java | 7 +--
.../corba/processors/idl/SequenceVisitor.java | 5 +-
.../corba/processors/idl/StringVisitor.java | 12 +---
.../corba/processors/idl/StructVisitor.java | 5 +-
.../corba/processors/idl/TypedefVisitor.java | 5 +-
.../corba/processors/idl/UnionVisitor.java | 5 +-
.../corba/processors/wsdl/WSDLParameter.java | 5 +-
.../processors/wsdl/WSDLToCorbaHelper.java | 6 +-
.../java2wsdl/processor/FrontendFactory.java | 10 +---
.../processor/internal/AntGenerator.java | 5 +-
.../internal/jaxws/JAXWSFrontEndProcessor.java | 5 +-
.../jaxws/generator/JaxwsClientGenerator.java | 7 +--
.../jaxws/generator/JaxwsImplGenerator.java | 7 +--
.../jaxws/generator/JaxwsSEIGenerator.java | 7 +--
.../jaxws/generator/JaxwsServerGenerator.java | 5 +-
.../simple/generator/SimpleClientGenerator.java | 5 +-
.../simple/generator/SimpleImplGenerator.java | 9 +--
.../simple/generator/SimpleSEIGenerator.java | 8 +--
.../simple/generator/SimpleServerGenerator.java | 5 +-
.../tools/wadlto/jaxb/CustomizationParser.java | 6 +-
.../cxf/tools/wsdlto/WSDLToJavaContainer.java | 5 +-
.../customization/CustomizationParser.java | 20 +------
.../jaxws/customization/JAXWSBindingParser.java | 8 +--
.../frontend/jaxws/generators/AntGenerator.java | 7 +--
.../jaxws/generators/ClientGenerator.java | 15 +----
.../jaxws/generators/FaultGenerator.java | 8 +--
.../jaxws/generators/ImplGenerator.java | 19 +------
.../frontend/jaxws/generators/SEIGenerator.java | 9 +--
.../jaxws/generators/ServerGenerator.java | 13 +----
.../jaxws/generators/ServiceGenerator.java | 7 +--
.../processor/internal/ParameterProcessor.java | 5 +-
.../processor/internal/ServiceProcessor.java | 7 +--
.../internal/annotator/BindingAnnotator.java | 7 +--
.../WrapperStyleNameCollisionValidator.java | 5 +-
.../misc/processor/WSDLToSoapProcessor.java | 7 +--
141 files changed, 259 insertions(+), 908 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/core/src/main/java/org/apache/cxf/common/util/PropertyUtils.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/cxf/common/util/PropertyUtils.java b/core/src/main/java/org/apache/cxf/common/util/PropertyUtils.java
index ec09abb..ded47cd 100644
--- a/core/src/main/java/org/apache/cxf/common/util/PropertyUtils.java
+++ b/core/src/main/java/org/apache/cxf/common/util/PropertyUtils.java
@@ -65,11 +65,7 @@ public final class PropertyUtils {
return false;
}
- if (Boolean.TRUE.equals(property) || "true".equalsIgnoreCase(property.toString())) {
- return true;
- }
-
- return false;
+ return Boolean.TRUE.equals(property) || "true".equalsIgnoreCase(property.toString());
}
/**
@@ -85,11 +81,7 @@ public final class PropertyUtils {
return false;
}
- if (Boolean.FALSE.equals(property) || "false".equalsIgnoreCase(property.toString())) {
- return true;
- }
-
- return false;
+ return Boolean.FALSE.equals(property) || "false".equalsIgnoreCase(property.toString());
}
public static Long getLong(Message message, String key) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/core/src/main/java/org/apache/cxf/common/util/StringUtils.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/cxf/common/util/StringUtils.java b/core/src/main/java/org/apache/cxf/common/util/StringUtils.java
index aa34a6c..8eb14f5 100644
--- a/core/src/main/java/org/apache/cxf/common/util/StringUtils.java
+++ b/core/src/main/java/org/apache/cxf/common/util/StringUtils.java
@@ -78,10 +78,7 @@ public final class StringUtils {
if (list == null || list.size() == 0) {
return true;
}
- if (list.size() == 1 && isEmpty(list.get(0))) {
- return true;
- }
- return false;
+ return list.size() == 1 && isEmpty(list.get(0));
}
public static String diff(String str1, String str2) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/core/src/main/java/org/apache/cxf/interceptor/FIStaxInInterceptor.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/cxf/interceptor/FIStaxInInterceptor.java b/core/src/main/java/org/apache/cxf/interceptor/FIStaxInInterceptor.java
index 176792f..caa71d8 100644
--- a/core/src/main/java/org/apache/cxf/interceptor/FIStaxInInterceptor.java
+++ b/core/src/main/java/org/apache/cxf/interceptor/FIStaxInInterceptor.java
@@ -91,11 +91,8 @@ public class FIStaxInInterceptor extends AbstractPhaseInterceptor<Message> {
protected boolean isHttpVerbSupported(Message message) {
if (isGET(message)) {
- if (isRequestor(message)
- && MessageUtils.isTrue(message.getContextualProperty(FI_GET_SUPPORTED))) {
- return true;
- }
- return false;
+ return isRequestor(message)
+ && MessageUtils.isTrue(message.getContextualProperty(FI_GET_SUPPORTED));
} else {
return true;
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/core/src/main/java/org/apache/cxf/staxutils/StaxUtils.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/cxf/staxutils/StaxUtils.java b/core/src/main/java/org/apache/cxf/staxutils/StaxUtils.java
index 78b0e11..49a551f 100644
--- a/core/src/main/java/org/apache/cxf/staxutils/StaxUtils.java
+++ b/core/src/main/java/org/apache/cxf/staxutils/StaxUtils.java
@@ -345,16 +345,13 @@ public final class StaxUtils {
private static boolean setRestrictionProperties(XMLInputFactory factory) {
//For now, we can only support Woodstox 4.2.x and newer as none of the other
//stax parsers support these settings
- if (setProperty(factory, "com.ctc.wstx.maxAttributesPerElement", maxAttributeCount)
+ return setProperty(factory, "com.ctc.wstx.maxAttributesPerElement", maxAttributeCount)
&& setProperty(factory, "com.ctc.wstx.maxAttributeSize", maxAttributeSize)
&& setProperty(factory, "com.ctc.wstx.maxChildrenPerElement", innerElementCountThreshold)
&& setProperty(factory, "com.ctc.wstx.maxElementCount", maxElementCount)
&& setProperty(factory, "com.ctc.wstx.maxElementDepth", innerElementLevelThreshold)
&& setProperty(factory, "com.ctc.wstx.maxCharacters", maxXMLCharacters)
- && setProperty(factory, "com.ctc.wstx.maxTextLength", maxTextLength)) {
- return true;
- }
- return false;
+ && setProperty(factory, "com.ctc.wstx.maxTextLength", maxTextLength);
}
private static boolean setProperty(XMLInputFactory f, String p, Object o) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/core/src/main/java/org/apache/cxf/staxutils/transform/OutTransformWriter.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/apache/cxf/staxutils/transform/OutTransformWriter.java b/core/src/main/java/org/apache/cxf/staxutils/transform/OutTransformWriter.java
index 88bc7bb..c4541bf 100644
--- a/core/src/main/java/org/apache/cxf/staxutils/transform/OutTransformWriter.java
+++ b/core/src/main/java/org/apache/cxf/staxutils/transform/OutTransformWriter.java
@@ -398,11 +398,8 @@ public class OutTransformWriter extends DelegatingXMLStreamWriter {
}
private boolean matchesDropped(boolean shallow) {
- if ((dropDepth > 0 && dropDepth <= currentDepth)
- || (shallow && (elementsStack.size() > 0 && dropElements.contains(elementsStack.get(0))))) {
- return true;
- }
- return false;
+ return (dropDepth > 0 && dropDepth <= currentDepth)
+ || (shallow && (elementsStack.size() > 0 && dropElements.contains(elementsStack.get(0))));
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/integration/jca/src/main/java/org/apache/cxf/jca/cxf/ManagedConnectionImpl.java
----------------------------------------------------------------------
diff --git a/integration/jca/src/main/java/org/apache/cxf/jca/cxf/ManagedConnectionImpl.java b/integration/jca/src/main/java/org/apache/cxf/jca/cxf/ManagedConnectionImpl.java
index b504a72..58a4358 100644
--- a/integration/jca/src/main/java/org/apache/cxf/jca/cxf/ManagedConnectionImpl.java
+++ b/integration/jca/src/main/java/org/apache/cxf/jca/cxf/ManagedConnectionImpl.java
@@ -164,10 +164,7 @@ public class ManagedConnectionImpl
if (cls == null) {
return false;
}
- if (null != cls.getAnnotation(WebService.class)) {
- return true;
- }
- return false;
+ return null != cls.getAnnotation(WebService.class);
}
public boolean isBound() {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/integration/jca/src/main/java/org/apache/cxf/jca/outbound/CXFConnectionSpec.java
----------------------------------------------------------------------
diff --git a/integration/jca/src/main/java/org/apache/cxf/jca/outbound/CXFConnectionSpec.java b/integration/jca/src/main/java/org/apache/cxf/jca/outbound/CXFConnectionSpec.java
index 5c60479..1bc611a 100644
--- a/integration/jca/src/main/java/org/apache/cxf/jca/outbound/CXFConnectionSpec.java
+++ b/integration/jca/src/main/java/org/apache/cxf/jca/outbound/CXFConnectionSpec.java
@@ -144,11 +144,7 @@ public class CXFConnectionSpec implements ConnectionRequestInfo {
return false;
}
- if (!ObjectUtils.equals(that.getAddress(), address)) {
- return false;
- }
-
- return true;
+ return ObjectUtils.equals(that.getAddress(), address);
}
// Required by JCA Spec
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/integration/jca/src/main/java/org/apache/cxf/jca/outbound/ManagedConnectionFactoryImpl.java
----------------------------------------------------------------------
diff --git a/integration/jca/src/main/java/org/apache/cxf/jca/outbound/ManagedConnectionFactoryImpl.java b/integration/jca/src/main/java/org/apache/cxf/jca/outbound/ManagedConnectionFactoryImpl.java
index cbc6cf4..19f7e8b 100644
--- a/integration/jca/src/main/java/org/apache/cxf/jca/outbound/ManagedConnectionFactoryImpl.java
+++ b/integration/jca/src/main/java/org/apache/cxf/jca/outbound/ManagedConnectionFactoryImpl.java
@@ -121,11 +121,7 @@ public class ManagedConnectionFactoryImpl implements ManagedConnectionFactory,
ManagedConnectionFactoryImpl that = (ManagedConnectionFactoryImpl)o;
- if (!ObjectUtils.equals(that.getBusConfigURL(), busConfigURL)) {
- return false;
- }
-
- return true;
+ return ObjectUtils.equals(that.getBusConfigURL(), busConfigURL);
}
public PrintWriter getLogWriter() throws ResourceException {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/integration/jca/src/main/java/org/apache/cxf/jca/servant/EJBEndpoint.java
----------------------------------------------------------------------
diff --git a/integration/jca/src/main/java/org/apache/cxf/jca/servant/EJBEndpoint.java b/integration/jca/src/main/java/org/apache/cxf/jca/servant/EJBEndpoint.java
index 4f7d6cf..cd17900 100644
--- a/integration/jca/src/main/java/org/apache/cxf/jca/servant/EJBEndpoint.java
+++ b/integration/jca/src/main/java/org/apache/cxf/jca/servant/EJBEndpoint.java
@@ -150,10 +150,7 @@ public class EJBEndpoint {
if (cls == null) {
return false;
}
- if (null != cls.getAnnotation(WebService.class)) {
- return true;
- }
- return false;
+ return null != cls.getAnnotation(WebService.class);
}
public String getEjbServantBaseURL() {
@@ -165,10 +162,7 @@ public class EJBEndpoint {
}
private static boolean isNotNull(String value) {
- if (value != null && !"".equals(value.trim())) {
- return true;
- }
- return false;
+ return value != null && !"".equals(value.trim());
}
public WorkManager getWorkManager() {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/parent/pom.xml
----------------------------------------------------------------------
diff --git a/parent/pom.xml b/parent/pom.xml
index 4bc5718..71d4984 100644
--- a/parent/pom.xml
+++ b/parent/pom.xml
@@ -347,12 +347,12 @@
<dependency>
<groupId>net.sourceforge.pmd</groupId>
<artifactId>pmd-core</artifactId>
- <version>5.2.3</version>
+ <version>5.3.0</version>
</dependency>
<dependency>
<groupId>net.sourceforge.pmd</groupId>
<artifactId>pmd-java</artifactId>
- <version>5.2.3</version>
+ <version>5.3.0</version>
</dependency>
</dependencies>
<configuration>
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/utils/CorbaUtils.java
----------------------------------------------------------------------
diff --git a/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/utils/CorbaUtils.java b/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/utils/CorbaUtils.java
index a51354e..a672de3 100644
--- a/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/utils/CorbaUtils.java
+++ b/rt/bindings/corba/src/main/java/org/apache/cxf/binding/corba/utils/CorbaUtils.java
@@ -483,10 +483,7 @@ public final class CorbaUtils {
}
public static boolean isIOR(String location) {
- if ((location.startsWith("ior:")) || (location.startsWith("IOR:"))) {
- return true;
- }
- return false;
+ return location.startsWith("ior:") || location.startsWith("IOR:");
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/type/basic/BeanTypeInfo.java
----------------------------------------------------------------------
diff --git a/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/type/basic/BeanTypeInfo.java b/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/type/basic/BeanTypeInfo.java
index 96a4e4d..a5f2a3b 100644
--- a/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/type/basic/BeanTypeInfo.java
+++ b/rt/databinding/aegis/src/main/java/org/apache/cxf/aegis/type/basic/BeanTypeInfo.java
@@ -120,11 +120,7 @@ public class BeanTypeInfo {
}
public boolean isMapped(PropertyDescriptor pd) {
- if (pd.getReadMethod() == null) {
- return false;
- }
-
- return true;
+ return pd.getReadMethod() != null;
}
protected void mapProperty(PropertyDescriptor pd) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/BeanA.java
----------------------------------------------------------------------
diff --git a/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/BeanA.java b/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/BeanA.java
index 51a375c..83a4fc6 100644
--- a/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/BeanA.java
+++ b/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/BeanA.java
@@ -44,11 +44,7 @@ public class BeanA implements java.io.Serializable {
final BeanA beanA = (BeanA)o;
- if ((propA != null) ? (!propA.equals(beanA.propA)) : (beanA.propA != null)) {
- return false;
- }
-
- return true;
+ return !((propA != null) ? (!propA.equals(beanA.propA)) : (beanA.propA != null));
}
public int hashCode() {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/BeanB.java
----------------------------------------------------------------------
diff --git a/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/BeanB.java b/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/BeanB.java
index b7fd78d..df38063 100644
--- a/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/BeanB.java
+++ b/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/BeanB.java
@@ -47,11 +47,7 @@ public class BeanB extends BeanA {
final BeanB beanB = (BeanB)o;
- if ((propB != null) ? (!propB.equals(beanB.propB)) : (beanB.propB != null)) {
- return false;
- }
-
- return true;
+ return !((propB != null) ? (!propB.equals(beanB.propB)) : (beanB.propB != null));
}
public int hashCode() {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/BeanC.java
----------------------------------------------------------------------
diff --git a/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/BeanC.java b/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/BeanC.java
index 501ea5a..89eb648 100644
--- a/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/BeanC.java
+++ b/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/BeanC.java
@@ -56,11 +56,7 @@ public class BeanC extends BeanB {
final BeanC beanC = (BeanC)o;
- if ((propC != null) ? (!propC.equals(beanC.propC)) : (beanC.propC != null)) {
- return false;
- }
-
- return true;
+ return !((propC != null) ? (!propC.equals(beanC.propC)) : (beanC.propC != null));
}
public int hashCode() {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/ResultBean.java
----------------------------------------------------------------------
diff --git a/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/ResultBean.java b/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/ResultBean.java
index baa2d78..691704f 100644
--- a/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/ResultBean.java
+++ b/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/ResultBean.java
@@ -53,11 +53,7 @@ public class ResultBean {
if (!Arrays.equals(result1, that.result1)) {
return false;
}
- if (!Arrays.equals(result2, that.result2)) {
- return false;
- }
-
- return true;
+ return Arrays.equals(result2, that.result2);
}
public int hashCode() {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/RootBean.java
----------------------------------------------------------------------
diff --git a/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/RootBean.java b/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/RootBean.java
index b3e6fa6..ca53c6d 100644
--- a/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/RootBean.java
+++ b/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/RootBean.java
@@ -55,11 +55,7 @@ public class RootBean {
if (child != null ? !child.equals(rootBean.child) : rootBean.child != null) {
return false;
}
- if (id != null ? !id.equals(rootBean.id) : rootBean.id != null) {
- return false;
- }
-
- return true;
+ return !(id != null ? !id.equals(rootBean.id) : rootBean.id != null);
}
public int hashCode() {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/WS1Exception.java
----------------------------------------------------------------------
diff --git a/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/WS1Exception.java b/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/WS1Exception.java
index b015375..68dfe51 100644
--- a/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/WS1Exception.java
+++ b/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/WS1Exception.java
@@ -79,11 +79,7 @@ public class WS1Exception extends Exception {
return false;
}
- if (errorCode != that.errorCode) {
- return false;
- }
-
- return true;
+ return errorCode == that.errorCode;
}
public int hashCode() {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/WS1ExtendedException.java
----------------------------------------------------------------------
diff --git a/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/WS1ExtendedException.java b/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/WS1ExtendedException.java
index f80e018..522090a 100644
--- a/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/WS1ExtendedException.java
+++ b/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws1/WS1ExtendedException.java
@@ -65,11 +65,7 @@ public class WS1ExtendedException extends WS1Exception {
final WS1ExtendedException that = (WS1ExtendedException)o;
- if (extendedCode != that.extendedCode) {
- return false;
- }
-
- return true;
+ return extendedCode == that.extendedCode;
}
public int hashCode() {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws2/common/ParentBean.java
----------------------------------------------------------------------
diff --git a/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws2/common/ParentBean.java b/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws2/common/ParentBean.java
index 2bc5eed..d29a25e 100644
--- a/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws2/common/ParentBean.java
+++ b/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws2/common/ParentBean.java
@@ -66,11 +66,7 @@ public class ParentBean {
if (content != null ? !content.equals(that.content) : that.content != null) {
return false;
}
- if (id != null ? !id.equals(that.id) : that.id != null) {
- return false;
- }
-
- return true;
+ return !(id != null ? !id.equals(that.id) : that.id != null);
}
public int hashCode() {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws2/common/exception/AlreadyExistsException.java
----------------------------------------------------------------------
diff --git a/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws2/common/exception/AlreadyExistsException.java b/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws2/common/exception/AlreadyExistsException.java
index ea1e06a..0a58e2d 100644
--- a/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws2/common/exception/AlreadyExistsException.java
+++ b/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws2/common/exception/AlreadyExistsException.java
@@ -49,11 +49,7 @@ public class AlreadyExistsException extends Exception {
return false;
}
- if (id != null ? !id.equals(that.id) : that.id != null) {
- return false;
- }
-
- return true;
+ return !(id != null ? !id.equals(that.id) : that.id != null);
}
public int hashCode() {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws2/common/exception/NotFoundException.java
----------------------------------------------------------------------
diff --git a/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws2/common/exception/NotFoundException.java b/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws2/common/exception/NotFoundException.java
index c0d4b50..e0752bd 100644
--- a/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws2/common/exception/NotFoundException.java
+++ b/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws2/common/exception/NotFoundException.java
@@ -49,11 +49,7 @@ public class NotFoundException extends Exception {
return false;
}
- if (id != null ? !id.equals(that.id) : that.id != null) {
- return false;
- }
-
- return true;
+ return !(id != null ? !id.equals(that.id) : that.id != null);
}
public int hashCode() {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws2/common/pack1/ContentBean1.java
----------------------------------------------------------------------
diff --git a/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws2/common/pack1/ContentBean1.java b/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws2/common/pack1/ContentBean1.java
index 8a39d58..2d6135f 100644
--- a/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws2/common/pack1/ContentBean1.java
+++ b/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws2/common/pack1/ContentBean1.java
@@ -50,11 +50,7 @@ public class ContentBean1 {
final ContentBean1 that = (ContentBean1)o;
- if (data1 != null ? !data1.equals(that.data1) : that.data1 != null) {
- return false;
- }
-
- return true;
+ return !(data1 != null ? !data1.equals(that.data1) : that.data1 != null);
}
public int hashCode() {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws2/common/pack2/ContentBean2.java
----------------------------------------------------------------------
diff --git a/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws2/common/pack2/ContentBean2.java b/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws2/common/pack2/ContentBean2.java
index a29c7aa..0d1cde0 100644
--- a/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws2/common/pack2/ContentBean2.java
+++ b/rt/databinding/aegis/src/test/java/org/apache/cxf/aegis/inheritance/ws2/common/pack2/ContentBean2.java
@@ -56,11 +56,7 @@ public class ContentBean2 extends ContentBean1 {
final ContentBean2 that = (ContentBean2)o;
- if (content2 != null ? !content2.equals(that.content2) : that.content2 != null) {
- return false;
- }
-
- return true;
+ return !(content2 != null ? !content2.equals(that.content2) : that.content2 != null);
}
public int hashCode() {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/databinding/jaxb/src/main/java/org/apache/cxf/jaxb/JAXBEncoderDecoder.java
----------------------------------------------------------------------
diff --git a/rt/databinding/jaxb/src/main/java/org/apache/cxf/jaxb/JAXBEncoderDecoder.java b/rt/databinding/jaxb/src/main/java/org/apache/cxf/jaxb/JAXBEncoderDecoder.java
index b7b0478..f66baee 100644
--- a/rt/databinding/jaxb/src/main/java/org/apache/cxf/jaxb/JAXBEncoderDecoder.java
+++ b/rt/databinding/jaxb/src/main/java/org/apache/cxf/jaxb/JAXBEncoderDecoder.java
@@ -786,10 +786,7 @@ public final class JAXBEncoderDecoder {
}
private static boolean isList(Type cls) {
- if (cls instanceof ParameterizedType) {
- return true;
- }
- return false;
+ return cls instanceof ParameterizedType;
}
private static boolean isList(MessagePartInfo part) {
if (part.getTypeClass().isArray() && !part.getTypeClass().getComponentType().isPrimitive()) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/MultipartProvider.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/MultipartProvider.java b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/MultipartProvider.java
index 91e2c96..643979f 100644
--- a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/MultipartProvider.java
+++ b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/MultipartProvider.java
@@ -125,15 +125,12 @@ public class MultipartProvider extends AbstractConfigurableProvider
private boolean isSupported(Class<?> type, Type genericType, Annotation[] anns,
MediaType mt) {
- if (mediaTypeSupported(mt)
+ return mediaTypeSupported(mt)
&& (WELL_KNOWN_MULTIPART_CLASSES.contains(type)
|| Collection.class.isAssignableFrom(type)
|| Map.class.isAssignableFrom(type) && type != MultivaluedMap.class
|| AnnotationUtils.getAnnotation(anns, Multipart.class) != null
- || MessageUtils.isTrue(mc.getContextualProperty(SUPPORT_TYPE_AS_MULTIPART)))) {
- return true;
- }
- return false;
+ || MessageUtils.isTrue(mc.getContextualProperty(SUPPORT_TYPE_AS_MULTIPART)));
}
protected void checkContentLength() {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/RequestDispatcherProvider.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/RequestDispatcherProvider.java b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/RequestDispatcherProvider.java
index c7b33e6..d13486e 100644
--- a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/RequestDispatcherProvider.java
+++ b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/provider/RequestDispatcherProvider.java
@@ -144,10 +144,7 @@ public class RequestDispatcherProvider extends AbstractConfigurableProvider
}
}
}
- if (mc != null && mc.get(MESSAGE_RESOURCE_PATH_PROPERTY) != null) {
- return true;
- }
- return false;
+ return mc != null && mc.get(MESSAGE_RESOURCE_PATH_PROPERTY) != null;
}
private boolean classResourceSupported(Class<?> type) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/ExceptionUtils.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/ExceptionUtils.java b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/ExceptionUtils.java
index 20977ee..5e9c257 100644
--- a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/ExceptionUtils.java
+++ b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/ExceptionUtils.java
@@ -57,11 +57,7 @@ public final class ExceptionUtils {
return true;
}
- if (Boolean.TRUE.equals(value) || "true".equalsIgnoreCase(value.toString())) {
- return true;
- }
-
- return false;
+ return Boolean.TRUE.equals(value) || "true".equalsIgnoreCase(value.toString());
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/JAXRSUtils.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/JAXRSUtils.java b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/JAXRSUtils.java
index 3aac1a0..08e63fb 100644
--- a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/JAXRSUtils.java
+++ b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/JAXRSUtils.java
@@ -622,12 +622,9 @@ public final class JAXRSUtils {
}
private static boolean matchHttpMethod(String expectedMethod, String httpMethod) {
- if (expectedMethod.equalsIgnoreCase(httpMethod)
+ return expectedMethod.equalsIgnoreCase(httpMethod)
|| headMethodPossible(expectedMethod, httpMethod)
- || expectedMethod.equals(DefaultMethod.class.getSimpleName())) {
- return true;
- }
- return false;
+ || expectedMethod.equals(DefaultMethod.class.getSimpleName());
}
public static boolean headMethodPossible(String expectedMethod, String httpMethod) {
@@ -1400,11 +1397,8 @@ public final class JAXRSUtils {
MediaType acceptContentType,
OperationResourceInfo ori) {
- if (intersectMimeTypes(ori.getConsumeTypes(), requestContentType).size() != 0
- && intersectMimeTypes(ori.getProduceTypes(), acceptContentType).size() != 0) {
- return true;
- }
- return false;
+ return intersectMimeTypes(ori.getConsumeTypes(), requestContentType).size() != 0
+ && intersectMimeTypes(ori.getProduceTypes(), acceptContentType).size() != 0;
}
public static List<MediaType> parseMediaTypes(String types) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/multipart/AttachmentUtils.java
----------------------------------------------------------------------
diff --git a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/multipart/AttachmentUtils.java b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/multipart/AttachmentUtils.java
index 765102a..6b89115 100644
--- a/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/multipart/AttachmentUtils.java
+++ b/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/utils/multipart/AttachmentUtils.java
@@ -188,10 +188,7 @@ public final class AttachmentUtils {
return true;
}
ContentDisposition cd = at.getContentDisposition();
- if (cd != null && value.equals(cd.getParameter("name"))) {
- return true;
- }
- return false;
+ return cd != null && value.equals(cd.getParameter("name"));
}
public static MultivaluedMap<String, String> populateFormMap(MessageContext mc,
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/frontend/simple/src/main/java/org/apache/cxf/frontend/WSDLGetInterceptor.java
----------------------------------------------------------------------
diff --git a/rt/frontend/simple/src/main/java/org/apache/cxf/frontend/WSDLGetInterceptor.java b/rt/frontend/simple/src/main/java/org/apache/cxf/frontend/WSDLGetInterceptor.java
index 37e2af8..c50608f 100644
--- a/rt/frontend/simple/src/main/java/org/apache/cxf/frontend/WSDLGetInterceptor.java
+++ b/rt/frontend/simple/src/main/java/org/apache/cxf/frontend/WSDLGetInterceptor.java
@@ -132,10 +132,6 @@ public class WSDLGetInterceptor extends AbstractPhaseInterceptor<Message> {
}
private boolean isRecognizedQuery(Map<String, String> map) {
-
- if (map.containsKey("wsdl") || map.containsKey("xsd")) {
- return true;
- }
- return false;
+ return map.containsKey("wsdl") || map.containsKey("xsd");
}
}
\ No newline at end of file
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/javascript/javascript-rt/src/main/java/org/apache/cxf/javascript/JavascriptGetInterceptor.java
----------------------------------------------------------------------
diff --git a/rt/javascript/javascript-rt/src/main/java/org/apache/cxf/javascript/JavascriptGetInterceptor.java b/rt/javascript/javascript-rt/src/main/java/org/apache/cxf/javascript/JavascriptGetInterceptor.java
index a4aa43b..5753a25 100644
--- a/rt/javascript/javascript-rt/src/main/java/org/apache/cxf/javascript/JavascriptGetInterceptor.java
+++ b/rt/javascript/javascript-rt/src/main/java/org/apache/cxf/javascript/JavascriptGetInterceptor.java
@@ -97,10 +97,7 @@ public class JavascriptGetInterceptor extends AbstractPhaseInterceptor<Message>
if (uri == null) {
return false;
}
- if (map.containsKey(CODE_QUERY_KEY)) {
- return true;
- }
- return false;
+ return map.containsKey(CODE_QUERY_KEY);
}
public static void writeUtilsToResponseStream(Class<?> referenceClass, OutputStream outputStream) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/management-web/src/main/java/org/apache/cxf/management/web/browser/client/service/settings/Subscription.java
----------------------------------------------------------------------
diff --git a/rt/management-web/src/main/java/org/apache/cxf/management/web/browser/client/service/settings/Subscription.java b/rt/management-web/src/main/java/org/apache/cxf/management/web/browser/client/service/settings/Subscription.java
index 4177237..c8009e4 100644
--- a/rt/management-web/src/main/java/org/apache/cxf/management/web/browser/client/service/settings/Subscription.java
+++ b/rt/management-web/src/main/java/org/apache/cxf/management/web/browser/client/service/settings/Subscription.java
@@ -62,11 +62,7 @@ public class Subscription {
Subscription that = (Subscription)o;
- if (id != null ? !id.equals(that.id) : that.id != null) {
- return false;
- }
-
- return true;
+ return !(id != null ? !id.equals(that.id) : that.id != null);
}
@Override
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/management-web/src/main/java/org/apache/cxf/management/web/logging/atom/AtomPushEngine.java
----------------------------------------------------------------------
diff --git a/rt/management-web/src/main/java/org/apache/cxf/management/web/logging/atom/AtomPushEngine.java b/rt/management-web/src/main/java/org/apache/cxf/management/web/logging/atom/AtomPushEngine.java
index 7355c47..5a6a788 100644
--- a/rt/management-web/src/main/java/org/apache/cxf/management/web/logging/atom/AtomPushEngine.java
+++ b/rt/management-web/src/main/java/org/apache/cxf/management/web/logging/atom/AtomPushEngine.java
@@ -118,11 +118,7 @@ final class AtomPushEngine {
//System.err.println("converter is not set");
return false;
}
- if (executor.isShutdown()) {
- //System.err.println("engine shutdown");
- return false;
- }
- return true;
+ return !executor.isShutdown();
}
private void publishBatch(final List<LogRecord> batch,
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/management/src/main/java/org/apache/cxf/management/interceptor/AbstractMessageResponseTimeInterceptor.java
----------------------------------------------------------------------
diff --git a/rt/management/src/main/java/org/apache/cxf/management/interceptor/AbstractMessageResponseTimeInterceptor.java b/rt/management/src/main/java/org/apache/cxf/management/interceptor/AbstractMessageResponseTimeInterceptor.java
index c6e7c13..bbfd589 100644
--- a/rt/management/src/main/java/org/apache/cxf/management/interceptor/AbstractMessageResponseTimeInterceptor.java
+++ b/rt/management/src/main/java/org/apache/cxf/management/interceptor/AbstractMessageResponseTimeInterceptor.java
@@ -154,10 +154,7 @@ public abstract class AbstractMessageResponseTimeInterceptor extends AbstractPha
ObjectName serviceCounterName = getServiceCounterName(ex);
Counter serviceCounter = counterRepo.getCounter(serviceCounterName);
//If serviceCounter is null, we need to wait ResponseTimeOutInterceptor to create it , hence set to true
- if (serviceCounter == null || serviceCounter.isEnabled()) {
- return true;
- }
- return false;
+ return serviceCounter == null || serviceCounter.isEnabled();
}
protected ObjectName getOperationCounterName(Exchange ex, ObjectName sericeCounterName) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/ClientProxyImpl.java
----------------------------------------------------------------------
diff --git a/rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/ClientProxyImpl.java b/rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/ClientProxyImpl.java
index f18530f..bb70bad 100644
--- a/rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/ClientProxyImpl.java
+++ b/rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/ClientProxyImpl.java
@@ -270,11 +270,8 @@ public class ClientProxyImpl extends AbstractClient implements
if (p.getType() == ParameterType.CONTEXT) {
return true;
}
- if (p.getType() == ParameterType.REQUEST_BODY
- && m.getParameterTypes()[p.getIndex()] == AsyncResponse.class) {
- return true;
- }
- return false;
+ return p.getType() == ParameterType.REQUEST_BODY
+ && m.getParameterTypes()[p.getIndex()] == AsyncResponse.class;
}
private static int getBodyIndex(MultivaluedMap<ParameterType, Parameter> map,
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/rs/extensions/providers/src/main/java/org/apache/cxf/jaxrs/provider/atom/AtomPojoProvider.java
----------------------------------------------------------------------
diff --git a/rt/rs/extensions/providers/src/main/java/org/apache/cxf/jaxrs/provider/atom/AtomPojoProvider.java b/rt/rs/extensions/providers/src/main/java/org/apache/cxf/jaxrs/provider/atom/AtomPojoProvider.java
index c1c7237..46823b8 100644
--- a/rt/rs/extensions/providers/src/main/java/org/apache/cxf/jaxrs/provider/atom/AtomPojoProvider.java
+++ b/rt/rs/extensions/providers/src/main/java/org/apache/cxf/jaxrs/provider/atom/AtomPojoProvider.java
@@ -564,10 +564,7 @@ public class AtomPojoProvider extends AbstractConfigurableProvider
}
protected boolean isFeedRequested(MediaType mt) {
- if ("entry".equalsIgnoreCase(mt.getParameters().get("type"))) {
- return false;
- }
- return true;
+ return !"entry".equalsIgnoreCase(mt.getParameters().get("type"));
}
public void setAtomWriters(Map<String, AtomElementWriter<?, ?>> writers) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/rs/extensions/search/src/main/java/org/apache/cxf/jaxrs/ext/search/AbstractSearchConditionParser.java
----------------------------------------------------------------------
diff --git a/rt/rs/extensions/search/src/main/java/org/apache/cxf/jaxrs/ext/search/AbstractSearchConditionParser.java b/rt/rs/extensions/search/src/main/java/org/apache/cxf/jaxrs/ext/search/AbstractSearchConditionParser.java
index 5721c8b..74c7a68 100644
--- a/rt/rs/extensions/search/src/main/java/org/apache/cxf/jaxrs/ext/search/AbstractSearchConditionParser.java
+++ b/rt/rs/extensions/search/src/main/java/org/apache/cxf/jaxrs/ext/search/AbstractSearchConditionParser.java
@@ -233,10 +233,7 @@ public abstract class AbstractSearchConditionParser<T> implements SearchConditio
private boolean paramConverterAvailable(Class<?> pClass) {
Message m = JAXRSUtils.getCurrentMessage();
ServerProviderFactory pf = m == null ? null : ServerProviderFactory.getInstance(m);
- if (pf != null && pf.createParameterHandler(pClass, pClass, EMPTY_ANNOTTAIONS) != null) {
- return true;
- }
- return false;
+ return pf != null && pf.createParameterHandler(pClass, pClass, EMPTY_ANNOTTAIONS) != null;
}
private CollectionCheck getCollectionCheck(String propName, boolean isCollection, Class<?> actualCls) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/data/Client.java
----------------------------------------------------------------------
diff --git a/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/data/Client.java b/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/data/Client.java
index 6fd429c..b1c4676 100644
--- a/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/data/Client.java
+++ b/rt/rs/security/oauth-parent/oauth/src/main/java/org/apache/cxf/rs/security/oauth/data/Client.java
@@ -157,11 +157,7 @@ public class Client {
if (!consumerKey.equals(that.consumerKey)) {
return false;
}
- if (!secretKey.equals(that.secretKey)) {
- return false;
- }
-
- return true;
+ return secretKey.equals(that.secretKey);
}
@Override
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/rs/security/sso/saml/src/main/java/org/apache/cxf/rs/security/saml/sso/AbstractSSOSpHandler.java
----------------------------------------------------------------------
diff --git a/rt/rs/security/sso/saml/src/main/java/org/apache/cxf/rs/security/saml/sso/AbstractSSOSpHandler.java b/rt/rs/security/sso/saml/src/main/java/org/apache/cxf/rs/security/saml/sso/AbstractSSOSpHandler.java
index 548ea6e..d9d4572 100644
--- a/rt/rs/security/sso/saml/src/main/java/org/apache/cxf/rs/security/saml/sso/AbstractSSOSpHandler.java
+++ b/rt/rs/security/sso/saml/src/main/java/org/apache/cxf/rs/security/saml/sso/AbstractSSOSpHandler.java
@@ -133,11 +133,7 @@ public class AbstractSSOSpHandler {
return true;
}
- if (expiresAt > 0 && currentTime.after(new Date(expiresAt))) {
- return true;
- }
-
- return false;
+ return expiresAt > 0 && currentTime.after(new Date(expiresAt));
}
public void setStateProvider(SPStateManager stateProvider) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/transports/http/src/main/java/org/apache/cxf/transport/http/HTTPConduit.java
----------------------------------------------------------------------
diff --git a/rt/transports/http/src/main/java/org/apache/cxf/transport/http/HTTPConduit.java b/rt/transports/http/src/main/java/org/apache/cxf/transport/http/HTTPConduit.java
index 4b9adc1..1d9ae9c 100644
--- a/rt/transports/http/src/main/java/org/apache/cxf/transport/http/HTTPConduit.java
+++ b/rt/transports/http/src/main/java/org/apache/cxf/transport/http/HTTPConduit.java
@@ -1552,10 +1552,7 @@ public abstract class HTTPConduit
return true;
}
// 2. Robust OneWays could have a fault
- if (responseCode == 500 && MessageUtils.getContextualBoolean(message, Message.ROBUST_ONEWAY, false)) {
- return true;
- }
- return false;
+ return responseCode == 500 && MessageUtils.getContextualBoolean(message, Message.ROBUST_ONEWAY, false);
}
protected void handleResponseInternal() throws IOException {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/transports/http/src/main/java/org/apache/cxf/transport/https/AllowAllHostnameVerifier.java
----------------------------------------------------------------------
diff --git a/rt/transports/http/src/main/java/org/apache/cxf/transport/https/AllowAllHostnameVerifier.java b/rt/transports/http/src/main/java/org/apache/cxf/transport/https/AllowAllHostnameVerifier.java
index cfabf5f..5b660bf 100644
--- a/rt/transports/http/src/main/java/org/apache/cxf/transport/https/AllowAllHostnameVerifier.java
+++ b/rt/transports/http/src/main/java/org/apache/cxf/transport/https/AllowAllHostnameVerifier.java
@@ -34,14 +34,9 @@ class AllowAllHostnameVerifier implements javax.net.ssl.HostnameVerifier {
public boolean verify(String host, SSLSession session) {
try {
Certificate[] certs = session.getPeerCertificates();
- if (certs != null && certs[0] instanceof X509Certificate) {
- return true;
- }
- return false;
+ return certs != null && certs[0] instanceof X509Certificate;
} catch (SSLException e) {
return false;
}
}
-
-
}
\ No newline at end of file
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/addr/src/main/java/org/apache/cxf/ws/addressing/impl/MAPAggregatorImpl.java
----------------------------------------------------------------------
diff --git a/rt/ws/addr/src/main/java/org/apache/cxf/ws/addressing/impl/MAPAggregatorImpl.java b/rt/ws/addr/src/main/java/org/apache/cxf/ws/addressing/impl/MAPAggregatorImpl.java
index 15ca563..e013431 100644
--- a/rt/ws/addr/src/main/java/org/apache/cxf/ws/addressing/impl/MAPAggregatorImpl.java
+++ b/rt/ws/addr/src/main/java/org/apache/cxf/ws/addressing/impl/MAPAggregatorImpl.java
@@ -243,12 +243,7 @@ public class MAPAggregatorImpl extends MAPAggregator {
if (null == aim) {
return false;
}
- if (null == aim.get(MetadataConstants.ADDRESSING_ASSERTION_QNAME)) {
- return false;
- }
- // no need to analyse the content of the Addressing assertion here
-
- return true;
+ return null != aim.get(MetadataConstants.ADDRESSING_ASSERTION_QNAME);
}
/**
@@ -271,10 +266,7 @@ public class MAPAggregatorImpl extends MAPAggregator {
if (null != aim.get(MetadataConstants.USING_ADDRESSING_2005_QNAME)) {
return true;
}
- if (null != aim.get(MetadataConstants.USING_ADDRESSING_2006_QNAME)) {
- return true;
- }
- return false;
+ return null != aim.get(MetadataConstants.USING_ADDRESSING_2006_QNAME);
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/addr/src/main/java/org/apache/cxf/ws/addressing/soap/DecoupledFaultHandler.java
----------------------------------------------------------------------
diff --git a/rt/ws/addr/src/main/java/org/apache/cxf/ws/addressing/soap/DecoupledFaultHandler.java b/rt/ws/addr/src/main/java/org/apache/cxf/ws/addressing/soap/DecoupledFaultHandler.java
index 92043ab..483a46b 100644
--- a/rt/ws/addr/src/main/java/org/apache/cxf/ws/addressing/soap/DecoupledFaultHandler.java
+++ b/rt/ws/addr/src/main/java/org/apache/cxf/ws/addressing/soap/DecoupledFaultHandler.java
@@ -94,9 +94,6 @@ public class DecoupledFaultHandler extends AbstractSoapInterceptor {
}
private boolean isWSAHeader(Header header) {
- if (header.getName().getNamespaceURI().startsWith(Names.WSA_NAMESPACE_NAME)) {
- return true;
- }
- return false;
+ return header.getName().getNamespaceURI().startsWith(Names.WSA_NAMESPACE_NAME);
}
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/policy/src/main/java/org/apache/cxf/ws/policy/attachment/external/URIDomainExpression.java
----------------------------------------------------------------------
diff --git a/rt/ws/policy/src/main/java/org/apache/cxf/ws/policy/attachment/external/URIDomainExpression.java b/rt/ws/policy/src/main/java/org/apache/cxf/ws/policy/attachment/external/URIDomainExpression.java
index 1c2a412..a2d8fe1 100644
--- a/rt/ws/policy/src/main/java/org/apache/cxf/ws/policy/attachment/external/URIDomainExpression.java
+++ b/rt/ws/policy/src/main/java/org/apache/cxf/ws/policy/attachment/external/URIDomainExpression.java
@@ -50,12 +50,9 @@ public class URIDomainExpression implements DomainExpression {
&& wsdl11XPointer.matchesService(si.getTargetNamespace(), si.getName().getLocalPart())) {
return true;
}
- if ((si.getInterface() != null) && (si.getInterface().getName() != null)
+ return (si.getInterface() != null) && (si.getInterface().getName() != null)
&& wsdl11XPointer.matchesPortType(si.getTargetNamespace(),
- si.getInterface().getName().getLocalPart())) {
- return true;
- }
- return false;
+ si.getInterface().getName().getLocalPart());
}
@Override
@@ -63,13 +60,10 @@ public class URIDomainExpression implements DomainExpression {
if (ei == null) {
return false;
}
- if ((ei.getService() != null) && (ei.getService().getName() != null) && (ei.getName() != null)
+ return (ei.getService() != null) && (ei.getService().getName() != null) && (ei.getName() != null)
&& wsdl11XPointer.matchesPort(ei.getService().getTargetNamespace(),
ei.getService().getName().getLocalPart(),
- ei.getName().getLocalPart())) {
- return true;
- }
- return false;
+ ei.getName().getLocalPart());
}
@Override
@@ -90,15 +84,12 @@ public class URIDomainExpression implements DomainExpression {
boi.getName().getLocalPart())) {
return true;
}
- if ((boi.getOperationInfo() != null) && (boi.getOperationInfo().getInterface() != null)
+ return (boi.getOperationInfo() != null) && (boi.getOperationInfo().getInterface() != null)
&& (boi.getOperationInfo().getInterface().getName() != null) && (boi.getOperationInfo().getName() != null)
&& wsdl11XPointer.matchesPortTypeOperation(
boi.getOperationInfo().getInterface().getName().getNamespaceURI(),
boi.getOperationInfo().getInterface().getName().getLocalPart(),
- boi.getOperationInfo().getName().getLocalPart())) {
- return true;
- }
- return false;
+ boi.getOperationInfo().getName().getLocalPart());
}
@Override
@@ -117,10 +108,7 @@ public class URIDomainExpression implements DomainExpression {
return true;
}
- if (checkPortTypeOperationInOut(bmi)) {
- return true;
- }
- return false;
+ return checkPortTypeOperationInOut(bmi);
}
@@ -134,10 +122,7 @@ public class URIDomainExpression implements DomainExpression {
return true;
}
- if (checkPortTypeOperationFault(bfi)) {
- return true;
- }
- return false;
+ return checkPortTypeOperationFault(bfi);
}
private boolean checkBindingOperationInOut(BindingMessageInfo bmi) {
@@ -196,21 +181,18 @@ public class URIDomainExpression implements DomainExpression {
ini = bfi.getBindingOperation().getOperationInfo().getInterface();
}
- if ((ini != null) && (ini.getName() != null)
+ return (ini != null) && (ini.getName() != null)
&& (bfi.getBindingOperation().getName() != null)
&& (bfi.getFaultInfo().getFaultName() != null)
&& wsdl11XPointer.matchesPortTypeOperationFault(
ini.getName().getNamespaceURI(),
ini.getName().getLocalPart(),
bfi.getBindingOperation().getName().getLocalPart(),
- bfi.getFaultInfo().getFaultName().getLocalPart())) {
- return true;
- }
- return false;
+ bfi.getFaultInfo().getFaultName().getLocalPart());
}
private boolean checkBindingOperationFault(BindingFaultInfo bfi) {
- if ((bfi.getFaultInfo() != null) && (bfi.getFaultInfo().getFaultName() != null)
+ return (bfi.getFaultInfo() != null) && (bfi.getFaultInfo().getFaultName() != null)
&& (bfi.getBindingOperation().getName() != null)
&& (bfi.getBindingOperation().getBinding() != null)
&& (bfi.getBindingOperation().getBinding().getName() != null)
@@ -218,10 +200,7 @@ public class URIDomainExpression implements DomainExpression {
bfi.getFaultInfo().getFaultName().getNamespaceURI(),
bfi.getBindingOperation().getBinding().getName().getLocalPart(),
bfi.getBindingOperation().getName().getLocalPart(),
- bfi.getFaultInfo().getFaultName().getLocalPart())) {
- return true;
- }
- return false;
+ bfi.getFaultInfo().getFaultName().getLocalPart());
}
public static class Wsdl11XPointer {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/AbstractRMInterceptor.java
----------------------------------------------------------------------
diff --git a/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/AbstractRMInterceptor.java b/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/AbstractRMInterceptor.java
index 894fb95..b14ced7 100644
--- a/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/AbstractRMInterceptor.java
+++ b/rt/ws/rm/src/main/java/org/apache/cxf/ws/rm/AbstractRMInterceptor.java
@@ -124,10 +124,7 @@ public abstract class AbstractRMInterceptor<T extends Message> extends AbstractP
if (assertionMap == null) {
return false;
}
- if (!RMPolicyUtilities.collectRMAssertions(assertionMap).isEmpty()) {
- return true;
- }
- return false;
+ return !RMPolicyUtilities.collectRMAssertions(assertionMap).isEmpty();
}
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/policy/PolicyUtils.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/policy/PolicyUtils.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/policy/PolicyUtils.java
index 6bed02e..48a1e61 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/policy/PolicyUtils.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/policy/PolicyUtils.java
@@ -104,11 +104,7 @@ public final class PolicyUtils {
Collection<AssertionInfo> sp11Ais = aim.get(new QName(SP11Constants.SP_NS, localname));
Collection<AssertionInfo> sp12Ais = aim.get(new QName(SP12Constants.SP_NS, localname));
- if ((sp11Ais != null && !sp11Ais.isEmpty()) || (sp12Ais != null && !sp12Ais.isEmpty())) {
- return true;
- }
-
- return false;
+ return (sp11Ais != null && !sp11Ais.isEmpty()) || (sp12Ais != null && !sp12Ais.isEmpty());
}
public static AbstractBinding getSecurityBinding(AssertionInfoMap aim) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/trust/STSUtils.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/trust/STSUtils.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/trust/STSUtils.java
index f9fda5f..670d95d 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/trust/STSUtils.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/trust/STSUtils.java
@@ -157,11 +157,7 @@ public final class STSUtils {
}
public static boolean configureViaEPR(STSClient client, EndpointReferenceType epr) {
- if (epr != null && client.getLocation() == null && client.getWsdlLocation() == null) {
- return true;
- }
-
- return false;
+ return epr != null && client.getLocation() == null && client.getWsdlLocation() == null;
}
private static STSClient createSTSClient(Message message, String type) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/AbstractTokenInterceptor.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/AbstractTokenInterceptor.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/AbstractTokenInterceptor.java
index 7501ce7..258d043 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/AbstractTokenInterceptor.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/AbstractTokenInterceptor.java
@@ -137,10 +137,7 @@ public abstract class AbstractTokenInterceptor extends AbstractSoapInterceptor {
protected boolean isTLSInUse(SoapMessage message) {
// See whether TLS is in use or not
TLSSessionInfo tlsInfo = message.get(TLSSessionInfo.class);
- if (tlsInfo != null) {
- return true;
- }
- return false;
+ return tlsInfo != null;
}
protected TokenStore getTokenStore(SoapMessage message) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/PolicyBasedWSS4JInInterceptor.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/PolicyBasedWSS4JInInterceptor.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/PolicyBasedWSS4JInInterceptor.java
index c4c8b37..ad65a3c 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/PolicyBasedWSS4JInInterceptor.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/PolicyBasedWSS4JInInterceptor.java
@@ -927,11 +927,7 @@ public class PolicyBasedWSS4JInInterceptor extends WSS4JInInterceptor {
}
ai = PolicyUtils.getFirstAssertionByLocalname(aim, SPConstants.CONTENT_ENCRYPTED_ELEMENTS);
- if (ai != null) {
- return true;
- }
-
- return false;
+ return ai != null;
}
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/StaxCryptoCoverageChecker.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/StaxCryptoCoverageChecker.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/StaxCryptoCoverageChecker.java
index d4c901c..14c60c5 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/StaxCryptoCoverageChecker.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/StaxCryptoCoverageChecker.java
@@ -321,90 +321,62 @@ public class StaxCryptoCoverageChecker extends AbstractPhaseInterceptor<SoapMess
}
private boolean isEnvelope(QName qname) {
- if ("Envelope".equals(qname.getLocalPart())
+ return "Envelope".equals(qname.getLocalPart())
&& (SOAP_NS.equals(qname.getNamespaceURI())
- || SOAP12_NS.equals(qname.getNamespaceURI()))) {
- return true;
- }
- return false;
+ || SOAP12_NS.equals(qname.getNamespaceURI()));
}
private boolean isSoapHeader(QName qname) {
- if ("Header".equals(qname.getLocalPart())
+ return "Header".equals(qname.getLocalPart())
&& (SOAP_NS.equals(qname.getNamespaceURI())
- || SOAP12_NS.equals(qname.getNamespaceURI()))) {
- return true;
- }
- return false;
+ || SOAP12_NS.equals(qname.getNamespaceURI()));
}
private boolean isSecurityHeader(QName qname) {
- if ("Security".equals(qname.getLocalPart()) && WSSE_NS.equals(qname.getNamespaceURI())) {
- return true;
- }
- return false;
+ return "Security".equals(qname.getLocalPart()) && WSSE_NS.equals(qname.getNamespaceURI());
}
private boolean isTimestamp(List<QName> qnames) {
- if (qnames != null && qnames.size() == 4
+ return qnames != null
+ && qnames.size() == 4
&& isEnvelope(qnames.get(0))
&& isSoapHeader(qnames.get(1))
&& isSecurityHeader(qnames.get(2))
&& "Timestamp".equals(qnames.get(3).getLocalPart())
- && WSU_NS.equals(qnames.get(3).getNamespaceURI())) {
- return true;
- }
-
- return false;
+ && WSU_NS.equals(qnames.get(3).getNamespaceURI());
}
private boolean isReplyTo(List<QName> qnames) {
- if (qnames != null && qnames.size() == 3
+ return qnames != null && qnames.size() == 3
&& isEnvelope(qnames.get(0))
&& isSoapHeader(qnames.get(1))
&& "ReplyTo".equals(qnames.get(2).getLocalPart())
- && WSA_NS.equals(qnames.get(2).getNamespaceURI())) {
- return true;
- }
-
- return false;
+ && WSA_NS.equals(qnames.get(2).getNamespaceURI());
}
private boolean isFaultTo(List<QName> qnames) {
- if (qnames != null && qnames.size() == 3
+ return qnames != null && qnames.size() == 3
&& isEnvelope(qnames.get(0))
&& isSoapHeader(qnames.get(1))
&& "FaultTo".equals(qnames.get(2).getLocalPart())
- && WSA_NS.equals(qnames.get(2).getNamespaceURI())) {
- return true;
- }
-
- return false;
+ && WSA_NS.equals(qnames.get(2).getNamespaceURI());
}
private boolean isBody(List<QName> qnames) {
- if (qnames != null && qnames.size() == 2
+ return qnames != null && qnames.size() == 2
&& isEnvelope(qnames.get(0))
&& "Body".equals(qnames.get(1).getLocalPart())
&& (SOAP_NS.equals(qnames.get(1).getNamespaceURI())
- || SOAP12_NS.equals(qnames.get(1).getNamespaceURI()))) {
- return true;
- }
-
- return false;
+ || SOAP12_NS.equals(qnames.get(1).getNamespaceURI()));
}
private boolean isUsernameToken(List<QName> qnames) {
- if (qnames != null && qnames.size() == 4
+ return qnames != null && qnames.size() == 4
&& isEnvelope(qnames.get(0))
&& isSoapHeader(qnames.get(1))
&& isSecurityHeader(qnames.get(2))
&& "UsernameToken".equals(qnames.get(3).getLocalPart())
- && WSSE_NS.equals(qnames.get(3).getNamespaceURI())) {
- return true;
- }
-
- return false;
+ && WSSE_NS.equals(qnames.get(3).getNamespaceURI());
}
public boolean isSignBody() {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/StaxSecurityContextInInterceptor.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/StaxSecurityContextInInterceptor.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/StaxSecurityContextInInterceptor.java
index 5ba0457..12d8299 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/StaxSecurityContextInInterceptor.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/StaxSecurityContextInInterceptor.java
@@ -191,12 +191,8 @@ public class StaxSecurityContextInInterceptor extends AbstractPhaseInterceptor<S
}
// Now check that a PublicKey/X509Certificate was used
- if (token.getPublicKey() != null
- || (token.getX509Certificates() != null && token.getX509Certificates().length > 0)) {
- return true;
- }
-
- return false;
+ return token.getPublicKey() != null
+ || (token.getX509Certificates() != null && token.getX509Certificates().length > 0);
}
private boolean isSamlEventSigned(SamlTokenSecurityEvent event) {
@@ -204,13 +200,9 @@ public class StaxSecurityContextInInterceptor extends AbstractPhaseInterceptor<S
return false;
}
- if (event.getSecurityToken() != null
+ return event.getSecurityToken() != null
&& event.getSecurityToken().getSamlAssertionWrapper() != null
- && event.getSecurityToken().getSamlAssertionWrapper().isSigned()) {
- return true;
- }
-
- return false;
+ && event.getSecurityToken().getSamlAssertionWrapper().isSigned();
}
private SecurityContext createSecurityContext(final Principal p) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/UsernameTokenInterceptor.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/UsernameTokenInterceptor.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/UsernameTokenInterceptor.java
index 59f7005..a083eeb 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/UsernameTokenInterceptor.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/UsernameTokenInterceptor.java
@@ -358,11 +358,8 @@ public class UsernameTokenInterceptor extends AbstractTokenInterceptor {
org.apache.wss4j.policy.model.UsernameToken usernameTokenPolicy
) {
AbstractSecurityAssertion supportingToken = usernameTokenPolicy.getParentAssertion();
- if (supportingToken instanceof SupportingTokens
- && ((SupportingTokens)supportingToken).isEndorsing()) {
- return false;
- }
- return true;
+ return !(supportingToken instanceof SupportingTokens
+ && ((SupportingTokens)supportingToken).isEndorsing());
}
protected void addToken(SoapMessage message) {
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/WSS4JInInterceptor.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/WSS4JInInterceptor.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/WSS4JInInterceptor.java
index 78a7647..8212eed 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/WSS4JInInterceptor.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/WSS4JInInterceptor.java
@@ -451,30 +451,21 @@ public class WSS4JInInterceptor extends AbstractWSS4JInterceptor {
* Is a Nonce Cache required, i.e. are we expecting a UsernameToken
*/
protected boolean isNonceCacheRequired(List<Integer> actions, SoapMessage msg) {
- if (actions.contains(WSConstants.UT) || actions.contains(WSConstants.UT_NOPASSWORD)) {
- return true;
- }
- return false;
+ return actions.contains(WSConstants.UT) || actions.contains(WSConstants.UT_NOPASSWORD);
}
/**
* Is a Timestamp cache required, i.e. are we expecting a Timestamp
*/
protected boolean isTimestampCacheRequired(List<Integer> actions, SoapMessage msg) {
- if (actions.contains(WSConstants.TS)) {
- return true;
- }
- return false;
+ return actions.contains(WSConstants.TS);
}
/**
* Is a SAML Cache required, i.e. are we expecting a SAML Token
*/
protected boolean isSamlCacheRequired(List<Integer> actions, SoapMessage msg) {
- if (actions.contains(WSConstants.ST_UNSIGNED) || actions.contains(WSConstants.ST_SIGNED)) {
- return true;
- }
- return false;
+ return actions.contains(WSConstants.ST_UNSIGNED) || actions.contains(WSConstants.ST_SIGNED);
}
/**
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyhandlers/SymmetricBindingHandler.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyhandlers/SymmetricBindingHandler.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyhandlers/SymmetricBindingHandler.java
index bfc67e0..944b035 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyhandlers/SymmetricBindingHandler.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyhandlers/SymmetricBindingHandler.java
@@ -976,12 +976,8 @@ public class SymmetricBindingHandler extends AbstractBindingBuilder {
}
private boolean hasSignedPartsOrElements() {
- if (PolicyUtils.getFirstAssertionByLocalname(aim, SPConstants.SIGNED_PARTS) != null
- || PolicyUtils.getFirstAssertionByLocalname(aim, SPConstants.SIGNED_ELEMENTS) != null) {
- return true;
- }
-
- return false;
+ return PolicyUtils.getFirstAssertionByLocalname(aim, SPConstants.SIGNED_PARTS) != null
+ || PolicyUtils.getFirstAssertionByLocalname(aim, SPConstants.SIGNED_ELEMENTS) != null;
}
}
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/AbstractBindingPolicyValidator.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/AbstractBindingPolicyValidator.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/AbstractBindingPolicyValidator.java
index 55a00b5..f43ab81 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/AbstractBindingPolicyValidator.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/AbstractBindingPolicyValidator.java
@@ -82,10 +82,7 @@ public abstract class AbstractBindingPolicyValidator implements SecurityPolicyVa
if (includeTimestamp && (timestampResults == null || timestampResults.size() != 1)) {
return false;
} else if (!includeTimestamp) {
- if (timestampResults == null || timestampResults.isEmpty()) {
- return true;
- }
- return false;
+ return timestampResults == null || timestampResults.isEmpty();
}
// At this point we received a (required) Timestamp. Now check that it is integrity protected.
@@ -239,10 +236,7 @@ public abstract class AbstractBindingPolicyValidator implements SecurityPolicyVa
signed = true;
}
if (actInt.intValue() == WSConstants.ENCR && el != null) {
- if (signed) {
- return true;
- }
- return false;
+ return signed;
}
}
return false;
@@ -265,10 +259,7 @@ public abstract class AbstractBindingPolicyValidator implements SecurityPolicyVa
// Don't count an endorsing signature
if (actInt.intValue() == WSConstants.SIGN && el != null
&& !(el.size() == 1 && el.get(0).getName().equals(SIG_QNAME))) {
- if (encrypted) {
- return true;
- }
- return false;
+ return encrypted;
}
}
return false;
http://git-wip-us.apache.org/repos/asf/cxf/blob/3309231e/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/AbstractSupportingTokenPolicyValidator.java
----------------------------------------------------------------------
diff --git a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/AbstractSupportingTokenPolicyValidator.java b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/AbstractSupportingTokenPolicyValidator.java
index ad0c835..982be23 100644
--- a/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/AbstractSupportingTokenPolicyValidator.java
+++ b/rt/ws/security/src/main/java/org/apache/cxf/ws/security/wss4j/policyvalidators/AbstractSupportingTokenPolicyValidator.java
@@ -122,16 +122,12 @@ public abstract class AbstractSupportingTokenPolicyValidator extends AbstractSec
}
}
- if ((isEndorsing() && !checkEndorsed(tokenResults, parameters.getSignedResults(),
+ return !((isEndorsing() && !checkEndorsed(tokenResults, parameters.getSignedResults(),
parameters.getMessage(),
parameters.getTimestampElement()))
|| !validateSignedEncryptedPolicies(tokenResults, parameters.getSignedResults(),
parameters.getEncryptedResults(),
- parameters.getMessage())) {
- return false;
- }
-
- return true;
+ parameters.getMessage()));
}
@@ -159,13 +155,9 @@ public abstract class AbstractSupportingTokenPolicyValidator extends AbstractSec
return false;
}
- if (!validateSignedEncryptedPolicies(parameters.getSamlResults(), parameters.getSignedResults(),
+ return validateSignedEncryptedPolicies(parameters.getSamlResults(), parameters.getSignedResults(),
parameters.getEncryptedResults(),
- parameters.getMessage())) {
- return false;
- }
-
- return true;
+ parameters.getMessage());
}
@@ -218,13 +210,9 @@ public abstract class AbstractSupportingTokenPolicyValidator extends AbstractSec
return false;
}
- if (!validateSignedEncryptedPolicies(tokenResults, parameters.getSignedResults(),
+ return validateSignedEncryptedPolicies(tokenResults, parameters.getSignedResults(),
parameters.getEncryptedResults(),
- parameters.getMessage())) {
- return false;
- }
-
- return true;
+ parameters.getMessage());
}
@@ -278,13 +266,9 @@ public abstract class AbstractSupportingTokenPolicyValidator extends AbstractSec
return false;
}
- if (!validateSignedEncryptedPolicies(tokenResults, parameters.getSignedResults(),
+ return validateSignedEncryptedPolicies(tokenResults, parameters.getSignedResults(),
parameters.getEncryptedResults(),
- parameters.getMessage())) {
- return false;
- }
-
- return true;
+ parameters.getMessage());
}
/**
@@ -322,13 +306,9 @@ public abstract class AbstractSupportingTokenPolicyValidator extends AbstractSec
return false;
}
- if (!validateSignedEncryptedPolicies(tokenResults, parameters.getSignedResults(),
+ return validateSignedEncryptedPolicies(tokenResults, parameters.getSignedResults(),
parameters.getEncryptedResults(),
- parameters.getMessage())) {
- return false;
- }
-
- return true;
+ parameters.getMessage());
}
/**
@@ -351,12 +331,7 @@ public abstract class AbstractSupportingTokenPolicyValidator extends AbstractSec
return false;
}
- if (!validateSignedEncryptedElements(encryptedElements, false, encryptedResults, tokenResults,
- message)) {
- return false;
- }
-
- return true;
+ return validateSignedEncryptedElements(encryptedElements, false, encryptedResults, tokenResults, message);
}
@@ -399,13 +374,9 @@ public abstract class AbstractSupportingTokenPolicyValidator extends AbstractSec
return false;
}
- if (!validateSignedEncryptedPolicies(tokenResults, parameters.getSignedResults(),
+ return validateSignedEncryptedPolicies(tokenResults, parameters.getSignedResults(),
parameters.getEncryptedResults(),
- parameters.getMessage())) {
- return false;
- }
-
- return true;
+ parameters.getMessage());
}
/**
@@ -462,10 +433,7 @@ public abstract class AbstractSupportingTokenPolicyValidator extends AbstractSec
private boolean isTLSInUse(Message message) {
// See whether TLS is in use or not
TLSSessionInfo tlsInfo = message.get(TLSSessionInfo.class);
- if (tlsInfo != null) {
- return true;
- }
- return false;
+ return tlsInfo != null;
}
/**