You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by do...@apache.org on 2007/10/17 19:32:48 UTC
svn commit: r585589 -
/ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/ObjectType.java
Author: doogie
Date: Wed Oct 17 10:32:47 2007
New Revision: 585589
URL: http://svn.apache.org/viewvc?rev=585589&view=rev
Log:
In simpleTypeConvert, where ever there was an if block that returned
TRUE/FALSE, they are now inlined, using ?:. Closes
https://issues.apache.org/jira/browse/OFBIZ-1336,
Modified:
ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/ObjectType.java
Modified: ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/ObjectType.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/ObjectType.java?rev=585589&r1=585588&r2=585589&view=diff
==============================================================================
--- ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/ObjectType.java (original)
+++ ofbiz/trunk/framework/base/src/base/org/ofbiz/base/util/ObjectType.java Wed Oct 17 10:32:47 2007
@@ -478,13 +478,7 @@
if ("Boolean".equals(type) || "java.lang.Boolean".equals(type)) {
str = StringUtil.removeSpaces(str);
- Boolean value = null;
- if (str.equalsIgnoreCase("TRUE")) {
- value = Boolean.TRUE;
- } else {
- value = Boolean.FALSE;
- }
- return value;
+ return str.equalsIgnoreCase("TRUE") ? Boolean.TRUE : Boolean.FALSE;
} else if ("Locale".equals(type) || "java.util.Locale".equals(type)) {
Locale loc = UtilMisc.parseLocale(str);
if (loc != null) {
@@ -1049,11 +1043,7 @@
// have converted value 2, now before converting value 1 see if it is a Collection and we are doing a contains comparison
if ("contains".equals(operator) && value1 instanceof Collection) {
Collection col1 = (Collection) value1;
- if (col1.contains(convertedValue2)) {
- return Boolean.TRUE;
- } else {
- return Boolean.FALSE;
- }
+ return col1.contains(convertedValue2) ? Boolean.TRUE : Boolean.FALSE;
}
Object convertedValue1 = null;
@@ -1068,17 +1058,9 @@
// handle null values...
if (convertedValue1 == null || convertedValue2 == null) {
if ("equals".equals(operator)) {
- if (convertedValue1 == null && convertedValue2 == null) {
- return Boolean.TRUE;
- } else {
- return Boolean.FALSE;
- }
+ return convertedValue1 == null && convertedValue2 == null ? Boolean.TRUE : Boolean.FALSE;
} else if ("not-equals".equals(operator)) {
- if (convertedValue1 == null && convertedValue2 == null) {
- return Boolean.FALSE;
- } else {
- return Boolean.TRUE;
- }
+ return convertedValue1 == null && convertedValue2 == null ? Boolean.FALSE : Boolean.TRUE;
} else if ("is-not-empty".equals(operator) || "is-empty".equals(operator)) {
// do nothing, handled later...
} else {
@@ -1098,11 +1080,7 @@
String str1 = (String) convertedValue1;
String str2 = (String) convertedValue2;
- if (str1.indexOf(str2) < 0) {
- return Boolean.FALSE;
- } else {
- return Boolean.TRUE;
- }
+ return str1.indexOf(str2) < 0 ? Boolean.FALSE : Boolean.TRUE;
} else {
messages.add("Error in XML file: cannot do a contains compare between a String and a non-String type");
return null;
@@ -1135,17 +1113,9 @@
if (str1.length() == 0 || str2.length() == 0) {
if ("equals".equals(operator)) {
- if (str1.length() == 0 && str2.length() == 0) {
- return Boolean.TRUE;
- } else {
- return Boolean.FALSE;
- }
+ return str1.length() == 0 && str2.length() == 0 ? Boolean.TRUE : Boolean.FALSE;
} else if ("not-equals".equals(operator)) {
- if (str1.length() == 0 && str2.length() == 0) {
- return Boolean.FALSE;
- } else {
- return Boolean.TRUE;
- }
+ return str1.length() == 0 && str2.length() == 0 ? Boolean.FALSE : Boolean.TRUE;
} else {
messages.add("ERROR: Could not do a compare between strings with one empty string for the operator " + operator);
return null;