Merge pull request #128 from BentoBoxWorld/develop

Release 1.9.4

Author: tastybento

pom.xml

@@ -47,7 +47,7 @@
         <!-- Revision variable removes warning about dynamic version -->
         <revision>${build.version}-SNAPSHOT</revision>
         <!-- This allows to change between versions and snapshots. -->
-        <build.version>1.9.3</build.version>
+        <build.version>1.9.4</build.version>
         <build.number>-LOCAL</build.number>
         <sonar.projectKey>BentoBoxWorld_Greenhouses</sonar.projectKey>
         <sonar.organization>bentobox-world</sonar.organization>

src/main/java/world/bentobox/greenhouses/greenhouse/BiomeRecipe.java

@@ -153,7 +153,7 @@ public boolean addMobs(EntityType mobType, double mobProbability, Material mobSp
             mobTree.put(lastProb + probability, new GreenhouseMob(mobType, mobSpawnOn, customizer));
             return true;
         } else {
-            addon.logError("Mob chances add up to > 100% in " + type + " biome recipe! Skipping " + mobType);
+            addon.logError("Mob chances add up to > 100% in " + type.name() + " biome recipe! Skipping " + mobType);
             return false;
         }
     }
@@ -179,7 +179,7 @@ public boolean addPlants(Material plantMaterial, double plantProbability, Materi
             // Add to probability tree
             map.put(lastProb + probability, new GreenhousePlant(plantMaterial, plantGrowOn));
         } else {
-            addon.logError("Plant chances add up to > 100% in " + type + " biome recipe! Skipping "
+            addon.logError("Plant chances add up to > 100% in " + type.name() + " biome recipe! Skipping "
                     + plantMaterial.toString());
             return false;
         }
@@ -403,8 +403,9 @@ public boolean spawnMob(Block b) {
         return getRandomMob()
                 // Check if the spawn on block matches, if it exists
                 .filter(m -> Optional.of(m.mobSpawnOn())
-                        .map(b.getRelative(BlockFace.DOWN).getType()::equals)
+                        .map(getEffectiveBlock(b.getRelative(BlockFace.DOWN))::equals)
                         .orElse(true))
+
                 // If spawn occurs, check if it can fit inside greenhouse
                 .map(m -> {
                     Entity entity = b.getWorld().spawnEntity(spawnLoc, m.mobType());
@@ -425,6 +426,19 @@ public boolean spawnMob(Block b) {
                 }).orElse(false);
     }
 
+    /**
+     * Get the effective block material, allowing bubble columns to act as water.
+     * @param b - block to check
+     * @return the block
+     */
+    private Material getEffectiveBlock(Block b) {
+        // Treat Bubble Column as water
+        if (b.getType() == Material.BUBBLE_COLUMN) {
+            return Material.WATER;
+        }
+        return b.getType();
+    }
+
     /**
      * Prevent hoglins and piglins from zombifying if they spawn in the overworld
      * @param entity - spawned entity

src/main/java/world/bentobox/greenhouses/listeners/GreenhouseEvents.java

@@ -12,6 +12,8 @@
 import org.bukkit.World;
 import org.bukkit.block.Biome;
 import org.bukkit.block.Block;
+import org.bukkit.block.data.BlockData;
+import org.bukkit.block.data.Waterlogged;
 import org.bukkit.event.EventHandler;
 import org.bukkit.event.EventPriority;
 import org.bukkit.event.Listener;
@@ -50,11 +52,17 @@ public void onPlayerInteractInNether(PlayerBucketEmptyEvent e) {
         if (!e.getBucket().equals(Material.WATER_BUCKET)) {
             return;
         }
+        BlockData blockData = e.getBlockClicked().getBlockData();
         Block b = e.getBlockClicked().getRelative(e.getBlockFace());
         if (e.getPlayer().getWorld().getEnvironment().equals(World.Environment.NETHER)
                 && !addon.getManager().getMap().getGreenhouse(b.getLocation())
                 .map(gh -> gh.getBiomeRecipe().getBiome()).map(NETHER_BIOMES::contains).orElse(true)) {
             // In Nether not a nether greenhouse
+            if (blockData instanceof Waterlogged w) {
+                w.setWaterlogged(true);
+                e.getBlockClicked().setBlockData(w);
+                return;
+            }
             b.setType(Material.WATER);
         } else if (!e.getPlayer().getWorld().getEnvironment().equals(World.Environment.NETHER)
                 && addon.getManager().getMap().getGreenhouse(b.getLocation())

src/main/java/world/bentobox/greenhouses/managers/EcoSystemManager.java

@@ -241,16 +241,14 @@ protected List<GrowthBlock> getAvailableBlocks(Greenhouse gh, boolean ignoreLiqu
             for (double z = ibb.getMinZ(); z < ibb.getMaxZ(); z++) {
                 for (double y = ibb.getMaxY() - 1; y >= bb.getMinY(); y--) {
                     Block b = gh.getWorld().getBlockAt(NumberConversions.floor(x), NumberConversions.floor(y), NumberConversions.floor(z));
-                    if (checkBlock(result, b, ignoreLiquid)) {
-                        break;
-                    }
+                    checkBlock(result, b, ignoreLiquid);
                 }
             }
         }
         return result;
     }
 
-    private boolean checkBlock(List<GrowthBlock> result, Block b, boolean ignoreLiquid) {
+    private void checkBlock(List<GrowthBlock> result, Block b, boolean ignoreLiquid) {
         // Check floor blocks
         if (!ignoreLiquid) {
             // Check ceiling blocks
@@ -264,15 +262,12 @@ private boolean checkBlock(List<GrowthBlock> result, Block b, boolean ignoreLiqu
                             )
                     ) {
                 result.add(new GrowthBlock(b.getRelative(BlockFace.UP), true));
-                return true;
             }
         } else {
             if (!b.isEmpty() && !b.isLiquid() && b.getRelative(BlockFace.UP).isLiquid()) {
                 result.add(new GrowthBlock(b.getRelative(BlockFace.UP), true));
-                return true;
             }
         }
-        return false;
     }
 
     private int getBoneMeal(Greenhouse gh) {

src/main/java/world/bentobox/greenhouses/ui/panel/Panel.java

@@ -36,7 +36,7 @@ public void showPanel(User user) {
     private List<String> getDescription(User user, BiomeRecipe br) {
         List<String> d = new ArrayList<>();
         // Make description
-        d.add(user.getTranslation("greenhouses.recipe.title", "[biome]", Util.prettifyText(br.getBiome().toString())));
+        d.add(user.getTranslation("greenhouses.recipe.title", "[biome]", Util.prettifyText(br.getBiome().getKey().getKey())));
         if (!br.getRecipeBlocks().isEmpty()) {
             d.add(user.getTranslation("greenhouses.recipe.minimumblockstitle"));
             br.getRecipeBlocks().forEach(b -> d.add(user.getTranslation("greenhouses.recipe.blockscolor") + b));

src/main/resources/biomes.yml

@@ -339,9 +339,9 @@ biomes:
       HORSE: 18:GRASS_BLOCK
       DONKEY: 2:GRASS_BLOCK
       COW: 20:GRASS_BLOCK
-      CHICKEN: 25:GRASS_BLOCK
-      PIG: 25:GRASS_BLOCK
-      SHEEP: 25:GRASS_BLOCK
+      CHICKEN: 20:GRASS_BLOCK
+      PIG: 20:GRASS_BLOCK
+      SHEEP: 20:GRASS_BLOCK
     moblimit: 1
     # Maxmobs - this is the maximum number of greenhouse-spawed mobs allowed in 
     # the greenhouse at once. Spawning will stop when this limit is reached. 

src/main/resources/locales/cs.yml

@@ -22,7 +22,7 @@ greenhouses:
     no-rank: "&cNa toto nemáš hodnost."
     notyours: "Toto není tvůj skleník!"
     not-inside: "&cNejsi ve skleníku!"
-    tooexpensive: "Nemůžeš si dovolit [price]" 
+    tooexpensive: "Nemůžeš si dovolit [price]"
     alreadyexists: "Skleník již existuje!"
     norecipe: "Nelze vytvořit skleník!"
 
@@ -38,154 +38,144 @@ greenhouses:
     icemustbe: "Ledové bloky > [coverage]% povrchu podlahy."
     lavamustbe: "Láva > [coverage]% povrchu podlahy."
     minimumblockstitle: "[Minimum bloků vyžadováno]"
-    nootherblocks: "Žádné další bloky nejsou vyžadovány."
+    nootherblocks: "Žádné další bloky již nejsou vyžadovány."
     missing: "Skleník chybí"
-
+   
   commands:
     user:
+      description: "Otevře GUI pro výběr skleníku"
       remove:
-        description: "Odstraní skleník, ve kterém stojíš, jsi-li vlastníkem"
+        description: "Odstraní skleník, ve kterém stojíš, pokud jsi jeho vlastník"        
       make:
-        description: "Pokusit se o výrobu skelníku"
-        parameters: "<recipe>"
+        description: "Pokusit se vytvořit skleník"
+        parameters: "<recept>"
         error:
-          already: "&cTady už je skleník!"
+          already: "&cZde již skleník existuje!"
           FAIL_BAD_ROOF_BLOCKS: "&cStřecha obsahuje nepovolené bloky!"
-          FAIL_BAD_WALL_BLOCKS: "&cStěny obsahují nepovolené bloky!"
-          FAIL_BELOW: "&cMusíš být uvnitř skleníku, aby ses pokusis jej vytvořit"
-          FAIL_BLOCKS_ABOVE: "&cNad skleníkem nesmí být žádné bloky! Bloky červeného skla by měly ukázat problémové bloky."
-          FAIL_HOLE_IN_ROOF: "&cVe střeše je díra nebo není rovná! Bloky červeného skla by měly ukázat problém."
-          FAIL_HOLE_IN_WALL: "&cVe stěně je díra!"
-          FAIL_NO_ROOF: "&cVypadá to, že tu není žádná střecha!"
-          FAIL_TOO_MANY_DOORS: "&cNemůžeš mít více, než 4 dveře ve skleníku!"
-          FAIL_TOO_MANY_HOPPERS: "&cVe stěnách nebo střeše je povolena jen jedna násypka."
-          FAIL_UNEVEN_WALLS: "&cStěny jsou nepravidelné. Bloky červeného skla by měly ukázat problémové bloky."
+          FAIL_BAD_WALL_BLOCKS: "&cStěna obsahuje nepovolené bloky!"
+          FAIL_BELOW: "&cMusíš být uvnitř skleníku, abys ho mohl zkusit vytvořit"
+          FAIL_BLOCKS_ABOVE: "&cNad skleníkem nesmí být žádné bloky! Červené skleněné bloky by měly ukázat problematické bloky."
+          FAIL_HOLE_IN_ROOF: |
+            &cVe střeše je díra nebo není rovná!
+            &cČervené skleněné bloky by měly ukázat problém.
+            &cUjisti se, že jsi uvnitř skleníku, abys ho mohl vytvořit.
+          FAIL_HOLE_IN_WALL: "&cVestěně je díra!"
+          FAIL_NO_ROOF: "&cZdá se, že zde není střecha! Ujisti se, že jsi uvnitř skleníku, abys ho mohl vytvořit."
+          FAIL_TOO_MANY_DOORS: "&cVe skleníku nemůžeš mít více než 4 dveře!"
+          FAIL_TOO_MANY_HOPPERS: "&cVestěnách nebo střeše je povolen pouze jeden násypník (hopper)."
+          FAIL_UNEVEN_WALLS: "&cStěny jsou nerovnoměrné. Červené skleněné bloky by měly ukázat problematické bloky."
           FAIL_INSUFFICIENT_ICE: "&cNedostatek ledu k vytvoření tohoto receptu"
           FAIL_INSUFFICIENT_LAVA: "&cNedostatek lávy k vytvoření tohoto receptu"
           FAIL_INSUFFICIENT_WATER: "&cNedostatek vody k vytvoření tohoto receptu"
-          FAIL_NO_ICE: "&cLed je potřeba k vytvoření tohoto receptu"
-          FAIL_NO_LAVA: "&cLáva je potřeba k vytvoření tohoto receptu"
-          FAIL_NO_WATER: "&cVoda je potřeba k vytvoření tohoto receptu"
-          FAIL_INSUFFICIENT_BLOCKS: "&cVíce bloků je potřeba k vytvoření tohoto receptu!"
-          FAIL_OVERLAPPING: "&cPardon, skleníky nesmí sdílet stěny."
-        success: "&2Úspěšně jsi vytvořil skleník biomu [biome]! Biom se synchronizuje s dalším teleportem nebo přihlášením."
+          FAIL_NO_ICE: "&cK vytvoření tohoto receptu je vyžadován led"
+          FAIL_NO_LAVA: "&cK vytvoření tohoto receptu je vyžadována láva"
+          FAIL_NO_WATER: "&cK vytvoření tohoto receptu je vyžadována voda"
+          FAIL_NO_RECIPE_FOUND: "&cNebyl nalezen žádný recept, který by odpovídal tomuto skleníku"
+          FAIL_INSUFFICIENT_BLOCKS: "&cK vytvoření tohoto receptu je vyžadováno více bloků!"
+          FAIL_OVERLAPPING: "&cSkleníky nemohou sdílet stěny, omlouváme se."
+        success: "&2Úspěšně jsi vytvořil skleník s biomem [biome]! Biom se synchronizuje při příštím teleportu nebo přihlášení."
         missing-blocks: "&cChybí [material] x [number]"
         unknown-recipe: "&cNeznámý recept"
-        try-these: "&cZkus něco z tohoto:"
+        try-these: "&cZkus jeden z těchto:"
         recipe-format: "&3[name]"
       info:
-        title: "&A[Jak postavit skleník]"
+        title: "&a [Jak postavit skleník]"
         instructions: |
-          &EUdělej kostku ze skla se 4 stěnami a rovnou skleněnou
-          &Estřechou a přihoď k tomu &F4 dveře &Eve stěnách.
-          &EPlácni &F1 násypku &Edo stěny nebo střechy a přidej kbelíky vody.
-          &Epro sníh a/nebo kostní moučku pro automatický růst rostlin.
-          &EPodívej se do receptů biomu pro typy bloků, které musí být
-          &Euvnitř skleníku pro úspěšné vytvoření. 
+          &e Vyrob krabici ze skla se 4 stěnami a plochou skleněnou
+          &e střechou a přidej do stěn až &f 4 dveře.
+          &e Umísti &f 1 násypník (hopper) &e do stěny nebo střechy a přidej kbelíky s vodou,
+          &e abys automaticky vyráběl sníh a/nebo kostní moučku pro růst rostlin.
+          &e Podívej se na recepty biomů, jaké bloky musí být uvnitř skleníku,
+          &e aby byl úspěšně vytvořen.
 
-
-######### Old locale for reference
 help:
-  help: "nápověda"
-  make: "Pokusí se vytvořit skleník"
-  remove: "Odstraní skleník, ve kterém stojíš, pokud jsi vlastníkem"
-  info: "Jak vytvořit skleník"
-  list: "Vypíše seznam všech biomů, které lze použít ve skleníku"
-  recipe: "Napoví, jak vytvořit skleník biomu"
-  opengui: "Otevře GUI Skleníků"
+   help: "nápověda"
+   make: "Pokusí se vytvořit skleník"
+   remove: "Odstraní skleník, ve kterém stojíš, pokud jsi vlastník"
+   info: "Jak vyrobit skleník"
+   list: "Seznam všech biomů skleníků, které lze vytvořit"
+   recipe: "Řekne ti, jak vyrobit daný biom skleníku"
+   opengui: "Otevře GUI skleníku"
 
 list:
-  title: "[Recepty biomů skleníku]"
-  info: "Použij /greenhouse recipe <number> k zobrazení detailů, jak vytvořit jednotlivé skleníky"
-
+   title: "[Recepty biomů skleníků]"
+   info: "Použij /greenhouse recipe <číslo> k zobrazení podrobností o každém skleníku"
 
-################
-#General Errors#
-################
 error:
-  greenhouseProtected: "Skleník je chráněn"
-  move: "Nejdříve se přesuň do skleníku, který vlastníš."
-  notowner: "Na toto musíš být vlastník tohoto skleníku."
-  removing: "Odstraňuji skleník!"
-  notyours: "Toto není tvůj skleník!"
-  notinside: "Nejsi ve skleníku!"
-  tooexpensive: "Nemůžeš si dovolit [price]" 
-  alreadyexists: "Skleník již existuje!"
-  norecipe: "Nelze vytvořit skleník!"
+   greenhouseProtected: "Skleník je chráněn"
+   move: "Nejdříve se přesuň do skleníku, který vlastníš."
+   notowner: "K provedení této akce musíš být vlastníkem tohoto skleníku."
+   removing: "Odstraňování skleníku!"
+   notyours: "Toto není tvůj skleník!"
+   notinside: "Nejsi ve skleníku!"
+   tooexpensive: "Nemůžeš si dovolit [price]"
+   alreadyexists: "Skleník již existuje!"
+   norecipe: "Nelze vytvořit skleník!"
 
 messages:
-  enter: "Vcházíš do [biome] skleníku [owner]!"
-  leave: "Odcházíš ze skleníku [owner]."
-  youarein: "Právě jsi v [biome] skleníku [owner]!"
-  removed: "Tento skleník již neeexistuje..."
-  removedmessage: "Tvůj [biome] skleník již neeistuje!"
-  ecolost: "Tvůj skleník na [location] ztratil svůj ekosystém a byl odstraněn."
+   enter: "Vstupuješ do skleníku [owner], biom [biome]!"
+   leave: "Opouštíš skleník [owner]."
+   youarein: "Nyní jsi ve skleníku [owner], biom [biome]!"
+   removed: "Tento skleník již neexistuje..."
+   removedmessage: "Tvůj skleník [biome] již neexistuje!"
+   ecolost: "Tvůj skleník na [location] ztratil svůj ekosystém a byl odstraněn."
 
 info:
-  title: "&A[Jak postavit skleník]"
-  instructions: |
-    &EUdělej kostku ze skla se 4 stěnami a rovnou skleněnou
-    &Estřechou a přihoď k tomu &F4 dveře &Eve stěnách.
-    &EPlácni &F1 násypku &Edo stěny nebo střechy a přidej kbelíky vody.
-    &Epro sníh a/nebo kostní moučku pro automatický růst rostlin.
-    &EPodívej se do receptů biomu pro typy bloků, které musí být
-    &Euvnitř skleníku pro úspěšné vytvoření. 
-  info: "[Info o skleníku]"
-  none: "Žádné"
-  nomore: "&4Nemůžeš postavit další skleníky!"
-  onemore: "&6Můžeš postavit ještě jeden skleník."
-  youcanbuild: "&AMůžeš postavit ještě [number] dalších skleníků!"
-  unlimited: "&AMůžeš postavit bezpočet dalších skleníků!"
-  welcome: "&BVítej! Klikni sem pro instrukce"
+   title: "&A[Jak postavit skleník]"
+   instructions: |
+     &E Vyrob krabici ze skla se 4 stěnami a plochou skleněnou
+     &E střechou a přidej do stěn až &F 4 dveře.
+     &E Umísti &F 1 násypník &E do stěny nebo střechy a přidej kbelíky s vodou,
+     &E abys automaticky vyráběl sníh a/nebo kostní moučku pro růst rostlin.
+     &E Podívej se na recepty biomů, jaké bloky musí být uvnitř skleníku,
+     &E aby byl úspěšně vytvořen.
+   info: "[Informace o skleníku]"
+   none: "Žádné"
+   nomore: "&4 Nemůžeš postavit žádné další skleníky!"
+   onemore: "&6 Můžeš postavit ještě jeden skleník."
+   youcanbuild: "&A Můžeš postavit až [number] dalších skleníků!"
+   unlimited: "&A Můžeš postavit neomezený počet skleníků!"
+   welcome: "&B Vítej! Klikni zde pro instrukce"
 
 recipe:
-  blockscolor: "&f"
-  hint: "Použij /greenhouse list k zobrazení seznamu čísel receptů!"
-  wrongnumber: "Číslo receptu musí být mezi 1 a [size]"
-  title: "[Recept [biome]]"
-  nowater: "Voda není povolena."
-  noice: "Led není povolen."
-  nolava: "Láva není povolena."
-  watermustbe: "Voda > [coverage]% povrchu podlahy."
-  icemustbe: "Ledové bloky > [coverage]% povrchu podlahy."
-  lavamustbe: "Láva > [coverage]% povrchu podlahy."
-  minimumblockstitle: "[Minimum bloků vyžadováno]"
-  nootherblocks: "Žádné další bloky již nejsou vyžadovány."
-  missing: "Skleník chybí"
+   blockscolor: "&f"
+   hint: "Použij /greenhouse list k zobrazení seznamu čísel receptů!"
+   wrongnumber: "Číslo receptu musí být mezi 1 a [size]"
+   title: "[Recept [biome]]"
+   nowater: "Voda není povolena."
+   noice: "Led není povolen."
+   nolava: "Láva není povolena."
+   watermustbe: "Voda > [coverage]% povrchu podlahy."
+   icemustbe: "Ledové bloky > [coverage]% povrchu podlahy."
+   lavamustbe: "Láva > [coverage]% povrchu podlahy."
+   minimumblockstitle: "[Minimum bloků vyžadováno]"
+   nootherblocks: "Žádné další bloky již nejsou vyžadovány."
+   missing: "Skleník chybí"
 
 event:
    broke: "Zničil jsi tento skleník! Vracím biom zpět na [biome]!"
    fix: "Oprav skleník a pak ho znovu vytvoř."
    cannotplace: "Nemůžeš položit bloky nad skleník!"
    pistonerror: "Píst nemůže posunout bloky nad skleník!"
 
-
 limits:
    noneallowed: "Nemáš oprávnění vytvářet žádné skleníky, takže jich bylo [number] smazáno."
    limitedto: "Máš oprávnění vytvořit jen [limit] skleníků, takže jich bylo [number] smazáno."
-   
 
-##################################
-#Admin commands that use /gadmin #
-##################################
-
-#Help
 adminHelp:
    reload: "obnovit konfiguraci ze souboru."
-   info: "poskytuje info o skleníku, v němž se nacházíš"
+   info: "poskytuje informace o skleníku, ve kterém se nacházíš"
 
-#reload
 reload:
-   configReloaded: "Obnovena konfigurace ze souboru."
+   configReloaded: "Konfigurace byla znovu načtena ze souboru."
 
 admininfo:
-   error: "Info o skleníku je přístupné jen ve hře"
-   error2: "Postav se do skleníku, abys viděl info."
-   flags: "[Vlaječky skleníku]"
+   error: "Informace o skleníku jsou dostupné pouze ve hře"
+   error2: "Vstup do skleníku, abys viděl informace."
+   flags: "[Vlajky skleníku]"
 
 news:
-   headline: "[Novinky skleníku]"
+   headline: "[Novinky skleníků]"
 
 controlpanel:
-   title: "&ASkleníky"
-
+   title: "&ASkleníky"