Update of /var/cvs/src/org/mmbase/datatypes/processors In directory james.mmbase.org:/tmp/cvs-serv11236/processors
Modified Files: Age.java ChecksumProcessorFactory.java Log Message: logging only See also: http://cvs.mmbase.org/viewcvs/src/org/mmbase/datatypes/processors Index: Age.java =================================================================== RCS file: /var/cvs/src/org/mmbase/datatypes/processors/Age.java,v retrieving revision 1.6 retrieving revision 1.7 diff -u -b -r1.6 -r1.7 --- Age.java 4 Mar 2009 11:32:09 -0000 1.6 +++ Age.java 30 Apr 2009 14:40:59 -0000 1.7 @@ -10,6 +10,7 @@ package org.mmbase.datatypes.processors; import org.mmbase.bridge.*; +import org.mmbase.bridge.util.*; import org.mmbase.util.*; import java.util.*; import org.mmbase.util.logging.*; @@ -39,8 +40,10 @@ try { // educated guess for the birth date: Date date = DynamicDate.eval(DynamicDate.getInstance("today - 6 month - " + value + " year")); - log.debug("setting age to " + value + " -> " + date); node.setValueWithoutProcess(birthdateField, date); + if (log.isDebugEnabled()) { + log.debug("setting age to " + value + " in " + birthdateField + " -> " + date + " -> " + new NodeMap(node)); + } } catch (org.mmbase.util.dateparser.ParseException pe) { log.warn(pe); } @@ -61,7 +64,9 @@ Date birthDate = node.getDateValue(birthdateField); Date now = new Date(); int age = (int) Math.floor((double) (now.getTime() - birthDate.getTime()) / (1000 * 3600 * 24 * 365.25)); - log.debug("getting age for " + birthDate + " --> " + age); + if (log.isDebugEnabled()) { + log.debug("getting age for " + birthDate + " --> " + age + " from " + new NodeMap(node)); + } return Casting.toType(value.getClass(), age); } } Index: ChecksumProcessorFactory.java =================================================================== RCS file: /var/cvs/src/org/mmbase/datatypes/processors/ChecksumProcessorFactory.java,v retrieving revision 1.8 retrieving revision 1.9 diff -u -b -r1.8 -r1.9 --- ChecksumProcessorFactory.java 24 Sep 2008 20:26:08 -0000 1.8 +++ ChecksumProcessorFactory.java 30 Apr 2009 14:40:59 -0000 1.9 @@ -12,6 +12,8 @@ import org.mmbase.util.transformers.*; import org.mmbase.util.functions.*; import java.io.StringWriter; +import org.mmbase.util.logging.*; + /** * See {...@link org.mmbase.util.transformers.ChecksumFactory}. This produces CommitProcessors meant @@ -19,12 +21,14 @@ * Checksum 'processor', and the field for which this field is a checksum. * * @author Michiel Meeuwissen - * @version $Id: ChecksumProcessorFactory.java,v 1.8 2008/09/24 20:26:08 michiel Exp $ + * @version $Id: ChecksumProcessorFactory.java,v 1.9 2009/04/30 14:40:59 michiel Exp $ * @since MMBase-1.8 */ public class ChecksumProcessorFactory implements ParameterizedCommitProcessorFactory, java.io.Serializable { + private static final Logger log = Logging.getLoggerInstance(ChecksumProcessorFactory.class); + private static final long serialVersionUID = 1L; protected static final Parameter[] PARAMS = new Parameter[] { @@ -47,14 +51,23 @@ if (!field.isVirtual()) { if (node.getChanged().contains(sourceField)) { if (node.isNull(sourceField)) { + log.debug("Source field is null"); // set checksum null too. node.setValue(field.getName(), null); return; } StringWriter writer = new StringWriter(); transformer.transform(node.getInputStreamValue(sourceField), writer); - node.setStringValue(field.getName(), writer.toString()); + String checksum = writer.toString(); + if (log.isDebugEnabled()) { + log.debug("Setting checksum field '" + field.getName() + "' to " + checksum); + } + node.setStringValue(field.getName(), checksum); + } else { + log.debug("Ignoring because '" + sourceField + "' is not changed"); } + } else { + log.debug("Ignoring because '" + field + "' is virtual"); } } public String toString() { _______________________________________________ Cvs mailing list Cvs@lists.mmbase.org http://lists.mmbase.org/mailman/listinfo/cvs