do not return null when set is expected

NucleoTeam · Nov 8, 2023 · c9a4fde · c9a4fde
1 parent bd3e2d8
commit c9a4fde
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 9 deletions.
diff --git a/buildSrc/src/main/groovy/nucleodb.library.gradle b/buildSrc/src/main/groovy/nucleodb.library.gradle
@@ -3,7 +3,7 @@ plugins {
 }
 
 group = 'com.nucleodb'
-version = '1.9.14'
+version = '1.9.15'
 
 repositories {
     mavenCentral()

diff --git a/library/src/main/java/com/nucleocore/library/database/tables/ConnectionHandler.java b/library/src/main/java/com/nucleocore/library/database/tables/ConnectionHandler.java
@@ -148,51 +148,51 @@ public Set<Connection> getByFrom(DataEntry de){
     if(tmp!=null){
       return tmp.stream().map(c->c.clone()).collect(Collectors.toSet());
     }
-    return null;
+    return new TreeSetExt<>();
   }
 
   public Set<Connection> getByFromAndLabel(DataEntry from, String label){
     Set<Connection> tmp = connections.get(from.getKey()+label);
     if(tmp!=null) {
       return tmp.stream().map(c->c.clone()).collect(Collectors.toSet());
     }
-    return null;
+    return new TreeSetExt<>();
   }
 
   public Set<Connection> getByFromAndLabelAndTo(DataEntry from, String label, DataEntry to){
     Set<Connection> tmp = connections.get(from.getKey()+to.getKey()+label);
     if(tmp!=null) {
       return tmp.stream().map(c->c.clone()).collect(Collectors.toSet());
     }
-    return null;
+    return new TreeSetExt<>();
   }
   public Set<Connection> getByFromAndTo(DataEntry from, DataEntry to){
     Set<Connection> tmp = connections.get(from.getKey()+to.getKey());
     if(tmp!=null) {
       return tmp.stream().map(c->c.clone()).collect(Collectors.toSet());
     }
-    return null;
+    return new TreeSetExt<>();
   }
   public Set<Connection> getReverseByLabelAndTo(String label, DataEntry to){
     Set<Connection> tmp = connectionsReverse.get(to.getKey()+label);
     if(tmp!=null) {
       return tmp.stream().map(c->c.clone()).collect(Collectors.toSet());
     }
-    return null;
+    return new TreeSetExt<>();
   }
   public Set<Connection> getReverseByFromAndLabelAndTo(DataEntry de, String label, DataEntry toDe){
     Set<Connection> tmp = connectionsReverse.get(de.getKey()+toDe.getKey()+label);
     if(tmp!=null) {
       return tmp.stream().map(c->c.clone()).collect(Collectors.toSet());
     }
-    return null;
+    return new TreeSetExt<>();
   }
   public Set<Connection> getReverseByFromAndTo(DataEntry from, DataEntry to){
     Set<Connection> tmp = connectionsReverse.get(from.getKey()+to.getKey());
     if(tmp!=null) {
       return tmp.stream().map(c->c.clone()).collect(Collectors.toSet());
     }
-    return null;
+    return new TreeSetExt<>();
   }
 
   private void putConnectionInKey(String key, Connection connection){

diff --git a/library/src/main/java/com/nucleocore/library/database/tables/DataTable.java b/library/src/main/java/com/nucleocore/library/database/tables/DataTable.java
@@ -234,7 +234,7 @@ public Set<DataEntry> createNewObject(Set<DataEntry> o) {
     } catch (JsonProcessingException e) {
       e.printStackTrace();
     }
-    return null;
+    return new TreeSetExt<>();
   }
 
   DataEntry createNewObject(DataEntry o) {