diff --git a/webapp/src/edu/cornell/mannlib/vitro/webapp/beans/DataPropertyComparator.java b/webapp/src/edu/cornell/mannlib/vitro/webapp/beans/DataPropertyComparator.java
index da32fc61c..de7a5b280 100644
--- a/webapp/src/edu/cornell/mannlib/vitro/webapp/beans/DataPropertyComparator.java
+++ b/webapp/src/edu/cornell/mannlib/vitro/webapp/beans/DataPropertyComparator.java
@@ -20,7 +20,9 @@ public class DataPropertyComparator implements Comparator<Individual> {
 
         int result;
         
-        // This pushes null values to the end of the list
+        // Push null values to the end of the list.
+        // Is this generally what's wanted? Or should this class be 
+        // NullLastDataPropertyComparator?
         if (dps1 == null) {
             result = (dps2 == null) ? 0 : 1;
         } else if (dps2 == null) {