commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From skitch...@apache.org
Subject cvs commit: jakarta-commons/digester/src/java/org/apache/commons/digester SetPropertyRule.java
Date Tue, 30 Nov 2004 03:06:27 GMT
skitching    2004/11/29 19:06:27

  Modified:    digester/src/java/org/apache/commons/digester
                        SetPropertyRule.java
  Log:
  Added comment only (re PropertyUtils.isWriteable).
  
  Revision  Changes    Path
  1.19      +4 -1      jakarta-commons/digester/src/java/org/apache/commons/digester/SetPropertyRule.java
  
  Index: SetPropertyRule.java
  ===================================================================
  RCS file: /home/cvs/jakarta-commons/digester/src/java/org/apache/commons/digester/SetPropertyRule.java,v
  retrieving revision 1.18
  retrieving revision 1.19
  diff -u -r1.18 -r1.19
  --- SetPropertyRule.java	10 May 2004 06:30:06 -0000	1.18
  +++ SetPropertyRule.java	30 Nov 2004 03:06:27 -0000	1.19
  @@ -130,6 +130,9 @@
   
           // Force an exception if the property does not exist
           // (BeanUtils.setProperty() silently returns in this case)
  +        //
  +        // This code should probably use PropertyUtils.isWriteable(), 
  +        // like SetPropertiesRule does.
           if (top instanceof DynaBean) {
               DynaProperty desc =
                   ((DynaBean) top).getDynaClass().getDynaProperty(actualName);
  
  
  

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


Mime
View raw message