You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@empire-db.apache.org by do...@apache.org on 2019/12/29 12:02:58 UTC
[empire-db] branch master updated: EMPIREDB-282 JSF options: allow
custom text for null value
This is an automated email from the ASF dual-hosted git repository.
doebele pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/empire-db.git
The following commit(s) were added to refs/heads/master by this push:
new 7726733 EMPIREDB-282 JSF options: allow custom text for null value
7726733 is described below
commit 7726733c2940b76fad2ab920d65da2bde0a4c41b
Author: Rainer Döbele <do...@apache.org>
AuthorDate: Sun Dec 29 13:02:54 2019 +0100
EMPIREDB-282
JSF options: allow custom text for null value
---
.../main/java/org/apache/empire/jsf2/controls/InputControl.java | 7 ++++---
.../java/org/apache/empire/jsf2/controls/RadioInputControl.java | 8 ++++----
.../java/org/apache/empire/jsf2/controls/SelectInputControl.java | 6 ++++--
.../java/org/apache/empire/jsf2/controls/TextInputControl.java | 5 +++--
4 files changed, 15 insertions(+), 11 deletions(-)
diff --git a/empire-db-jsf2/src/main/java/org/apache/empire/jsf2/controls/InputControl.java b/empire-db-jsf2/src/main/java/org/apache/empire/jsf2/controls/InputControl.java
index fca3d70..c3d7397 100644
--- a/empire-db-jsf2/src/main/java/org/apache/empire/jsf2/controls/InputControl.java
+++ b/empire-db-jsf2/src/main/java/org/apache/empire/jsf2/controls/InputControl.java
@@ -698,12 +698,13 @@ public abstract class InputControl
// Lookup and Print value
Options options = vi.getOptions();
if (options != null && !options.isEmpty() && !hasFormatOption(vi, "nolookup"))
- { // Check for Options
+ { // Check for Options
String text = options.get(value);
- if (text != null)
+ if (StringUtils.isNotEmpty(text))
return vi.getText(text);
// Error
- InputControl.log.error("The element '" + String.valueOf(value) + "' is not part of the supplied option list.");
+ if (value!=null)
+ InputControl.log.error("The element '" + String.valueOf(value) + "' is not part of the supplied option list.");
}
// value
if (value == null)
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 de2ad93..955f981 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
@@ -33,6 +33,7 @@ import javax.faces.context.ResponseWriter;
import org.apache.empire.commons.ObjectUtils;
import org.apache.empire.commons.OptionEntry;
import org.apache.empire.commons.Options;
+import org.apache.empire.commons.StringUtils;
import org.apache.empire.data.Column;
import org.apache.empire.exceptions.InvalidArgumentException;
import org.apache.empire.exceptions.UnexpectedReturnValueException;
@@ -299,12 +300,11 @@ public class RadioInputControl extends InputControl
if (attrValue instanceof Options)
{ // Check for Options
String text = ((Options) attrValue).get(value);
- if (text != null)
- {
+ if (StringUtils.isNotEmpty(text))
return vi.getText(text);
- }
// Error
- RadioInputControl.log.error("The element '" + String.valueOf(value) + "' is not part of the supplied option list.");
+ if (value!=null)
+ RadioInputControl.log.error("The element '" + String.valueOf(value) + "' is not part of the supplied option list.");
}
}
}
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 36679ef..2a06c40 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
@@ -30,6 +30,7 @@ import javax.faces.context.FacesContext;
import org.apache.empire.commons.ObjectUtils;
import org.apache.empire.commons.OptionEntry;
import org.apache.empire.commons.Options;
+import org.apache.empire.commons.StringUtils;
import org.apache.empire.data.Column;
import org.apache.empire.exceptions.InvalidArgumentException;
import org.apache.empire.exceptions.UnexpectedReturnValueException;
@@ -320,10 +321,11 @@ public class SelectInputControl extends InputControl
if (attrValue instanceof Options)
{ // Check for Options
String text = ((Options) attrValue).get(value);
- if (text != null)
+ if (StringUtils.isNotEmpty(text))
return vi.getText(text);
// Error
- SelectInputControl.log.error("The element '" + String.valueOf(value) + "' is not part of the supplied option list.");
+ if (value!=null)
+ SelectInputControl.log.error("The element '" + String.valueOf(value) + "' is not part of the supplied option list.");
}
}
}
diff --git a/empire-db-jsf2/src/main/java/org/apache/empire/jsf2/controls/TextInputControl.java b/empire-db-jsf2/src/main/java/org/apache/empire/jsf2/controls/TextInputControl.java
index 7e52065..cdef3b1 100644
--- a/empire-db-jsf2/src/main/java/org/apache/empire/jsf2/controls/TextInputControl.java
+++ b/empire-db-jsf2/src/main/java/org/apache/empire/jsf2/controls/TextInputControl.java
@@ -238,10 +238,11 @@ public class TextInputControl extends InputControl
if (options != null && !options.isEmpty())
{ // Check for Options
String text = options.get(value);
- if (text != null)
+ if (StringUtils.isNotEmpty(text))
return vi.getText(text);
// Error
- log.error("The element '" + String.valueOf(value) + "' is not part of the supplied option list.");
+ if (value!=null)
+ TextInputControl.log.error("The element '" + String.valueOf(value) + "' is not part of the supplied option list.");
}
// Check Value
if (value == null)