EMPIREDB-304
code optimization
diff --git a/empire-db-jsf2/src/main/java/org/apache/empire/jsf2/controls/RadioInputControl.java b/empire-db-jsf2/src/main/java/org/apache/empire/jsf2/controls/RadioInputControl.java
index b43fbdc..de2ad93 100644
--- a/empire-db-jsf2/src/main/java/org/apache/empire/jsf2/controls/RadioInputControl.java
+++ b/empire-db-jsf2/src/main/java/org/apache/empire/jsf2/controls/RadioInputControl.java
@@ -326,9 +326,10 @@
@Override
protected Object parseInputValue(String value, InputInfo ii)
{
- if (ii.getColumn().isEnum())
+ Class<Enum<?>> enumType = ii.getColumn().getEnumType();
+ if (enumType!=null)
{ // convert to enum
- return ObjectUtils.getEnum(ii.getColumn().getEnumType(), value);
+ return ObjectUtils.getEnum(enumType, value);
}
return value;
}
diff --git a/empire-db-jsf2/src/main/java/org/apache/empire/jsf2/controls/SelectInputControl.java b/empire-db-jsf2/src/main/java/org/apache/empire/jsf2/controls/SelectInputControl.java
index 91a89e1..1e1ee54 100644
--- a/empire-db-jsf2/src/main/java/org/apache/empire/jsf2/controls/SelectInputControl.java
+++ b/empire-db-jsf2/src/main/java/org/apache/empire/jsf2/controls/SelectInputControl.java
@@ -325,9 +325,10 @@
{
if ((value instanceof Number))
{ // Check whether it's an Enum
- if (ii.getColumn().isEnum())
- { // Convert ordinal value to Enum-name
- Enum<?> enumVal = ObjectUtils.getEnum(ii.getColumn().getEnumType(), value);
+ Class<Enum<?>> enumType = ii.getColumn().getEnumType();
+ if (enumType!=null)
+ { // Convert ordinal to name
+ Enum<?> enumVal = ObjectUtils.getEnum(enumType, value);
value = enumVal.name();
}
}
@@ -349,9 +350,10 @@
@Override
protected Object parseInputValue(String value, InputInfo ii)
{
- if (ii.getColumn().isEnum())
+ Class<Enum<?>> enumType = ii.getColumn().getEnumType();
+ if (enumType!=null)
{ // convert to enum
- return ObjectUtils.getEnum(ii.getColumn().getEnumType(), value);
+ return ObjectUtils.getEnum(enumType, value);
}
return value;
}
diff --git a/empire-db-jsf2/src/main/java/org/apache/empire/jsf2/utils/TagEncodingHelper.java b/empire-db-jsf2/src/main/java/org/apache/empire/jsf2/utils/TagEncodingHelper.java
index a8b6429..ae83f15 100644
--- a/empire-db-jsf2/src/main/java/org/apache/empire/jsf2/utils/TagEncodingHelper.java
+++ b/empire-db-jsf2/src/main/java/org/apache/empire/jsf2/utils/TagEncodingHelper.java
@@ -170,12 +170,6 @@
}
@Override
- public boolean isEnum()
- {
- return getSourceColumn().isEnum();
- }
-
- @Override
public Class<Enum<?>> getEnumType()
{
return getSourceColumn().getEnumType();
diff --git a/empire-db/src/main/java/org/apache/empire/data/Column.java b/empire-db/src/main/java/org/apache/empire/data/Column.java
index 58e6e0d..fa92525 100644
--- a/empire-db/src/main/java/org/apache/empire/data/Column.java
+++ b/empire-db/src/main/java/org/apache/empire/data/Column.java
@@ -81,13 +81,6 @@
boolean isReadOnly();
/**
- * Returns true if an enum type has been set for this column
- * <P>
- * @return true if an enum type has been set for this column
- */
- boolean isEnum();
-
- /**
* Returns the enum type for this column
* <P>
* @return the enum type
diff --git a/empire-db/src/main/java/org/apache/empire/data/bean/BeanProperty.java b/empire-db/src/main/java/org/apache/empire/data/bean/BeanProperty.java
index ccbac58..63690b0 100644
--- a/empire-db/src/main/java/org/apache/empire/data/bean/BeanProperty.java
+++ b/empire-db/src/main/java/org/apache/empire/data/bean/BeanProperty.java
@@ -166,17 +166,6 @@
}
/**
- * Returns true if an enum type has been set for this column
- * <P>
- * @return eturns true if an enum type has been set for this column
- */
- @Override
- public boolean isEnum()
- {
- return (getEnumType()!=null);
- }
-
- /**
* Returns the enum type for this column
* <P>
* @return the enum type
diff --git a/empire-db/src/main/java/org/apache/empire/db/DBColumn.java b/empire-db/src/main/java/org/apache/empire/db/DBColumn.java
index c05632a..95b88b5 100644
--- a/empire-db/src/main/java/org/apache/empire/db/DBColumn.java
+++ b/empire-db/src/main/java/org/apache/empire/db/DBColumn.java
@@ -389,8 +389,7 @@
* <P>
* @return eturns true if an enum type has been set for this column
*/
- @Override
- public boolean isEnum()
+ public final boolean isEnum()
{
return (getEnumType()!=null);
}