Update of 
/var/cvs/applications/taglib/src/org/mmbase/bridge/jsp/taglib/typehandler
In directory james.mmbase.org:/tmp/cvs-serv31593

Modified Files:
        AbstractTypeHandler.java StringHandler.java 
Log Message:
  MMB-1607


See also: 
http://cvs.mmbase.org/viewcvs/applications/taglib/src/org/mmbase/bridge/jsp/taglib/typehandler
See also: http://www.mmbase.org/jira/browse/MMB-1607


Index: AbstractTypeHandler.java
===================================================================
RCS file: 
/var/cvs/applications/taglib/src/org/mmbase/bridge/jsp/taglib/typehandler/AbstractTypeHandler.java,v
retrieving revision 1.59
retrieving revision 1.60
diff -u -b -r1.59 -r1.60
--- AbstractTypeHandler.java    23 Feb 2008 16:00:44 -0000      1.59
+++ AbstractTypeHandler.java    25 Feb 2008 16:07:46 -0000      1.60
@@ -29,7 +29,7 @@
  * @author Gerard van de Looi
  * @author Michiel Meeuwissen
  * @since  MMBase-1.6
- * @version $Id: AbstractTypeHandler.java,v 1.59 2008/02/23 16:00:44 michiel 
Exp $
+ * @version $Id: AbstractTypeHandler.java,v 1.60 2008/02/25 16:07:46 michiel 
Exp $
  */
 
 public abstract class AbstractTypeHandler implements TypeHandler {
@@ -229,9 +229,9 @@
                             log.debug("Setting " + fieldName + " to " + 
fieldValue);
                         }
                         if ("".equals(fieldValue) && 
interpretEmptyAsNull(field)) {
-                            node.setValue(fieldName,  null);
+                            setValue(node, fieldName,  null);
                         } else {
-                            node.setValue(fieldName,  fieldValue);
+                            setValue(node, fieldName,  fieldValue);
                         }
                     } catch (Throwable t) {
                         // may throw exception like 'You cannot change the 
field"
@@ -270,6 +270,13 @@
         }
     }
 
+
+    /**
+     * @since MMBase-1.8.6
+     */
+    protected void setValue(Node node, String fieldName, Object value) {
+        node.setValue(fieldName, value);
+    }
     /**
      * @see TypeHandler#useHtmlInput(Node, Field)
      */
@@ -281,9 +288,9 @@
             return false;
         }  else {
             if ("".equals(fieldValue) && interpretEmptyAsNull(field)) {
-                node.setValue(fieldName,  null);
+                setValue(node, fieldName,  null);
             } else {
-                node.setValue(fieldName,  fieldValue);
+                setValue(node, fieldName,  fieldValue);
             }
             return true;
         }


Index: StringHandler.java
===================================================================
RCS file: 
/var/cvs/applications/taglib/src/org/mmbase/bridge/jsp/taglib/typehandler/StringHandler.java,v
retrieving revision 1.65
retrieving revision 1.66
diff -u -b -r1.65 -r1.66
--- StringHandler.java  21 Sep 2007 12:53:43 -0000      1.65
+++ StringHandler.java  25 Feb 2008 16:07:46 -0000      1.66
@@ -29,7 +29,7 @@
  * @author Gerard van de Looi
  * @author Michiel Meeuwissen
  * @since  MMBase-1.6
- * @version $Id: StringHandler.java,v 1.65 2007/09/21 12:53:43 michiel Exp $
+ * @version $Id: StringHandler.java,v 1.66 2008/02/25 16:07:46 michiel Exp $
  */
 
 public class StringHandler extends AbstractTypeHandler {
@@ -118,6 +118,11 @@
         }
     }
 
+
+    protected void setValue(Node node, String fieldName, Object value) {
+        node.setStringValue(fieldName, 
org.mmbase.util.Casting.toString(value));
+    }
+
     /**
      * @see TypeHandler#useHtmlInput(Node, Field)
      */
@@ -136,7 +141,7 @@
             String fieldName = field.getName();
             if (! fieldValue.equals(node.getValue(fieldName))) {
                 if (fieldValue.length() == 0 && node.getValue(fieldName) == 
null) return false;
-                node.setStringValue(fieldName,  fieldValue);
+                setValue(node, fieldName,  fieldValue);
                 return true;
             }
         }
_______________________________________________
Cvs mailing list
[email protected]
http://lists.mmbase.org/mailman/listinfo/cvs

Reply via email to