You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by sc...@apache.org on 2005/03/23 00:17:24 UTC
svn commit: r158694 - in jakarta/commons/proper/collections/trunk:
RELEASE-NOTES.html project.xml
src/java/org/apache/commons/collections/BeanMap.java
Author: scolebourne
Date: Tue Mar 22 15:17:21 2005
New Revision: 158694
URL: http://svn.apache.org/viewcvs?view=rev&rev=158694
Log:
Fix internal variable to not include non-existant write methods
bug 15895, from Dimiter Dimitrov
Modified:
jakarta/commons/proper/collections/trunk/RELEASE-NOTES.html
jakarta/commons/proper/collections/trunk/project.xml
jakarta/commons/proper/collections/trunk/src/java/org/apache/commons/collections/BeanMap.java
Modified: jakarta/commons/proper/collections/trunk/RELEASE-NOTES.html
URL: http://svn.apache.org/viewcvs/jakarta/commons/proper/collections/trunk/RELEASE-NOTES.html?view=diff&r1=158693&r2=158694
==============================================================================
--- jakarta/commons/proper/collections/trunk/RELEASE-NOTES.html (original)
+++ jakarta/commons/proper/collections/trunk/RELEASE-NOTES.html Tue Mar 22 15:17:21 2005
@@ -58,6 +58,7 @@
<li>MultiHashMap.remove(key, item) - Was returning the item even when nothing was removed [32366]</li>
<li>SetUniqueList.set(int,Object) - Destroyed set status in certain circumstances [33294]</li>
<li>AbstractLinkedMap.init() - Now calls createEntry() to create the map entry object [33706]</li>
+<li>BeanMap.initialize() - Internal variable now correctly initialised with only write methods that actually exist [15895]</li>
</ul>
<center><h3>JAVADOC</h3></center>
Modified: jakarta/commons/proper/collections/trunk/project.xml
URL: http://svn.apache.org/viewcvs/jakarta/commons/proper/collections/trunk/project.xml?view=diff&r1=158693&r2=158694
==============================================================================
--- jakarta/commons/proper/collections/trunk/project.xml (original)
+++ jakarta/commons/proper/collections/trunk/project.xml Tue Mar 22 15:17:21 2005
@@ -168,6 +168,9 @@
<name>Eric Crampton</name>
</contributor>
<contributor>
+ <name>Dimiter Dimitrov</name>
+ </contributor>
+ <contributor>
<name>Peter Donald</name>
</contributor>
<contributor>
Modified: jakarta/commons/proper/collections/trunk/src/java/org/apache/commons/collections/BeanMap.java
URL: http://svn.apache.org/viewcvs/jakarta/commons/proper/collections/trunk/src/java/org/apache/commons/collections/BeanMap.java?view=diff&r1=158693&r2=158694
==============================================================================
--- jakarta/commons/proper/collections/trunk/src/java/org/apache/commons/collections/BeanMap.java (original)
+++ jakarta/commons/proper/collections/trunk/src/java/org/apache/commons/collections/BeanMap.java Tue Mar 22 15:17:21 2005
@@ -1,5 +1,5 @@
/*
- * Copyright 2001-2004 The Apache Software Foundation
+ * Copyright 2001-2005 The Apache Software Foundation
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@@ -46,6 +46,7 @@
*
* @author James Strachan
* @author Stephen Colebourne
+ * @author Dimiter Dimitrov
*/
public class BeanMap extends AbstractMap implements Cloneable {
@@ -611,7 +612,7 @@
if ( readMethod != null ) {
readMethods.put( name, readMethod );
}
- if ( writeMethods != null ) {
+ if ( writeMethod != null ) {
writeMethods.put( name, writeMethod );
}
types.put( name, aType );
---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org