diff options
author | Daniel Bowman <daniel@danielbowman.co.uk> | 2018-10-27 14:46:48 +0100 |
---|---|---|
committer | Daniel Bowman <daniel@danielbowman.co.uk> | 2018-10-27 14:46:48 +0100 |
commit | ac260b65765e55910517d1d468927936d1688d04 (patch) | |
tree | 0b81db9e002ba971970f68af1473e03f81a7a716 /app/src/main/java | |
parent | 0a4f6215972d970e69859fa690938ef9f0ea2e0a (diff) | |
download | dsub-ac260b65765e55910517d1d468927936d1688d04.tar.gz dsub-ac260b65765e55910517d1d468927936d1688d04.tar.bz2 dsub-ac260b65765e55910517d1d468927936d1688d04.zip |
Fix Genre comparator to pass unit tests
Diffstat (limited to 'app/src/main/java')
-rw-r--r-- | app/src/main/java/github/daneren2005/dsub/domain/Genre.java | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/app/src/main/java/github/daneren2005/dsub/domain/Genre.java b/app/src/main/java/github/daneren2005/dsub/domain/Genre.java index 4b6ac344..dcf12a1d 100644 --- a/app/src/main/java/github/daneren2005/dsub/domain/Genre.java +++ b/app/src/main/java/github/daneren2005/dsub/domain/Genre.java @@ -1,16 +1,10 @@ package github.daneren2005.dsub.domain; -import android.content.Context; -import android.content.SharedPreferences; - import java.io.Serializable; import java.util.Collections; import java.util.Comparator; import java.util.List; -import github.daneren2005.dsub.util.Constants; -import github.daneren2005.dsub.util.Util; - public class Genre implements Serializable { private String name; private String index; @@ -57,7 +51,10 @@ public class Genre implements Serializable { public static class GenreComparator implements Comparator<Genre> { @Override public int compare(Genre genre1, Genre genre2) { - return genre1.getName().compareToIgnoreCase(genre2.getName()); + String genre1Name = genre1.getName() != null ? genre1.getName() : ""; + String genre2Name = genre2.getName() != null ? genre2.getName() : ""; + + return genre1Name.compareToIgnoreCase(genre2Name); } public static List<Genre> sort(List<Genre> genres) { |