[fix] wrong "equals" implementation

crazywenza · Jun 16, 2017 · abb47a0 · abb47a0
1 parent 3812ce4
commit abb47a0
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/matisse/src/main/java/com/zhihu/matisse/internal/entity/Item.java b/matisse/src/main/java/com/zhihu/matisse/internal/entity/Item.java
@@ -132,7 +132,13 @@ public boolean equals(Object obj) {
         }
 
         Item other = (Item) obj;
-        return other.uri.equals(uri);
+        return id == other.id
+                && (mimeType != null && mimeType.equals(other.mimeType)
+                    || (mimeType == null && other.mimeType == null))
+                && (uri != null && uri.equals(other.uri)
+                    || (uri == null && other.uri == null))
+                && size == other.size
+                && duration == other.duration;
     }
 
     @Override
@@ -145,4 +151,4 @@ public int hashCode() {
         result = 31 * result + Long.valueOf(duration).hashCode();
         return result;
     }
-}
+}