Fix some problems brought from merge conflicts

.github/workflows/pullrequest.yml

@@ -19,6 +19,8 @@ jobs:
         uses: actions/setup-java@v1
         with:
           java-version: 1.8
+      - name: submodules-init
+        uses: snickerbockers/submodules-init@v4
       - name: Build with Maven
         run: mvn -B package
       - name: Archive artifacts

connector/src/main/java/org/geysermc/connector/network/session/GeyserSession.java

@@ -165,9 +165,9 @@ public class GeyserSession implements CommandSender {
 
         ChunkUtils.sendEmptyChunks(this, playerEntity.getPosition().toInt(), 0, false);
 
-        BiomeDefinitionListPacket biomePacket = new BiomeDefinitionListPacket();
+        BiomeDefinitionListPacket biomeDefinitionListPacket = new BiomeDefinitionListPacket();
-        biomePacket.setTag(CompoundTag.EMPTY);
+        biomeDefinitionListPacket.setTag(Toolbox.BIOMES);
-        upstream.sendPacket(biomePacket);
+        upstream.sendPacket(biomeDefinitionListPacket);
 
         AvailableEntityIdentifiersPacket entityPacket = new AvailableEntityIdentifiersPacket();
         entityPacket.setTag(CompoundTag.EMPTY);

connector/src/main/java/org/geysermc/connector/utils/MessageUtils.java

@@ -157,14 +157,14 @@ public class MessageUtils {
                 base += "r";
                 break;
             default:
-                return "";
+                break;
         }
 
         return base;
     }
 
     private static String getFormat(List<ChatFormat> formats) {
-        String str = "";
+        StringBuilder str = new StringBuilder();
         for (ChatFormat cf : formats) {
             String base = "\u00a7";
             switch (cf) {
@@ -187,10 +187,10 @@ public class MessageUtils {
                     break;
             }
 
-            str += base;
+            str.append(base);
         }
 
-        return str;
+        return str.toString();
     }
 
     public static boolean isMessage(String text) {