Merge branch 'develop' into master

# Conflicts:
#	pom.xml

.github/workflows/build.yml

@@ -14,10 +14,10 @@ jobs:
       - uses: actions/checkout@v2
         with:
           fetch-depth: 0  # Shallow clones should be disabled for a better relevancy of analysis
-      - name: Set up JDK 16
+      - name: Set up JDK 17
         uses: actions/setup-java@v1
         with:
-          java-version: 16
+          java-version: 17
       - name: Cache SonarCloud packages
         uses: actions/cache@v1
         with:

pom.xml

@@ -45,12 +45,12 @@
         <project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
         <java.version>17</java.version>
         <!-- More visible way how to change dependency versions -->
-        <spigot.version>1.19.2-R0.1-SNAPSHOT</spigot.version>
+        <spigot.version>1.19.4-R0.1-SNAPSHOT</spigot.version>
-        <bentobox.version>1.19.0</bentobox.version>
+        <bentobox.version>1.23.0</bentobox.version>
         <!-- Revision variable removes warning about dynamic version -->
         <revision>${build.version}-SNAPSHOT</revision>
         <!-- This allows to change between versions and snapshots. -->
-        <build.version>1.17.0</build.version>
+        <build.version>1.18.0</build.version>
         <build.number>-LOCAL</build.number>
         <sonar.organization>bentobox-world</sonar.organization>
         <sonar.host.url>https://sonarcloud.io</sonar.host.url>
@@ -114,12 +114,6 @@
             <version>${spigot.version}</version>
             <scope>provided</scope>
         </dependency>
-        <dependency>
-            <groupId>org.spigotmc</groupId>
-            <artifactId>plugin-annotations</artifactId>
-            <version>1.2.3-SNAPSHOT</version>
-            <scope>provided</scope>
-        </dependency>
         <dependency>
             <groupId>world.bentobox</groupId>
             <artifactId>bentobox</artifactId>
@@ -192,15 +186,17 @@
              <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-javadoc-plugin</artifactId>
-                <version>3.0.1</version>
+                <version>3.3.0</version>
                 <configuration>
-                    <show>public</show>
                     <failOnError>false</failOnError>
                     <additionalJOption>-Xdoclint:none</additionalJOption>
+                    <!-- To compile with Java 11, this tag may be required -->
+                    <javadocExecutable>${java.home}/bin/javadoc</javadocExecutable>
                 </configuration>
                 <executions>
                     <execution>
                         <id>attach-javadocs</id>
+                        <phase>install</phase>
                         <goals>
                             <goal>jar</goal>
                         </goals>

src/main/java/world/bentobox/caveblock/CaveBlock.java

@@ -282,6 +282,16 @@ public class CaveBlock extends GameModeAddon
     }
 
 
+    /**
+     * This addon uses the new chunk generation API for the sea bottom
+     */
+    @Override
+    public boolean isUsesNewChunkGeneration()
+    {
+        return true;
+    }
+
+
     // ---------------------------------------------------------------------
     // Section: Variables
     // ---------------------------------------------------------------------

src/main/java/world/bentobox/caveblock/CaveBlockPladdon.java

@@ -1,16 +1,10 @@
 package world.bentobox.caveblock;
 
-import org.bukkit.plugin.java.annotation.dependency.Dependency;
-import org.bukkit.plugin.java.annotation.plugin.ApiVersion;
-import org.bukkit.plugin.java.annotation.plugin.Plugin;
 
 import world.bentobox.bentobox.api.addons.Addon;
 import world.bentobox.bentobox.api.addons.Pladdon;
 
 
-@Plugin(name = "Pladdon", version = "1.0")
-@ApiVersion(ApiVersion.Target.v1_18)
-@Dependency(value = "BentoBox")
 public class CaveBlockPladdon extends Pladdon {
 
     @Override

src/main/java/world/bentobox/caveblock/Settings.java

@@ -1,12 +1,7 @@
 package world.bentobox.caveblock;
 
 
-import java.util.ArrayList;
+import java.util.*;
-import java.util.HashMap;
-import java.util.HashSet;
-import java.util.List;
-import java.util.Map;
-import java.util.Set;
 
 import org.bukkit.Difficulty;
 import org.bukkit.GameMode;
@@ -333,7 +328,7 @@ public class Settings implements WorldSettings
      * @return the defaultIslandFlags object.
      */
     @Override
-    public Map<Flag, Integer> getDefaultIslandFlags()
+    public Map<String, Integer> getDefaultIslandFlagNames()
     {
         return defaultIslandFlags;
     }
@@ -344,7 +339,7 @@ public class Settings implements WorldSettings
      * @return the defaultIslandSettings object.
      */
     @Override
-    public Map<Flag, Integer> getDefaultIslandSettings()
+    public Map<String, Integer> getDefaultIslandSettingNames()
     {
         return defaultIslandSettings;
     }
@@ -1179,7 +1174,7 @@ public class Settings implements WorldSettings
      * @param defaultIslandFlags the defaultIslandFlags object new value.
      *
      */
-    public void setDefaultIslandFlags(Map<Flag, Integer> defaultIslandFlags)
+    public void setDefaultIslandFlags(Map<String, Integer> defaultIslandFlags)
     {
         this.defaultIslandFlags = defaultIslandFlags;
     }
@@ -1190,7 +1185,7 @@ public class Settings implements WorldSettings
      * @param defaultIslandSettings the defaultIslandSettings object new value.
      *
      */
-    public void setDefaultIslandSettings(Map<Flag, Integer> defaultIslandSettings)
+    public void setDefaultIslandSettings(Map<String, Integer> defaultIslandSettings)
     {
         this.defaultIslandSettings = defaultIslandSettings;
     }
@@ -2143,6 +2138,20 @@ public class Settings implements WorldSettings
     }
 
 
+    @Override
+    public Map<Flag, Integer> getDefaultIslandFlags()
+    {
+        return Collections.emptyMap();
+    }
+
+
+    @Override
+    public Map<Flag, Integer> getDefaultIslandSettings()
+    {
+        return Collections.emptyMap();
+    }
+
+
     // ---------------------------------------------------------------------
     // Section: Variables
     // ---------------------------------------------------------------------
@@ -2439,13 +2448,11 @@ public class Settings implements WorldSettings
     @ConfigComment("  SUB-OWNER = 900")
     @ConfigComment("  OWNER     = 1000")
     @ConfigEntry(path = "world.default-cave-flags")
-    @Adapter(FlagSerializer.class)
+    private Map<String, Integer> defaultIslandFlags = new HashMap<>();
-    private Map<Flag, Integer> defaultIslandFlags = new HashMap<>();
 
     @ConfigComment("These are the default settings for new caves")
     @ConfigEntry(path = "world.default-cave-settings")
-    @Adapter(FlagSerializer2.class)
+    private Map<String, Integer> defaultIslandSettings = new HashMap<>();
-    private Map<Flag, Integer> defaultIslandSettings = new HashMap<>();
 
     @ConfigComment("These settings/flags are hidden from users")
     @ConfigComment("Ops can toggle hiding in-game using SHIFT-LEFT-CLICK on flags in settings")

src/main/resources/addon.yml

@@ -1,7 +1,7 @@
 name: CaveBlock
 main: world.bentobox.caveblock.CaveBlock
 version: ${version}${build.number}
-api-version: 1.19.0
+api-version: 1.23.0
 metrics: true
 repository: "BentoBoxWorld/CaveBlock"
 icon: "STONE_PICKAXE"

src/main/resources/plugin.yml

@@ -0,0 +1,9 @@
+name: BentoBox-CaveBlock
+main: world.bentobox.caveblock.CaveBlockPladdon
+version: ${project.version}${build.number}
+api-version: "1.19"
+
+authors: [tastybento, BONNe]
+contributors: ["The BentoBoxWorld Community"]
+website: https://bentobox.world
+description: ${project.description}