You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@deltaspike.apache.org by jo...@apache.org on 2016/07/13 04:01:40 UTC
[1/3] deltaspike git commit: DELTASPIKE-645 squid:S2583 - Conditions
should not unconditionally evaluate to 'TRUE' or to 'FALSE'
Repository: deltaspike
Updated Branches:
refs/heads/master 7b3606bb4 -> 712fc6dca
DELTASPIKE-645 squid:S2583 - Conditions should not unconditionally evaluate to 'TRUE' or to 'FALSE'
This closes #55
Project: http://git-wip-us.apache.org/repos/asf/deltaspike/repo
Commit: http://git-wip-us.apache.org/repos/asf/deltaspike/commit/2bc54060
Tree: http://git-wip-us.apache.org/repos/asf/deltaspike/tree/2bc54060
Diff: http://git-wip-us.apache.org/repos/asf/deltaspike/diff/2bc54060
Branch: refs/heads/master
Commit: 2bc54060a4aff2e01ecfdd0c1eaaeec52a39b738
Parents: 7b3606b
Author: George Kankava <ge...@devfactory.com>
Authored: Tue Jun 28 21:40:34 2016 +0400
Committer: John D. Ament <jo...@apache.org>
Committed: Wed Jul 13 00:01:20 2016 -0400
----------------------------------------------------------------------
.../deltaspike/data/api/mapping/SimpleQueryInOutMapperBase.java | 5 +++--
.../org/apache/deltaspike/data/impl/meta/RepositoryMethod.java | 2 +-
.../jsf/impl/component/window/WindowIdHtmlRenderer.java | 2 +-
3 files changed, 5 insertions(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/deltaspike/blob/2bc54060/deltaspike/modules/data/api/src/main/java/org/apache/deltaspike/data/api/mapping/SimpleQueryInOutMapperBase.java
----------------------------------------------------------------------
diff --git a/deltaspike/modules/data/api/src/main/java/org/apache/deltaspike/data/api/mapping/SimpleQueryInOutMapperBase.java b/deltaspike/modules/data/api/src/main/java/org/apache/deltaspike/data/api/mapping/SimpleQueryInOutMapperBase.java
index 874e535..e82fe31 100644
--- a/deltaspike/modules/data/api/src/main/java/org/apache/deltaspike/data/api/mapping/SimpleQueryInOutMapperBase.java
+++ b/deltaspike/modules/data/api/src/main/java/org/apache/deltaspike/data/api/mapping/SimpleQueryInOutMapperBase.java
@@ -73,15 +73,16 @@ public abstract class SimpleQueryInOutMapperBase<Entity, Dto> implements QueryIn
@Override
public Object mapResultList(final List<Entity> result)
{
- final List<Object> mapped = new ArrayList<Object>(result.size());
if (result != null)
{
+ final List<Object> mapped = new ArrayList<Object>(result.size());
for (final Entity a : result)
{
mapped.add(mapResult(a));
}
+ return mapped;
}
- return mapped;
+ return new ArrayList<Object>();
}
@Override
http://git-wip-us.apache.org/repos/asf/deltaspike/blob/2bc54060/deltaspike/modules/data/impl/src/main/java/org/apache/deltaspike/data/impl/meta/RepositoryMethod.java
----------------------------------------------------------------------
diff --git a/deltaspike/modules/data/impl/src/main/java/org/apache/deltaspike/data/impl/meta/RepositoryMethod.java b/deltaspike/modules/data/impl/src/main/java/org/apache/deltaspike/data/impl/meta/RepositoryMethod.java
index e349e56..0522b32 100644
--- a/deltaspike/modules/data/impl/src/main/java/org/apache/deltaspike/data/impl/meta/RepositoryMethod.java
+++ b/deltaspike/modules/data/impl/src/main/java/org/apache/deltaspike/data/impl/meta/RepositoryMethod.java
@@ -189,7 +189,7 @@ public class RepositoryMethod
return;
}
- if (queryInOutMapperIsNormalScope != null && beanManager != null)
+ if (beanManager != null)
{
final Set<Bean<?>> beans = beanManager.getBeans(mapper);
final Class<? extends Annotation> scope = beanManager.resolve(beans).getScope();
http://git-wip-us.apache.org/repos/asf/deltaspike/blob/2bc54060/deltaspike/modules/jsf/impl/src/main/java/org/apache/deltaspike/jsf/impl/component/window/WindowIdHtmlRenderer.java
----------------------------------------------------------------------
diff --git a/deltaspike/modules/jsf/impl/src/main/java/org/apache/deltaspike/jsf/impl/component/window/WindowIdHtmlRenderer.java b/deltaspike/modules/jsf/impl/src/main/java/org/apache/deltaspike/jsf/impl/component/window/WindowIdHtmlRenderer.java
index 755f0c6..ed2752d 100644
--- a/deltaspike/modules/jsf/impl/src/main/java/org/apache/deltaspike/jsf/impl/component/window/WindowIdHtmlRenderer.java
+++ b/deltaspike/modules/jsf/impl/src/main/java/org/apache/deltaspike/jsf/impl/component/window/WindowIdHtmlRenderer.java
@@ -88,7 +88,7 @@ public class WindowIdHtmlRenderer extends Renderer
+ clientWindowConfig.isClientWindowStoreWindowTreeEnabledOnButtonClick());
// see #729
- if (!delegatedWindowMode && clientWindow.isInitialRedirectSupported(context))
+ if (clientWindow.isInitialRedirectSupported(context))
{
Object cookie = ClientWindowHelper.getRequestWindowIdCookie(context, windowId);
if (cookie != null && cookie instanceof Cookie)
[2/3] deltaspike git commit: DELTASPIKE-645 squid:S2131 - Primitives
should not be boxed just for 'String' conversion
Posted by jo...@apache.org.
DELTASPIKE-645 squid:S2131 - Primitives should not be boxed just for 'String' conversion
This closes #54
Project: http://git-wip-us.apache.org/repos/asf/deltaspike/repo
Commit: http://git-wip-us.apache.org/repos/asf/deltaspike/commit/a56d0625
Tree: http://git-wip-us.apache.org/repos/asf/deltaspike/tree/a56d0625
Diff: http://git-wip-us.apache.org/repos/asf/deltaspike/diff/a56d0625
Branch: refs/heads/master
Commit: a56d06256838c734e4b5f1269117a1095457df34
Parents: 2bc5406
Author: George Kankava <ge...@devfactory.com>
Authored: Tue Jun 28 21:25:30 2016 +0400
Committer: John D. Ament <jo...@apache.org>
Committed: Wed Jul 13 00:01:21 2016 -0400
----------------------------------------------------------------------
.../org/apache/deltaspike/core/impl/jmx/DynamicMBeanWrapper.java | 2 +-
.../impl/scope/window/strategy/AbstractClientWindowStrategy.java | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/deltaspike/blob/a56d0625/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/jmx/DynamicMBeanWrapper.java
----------------------------------------------------------------------
diff --git a/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/jmx/DynamicMBeanWrapper.java b/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/jmx/DynamicMBeanWrapper.java
index 00b0cb0..1ff1365 100644
--- a/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/jmx/DynamicMBeanWrapper.java
+++ b/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/jmx/DynamicMBeanWrapper.java
@@ -159,7 +159,7 @@ public class DynamicMBeanWrapper extends NotificationBroadcasterSupport implemen
}
else
{
- methodName = "" + Character.toUpperCase(fieldName.charAt(0));
+ methodName = Character.toString(Character.toUpperCase(fieldName.charAt(0)));
}
Method setter = null;
http://git-wip-us.apache.org/repos/asf/deltaspike/blob/a56d0625/deltaspike/modules/jsf/impl/src/main/java/org/apache/deltaspike/jsf/impl/scope/window/strategy/AbstractClientWindowStrategy.java
----------------------------------------------------------------------
diff --git a/deltaspike/modules/jsf/impl/src/main/java/org/apache/deltaspike/jsf/impl/scope/window/strategy/AbstractClientWindowStrategy.java b/deltaspike/modules/jsf/impl/src/main/java/org/apache/deltaspike/jsf/impl/scope/window/strategy/AbstractClientWindowStrategy.java
index 111f85b..4078e45 100644
--- a/deltaspike/modules/jsf/impl/src/main/java/org/apache/deltaspike/jsf/impl/scope/window/strategy/AbstractClientWindowStrategy.java
+++ b/deltaspike/modules/jsf/impl/src/main/java/org/apache/deltaspike/jsf/impl/scope/window/strategy/AbstractClientWindowStrategy.java
@@ -101,12 +101,12 @@ public abstract class AbstractClientWindowStrategy implements ClientWindow
protected String generateNewWindowId()
{
//X TODO proper mechanism
- return "" + (new Random()).nextInt() % 10000;
+ return Integer.toString((new Random()).nextInt() % 10000);
}
protected String generateNewRequestToken()
{
- return "" + ((int) Math.floor(Math.random() * 999));
+ return Integer.toString((int) Math.floor(Math.random() * 999));
}
protected boolean isPost(FacesContext facesContext)
[3/3] deltaspike git commit: DELTASPIKE-645 squid:S1155 -
Collection.isEmpty() should be used to test for emptiness
Posted by jo...@apache.org.
DELTASPIKE-645 squid:S1155 - Collection.isEmpty() should be used to test for emptiness
This closes #53
Project: http://git-wip-us.apache.org/repos/asf/deltaspike/repo
Commit: http://git-wip-us.apache.org/repos/asf/deltaspike/commit/712fc6dc
Tree: http://git-wip-us.apache.org/repos/asf/deltaspike/tree/712fc6dc
Diff: http://git-wip-us.apache.org/repos/asf/deltaspike/diff/712fc6dc
Branch: refs/heads/master
Commit: 712fc6dca9bb732bee094b9107929b20d439c952
Parents: a56d062
Author: George Kankava <ge...@devfactory.com>
Authored: Tue Jun 28 21:13:01 2016 +0400
Committer: John D. Ament <jo...@apache.org>
Committed: Wed Jul 13 00:01:21 2016 -0400
----------------------------------------------------------------------
.../core/api/resourceloader/ClasspathResourceProvider.java | 2 +-
.../deltaspike/data/impl/builder/result/DefaultQueryResult.java | 2 +-
.../deltaspike/data/impl/builder/result/QueryProcessorFactory.java | 2 +-
.../org/apache/deltaspike/data/impl/criteria/QueryCriteria.java | 2 +-
.../jpa/impl/transaction/context/TransactionBeanStorage.java | 2 +-
.../org/apache/deltaspike/proxy/api/DeltaSpikeProxyFactory.java | 2 +-
.../org/apache/deltaspike/proxy/impl/AsmProxyClassGenerator.java | 2 +-
.../proxy/impl/invocation/AbstractManualInvocationHandler.java | 2 +-
.../org/apache/deltaspike/scheduler/impl/SchedulerExtension.java | 2 +-
.../security/impl/authorization/SecuredAnnotationAuthorizer.java | 2 +-
10 files changed, 10 insertions(+), 10 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/deltaspike/blob/712fc6dc/deltaspike/core/api/src/main/java/org/apache/deltaspike/core/api/resourceloader/ClasspathResourceProvider.java
----------------------------------------------------------------------
diff --git a/deltaspike/core/api/src/main/java/org/apache/deltaspike/core/api/resourceloader/ClasspathResourceProvider.java b/deltaspike/core/api/src/main/java/org/apache/deltaspike/core/api/resourceloader/ClasspathResourceProvider.java
index 618ae62..47aeb77 100644
--- a/deltaspike/core/api/src/main/java/org/apache/deltaspike/core/api/resourceloader/ClasspathResourceProvider.java
+++ b/deltaspike/core/api/src/main/java/org/apache/deltaspike/core/api/resourceloader/ClasspathResourceProvider.java
@@ -120,7 +120,7 @@ public class ClasspathResourceProvider extends AbstractResourceProvider
private String urlsToString(List<URL> urls, String name)
{
- if (urls.size() == 0)
+ if (urls.isEmpty())
{
return String.format("No resources found for '%s'",name);
}
http://git-wip-us.apache.org/repos/asf/deltaspike/blob/712fc6dc/deltaspike/modules/data/impl/src/main/java/org/apache/deltaspike/data/impl/builder/result/DefaultQueryResult.java
----------------------------------------------------------------------
diff --git a/deltaspike/modules/data/impl/src/main/java/org/apache/deltaspike/data/impl/builder/result/DefaultQueryResult.java b/deltaspike/modules/data/impl/src/main/java/org/apache/deltaspike/data/impl/builder/result/DefaultQueryResult.java
index 2548984..bc70a1f 100644
--- a/deltaspike/modules/data/impl/src/main/java/org/apache/deltaspike/data/impl/builder/result/DefaultQueryResult.java
+++ b/deltaspike/modules/data/impl/src/main/java/org/apache/deltaspike/data/impl/builder/result/DefaultQueryResult.java
@@ -231,7 +231,7 @@ public class DefaultQueryResult<T> implements QueryResult<T>
public T getAnyResult()
{
List<T> queryResult = getResultList();
- return queryResult.size() > 0 ? queryResult.get(0) : null;
+ return !queryResult.isEmpty() ? queryResult.get(0) : null;
}
@Override
http://git-wip-us.apache.org/repos/asf/deltaspike/blob/712fc6dc/deltaspike/modules/data/impl/src/main/java/org/apache/deltaspike/data/impl/builder/result/QueryProcessorFactory.java
----------------------------------------------------------------------
diff --git a/deltaspike/modules/data/impl/src/main/java/org/apache/deltaspike/data/impl/builder/result/QueryProcessorFactory.java b/deltaspike/modules/data/impl/src/main/java/org/apache/deltaspike/data/impl/builder/result/QueryProcessorFactory.java
index 9a72dca..4bc5777 100644
--- a/deltaspike/modules/data/impl/src/main/java/org/apache/deltaspike/data/impl/builder/result/QueryProcessorFactory.java
+++ b/deltaspike/modules/data/impl/src/main/java/org/apache/deltaspike/data/impl/builder/result/QueryProcessorFactory.java
@@ -149,7 +149,7 @@ public final class QueryProcessorFactory
default:
@SuppressWarnings("unchecked")
List<Object> queryResult = query.getResultList();
- result = queryResult.size() > 0 ? queryResult.get(0) : null;
+ result = !queryResult.isEmpty() ? queryResult.get(0) : null;
}
if (context.isOptional())
{
http://git-wip-us.apache.org/repos/asf/deltaspike/blob/712fc6dc/deltaspike/modules/data/impl/src/main/java/org/apache/deltaspike/data/impl/criteria/QueryCriteria.java
----------------------------------------------------------------------
diff --git a/deltaspike/modules/data/impl/src/main/java/org/apache/deltaspike/data/impl/criteria/QueryCriteria.java b/deltaspike/modules/data/impl/src/main/java/org/apache/deltaspike/data/impl/criteria/QueryCriteria.java
index ab66083..4157114 100644
--- a/deltaspike/modules/data/impl/src/main/java/org/apache/deltaspike/data/impl/criteria/QueryCriteria.java
+++ b/deltaspike/modules/data/impl/src/main/java/org/apache/deltaspike/data/impl/criteria/QueryCriteria.java
@@ -132,7 +132,7 @@ public class QueryCriteria<C, R> implements Criteria<C, R>
public R getAnyResult()
{
List<R> queryResult = getResultList();
- return queryResult.size() > 0 ? queryResult.get(0) : null;
+ return !queryResult.isEmpty() ? queryResult.get(0) : null;
}
@Override
http://git-wip-us.apache.org/repos/asf/deltaspike/blob/712fc6dc/deltaspike/modules/jpa/impl/src/main/java/org/apache/deltaspike/jpa/impl/transaction/context/TransactionBeanStorage.java
----------------------------------------------------------------------
diff --git a/deltaspike/modules/jpa/impl/src/main/java/org/apache/deltaspike/jpa/impl/transaction/context/TransactionBeanStorage.java b/deltaspike/modules/jpa/impl/src/main/java/org/apache/deltaspike/jpa/impl/transaction/context/TransactionBeanStorage.java
index 1a37fef..376051e 100644
--- a/deltaspike/modules/jpa/impl/src/main/java/org/apache/deltaspike/jpa/impl/transaction/context/TransactionBeanStorage.java
+++ b/deltaspike/modules/jpa/impl/src/main/java/org/apache/deltaspike/jpa/impl/transaction/context/TransactionBeanStorage.java
@@ -183,7 +183,7 @@ public class TransactionBeanStorage
destroyBeans(currentTci.contextualInstances);
- if (oldTci.size() > 0)
+ if (!oldTci.isEmpty())
{
currentTci = oldTci.pop();
endTransactionScope();
http://git-wip-us.apache.org/repos/asf/deltaspike/blob/712fc6dc/deltaspike/modules/proxy/api/src/main/java/org/apache/deltaspike/proxy/api/DeltaSpikeProxyFactory.java
----------------------------------------------------------------------
diff --git a/deltaspike/modules/proxy/api/src/main/java/org/apache/deltaspike/proxy/api/DeltaSpikeProxyFactory.java b/deltaspike/modules/proxy/api/src/main/java/org/apache/deltaspike/proxy/api/DeltaSpikeProxyFactory.java
index b30c398..19a8099 100644
--- a/deltaspike/modules/proxy/api/src/main/java/org/apache/deltaspike/proxy/api/DeltaSpikeProxyFactory.java
+++ b/deltaspike/modules/proxy/api/src/main/java/org/apache/deltaspike/proxy/api/DeltaSpikeProxyFactory.java
@@ -110,7 +110,7 @@ public abstract class DeltaSpikeProxyFactory
// check if a interceptor is defined on class level. if not, skip interceptor methods
if (interceptMethods != null
- && interceptMethods.size() > 0
+ && !interceptMethods.isEmpty()
&& !containsInterceptorBinding(beanManager, targetClass.getDeclaredAnnotations()))
{
// loop every method and check if a interceptor is defined on the method -> otherwise don't overwrite
http://git-wip-us.apache.org/repos/asf/deltaspike/blob/712fc6dc/deltaspike/modules/proxy/impl-asm5/src/main/java/org/apache/deltaspike/proxy/impl/AsmProxyClassGenerator.java
----------------------------------------------------------------------
diff --git a/deltaspike/modules/proxy/impl-asm5/src/main/java/org/apache/deltaspike/proxy/impl/AsmProxyClassGenerator.java b/deltaspike/modules/proxy/impl-asm5/src/main/java/org/apache/deltaspike/proxy/impl/AsmProxyClassGenerator.java
index a887c28..64408a8 100644
--- a/deltaspike/modules/proxy/impl-asm5/src/main/java/org/apache/deltaspike/proxy/impl/AsmProxyClassGenerator.java
+++ b/deltaspike/modules/proxy/impl-asm5/src/main/java/org/apache/deltaspike/proxy/impl/AsmProxyClassGenerator.java
@@ -337,7 +337,7 @@ public class AsmProxyClassGenerator implements ProxyClassGenerator
// catch checked exceptions and rethrow it
boolean throwableCatched = false;
- if (exceptionsToCatch.size() > 0)
+ if (!exceptionsToCatch.isEmpty())
{
rethrow = mg.mark();
mg.visitVarInsn(Opcodes.ASTORE, 1);
http://git-wip-us.apache.org/repos/asf/deltaspike/blob/712fc6dc/deltaspike/modules/proxy/impl-asm5/src/main/java/org/apache/deltaspike/proxy/impl/invocation/AbstractManualInvocationHandler.java
----------------------------------------------------------------------
diff --git a/deltaspike/modules/proxy/impl-asm5/src/main/java/org/apache/deltaspike/proxy/impl/invocation/AbstractManualInvocationHandler.java b/deltaspike/modules/proxy/impl-asm5/src/main/java/org/apache/deltaspike/proxy/impl/invocation/AbstractManualInvocationHandler.java
index 37f763c..92bfc3c 100644
--- a/deltaspike/modules/proxy/impl-asm5/src/main/java/org/apache/deltaspike/proxy/impl/invocation/AbstractManualInvocationHandler.java
+++ b/deltaspike/modules/proxy/impl-asm5/src/main/java/org/apache/deltaspike/proxy/impl/invocation/AbstractManualInvocationHandler.java
@@ -36,7 +36,7 @@ public abstract class AbstractManualInvocationHandler implements InvocationHandl
// check if interceptors are defined, otherwise just call the original logik
List<Interceptor<?>> interceptors = interceptorLookup.lookup(proxy, method);
- if (interceptors != null && interceptors.size() > 0)
+ if (interceptors != null && !interceptors.isEmpty())
{
try
{
http://git-wip-us.apache.org/repos/asf/deltaspike/blob/712fc6dc/deltaspike/modules/scheduler/impl/src/main/java/org/apache/deltaspike/scheduler/impl/SchedulerExtension.java
----------------------------------------------------------------------
diff --git a/deltaspike/modules/scheduler/impl/src/main/java/org/apache/deltaspike/scheduler/impl/SchedulerExtension.java b/deltaspike/modules/scheduler/impl/src/main/java/org/apache/deltaspike/scheduler/impl/SchedulerExtension.java
index 1780373..6bb5790 100644
--- a/deltaspike/modules/scheduler/impl/src/main/java/org/apache/deltaspike/scheduler/impl/SchedulerExtension.java
+++ b/deltaspike/modules/scheduler/impl/src/main/java/org/apache/deltaspike/scheduler/impl/SchedulerExtension.java
@@ -221,7 +221,7 @@ public class SchedulerExtension implements Extension, Deactivatable
afterBeanDiscovery.addDefinitionError(t);
}
}
- else if (this.foundManagedJobClasses.size() > 0)
+ else if (!this.foundManagedJobClasses.isEmpty())
{
LOG.warning(
this.foundManagedJobClasses.size() + " scheduling-jobs found, but there is no configured scheduler");
http://git-wip-us.apache.org/repos/asf/deltaspike/blob/712fc6dc/deltaspike/modules/security/impl/src/main/java/org/apache/deltaspike/security/impl/authorization/SecuredAnnotationAuthorizer.java
----------------------------------------------------------------------
diff --git a/deltaspike/modules/security/impl/src/main/java/org/apache/deltaspike/security/impl/authorization/SecuredAnnotationAuthorizer.java b/deltaspike/modules/security/impl/src/main/java/org/apache/deltaspike/security/impl/authorization/SecuredAnnotationAuthorizer.java
index 4ece4bb..a718137 100644
--- a/deltaspike/modules/security/impl/src/main/java/org/apache/deltaspike/security/impl/authorization/SecuredAnnotationAuthorizer.java
+++ b/deltaspike/modules/security/impl/src/main/java/org/apache/deltaspike/security/impl/authorization/SecuredAnnotationAuthorizer.java
@@ -125,7 +125,7 @@ public class SecuredAnnotationAuthorizer
violations = voter.checkPermission(voterContext);
- if (violations != null && violations.size() > 0)
+ if (violations != null && !violations.isEmpty())
{
if (voterContext instanceof EditableAccessDecisionVoterContext)
{