| commit | a56d06256838c734e4b5f1269117a1095457df34 | [log] [tgz] |
|---|---|---|
| author | George Kankava <george.kankava@devfactory.com> | Tue Jun 28 21:25:30 2016 +0400 |
| committer | John D. Ament <johndament@apache.org> | Wed Jul 13 00:01:21 2016 -0400 |
| tree | c64a7099e3a0e5e0546d3cf60a10ec612274a01f | |
| parent | 2bc54060a4aff2e01ecfdd0c1eaaeec52a39b738 [diff] |
DELTASPIKE-645 squid:S2131 - Primitives should not be boxed just for 'String' conversion This closes #54
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 @@ } else { - methodName = "" + Character.toUpperCase(fieldName.charAt(0)); + methodName = Character.toString(Character.toUpperCase(fieldName.charAt(0))); } Method setter = null;
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 @@ 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)