Author: kono
Date: 2011-03-24 14:24:07 -0700 (Thu, 24 Mar 2011)
New Revision: 24585

Modified:
   
core3/default-mappingcalculators/trunk/src/main/java/org/cytoscape/view/vizmap/mappings/ContinuousMappingFactory.java
Log:
Fixed condition to check the compatible data type for continuous mapping.

Modified: 
core3/default-mappingcalculators/trunk/src/main/java/org/cytoscape/view/vizmap/mappings/ContinuousMappingFactory.java
===================================================================
--- 
core3/default-mappingcalculators/trunk/src/main/java/org/cytoscape/view/vizmap/mappings/ContinuousMappingFactory.java
       2011-03-24 21:22:29 UTC (rev 24584)
+++ 
core3/default-mappingcalculators/trunk/src/main/java/org/cytoscape/view/vizmap/mappings/ContinuousMappingFactory.java
       2011-03-24 21:24:07 UTC (rev 24585)
@@ -3,15 +3,21 @@
 import org.cytoscape.view.model.VisualProperty;
 import org.cytoscape.view.vizmap.VisualMappingFunction;
 import org.cytoscape.view.vizmap.VisualMappingFunctionFactory;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public class ContinuousMappingFactory implements VisualMappingFunctionFactory {
+       
+       private static final Logger logger = 
LoggerFactory.getLogger(ContinuousMappingFactory.class);
 
        @Override
        public <K, V> VisualMappingFunction<K, V> 
createVisualMappingFunction(final String attributeName, 
                        Class<K> attrValueType, VisualProperty<V> vp) {
                
+               logger.debug("Trying to create Continuous mapping.  Data Type 
is " + attrValueType);
+               
                // Validate attribute type: Continuous Mapping is compatible 
with Numbers only.
-               if(attrValueType.isAssignableFrom(Number.class) == false)
+               if(Number.class.isAssignableFrom(attrValueType) == false)
                        throw new IllegalArgumentException("ContinuousMapping 
can be used for numerical attributes only.");
                
                return new ContinuousMapping<K, V>(attributeName, 
attrValueType,  vp);

-- 
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