You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by rd...@apache.org on 2003/07/01 21:09:49 UTC

cvs commit: jakarta-commons/betwixt/src/java/org/apache/commons/betwixt/digester XMLIntrospectorHelper.java

rdonkin     2003/07/01 12:09:49

  Modified:    betwixt/src/java/org/apache/commons/betwixt/digester
                        XMLIntrospectorHelper.java
  Log:
  Added fix for bug in reading nested looping elements.
  
  Revision  Changes    Path
  1.22      +9 -1      jakarta-commons/betwixt/src/java/org/apache/commons/betwixt/digester/XMLIntrospectorHelper.java
  
  Index: XMLIntrospectorHelper.java
  ===================================================================
  RCS file: /home/cvs/jakarta-commons/betwixt/src/java/org/apache/commons/betwixt/digester/XMLIntrospectorHelper.java,v
  retrieving revision 1.21
  retrieving revision 1.22
  diff -u -r1.21 -r1.22
  --- XMLIntrospectorHelper.java	5 Jun 2003 20:59:16 -0000	1.21
  +++ XMLIntrospectorHelper.java	1 Jul 2003 19:09:49 -0000	1.22
  @@ -185,11 +185,14 @@
           } else if ( isLoopType( type ) ) {
               if (log.isTraceEnabled()) {
                   log.trace("Loop type: " + name);
  +                log.trace("Wrap in collections? " + introspector.isWrapCollectionsInElement());
               }
               ElementDescriptor loopDescriptor = new ElementDescriptor();
               loopDescriptor.setContextExpression(
                   new IteratorExpression( new MethodExpression( readMethod ) )
               );
  +            loopDescriptor.setWrapCollectionsInElement(
  +                        introspector.isWrapCollectionsInElement());
               // XXX: need to support some kind of 'add' or handle arrays, Lists or indexed properties
               //loopDescriptor.setUpdater( new MethodUpdater( writeMethod ) );
               if ( Map.class.isAssignableFrom( type ) ) {
  @@ -233,6 +236,11 @@
           // XXX: associate more bean information with the descriptor?
           //nodeDescriptor.setDisplayName( propertyDescriptor.getDisplayName() );
           //nodeDescriptor.setShortDescription( propertyDescriptor.getShortDescription() );
  +        
  +        if (log.isTraceEnabled()) {
  +            log.trace("Created descriptor:");
  +            log.trace(nodeDescriptor);
  +        }
           return nodeDescriptor;
       }
       
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org