[jboss-cvs] jboss-seam/src/ui/org/jboss/seam/ui ...

[Gavin King]

  User: gavin   
  Date: 07/01/31 13:30:58

  Modified:    src/ui/org/jboss/seam/ui  UISelectItems.java
  Log:
  use toString() method instead of typecase
  
  Revision  Changes    Path
  1.4       +6 -4      jboss-seam/src/ui/org/jboss/seam/ui/UISelectItems.java
  
  (In the diff below, changes in quantity of whitespace are not shown.)
  
  Index: UISelectItems.java
  ===================================================================
  RCS file: /cvsroot/jboss/jboss-seam/src/ui/org/jboss/seam/ui/UISelectItems.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -b -r1.3 -r1.4
  --- UISelectItems.java	30 Jan 2007 09:01:33 -0000	1.3
  +++ UISelectItems.java	31 Jan 2007 18:30:58 -0000	1.4
  @@ -122,16 +122,18 @@
   	}
   	
   	private String getString(String localName) {
  -		if (getValueBinding(localName) == null) {
  +		ValueBinding vb = getValueBinding(localName);
  +      if (vb == null) {
   			return null;
   		} else {
  -			return (String) getValueBinding(localName).getValue(getFacesContext());
  +			return vb.getValue(getFacesContext()).toString();
   		}
   	}
   	
   	private Boolean getBoolean(String localName) {
  -		if (getString(localName) != null) {
  -			return Boolean.valueOf(getString(localName));
  +		String string = getString(localName);
  +      if (string != null) {
  +			return Boolean.valueOf(string);
   		} else {
   			return null;
   		}