Libraries: fixed incorrect handling of null types

Author: Federico Fissore

app/src/cc/arduino/contributions/libraries/LibraryByTypeComparator.java

@@ -16,6 +16,12 @@ public LibraryByTypeComparator(LibraryTypeComparator libraryTypeComparator) {
 
   @Override
   public int compare(ContributedLibrary o1, ContributedLibrary o2) {
+    if (o1.getTypes() == null) {
+      return 1;
+    }
+    if (o2.getTypes() == null) {
+      return -1;
+    }
     return libraryTypeComparator.compare(o1.getTypes().get(0), o2.getTypes().get(0));
   }
 

app/src/cc/arduino/contributions/libraries/LibraryOfSameTypeComparator.java

@@ -6,6 +6,12 @@ public class LibraryOfSameTypeComparator implements Comparator<ContributedLibrar
 
   @Override
   public int compare(ContributedLibrary o1, ContributedLibrary o2) {
+    if (o1.getTypes() == null) {
+      return 1;
+    }
+    if (o2.getTypes() == null) {
+      return -1;
+    }
     if (!o1.getTypes().get(0).equals(o2.getTypes().get(0))) {
       return 0;
     }

arduino-core/src/processing/app/packages/LegacyUserLibrary.java

@@ -32,8 +32,6 @@
 import java.util.Arrays;
 import java.util.List;
 
-import cc.arduino.contributions.libraries.ContributedLibraryReference;
-
 public class LegacyUserLibrary extends UserLibrary {
 
   private String name;
@@ -45,6 +43,8 @@ public static LegacyUserLibrary create(File libFolder) {
     res.setInstalled(true);
     res.layout = LibraryLayout.FLAT;
     res.name = libFolder.getName();
+    res.setTypes(Arrays.asList("Contributed"));
+    res.setCategory("Uncategorized");
     return res;
   }
 
@@ -58,66 +58,6 @@ public List<String> getArchitectures() {
     return Arrays.asList("*");
   }
 
-  @Override
-  public String getAuthor() {
-    return null;
-  }
-
-  @Override
-  public String getParagraph() {
-    return null;
-  }
-
-  @Override
-  public String getSentence() {
-    return null;
-  }
-
-  @Override
-  public String getWebsite() {
-    return null;
-  }
-
-  @Override
-  public String getCategory() {
-    return "Uncategorized";
-  }
-
-  @Override
-  public String getLicense() {
-    return null;
-  }
-
-  @Override
-  public String getVersion() {
-    return null;
-  }
-
-  @Override
-  public String getMaintainer() {
-    return null;
-  }
-
-  @Override
-  public String getChecksum() {
-    return null;
-  }
-
-  @Override
-  public long getSize() {
-    return 0;
-  }
-
-  @Override
-  public String getUrl() {
-    return null;
-  }
-
-  @Override
-  public List<ContributedLibraryReference> getRequires() {
-    return null;
-  }
-
   @Override
   public String toString() {
     return "LegacyLibrary:" + name + "\n";