Author: ruschein
Date: 2010-12-08 15:28:38 -0800 (Wed, 08 Dec 2010)
New Revision: 23138

Modified:
   
core3/model-impl/trunk/impl/src/main/java/org/cytoscape/model/internal/CyTableImpl.java
Log:
Fixed a potential bug and added another null-pointer check.

Modified: 
core3/model-impl/trunk/impl/src/main/java/org/cytoscape/model/internal/CyTableImpl.java
===================================================================
--- 
core3/model-impl/trunk/impl/src/main/java/org/cytoscape/model/internal/CyTableImpl.java
     2010-12-08 23:27:39 UTC (rev 23137)
+++ 
core3/model-impl/trunk/impl/src/main/java/org/cytoscape/model/internal/CyTableImpl.java
     2010-12-08 23:28:38 UTC (rev 23138)
@@ -128,6 +128,7 @@
                // so that we don't fire an event.
                types.put(primaryKey, primaryKeyType);
                attributes.put(primaryKey, new HashMap<Object, Object>());
+               reverse.put(primaryKey, new HashMap<Object, Set<Object>>());
        }
 
        /**
@@ -358,6 +359,8 @@
                ++counter;
                if (columnName == null)
                        throw new NullPointerException("columnName must not be 
null!");
+               if (value == null)
+                       throw new NullPointerException("value must not be 
null!");
 
                final Class<?> columnType = types.get(columnName);
                if (columnType == null || !attributes.containsKey(columnName))

-- 
You received this message because you are subscribed to the Google Groups 
"cytoscape-cvs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/cytoscape-cvs?hl=en.

Reply via email to