fixed bug around enum/int confusion

4 files changed, 15 insertions, 13 deletions

diff --git a/cmpcj/src/de/schwarzrot/cmpc/util/MediaComparator.java b/cmpcj/src/de/schwarzrot/cmpc/util/MediaComparator.java
index 8e9ce48..a719032 100644
--- a/cmpcj/src/de/schwarzrot/cmpc/util/MediaComparator.java
+++ b/cmpcj/src/de/schwarzrot/cmpc/util/MediaComparator.java
@@ -35,6 +35,6 @@ public class MediaComparator implements Comparator<Media> {
     public int compare(Media a, Media b) {
         if (a.getType() == b.getType())
             return a.getName().compareToIgnoreCase(b.getName());
-        return a.getType() - b.getType();
+        return a.getType().ordinal() - b.getType().ordinal();
     }
 }
diff --git a/cmpcj/src/de/schwarzrot/cmpc/util/MediaForTypeMatcher.java b/cmpcj/src/de/schwarzrot/cmpc/util/MediaForTypeMatcher.java
index c3a3469..18820cf 100644
--- a/cmpcj/src/de/schwarzrot/cmpc/util/MediaForTypeMatcher.java
+++ b/cmpcj/src/de/schwarzrot/cmpc/util/MediaForTypeMatcher.java
@@ -34,7 +34,7 @@ import de.schwarzrot.cmpc.domain.Media;
 
 
 public class MediaForTypeMatcher implements Matcher<Media> {
-    public MediaForTypeMatcher(Collection<Integer> mediaTypes) {
+    public MediaForTypeMatcher(Collection<Media.SupportedMediaType> mediaTypes) {
         this.mediaTypes.addAll(mediaTypes);
     }
 
@@ -45,10 +45,10 @@ public class MediaForTypeMatcher implements Matcher<Media> {
             return false;
         if (mediaTypes.isEmpty())
             return true;
-        Integer mt = m.getType();
+        Media.SupportedMediaType mt = m.getType();
 
         return mediaTypes.contains(mt);
     }
 
-    private Set<Integer> mediaTypes = new HashSet<Integer>();
+    private Set<Media.SupportedMediaType> mediaTypes = new HashSet<Media.SupportedMediaType>();
 }
diff --git a/cmpcj/src/de/schwarzrot/cmpc/util/MediaPool2TypeList.java b/cmpcj/src/de/schwarzrot/cmpc/util/MediaPool2TypeList.java
index ce541ac..de93246 100644
--- a/cmpcj/src/de/schwarzrot/cmpc/util/MediaPool2TypeList.java
+++ b/cmpcj/src/de/schwarzrot/cmpc/util/MediaPool2TypeList.java
@@ -32,7 +32,7 @@ import ca.odell.glazedlists.event.ListEvent;
 import de.schwarzrot.cmpc.domain.Media;
 
 
-public class MediaPool2TypeList extends TransformedList<Media, Integer> {
+public class MediaPool2TypeList extends TransformedList<Media, Media.SupportedMediaType> {
     public MediaPool2TypeList(EventList<Media> source) {
         super(source);
         source.addListEventListener(this);
@@ -40,13 +40,14 @@ public class MediaPool2TypeList extends TransformedList<Media, Integer> {
 
 
     @Override
-    public Integer get(int index) {
+    public Media.SupportedMediaType get(int index) {
         Media m = source.get(index);
 
         return m.getType();
     }
 
 
+    @Override
     public void listChanged(ListEvent<Media> listChanges) {
         updates.forwardEvent(listChanges);
     }
diff --git a/cmpcj/src/de/schwarzrot/cmpc/util/MediaTypeSelect.java b/cmpcj/src/de/schwarzrot/cmpc/util/MediaTypeSelect.java
index 9ba948e..033ee53 100644
--- a/cmpcj/src/de/schwarzrot/cmpc/util/MediaTypeSelect.java
+++ b/cmpcj/src/de/schwarzrot/cmpc/util/MediaTypeSelect.java
@@ -67,7 +67,7 @@ public class MediaTypeSelect extends AbstractMatcherEditor<Media> implements Lis
                 setBackground(list.getBackground());
                 setForeground(list.getForeground());
             }
-            ImageIcon icon = images[(Integer) value];
+            ImageIcon icon = images[((Media.SupportedMediaType) value).ordinal()];
             setSize(110, 110);
             setHorizontalAlignment(JLabel.CENTER);
 
@@ -105,13 +105,14 @@ public class MediaTypeSelect extends AbstractMatcherEditor<Media> implements Lis
 
 
     public MediaTypeSelect(EventList<Media> source) {
-        EventList<Integer> mediaTypeNonUnique = new MediaPool2TypeList(source);
+        EventList<Media.SupportedMediaType> mediaTypeNonUnique = new MediaPool2TypeList(source);
 
-        mediaTypeList = new UniqueList<Integer>(mediaTypeNonUnique);
-        DefaultEventListModel<Integer> mediaTypeListModel = new DefaultEventListModel<Integer>(mediaTypeList);
+        mediaTypeList = new UniqueList<Media.SupportedMediaType>(mediaTypeNonUnique);
+        DefaultEventListModel<Media.SupportedMediaType> mediaTypeListModel = new DefaultEventListModel<Media.SupportedMediaType>(
+                mediaTypeList);
 
         mediaTypeJList = new JList(mediaTypeListModel);
-        DefaultEventSelectionModel<Integer> mediaTypeSelectionModel = new DefaultEventSelectionModel<Integer>(
+        DefaultEventSelectionModel<Media.SupportedMediaType> mediaTypeSelectionModel = new DefaultEventSelectionModel<Media.SupportedMediaType>(
                 mediaTypeList);
 
         mediaTypeJList.setSelectionModel(mediaTypeSelectionModel);
@@ -134,8 +135,8 @@ public class MediaTypeSelect extends AbstractMatcherEditor<Media> implements Lis
         fireChanged(newMatcher);
     }
 
-    private EventList<Integer> mediaTypeList;
-    private EventList<Integer> selectedTypes;
+    private EventList<Media.SupportedMediaType> mediaTypeList;
+    private EventList<Media.SupportedMediaType> selectedTypes;
     private JList mediaTypeJList;
     private static final String[] iconFiles;
     static {